Changeset 31264 in osm


Ignore:
Timestamp:
2015-06-13T13:17:09+02:00 (9 years ago)
Author:
nokutu
Message:

Code cleaned and commented

Location:
applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary
Files:
8 edited

Legend:

Unmodified
Added
Removed
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryAbstractImage.java

    r31257 r31264  
    33import org.openstreetmap.josm.data.coor.LatLon;
    44
     5/**
     6 * Abstract supperclass for all image objects. At the moment there is just 2,
     7 * {@code MapillaryImportedImage} and {@code MapillaryImage}.
     8 *
     9 * @author nokutu
     10 *
     11 */
    512public abstract class MapillaryAbstractImage {
    613
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryImportedImage.java

    r31261 r31264  
    99public class MapillaryImportedImage extends MapillaryAbstractImage {
    1010
     11        /**
     12         * The picture file.
     13         */
    1114        protected File file;
    1215
     
    1619        }
    1720
     21        /**
     22         * Returns the pictures of the file.
     23         *
     24         * @return
     25         * @throws IOException
     26         */
    1827        public BufferedImage getImage() throws IOException {
    1928                return ImageIO.read(file);
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/MapillaryMouseAdapter.java

    r31255 r31264  
    1212import org.openstreetmap.josm.plugins.mapillary.commands.MapillaryRecord;
    1313
     14/**
     15 * Handles the input event related with the layer. Mainly clicks.
     16 *
     17 * @author nokutu
     18 *
     19 */
    1420public class MapillaryMouseAdapter extends MouseAdapter {
    1521        private Point start;
     
    1925        private MapillaryData mapillaryData;
    2026        private MapillaryRecord record;
    21        
    22         public MapillaryMouseAdapter(){
     27
     28        public MapillaryMouseAdapter() {
    2329                mapillaryData = MapillaryData.getInstance();
    2430                record = MapillaryRecord.getInstance();
     
    3036                if (e.getButton() != MouseEvent.BUTTON1)
    3137                        return;
    32                 if (Main.map.mapView.getActiveLayer() != MapillaryLayer
    33                                 .getInstance())
     38                if (Main.map.mapView.getActiveLayer() != MapillaryLayer.getInstance())
    3439                        return;
    3540                MapillaryAbstractImage closestTemp = getClosest(e.getPoint());
    36                 if (closestTemp instanceof MapillaryImage
    37                                 || closestTemp == null) {
     41                if (closestTemp instanceof MapillaryImage || closestTemp == null) {
    3842                        MapillaryImage closest = (MapillaryImage) closestTemp;
    3943                        // Doube click
     
    4953                        this.lastClicked = this.closest;
    5054                        this.closest = closest;
    51                         if (mapillaryData.getMultiSelectedImages()
    52                                         .contains(closest))
     55                        if (mapillaryData.getMultiSelectedImages().contains(closest))
    5356                                return;
    5457                        // ctrl+click
     
    6467                                                && ((MapillaryImage) this.closest).getSequence() == ((MapillaryImage) this.lastClicked)
    6568                                                                .getSequence()) {
    66                                         int i = ((MapillaryImage) this.closest).getSequence().getImages()
    67                                                         .indexOf(this.closest);
    68                                         int j = ((MapillaryImage) this.lastClicked).getSequence().getImages()
    69                                                         .indexOf(this.lastClicked);
     69                                        int i = ((MapillaryImage) this.closest).getSequence()
     70                                                        .getImages().indexOf(this.closest);
     71                                        int j = ((MapillaryImage) this.lastClicked).getSequence()
     72                                                        .getImages().indexOf(this.lastClicked);
    7073                                        if (i < j)
    7174                                                mapillaryData
    7275                                                                .addMultiSelectedImage(new ArrayList<MapillaryAbstractImage>(
    73                                                                                 ((MapillaryImage) this.closest).getSequence()
    74                                                                                                 .getImages()
     76                                                                                ((MapillaryImage) this.closest)
     77                                                                                                .getSequence().getImages()
    7578                                                                                                .subList(i, j + 1)));
    7679                                        else
    7780                                                mapillaryData
    7881                                                                .addMultiSelectedImage(new ArrayList<MapillaryAbstractImage>(
    79                                                                                 ((MapillaryImage) this.closest).getSequence()
    80                                                                                                 .getImages()
     82                                                                                ((MapillaryImage) this.closest)
     83                                                                                                .getSequence().getImages()
    8184                                                                                                .subList(j, i + 1)));
    8285                                }
     
    8487                        } else
    8588                                mapillaryData.setSelectedImage(closest);
    86                 // If you select and imported image
     89                // If you select an imported image
    8790                } else if (closestTemp instanceof MapillaryImportedImage) {
    8891                        MapillaryImportedImage closest = (MapillaryImportedImage) closestTemp;
     
    9093                        this.lastClicked = this.closest;
    9194                        this.closest = closest;
    92                         if (mapillaryData.getMultiSelectedImages()
    93                                         .contains(closest))
     95                        if (mapillaryData.getMultiSelectedImages().contains(closest))
    9496                                return;
    9597                        if (e.getModifiers() == (MouseEvent.BUTTON1_MASK | MouseEvent.CTRL_MASK)
     
    106108                MapillaryAbstractImage closest = null;
    107109                for (MapillaryAbstractImage image : mapillaryData.getImages()) {
    108                         Point imagePoint = Main.map.mapView.getPoint(image
    109                                         .getLatLon());
    110                         imagePoint
    111                                         .setLocation(imagePoint.getX(), imagePoint.getY());
     110                        Point imagePoint = Main.map.mapView.getPoint(image.getLatLon());
     111                        imagePoint.setLocation(imagePoint.getX(), imagePoint.getY());
    112112                        double dist = clickPoint.distanceSq(imagePoint);
    113113                        if (minDistance > dist
     
    122122        @Override
    123123        public void mouseDragged(MouseEvent e) {
    124                 if (Main.map.mapView.getActiveLayer() != MapillaryLayer
    125                                 .getInstance())
     124                if (Main.map.mapView.getActiveLayer() != MapillaryLayer.getInstance())
    126125                        return;
    127126                if (MapillaryData.getInstance().getSelectedImage() != null) {
    128127                        if (lastButton == MouseEvent.BUTTON1 && !e.isShiftDown()) {
    129                                 LatLon to = Main.map.mapView.getLatLon(e.getX(),
    130                                                 e.getY());
     128                                LatLon to = Main.map.mapView.getLatLon(e.getX(), e.getY());
    131129                                LatLon from = Main.map.mapView.getLatLon(start.getX(),
    132130                                                start.getY());
    133                                 for (MapillaryAbstractImage img : MapillaryData
    134                                                 .getInstance().getMultiSelectedImages()) {
    135                                         img.move(to.getX() - from.getX(),
    136                                                         to.getY() - from.getY());
     131                                for (MapillaryAbstractImage img : MapillaryData.getInstance()
     132                                                .getMultiSelectedImages()) {
     133                                        img.move(to.getX() - from.getX(), to.getY() - from.getY());
    137134                                }
    138135                                Main.map.repaint();
    139                         } else if (lastButton == MouseEvent.BUTTON1
    140                                         && e.isShiftDown()) {
     136                        } else if (lastButton == MouseEvent.BUTTON1 && e.isShiftDown()) {
    141137                                this.closest.turn(Math.toDegrees(Math.atan2(
    142138                                                (e.getX() - start.x), -(e.getY() - start.y)))
    143139                                                - closest.getTempCa());
    144                                 for (MapillaryAbstractImage img : MapillaryData
    145                                                 .getInstance().getMultiSelectedImages()) {
    146                                         img.turn(Math.toDegrees(Math.atan2(
    147                                                         (e.getX() - start.x), -(e.getY() - start.y)))
    148                                                         - closest.getTempCa());
     140                                for (MapillaryAbstractImage img : MapillaryData.getInstance()
     141                                                .getMultiSelectedImages()) {
     142                                        img.turn(Math.toDegrees(Math.atan2((e.getX() - start.x),
     143                                                        -(e.getY() - start.y))) - closest.getTempCa());
    149144                                }
    150145                                Main.map.repaint();
     
    165160                } else if (mapillaryData.getSelectedImage().getTempLatLon() != mapillaryData
    166161                                .getSelectedImage().getLatLon()) {
    167                         LatLon from = mapillaryData.getSelectedImage()
    168                                         .getTempLatLon();
     162                        LatLon from = mapillaryData.getSelectedImage().getTempLatLon();
    169163                        LatLon to = mapillaryData.getSelectedImage().getLatLon();
    170164                        record.addCommand(new CommandMoveImage(mapillaryData
    171                                         .getMultiSelectedImages(), to.getX() - from.getX(),
    172                                         to.getY() - from.getY()));
     165                                        .getMultiSelectedImages(), to.getX() - from.getX(), to
     166                                        .getY() - from.getY()));
    173167                }
    174168                for (MapillaryAbstractImage img : mapillaryData
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/actions/MapillaryDownloadAction.java

    r31252 r31264  
    3333        }
    3434
    35         /*
    36          * (non-Javadoc)
    37          *
    38          * @see
    39          * java.awt.event.ActionListener#actionPerformed(java.awt.event.ActionEvent)
    40          */
    4135        @Override
    4236        public void actionPerformed(ActionEvent arg0) {
    4337                this.layer = null;
    4438                if (Main.map == null || Main.map.mapView == null
    45                                 || Main.map.mapView.getEditLayer() == null) {
     39                                || Main.map.mapView.getEditLayer() == null)
    4640                        return;
    47                 }
    48                 for (Layer layer : Main.map.mapView.getAllLayers()) {
    49                         if (layer instanceof MapillaryLayer) {
     41                for (Layer layer : Main.map.mapView.getAllLayers())
     42                        if (layer instanceof MapillaryLayer)
    5043                                this.layer = (MapillaryLayer) layer;
    51                         }
    52                 }
    5344
    5445                if (this.layer == null)
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/actions/MapillaryExportAction.java

    r31261 r31264  
    6969                                export(MapillaryData.getInstance().getMultiSelectedImages());
    7070                        }
     71                // This option ignores the selected directory.
    7172                 } else if (dialog.group.isSelected(dialog.rewrite.getModel())) {
    7273                        ArrayList<MapillaryImportedImage> images = new ArrayList<>();
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/commands/MapillaryCommand.java

    r31257 r31264  
    1717
    1818        public abstract void redo();
    19        
     19
     20        /**
     21         * If two equal commands are applied consecutively to the same set of
     22         * images, they are summed in order to reduce them to just one command.
     23         *
     24         * @param command
     25         */
    2026        public abstract void sum(MapillaryCommand command);
    21        
     27
     28        /**
     29         * Checks if the image has been modified, compairing with its original
     30         * values.
     31         */
    2232        public void checkModified() {
    2333                for (MapillaryAbstractImage image : images)
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/downloads/MapillaryImageInfoDownloaderThread.java

    r31259 r31264  
    3939                        JsonObject jsonobj = Json.createReader(br).readObject();
    4040                        if (!jsonobj.getBoolean("more"))
    41                                 ex.shutdownNow();
     41                                ex.shutdown();
    4242                        JsonArray jsonarr = jsonobj.getJsonArray("ims");
    4343                        JsonObject data;
     
    5353                                                        ((MapillaryImage) image).setCapturedAt(data
    5454                                                                        .getJsonNumber("captured_at").intValue());
    55                                                         ((MapillaryImage) image).setLocation(data.getString("location"));
     55                                                        ((MapillaryImage) image).setLocation(data
     56                                                                        .getString("location"));
    5657                                                }
    5758                                        }
  • applications/editors/josm/plugins/mapillary/src/org/openstreetmap/josm/plugins/mapillary/gui/MapillaryToggleDialog.java

    r31263 r31264  
    4545                ICachedLoaderListener, MapillaryDataListener {
    4646
     47        public final static int NORMAL_MODE = 0;
     48        public final static int SIGNAL_MODE = 1;
     49
    4750        public static MapillaryToggleDialog INSTANCE;
    4851
     
    6366
    6467        private int mode;
    65 
    66         public final static int NORMAL_MODE = 0;
    67         public final static int SIGNAL_MODE = 1;
    6868
    6969        private JPanel buttonsPanel;
     
    182182                                                c++;
    183183                                        }
    184                                         if (first >= i) {
     184                                        if (first >= i)
    185185                                                previousSignalButton.setEnabled(false);
    186                                         }
    187                                         if (last <= i) {
     186                                        if (last <= i)
    188187                                                nextSignalButton.setEnabled(false);
    189                                         }
    190188                                }
    191189
     
    277275        }
    278276
     277        /**
     278         * Action class to jump to the image following the red line.
     279         *
     280         * @author nokutu
     281         *
     282         */
    279283        class redAction extends AbstractAction {
    280284                public redAction() {
     
    294298        }
    295299
     300        /**
     301         * Action class to jump to the image following the blue line.
     302         *
     303         * @author nokutu
     304         *
     305         */
    296306        class blueAction extends AbstractAction {
    297307                public blueAction() {
     
    377387        }
    378388
     389        /**
     390         * Action class to jump to the next picture containing a signal.
     391         *
     392         * @author nokutu
     393         *
     394         */
    379395        class NextSignalAction extends AbstractAction {
    380396                public NextSignalAction() {
    381397                        putValue(NAME, tr("Next Signal"));
    382398                        putValue(SHORT_DESCRIPTION,
    383                                         tr("Jumps to the next picture that contains a siganl"));
     399                                        tr("Jumps to the next picture that contains a signal"));
    384400                }
    385401
     
    406422        }
    407423
     424        /**
     425         * Action class to jump to the previous picture containing a signal.
     426         *
     427         * @author nokutu
     428         *
     429         */
    408430        class PreviousSignalAction extends AbstractAction {
    409431                public PreviousSignalAction() {
    410432                        putValue(NAME, tr("Previous Signal"));
    411433                        putValue(SHORT_DESCRIPTION,
    412                                         tr("Jumps to the previous picture that contains a siganl"));
     434                                        tr("Jumps to the previous picture that contains a signal"));
    413435                }
    414436
Note: See TracChangeset for help on using the changeset viewer.