Ignore:
Timestamp:
2015-10-10T01:40:42+02:00 (4 years ago)
Author:
Don-vip
Message:

sonar - fb-contrib - minor performance improvements:

  • Method passes constant String of length 1 to character overridden method
  • Method needlessly boxes a boolean constant
  • Method uses iterator().next() on a List to get the first item
  • Method converts String to boxed primitive using excessive boxing
  • Method converts String to primitive using excessive boxing
  • Method creates array using constants
  • Class defines List based fields but uses them like Sets
Location:
trunk/src/org/openstreetmap/josm/actions
Files:
20 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/actions/AddImageryLayerAction.java

    r8763 r8846  
    136136            ImageryInfo ret = new ImageryInfo(info.getName(), url, "wms", info.getEulaAcceptanceRequired(), info.getCookies());
    137137            if (layersString.length() > 2) {
    138                 ret.setName(ret.getName() + " " + layersString.substring(0, layersString.length() - 2));
     138                ret.setName(ret.getName() + ' ' + layersString.substring(0, layersString.length() - 2));
    139139            }
    140140            ret.setServerProjections(supportedCrs);
  • trunk/src/org/openstreetmap/josm/actions/AutoScaleAction.java

    r8836 r8846  
    188188            }
    189189        }
    190         putValue("active", true);
     190        putValue("active", Boolean.TRUE);
    191191    }
    192192
  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r8840 r8846  
    497497            if (predecessors.get(n) == null) return true;
    498498            if (predecessors.get(n).size() == 1) {
    499                 NodePair p1 = successors.get(n).iterator().next();
    500                 NodePair p2 = predecessors.get(n).iterator().next();
     499                NodePair p1 = successors.get(n).get(0);
     500                NodePair p2 = predecessors.get(n).get(0);
    501501                return p1.equals(p2.swap());
    502502            }
  • trunk/src/org/openstreetmap/josm/actions/HistoryInfoWebAction.java

    r8475 r8846  
    3030        if (infoObject instanceof OsmPrimitive) {
    3131            OsmPrimitive primitive = (OsmPrimitive) infoObject;
    32             return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId() + "/history";
     32            return Main.getBaseBrowseUrl() + '/' + OsmPrimitiveType.from(primitive).getAPIName() + '/' + primitive.getId() + "/history";
    3333        } else {
    3434            return null;
  • trunk/src/org/openstreetmap/josm/actions/ImageryAdjustAction.java

    r8836 r8846  
    262262            try (Formatter us = new Formatter(Locale.US)) {
    263263                tOffset.setText(us.format(
    264                     "%1." + precision + "f; %1." + precision + "f",
     264                    "%1." + precision + "f; %1." + precision + 'f',
    265265                    layer.getDx(), layer.getDy()).toString());
    266266            }
  • trunk/src/org/openstreetmap/josm/actions/InfoWebAction.java

    r8510 r8846  
    3636        if (infoObject instanceof OsmPrimitive) {
    3737            OsmPrimitive primitive = (OsmPrimitive) infoObject;
    38             return Main.getBaseBrowseUrl() + "/" + OsmPrimitiveType.from(primitive).getAPIName() + "/" + primitive.getId();
     38            return Main.getBaseBrowseUrl() + '/' + OsmPrimitiveType.from(primitive).getAPIName() + '/' + primitive.getId();
    3939        } else if (infoObject instanceof Note) {
    4040            Note note = (Note) infoObject;
  • trunk/src/org/openstreetmap/josm/actions/MapRectifierWMSmenuAction.java

    r8598 r8846  
    188188                if (s.wmsUrl.isEmpty()) {
    189189                    try {
    190                         addWMSLayer(s.name + " (" + text + ")", text);
     190                        addWMSLayer(s.name + " (" + text + ')', text);
    191191                        break outer;
    192192                    } catch (IllegalStateException ex) {
     
    200200                    String id = m.group(1);
    201201                    String newURL = s.wmsUrl.replaceAll("__s__", id);
    202                     String title = s.name + " (" + id + ")";
     202                    String title = s.name + " (" + id + ')';
    203203                    addWMSLayer(title, newURL);
    204204                    break outer;
     
    207207                if (s.idValidator.matcher(text).matches()) {
    208208                    String newURL = s.wmsUrl.replaceAll("__s__", text);
    209                     String title = s.name + " (" + text + ")";
     209                    String title = s.name + " (" + text + ')';
    210210                    addWMSLayer(title, newURL);
    211211                    break outer;
  • trunk/src/org/openstreetmap/josm/actions/PasteTagsAction.java

    r8510 r8846  
    305305            Main.main.undoRedo.add(
    306306                    new SequenceCommand(
    307                             title1 + " " + title2,
     307                            title1 + ' ' + title2,
    308308                            commands
    309309                    ));
  • trunk/src/org/openstreetmap/josm/actions/RestartAction.java

    r8736 r8846  
    158158                // else it's a .class, add the classpath and mainClass
    159159                cmd.add("-cp");
    160                 cmd.add("\"" + System.getProperty("java.class.path") + "\"");
     160                cmd.add('"' + System.getProperty("java.class.path") + '"');
    161161                cmd.add(mainCommand[0]);
    162162            }
  • trunk/src/org/openstreetmap/josm/actions/SaveActionBase.java

    r8802 r8846  
    172172            String fn = file.getPath();
    173173            if (ff instanceof ExtensionFileFilter) {
    174                 fn += "." + ((ExtensionFileFilter) ff).getDefaultExtension();
     174                fn += '.' + ((ExtensionFileFilter) ff).getDefaultExtension();
    175175            } else if (extension != null) {
    176                 fn += "." + extension;
     176                fn += '.' + extension;
    177177            }
    178178            file = new File(fn);
  • trunk/src/org/openstreetmap/josm/actions/SessionLoadAction.java

    r8540 r8846  
    115115                    if (canceled) return;
    116116                    if (!layers.isEmpty()) {
    117                         Layer firstLayer = layers.iterator().next();
     117                        Layer firstLayer = layers.get(0);
    118118                        boolean noMap = Main.map == null;
    119119                        if (noMap) {
  • trunk/src/org/openstreetmap/josm/actions/ShowStatusReportAction.java

    r8513 r8846  
    6060    private static void shortenParam(ListIterator<String> it, String[] param, String source, String target) {
    6161        if (source != null && target.length() < source.length() && param[1].startsWith(source)) {
    62             it.set(param[0] + "=" + param[1].replace(source, target));
     62            it.set(param[0] + '=' + param[1].replace(source, target));
    6363        }
    6464    }
  • trunk/src/org/openstreetmap/josm/actions/SplitWayAction.java

    r8540 r8846  
    455455                                Way w = relationMembers.get(i_r - k).getWay();
    456456                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    457                                     backwards = false;
     457                                    backwards = Boolean.FALSE;
    458458                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    459                                     backwards = true;
     459                                    backwards = Boolean.TRUE;
    460460                                }
    461461                                break;
     
    464464                                Way w = relationMembers.get(i_r + k).getWay();
    465465                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    466                                     backwards = true;
     466                                    backwards = Boolean.TRUE;
    467467                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    468                                     backwards = false;
     468                                    backwards = Boolean.FALSE;
    469469                                }
    470470                                break;
  • trunk/src/org/openstreetmap/josm/actions/UnGlueAction.java

    r8513 r8846  
    135135            errorTime = Notification.TIME_VERY_LONG;
    136136            errMsg =
    137                 tr("The current selection cannot be used for unglueing.")+"\n"+
    138                 "\n"+
    139                 tr("Select either:")+"\n"+
    140                 tr("* One tagged node, or")+"\n"+
    141                 tr("* One node that is used by more than one way, or")+"\n"+
    142                 tr("* One node that is used by more than one way and one of those ways, or")+"\n"+
    143                 tr("* One way that has one or more nodes that are used by more than one way, or")+"\n"+
    144                 tr("* One way and one or more of its nodes that are used by more than one way.")+"\n"+
    145                 "\n"+
     137                tr("The current selection cannot be used for unglueing.")+'\n'+
     138                '\n'+
     139                tr("Select either:")+'\n'+
     140                tr("* One tagged node, or")+'\n'+
     141                tr("* One node that is used by more than one way, or")+'\n'+
     142                tr("* One node that is used by more than one way and one of those ways, or")+'\n'+
     143                tr("* One way that has one or more nodes that are used by more than one way, or")+'\n'+
     144                tr("* One way and one or more of its nodes that are used by more than one way.")+'\n'+
     145                '\n'+
    146146                tr("Note: If a way is selected, this way will get fresh copies of the unglued\n"+
    147147                        "nodes and the new nodes will be selected. Otherwise, all ways will get their\n"+
  • trunk/src/org/openstreetmap/josm/actions/downloadtasks/DownloadOsmTask.java

    r8840 r8846  
    363363                // TODO: proper i18n after stabilization
    364364                Collection<String> items = new ArrayList<>();
    365                 items.add(tr("OSM Server URL:") + " " + url.getHost());
     365                items.add(tr("OSM Server URL:") + ' ' + url.getHost());
    366366                items.add(tr("Command")+": "+url.getPath());
    367367                if (url.getQuery() != null) {
  • trunk/src/org/openstreetmap/josm/actions/mapmode/ExtrudeAction.java

    r8840 r8846  
    167167        @Override
    168168        public String toString() {
    169             return "ReferenceSegment[en=" + en + ", p1=" + p1 + ", p2=" + p2 + ", perp=" + perpendicular + "]";
     169            return "ReferenceSegment[en=" + en + ", p1=" + p1 + ", p2=" + p2 + ", perp=" + perpendicular + ']';
    170170        }
    171171    }
  • trunk/src/org/openstreetmap/josm/actions/mapmode/SelectAction.java

    r8840 r8846  
    960960            else if (mode == Mode.MOVE && (System.currentTimeMillis() - mouseDownTime >= initialMoveDelay)) {
    961961                final boolean canMerge = getCurrentDataSet() != null && !getCurrentDataSet().getSelectedNodes().isEmpty();
    962                 final String mergeHelp = canMerge ? " " + tr("Ctrl to merge with nearest node.") : "";
     962                final String mergeHelp = canMerge ? ' ' + tr("Ctrl to merge with nearest node.") : "";
    963963                return tr("Release the mouse button to stop moving.") + mergeHelp;
    964964            } else if (mode == Mode.ROTATE)
  • trunk/src/org/openstreetmap/josm/actions/search/PushbackTokenizer.java

    r8674 r8846  
    3434        @Override
    3535        public String toString() {
    36             return "Range [start=" + start + ", end=" + end + "]";
     36            return "Range [start=" + start + ", end=" + end + ']';
    3737        }
    3838    }
  • trunk/src/org/openstreetmap/josm/actions/search/SearchAction.java

    r8840 r8846  
    207207                        try {
    208208                            JTextComponent tf = (JTextComponent) hcb.getEditor().getEditorComponent();
    209                             tf.getDocument().insertString(tf.getCaretPosition(), " " + insertText, null);
     209                            tf.getDocument().insertString(tf.getCaretPosition(), ' ' + insertText, null);
    210210                        } catch (BadLocationException ex) {
    211211                            throw new RuntimeException(ex.getMessage(), ex);
     
    628628            String all = allElements ? ", " +
    629629                            /*all elements*/ trc("search", "A") : "";
    630             return "\"" + text + "\" (" + cs + rx + css + all + ", " + mode + ")";
     630            return '"' + text + "\" (" + cs + rx + css + all + ", " + mode + ')';
    631631        }
    632632
  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r8840 r8846  
    159159                    case "timestamp":
    160160                        // add leading/trailing space in order to get expected split (e.g. "a--" => {"a", ""})
    161                         String rangeS = " " + tokenizer.readTextOrNumber() + " ";
     161                        String rangeS = ' ' + tokenizer.readTextOrNumber() + ' ';
    162162                        String[] rangeA = rangeS.split("/");
    163163                        if (rangeA.length == 1) {
     
    371371        @Override
    372372        public String toString() {
    373             return key + "?";
     373            return key + '?';
    374374        }
    375375    }
     
    607607        @Override
    608608        public String toString() {
    609             return key + "=" + value;
     609            return key + '=' + value;
    610610        }
    611611    }
     
    623623            Double v = null;
    624624            try {
    625                 v = Double.parseDouble(referenceValue);
     625                v = Double.valueOf(referenceValue);
    626626            } catch (NumberFormatException ignore) {
    627627            }
     
    10191019        @Override
    10201020        public String toString() {
    1021             return getString() + "=" + min + "-" + max;
     1021            return getString() + '=' + min + '-' + max;
    10221022        }
    10231023    }
     
    12441244        @Override
    12451245        public String toString() {
    1246             return "parent(" + match + ")";
     1246            return "parent(" + match + ')';
    12471247        }
    12481248    }
     
    12681268        @Override
    12691269        public String toString() {
    1270             return "child(" + match + ")";
     1270            return "child(" + match + ')';
    12711271        }
    12721272    }
Note: See TracChangeset for help on using the changeset viewer.