Ignore:
Timestamp:
2014-09-23T12:02:21+02:00 (11 years ago)
Author:
donvip
Message:

[josm_commandline] code cleanup

Location:
applications/editors/josm/plugins/CommandLine/src/CommandLine
Files:
9 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/Command.java

    r30669 r30671  
    2828    public boolean asynchronous;
    2929
    30     public Command () { parameters = new ArrayList<Parameter>(); optParameters = new ArrayList<Parameter>(); currentParameterNum = 0; tracks = false; asynchronous = false; icon = ""; }
     30    public Command () {
     31        parameters = new ArrayList<>();
     32        optParameters = new ArrayList<>();
     33        currentParameterNum = 0;
     34        tracks = false;
     35        asynchronous = false;
     36        icon = "";
     37    }
    3138
    3239    @SuppressWarnings("unchecked")
    … …  
    5360                        multiValue.add((OsmPrimitive)obj);
    5461                        return true;
    55                     }
    56                     else {
    57                         if (nextParameter() && multiValue.size() > 0) {
    58                             return loadObject(obj);
    59                         }
    60                     }
    61                 }
    62                 else {
    63                     if (nextParameter()) {
     62                    } else if (nextParameter() && multiValue.size() > 0) {
    6463                        return loadObject(obj);
    6564                    }
     65                }
     66                else if (nextParameter()) {
     67                    return loadObject(obj);
    6668                }
    6769            }
    … …  
    167169
    168170    public Collection<OsmPrimitive> getDepsObjects() {
    169         ArrayList<OsmPrimitive> depsObjects = new ArrayList<OsmPrimitive>();
     171        ArrayList<OsmPrimitive> depsObjects = new ArrayList<>();
    170172        for (Parameter parameter : parameters) {
    171173            if (!parameter.isOsm())
    … …  
    184186
    185187    public Collection<OsmPrimitive> getDepsObjects(Collection<OsmPrimitive> currentObjects, OsmPrimitive primitive) {
    186         ArrayList<OsmPrimitive> depsObjects = new ArrayList<OsmPrimitive>();
     188        ArrayList<OsmPrimitive> depsObjects = new ArrayList<>();
    187189        if (!currentObjects.contains(primitive)) {
    188190            if (primitive instanceof Way) {

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/CommandLine.java

    r30670 r30671  
    314314                Layer layer = Main.map.mapView.getActiveLayer();
    315315                if (layer != null) {
    316                     if (layer instanceof ImageryLayer) {
    317                     }
    318                     else {
     316                    if (!(layer instanceof ImageryLayer)) {
    319317                        List<ImageryLayer> imageryLayers = Main.map.mapView.getLayersOfType(ImageryLayer.class);
    320318                        if (imageryLayers.size() == 1) {
    … …  
    501499                Collection<OsmPrimitive> pObjects;
    502500                osmWriter.header();
    503                 Collection<OsmPrimitive> contents = new ArrayList<OsmPrimitive>();
     501                Collection<OsmPrimitive> contents = new ArrayList<>();
    504502                for (OsmPrimitive primitive : refObjects) {
    505503                    contents.add(primitive);
    … …  
    518516                    if (!parameter.isOsm())
    519517                        continue;
    520                     contents = new ArrayList<OsmPrimitive>();
     518                    contents = new ArrayList<>();
    521519                    osmWriter.header();
    522520                    pObjects = parameter.getParameterObjects();

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/GpxFilter.java

    r30669 r30671  
    3636        Collection<WayPoint> currentSegment;
    3737        for (GpxTrack track : data.tracks) {
    38             currentTrack = new ArrayList<Collection<WayPoint>>();
     38            currentTrack = new ArrayList<>();
    3939            for (GpxTrackSegment segment : track.getSegments()) {
    40                 currentSegment = new ArrayList<WayPoint>();
     40                currentSegment = new ArrayList<>();
    4141                for (WayPoint wp : segment.getWayPoints()) {
    4242                    if ( bbox.bounds(wp.getCoor()) ) {
    … …  
    4545                        if (currentSegment.size() > 1) {
    4646                            currentTrack.add(currentSegment);
    47                             currentSegment = new ArrayList<WayPoint>();
     47                            currentSegment = new ArrayList<>();
    4848                        }
    4949                    }
    … …  
    5151                if (currentSegment.size() > 1) {
    5252                    currentTrack.add(currentSegment);
    53                     currentSegment = new ArrayList<WayPoint>();
     53                    currentSegment = new ArrayList<>();
    5454                }
    5555            }

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/History.java

    r25038 r30671  
    11/*
    22 *      History.java
    3  *     
     3 *
    44 *      Copyright 2010 Hind <foxhind@gmail.com>
    5  *     
     5 *
    66 */
    7  
     7
    88package CommandLine;
    99
    … …  
    1111
    1212public class History {
    13         private LinkedList<String> historyList;
    14         private int maxLen;
    15         private int num;
    16        
    17         public History(int len) {
    18                 num = 0;
    19                 maxLen = len;
    20                 historyList = new LinkedList<String>();
    21         }
    22        
    23         public void addItem(String item) {
    24                 if (!item.equals("")) {
    25                         String prevItem = historyList.peekFirst();
    26                         if (prevItem == null) {
    27                                 historyList.addFirst(item);
    28                         }
    29                         else {
    30                                 if (!prevItem.equalsIgnoreCase(item))
    31                                         historyList.addFirst(item);
    32                         }
    33                         if (historyList.size() > maxLen) {
    34                                 historyList.removeLast();
    35                         }
    36                 }
    37                 num = -1;
    38         }
    39        
    40         public String getPrevItem() {
    41                 num += 1;
    42                 if (num >= historyList.size()) {
    43                         num = historyList.size() - 1;
    44                 }
    45                 if (num < 0) {
    46                         num = -1;
    47                         return "";
    48                 }
    49                 return historyList.get(num);
    50         }
    51        
    52         public String getLastItem() {
    53                 if (historyList.size() > 0)
    54                         return historyList.get(0);
    55                 return "";
    56         }
     13    private final LinkedList<String> historyList;
     14    private final int maxLen;
     15    private int num;
    5716
    58         public String getNextItem() {
    59                 num -= 1;
    60                 if (num < 0) {
    61                         num = -1;
    62                         return "";
    63                 }
    64                 return historyList.get(num);
    65         }
     17    public History(int len) {
     18        num = 0;
     19        maxLen = len;
     20        historyList = new LinkedList<>();
     21    }
     22
     23    public void addItem(String item) {
     24        if (!item.equals("")) {
     25            String prevItem = historyList.peekFirst();
     26            if (prevItem == null) {
     27                historyList.addFirst(item);
     28            }
     29            else {
     30                if (!prevItem.equalsIgnoreCase(item))
     31                    historyList.addFirst(item);
     32            }
     33            if (historyList.size() > maxLen) {
     34                historyList.removeLast();
     35            }
     36        }
     37        num = -1;
     38    }
     39
     40    public String getPrevItem() {
     41        num += 1;
     42        if (num >= historyList.size()) {
     43            num = historyList.size() - 1;
     44        }
     45        if (num < 0) {
     46            num = -1;
     47            return "";
     48        }
     49        return historyList.get(num);
     50    }
     51
     52    public String getLastItem() {
     53        if (historyList.size() > 0)
     54            return historyList.get(0);
     55        return "";
     56    }
     57
     58    public String getNextItem() {
     59        num -= 1;
     60        if (num < 0) {
     61            num = -1;
     62            return "";
     63        }
     64        return historyList.get(num);
     65    }
    6666}
    6767

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/Loader.java

    r30669 r30671  
    3131        dirToScan = dir;
    3232        currentTag = "";
    33         loadingCommands = new ArrayList<Command>();
     33        loadingCommands = new ArrayList<>();
    3434    }
    3535

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/OsmToCmd.java

    r30669 r30671  
    5454    private final CommandLine parentPlugin;
    5555    private final DataSet targetDataSet;
    56     private final LinkedList<Command> cmds = new LinkedList<Command>();
     56    private final LinkedList<Command> cmds = new LinkedList<>();
    5757    private final HashMap<PrimitiveId, OsmPrimitive> externalIdMap; // Maps external ids to internal primitives
    5858
    … …  
    6060        this.parentPlugin = parentPlugin;
    6161        this.targetDataSet = targetDataSet;
    62         externalIdMap = new HashMap<PrimitiveId, OsmPrimitive>();
     62        externalIdMap = new HashMap<>();
    6363    }
    6464
    … …  
    9999        private OsmPrimitive currentPrimitive;
    100100        //private long currentExternalId;
    101         private final List<Node> currentWayNodes = new ArrayList<Node>();
    102         private final List<RelationMember> currentRelationMembers = new ArrayList<RelationMember>();
     101        private final List<Node> currentWayNodes = new ArrayList<>();
     102        private final List<RelationMember> currentRelationMembers = new ArrayList<>();
    103103
    104104        @Override
    … …  
    108108                    if (atts == null) {
    109109                        throwException(tr("Missing mandatory attribute ''{0}'' of XML element {1}.", "version", "osm"));
     110                        return;
    110111                    }
    111112                    String v = atts.getValue("version");
    112113                    if (v == null) {
    113114                        throwException(tr("Missing mandatory attribute ''{0}''.", "version"));
     115                        return;
    114116                    }
    115117                    if ( !(v.equals("0.6")) ) {
    116118                        throwException(tr("Unsupported version: {0}", v));
     119                        return;
    117120                    }
    118121
    … …  
    224227                    if (key == null || value == null) {
    225228                        throwException(tr("Missing key or value attribute in tag."));
     229                        return;
    226230                    }
    227231                    currentPrimitive.put(key.intern(), value.intern());

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/Parameter.java

    r29505 r30671  
    11/*
    22 *      Parameter.java
    3  *     
     3 *
    44 *      Copyright 2011 Hind <foxhind@gmail.com>
    5  *     
     5 *
    66 */
    7  
     7
    88package CommandLine;
    99
    … …  
    1616
    1717public class Parameter {
    18         public boolean required;
    19         public Type type;
    20         public String name;
    21         public String description;
    22         private Object value;
    23         private ArrayList<OsmPrimitive> valueList;
    24         protected float maxVal;
    25         protected float minVal;
    26         protected int maxInstances;
    27        
    28         public Parameter () { required = false; maxInstances = 1; maxVal = 0; minVal = 0; value = ""; valueList = new ArrayList<OsmPrimitive>(); }
    29         public String getValue() {
    30                 String out = "";
    31                 switch (type) {
    32                         case POINT:
    33                                 out = (String)value;
    34                                 break;
    35                         case LENGTH:
    36                                 out = String.valueOf(value);
    37                                 break;
    38                         case NATURAL:
    39                                 out = String.valueOf(value);
    40                                 break;
    41                         case STRING:
    42                                 out = String.valueOf(value);
    43                                 break;
    44                         case RELAY:
    45                                 out = String.valueOf(((Relay)value).getValue());
    46                                 break;
    47                         case NODE:
    48                                 out = String.valueOf(valueList.size()) + " " + tr("nodes");
    49                                 break;
    50                         case WAY:
    51                                 out = String.valueOf(valueList.size()) + " " + tr("ways");
    52                                 break;
    53                         case RELATION:
    54                                 out = String.valueOf(valueList.size()) + " " + tr("relations");
    55                                 break;
    56                         case ANY:
    57                                 out = String.valueOf(valueList.size()) + " " + tr("OSM objects");
    58                                 break;
    59                         case USERNAME:
    60                                 out = String.valueOf(value);
    61                                 break;
    62                         case IMAGERYURL:
    63                                 out = String.valueOf(value);
    64                                 break;
    65                         case IMAGERYOFFSET:
    66                                 out = String.valueOf(value);
    67                                 break;
    68                 }
    69                 return out;
    70         }
     18    public boolean required;
     19    public Type type;
     20    public String name;
     21    public String description;
     22    private Object value;
     23    private final ArrayList<OsmPrimitive> valueList;
     24    protected float maxVal;
     25    protected float minVal;
     26    protected int maxInstances;
    7127
    72         public Object getRawValue() {
    73                 return value;
    74         }
     28    public Parameter() {
     29        required = false;
     30        maxInstances = 1;
     31        maxVal = 0;
     32        minVal = 0;
     33        value = "";
     34        valueList = new ArrayList<>();
     35    }
    7536
    76         public ArrayList<OsmPrimitive> getValueList() {
    77                 return valueList;
    78         }
     37    public String getValue() {
     38        String out = "";
     39        switch (type) {
     40        case POINT:
     41            out = (String)value;
     42            break;
     43        case LENGTH:
     44            out = String.valueOf(value);
     45            break;
     46        case NATURAL:
     47            out = String.valueOf(value);
     48            break;
     49        case STRING:
     50            out = String.valueOf(value);
     51            break;
     52        case RELAY:
     53            out = String.valueOf(((Relay)value).getValue());
     54            break;
     55        case NODE:
     56            out = String.valueOf(valueList.size()) + " " + tr("nodes");
     57            break;
     58        case WAY:
     59            out = String.valueOf(valueList.size()) + " " + tr("ways");
     60            break;
     61        case RELATION:
     62            out = String.valueOf(valueList.size()) + " " + tr("relations");
     63            break;
     64        case ANY:
     65            out = String.valueOf(valueList.size()) + " " + tr("OSM objects");
     66            break;
     67        case USERNAME:
     68            out = String.valueOf(value);
     69            break;
     70        case IMAGERYURL:
     71            out = String.valueOf(value);
     72            break;
     73        case IMAGERYOFFSET:
     74            out = String.valueOf(value);
     75            break;
     76        }
     77        return out;
     78    }
    7979
    80         public void setValue(Object obj) {
    81                 if (type == Type.RELAY && obj instanceof String && value instanceof Relay) {
    82                         ((Relay)value).setValue((String)obj);
    83                 }
    84                 else
    85                         value = obj;
    86         }
     80    public Object getRawValue() {
     81        return value;
     82    }
    8783
    88         public Collection<OsmPrimitive> getParameterObjects() {
    89                 ArrayList<OsmPrimitive> pObjects = new ArrayList<OsmPrimitive>();
    90                 if (isOsm()) {
    91                         if (maxInstances == 1) {
    92                                 pObjects.add((OsmPrimitive)value);
    93                         }
    94                         else {
    95                                 return valueList;
    96                         }
    97                 }
    98                 return pObjects;
    99         }
     84    public ArrayList<OsmPrimitive> getValueList() {
     85        return valueList;
     86    }
    10087
    101         public boolean isOsm() {
    102                 return type == Type.NODE || type == Type.WAY || type == Type.RELATION || type == Type.ANY;
    103         }
     88    public void setValue(Object obj) {
     89        if (type == Type.RELAY && obj instanceof String && value instanceof Relay) {
     90            ((Relay)value).setValue((String)obj);
     91        }
     92        else
     93            value = obj;
     94    }
     95
     96    public Collection<OsmPrimitive> getParameterObjects() {
     97        ArrayList<OsmPrimitive> pObjects = new ArrayList<>();
     98        if (isOsm()) {
     99            if (maxInstances == 1) {
     100                pObjects.add((OsmPrimitive)value);
     101            }
     102            else {
     103                return valueList;
     104            }
     105        }
     106        return pObjects;
     107    }
     108
     109    public boolean isOsm() {
     110        return type == Type.NODE || type == Type.WAY || type == Type.RELATION || type == Type.ANY;
     111    }
    104112}

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/PointAction.java

    r30669 r30671  
    4747        currentCursor = cursorCrosshair;
    4848        nearestNode = null;
    49         pointList = new ArrayList<String>();
     49        pointList = new ArrayList<>();
    5050    }
    5151

  • applications/editors/josm/plugins/CommandLine/src/CommandLine/Relay.java

    r25038 r30671  
    11/*
    22 *      Relay.java
    3  *     
     3 *
    44 *      Copyright 2010 Hind <foxhind@gmail.com>
    5  *     
     5 *
    66 */
    77
    … …  
    1111
    1212public class Relay {
    13   static String marker = "\u0332";
    14   private String optionsString;
    15   private HashMap<String, String> options;
    16   private String value;
     13    static String marker = "\u0332";
     14    private String optionsString;
     15    private final HashMap<String, String> options;
     16    private String value;
    1717
    18   public Relay() {
    19     optionsString = "";
    20     value = "";
    21     options = new HashMap<String, String>();
    22   }
     18    public Relay() {
     19        optionsString = "";
     20        value = "";
     21        options = new HashMap<>();
     22    }
    2323
    24   public void setValue(String value) {
    25     if (options.containsKey(value))
    26       this.value = options.get(value);
    27     else if (options.containsValue(value))
    28       this.value = value;
    29   }
     24    public void setValue(String value) {
     25        if (options.containsKey(value))
     26            this.value = options.get(value);
     27        else if (options.containsValue(value))
     28            this.value = value;
     29    }
    3030
    31   public void addValue(String value) {
    32     String letter = null;
    33     if (!(options.containsValue(value))) {
    34       int i = 0;
    35       for (; i < value.length() ; i++) {
    36         letter = value.substring(i, i + 1).toLowerCase();
    37         if (!options.containsKey(letter))
    38           break;
    39       }
    40       if (i == value.length()) {
    41         letter = String.valueOf(System.currentTimeMillis());
    42         optionsString = optionsString + (optionsString.length() == 0 ? "" : ", ") + value;
    43       }
    44       else
    45         optionsString = optionsString + (optionsString.length() == 0 ? "" : ", ") + value.substring(0, i) + marker + value.substring(i);
    46       options.put(letter, value);
     31    public void addValue(String value) {
     32        String letter = null;
     33        if (!(options.containsValue(value))) {
     34            int i = 0;
     35            for (; i < value.length() ; i++) {
     36                letter = value.substring(i, i + 1).toLowerCase();
     37                if (!options.containsKey(letter))
     38                    break;
     39            }
     40            if (i == value.length()) {
     41                letter = String.valueOf(System.currentTimeMillis());
     42                optionsString = optionsString + (optionsString.length() == 0 ? "" : ", ") + value;
     43            }
     44            else
     45                optionsString = optionsString + (optionsString.length() == 0 ? "" : ", ") + value.substring(0, i) + marker + value.substring(i);
     46            options.put(letter, value);
     47        }
     48        this.value = value;
    4749    }
    48     this.value = value;
    49   }
    5050
    51   public String getValue() {
    52     return value;
    53   }
     51    public String getValue() {
     52        return value;
     53    }
    5454
    55   public boolean isCorrectValue(String value) {
    56     return options.containsValue(value) || options.containsKey(value.toLowerCase());
    57   }
     55    public boolean isCorrectValue(String value) {
     56        return options.containsValue(value) || options.containsKey(value.toLowerCase());
     57    }
    5858
    59   public String getOptionsString() {
    60     return optionsString;
    61   }
     59    public String getOptionsString() {
     60        return optionsString;
     61    }
    6262}
Note: See TracChangeset for help on using the changeset viewer.