Ignore:
Timestamp:
2016-07-05T23:46:31+02:00 (8 years ago)
Author:
donvip
Message:

checkstyle

Location:
applications/editors/josm/plugins/OpeningHoursEditor
Files:
1 added
8 edited

Legend:

Unmodified
Added
Removed
  • applications/editors/josm/plugins/OpeningHoursEditor/.project

    r32286 r32583  
    2121                        </arguments>
    2222                </buildCommand>
     23                <buildCommand>
     24                        <name>net.sf.eclipsecs.core.CheckstyleBuilder</name>
     25                        <arguments>
     26                        </arguments>
     27                </buildCommand>
    2328        </buildSpec>
    2429        <natures>
    2530                <nature>org.eclipse.jdt.core.javanature</nature>
    2631                <nature>sf.eclipse.javacc.javaccnature</nature>
     32                <nature>net.sf.eclipsecs.core.CheckstyleNature</nature>
    2733        </natures>
    2834</projectDescription>
  • applications/editors/josm/plugins/OpeningHoursEditor/build.xml

    r32448 r32583  
    44    <property name="commit.message" value="fixed main version"/>
    55    <!-- enter the *lowest* JOSM version this plugin is currently compatible with -->
    6     <property name="plugin.main.version" value="10353"/>
     6    <property name="plugin.main.version" value="10420"/>
    77       
    88    <property name="plugin.author" value="Falko Thomale"/>
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/ClockSystem.java

    r30116 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe;
    23
     
    45import java.text.ParseException;
    56import java.util.Locale;
     7
     8import org.openstreetmap.josm.Main;
    69
    710/**
     
    2730            midnight = localeFormat.format(stdFormat.parse("12:00 AM"));
    2831        } catch (ParseException ignore) {
     32            Main.trace(ignore);
    2933        }
    3034        if (midnight.contains("12"))
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/OhePlugin.java

    r32448 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe;
    23
     
    5960     */
    6061    private final String[][] TAG_EDIT_STRINGS = new String[][] {
    61             { "opening_hours", ".*", "opening_hours" },
    62             { "collection_times", ".*", "collection_times" },
    63             { "collection_times:local", ".*", "collection_times:local" },
    64             { "shop", ".*", "opening_hours" },
    65             { "amenity", "post_box", "collection_times" },
    66             { "amenity", "recycling", "collection_times" },
    67             { "amenity", ".*", "opening_hours" },
    68             { "lit", ".*", "lit" },
    69             { "highway", ".*", "lit" } };
     62            {"opening_hours", ".*", "opening_hours"},
     63            {"collection_times", ".*", "collection_times"},
     64            {"collection_times:local", ".*", "collection_times:local"},
     65            {"shop", ".*", "opening_hours"},
     66            {"amenity", "post_box", "collection_times"},
     67            {"amenity", "recycling", "collection_times"},
     68            {"amenity", ".*", "opening_hours"},
     69            {"lit", ".*", "lit"},
     70            {"highway", ".*", "lit"} };
    7071
    7172    /**
     
    8687        private static final long serialVersionUID = 1456257438391417756L;
    8788
    88         public OheMenuAction() {
     89        OheMenuAction() {
    8990            super(tr("Edit opening hours"), "opening_hours.png",
    9091                    tr("Edit time-tag of selected element in a graphical interface"), Shortcut.registerShortcut(
     
    149150                }
    150151            };
    151             propertyData.setColumnIdentifiers(new String[] { tr("Key"), tr("Value") });
     152            propertyData.setColumnIdentifiers(new String[] {tr("Key"), tr("Value")});
    152153            for (Entry<String, Map<String, Integer>> e : valueCount.entrySet()) {
    153154                int count = 0;
     
    158159                    e.getValue().put("", selection.size() - count);
    159160                }
    160                 propertyData.addRow(new Object[] { e.getKey(), e.getValue() });
     161                propertyData.addRow(new Object[] {e.getKey(), e.getValue()});
    161162            }
    162163            final JTable propertyTable = new JTable(propertyData);
     
    229230                            break searchLoop;
    230231                        } else if (value instanceof Map<?, ?>) {
    231                             for (String v : ((Map<String, Integer>) value).keySet())
     232                            for (String v : ((Map<String, Integer>) value).keySet()) {
    232233                                if (valuePattern.matcher(v).matches()) {
    233234                                    preSelectedKey = pattern[2];
    234235                                    break searchLoop;
    235236                                }
     237                            }
    236238                        }
    237239                    }
     
    239241            }
    240242            int preSelectedRow = -1;
    241             for (int i = 0; i < propertyData.getRowCount(); ++i)
     243            for (int i = 0; i < propertyData.getRowCount(); ++i) {
    242244                if (preSelectedKey.equals(propertyData.getValueAt(i, 0))) {
    243245                    preSelectedRow = i;
    244246                }
     247            }
    245248            if (preSelectedRow != -1) {
    246249                propertyTable.setEnabled(true);
     
    304307            String[] changedKeyValuePair = null;
    305308            answer = optionPane.getValue();
    306             if (!(answer == null || answer == JOptionPane.UNINITIALIZED_VALUE || (answer instanceof Integer && (Integer) answer != JOptionPane.OK_OPTION))) {
     309            if (!(answer == null || answer == JOptionPane.UNINITIALIZED_VALUE
     310              || (answer instanceof Integer && (Integer) answer != JOptionPane.OK_OPTION))) {
    307311                changedKeyValuePair = panel.getChangedKeyValuePair();
    308312            }
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/OpeningTimeUtils.java

    r30737 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe;
    23
     
    910 * Collection of utility methods.
    1011 */
    11 public class OpeningTimeUtils {
     12public final class OpeningTimeUtils {
     13   
     14    private OpeningTimeUtils() {
     15        // Hide default constructors for utilities classes
     16    }
    1217   
    1318    /**
     
    5358                    for (DaytimeSpan timespan : dateTime2.daytimeSpans) {
    5459                        if (!timespan.isOff()) {
    55                             ret.add(new int[] { dayspan.startDay, dayspan.endDay, timespan.startMinute,
     60                            ret.add(new int[] {dayspan.startDay, dayspan.endDay, timespan.startMinute,
    5661                                    timespan.endMinute });
    5762                        }
     
    245250
    246251    private static boolean isArrayEmpty(boolean[] bs) {
    247         for (int i = 0; i < bs.length; i++)
     252        for (int i = 0; i < bs.length; i++) {
    248253            if (bs[i])
    249254                return false;
     255        }
    250256        return true;
    251257    }
     
    265271     * @throws IllegalArgumentException if the day is invalid
    266272     */
    267     public static final void ensureValidDay(int day, String paramName) throws IllegalArgumentException {
     273    public static void ensureValidDay(int day, String paramName) throws IllegalArgumentException {
    268274        if (day < 0 || day > 6) {
    269275            throw new IllegalArgumentException(paramName + " is not a valid day (0-6). Given value is " + day);
     
    277283     * @throws IllegalArgumentException if the minute is invalid
    278284     */
    279     public static final void ensureValidMinute(int minute, String paramName) throws IllegalArgumentException {
     285    public static void ensureValidMinute(int minute, String paramName) throws IllegalArgumentException {
    280286        if (minute < 0 || minute > 24*60+1) {
    281287            throw new IllegalArgumentException(paramName + " is not a valid minute (0-1441). Given value is " + minute);
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/gui/OheDialogPanel.java

    r30722 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe.gui;
    23
     
    4748     * The Panel for editing the time-values.
    4849     *
    49      * @param plugin
    50      * @param key
    5150     * @param valuesToEdit
    52      *            can be a String or a Map<String, Integer> which contains
     51     *            can be a String or a Map&lt;String, Integer&gt; which contains
    5352     *            multiple values and their number of occurences
    5453     */
     
    7069                // TODO let the user choose which value he wants to edit (e.g. with a combobox)
    7170                int mostOccurences = 0;
    72                 for (String v : valuesMap.keySet())
     71                for (String v : valuesMap.keySet()) {
    7372                    if (valuesMap.get(v) > mostOccurences) {
    7473                        value = v;
    7574                        mostOccurences = valuesMap.get(v);
    7675                    }
     76                }
    7777            }
    7878        }
     
    118118
    119119    public String[] getChangedKeyValuePair() {
    120         return new String[] { oldkey, keyField.getText(), valueField.getText() };
     120        return new String[] {oldkey, keyField.getText(), valueField.getText()};
    121121    }
    122122
     
    135135                Main.warn(t);
    136136               
    137                 int tColumns[] = null;
     137                int[] tColumns = null;
    138138                String info = t.getMessage();
    139139
    140140                if (t instanceof ParseException) {
    141141                    ParseException parserExc = (ParseException) t;
    142                     tColumns = new int[] { parserExc.currentToken.beginColumn - 1, parserExc.currentToken.endColumn + 1 };
     142                    tColumns = new int[] {parserExc.currentToken.beginColumn - 1, parserExc.currentToken.endColumn + 1};
    143143                } else if (t instanceof SyntaxException) {
    144144                    SyntaxException syntaxError = (SyntaxException) t;
    145                     tColumns = new int[] { syntaxError.getStartColumn(), syntaxError.getEndColumn() };
     145                    tColumns = new int[] {syntaxError.getStartColumn(), syntaxError.getEndColumn()};
    146146                } else if (t instanceof TokenMgrError) {
    147147                    try {
     
    150150                        if (idx > -1) {
    151151                            int col = Integer.parseInt(info.substring(idx+"column ".length(), info.indexOf('.', idx)));
    152                             tColumns = new int[] { col - 1, col + 1 };
     152                            tColumns = new int[] {col - 1, col + 1};
    153153                        }
    154154                    } catch (IndexOutOfBoundsException e) {
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/gui/OheEditor.java

    r30737 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe.gui;
    23
     
    2021
    2122public class OheEditor extends JPanel implements MouseListener, MouseMotionListener {
     23    private final class OhePanel extends JPanel {
     24        @Override
     25        public void setSize(Dimension d) {
     26            super.setSize(d);
     27            repositionTimeRects();
     28        }
     29
     30        @Override
     31        public void paintComponent(Graphics g) {
     32            if (OheEditor.this.isEnabled()) {
     33                g.setColor(Color.WHITE);
     34                g.fillRect(0, 0, getWidth(), getHeight());
     35               
     36                // draw the time from 12PM to 00AM in a different color
     37                if (dialog.getHourMode() == ClockSystem.TWELVE_HOURS) {
     38                    g.setColor(new Color(255, 255, 218));
     39                    g.fillRect(0, getMinutePosition(12 * 60), getWidth(), getHeight() - getMinutePosition(12 * 60));
     40                }
     41
     42                // horizontal Lines
     43                for (int i = 0; i <= 24; ++i) {
     44                    if (i % 3 == 0) {
     45                        g.setColor(Color.BLACK);
     46                    } else {
     47                        g.setColor(Color.LIGHT_GRAY);
     48                    }
     49                    g.drawLine(0, getMinutePosition(i * 60), getWidth(), getMinutePosition(i * 60));
     50                }
     51
     52                // vertical Lines
     53                g.setColor(Color.BLACK);
     54                for (int i = 0; i <= 7; ++i) {
     55                    g.drawLine(getDayPosition(i), 0, getDayPosition(i), getHeight());
     56                }
     57
     58                // if a new Rect is dragged draw it
     59                if (day0 >= 0) {
     60                    Graphics2D g2D = (Graphics2D) g;
     61                    int day2 = Math.min(day0, day1);
     62                    int day3 = Math.max(day0, day1);
     63                    int minute2 = Math.min(minute0, minute1);
     64                    int minute3 = Math.max(minute0, minute1);
     65                    Rectangle bounds = getPanelBoundsForTimeinterval(day2, day3 + 1, minute2, minute3);
     66                    TimeRect.drawTimeRect(g2D, bounds, minute2 == minute3, false);
     67                }
     68            } else {
     69                g.setColor(Color.LIGHT_GRAY);
     70                g.fillRect(0, 0, getWidth(), getHeight());
     71            }
     72        }
     73    }
     74
    2275    final OheDialogPanel dialog;
    2376
    24     final private JScrollPane scrollPane;
     77    private final JScrollPane scrollPane;
    2578    final JPanel contentPanel;
    2679
    2780    ArrayList<TimeRect> timeRects;
    2881
    29     final private int dayAxisHeight = 20;
    30     final private int timeAxisWidth = 45;
     82    private final int dayAxisHeight = 20;
     83    private final int timeAxisWidth = 45;
    3184
    3285    public OheEditor(OheDialogPanel oheDialogPanel) {
     
    3487
    3588        // the MainPanel for showing the TimeRects
    36         contentPanel = new JPanel() {
    37             @Override
    38             public void setSize(Dimension d) {
    39                 super.setSize(d);
    40                 repositionTimeRects();
    41             }
    42 
    43             @Override
    44             public void paintComponent(Graphics g) {
    45                 if (OheEditor.this.isEnabled()) {
    46                     g.setColor(Color.WHITE);
    47                     g.fillRect(0, 0, getWidth(), getHeight());
    48                    
    49                     // draw the time from 12PM to 00AM in a different color
    50                     if (dialog.getHourMode() == ClockSystem.TWELVE_HOURS) {
    51                         g.setColor(new Color(255, 255, 218));
    52                         g.fillRect(0, getMinutePosition(12 * 60), getWidth(), getHeight() - getMinutePosition(12 * 60));
    53                     }
    54 
    55                     // horizontal Lines
    56                     for (int i = 0; i <= 24; ++i) {
    57                         if (i % 3 == 0) {
    58                             g.setColor(Color.BLACK);
    59                         } else {
    60                             g.setColor(Color.LIGHT_GRAY);
    61                         }
    62 
    63                         g.drawLine(0, getMinutePosition(i * 60), getWidth(), getMinutePosition(i * 60));
    64                     }
    65 
    66                     // vertical Lines
    67                     g.setColor(Color.BLACK);
    68                     for (int i = 0; i <= 7; ++i) {
    69                         g.drawLine(getDayPosition(i), 0, getDayPosition(i), getHeight());
    70                     }
    71 
    72                     // if a new Rect is dragged draw it
    73                     if (day0 >= 0) {
    74                         Graphics2D g2D = (Graphics2D) g;
    75 
    76                         int day2 = Math.min(day0, day1);
    77                         int day3 = Math.max(day0, day1);
    78                         int minute2 = Math.min(minute0, minute1);
    79                         int minute3 = Math.max(minute0, minute1);
    80                         Rectangle bounds = getPanelBoundsForTimeinterval(day2, day3 + 1, minute2, minute3);
    81 
    82                         TimeRect.drawTimeRect(g2D, bounds, minute2 == minute3, false);
    83                     }
    84                 } else {
    85                     g.setColor(Color.LIGHT_GRAY);
    86                     g.fillRect(0, 0, getWidth(), getHeight());
    87                 }
    88             }
    89         };
     89        contentPanel = new OhePanel();
    9090        contentPanel.addMouseListener(this);
    9191        contentPanel.addMouseMotionListener(this);
  • applications/editors/josm/plugins/OpeningHoursEditor/src/org/openstreetmap/josm/plugins/ohe/gui/TimeRect.java

    r30519 r32583  
     1// License: GPL. For details, see LICENSE file.
    12package org.openstreetmap.josm.plugins.ohe.gui;
    23
     
    3435
    3536    public static final boolean[][] transformDirections = new boolean[][] {
    36             { true, true, true, true }, // Drag
    37             { true, false, false, false }, // N
    38             { true, true, false, false }, // NE
    39             { false, true, false, false }, // E
    40             { false, true, true, false }, // SE
    41             { false, false, true, false }, // S
    42             { false, false, true, true }, // SW
    43             { false, false, false, true }, // W
    44             { true, false, false, true }, // NW
     37            {true, true, true, true}, // Drag
     38            {true, false, false, false}, // N
     39            {true, true, false, false}, // NE
     40            {false, true, false, false}, // E
     41            {false, true, true, false}, // SE
     42            {false, false, true, false}, // S
     43            {false, false, true, true}, // SW
     44            {false, false, false, true}, // W
     45            {true, false, false, true}, // NW
    4546    };
    4647
Note: See TracChangeset for help on using the changeset viewer.