Changeset 8461 in josm


Ignore:
Timestamp:
2015-06-03T14:05:30+02:00 (9 years ago)
Author:
Don-vip
Message:

global use of String.isEmpty()

Location:
trunk/src/org/openstreetmap/josm
Files:
26 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/search/SearchCompiler.java

    r8413 r8461  
    651651            }
    652652
    653             if (regexp && key.length() > 0 && !"*".equals(key)) {
     653            if (regexp && !key.isEmpty() && !"*".equals(key)) {
    654654                try {
    655655                    keyPattern = Pattern.compile(key, regexFlags(false));
    … …  
    662662                keyPattern = null;
    663663            }
    664             if (regexp && this.value.length() > 0 && !"*".equals(this.value)) {
     664            if (regexp && !this.value.isEmpty() && !"*".equals(this.value)) {
    665665                try {
    666666                    valuePattern = Pattern.compile(this.value, regexFlags(false));

  • trunk/src/org/openstreetmap/josm/data/CustomConfigurator.java

    r8449 r8461  
    178178    public static int askForOption(String text, String opts) {
    179179        Integer answer;
    180         if (opts.length()>0) {
     180        if (!opts.isEmpty()) {
    181181            String[] options = opts.split(";");
    182182            answer = JOptionPane.showOptionDialog(Main.parent, text, "Question", JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.QUESTION_MESSAGE, null, options, 0);
    … …  
    189189    public static String askForText(String text) {
    190190        String s = JOptionPane.showInputDialog(Main.parent, text, tr("Enter text"), JOptionPane.QUESTION_MESSAGE);
    191         if (s!=null && (s=s.trim()).length()>0) {
     191        if (s!=null && !(s=s.trim()).isEmpty()) {
    192192            return s;
    193193        } else {
    … …  
    558558            PreferencesUtils.showPrefs(tmpPref);
    559559
    560             if (id.length()>0) {
     560            if (!id.isEmpty()) {
    561561                try {
    562562                    String fragmentVar = "API.fragments['"+id+"']";
    … …  
    621621            String text = evalVars(elem.getAttribute("text"));
    622622            String locText = evalVars(elem.getAttribute(LanguageInfo.getJOSMLocaleCode()+".text"));
    623             if (locText!=null && locText.length()>0) text=locText;
     623            if (locText!=null && !locText.isEmpty()) text=locText;
    624624
    625625            String type = evalVars(elem.getAttribute("type"));
    … …  
    630630            String text = evalVars(elem.getAttribute("text"));
    631631            String locText = evalVars(elem.getAttribute(LanguageInfo.getJOSMLocaleCode()+".text"));
    632             if (locText.length()>0) text=locText;
     632            if (!locText.isEmpty()) text=locText;
    633633            String var = elem.getAttribute("var");
    634634            if (var.isEmpty()) var="result";
    … …  
    640640                String opts = evalVars(elem.getAttribute("options"));
    641641                String locOpts = evalVars(elem.getAttribute(LanguageInfo.getJOSMLocaleCode()+".options"));
    642                 if (locOpts.length()>0) opts=locOpts;
     642                if (!locOpts.isEmpty()) opts=locOpts;
    643643                setVar(var, String.valueOf(askForOption(text, opts)));
    644644            }

  • trunk/src/org/openstreetmap/josm/data/validation/routines/InetAddressValidator.java

    r7937 r8461  
    7474        for (int i = 0; i <= 3; i++) {
    7575            String ipSegment = groups[i];
    76             if (ipSegment == null || ipSegment.length() <= 0) {
     76            if (ipSegment == null || ipSegment.isEmpty()) {
    7777                return false;
    7878            }

  • trunk/src/org/openstreetmap/josm/data/validation/routines/UrlValidator.java

    r7937 r8461  
    405405
    406406        String extra = authorityMatcher.group(PARSE_AUTHORITY_EXTRA);
    407         if (extra != null && extra.trim().length() > 0){
     407        if (extra != null && !extra.trim().isEmpty()){
    408408            return false;
    409409        }

  • trunk/src/org/openstreetmap/josm/data/validation/tests/RelationChecker.java

    r8401 r8461  
    301301                }));
    302302
    303                 if (key.length() > 0) {
     303                if (!key.isEmpty()) {
    304304                    String s = marktr("Role {0} unknown in templates {1}");
    305305

  • trunk/src/org/openstreetmap/josm/data/validation/tests/TagChecker.java

    r8444 r8461  
    177177                boolean isFirstLine = true;
    178178                String line;
    179                 while ((line = reader.readLine()) != null && (tagcheckerfile || line.length() != 0)) {
     179                while ((line = reader.readLine()) != null && (tagcheckerfile || !line.isEmpty())) {
    180180                    if (line.startsWith("#")) {
    181181                        if (line.startsWith("# JOSM TagChecker")) {
    … …  
    218218                        }
    219219                    } else if (tagcheckerfile) {
    220                         if (line.length() > 0) {
     220                        if (!line.isEmpty()) {
    221221                            CheckerData d = new CheckerData();
    222222                            String err = d.getData(line);
    … …  
    247247        }
    248248
    249         if (errorSources.length() > 0)
     249        if (!errorSources.isEmpty())
    250250            throw new IOException(tr("Could not access data file(s):\n{0}", errorSources));
    251251    }
    … …  
    382382                withErrors.put(p, "HTML");
    383383            }
    384             if (checkValues && key != null && value != null && value.length() > 0 && presetsValueData != null) {
     384            if (checkValues && key != null && value != null && !value.isEmpty() && presetsValueData != null) {
    385385                final Set<String> values = presetsValueData.get(key);
    386386                final boolean keyInPresets = values != null;
    … …  
    440440                }
    441441            }
    442             if (checkFixmes && key != null && value != null && value.length() > 0) {
     442            if (checkFixmes && key != null && value != null && !value.isEmpty()) {
    443443                if ((value.toLowerCase(Locale.ENGLISH).contains("fixme")
    444444                        || value.contains("check and delete")

  • trunk/src/org/openstreetmap/josm/gui/DefaultNameFormatter.java

    r8443 r8461  
    525525              .append("</strong>=");
    526526            String value = tags.get(key);
    527             while(value.length() != 0) {
     527            while (!value.isEmpty()) {
    528528                sb.append(value.substring(0,Math.min(50, value.length())));
    529529                if (value.length() > 50) {

  • trunk/src/org/openstreetmap/josm/gui/ExtendedDialog.java

    r8441 r8461  
    492492
    493493        // Ensure all required variables are available
    494         if(rememberSizePref.length() != 0 && defaultWindowGeometry != null) {
     494        if (!rememberSizePref.isEmpty() && defaultWindowGeometry != null) {
    495495            if(visible) {
    496496                initWindowGeometry().applySafe(this);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/changeset/query/AdvancedChangesetQueryPanel.java

    r8444 r8461  
    10251025        @Override
    10261026        public boolean isValid() {
    1027             return getComponent().getText().trim().length() > 0;
     1027            return !getComponent().getText().trim().isEmpty();
    10281028        }
    10291029

  • trunk/src/org/openstreetmap/josm/gui/download/PlaceSelection.java

    r8394 r8461  
    305305
    306306        protected final void updateEnabledState() {
    307             setEnabled(cbSearchExpression.getText().trim().length() > 0);
     307            setEnabled(!cbSearchExpression.getText().trim().isEmpty());
    308308        }
    309309

  • trunk/src/org/openstreetmap/josm/gui/layer/NoteLayer.java

    r8444 r8461  
    117117                String commentText = comment.getText();
    118118                //closing a note creates an empty comment that we don't want to show
    119                 if (commentText != null && commentText.trim().length() > 0) {
     119                if (commentText != null && !commentText.trim().isEmpty()) {
    120120                    sb.append("<hr/>");
    121121                    String userName = XmlWriter.encode(comment.getUser().getName());

  • trunk/src/org/openstreetmap/josm/gui/oauth/FullyAutomaticAuthorizationUI.java

    r8442 r8461  
    408408        @Override
    409409        public boolean isValid() {
    410             return getComponent().getText().trim().length() > 0;
     410            return !getComponent().getText().trim().isEmpty();
    411411        }
    412412
    … …  
    429429        @Override
    430430        public boolean isValid() {
    431             return getComponent().getText().trim().length() > 0;
     431            return !getComponent().getText().trim().isEmpty();
    432432        }
    433433

  • trunk/src/org/openstreetmap/josm/gui/oauth/FullyAutomaticPropertiesPanel.java

    r6890 r8461  
    8282        @Override
    8383        public boolean isValid() {
    84             return getComponent().getText().trim().length() > 0;
     84            return !getComponent().getText().trim().isEmpty();
    8585        }
    8686

  • trunk/src/org/openstreetmap/josm/gui/preferences/PreferenceTabbedPane.java

    r8444 r8461  
    489489                    // Get icon
    490490                    String iconName = tps.getIconName();
    491                     ImageIcon icon = iconName != null && iconName.length() > 0 ? ImageProvider.get("preferences", iconName) : null;
     491                    ImageIcon icon = iconName != null && !iconName.isEmpty() ? ImageProvider.get("preferences", iconName) : null;
    492492                    // See #6985 - Force icons to be 48x48 pixels
    493493                    if (icon != null && (icon.getIconHeight() != 48 || icon.getIconWidth() != 48)) {

  • trunk/src/org/openstreetmap/josm/gui/preferences/ToolbarPreferences.java

    r8444 r8461  
    218218                    skip('=');
    219219                    String paramValue = readTillChar(',',')');
    220                     if (paramName.length() > 0 && paramValue.length() > 0) {
     220                    if (!paramName.isEmpty() && !paramValue.isEmpty()) {
    221221                        ActionParameter<?> actionParam = actionParams.get(paramName);
    222222                        if (actionParam != null) {
    … …  
    235235                    skip('=');
    236236                    String paramValue = readTillChar(',','}');
    237                     if ("icon".equals(paramName) && paramValue.length() > 0) {
     237                    if ("icon".equals(paramName) && !paramValue.isEmpty()) {
    238238                        result.setIcon(paramValue);
    239                     } else if("name".equals(paramName) && paramValue.length() > 0) {
     239                    } else if("name".equals(paramName) && !paramValue.isEmpty()) {
    240240                        result.setName(paramValue);
    241241                    }
    … …  
    289289                boolean first = true;
    290290                String tmp = action.getName();
    291                 if(tmp.length() != 0) {
     291                if (!tmp.isEmpty()) {
    292292                    result.append(first ? "{" : ",");
    293293                    result.append("name=");
    … …  
    296296                }
    297297                tmp = action.getIcon();
    298                 if(tmp.length() != 0) {
     298                if (!tmp.isEmpty()) {
    299299                    result.append(first ? "{" : ",");
    300300                    result.append("icon=");
    … …  
    385385            ActionParameter<Object> param = getParam(paramIndex);
    386386
    387             if (param != null && val.length() > 0) {
     387            if (param != null && !val.isEmpty()) {
    388388                currentAction.getParameters().put(param.getName(), param.readFromString((String)aValue));
    389389            }

  • trunk/src/org/openstreetmap/josm/gui/preferences/server/UserNameValidator.java

    r6529 r8461  
    2323    @Override
    2424    public boolean isValid() {
    25         return getComponent().getText().trim().length() > 0;
     25        return !getComponent().getText().trim().isEmpty();
    2626    }
    2727

  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletingComboBox.java

    r8441 r8461  
    104104                try {
    105105                    Long.parseLong(str);
    106                     if (curText.length() != 0)
     106                    if (!curText.isEmpty())
    107107                        Long.parseLong(curText);
    108108                    item = lookupItem(curText, true);

  • trunk/src/org/openstreetmap/josm/io/GpxExporter.java

    r8443 r8461  
    147147
    148148        Main.pref.put("lastAddAuthor", author.isSelected());
    149         if (authorName.getText().length() != 0) {
     149        if (!authorName.getText().isEmpty()) {
    150150            Main.pref.put("lastAuthorName", authorName.getText());
    151151        }
    152         if (copyright.getText().length() != 0) {
     152        if (!copyright.getText().isEmpty()) {
    153153            Main.pref.put("lastCopyright", copyright.getText());
    154154        }
    … …  
    164164        // add author and copyright details to the gpx data
    165165        if (author.isSelected()) {
    166             if (authorName.getText().length() > 0) {
     166            if (!authorName.getText().isEmpty()) {
    167167                gpxData.put(META_AUTHOR_NAME, authorName.getText());
    168168                gpxData.put(META_COPYRIGHT_AUTHOR, authorName.getText());
    169169            }
    170             if (email.getText().length() > 0) {
     170            if (!email.getText().isEmpty()) {
    171171                gpxData.put(META_AUTHOR_EMAIL, email.getText());
    172172            }
    173             if (copyright.getText().length() > 0) {
     173            if (!copyright.getText().isEmpty()) {
    174174                gpxData.put(META_COPYRIGHT_LICENSE, copyright.getText());
    175175            }
    176             if (copyrightYear.getText().length() > 0) {
     176            if (!copyrightYear.getText().isEmpty()) {
    177177                gpxData.put(META_COPYRIGHT_YEAR, copyrightYear.getText());
    178178            }
    … …  
    180180
    181181        // add the description to the gpx data
    182         if (desc.getText().length() > 0) {
     182        if (!desc.getText().isEmpty()) {
    183183            gpxData.put(META_DESC, desc.getText());
    184184        }
    185185
    186186        // add keywords to the gpx data
    187         if (keywords.getText().length() > 0) {
     187        if (!keywords.getText().isEmpty()) {
    188188            gpxData.put(META_KEYWORDS, keywords.getText());
    189189        }
    … …  
    271271                    email.setText("");
    272272                }
    273                 boolean isAuthorSet = authorName.getText().length() != 0;
    274                 GpxExporter.enableCopyright(data, copyright, predefined, copyrightYear, copyrightLabel, copyrightYearLabel, warning, b && isAuthorSet);
     273                boolean isAuthorSet = !authorName.getText().isEmpty();
     274                GpxExporter.enableCopyright(data, copyright, predefined, copyrightYear, copyrightLabel, copyrightYearLabel, warning,
     275                        b && isAuthorSet);
    275276            }
    276277        };
    … …  
    279280        KeyAdapter authorNameListener = new KeyAdapter(){
    280281            @Override public void keyReleased(KeyEvent e) {
    281                 boolean b = authorName.getText().length()!=0 && author.isSelected();
     282                boolean b = !authorName.getText().isEmpty() && author.isSelected();
    282283                GpxExporter.enableCopyright(data, copyright, predefined, copyrightYear, copyrightLabel, copyrightYearLabel, warning, b);
    283284            }

  • trunk/src/org/openstreetmap/josm/io/GpxReader.java

    r8390 r8461  
    364364                    break;
    365365                case "link":
    366                     if (currentLink.uri == null && accumulator != null && accumulator.toString().length() != 0) {
     366                    if (currentLink.uri == null && accumulator != null && !accumulator.toString().isEmpty()) {
    367367                        currentLink = new GpxLink(accumulator.toString());
    368368                    }

  • trunk/src/org/openstreetmap/josm/io/GpxWriter.java

    r8449 r8461  
    250250     */
    251251    private void simpleTag(String tag, String content) {
    252         if (content != null && content.length() > 0) {
     252        if (content != null && !content.isEmpty()) {
    253253            open(tag);
    254254            out.print(encode(content));

  • trunk/src/org/openstreetmap/josm/io/OsmReader.java

    r8415 r8461  
    436436
    437437        String time = parser.getAttributeValue(null, "timestamp");
    438         if (time != null && time.length() != 0) {
     438        if (time != null && !time.isEmpty()) {
    439439            current.setTimestamp(DateUtils.fromString(time));
    440440        }

  • trunk/src/org/openstreetmap/josm/plugins/ReadRemotePluginInformationTask.java

    r8443 r8461  
    147147        String pl = Utils.join(",", Main.pref.getCollection("plugins"));
    148148        String printsite = site.replaceAll("%<(.*)>", "");
    149         if (pl != null && pl.length() != 0) {
     149        if (pl != null && !pl.isEmpty()) {
    150150            site = site.replaceAll("%<(.*)>", "$1"+pl);
    151151        } else {

  • trunk/src/org/openstreetmap/josm/tools/ColorHelper.java

    r8404 r8461  
    2020     */
    2121    public static Color html2color(String html) {
    22         if (html.length() > 0 && html.charAt(0) == '#')
     22        if (!html.isEmpty() && html.charAt(0) == '#')
    2323            html = html.substring(1);
    2424        if (html.length() == 3) {

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookOsx.java

    r8377 r8461  
    292292        }
    293293        result += name;
    294         if (sc != null && sc.getKeyText().length() != 0) {
     294        if (sc != null && !sc.getKeyText().isEmpty()) {
    295295            result += " ";
    296296            if (canHtml) {

  • trunk/src/org/openstreetmap/josm/tools/PlatformHookUnixoid.java

    r8404 r8461  
    147147        result += "<html>";
    148148        result += name;
    149         if (sc != null && sc.getKeyText().length() != 0) {
     149        if (sc != null && !sc.getKeyText().isEmpty()) {
    150150            result += " ";
    151151            result += "<font size='-2'>";

  • trunk/src/org/openstreetmap/josm/tools/TextTagParser.java

    r8443 r8461  
    8585                    esc = true;
    8686                } else if (c == '\"' && !quotesStarted) { // opening "
    87                     if (s.toString().trim().length()>0) { // we had   ||some text"||
     87                    if (!s.toString().trim().isEmpty()) { // we had   ||some text"||
    8888                        s.append(c); // just add ", not open
    8989                    } else {
Note: See TracChangeset for help on using the changeset viewer.