Changeset 11914 in josm for trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

Timestamp:

2017-04-15T02:00:43+02:00 (7 years ago)

Author:

Don-vip

Message:

sonar - squid:S2972 - Inner classes should not have too many lines of code

File:

• trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java (modified) (2 diffs)

trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

@@ -35 +35 @@
 import java.util.Hashtable;
 import java.util.List;
-import java.util.Locale;
-import java.util.Objects;
 import java.util.Optional;
 import java.util.TimeZone;
@@ -1251 +1249 @@
         return endIndex;
     }
-
-    static final class Timezone {
-
-        static final Timezone ZERO = new Timezone(0.0);
-        private final double timezone;
-
-        Timezone(double hours) {
-            this.timezone = hours;
-        }
-
-        public double getHours() {
-            return timezone;
-        }
-
-        String formatTimezone() {
-            StringBuilder ret = new StringBuilder();
-
-            double timezone = this.timezone;
-            if (timezone < 0) {
-                ret.append('-');
-                timezone = -timezone;
-            } else {
-                ret.append('+');
-            }
-            ret.append((long) timezone).append(':');
-            int minutes = (int) ((timezone % 1) * 60);
-            if (minutes < 10) {
-                ret.append('0');
-            }
-            ret.append(minutes);
-
-            return ret.toString();
-        }
-
-        static Timezone parseTimezone(String timezone) throws ParseException {
-
-            if (timezone.isEmpty())
-                return ZERO;
-
-            String error = tr("Error while parsing timezone.\nExpected format: {0}", "+H:MM");
-
-            char sgnTimezone = '+';
-            StringBuilder hTimezone = new StringBuilder();
-            StringBuilder mTimezone = new StringBuilder();
-            int state = 1; // 1=start/sign, 2=hours, 3=minutes.
-            for (int i = 0; i < timezone.length(); i++) {
-                char c = timezone.charAt(i);
-                switch (c) {
-                    case ' ':
-                        if (state != 2 || hTimezone.length() != 0)
-                            throw new ParseException(error, i);
-                        break;
-                    case '+':
-                    case '-':
-                        if (state == 1) {
-                            sgnTimezone = c;
-                            state = 2;
-                        } else
-                            throw new ParseException(error, i);
-                        break;
-                    case ':':
-                    case '.':
-                        if (state == 2) {
-                            state = 3;
-                        } else
-                            throw new ParseException(error, i);
-                        break;
-                    case '0':
-                    case '1':
-                    case '2':
-                    case '3':
-                    case '4':
-                    case '5':
-                    case '6':
-                    case '7':
-                    case '8':
-                    case '9':
-                        switch (state) {
-                            case 1:
-                            case 2:
-                                state = 2;
-                                hTimezone.append(c);
-                                break;
-                            case 3:
-                                mTimezone.append(c);
-                                break;
-                            default:
-                                throw new ParseException(error, i);
-                        }
-                        break;
-                    default:
-                        throw new ParseException(error, i);
-                }
-            }
-
-            int h = 0;
-            int m = 0;
-            try {
-                h = Integer.parseInt(hTimezone.toString());
-                if (mTimezone.length() > 0) {
-                    m = Integer.parseInt(mTimezone.toString());
-                }
-            } catch (NumberFormatException nfe) {
-                // Invalid timezone
-                throw (ParseException) new ParseException(error, 0).initCause(nfe);
-            }
-
-            if (h > 12 || m > 59)
-                throw new ParseException(error, 0);
-            else
-                return new Timezone((h + m / 60.0) * (sgnTimezone == '-' ? -1 : 1));
-        }
-
-        @Override
-        public boolean equals(Object o) {
-            if (this == o) return true;
-            if (!(o instanceof Timezone)) return false;
-            Timezone timezone1 = (Timezone) o;
-            return Double.compare(timezone1.timezone, timezone) == 0;
-        }
-
-        @Override
-        public int hashCode() {
-            return Objects.hash(timezone);
-        }
-    }
-
-    static final class Offset {
-
-        static final Offset ZERO = new Offset(0);
-        private final long milliseconds;
-
-        private Offset(long milliseconds) {
-            this.milliseconds = milliseconds;
-        }
-
-        static Offset milliseconds(long milliseconds) {
-            return new Offset(milliseconds);
-        }
-
-        static Offset seconds(long seconds) {
-            return new Offset(1000 * seconds);
-        }
-
-        long getMilliseconds() {
-            return milliseconds;
-        }
-
-        long getSeconds() {
-            return milliseconds / 1000;
-        }
-
-        String formatOffset() {
-            if (milliseconds % 1000 == 0) {
-                return Long.toString(milliseconds / 1000);
-            } else if (milliseconds % 100 == 0) {
-                return String.format(Locale.ENGLISH, "%.1f", milliseconds / 1000.);
-            } else {
-                return String.format(Locale.ENGLISH, "%.3f", milliseconds / 1000.);
-            }
-        }
-
-        static Offset parseOffset(String offset) throws ParseException {
-            String error = tr("Error while parsing offset.\nExpected format: {0}", "number");
-
-            if (!offset.isEmpty()) {
-                try {
-                    if (offset.startsWith("+")) {
-                        offset = offset.substring(1);
-                    }
-                    return Offset.milliseconds(Math.round(Double.parseDouble(offset) * 1000));
-                } catch (NumberFormatException nfe) {
-                    throw (ParseException) new ParseException(error, 0).initCause(nfe);
-                }
-            } else {
-                return Offset.ZERO;
-            }
-        }
-
-        int getDayOffset() {
-            // Find day difference
-            return (int) Math.round(((double) getMilliseconds()) / TimeUnit.DAYS.toMillis(1));
-        }
-
-        Offset withoutDayOffset() {
-            return milliseconds(getMilliseconds() - TimeUnit.DAYS.toMillis(getDayOffset()));
-        }
-
-        Pair<Timezone, Offset> splitOutTimezone() {
-            // In hours
-            final double tz = ((double) withoutDayOffset().getSeconds()) / TimeUnit.HOURS.toSeconds(1);
-
-            // Due to imprecise clocks we might get a "+3:28" timezone, which should obviously be 3:30 with
-            // -2 minutes offset. This determines the real timezone and finds offset.
-            final double timezone = (double) Math.round(tz * 2) / 2; // hours, rounded to one decimal place
-            final long delta = Math.round(getMilliseconds() - timezone * TimeUnit.HOURS.toMillis(1));
-            return Pair.create(new Timezone(timezone), Offset.milliseconds(delta));
-        }
-
-        @Override
-        public boolean equals(Object o) {
-            if (this == o) return true;
-            if (!(o instanceof Offset)) return false;
-            Offset offset = (Offset) o;
-            return milliseconds == offset.milliseconds;
-        }
-
-        @Override
-        public int hashCode() {
-            return Objects.hash(milliseconds);
-        }
-    }
 }