Changeset 14435 in josm


Ignore:
Timestamp:
2018-11-20T01:51:27+01:00 (3 weeks ago)
Author:
Don-vip
Message:

fix SonarQube issues

Location:
trunk
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/projection/ProjectionCLI.java

    r14415 r14435  
    2929public class ProjectionCLI implements CLIModule {
    3030
     31    /** The unique instance **/
    3132    public static final ProjectionCLI INSTANCE = new ProjectionCLI();
    3233
     
    4344    public void processArguments(String[] argArray) {
    4445        List<String> positionalArguments = new OptionParser("JOSM projection")
    45             .addFlagParameter("help", this::showHelp)
     46            .addFlagParameter("help", ProjectionCLI::showHelp)
    4647            .addShortAlias("help", "h")
    4748            .addFlagParameter("inverse", () -> argInverse = true)
     
    8485     * Displays help on the console
    8586     */
    86     private void showHelp() {
     87    private static void showHelp() {
    8788        System.out.println(getHelp());
    8889        System.exit(0);
  • trunk/src/org/openstreetmap/josm/gui/download/DownloadDialog.java

    r14418 r14435  
    676676            });
    677677
    678             addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> {
    679                 heightAdjustedExplicitly = true;
    680             });
     678            addPropertyChangeListener(DIVIDER_LOCATION_PROPERTY, e -> heightAdjustedExplicitly = true);
    681679        }
    682680
  • trunk/src/org/openstreetmap/josm/gui/layer/AbstractTileSourceLayer.java

    r14428 r14435  
    17761776
    17771777        /**
     1778         * Constructs a new {@code PrecacheTask}.
    17781779         * @param progressMonitor that will be notified about progess of the task
    17791780         * @param bufferY buffer Y in degrees around which to download tiles
  • trunk/src/org/openstreetmap/josm/tools/OptionParser.java

    r14419 r14435  
    168168                        parameter = split[1];
    169169                    } else {
    170                         if (toHandle.isEmpty() || toHandle.getFirst().equals("--")) {
     170                        if (toHandle.isEmpty() || "--".equals(toHandle.getFirst())) {
    171171                            throw new OptionParseException(tr("{0}: option ''{1}'' requires an argument", program));
    172172                        }
     
    198198                option.option.runFor(option.parameter);
    199199            } catch (OptionParseException e) {
    200                 String message;
     200                StringBuilder message = new StringBuilder();
    201201                // Just add a nicer error message
    202202                if (option.parameter == null) {
    203                     message = tr("{0}: Error while handling option ''{1}''", program, option.optionName);
     203                    message.append(tr("{0}: Error while handling option ''{1}''", program, option.optionName));
    204204                } else {
    205                     message = tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
    206                             option.parameter);
     205                    message.append(tr("{0}: Invalid value {2} for option ''{1}''", program, option.optionName,
     206                            option.parameter));
    207207                }
    208208                if (!e.getLocalizedMessage().isEmpty()) {
    209                     message += ": " + e.getLocalizedMessage().isEmpty();
     209                    message.append(": ").append(e.getLocalizedMessage().isEmpty());
    210210                }
    211                 throw new OptionParseException(message, e);
     211                throw new OptionParseException(message.toString(), e);
    212212            }
    213213        }
     
    261261    }
    262262
    263     protected abstract static class AvailableOption {
    264 
    265         public boolean requiresParameter() {
     263    protected interface AvailableOption {
     264
     265        /**
     266         * Determines if this option requires a parameter.
     267         * @return {@code true} if this option requires a parameter ({@code false} by default)
     268         */
     269        default boolean requiresParameter() {
    266270            return false;
    267271        }
    268272
    269         public OptionCount getRequiredCount() {
     273        /**
     274         * Determines how often this option may / must be specified on the command line.
     275         * @return how often this option may / must be specified on the command line
     276         */
     277        default OptionCount getRequiredCount() {
    270278            return OptionCount.OPTIONAL;
    271279        }
     
    275283         * @param parameter The parameter if {@link #requiresParameter()} is true, <code>null</code> otherwise.
    276284         */
    277         public abstract void runFor(String parameter);
    278 
     285        void runFor(String parameter);
    279286    }
    280287
     
    292299
    293300    /**
     301     * Exception thrown when an option cannot be parsed.
    294302     * @author Michael Zangl
    295303     */
     
    307315
    308316        /**
     317         * Create an error with a localized description
    309318         * @param localizedMessage The message to display to the user.
    310319         */
     
    315324
    316325        /**
     326         * Create an error with a localized description and a root cause
    317327         * @param localizedMessage The message to display to the user.
    318328         * @param t The error that caused this message to be displayed.
  • trunk/test/unit/org/openstreetmap/josm/tools/OptionParserTest.java

    r14415 r14435  
    33
    44import static org.junit.Assert.assertEquals;
     5import static org.junit.Assert.assertFalse;
     6import static org.junit.Assert.assertTrue;
    57
    68import java.util.ArrayList;
     
    2022public class OptionParserTest {
    2123
    22     // A reason for moving to jupter...
     24    // A reason for moving to jupiter...
    2325    @Test(expected = OptionParseException.class)
    2426    public void testEmptyParserRejectsLongopt() {
     
    4345
    4446    @Test
    45     public void testparserOption() {
     47    public void testParserOption() {
    4648        AtomicReference<String> argFound = new AtomicReference<>();
    4749        OptionParser parser = new OptionParser("test")
     
    5355
    5456    @Test(expected = OptionParseException.class)
    55     public void testparserOptionFailsIfMissing() {
     57    public void testParserOptionFailsIfMissing() {
    5658        AtomicReference<String> argFound = new AtomicReference<>();
    5759        OptionParser parser = new OptionParser("test")
     
    6264
    6365    @Test(expected = OptionParseException.class)
    64     public void testparserOptionFailsIfMissingArgument() {
     66    public void testParserOptionFailsIfMissingArgument() {
    6567        AtomicReference<String> argFound = new AtomicReference<>();
    6668        OptionParser parser = new OptionParser("test")
     
    7173
    7274    @Test(expected = OptionParseException.class)
    73     public void testparserOptionFailsIfMissing2() {
     75    public void testParserOptionFailsIfMissing2() {
    7476        AtomicReference<String> argFound = new AtomicReference<>();
    7577        OptionParser parser = new OptionParser("test")
     
    8082
    8183    @Test(expected = OptionParseException.class)
    82     public void testparserOptionFailsIfTwice() {
     84    public void testParserOptionFailsIfTwice() {
    8385        AtomicReference<String> argFound = new AtomicReference<>();
    8486        OptionParser parser = new OptionParser("test")
     
    8991
    9092    @Test(expected = OptionParseException.class)
    91     public void testparserOptionFailsIfTwiceForAlias() {
     93    public void testParserOptionFailsIfTwiceForAlias() {
    9294        AtomicReference<String> argFound = new AtomicReference<>();
    9395        OptionParser parser = new OptionParser("test")
     
    226228        assertEquals("arg", argFound.get());
    227229        assertEquals(Arrays.asList("m1", "m2"), multiFound);
    228         assertEquals(true, usedFlag.get());
    229         assertEquals(false, unusedFlag.get());
     230        assertTrue(usedFlag.get());
     231        assertFalse(unusedFlag.get());
    230232    }
    231233
     
    245247        assertEquals(Arrays.asList(), remaining);
    246248        assertEquals("arg", argFound.get());
    247         assertEquals(true, usedFlag.get());
    248         assertEquals(false, unusedFlag.get());
    249     }
    250 
    251     @Test(expected = OptionParseException.class)
    252     public void testAbigiousAlternatives() {
     249        assertTrue(usedFlag.get());
     250        assertFalse(unusedFlag.get());
     251    }
     252
     253    @Test(expected = OptionParseException.class)
     254    public void testAmbiguousAlternatives() {
    253255        AtomicReference<String> argFound = new AtomicReference<>();
    254256        AtomicBoolean usedFlag = new AtomicBoolean();
     
    280282        assertEquals(Arrays.asList("x"), remaining);
    281283        assertEquals("arg", argFound.get());
    282         assertEquals(true, usedFlag.get());
    283         assertEquals(false, unusedFlag.get());
     284        assertTrue(usedFlag.get());
     285        assertFalse(unusedFlag.get());
    284286
    285287        remaining = parser.parseOptions(Arrays.asList("-ft", "arg", "x"));
     
    287289        assertEquals(Arrays.asList("x"), remaining);
    288290        assertEquals("arg", argFound.get());
    289         assertEquals(true, usedFlag.get());
    290         assertEquals(false, unusedFlag.get());
     291        assertTrue(usedFlag.get());
     292        assertFalse(unusedFlag.get());
    291293
    292294        remaining = parser.parseOptions(Arrays.asList("-f", "-t=arg", "x"));
     
    294296        assertEquals(Arrays.asList("x"), remaining);
    295297        assertEquals("arg", argFound.get());
    296         assertEquals(true, usedFlag.get());
    297         assertEquals(false, unusedFlag.get());
     298        assertTrue(usedFlag.get());
     299        assertFalse(unusedFlag.get());
    298300    }
    299301
     
    319321
    320322    @Test(expected = IllegalArgumentException.class)
    321     public void testDupplicateOptionName() {
     323    public void testDuplicateOptionName() {
    322324        new OptionParser("test").addFlagParameter("test", this::nop).addFlagParameter("test", this::nop);
    323325    }
    324326
    325327    @Test(expected = IllegalArgumentException.class)
    326     public void testDupplicateOptionName2() {
     328    public void testDuplicateOptionName2() {
    327329        new OptionParser("test").addFlagParameter("test", this::nop)
    328330            .addArgumentParameter("test", OptionCount.OPTIONAL, this::nop);
     
    345347
    346348    @Test(expected = IllegalArgumentException.class)
    347     public void testDupplicateShortAlias() {
     349    public void testDuplicateShortAlias() {
    348350        new OptionParser("test").addFlagParameter("test", this::nop)
    349351        .addFlagParameter("test2", this::nop)
Note: See TracChangeset for help on using the changeset viewer.