Changeset 14435 in josm for trunk/src/org/openstreetmap

Timestamp:

2018-11-20T01:51:27+01:00 (6 years ago)

Author:

Don-vip

Message:

fix SonarQube issues

Location:

trunk/src/org/openstreetmap/josm

Files:

• data/projection/ProjectionCLI.java (modified) (3 diffs)
• gui/download/DownloadDialog.java (modified) (1 diff)
• gui/layer/AbstractTileSourceLayer.java (modified) (1 diff)
• tools/OptionParser.java (modified) (7 diffs)

trunk/src/org/openstreetmap/josm/data/projection/ProjectionCLI.java

@@ -29 +29 @@
 public class ProjectionCLI implements CLIModule {
 
+    /** The unique instance **/
     public static final ProjectionCLI INSTANCE = new ProjectionCLI();
 
@@ -43 +44 @@
     public void processArguments(String[] argArray) {
         List<String> positionalArguments = new OptionParser("JOSM projection")
-            .addFlagParameter("help", this::showHelp)
+            .addFlagParameter("help", ProjectionCLI::showHelp)
             .addShortAlias("help", "h")
             .addFlagParameter("inverse", () -> argInverse = true)
@@ -84 +85 @@
      * Displays help on the console
      */
-    private void showHelp() {
+    private static void showHelp() {
         System.out.println(getHelp());
         System.exit(0);

trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

@@ -676 +676 @@
             });
 
-            addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> {
-                heightAdjustedExplicitly = true;
-            });
+            addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> heightAdjustedExplicitly = true);
         }
 

trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

@@ -1776 +1776 @@
 
         /**
+         * Constructs a new {@code PrecacheTask}.
          * @param progressMonitor that will be notified about progess of the task
          * @param bufferY buffer Y in degrees around which to download tiles

trunk/src/org/openstreetmap/josm/tools/OptionParser.java

@@ -168 +168 @@
                         parameter = split[1];
                     } else {
-                        if (toHandle.isEmpty() || toHandle.getFirst().equals("--")) {
+                        if (toHandle.isEmpty() || "--".equals(toHandle.getFirst())) {
                             throw new OptionParseException(tr("{0}: option ''{1}'' requires an argument", program));
                         }
@@ -198 +198 @@
                 option.option.runFor(option.parameter);
             } catch (OptionParseException e) {
-                String message;
+                StringBuilder message = new StringBuilder();
                 // Just add a nicer error message
                 if (option.parameter == null) {
-                    message = tr("{0}: Error while handling option ''{1}''", program, option.optionName);
+                    message.append(tr("{0}: Error while handling option ''{1}''", program, option.optionName));
                 } else {
-                    message = tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
-                            option.parameter);
+                    message.append(tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
+                            option.parameter));
                 }
                 if (!e.getLocalizedMessage().isEmpty()) {
-                    message += ": " + e.getLocalizedMessage().isEmpty();
+                    message.append(": ").append(e.getLocalizedMessage().isEmpty());
                 }
-                throw new OptionParseException(message, e);
+                throw new OptionParseException(message.toString(), e);
             }
         }
@@ -261 +261 @@
     }
 
-    protected abstract static class AvailableOption {
-
-        public boolean requiresParameter() {
+    protected interface AvailableOption {
+
+        /**
+         * Determines if this option requires a parameter.
+         * @return {@code true} if this option requires a parameter ({@code false} by default)
+         */
+        default boolean requiresParameter() {
             return false;
         }
 
-        public OptionCount getRequiredCount() {
+        /**
+         * Determines how often this option may / must be specified on the command line.
+         * @return how often this option may / must be specified on the command line
+         */
+        default OptionCount getRequiredCount() {
             return OptionCount.OPTIONAL;
         }
@@ -275 +283 @@
          * @param parameter The parameter if {@link #requiresParameter()} is true, <code>null</code> otherwise.
          */
-        public abstract void runFor(String parameter);
-
+        void runFor(String parameter);
     }
 
@@ -292 +299 @@
 
     /**
+     * Exception thrown when an option cannot be parsed.
      * @author Michael Zangl
      */
@@ -307 +315 @@
 
         /**
+         * Create an error with a localized description
          * @param localizedMessage The message to display to the user.
          */
@@ -315 +324 @@
 
         /**
+         * Create an error with a localized description and a root cause
          * @param localizedMessage The message to display to the user.
          * @param t The error that caused this message to be displayed.