Ignore:
Timestamp:
2010-11-25T09:45:38+01:00 (11 years ago)
Author:
bastiK
Message:

adapt coding style (to some degree); there shouldn't be any semantic changes in this commit

Location:
trunk/src/org/openstreetmap/josm/data/validation/util
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/validation/util/AgregatePrimitivesVisitor.java

    r3669 r3671  
    1919 * @author frsantos
    2020 */
    21 public class AgregatePrimitivesVisitor extends AbstractVisitor
    22 {
     21public class AgregatePrimitivesVisitor extends AbstractVisitor {
    2322    /** Aggregated data */
    2423    final Collection<OsmPrimitive> aggregatedData = new HashSet<OsmPrimitive>();
     
    2928     * @return The aggregated primitives
    3029     */
    31     public Collection<OsmPrimitive> visit(Collection<OsmPrimitive> data)
    32     {
    33         for (OsmPrimitive osm : data)
    34         {
     30    public Collection<OsmPrimitive> visit(Collection<OsmPrimitive> data) {
     31        for (OsmPrimitive osm : data) {
    3532            osm.visit(this);
    3633        }
     
    3936    }
    4037
    41     public void visit(Node n)
    42     {
    43         if(!aggregatedData.contains(n))
     38    @Override
     39    public void visit(Node n) {
     40        if (!aggregatedData.contains(n)) {
    4441            aggregatedData.add(n);
    45     }
    46 
    47     public void visit(Way w)
    48     {
    49         if(!aggregatedData.contains(w))
    50         {
    51             aggregatedData.add(w);
    52             for (Node n : w.getNodes())
    53                 visit(n);
    5442        }
    5543    }
    5644
     45    @Override
     46    public void visit(Way w) {
     47        if (!aggregatedData.contains(w)) {
     48            aggregatedData.add(w);
     49            for (Node n : w.getNodes()) {
     50                visit(n);
     51            }
     52        }
     53    }
     54
     55    @Override
    5756    public void visit(Relation r) {
    5857        if (!aggregatedData.contains(r)) {
  • trunk/src/org/openstreetmap/josm/data/validation/util/MultipleNameVisitor.java

    r3669 r3671  
    3131     * @param data The collection of primitives
    3232     */
    33     public void visit(Collection<? extends OsmPrimitive> data)
    34     {
     33    public void visit(Collection<? extends OsmPrimitive> data) {
    3534        String multipleName = null;
    3635        String multiplePluralClassname = null;
     
    4039
    4140        multipleClassname = null;
    42         for (OsmPrimitive osm : data)
    43         {
     41        for (OsmPrimitive osm : data) {
    4442            String name = osm.get("name");
    45             if(name == null) name = osm.get("ref");
    46             if(!initializedname)
    47             {
     43            if (name == null) {
     44                name = osm.get("ref");
     45            }
     46            if (!initializedname) {
    4847                multipleName = name; initializedname = true;
    49             }
    50             else if(multipleName != null && (name == null  || !name.equals(multipleName)))
    51             {
     48            } else if (multipleName != null && (name == null  || !name.equals(multipleName))) {
    5249                multipleName = null;
    5350            }
    5451
    55             if(firstName == null && name != null)
     52            if (firstName == null && name != null) {
    5653                firstName = name;
     54            }
    5755            osm.visit(this);
    58             if (multipleClassname == null)
    59             {
     56            if (multipleClassname == null) {
    6057                multipleClassname = className;
    6158                multiplePluralClassname = classNamePlural;
    62             }
    63             else if (!multipleClassname.equals(className))
    64             {
     59            } else if (!multipleClassname.equals(className)) {
    6560                multipleClassname = "object";
    6661                multiplePluralClassname = trn("object", "objects", 2);
     
    6863        }
    6964
    70         if( size == 1 )
     65        if (size == 1) {
    7166            displayName = name;
    72         else if(multipleName != null)
     67        } else if (multipleName != null) {
    7368            displayName = size + " " + trn(multipleClassname, multiplePluralClassname, size) + ": " + multipleName;
    74         else if(firstName != null)
     69        } else if (firstName != null) {
    7570            displayName = size + " " + trn(multipleClassname, multiplePluralClassname, size) + ": " + tr("{0}, ...", firstName);
    76         else
     71        } else {
    7772            displayName = size + " " + trn(multipleClassname, multiplePluralClassname, size);
     73        }
    7874    }
    7975
    8076    @Override
    81     public JLabel toLabel()
    82     {
     77    public JLabel toLabel() {
    8378        return new JLabel(getText(), getIcon(), JLabel.HORIZONTAL);
    8479    }
     
    8883     * @return the name of the items
    8984     */
    90     public String getText()
    91     {
     85    public String getText() {
    9286        return displayName;
    9387    }
     
    9791     * @return the icon of the items
    9892     */
    99     public Icon getIcon()
    100     {
    101         if( size == 1 )
     93    public Icon getIcon() {
     94        if (size == 1)
    10295            return icon;
    10396        else
  • trunk/src/org/openstreetmap/josm/data/validation/util/NameVisitor.java

    r3669 r3671  
    4343     * is displayed.
    4444     */
     45    @Override
    4546    public void visit(Node n) {
    4647        name = n.getDisplayName(DefaultNameFormatter.getInstance());
     
    5556     * is displayed with x being the number of nodes in the way.
    5657     */
     58    @Override
    5759    public void visit(Way w) {
    5860        name = w.getDisplayName(DefaultNameFormatter.getInstance());
     
    6567    /**
    6668     */
     69    @Override
    6770    public void visit(Relation e) {
    6871        name = e.getDisplayName(DefaultNameFormatter.getInstance());
     
    7780    }
    7881
    79 
    8082    private void addId(OsmPrimitive osm) {
    81         if (Main.pref.getBoolean("osm-primitives.showid"))
     83        if (Main.pref.getBoolean("osm-primitives.showid")) {
    8284            name += tr(" [id: {0}]", osm.getId());
     85        }
    8386    }
    8487}
  • trunk/src/org/openstreetmap/josm/data/validation/util/ValUtil.java

    r3670 r3671  
    3838     * @return A list with all the cells the way starts or ends
    3939     */
    40     public static List<List<Way>> getWaysInCell(Way w, Map<Point2D,List<Way>> cellWays)
    41     {
     40    public static List<List<Way>> getWaysInCell(Way w, Map<Point2D,List<Way>> cellWays) {
    4241        if (w.getNodesCount() == 0)
    4342            return Collections.emptyList();
     
    5958        cell = new Point2D.Double(x0, y0);
    6059        cellNodes.add(cell);
    61         List<Way> ways = cellWays.get( cell );
    62         if( ways == null )
    63         {
     60        List<Way> ways = cellWays.get(cell);
     61        if (ways == null) {
    6462            ways = new ArrayList<Way>();
    6563            cellWays.put(cell, ways);
     
    6967        // End of the way
    7068        cell = new Point2D.Double(x1, y1);
    71         if( !cellNodes.contains(cell) )
    72         {
     69        if (!cellNodes.contains(cell)) {
    7370            cellNodes.add(cell);
    7471            ways = cellWays.get( cell );
    75             if( ways == null )
    76             {
     72            if (ways == null) {
    7773                ways = new ArrayList<Way>();
    7874                cellWays.put(cell, ways);
     
    8278
    8379        // Then floor coordinates, in case the way is in the border of the cell.
    84         x0 = (long)Math.floor(n1.getEastNorth().east()  * OsmValidator.griddetail);
    85         y0 = (long)Math.floor(n1.getEastNorth().north() * OsmValidator.griddetail);
    86         x1 = (long)Math.floor(n2.getEastNorth().east()  * OsmValidator.griddetail);
    87         y1 = (long)Math.floor(n2.getEastNorth().north() * OsmValidator.griddetail);
     80        x0 = (long) Math.floor(n1.getEastNorth().east()  * OsmValidator.griddetail);
     81        y0 = (long) Math.floor(n1.getEastNorth().north() * OsmValidator.griddetail);
     82        x1 = (long) Math.floor(n2.getEastNorth().east()  * OsmValidator.griddetail);
     83        y1 = (long) Math.floor(n2.getEastNorth().north() * OsmValidator.griddetail);
    8884
    8985        // Start of the way
    9086        cell = new Point2D.Double(x0, y0);
    91         if( !cellNodes.contains(cell) )
    92         {
     87        if (!cellNodes.contains(cell)) {
    9388            cellNodes.add(cell);
    94             ways = cellWays.get( cell );
    95             if( ways == null )
    96             {
     89            ways = cellWays.get(cell);
     90            if (ways == null) {
    9791                ways = new ArrayList<Way>();
    9892                cellWays.put(cell, ways);
     
    10397        // End of the way
    10498        cell = new Point2D.Double(x1, y1);
    105         if( !cellNodes.contains(cell) )
    106         {
     99        if (!cellNodes.contains(cell)) {
    107100            cellNodes.add(cell);
    108             ways = cellWays.get( cell );
    109             if( ways == null )
    110             {
     101            ways = cellWays.get(cell);
     102            if (ways == null) {
    111103                ways = new ArrayList<Way>();
    112104                cellWays.put(cell, ways);
     
    114106            cells.add(ways);
    115107        }
    116 
    117108        return cells;
    118109    }
     
    128119     * @return A list with the coordinates of all cells
    129120     */
    130     public static List<Point2D> getSegmentCells(Node n1, Node n2, double gridDetail)
    131     {
     121    public static List<Point2D> getSegmentCells(Node n1, Node n2, double gridDetail) {
    132122        List<Point2D> cells = new ArrayList<Point2D>();
    133123        double x0 = n1.getEastNorth().east() * gridDetail;
     
    136126        double y1 = n2.getEastNorth().north() * gridDetail + 1;
    137127
    138         if( x0 > x1 )
    139         {
     128        if (x0 > x1) {
    140129            // Move to 1st-4th cuadrants
    141130            double aux;
     
    147136        double dy  = y1 - y0;
    148137        long stepY = y0 <= y1 ? 1 : -1;
    149         long gridX0 = (long)Math.floor(x0);
    150         long gridX1 = (long)Math.floor(x1);
    151         long gridY0 = (long)Math.floor(y0);
    152         long gridY1 = (long)Math.floor(y1);
     138        long gridX0 = (long) Math.floor(x0);
     139        long gridX1 = (long) Math.floor(x1);
     140        long gridY0 = (long) Math.floor(y0);
     141        long gridY1 = (long) Math.floor(y1);
    153142
    154143        long maxSteps = (gridX1 - gridX0) + Math.abs(gridY1 - gridY0) + 1;
    155         while( (gridX0 <= gridX1 && (gridY0 - gridY1)*stepY <= 0) && maxSteps-- > 0)
    156         {
    157             cells.add( new Point2D.Double(gridX0, gridY0) );
     144        while ((gridX0 <= gridX1 && (gridY0 - gridY1)*stepY <= 0) && maxSteps-- > 0) {
     145            cells.add( new Point2D.Double(gridX0, gridY0));
    158146
    159147            // Is the cross between the segment and next vertical line nearer than the cross with next horizontal line?
     
    166154            double distY = Math.pow(scanX - x0, 2) + Math.pow(gridY0 + stepY - y0, 2);
    167155
    168             if( distX < distY)
     156            if (distX < distY) {
    169157                gridX0 += 1;
    170             else
     158            } else {
    171159                gridY0 += stepY;
     160            }
    172161        }
    173 
    174162        return cells;
    175163    }
Note: See TracChangeset for help on using the changeset viewer.