Changeset 30783 in osm for applications/editors/josm


Ignore:
Timestamp:
2014-10-29T22:12:34+01:00 (10 years ago)
Author:
donvip
Message:

[josm-merge_overlap] code cleanup

File:
1 edited

Legend:

Unmodified
Added
Removed
  • applications/editors/josm/plugins/merge-overlap/src/mergeoverlap/MergeOverlapAction.java

    r30782 r30783  
    186186            combine.add(way);
    187187            for (Way opositWay : ways) {
    188                 if (way != opositWay
    189                         && way.getNodesCount() == opositWay.getNodesCount()) {
     188                if (way != opositWay && way.getNodesCount() == opositWay.getNodesCount()) {
    190189                    boolean equals1 = true;
    191190                    for (int i = 0; i < way.getNodesCount(); i++) {
     
    197196                    boolean equals2 = true;
    198197                    for (int i = 0; i < way.getNodesCount(); i++) {
    199                         if (way.getNode(i) != opositWay.getNode(way
    200                                 .getNodesCount()
    201                                 - i - 1)) {
     198                        if (way.getNode(i) != opositWay.getNode(way.getNodesCount() - i - 1)) {
    202199                            equals2 = false;
    203200                            break;
     
    283280    private boolean follows(Way way1, Way way2, NodePos np1, NodePos np2,
    284281            int incr) {
    285         if (way2.isClosed() && incr == 1
    286                 && np1.opositPos == way2.getNodesCount() - 2) {
     282        if (way2.isClosed() && incr == 1 && np1.opositPos == way2.getNodesCount() - 2) {
    287283            return np2.pos == np1.pos + 1 && np2.opositPos == 0;
    288284        } else if (way2.isClosed() && incr == 1 && np1.opositPos == 0) {
     
    290286                    || np2.pos == np1.pos + 1 && np2.opositPos == 1;
    291287        } else if (way2.isClosed() && incr == -1 && np1.opositPos == 0) {
    292             return np2.pos == np1.pos && np2.opositPos == 0
    293                     || np2.pos == np1.pos + 1
     288            return np2.pos == np1.pos && np2.opositPos == 0 || np2.pos == np1.pos + 1
    294289                    && np2.opositPos == way2.getNodesCount() - 2;
    295290        } else {
    296             return np2.pos == np1.pos + 1
    297                     && np2.opositPos == np1.opositPos + incr;
     291            return np2.pos == np1.pos + 1 && np2.opositPos == np1.opositPos + incr;
    298292        }
    299293    }
     
    307301     * @return
    308302     */
    309     private SplitWayResult splitWay(OsmDataLayer layer, Way way,
    310             List<List<Node>> wayChunks) {
     303    private SplitWayResult splitWay(OsmDataLayer layer, Way way, List<List<Node>> wayChunks) {
    311304        // build a list of commands, and also a new selection list
    312         Collection<Command> commandList = new ArrayList<>(wayChunks
    313                 .size());
     305        Collection<Command> commandList = new ArrayList<>(wayChunks.size());
    314306
    315307        Iterator<List<Node>> chunkIt = wayChunks.iterator();
    316         Collection<String> nowarnroles = Main.pref.getCollection(
    317                 "way.split.roles.nowarn", Arrays.asList(new String[] { "outer",
    318                         "inner", "forward", "backward" }));
     308        Collection<String> nowarnroles = Main.pref.getCollection("way.split.roles.nowarn",
     309                Arrays.asList(new String[] { "outer", "inner", "forward", "backward" }));
    319310
    320311        // First, change the original way
     
    402393                            warnme = true;
    403394                        }
    404                     } else if (!("route".equals(type))
    405                             && !("multipolygon".equals(type))) {
     395                    } else if (!("route".equals(type)) && !("multipolygon".equals(type))) {
    406396                        warnme = true;
    407397                    }
     
    418408                        int k = 1;
    419409                        while (ir - k >= 0 || ir + k < relationMembers.size()) {
    420                             if ((ir - k >= 0)
    421                                     && relationMembers.get(ir - k).isWay()) {
     410                            if ((ir - k >= 0) && relationMembers.get(ir - k).isWay()) {
    422411                                Way w = relationMembers.get(ir - k).getWay();
    423                                 if ((w.lastNode() == way.firstNode())
    424                                         || w.firstNode() == way.firstNode()) {
     412                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    425413                                    backwards = false;
    426                                 } else if ((w.firstNode() == way.lastNode())
    427                                         || w.lastNode() == way.lastNode()) {
     414                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    428415                                    backwards = true;
    429416                                }
    430417                                break;
    431418                            }
    432                             if ((ir + k < relationMembers.size())
    433                                     && relationMembers.get(ir + k).isWay()) {
     419                            if ((ir + k < relationMembers.size()) && relationMembers.get(ir + k).isWay()) {
    434420                                Way w = relationMembers.get(ir + k).getWay();
    435                                 if ((w.lastNode() == way.firstNode())
    436                                         || w.firstNode() == way.firstNode()) {
     421                                if ((w.lastNode() == way.firstNode()) || w.firstNode() == way.firstNode()) {
    437422                                    backwards = true;
    438                                 } else if ((w.firstNode() == way.lastNode())
    439                                         || w.lastNode() == way.lastNode()) {
     423                                } else if ((w.firstNode() == way.lastNode()) || w.lastNode() == way.lastNode()) {
    440424                                    backwards = false;
    441425                                }
     
    447431                        int j = ic;
    448432                        for (Way wayToAdd : newWays) {
    449                             RelationMember em = new RelationMember(
    450                                     rm.getRole(), wayToAdd);
     433                            RelationMember em = new RelationMember(rm.getRole(), wayToAdd);
    451434                            j++;
    452435                            if ((backwards != null) && backwards) {
     
    469452        }
    470453        if (warnmerole) {
    471             JOptionPane.showMessageDialog(
    472                 Main.parent,
     454            JOptionPane.showMessageDialog(Main.parent,
    473455                tr("<html>A role based relation membership was copied to all new ways.<br>You should verify this and correct it when necessary.</html>"),
    474456                tr("Warning"), JOptionPane.WARNING_MESSAGE);
    475457        } else if (warnme) {
    476             JOptionPane.showMessageDialog(
    477                 Main.parent,
     458            JOptionPane.showMessageDialog(Main.parent,
    478459                tr("<html>A relation membership was copied to all new ways.<br>You should verify this and correct it when necessary.</html>"),
    479460                tr("Warning"), JOptionPane.WARNING_MESSAGE);
    480461        }
    481462
    482         return new SplitWayResult(
    483                 new SequenceCommand(tr("Split way"), commandList), null,
    484                 changedWay, newWays);
     463        return new SplitWayResult(new SequenceCommand(tr("Split way"), commandList), null, changedWay, newWays);
    485464    }
    486465
     
    491470     * @throws UserCancelException
    492471     */
    493     private Pair<Way, List<Command>> combineWaysWorker(Collection<Way> ways)
    494             throws UserCancelException {
     472    private Pair<Way, List<Command>> combineWaysWorker(Collection<Way> ways) throws UserCancelException {
    495473
    496474        // prepare and clean the list of ways to combine
    497475        if (ways == null || ways.isEmpty())
    498476            return null;
    499         ways.remove(null); // just in case - remove all null ways from the
    500                             // collection
     477        ways.remove(null); // just in case - remove all null ways from the collection
    501478
    502479        // remove duplicates, preserving order
     
    514491        List<Way> unreversedWays = new LinkedList<>();
    515492        for (Way w : ways) {
    516             if ((path.indexOf(w.getNode(0)) + 1) == path.lastIndexOf(w
    517                     .getNode(1))) {
     493            if ((path.indexOf(w.getNode(0)) + 1) == path.lastIndexOf(w.getNode(1))) {
    518494                unreversedWays.add(w);
    519495            } else {
     
    529505        if ((reversedWays != null) && !reversedWays.isEmpty()) {
    530506            // filter out ways that have no direction-dependent tags
    531             unreversedWays = ReverseWayTagCorrector
    532                     .irreversibleWays(unreversedWays);
    533             reversedWays = ReverseWayTagCorrector
    534                     .irreversibleWays(reversedWays);
     507            unreversedWays = ReverseWayTagCorrector.irreversibleWays(unreversedWays);
     508            reversedWays = ReverseWayTagCorrector.irreversibleWays(reversedWays);
    535509            // reverse path if there are more reversed than unreversed ways with
    536510            // direction-dependent tags
     
    552526                    Way wnew = new Way(w);
    553527                    reversedTagWays.add(wnew);
    554                     changePropertyCommands = reverseWayTagCorrector.execute(w,
    555                             wnew);
    556                 }
    557                 if ((changePropertyCommands != null)
    558                         && !changePropertyCommands.isEmpty()) {
     528                    changePropertyCommands = reverseWayTagCorrector.execute(w, wnew);
     529                }
     530                if ((changePropertyCommands != null) && !changePropertyCommands.isEmpty()) {
    559531                    for (Command c : changePropertyCommands) {
    560532                        c.executeCommand();
Note: See TracChangeset for help on using the changeset viewer.