Changeset 19624 in osm for applications/editors/josm


Ignore:
Timestamp:
2010-01-25T20:22:13+01:00 (15 years ago)
Author:
jttt
Message:

Code cleanup

Location:
applications/editors/josm/plugins/lakewalker
Files:
2 added
1 deleted
12 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/BooleanConfigurer.java

    r13497 r19624  
    1414 *
    1515 * You should have received a copy of the GNU Library General Public
    16  * License along with this library; if not, copies are available 
     16 * License along with this library; if not, copies are available
    1717 * at http://www.opensource.org.
    1818 */
    … …  
    2323 */
    2424public class BooleanConfigurer extends Configurer {
    25   private javax.swing.JCheckBox box;
     25        private javax.swing.JCheckBox box;
    2626
    27   public BooleanConfigurer() {
    28     this(false);
    29   }
    30  
    31   public BooleanConfigurer(boolean val) {
    32     this(null, "", val);
    33   }
    34  
    35   public BooleanConfigurer(String key, String name, Boolean val) {
    36     super(key, name, val);
    37   }
     27        public BooleanConfigurer() {
     28                this(false);
     29        }
    3830
    39   public BooleanConfigurer(String key, String name, boolean val) {
    40     super(key, name, val ? Boolean.TRUE : Boolean.FALSE);
    41   }
     31        public BooleanConfigurer(boolean val) {
     32                this(null, "", val);
     33        }
    4234
    43   public BooleanConfigurer(String key, String name) {
    44     this(key, name, Boolean.FALSE);
    45   }
     35        public BooleanConfigurer(String key, String name, Boolean val) {
     36                super(key, name, val);
     37        }
    4638
    47   public String getValueString() {
    48     return booleanValue().toString();
    49   }
     39        public BooleanConfigurer(String key, String name, boolean val) {
     40                super(key, name, val ? Boolean.TRUE : Boolean.FALSE);
     41        }
    5042
    51   public void setValue(Object o) {
    52     super.setValue(o);
    53     if (box != null
    54       && !o.equals(new Boolean(box.isSelected()))) {
    55       box.setSelected(booleanValue().booleanValue());
    56     }
    57   }
     43        public BooleanConfigurer(String key, String name) {
     44                this(key, name, Boolean.FALSE);
     45        }
    5846
    59   public void setValue(String s) {
    60     setValue(Boolean.valueOf(s));
    61   }
     47        @Override
     48        public String getValueString() {
     49                return booleanValue().toString();
     50        }
    6251
    63   public void setName(String s) {
    64     super.setName(s);
    65     if (box != null) {
    66       box.setText(s);
    67     }
    68   }
     52        @Override
     53        public void setValue(Object o) {
     54                super.setValue(o);
     55                if (box != null
     56                                && !o.equals(box.isSelected())) {
     57                        box.setSelected(booleanValue().booleanValue());
     58                }
     59        }
    6960
    70   public java.awt.Component getControls() {
    71     if (box == null) {
    72       box = new javax.swing.JCheckBox(getName());
    73       box.setSelected(booleanValue().booleanValue());
    74       box.addItemListener(new java.awt.event.ItemListener() {
    75         public void itemStateChanged(java.awt.event.ItemEvent e) {
    76           setValue(new Boolean(box.isSelected()));
    77         }
    78       });
    79     }
    80     return box;
    81   }
     61        @Override
     62        public void setValue(String s) {
     63                setValue(Boolean.valueOf(s));
     64        }
    8265
    83   public Boolean booleanValue() {
    84     return (Boolean) value;
    85   }
     66        @Override
     67        public void setName(String s) {
     68                super.setName(s);
     69                if (box != null) {
     70                        box.setText(s);
     71                }
     72        }
     73
     74        @Override
     75        public java.awt.Component getControls() {
     76                if (box == null) {
     77                        box = new javax.swing.JCheckBox(getName());
     78                        box.setSelected(booleanValue().booleanValue());
     79                        box.addItemListener(new java.awt.event.ItemListener() {
     80                                public void itemStateChanged(java.awt.event.ItemEvent e) {
     81                                        setValue(box.isSelected());
     82                                }
     83                        });
     84                }
     85                return box;
     86        }
     87
     88        public Boolean booleanValue() {
     89                return (Boolean) value;
     90        }
    8691}

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/DoubleConfigurer.java

    r18827 r19624  
    3636    }
    3737
    38     public void setValue(String s) {
     38    @Override
     39        public void setValue(String s) {
    3940        Double d = null;
    4041        try {
    … …  
    4849    }
    4950
    50     public void setValue(Object o) {
     51    @Override
     52        public void setValue(Object o) {
    5153        if (!noUpdate && nameField != null && o != null) {
    5254            nameField.setText(o.toString());
    … …  
    5557    }
    5658
    57     public String getValueString() {
     59    @Override
     60        public String getValueString() {
    5861        if (value == null || value.equals("")) {
    5962            return null;

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/IntConfigurer.java

    r18828 r19624  
    2323 */
    2424public class IntConfigurer extends StringConfigurer {
    25  
    26   public IntConfigurer() {
    27     super();
    28   }
    29  
    30   public IntConfigurer(String key, String name) {
    31     this(key, name, new Integer(0));
    32   }
    3325
    34   public IntConfigurer(String key, String name, Integer val) {
    35     super(key, name);
    36     if (val != null) {
    37       setValue(val);
    38     }
    39   }
     26        public IntConfigurer() {
     27                super();
     28        }
    4029
    41   public void setValue(String s) {
    42     Integer i = null;
    43     try {
    44       i = Integer.valueOf(s);
    45     }
    46     catch (NumberFormatException e) {
    47       i = null;
    48     }
    49     if (i != null) {
    50       setValue(i);
    51     }
    52   }
    53  
    54   public int getIntValue(int defaultValue) {
    55     if (getValue() instanceof Integer) {
    56       return ((Integer)getValue()).intValue();
    57     }
    58     else {
    59       return defaultValue;
    60     }
    61   }
     30        public IntConfigurer(String key, String name) {
     31                this(key, name, 0);
     32        }
    6233
    63   public void setValue(Object o) {
    64     if (!noUpdate && nameField != null && o != null) {
    65       nameField.setText(o.toString());
    66     }
    67     super.setValue(o);
    68   }
     34        public IntConfigurer(String key, String name, Integer val) {
     35                super(key, name);
     36                if (val != null) {
     37                        setValue(val);
     38                }
     39        }
    6940
    70   public String getValueString() {
    71     return value == null ? null : value.toString();
    72   }
     41        @Override
     42        public void setValue(String s) {
     43                Integer i = null;
     44                try {
     45                        i = Integer.valueOf(s);
     46                }
     47                catch (NumberFormatException e) {
     48                        i = null;
     49                }
     50                if (i != null) {
     51                        setValue(i);
     52                }
     53        }
     54
     55        public int getIntValue(int defaultValue) {
     56                if (getValue() instanceof Integer) {
     57                        return ((Integer)getValue()).intValue();
     58                }
     59                else {
     60                        return defaultValue;
     61                }
     62        }
     63
     64        @Override
     65        public void setValue(Object o) {
     66                if (!noUpdate && nameField != null && o != null) {
     67                        nameField.setText(o.toString());
     68                }
     69                super.setValue(o);
     70        }
     71
     72        @Override
     73        public String getValueString() {
     74                return value == null ? null : value.toString();
     75        }
    7376}

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/Lakewalker.java

    r16586 r19624  
    55import java.io.File;
    66import java.util.ArrayList;
    7 import java.util.Collection;
    8 import java.util.LinkedList;
    9 
    10 import org.openstreetmap.josm.command.Command;
    11 import org.openstreetmap.josm.data.osm.Way;
     7import java.util.List;
     8
    129import org.openstreetmap.josm.gui.progress.ProgressMonitor;
    1310
    1411public class Lakewalker {
    15     protected Collection<Command> commands = new LinkedList<Command>();
    16     protected Collection<Way> ways = new ArrayList<Way>();
    1712    protected boolean cancel;
    1813
    19     private int waylen;
    2014    private int maxnode;
    2115    private int threshold;
    22     private double epsilon;
    2316    private int resolution;
    2417    private int tilesize;
    … …  
    3528
    3629    public Lakewalker(int waylen, int maxnode, int threshold, double epsilon, int resolution, int tilesize, String startdir, String wmslayer, File workingdir){
    37         this.waylen = waylen;
    3830        this.maxnode = maxnode;
    3931        this.threshold = threshold;
    40         this.epsilon = epsilon;
    4132        this.resolution = resolution;
    4233        this.tilesize = tilesize;
    … …  
    383374        }
    384375
    385         ArrayList<double[]> seg_a = new ArrayList<double[]>();
    386         ArrayList<double[]> seg_b = new ArrayList<double[]>();
     376        List<double[]> seg_a;
     377        List<double[]> seg_b;
    387378
    388379        if(farthest_dist > epsilon){
    … …  
    440431     * @author Jason Reid
    441432     */
    442     private class LakewalkerBBox {
     433    private static class LakewalkerBBox {
    443434
    444435        private double top = 90;

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerAction.java

    r18880 r19624  
    7373    private void cleanupCache() {
    7474        final long maxCacheAge = System.currentTimeMillis()-Main.pref.getInteger(LakewalkerPreferences.PREF_MAXCACHEAGE, 100)*24*60*60*1000L;
    75         final long maxCacheSize = Main.pref.getInteger(LakewalkerPreferences.PREF_MAXCACHESIZE, 300)*1024*1024;
     75        final long maxCacheSize = Main.pref.getInteger(LakewalkerPreferences.PREF_MAXCACHESIZE, 300)*1024*1024L;
    7676
    7777        for (String wmsFolder : LakewalkerPreferences.WMSLAYERS) {
    … …  
    177177                        progressMonitor.createSubTaskMonitor(1, false));
    178178            } catch(LakewalkerException e){
    179                 System.out.println(e.getError());
     179                System.out.println(e.getMessage());
    180180            }
    181181
    … …  
    247247
    248248                } catch (Exception ex) {
     249                        ex.printStackTrace();
    249250                }
    250251

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerApp.java

    r16586 r19624  
    4040        try {
    4141            nodelist = lw.trace(lat,lon,leftlon,rightlon,toplat,botlat, NullProgressMonitor.INSTANCE);
     42
     43            System.out.println(nodelist.size()+" nodes generated");
     44
     45            nodelist = lw.vertexReduce(nodelist, dp);
     46
     47            System.out.println("After vertex reduction, "+nodelist.size()+" nodes remain.");
     48
     49            nodelist = lw.douglasPeucker(nodelist, dp, 0);
     50            System.out.println("After dp approximation, "+nodelist.size()+" nodes remain.");
     51
    4252        } catch(LakewalkerException e){
    43             System.out.println(e.getError());
     53            System.out.println(e.getMessage());
     54            e.printStackTrace();
    4455        }
    45 
    46         System.out.println(nodelist.size()+" nodes generated");
    47 
    48         nodelist = lw.vertexReduce(nodelist, dp);
    49 
    50         System.out.println("After vertex reduction, "+nodelist.size()+" nodes remain.");
    51 
    52         nodelist = lw.douglasPeucker(nodelist, dp, 0);
    53 
    54         System.out.println("After dp approximation, "+nodelist.size()+" nodes remain.");
    55 
    56 
    5756
    5857    }

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerException.java

    r15959 r19624  
    44
    55class LakewalkerException extends Exception {
    6     String error;
    7 
    86    public LakewalkerException(){
    9         super();
    10         this.error = tr("An unknown error has occurred");
     7        super(tr("An unknown error has occurred"));
    118    }
    129
    1310    public LakewalkerException(String err){
    14         super();
    15         this.error = err;
    16     }
    17 
    18     public String getError(){
    19       return this.error;
     11        super(err);
    2012    }
    2113}

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerPlugin.java

    r19446 r19624  
    1515 */
    1616public class LakewalkerPlugin extends Plugin {
    17   public LakewalkerPlugin(PluginInformation info) {
    18           super(info);
    19     MainMenu.add(Main.main.menu.toolsMenu, new LakewalkerAction(tr("Lake Walker")));
    20   }
     17        public LakewalkerPlugin(PluginInformation info) {
     18                super(info);
     19                MainMenu.add(Main.main.menu.toolsMenu, new LakewalkerAction(tr("Lake Walker")));
     20        }
    2121
    22   public PreferenceSetting getPreferenceSetting()
    23   {
    24     return new LakewalkerPreferences();
    25   }
     22        @Override
     23        public PreferenceSetting getPreferenceSetting()
     24        {
     25                return new LakewalkerPreferences();
     26        }
    2627
    2728}

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerPreferences.java

    r19294 r19624  
    33import static org.openstreetmap.josm.tools.I18n.marktr;
    44import static org.openstreetmap.josm.tools.I18n.tr;
     5
     6import java.awt.GridBagConstraints;
    57
    68import javax.swing.Box;
    … …  
    911
    1012import org.openstreetmap.josm.Main;
    11 import org.openstreetmap.josm.gui.preferences.PreferenceDialog;
    1213import org.openstreetmap.josm.gui.preferences.PreferenceSetting;
    1314import org.openstreetmap.josm.gui.preferences.PreferenceTabbedPane;
    … …  
    7475        landsatResConfig.setToolTipText(tr("Resolution of Landsat tiles, measured in pixels per degree. Default 4000."));
    7576        landsatSizeConfig.setToolTipText(tr("Size of one landsat tile, measured in pixels. Default 2000."));
    76         eastOffsetConfig.setToolTipText(tr("Offset all points in East direction (degrees). Default 0."));   
    77         northOffsetConfig.setToolTipText(tr("Offset all points in North direction (degrees). Default 0."));   
     77        eastOffsetConfig.setToolTipText(tr("Offset all points in East direction (degrees). Default 0."));
     78        northOffsetConfig.setToolTipText(tr("Offset all points in North direction (degrees). Default 0."));
    7879        startDirConfig.setToolTipText(tr("Direction to search for land. Default east."));
    7980        lakeTypeConfig.setToolTipText(tr("Tag ways as water, coastline, land or nothing. Default is water."));
    … …  
    105106    public void buildPreferences(JPanel prefPanel) {
    106107        GBC labelConstraints = GBC.std().insets(10,5,5,0);
    107         GBC dataConstraints = GBC.eol().insets(0,5,0,0).fill(GBC.HORIZONTAL);
     108        GBC dataConstraints = GBC.eol().insets(0,5,0,0).fill(GridBagConstraints.HORIZONTAL);
    108109
    109110        prefPanel.add(maxSegsLabel, labelConstraints);
    … …  
    118119        prefPanel.add(landsatResConfig.getControls(), dataConstraints);
    119120        prefPanel.add(landsatSizeLabel, labelConstraints);
    120         prefPanel.add(landsatSizeConfig.getControls(), dataConstraints);   
     121        prefPanel.add(landsatSizeConfig.getControls(), dataConstraints);
    121122        prefPanel.add(eastOffsetLabel, labelConstraints);
    122123        prefPanel.add(eastOffsetConfig.getControls(), dataConstraints);
    … …  
    124125        prefPanel.add(northOffsetConfig.getControls(), dataConstraints);
    125126        prefPanel.add(startDirLabel, labelConstraints);
    126         prefPanel.add(startDirConfig.getControls(), dataConstraints);   
     127        prefPanel.add(startDirConfig.getControls(), dataConstraints);
    127128        prefPanel.add(lakeTypeLabel, labelConstraints);
    128129        prefPanel.add(lakeTypeConfig.getControls(), dataConstraints);
    … …  
    136137        prefPanel.add(sourceConfig.getControls(), dataConstraints);
    137138
    138         prefPanel.add(Box.createVerticalGlue(), GBC.eol().fill(GBC.VERTICAL));
     139        prefPanel.add(Box.createVerticalGlue(), GBC.eol().fill(GridBagConstraints.VERTICAL));
    139140    }
    140141

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/LakewalkerWMS.java

    r16586 r19624  
    1313import java.text.NumberFormat;
    1414import java.util.HashMap;
     15import java.util.List;
    1516import java.util.Locale;
     17import java.util.Map;
    1618import java.util.Vector;
    1719
    … …  
    2729
    2830    // Vector to cache images in memory
    29     private Vector<BufferedImage> images = new Vector<BufferedImage>();
     31    private List<BufferedImage> images = new Vector<BufferedImage>();
    3032    // Hashmap to hold the mapping of cached images
    31     private HashMap<String,Integer> imageindex = new HashMap<String,Integer>();
     33    private Map<String,Integer> imageindex = new HashMap<String,Integer>();
    3234
    3335    private int resolution;
    … …  
    179181            image = this.getTile(x,y, progressMonitor);
    180182        } catch(LakewalkerException e){
    181             System.out.println(e.getError());
    182             throw new LakewalkerException(e.getMessage());
     183            System.out.println(e.getMessage());
     184            throw e;
    183185        }
    184186

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/StringConfigurer.java

    r13497 r19624  
    1414 *
    1515 * You should have received a copy of the GNU Library General Public
    16  * License along with this library; if not, copies are available 
     16 * License along with this library; if not, copies are available
    1717 * at http://www.opensource.org.
    1818 */
    … …  
    3434    this(null, "");
    3535  }
    36  
     36
    3737  public StringConfigurer(String key, String name) {
    3838    this(key, name, "");
    … …  
    4343  }
    4444
    45   public String getValueString() {
     45  @Override
     46public String getValueString() {
    4647    return (String) value;
    4748  }
    4849
    49   public void setValue(String s) {
     50  @Override
     51public void setValue(String s) {
    5052    if (!noUpdate && nameField != null) {
    5153      nameField.setText(s);
    … …  
    5759    nameField.setToolTipText(s);
    5860  }
    59  
    60   public java.awt.Component getControls() {
     61
     62  @Override
     63public java.awt.Component getControls() {
    6164    if (p == null) {
    6265      p = new JPanel();
    … …  
    6972      p.add(nameField);
    7073      nameField.addKeyListener(new java.awt.event.KeyAdapter() {
    71         public void keyReleased(java.awt.event.KeyEvent evt) {
     74        @Override
     75                public void keyReleased(java.awt.event.KeyEvent evt) {
    7276          noUpdate = true;
    7377          setValue(nameField.getText());

  • applications/editors/josm/plugins/lakewalker/src/org/openstreetmap/josm/plugins/lakewalker/StringEnumConfigurer.java

    r13497 r19624  
    3333import java.awt.event.ActionEvent;
    3434import java.awt.event.ActionListener;
     35
    3536import javax.swing.Box;
    36 import javax.swing.DefaultComboBoxModel;
    3737import javax.swing.JComboBox;
    3838import javax.swing.JLabel;
    … …  
    6363        tooltipText = s;
    6464    }
    65     public Component getControls() {
     65    @Override
     66        public Component getControls() {
    6667        if (panel == null) {
    6768            panel = Box.createHorizontalBox();
    … …  
    8283        return panel;
    8384    }
    84     public void setValue(Object o) {
     85
     86    @Override
     87        public void setValue(Object o) {
    8588        if(o == null)
    86             o = new Integer(0);
     89            o = 0;
    8790        super.setValue(o);
    8891        if(!noUpdate && box != null)
    … …  
    9093    }
    9194
    92     public void setValue(String s) {
     95    @Override
     96        public void setValue(String s) {
    9397        Integer n = 0;
    9498        for (int i = 0; i < transValues.length; ++i)
    … …  
    102106    }
    103107
    104     public String getValueString() {
     108    @Override
     109        public String getValueString() {
    105110        return validValues[(Integer)value];
    106111    }
Note: See TracChangeset for help on using the changeset viewer.