Ignore:
Timestamp:
2009-11-02T07:51:28+01:00 (14 years ago)
Author:
jttt
Message:

Change most occurrences of Dataset.nodes/ways/relations with getNodes()/../.. or addPrimitive

Location:
trunk/src/org/openstreetmap/josm/gui
Files:
12 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/NavigatableComponent.java

    r2327 r2381  
    304304        Node minPrimitive = null;
    305305        DataSet ds = getCurrentDataSet();
    306         if(ds == null)
     306        if (ds == null)
    307307            return null;
    308         for (Node n : ds.nodes) {
     308        for (Node n : ds.getNodes()) {
    309309            if (!n.isUsable()) {
    310310                continue;
    … …  
    317317            }
    318318            // when multiple nodes on one point, prefer new or selected nodes
    319             else if(dist == minDistanceSq && minPrimitive != null
     319            else if (dist == minDistanceSq && minPrimitive != null
    320320                    && ((n.isNew() && ds.isSelected(n))
    321                             || (!ds.isSelected(minPrimitive) && (ds.isSelected(n) || n.isNew())))) {
     321                    || (!ds.isSelected(minPrimitive) && (ds.isSelected(n) || n.isNew())))) {
    322322                minPrimitive = n;
    323323            }
    … …  
    335335        TreeMap<Double, List<WaySegment>> nearest = new TreeMap<Double, List<WaySegment>>();
    336336        DataSet ds = getCurrentDataSet();
    337         if(ds == null)
     337        if (ds == null)
    338338            return null;
    339         for (Way w : ds.ways) {
     339        for (Way w : ds.getWays()) {
    340340            if (!w.isUsable()) {
    341341                continue;
    … …  
    358358                double a = p.distanceSq(B);
    359359                double b = p.distanceSq(A);
    360                 double perDist = a-(a-b+c)*(a-b+c)/4/c; // perpendicular distance squared
    361                 if (perDist < snapDistance && a < c+snapDistance && b < c+snapDistance) {
    362                     if(ds.isSelected(w)) {
     360                double perDist = a - (a - b + c) * (a - b + c) / 4 / c; // perpendicular distance squared
     361                if (perDist < snapDistance && a < c + snapDistance && b < c + snapDistance) {
     362                    if (ds.isSelected(w)) {
    363363                        perDist -= 0.00001;
    364364                    }
    … …  
    459459        Collection<OsmPrimitive> nearest = new HashSet<OsmPrimitive>();
    460460        DataSet ds = getCurrentDataSet();
    461         if(ds == null)
     461        if (ds == null)
    462462            return null;
    463         for (Way w : ds.ways) {
     463        for (Way w : ds.getWays()) {
    464464            if (!w.isUsable()) {
    465465                continue;
    … …  
    479479                double a = p.distanceSq(B);
    480480                double b = p.distanceSq(A);
    481                 double perDist = a-(a-b+c)*(a-b+c)/4/c; // perpendicular distance squared
    482                 if (perDist < snapDistance && a < c+snapDistance && b < c+snapDistance) {
     481                double perDist = a - (a - b + c) * (a - b + c) / 4 / c; // perpendicular distance squared
     482                if (perDist < snapDistance && a < c + snapDistance && b < c + snapDistance) {
    483483                    nearest.add(w);
    484484                    break;
    … …  
    487487            }
    488488        }
    489         for (Node n : ds.nodes) {
     489        for (Node n : ds.getNodes()) {
    490490            if (n.isUsable()
    491491                    && getPoint(n).distanceSq(p) < snapDistance) {
    … …  
    507507        Collection<Node> nearest = new HashSet<Node>();
    508508        DataSet ds = getCurrentDataSet();
    509         if(ds == null)
     509        if (ds == null)
    510510            return null;
    511         for (Node n : ds.nodes) {
     511        for (Node n : ds.getNodes()) {
    512512            if (n.isUsable()
    513513                    && getPoint(n).distanceSq(p) < snapDistance) {

  • trunk/src/org/openstreetmap/josm/gui/SelectionManager.java

    r2120 r2381  
    285285        } else {
    286286            // nodes
    287             for (Node n : nc.getCurrentDataSet().nodes) {
     287            for (Node n : nc.getCurrentDataSet().getNodes()) {
    288288                if (n.isUsable() && r.contains(nc.getPoint(n))) {
    289289                    selection.add(n);
    … …  
    292292
    293293            // ways
    294             for (Way w : nc.getCurrentDataSet().ways) {
    295                 if (!w.isUsable() || w.getNodesCount() == 0){
     294            for (Way w : nc.getCurrentDataSet().getWays()) {
     295                if (!w.isUsable() || w.getNodesCount() == 0) {
    296296                    continue;
    297297                }

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

    r2348 r2381  
    754754        Map<Relation, Collection<RelationMember>> roles = new HashMap<Relation, Collection<RelationMember>>();
    755755        if (Main.main.getCurrentDataSet() != null) {
    756             for (Relation r : Main.main.getCurrentDataSet().relations) {
     756            for (Relation r : Main.main.getCurrentDataSet().getRelations()) {
    757757                if (!r.isFiltered() && !r.incomplete && !r.isDeleted()) {
    758758                    for (RelationMember m : r.getMembers()) {

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

    r2348 r2381  
    147147    protected int getNumRelations() {
    148148        if (Main.main.getCurrentDataSet() == null) return 0;
    149         return Main.main.getCurrentDataSet().relations.size();
     149        return Main.main.getCurrentDataSet().getRelations().size();
    150150    }
    151151
    … …  
    158158     */
    159159    protected ArrayList<Relation> getDisplayedRelationsInSortOrder(DataSet ds) {
    160         ArrayList<Relation> relations = new ArrayList<Relation>(ds.relations.size());
    161         for (Relation r: ds.relations ){
     160        ArrayList<Relation> relations = new ArrayList<Relation>(ds.getRelations().size());
     161        for (Relation r : ds.getRelations()) {
    162162            if (!r.isUsable() || !r.isVisible()) {
    163163                continue;
    … …  
    170170                new Comparator<Relation>() {
    171171                    NameFormatter formatter = DefaultNameFormatter.getInstance();
     172
    172173                    public int compare(Relation r1, Relation r2) {
    173174                        return r1.getDisplayName(formatter).compareTo(r2.getDisplayName(formatter));

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

    r2273 r2381  
    258258                return;
    259259            HashSet<Relation> relations = new HashSet<Relation>();
    260             for (int i=0; i < selection.length;i++) {
    261                 relations.add((Relation)selection[i].getLastPathComponent());
     260            for (TreePath aSelection : selection) {
     261                relations.add((Relation) aSelection.getLastPathComponent());
    262262            }
    263263            Main.worker.submit(new DownloadRelationSetTask(getParentDialog(),relations));

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

    r2378 r2381  
    138138            //this.tagEditorModel.initFromPrimitive(relation);
    139139            this.memberTableModel.populate(relation);
    140             if (!getLayer().data.relations.contains(relation)) {
     140            if (!getLayer().data.getRelations().contains(relation)) {
    141141                // treat it as a new relation if it doesn't exist in the
    142142                // data set yet.
    … …  
    597597            );
    598598            switch(ret) {
    599                 case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return;
    600                 case JOptionPane.CLOSED_OPTION: return;
    601                 case JOptionPane.NO_OPTION: return;
    602                 case JOptionPane.YES_OPTION:
    603                     memberTableModel.removeMembersReferringTo(toCheck);
    604                     break;
     599            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return;
     600            case JOptionPane.CLOSED_OPTION: return;
     601            case JOptionPane.NO_OPTION: return;
     602            case JOptionPane.YES_OPTION:
     603                memberTableModel.removeMembersReferringTo(toCheck);
     604                break;
    605605            }
    606606        }
    … …  
    633633            );
    634634            switch(ret) {
    635                 case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION : return true;
    636                 case JOptionPane.YES_OPTION: return true;
    637                 case JOptionPane.NO_OPTION: return false;
    638                 case JOptionPane.CLOSED_OPTION: return false;
    639                 case JOptionPane.CANCEL_OPTION: throw new AddAbortException();
     635            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION : return true;
     636            case JOptionPane.YES_OPTION: return true;
     637            case JOptionPane.NO_OPTION: return false;
     638            case JOptionPane.CLOSED_OPTION: return false;
     639            case JOptionPane.CANCEL_OPTION: throw new AddAbortException();
    640640            }
    641641            // should not happen
    … …  
    12551255            );
    12561256            switch(ret) {
    1257                 case JOptionPane.YES_OPTION: return true;
    1258                 case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return true;
    1259                 default:
    1260                     return false;
     1257            case JOptionPane.YES_OPTION: return true;
     1258            case ConditionalOptionPaneUtil.DIALOG_DISABLED_OPTION: return true;
     1259            default:
     1260                return false;
    12611261            }
    12621262        }

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

    r2273 r2381  
    150150        }
    151151        parents.clear();
    152         for (Relation parent : referrers.relations) {
    153             parents.add((Relation)getLayer().data.getPrimitiveById(parent.getId(),OsmPrimitiveType.RELATION));
     152        for (Relation parent : referrers.getRelations()) {
     153            parents.add((Relation) getLayer().data.getPrimitiveById(parent.getId(), OsmPrimitiveType.RELATION));
    154154        }
    155155        if (continuation != null) {

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

    r2273 r2381  
    7979            return;
    8080        }
    81         for (Relation parent : ds.relations) {
     81        for (Relation parent : ds.getRelations()) {
    8282            if (isReferringRelation(parent)) {
    8383                referrers.add(parent);

  • trunk/src/org/openstreetmap/josm/gui/layer/GpxLayer.java

    r2322 r2381  
    776776                            n.setTimestamp(DateUtils.fromString(timestr));
    777777                        }
    778                         ds.nodes.add(n);
     778                        ds.addPrimitive(n);
    779779                        nodes.add(n);
    780780                    }
    781781                    Way w = new Way();
    782782                    w.setNodes(nodes);
    783                     ds.ways.add(w);
     783                    ds.addPrimitive(w);
    784784                }
    785785            }

  • trunk/src/org/openstreetmap/josm/gui/layer/OsmDataLayer.java

    r2327 r2381  
    33package org.openstreetmap.josm.gui.layer;
    44
     5import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    56import static org.openstreetmap.josm.tools.I18n.marktr;
    6 import static org.openstreetmap.josm.gui.help.HelpUtil.ht;
    77import static org.openstreetmap.josm.tools.I18n.tr;
    88import static org.openstreetmap.josm.tools.I18n.trn;
    … …  
    1919import java.awt.TexturePaint;
    2020import java.awt.event.ActionEvent;
    21 import java.awt.event.ActionListener;
    2221import java.awt.geom.Area;
    2322import java.awt.image.BufferedImage;
    … …  
    3231import javax.swing.AbstractAction;
    3332import javax.swing.Icon;
    34 import javax.swing.JButton;
    35 import javax.swing.JDialog;
    3633import javax.swing.JLabel;
    3734import javax.swing.JMenuItem;
    … …  
    6865import org.openstreetmap.josm.gui.dialogs.LayerListDialog;
    6966import org.openstreetmap.josm.gui.dialogs.LayerListPopup;
    70 import org.openstreetmap.josm.gui.help.HelpBrowser;
    71 import org.openstreetmap.josm.gui.help.HelpUtil;
    7267import org.openstreetmap.josm.tools.DateUtils;
    7368import org.openstreetmap.josm.tools.GBC;
    7469import org.openstreetmap.josm.tools.ImageProvider;
    75 import org.openstreetmap.josm.tools.WindowGeometry;
    7670
    7771/**
    … …  
    256250    @Override public String getToolTipText() {
    257251        String tool = "";
    258         tool += undeletedSize(data.nodes)+" "+trn("node", "nodes", undeletedSize(data.nodes))+", ";
    259         tool += undeletedSize(data.ways)+" "+trn("way", "ways", undeletedSize(data.ways));
     252        tool += undeletedSize(data.getNodes())+" "+trn("node", "nodes", undeletedSize(data.getNodes()))+", ";
     253        tool += undeletedSize(data.getWays())+" "+trn("way", "ways", undeletedSize(data.getWays()));
    260254        if (data.version != null) {
    261255            tool += ", " + tr("version {0}", data.version);
    … …  
    350344                    numRemainingConflicts,
    351345                    numRemainingConflicts
    352             );           
    353         }
    354        
     346            );
     347        }
     348
    355349        StringBuffer sb = new StringBuffer();
    356350        sb.append("<html>").append(msg1);
    … …  
    369363                            tr("Click to close this dialog and continue editing"),
    370364                            null /* no specific help */
    371                             )
     365                    )
    372366            };
    373367            HelpAwareOptionPane.showOptionDialog(
    … …  
    380374                    options[0],
    381375                    ht("/Concepts/Conflict#WarningAboutDetectedConflicts")
    382              );           
     376            );
    383377        }
    384378    }
    … …  
    435429
    436430    @Override public void visitBoundingBox(final BoundingXYVisitor v) {
    437         for (final Node n : data.nodes)
     431        for (final Node n: data.getNodes()) {
    438432            if (n.isUsable()) {
    439433                v.visit(n);
    440434            }
     435        }
    441436    }
    442437
    … …  
    558553        gpxData.storageFile = file;
    559554        HashSet<Node> doneNodes = new HashSet<Node>();
    560         for (Way w : data.ways) {
     555        for (Way w : data.getWays()) {
    561556            if (!w.isUsable()) {
    562557                continue;
    … …  
    583578                }
    584579                WayPoint wpt = new WayPoint(n.getCoor());
    585                 if (!n.isTimestampEmpty())
    586                 {
     580                if (!n.isTimestampEmpty()) {
    587581                    wpt.attr.put("time", DateUtils.fromDate(n.getTimestamp()));
    588582                    wpt.setTime();
    … …  
    594588        // what is this loop meant to do? it creates waypoints but never
    595589        // records them?
    596         for (Node n : data.nodes) {
     590        for (Node n : data.getNodes()) {
    597591            if (n.incomplete || n.isDeleted() || doneNodes.contains(n)) {
    598592                continue;
    … …  
    607601                wpt.attr.put("name", name);
    608602            }
     603            ;
    609604        }
    610605        return gpxData;

  • trunk/src/org/openstreetmap/josm/gui/layer/RawGpsLayer.java

    r2179 r2381  
    7878                for (GpsPoint p : c) {
    7979                    Node n = new Node(p.latlon);
    80                     ds.nodes.add(n);
     80                    ds.addPrimitive(n);
    8181                    nodes.add(n);
    8282                }
    8383                Way w = new Way();
    8484                w.setNodes(nodes);
    85                 ds.ways.add(w);
     85                ds.addPrimitive(w);
    8686            }
    8787            Main.main.addLayer(new OsmDataLayer(ds, tr("Converted from: {0}", RawGpsLayer.this.getName()), null));

  • trunk/src/org/openstreetmap/josm/gui/tagging/ac/AutoCompletionCache.java

    r2088 r2381  
    150150            cachePrimitive(primitive);
    151151        }
    152         for (Relation relation : layer.data.relations) {
     152        for (Relation relation : layer.data.getRelations()) {
    153153            if (relation.incomplete || relation.isDeleted()) {
    154154                continue;
Note: See TracChangeset for help on using the changeset viewer.