Ignore:
Timestamp:
2014-10-18T15:31:53+02:00 (11 years ago)
Author:
donvip
Message:

[josm_opendata] fix some sonar issues

Location:
applications/editors/josm/plugins/opendata/modules/fr.sncf
Files:
6 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/build.xml

    r30416 r30731  
    1414    <property name="module.dist.dir" value="../../dist"/>
    1515    <property name="module.jar" value="${module.dist.dir}/${ant.project.name}.jar"/>
    16         <!-- conditions -->
     16    <!-- conditions -->
    1717    <condition property="resources.exist">
    1818        <available file="resources" type="dir" />

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/src/org/openstreetmap/josm/plugins/opendata/modules/fr/sncf/SncfConstants.java

    r30340 r30731  
    55
    66public interface SncfConstants extends FrenchConstants {
    7        
    8         /**
    9         * Source
    10         */
    11         public static final String SOURCE = "SNCF";
    12        
    13         /**
    14         * Portal
    15         */
    16         public static final String PORTAL = "http://opendata-test.tumblr.com/";
     7   
     8    /**
     9    * Source
     10    */
     11    public static final String SOURCE = "SNCF";
     12   
     13    /**
     14    * Portal
     15    */
     16    public static final String PORTAL = "http://opendata-test.tumblr.com/";
    1717
    18         /**
    19         * Icons
    20         */
    21         public static final String ICON_16 = "data.fr.sncf_16.png";
    22         public static final String ICON_24 = "data.fr.sncf_24.png";
     18    /**
     19    * Icons
     20    */
     21    public static final String ICON_16 = "data.fr.sncf_16.png";
     22    public static final String ICON_24 = "data.fr.sncf_24.png";
    2323}

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/src/org/openstreetmap/josm/plugins/opendata/modules/fr/sncf/SncfLicense.java

    r30340 r30731  
    88public class SncfLicense extends License {
    99
    10         public SncfLicense() {
    11                 try {
    12                         setURL("http://test.data-sncf.com/licence", "fr");
    13                 } catch (MalformedURLException e) {
    14                         e.printStackTrace();
    15                 }
    16         }
     10    public SncfLicense() {
     11        try {
     12            setURL("http://test.data-sncf.com/licence", "fr");
     13        } catch (MalformedURLException e) {
     14            e.printStackTrace();
     15        }
     16    }
    1717}

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/src/org/openstreetmap/josm/plugins/opendata/modules/fr/sncf/SncfModule.java

    r30340 r30731  
    88public class SncfModule extends AbstractModule {
    99
    10         public SncfModule(ModuleInformation info) {
    11                 super(info);
    12                 handlers.add(EquipementsHandler.class);
     10    public SncfModule(ModuleInformation info) {
     11        super(info);
     12        handlers.add(EquipementsHandler.class);
    1313    }
    1414}

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/src/org/openstreetmap/josm/plugins/opendata/modules/fr/sncf/datasets/EquipementsHandler.java

    r30340 r30731  
    99public class EquipementsHandler extends SncfDataSetHandler {
    1010
    11         private class LambertIICsvHandler extends InternalCsvHandler {
    12                 @Override
    13                 public LatLon getCoor(EastNorth en, String[] fields) {
    14                         // Lambert II coordinates offset by 2000000 (see https://fr.wikipedia.org/wiki/Projection_conique_conforme_de_Lambert#Projections_officielles_en_France_métropolitaine)
    15                         return super.getCoor(new EastNorth(en.getX(), en.getY()-2000000), fields);
    16                 }
    17         }
    18        
    19         public EquipementsHandler() {
    20                 super("equipementsgares");
    21                 setCsvHandler(new LambertIICsvHandler());
    22                 setSingleProjection(lambert4Zones[1]); // Lambert II
    23         }
     11    private class LambertIICsvHandler extends InternalCsvHandler {
     12        @Override
     13        public LatLon getCoor(EastNorth en, String[] fields) {
     14            // Lambert II coordinates offset by 2000000 (see https://fr.wikipedia.org/wiki/Projection_conique_conforme_de_Lambert#Projections_officielles_en_France_métropolitaine)
     15            return super.getCoor(new EastNorth(en.getX(), en.getY()-2000000), fields);
     16        }
     17    }
     18   
     19    public EquipementsHandler() {
     20        super("equipementsgares");
     21        setCsvHandler(new LambertIICsvHandler());
     22        setSingleProjection(lambert4Zones[1]); // Lambert II
     23    }
    2424
    25         @Override
    26         public boolean acceptsFilename(String filename) {
    27                 return acceptsCsvXlsFilename(filename, "gare_20......");
    28         }
     25    @Override
     26    public boolean acceptsFilename(String filename) {
     27        return acceptsCsvXlsFilename(filename, "gare_20......");
     28    }
    2929
    30         @Override
    31         public void updateDataSet(DataSet ds) {
    32                 for (Node n : ds.getNodes()) {
    33                         replace(n, "nom gare", "name");
    34                         n.put("railway", "station");
    35                 }
    36         }
     30    @Override
     31    public void updateDataSet(DataSet ds) {
     32        for (Node n : ds.getNodes()) {
     33            replace(n, "nom gare", "name");
     34            n.put("railway", "station");
     35        }
     36    }
    3737}

  • applications/editors/josm/plugins/opendata/modules/fr.sncf/src/org/openstreetmap/josm/plugins/opendata/modules/fr/sncf/datasets/SncfDataSetHandler.java

    r30340 r30731  
    1010
    1111public abstract class SncfDataSetHandler extends FrenchDataSetHandler implements SncfConstants {
    12        
    13         public SncfDataSetHandler(String portalId) {
    14                 init(portalId);
    15         }
    16        
    17         public SncfDataSetHandler(String portalId, String relevantTag) {
    18                 super(relevantTag);
    19                 init(portalId);
    20         }
    21        
    22         public SncfDataSetHandler(String portalId, boolean relevantUnion, String ... relevantTags) {
    23                 super(relevantUnion, relevantTags);
    24                 init(portalId);
    25         }
     12   
     13    public SncfDataSetHandler(String portalId) {
     14        init(portalId);
     15    }
     16   
     17    public SncfDataSetHandler(String portalId, String relevantTag) {
     18        super(relevantTag);
     19        init(portalId);
     20    }
     21   
     22    public SncfDataSetHandler(String portalId, boolean relevantUnion, String ... relevantTags) {
     23        super(relevantUnion, relevantTags);
     24        init(portalId);
     25    }
    2626
    27         public SncfDataSetHandler(String portalId, String ... relevantTags) {
    28                 this(portalId, false, relevantTags);
    29         }
    30        
    31         public SncfDataSetHandler(String portalId, boolean relevantUnion, Tag ... relevantTags) {
    32                 super(relevantUnion, relevantTags);
    33                 init(portalId);
    34         }
     27    public SncfDataSetHandler(String portalId, String ... relevantTags) {
     28        this(portalId, false, relevantTags);
     29    }
     30   
     31    public SncfDataSetHandler(String portalId, boolean relevantUnion, Tag ... relevantTags) {
     32        super(relevantUnion, relevantTags);
     33        init(portalId);
     34    }
    3535
    36         private final void init(String portalId) {
    37                 setLicense(new SncfLicense());
    38                 if (portalId != null && !portalId.isEmpty()) {
    39                         try {
    40                                 setLocalPortalURL(PORTAL + portalId);
    41                         } catch (MalformedURLException e) {
    42                                 e.printStackTrace();
    43                         }
    44                 }
    45         }
     36    private final void init(String portalId) {
     37        setLicense(new SncfLicense());
     38        if (portalId != null && !portalId.isEmpty()) {
     39            try {
     40                setLocalPortalURL(PORTAL + portalId);
     41            } catch (MalformedURLException e) {
     42                e.printStackTrace();
     43            }
     44        }
     45    }
    4646
    47         @Override
    48         public String getSource() {
    49                 return SOURCE;
    50         }
     47    @Override
     48    public String getSource() {
     49        return SOURCE;
     50    }
    5151
    52         @Override
    53         public String getLocalPortalIconName() {
    54                 return ICON_24;
    55         }
     52    @Override
     53    public String getLocalPortalIconName() {
     54        return ICON_24;
     55    }
    5656
    57         @Override
    58         public String getDataLayerIconName() {
    59                 return ICON_16;
    60         }
     57    @Override
     58    public String getDataLayerIconName() {
     59        return ICON_16;
     60    }
    6161}
Note: See TracChangeset for help on using the changeset viewer.