Changeset 1937 in josm


Ignore:
Timestamp:
2009-08-08T20:07:43+02:00 (15 years ago)
Author:
jttt
Message:

Replace some occurrences of RelationMember.member with getters

Location:
trunk/src/org/openstreetmap/josm
Files:
16 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/actions/CombineWayAction.java

    r1930 r1937  
    9494            }
    9595            for (RelationMember rm : r.getMembers()) {
    96                 if (rm.member instanceof Way) {
     96                if (rm.isWay()) {
    9797                    for(Way w : selectedWays) {
    98                         if (rm.member == w) {
     98                        if (rm.getMember() == w) {
    9999                            Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.getRole());
    100100                            HashSet<Way> waylinks = new HashSet<Way>();
    … …  
    231231                // Don't copy the member if it to one of our ways, just keep a
    232232                // note to re-add it later on.
    233                 if (selectedWays.contains(rm.member)) {
     233                if (selectedWays.contains(rm.getMember())) {
    234234                    rolesToReAdd.add(rm.getRole());
    235235                } else {

  • trunk/src/org/openstreetmap/josm/actions/CopyAction.java

    r1925 r1937  
    9797                List<RelationMember> members = new ArrayList<RelationMember>();
    9898                for (RelationMember m : e.getMembers()) {
    99                     if (! map.containsKey(m.member)) {
    100                         m.member.visit(this);
     99                    if (! map.containsKey(m.getMember())) {
     100                        m.getMember().visit(this);
    101101                    }
    102                     RelationMember mnew = new RelationMember(m);
    103                     mnew.member = map.get(m.member);
     102                    RelationMember mnew = new RelationMember(m.getRole(), map.get(m.getMember()));
    104103                    members.add(mnew);
    105104                }

  • trunk/src/org/openstreetmap/josm/actions/MergeNodesAction.java

    r1930 r1937  
    131131            }
    132132            for (RelationMember rm : r.getMembers()) {
    133                 if (rm.member instanceof Node) {
     133                if (rm.isNode()) {
    134134                    for (Node n : allNodes) {
    135                         if (rm.member == n) {
     135                        if (rm.getMember() == n) {
    136136                            Pair<Relation,String> pair = new Pair<Relation,String>(r, rm.getRole());
    137137                            HashSet<Node> nodelinks = new HashSet<Node>();
    … …  
    281281                // Don't copy the member if it points to one of our nodes,
    282282                // just keep a note to re-add it later on.
    283                 if (allNodes.contains(rm.member)) {
     283                if (allNodes.contains(rm.getMember())) {
    284284                    rolesToReAdd.add(rm.getRole());
    285285                } else {

  • trunk/src/org/openstreetmap/josm/command/DeleteCommand.java

    r1930 r1937  
    165165        String role = new String();
    166166        for (RelationMember m : ref.getMembers()) {
    167             if (m.member == osm) {
     167            if (m.getMember() == osm) {
    168168                role = m.getRole();
    169169                break;

  • trunk/src/org/openstreetmap/josm/data/osm/DataSet.java

    r1925 r1937  
    404404            while(it.hasNext()) {
    405405                RelationMember member = it.next();
    406                 if (member.member.equals(primitive)) {
     406                if (member.getMember().equals(primitive)) {
    407407                    it.remove();
    408408                }

  • trunk/src/org/openstreetmap/josm/data/osm/RelationMember.java

    r1930 r1937  
    3434        return role != null && !"".equals(role);
    3535    }
     36
     37    /**
     38     *
     39     * @return True if member is relation
     40     * @since 1937
     41     */
     42    public boolean isRelation() {
     43        return member instanceof Relation;
     44    }
     45
     46    /**
     47    *
     48    * @return True if member is way
     49    * @since 1937
     50    */
     51    public boolean isWay() {
     52        return member instanceof Way;
     53    }
     54
     55    /**
     56    *
     57    * @return True if member is node
     58    * @since 1937
     59    */
     60    public boolean isNode() {
     61        return member instanceof Node;
     62    }
     63
     64    /**
     65     *
     66     * @return Member as relation
     67     * @since 1937
     68     */
     69    public Relation getRelation() {
     70        return (Relation)member;
     71    }
     72
     73    /**
     74    *
     75    * @return Member as way
     76    * @since 1937
     77    */
     78    public Way getWay() {
     79        return (Way)member;
     80    }
     81
     82    /**
     83    *
     84    * @return Member as node
     85    * @since 1937
     86    */
     87    public Node getNode() {
     88        return (Node)member;
     89    }
     90
     91    /**
     92    *
     93    * @return Member
     94    * @since 1937
     95    */
     96    public OsmPrimitive getMember() {
     97        return member;
     98    }
     99
    36100
    37101    /**

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/AllNodesVisitor.java

    r1925 r1937  
    4444    public void visit(Relation e) {
    4545        for (RelationMember m : e.getMembers())
    46             if (m.member instanceof Node) visit((Node)m.member);
     46            if (m.isNode()) visit(m.getNode());
    4747    }
    4848    /**

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/BoundingXYVisitor.java

    r1925 r1937  
    3434        // only use direct members
    3535        for (RelationMember m : e.getMembers()) {
    36             if (!(m.member instanceof Relation)) {
    37                 m.member.visit(this);
     36            if (!m.isRelation()) {
     37                m.getMember().visit(this);
    3838            }
    3939        }

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/CollectBackReferencesVisitor.java

    r1925 r1937  
    7474            if (r.incomplete || r.deleted) continue;
    7575            for (RelationMember m : r.getMembers()) {
    76                 if (m.member == p) {
     76                if (m.getMember() == p) {
    7777                    if (!data.contains(r)) {
    7878                        data.add(r);

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MapPaintVisitor.java

    r1930 r1937  
    487487            for (RelationMember m : r.getMembers())
    488488            {
    489                 if (m.member != null && !m.member.incomplete && !m.member.deleted
    490                 && m.member instanceof Node)
    491                 {
    492                     drawSelectedMember(m.member, styles != null ? getPrimitiveStyle(m.member) : null, true, true);
     489                if (m.isNode() && !m.getMember().incomplete && !m.getMember().deleted)
     490                {
     491                    drawSelectedMember(m.getMember(), styles != null ? getPrimitiveStyle(m.getMember()) : null, true, true);
    493492                }
    494493            }
    … …  
    509508            for (RelationMember m : r.getMembers())
    510509            {
    511                 if (m.member != null && !m.member.incomplete && !m.member.deleted
    512                 && m.member instanceof Way) /* nodes drawn on second call */
    513                 {
    514                     drawSelectedMember(m.member, styles != null ? getPrimitiveStyle(m.member)
     510                if (m.isWay() && !m.getMember().incomplete && !m.getMember().deleted) /* nodes drawn on second call */
     511                {
     512                    drawSelectedMember(m.getMember(), styles != null ? getPrimitiveStyle(m.getMember())
    515513                    : null, true, true);
    516514                }
    … …  
    537535            //    System.out.println("member " + m.member + " selected " + r.selected);
    538536
    539             if(m.member == null)
     537            if(m.getMember() == null)
     538                // TODO Nullable member will not be allowed after RelationMember.member is encalupsed
    540539                r.putError(tr("Empty member in relation."), true);
    541             else if(m.member.deleted)
     540            else if(m.getMember().deleted)
    542541                r.putError(tr("Deleted member ''{0}'' in relation.",
    543                 m.member.getName()), true);
    544             else if(m.member.incomplete)
     542                m.getMember().getName()), true);
     543            else if(m.getMember().incomplete)
    545544            {
    546545                return;
    … …  
    548547            else
    549548            {
    550                 if(m.member instanceof Way)
    551                 {
    552                     Way w = (Way) m.member;
     549                if(m.isWay())
     550                {
     551                    Way w = m.getWay();
    553552                    if(w.getNodesCount() < 2)
    554553                    {
    … …  
    577576                        r.putError(tr("Unknown role ''{0}''.", m.getRole()), true);
    578577                }
    579                 else if(m.member instanceof Node)
    580                 {
    581                     Node n = (Node) m.member;
     578                else if(m.isNode())
     579                {
     580                    Node n = m.getNode();
    582581                    if("via".equals(m.getRole()))
    583582                    {
    … …  
    591590                }
    592591                else
    593                     r.putError(tr("Unknown member type for ''{0}''.", m.member.getName()), true);
     592                    r.putError(tr("Unknown member type for ''{0}''.", m.getMember().getName()), true);
    594593            }
    595594        }
    … …  
    784783        for (RelationMember m : r.getMembers())
    785784        {
    786             if(m.member == null)
     785            if(m.getMember() == null)
     786                //TODO Remove useless nullcheck when RelationMember.member is encalupsed
    787787                r.putError(tr("Empty member in relation."), true);
    788             else if(m.member.deleted)
     788            else if(m.getMember().deleted)
    789789                r.putError(tr("Deleted member ''{0}'' in relation.",
    790                 m.member.getName()), true);
    791             else if(m.member.incomplete)
     790                m.getMember().getName()), true);
     791            else if(m.getMember().incomplete)
    792792                incomplete = true;
    793793            else
    794794            {
    795                 if(m.member instanceof Way)
    796                 {
    797                     Way w = (Way) m.member;
     795                if(m.isWay())
     796                {
     797                    Way w = m.getWay();
    798798                    if(w.getNodesCount() < 2)
    799799                    {
    … …  
    812812                            outer.add(w);
    813813                        else if(r.isSelected())
    814                             drawSelectedMember(m.member, styles != null
    815                             ? getPrimitiveStyle(m.member) : null, true, true);
     814                            drawSelectedMember(m.getMember(), styles != null
     815                            ? getPrimitiveStyle(m.getMember()) : null, true, true);
    816816                    }
    817817                }
    … …  
    819819                {
    820820                    r.putError(tr("Non-Way ''{0}'' in multipolygon.",
    821                     m.member.getName()), true);
     821                    m.getMember().getName()), true);
    822822                }
    823823            }

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeSourceBuildingVisitor.java

    r1925 r1937  
    9292        clone.members.clear();
    9393        for (RelationMember member: r.getMembers()) {
    94             RelationMember cloneMember = new RelationMember(member);
    95             cloneMember.member = mappedPrimitives.get(member.member);
     94            RelationMember cloneMember = new RelationMember(member.getRole(), mappedPrimitives.get(member.getMember()));
    9695            clone.members.add(cloneMember);
    9796        }
    … …  
    190189        rememberRelationPartial(r);
    191190        for (RelationMember member: r.getMembers()) {
    192             if (isAlreadyRemembered(member.member)) {
     191            if (isAlreadyRemembered(member.getMember())) {
    193192                // referred primitive already remembered
    194193                //
    195194                continue;
    196195            }
    197             if (member.member instanceof Node) {
    198                 Node node = (Node)member.member;
     196            if (member.isNode()) {
     197                Node node = member.getNode();
    199198                if (isInSelectionBase(node)) {
    200199                    rememberNode(node);
    … …  
    204203                    rememberNodeIncomplete(node);
    205204                }
    206             } else if (member.member instanceof Way) {
    207                 Way way = (Way)member.member;
     205            } else if (member.isWay()) {
     206                Way way = member.getWay();
    208207                if (isInSelectionBase(way)) {
    209208                    way.visit(this);
    … …  
    213212                    rememberWayIncomplete(way);
    214213                }
    215             } else if (member.member instanceof Relation) {
    216                 Relation relation = (Relation)member.member;
    217                 if (isInSelectionBase(member.member)) {
     214            } else if (member.isRelation()) {
     215                Relation relation = member.getRelation();
     216                if (isInSelectionBase(member.getMember())) {
    218217                    relation.visit(this);
    219218                } else if (isNew(relation)) {

  • trunk/src/org/openstreetmap/josm/data/osm/visitor/MergeVisitor.java

    r1926 r1937  
    193193        LinkedList<RelationMember> newMembers = new LinkedList<RelationMember>();
    194194        for (RelationMember myMember : r.getMembers()) {
    195             OsmPrimitive mergedMember = merged.get(myMember.member);
     195            OsmPrimitive mergedMember = merged.get(myMember.getMember());
    196196            if (mergedMember == null) {
    197197                newMembers.add(myMember);
    198198            } else {
    199199                if (! mergedMember.deleted) {
    200                     RelationMember newMember = new RelationMember(myMember);
    201                     newMember.member = mergedMember;
     200                    RelationMember newMember = new RelationMember(myMember.getRole(), mergedMember);
    202201                    newMembers.add(newMember);
    203202                }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/ConflictDialog.java

    r1925 r1937  
    191191            public void visit(Relation e) {
    192192                for (RelationMember em : e.getMembers()) {
    193                     em.member.visit(this);
     193                    em.getMember().visit(this);
    194194                }
    195195            }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/ChildRelationBrowser.java

    r1925 r1937  
    366366            downloadedRelationIds.add(parent.id);
    367367            for (RelationMember member: parent.getMembers()) {
    368                 if (member.member instanceof Relation) {
    369                     Relation child = (Relation)member.member;
     368                if (member.isRelation()) {
     369                    Relation child = member.getRelation();
    370370                    if (!downloadedRelationIds.contains(child)) {
    371371                        relationsToDownload.push(child);

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/GenericRelationEditor.java

    r1927 r1937  
    5555import org.openstreetmap.josm.command.ChangeCommand;
    5656import org.openstreetmap.josm.command.ConflictAddCommand;
    57 import org.openstreetmap.josm.data.SelectionChangedListener;
    5857import org.openstreetmap.josm.data.conflict.Conflict;
    5958import org.openstreetmap.josm.data.osm.DataSet;
    … …  
    933932     * Selects  members in the relation editor which refer to primitives in the current
    934933     * selection of the context layer.
    935      * 
     934     *
    936935     */
    937936    class SelectedMembersForSelectionAction extends AbstractAction implements TableModelListener {
    … …  
    967966     * Selects primitives in the layer this editor belongs to. The selected primitives are
    968967     * equal to the set of primitives the currently selected relation members refer to.
    969      * 
     968     *
    970969     */
    971970    class SelectPrimitivesForSelectedMembersAction extends AbstractAction implements ListSelectionListener {
    … …  
    14801479            Collection<OsmPrimitive> selection = getLayer().data.getSelected();
    14811480            for (RelationMember member: r.getMembers()) {
    1482                 if (selection.contains(member.member)) {
     1481                if (selection.contains(member.getMember())) {
    14831482                    members.add(member);
    14841483                }

  • trunk/src/org/openstreetmap/josm/gui/dialogs/relation/MemberTableModel.java

    r1930 r1937  
    8989                return members.get(rowIndex).getRole();
    9090            case 1:
    91                 return members.get(rowIndex).member;
     91                return members.get(rowIndex).getMember();
    9292            case 2:
    9393                return linked(rowIndex);
    … …  
    109109
    110110    public OsmPrimitive getReferredPrimitive(int idx) {
    111         return members.get(idx).member;
     111        return members.get(idx).getMember();
    112112    }
    113113
    … …  
    198198    public void updateMemberReferences(DataSet ds) {
    199199        for (RelationMember member : members) {
    200             if (member.member.id == 0) {
     200            if (member.getMember().id == 0) {
    201201                continue;
    202202            }
    203             OsmPrimitive primitive = ds.getPrimitiveById(member.member.id);
     203            OsmPrimitive primitive = ds.getPrimitiveById(member.getMember().id);
    204204            if (primitive != null) {
    205205                member.member = primitive;
    … …  
    215215        while (it.hasNext()) {
    216216            RelationMember member = it.next();
    217             if (primitives.contains(member.member)) {
     217            if (primitives.contains(member.getMember())) {
    218218                it.remove();
    219219            }
    … …  
    240240    public boolean hasIncompleteMembers() {
    241241        for (RelationMember member : members) {
    242             if (member.member.incomplete)
     242            if (member.getMember().incomplete)
    243243                return true;
    244244        }
    … …  
    333333        int count = 0;
    334334        for (RelationMember member : members) {
    335             if (member.member.equals(primitive)) {
     335            if (member.getMember().equals(primitive)) {
    336336                count++;
    337337            }
    … …  
    379379        HashSet<OsmPrimitive> ret = new HashSet<OsmPrimitive>();
    380380        for (RelationMember m: getSelectedMembers()) {
    381             ret.add(m.member);
     381            ret.add(m.getMember());
    382382        }
    383383        return ret;
    … …  
    393393        if (referenceSet == null) return null;
    394394        for (RelationMember m: members) {
    395             if (referenceSet.contains(m.member)) {
    396                 ret.add(m.member);
     395            if (referenceSet.contains(m.getMember())) {
     396                ret.add(m.getMember());
    397397            }
    398398        }
    … …  
    466466            return false;
    467467        RelationMember member = members.get(index);
    468         if (!(member.member instanceof Relation))
     468        if (!member.isRelation())
    469469            return false;
    470         Relation r = (Relation) member.member;
     470        Relation r = member.getRelation();
    471471        return !r.incomplete;
    472472    }
    … …  
    486486        HashSet<OsmPrimitive> referrers = new HashSet<OsmPrimitive>();
    487487        for(RelationMember member : members) {
    488             referrers.add(member.member);
     488            referrers.add(member.getMember());
    489489        }
    490490        Iterator<OsmPrimitive> it = primitives.iterator();
    … …  
    509509        for (int i=0; i< members.size();i++) {
    510510            RelationMember m = members.get(i);
    511             if (primitives.contains(m.member)) {
     511            if (primitives.contains(m.getMember())) {
    512512                this.getSelectionModel().addSelectionInterval(i,i);
    513513            }
    … …  
    541541        Node    result = null;
    542542
    543         if (element.member instanceof Way) {
    544             Way w = (Way) element.member;
    545             if (linked_element.member instanceof Way) {
    546                 Way x = (Way) linked_element.member;
     543        if (element.isWay()) {
     544            Way w = element.getWay();
     545            if (linked_element.isWay()) {
     546                Way x = linked_element.getWay();
    547547                if ((w.firstNode() == x.firstNode()) || (w.firstNode() == x.lastNode())) {
    548548                    result = w.lastNode();
    … …  
    550550                    result = w.firstNode();
    551551                }
    552             } else if (linked_element.member instanceof Node) {
    553                 Node m = (Node) linked_element.member;
     552            } else if (linked_element.isNode()) {
     553                Node m = linked_element.getNode();
    554554                if (w.firstNode() == m) {
    555555                    result = w.lastNode();
    … …  
    558558                }
    559559            }
    560         } else if (element.member instanceof Node) {
    561             Node n = (Node) element.member;
     560        } else if (element.isNode()) {
     561            Node n = element.getNode();
    562562            result = n;
    563563        }
    … …  
    609609                    // only one element in segment, so try to link against each linkable node of element
    610610                    RelationMember m = members.get(segment.getFirst());
    611                     if (m.member instanceof Way) {
    612                         Way w = (Way) m.member;
     611                    if (m.isWay()) {
     612                        Way w = m.getWay();
    613613                        endSearchNode = w.lastNode();
    614614                        if (w.lastNode() != w.firstNode())
    … …  
    616616                            startSearchNode = w.firstNode();
    617617                        }
    618                     } else if (m.member instanceof Node) {
    619                         Node n = (Node) m.member;
     618                    } else if (m.isNode()) {
     619                        Node n = m.getNode();
    620620                        endSearchNode = n;
    621621                    }
    … …  
    687687        Way way2 = null;
    688688
    689         if (m1.member instanceof Way) {
    690             way1 = (Way) m1.member;
    691         }
    692         if (m2.member instanceof Way) {
    693             way2 = (Way) m2.member;
     689        if (m1.isWay()) {
     690            way1 = m1.getWay();
     691        }
     692        if (m2.isWay()) {
     693            way2 = m2.getWay();
    694694        }
    695695        if ((way1 != null) && (way2 != null)) {
Note: See TracChangeset for help on using the changeset viewer.