Ignore:
Timestamp:
2016-03-17T01:50:12+01:00 (4 years ago)
Author:
Don-vip
Message:

sonar - Local variable and method parameter names should comply with a naming convention

Location:
trunk/src/org/openstreetmap/josm/io
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/io/NmeaReader.java

    r9740 r10001  
    173173        try (BufferedReader rd = new BufferedReader(new InputStreamReader(source, StandardCharsets.UTF_8))) {
    174174            StringBuilder sb = new StringBuilder(1024);
    175             int loopstart_char = rd.read();
     175            int loopstartChar = rd.read();
    176176            ps = new NMEAParserState();
    177             if (loopstart_char == -1)
     177            if (loopstartChar == -1)
    178178                //TODO tell user about the problem?
    179179                return;
    180             sb.append((char) loopstart_char);
     180            sb.append((char) loopstartChar);
    181181            ps.pDate = "010100"; // TODO date problem
    182182            while (true) {
  • trunk/src/org/openstreetmap/josm/io/OsmChangesetParser.java

    r8855 r10001  
    119119
    120120            // -- min_lon and min_lat
    121             String min_lon = atts.getValue("min_lon");
    122             String min_lat = atts.getValue("min_lat");
    123             String max_lon = atts.getValue("max_lon");
    124             String max_lat = atts.getValue("max_lat");
    125             if (min_lon != null && min_lat != null && max_lon != null && max_lat != null) {
     121            String minLonStr = atts.getValue("min_lon");
     122            String minLatStr = atts.getValue("min_lat");
     123            String maxLonStr = atts.getValue("max_lon");
     124            String maxLatStr = atts.getValue("max_lat");
     125            if (minLonStr != null && minLatStr != null && maxLonStr != null && maxLatStr != null) {
    126126                double minLon = 0;
    127127                try {
    128                     minLon = Double.parseDouble(min_lon);
    129                 } catch (NumberFormatException e) {
    130                     throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lon", min_lon));
     128                    minLon = Double.parseDouble(minLonStr);
     129                } catch (NumberFormatException e) {
     130                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lon", minLonStr));
    131131                }
    132132                double minLat = 0;
    133133                try {
    134                     minLat = Double.parseDouble(min_lat);
    135                 } catch (NumberFormatException e) {
    136                     throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lat", min_lat));
     134                    minLat = Double.parseDouble(minLatStr);
     135                } catch (NumberFormatException e) {
     136                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "min_lat", minLatStr));
    137137                }
    138138                current.setMin(new LatLon(minLat, minLon));
     
    142142                double maxLon = 0;
    143143                try {
    144                     maxLon = Double.parseDouble(max_lon);
    145                 } catch (NumberFormatException e) {
    146                     throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lon", max_lon));
     144                    maxLon = Double.parseDouble(maxLonStr);
     145                } catch (NumberFormatException e) {
     146                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lon", maxLonStr));
    147147                }
    148148                double maxLat = 0;
    149149                try {
    150                     maxLat = Double.parseDouble(max_lat);
    151                 } catch (NumberFormatException e) {
    152                     throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lat", max_lat));
     150                    maxLat = Double.parseDouble(maxLatStr);
     151                } catch (NumberFormatException e) {
     152                    throwException(tr("Illegal value for attribute ''{0}''. Got ''{1}''.", "max_lat", maxLatStr));
    153153                }
    154154                current.setMax(new LatLon(maxLon, maxLat));
  • trunk/src/org/openstreetmap/josm/io/OsmServerWriter.java

    r9528 r10001  
    6161    private long uploadStartTime;
    6262
    63     public String timeLeft(int progress, int list_size) {
     63    public String timeLeft(int progress, int listSize) {
    6464        long now = System.currentTimeMillis();
    6565        long elapsed = now - uploadStartTime;
     
    6767            elapsed = 1;
    6868        }
    69         double uploads_per_ms = (double) progress / elapsed;
    70         double uploads_left = list_size - progress;
    71         long ms_left = (long) (uploads_left / uploads_per_ms);
    72         long minutes_left = ms_left / MSECS_PER_MINUTE;
    73         long seconds_left = (ms_left / MSECS_PER_SECOND) % SECONDS_PER_MINUTE;
    74         StringBuilder time_left_str = new StringBuilder().append(minutes_left).append(':');
    75         if (seconds_left < 10) {
    76             time_left_str.append('0');
    77         }
    78         return time_left_str.append(seconds_left).toString();
     69        double uploadsPerMs = (double) progress / elapsed;
     70        double uploadsLeft = listSize - progress;
     71        long msLeft = (long) (uploadsLeft / uploadsPerMs);
     72        long minutesLeft = msLeft / MSECS_PER_MINUTE;
     73        long secondsLeft = (msLeft / MSECS_PER_SECOND) % SECONDS_PER_MINUTE;
     74        StringBuilder timeLeftStr = new StringBuilder().append(minutesLeft).append(':');
     75        if (secondsLeft < 10) {
     76            timeLeftStr.append('0');
     77        }
     78        return timeLeftStr.append(secondsLeft).toString();
    7979    }
    8080
     
    9494            for (OsmPrimitive osm : primitives) {
    9595                int progress = progressMonitor.getTicks();
    96                 String time_left_str = timeLeft(progress, primitives.size());
     96                String timeLeftStr = timeLeft(progress, primitives.size());
    9797                String msg = "";
    9898                switch(OsmPrimitiveType.from(osm)) {
     
    106106                                progress,
    107107                                primitives.size(),
    108                                 time_left_str,
     108                                timeLeftStr,
    109109                                osm.getName() == null ? osm.getId() : osm.getName(),
    110110                                        osm.getId()));
  • trunk/src/org/openstreetmap/josm/io/remotecontrol/RequestProcessor.java

    r10000 r10001  
    177177
    178178            Map<String, String> headers = new HashMap<>();
    179             int k = 0, MAX_HEADERS = 20;
    180             while (k < MAX_HEADERS) {
     179            int k = 0;
     180            int maxHeaders = 20;
     181            while (k < maxHeaders) {
    181182                get = in.readLine();
    182183                if (get == null) break;
Note: See TracChangeset for help on using the changeset viewer.