Changeset 14210 in josm


Ignore:
Timestamp:
2018-08-31T23:25:33+02:00 (11 months ago)
Author:
Don-vip
Message:

see #16681 - sonarqube - squid:HiddenFieldCheck

Location:
trunk/src/org/openstreetmap/josm/data/gpx
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/gpx/GpxImageEntry.java

    r14209 r14210  
    66import java.util.Date;
    77import java.util.Objects;
     8import java.util.function.Consumer;
    89
    910import org.openstreetmap.josm.data.coor.CachedLatLon;
     
    501502        try {
    502503            if (dirExif != null) {
    503                 int orientation = dirExif.getInt(ExifIFD0Directory.TAG_ORIENTATION);
    504                 setExifOrientation(orientation);
     504                setExifOrientation(dirExif.getInt(ExifIFD0Directory.TAG_ORIENTATION));
    505505            }
    506506        } catch (MetadataException ex) {
     
    511511            if (dir != null) {
    512512                // there are cases where these do not match width and height stored in dirExif
    513                 int width = dir.getInt(JpegDirectory.TAG_IMAGE_WIDTH);
    514                 int height = dir.getInt(JpegDirectory.TAG_IMAGE_HEIGHT);
    515                 setWidth(width);
    516                 setHeight(height);
     513                setWidth(dir.getInt(JpegDirectory.TAG_IMAGE_WIDTH));
     514                setHeight(dir.getInt(JpegDirectory.TAG_IMAGE_HEIGHT));
    517515            }
    518516        } catch (MetadataException ex) {
     
    526524        }
    527525
    528         final Double speed = ExifReader.readSpeed(dirGps);
    529         if (speed != null) {
    530             setSpeed(speed);
    531         }
    532 
    533         final Double ele = ExifReader.readElevation(dirGps);
    534         if (ele != null) {
    535             setElevation(ele);
    536         }
    537 
    538         try {
    539             final LatLon latlon = ExifReader.readLatLon(dirGps);
    540             setExifCoor(latlon);
     526        ifNotNull(ExifReader.readSpeed(dirGps), this::setSpeed);
     527        ifNotNull(ExifReader.readElevation(dirGps), this::setElevation);
     528
     529        try {
     530            setExifCoor(ExifReader.readLatLon(dirGps));
    541531            setPos(getExifCoor());
    542532        } catch (MetadataException | IndexOutOfBoundsException ex) { // (other exceptions, e.g. #5271)
     
    547537
    548538        try {
    549             final Double direction = ExifReader.readDirection(dirGps);
    550             if (direction != null) {
    551                 setExifImgDir(direction);
    552             }
     539            ifNotNull(ExifReader.readDirection(dirGps), this::setExifImgDir);
    553540        } catch (IndexOutOfBoundsException ex) { // (other exceptions, e.g. #5271)
    554541            Logging.debug(ex);
    555542        }
    556543
    557         final Date gpsDate = dirGps.getGpsDate();
    558         if (gpsDate != null) {
    559             setExifGpsTime(gpsDate);
     544        ifNotNull(dirGps.getGpsDate(), this::setExifGpsTime);
     545    }
     546
     547    private static <T> void ifNotNull(T value, Consumer<T> setter) {
     548        if (value != null) {
     549            setter.accept(value);
    560550        }
    561551    }
  • trunk/src/org/openstreetmap/josm/data/gpx/GpxTimezone.java

    r14205 r14210  
    4545        StringBuilder ret = new StringBuilder();
    4646
    47         double timezone = this.timezone;
    48         if (timezone < 0) {
     47        double tz = this.timezone;
     48        if (tz < 0) {
    4949            ret.append('-');
    50             timezone = -timezone;
     50            tz = -tz;
    5151        } else {
    5252            ret.append('+');
    5353        }
    54         ret.append((long) timezone).append(':');
    55         int minutes = (int) ((timezone % 1) * 60);
     54        ret.append((long) tz).append(':');
     55        int minutes = (int) ((tz % 1) * 60);
    5656        if (minutes < 10) {
    5757            ret.append('0');
Note: See TracChangeset for help on using the changeset viewer.