Ignore:
Timestamp:
2014-10-19T01:27:04+02:00 (10 years ago)
Author:
donvip
Message:

[josm_plugins] fix java 7 warnings / global usage of try-with-resource

Location:
applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin
Files:
3 edited

Legend:

Unmodified
Added
Removed

  • applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin/ImportImagePlugin.java

    r29805 r30738  
    9595            // add menu entries
    9696            //Main.main.menu.fileMenu.insert(loadFileAction, 8);
    97            
     97
    9898            //Main.main.menu.fileMenu.insertSeparator(9);
    99            
     99
    100100            ExtensionFileFilter.importers.add(new ImportImageFileImporter());
    101101
    … …  
    129129
    130130        // if properties file doesn't exist, install plugin
    131         if(!isInstalled)
    132         {
     131        if (!isInstalled) {
    133132
    134133            /*----------- Begin installation ---------------*/
    … …  
    147146
    148147            // create local properties file
    149             if(pluginProps == null || pluginProps.isEmpty()){
    150 
    151                 FileWriter fw = new FileWriter(new File(PLUGINPROPERTIES_PATH));
    152                 URL propertiesURL = pluginClassLoader.getResource("resources/" + PLUGINPROPERTIES_FILENAME);
    153                 pluginProps = new Properties();
    154                 pluginProps.load(propertiesURL.openStream());
    155                 pluginProps.store(fw, null);
    156                 fw.close();
     148            if (pluginProps == null || pluginProps.isEmpty()) {
     149                try (FileWriter fw = new FileWriter(new File(PLUGINPROPERTIES_PATH))) {
     150                        URL propertiesURL = pluginClassLoader.getResource("resources/" + PLUGINPROPERTIES_FILENAME);
     151                        pluginProps = new Properties();
     152                        pluginProps.load(propertiesURL.openStream());
     153                        pluginProps.store(fw, null);
     154                }
    157155                logger.debug("Plugin properties loaded");
    158156            }
    159157
    160             if(!new File(LOGGING_PROPERTIES_FILEPATH).exists())
    161             {
    162                 FileWriter fw = new FileWriter(new File(LOGGING_PROPERTIES_FILEPATH));
    163                 URL propertiesURL = pluginClassLoader.getResource("resources/log4j.properties");
    164                 Properties loggingProps = new Properties();
    165                 loggingProps.load(propertiesURL.openStream());
    166                 loggingProps.store(fw, null);
    167                 fw.close();
     158            if (!new File(LOGGING_PROPERTIES_FILEPATH).exists()) {
     159                try (FileWriter fw = new FileWriter(new File(LOGGING_PROPERTIES_FILEPATH))) {
     160                        URL propertiesURL = pluginClassLoader.getResource("resources/log4j.properties");
     161                        Properties loggingProps = new Properties();
     162                        loggingProps.load(propertiesURL.openStream());
     163                        loggingProps.store(fw, null);
     164                }
    168165                logger.debug("Logging properties created");
    169166            }
    … …  
    172169        }
    173170    }
    174 
    175171
    176172    /**

  • applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin/LayerPropertiesDialog.java

    r30532 r30738  
    3131import org.opengis.referencing.NoSuchAuthorityCodeException;
    3232import org.opengis.referencing.crs.CoordinateReferenceSystem;
     33import org.openstreetmap.josm.Main;
    3334
    3435/**
    … …  
    3637 * - general and spatial information about the georeferenced image
    3738 * - a possiblitly to change the source reference system of the image
    38  * 
    39  * 
     39 *
     40 *
    4041 * @author Christoph Beekmans, Fabian Kowitz, Anna Robaszkiewicz, Oliver Kuhn, Martin Ulitzny
    4142 *
    4243 */
    4344public class LayerPropertiesDialog extends JFrame{
    44    
     45
    4546    private Vector<String> supportedCRS;
    4647    private ImageLayer imageLayer;
    … …  
    8687
    8788    /**
    88      * This method initializes 
    89      * 
     89     * This method initializes
     90     *
    9091     */
    9192    public LayerPropertiesDialog(ImageLayer imageLayer, Vector<String> supportedCRS) {
    … …  
    9596        initialize();
    9697    }
    97    
    98     /**
    99      * This method initializes 
    100      * 
     98
     99    /**
     100     * This method initializes
     101     *
    101102     */
    102103    public LayerPropertiesDialog(Vector<String> supportedCRS) {
    … …  
    105106        initialize();
    106107    }
    107    
     108
    108109
    109110    /**
    110111     * This method initializes this
    111      * 
     112     *
    112113     */
    113114    private void initialize() {
    … …  
    115116        this.setContentPane(getMainPanel());
    116117        this.setPreferredSize(new Dimension(404, 485));
    117            
    118     }
    119 
    120     /**
    121      * This method initializes mainPanel   
    122      * 
    123      * @return javax.swing.JPanel   
     118
     119    }
     120
     121    /**
     122     * This method initializes mainPanel
     123     *
     124     * @return javax.swing.JPanel
    124125     */
    125126    private JPanel getMainPanel() {
    … …  
    134135
    135136    /**
    136      * This method initializes jPanel   
    137      * 
    138      * @return javax.swing.JPanel   
     137     * This method initializes jPanel
     138     *
     139     * @return javax.swing.JPanel
    139140     */
    140141    private JPanel getJPanel() {
    … …  
    155156
    156157    /**
    157      * This method initializes buttonPanel 
    158      * 
    159      * @return javax.swing.JPanel   
     158     * This method initializes buttonPanel
     159     *
     160     * @return javax.swing.JPanel
    160161     */
    161162    private JPanel getButtonPanel() {
    … …  
    170171
    171172    /**
    172      * This method initializes jTabbedPane 
    173      * 
    174      * @return javax.swing.JTabbedPane 
     173     * This method initializes jTabbedPane
     174     *
     175     * @return javax.swing.JTabbedPane
    175176     */
    176177    private JTabbedPane getJTabbedPane() {
    … …  
    184185
    185186    /**
    186      * This method initializes infoPanel   
    187      * 
    188      * @return javax.swing.JPanel   
     187     * This method initializes infoPanel
     188     *
     189     * @return javax.swing.JPanel
    189190     */
    190191    private JPanel getInfoPanel() {
    … …  
    226227            crsValueLabel = new JLabel();
    227228            crsValueLabel.setBounds(new Rectangle(150, 150, 226, 16));
    228            
     229
    229230            String crsDescription = "";
    230231            try {
    … …  
    233234            }
    234235            crsValueLabel.setText(crsDescription + "(" + imageLayer.getBbox().getCoordinateReferenceSystem().getName().toString() + ")");
    235            
     236
    236237            crsLabel = new JLabel();
    237238            crsLabel.setBounds(new Rectangle(15, 150, 118, 16));
    … …  
    281282
    282283    /**
    283      * This method initializes crsPanel 
    284      * 
    285      * @return javax.swing.JPanel   
     284     * This method initializes crsPanel
     285     *
     286     * @return javax.swing.JPanel
    286287     */
    287288    private JPanel getCrsPanel() {
    … …  
    295296            }
    296297            currentCRSValueLabel.setText(crsDescription);
    297            
     298
    298299            currentCRSLabel = new JLabel();
    299300            currentCRSLabel.setBounds(new Rectangle(15, 33, 52, 16));
    … …  
    339340
    340341    /**
    341      * This method initializes okButton 
    342      * 
    343      * @return javax.swing.JButton 
     342     * This method initializes okButton
     343     *
     344     * @return javax.swing.JButton
    344345     */
    345346    private JButton getOkButton() {
    … …  
    349350            okButton.setText("OK");
    350351            okButton.addActionListener(new java.awt.event.ActionListener() {
    351                 public void actionPerformed(java.awt.event.ActionEvent e) {
    352                    
     352                @Override
     353                                public void actionPerformed(java.awt.event.ActionEvent e) {
     354
    353355                    setVisible(false);
    354356                    dispose();
    … …  
    360362
    361363    /**
    362      * This method initializes searchField 
    363      * 
    364      * @return javax.swing.JTextField   
     364     * This method initializes searchField
     365     *
     366     * @return javax.swing.JTextField
    365367     */
    366368    private JTextField getSearchField() {
    … …  
    370372            searchField.setToolTipText("Enter keywords or EPSG codes");
    371373            searchField.addKeyListener(new java.awt.event.KeyAdapter() {
    372                 public void keyTyped(java.awt.event.KeyEvent e) {
    373                    
     374                @Override
     375                                public void keyTyped(java.awt.event.KeyEvent e) {
     376
    374377                    for (Iterator<String> iterator = supportedCRS.iterator(); iterator.hasNext();) {
    375                         String type = (String) iterator.next();
     378                        String type = iterator.next();
    376379                        if (type.contains(searchField.getText())) {
    377380                            crsJList.setSelectedIndex(supportedCRS.indexOf(type));
    … …  
    379382                            break;
    380383                        }
    381                        
     384
    382385                    }
    383386                }
    … …  
    388391
    389392    /**
    390      * This method initializes crsListScrollPane   
    391      * 
    392      * @return javax.swing.JScrollPane 
     393     * This method initializes crsListScrollPane
     394     *
     395     * @return javax.swing.JScrollPane
    393396     */
    394397    private JScrollPane getCrsListScrollPane() {
    … …  
    402405
    403406    /**
    404      * This method initializes crsJList 
    405      * 
    406      * @return javax.swing.JList   
     407     * This method initializes crsJList
     408     *
     409     * @return javax.swing.JList
    407410     */
    408411    private JList<String> getCrsJList() {
    … …  
    415418
    416419    /**
    417      * This method initializes useDefaultCRSButton 
    418      * 
    419      * @return javax.swing.JButton 
     420     * This method initializes useDefaultCRSButton
     421     *
     422     * @return javax.swing.JButton
    420423     */
    421424    private JButton getUseDefaultCRSButton() {
    … …  
    425428            useDefaultCRSButton.setText("Apply Default");
    426429            useDefaultCRSButton.addActionListener(new java.awt.event.ActionListener() {
    427                 public void actionPerformed(java.awt.event.ActionEvent e) {
    428                    
     430                @Override
     431                                public void actionPerformed(java.awt.event.ActionEvent e) {
     432
    429433                    try {
    430                        
     434
    431435                        setCursor(new Cursor(Cursor.WAIT_CURSOR));
    432436                        if(PluginOperations.defaultSourceCRS != null){
    … …  
    436440                            JOptionPane.showMessageDialog(getContentPane(), "<html>No default reference system available.<br>Please select one from the list</html>");
    437441                        }
    438                        
     442
    439443                    } catch (NoSuchAuthorityCodeException e1) {
    440444                        // TODO Auto-generated catch block
    … …  
    457461
    458462    /**
    459      * This method initializes applySelectedCRSButton   
    460      * 
    461      * @return javax.swing.JButton 
     463     * This method initializes applySelectedCRSButton
     464     *
     465     * @return javax.swing.JButton
    462466     */
    463467    private JButton getApplySelectedCRSButton() {
    … …  
    469473            applySelectedCRSButton.setText("<html>Apply<br>Selection</html>");
    470474            applySelectedCRSButton.addActionListener(new java.awt.event.ActionListener() {
    471                 public void actionPerformed(java.awt.event.ActionEvent e) {
    472                    
    473                     String selection = (String) crsJList.getSelectedValue();
     475                @Override
     476                                public void actionPerformed(java.awt.event.ActionEvent e) {
     477
     478                    String selection = crsJList.getSelectedValue();
    474479                    String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
    475                    
     480
    476481                    CoordinateReferenceSystem newRefSys = null;
    477482                    try {
    478483                        newRefSys = CRS.decode(code, eastingFirstCheckBox.isSelected());
    479                        
     484
    480485                        setCursor(new Cursor(Cursor.WAIT_CURSOR));
    481                        
     486
    482487                        imageLayer.resample(newRefSys);
    483488
    … …  
    495500                        setCursor(new Cursor(Cursor.DEFAULT_CURSOR));
    496501                    }
    497                    
    498                    
     502
     503
    499504                }
    500505            });
    … …  
    504509
    505510    /**
    506      * This method initializes setSelectedCRSAsDefaultButton   
    507      * 
    508      * @return javax.swing.JButton 
     511     * This method initializes setSelectedCRSAsDefaultButton
     512     *
     513     * @return javax.swing.JButton
    509514     */
    510515    private JButton getSetSelectedCRSAsDefaultButton() {
    … …  
    515520            setSelectedCRSAsDefaultButton
    516521                    .addActionListener(new java.awt.event.ActionListener() {
    517                         public void actionPerformed(java.awt.event.ActionEvent e) {
    518                            
     522                        @Override
     523                                                public void actionPerformed(java.awt.event.ActionEvent e) {
     524
    519525                            if(crsJList.getSelectedValue() != null){
    520                                 String selection = (String) crsJList.getSelectedValue();
     526                                String selection = crsJList.getSelectedValue();
    521527                                String code = selection.substring(selection.indexOf("[-") + 2, selection.indexOf("-]"));
    522                                
     528
    523529                                try {
    524530                                    PluginOperations.defaultSourceCRS = CRS.decode(code, eastingFirstCheckBox.isSelected());
    525531                                    PluginOperations.defaultSourceCRSDescription = selection;
    526                                    
     532
    527533                                    ImportImagePlugin.pluginProps.setProperty("default_crs_eastingfirst", "" + eastingFirstCheckBox.isSelected());
    528534                                    ImportImagePlugin.pluginProps.setProperty("default_crs_srid", code);
    529                                     FileWriter fileWriter = new FileWriter(new File(ImportImagePlugin.PLUGINPROPERTIES_PATH));
    530                                     ImportImagePlugin.pluginProps.store(fileWriter, null);
    531                                     fileWriter.close();
    532                                    
     535                                    try (FileWriter fileWriter = new FileWriter(new File(ImportImagePlugin.PLUGINPROPERTIES_PATH))) {
     536                                        ImportImagePlugin.pluginProps.store(fileWriter, null);
     537                                    }
     538
    533539                                    defaultCRSLabel.setText(selection);
    534                                    
    535                                 } catch (IOException e2) {
    536                                     // TODO Auto-generated catch block
    537                                     e2.printStackTrace();
    538                                 } catch (NoSuchAuthorityCodeException e3) {
    539                                     // TODO Auto-generated catch block
    540                                     e3.printStackTrace();
    541                                 } catch (FactoryException e4) {
    542                                     // TODO Auto-generated catch block
    543                                     e4.printStackTrace();
     540
     541                                } catch (IOException | FactoryException e2) {
     542                                    Main.error(e2);
    544543                                }
    545                             }else{
     544                            } else {
    546545                                JOptionPane.showMessageDialog(getContentPane(), "Please make a selection from the list.");
    547546                            }
    548 
    549                            
    550547                        }
    551548                    });
    … …  
    553550        return setSelectedCRSAsDefaultButton;
    554551    }
    555    
    556     /**
    557      * This method initializes eastingFirstCheckBox 
    558      * 
    559      * @return javax.swing.JCheckBox   
     552
     553    /**
     554     * This method initializes eastingFirstCheckBox
     555     *
     556     * @return javax.swing.JCheckBox
    560557     */
    561558    private JCheckBox getEastingFirstCheckBox() {
    … …  
    568565    }
    569566
    570    
    571    
     567
     568
    572569    /**
    573570     * Listener setting text in the search field if selection has changed.
    … …  
    575572     */
    576573    class ListSelectionHandler implements ListSelectionListener {
    577         public void valueChanged(ListSelectionEvent e) {
     574        @Override
     575                public void valueChanged(ListSelectionEvent e) {
    578576            if(e.getValueIsAdjusting())
    579577            {

  • applications/editors/josm/plugins/ImportImagePlugin/src/org/openstreetmap/josm/plugins/ImportImagePlugin/PluginOperations.java

    r30737 r30738  
    296296        logger.debug("Loading .prj file: " + prjFile.getAbsolutePath());
    297297
    298         StringBuilder sb = new StringBuilder();
    299         String content = null;
    300         BufferedReader br = new BufferedReader(new FileReader(prjFile));
    301         while((content = br.readLine()) != null)
    302         {
    303             sb.append(content);
    304         }
    305         br.close();
    306         try {
     298        try (BufferedReader br = new BufferedReader(new FileReader(prjFile))) {
     299            StringBuilder sb = new StringBuilder();
     300            String content = null;
     301            while((content = br.readLine()) != null) {
     302                sb.append(content);
     303            }
    307304            refSys = CRS.parseWKT(sb.toString().trim());
    308305        } catch (FactoryException e) {
Note: See TracChangeset for help on using the changeset viewer.