Ignore:
Timestamp:
2009-07-19T19:04:49+02:00 (10 years ago)
Author:
Gubaer
Message:

removed dependencies to Main.ds, removed Main.ds
removed AddVisitor, NameVisitor, DeleteVisitor - unnecessary double dispatching for these simple cases

Location:
trunk/src/org/openstreetmap/josm/corrector
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/corrector/ReverseWayTagCorrector.java

    r1762 r1814  
    115115        roleCorrectionMap.put(way, new ArrayList<RoleCorrection>());
    116116
    117         for (Relation relation : Main.ds.relations) {
     117        for (Relation relation : Main.main.getCurrentDataSet().relations) {
    118118            int position = 0;
    119119            for (RelationMember member : relation.members) {
  • trunk/src/org/openstreetmap/josm/corrector/RoleCorrectionTableModel.java

    r1001 r1814  
    66import java.util.List;
    77
    8 import org.openstreetmap.josm.data.osm.visitor.NameVisitor;
     8import org.openstreetmap.josm.gui.PrimitiveNameFormatter;
    99
    1010public class RoleCorrectionTableModel extends
    11         CorrectionTableModel<RoleCorrection> {
    12 
    13     private static NameVisitor nameVisitor = new NameVisitor();
     11CorrectionTableModel<RoleCorrection> {
     12    private static final PrimitiveNameFormatter NAME_FORMATTER = new PrimitiveNameFormatter();
    1413
    1514    public RoleCorrectionTableModel(List<RoleCorrection> roleCorrections) {
     
    4140        switch (colIndex) {
    4241        case 0:
    43             roleCorrection.relation.visit(nameVisitor);
    44             return nameVisitor.name;
     42            return NAME_FORMATTER.getName(roleCorrection.relation);
    4543        case 1:
    4644            return roleCorrection.member.role;
  • trunk/src/org/openstreetmap/josm/corrector/TagCorrector.java

    r1617 r1814  
    2525import org.openstreetmap.josm.data.osm.Node;
    2626import org.openstreetmap.josm.data.osm.OsmPrimitive;
     27import org.openstreetmap.josm.data.osm.OsmPrimitiveType;
    2728import org.openstreetmap.josm.data.osm.Relation;
    28 import org.openstreetmap.josm.data.osm.RelationMember;
    2929import org.openstreetmap.josm.data.osm.Way;
    30 import org.openstreetmap.josm.data.osm.visitor.NameVisitor;
    3130import org.openstreetmap.josm.gui.JMultilineLabel;
     31import org.openstreetmap.josm.gui.PrimitiveNameFormatter;
    3232import org.openstreetmap.josm.tools.GBC;
     33import org.openstreetmap.josm.tools.ImageProvider;
    3334
    3435public abstract class TagCorrector<P extends OsmPrimitive> {
     36    private static final PrimitiveNameFormatter NAME_FORMATTER = new PrimitiveNameFormatter();
    3537
    3638    public abstract Collection<Command> execute(P primitive, P oldprimitive)
    37         throws UserCancelException;
     39    throws UserCancelException;
    3840
    3941    private String[] applicationOptions = new String[] {
    40         tr("Apply selected changes"),
    41         tr("Don't apply changes"),
    42         tr("Cancel")
     42            tr("Apply selected changes"),
     43            tr("Don't apply changes"),
     44            tr("Cancel")
    4345    };
    4446
     
    5658        }
    5759
    58         if (!hasCorrections)
     60        if (!hasCorrections) {
    5961            for (List<RoleCorrection> roleCorrectionList : roleCorrectionMap
    6062                    .values()) {
     
    6466                }
    6567            }
     68        }
    6669
    6770        if (hasCorrections) {
     
    7275                new HashMap<OsmPrimitive, RoleCorrectionTable>();
    7376
    74             NameVisitor nameVisitor = new NameVisitor();
     77            //NameVisitor nameVisitor = new NameVisitor();
    7578
    7679            final JPanel p = new JPanel(new GridBagLayout());
     
    8790            for (OsmPrimitive primitive : tagCorrectionsMap.keySet()) {
    8891                final List<TagCorrection> tagCorrections = tagCorrectionsMap
    89                         .get(primitive);
    90 
    91                 if (tagCorrections.isEmpty())
     92                .get(primitive);
     93
     94                if (tagCorrections.isEmpty()) {
    9295                    continue;
    93 
    94                 primitive.visit(nameVisitor);
     96                }
    9597
    9698                final JLabel propertiesLabel = new JLabel(tr("Properties of "));
     
    98100
    99101                final JLabel primitiveLabel = new JLabel(
    100                         nameVisitor.name + ":", nameVisitor.icon, JLabel.LEFT);
     102                        NAME_FORMATTER.getName(primitive) + ":",
     103                        ImageProvider.get(OsmPrimitiveType.from(primitive)),
     104                        JLabel.LEFT
     105                );
    101106                p.add(primitiveLabel, GBC.eol());
    102107
     
    111116            for (OsmPrimitive primitive : roleCorrectionMap.keySet()) {
    112117                final List<RoleCorrection> roleCorrections = roleCorrectionMap
    113                         .get(primitive);
    114                 if (roleCorrections.isEmpty())
     118                .get(primitive);
     119                if (roleCorrections.isEmpty()) {
    115120                    continue;
    116 
    117                 primitive.visit(nameVisitor);
     121                }
    118122
    119123                final JLabel rolesLabel = new JLabel(
     
    122126
    123127                final JLabel primitiveLabel = new JLabel(
    124                         nameVisitor.name + ":", nameVisitor.icon, JLabel.LEFT);
     128                        NAME_FORMATTER.getName(primitive),
     129                        ImageProvider.get(OsmPrimitiveType.from(primitive)),
     130                        JLabel.LEFT
     131                );
    125132                p.add(primitiveLabel, GBC.eol());
    126133
     
    145152                    // create the clone
    146153                    OsmPrimitive clone = null;
    147                     if (primitive instanceof Way) clone = new Way((Way)primitive);
    148                     else if (primitive instanceof Node) clone = new Node((Node)primitive);
    149                     else if (primitive instanceof Relation) clone = new Relation((Relation)primitive);
     154                    if (primitive instanceof Way) {
     155                        clone = new Way((Way)primitive);
     156                    } else if (primitive instanceof Node) {
     157                        clone = new Node((Node)primitive);
     158                    } else if (primitive instanceof Relation) {
     159                        clone = new Relation((Relation)primitive);
     160                    }
    150161
    151162                    // use this structure to remember keys that have been set already so that
     
    157168                        if (tagTableMap.get(primitive).getCorrectionTableModel().getApply(i)) {
    158169                            TagCorrection tagCorrection = tagCorrections.get(i);
    159                             if (tagCorrection.isKeyChanged() && !keysChanged.contains(tagCorrection.oldKey)) clone.remove(tagCorrection.oldKey);
     170                            if (tagCorrection.isKeyChanged() && !keysChanged.contains(tagCorrection.oldKey)) {
     171                                clone.remove(tagCorrection.oldKey);
     172                            }
    160173                            clone.put(tagCorrection.newKey, tagCorrection.newValue);
    161174                            keysChanged.add(tagCorrection.newKey);
     
    164177
    165178                    // save the clone
    166                     if (!keysChanged.isEmpty()) commands.add(new ChangeCommand(primitive, clone));
     179                    if (!keysChanged.isEmpty()) {
     180                        commands.add(new ChangeCommand(primitive, clone));
     181                    }
    167182                }
    168183                for (OsmPrimitive primitive : roleCorrectionMap.keySet()) {
    169184                    List<RoleCorrection> roleCorrections = roleCorrectionMap
    170                             .get(primitive);
     185                    .get(primitive);
    171186
    172187                    for (int i = 0; i < roleCorrections.size(); i++) {
     
    177192                    }
    178193                }
    179             } else if (answer != JOptionPane.NO_OPTION) {
     194            } else if (answer != JOptionPane.NO_OPTION)
    180195                throw new UserCancelException();
    181             }
    182196            return commands;
    183197        }
Note: See TracChangeset for help on using the changeset viewer.