Changeset 2931 in josm


Ignore:
Timestamp:
03.02.2010 09:48:51 (2 years ago)
Author:
bastiK
Message:

cleanup for geoimage code (mainly getters and setters)

Location:
trunk/src/org/openstreetmap/josm/gui/layer/geoimage
Files:
5 edited

Legend:

Unmodified
Added
Removed

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/CorrelateGpxWithImages.java

    r2907 r2931  
    22// Copyright 2007 by Christian Gallioz (aka khris78) 
    33// Parts of code from Geotagged plugin (by Rob Neild) 
    4 // and the core JOSM source code (by Immanuel Scholz and others) 
    54 
    65package org.openstreetmap.josm.gui.layer.geoimage; 
    … …  
    333332            imgList = new JList(new AbstractListModel() { 
    334333                public Object getElementAt(int i) { 
    335                     return yLayer.data.get(i).file.getName(); 
     334                    return yLayer.data.get(i).getFile().getName(); 
    336335                } 
    337336 
    … …  
    345344                public void valueChanged(ListSelectionEvent arg0) { 
    346345                    int index = imgList.getSelectedIndex(); 
    347                     imgDisp.setImage(yLayer.data.get(index).file); 
    348                     Date date = yLayer.data.get(index).time; 
     346                    imgDisp.setImage(yLayer.data.get(index).getFile()); 
     347                    Date date = yLayer.data.get(index).getExifTime(); 
    349348                    if (date != null) { 
    350349                        lbExifTime.setText(new SimpleDateFormat("dd/MM/yyyy HH:mm:ss").format(date)); 
    … …  
    649648                    return tr("No gpx selected"); 
    650649 
    651                 lastNumMatched = matchGpxTrack(dateImgLst, selGpx.data, (long) (timezone * 3600) + delta); 
     650                final long offset_ms = ((long) (timezone * 3600) + delta) * 1000; // in milliseconds 
     651                lastNumMatched = matchGpxTrack(dateImgLst, selGpx.data, offset_ms); 
    652652 
    653653                return trn("<html>Matched <b>{0}</b> of <b>{1}</b> photo to GPX track.</html>", 
    … …  
    746746                    Main.pref.put("geoimage.showThumbs", yLayer.useThumbs); 
    747747 
    748                     yLayer.useThumbs = cbShowThumbs.isSelected();//FIXME 
     748                    yLayer.useThumbs = cbShowThumbs.isSelected(); 
    749749                    yLayer.loadThumbs(); 
    750750 
    … …  
    955955 
    956956            // Init variables 
    957             long firstExifDate = imgs.get(0).time.getTime()/1000; 
     957            long firstExifDate = imgs.get(0).getExifTime().getTime()/1000; 
    958958 
    959959            long firstGPXDate = -1; 
    … …  
    10411041        ArrayList<ImageEntry> dateImgLst = new ArrayList<ImageEntry>(yLayer.data.size()); 
    10421042        for (ImageEntry e : yLayer.data) { 
    1043             if (e.time == null) { 
     1043            if (e.getExifTime() == null) { 
    10441044                continue; 
    10451045            } 
    10461046 
    1047             if (e.exifCoor != null) { 
     1047            if (e.getExifCoor() != null) { 
    10481048                if (!exif) { 
    10491049                    continue; 
    … …  
    10511051            } 
    10521052 
    1053             if (e.isTagged() && e.exifCoor == null) { 
     1053            if (e.isTagged() && e.getExifCoor() == null) { 
    10541054                if (!tagged) { 
    10551055                    continue; 
    … …  
    10621062        Collections.sort(dateImgLst, new Comparator<ImageEntry>() { 
    10631063            public int compare(ImageEntry arg0, ImageEntry arg1) { 
    1064                 return arg0.time.compareTo(arg1.time); 
     1064                return arg0.getExifTime().compareTo(arg1.getExifTime()); 
    10651065            } 
    10661066        }); 
    … …  
    10821082    } 
    10831083 
    1084     private int matchGpxTrack(ArrayList<ImageEntry> dateImgLst, GpxData selectedGpx, long offset) { 
     1084    /** 
     1085     * Match a list of photos to a gpx track with a given offset. 
     1086     * All images need a exifTime attribute and the List must be sorted according to these times. 
     1087     */ 
     1088    private int matchGpxTrack(ArrayList<ImageEntry> images, GpxData selectedGpx, long offset) { 
    10851089        int ret = 0; 
    10861090 
    … …  
    10901094            for (GpxTrackSegment segment : trk.getSegments()) { 
    10911095 
    1092                 long prevDateWp = 0; 
     1096                long prevWpTime = 0; 
    10931097                WayPoint prevWp = null; 
    10941098 
    10951099                for (WayPoint curWp : segment.getWayPoints()) { 
    10961100 
    1097                     String curDateWpStr = (String) curWp.attr.get("time"); 
    1098                     if (curDateWpStr != null) { 
     1101                    String curWpTimeStr = (String) curWp.attr.get("time"); 
     1102                    if (curWpTimeStr != null) { 
    10991103 
    11001104                        try { 
    1101                             long curDateWp = dateParser.parse(curDateWpStr).getTime()/1000 + offset; 
    1102                             ret += matchPoints(dateImgLst, prevWp, prevDateWp, curWp, curDateWp); 
     1105                            long curWpTime = dateParser.parse(curWpTimeStr).getTime() + offset; 
     1106                            ret += matchPoints(images, prevWp, prevWpTime, curWp, curWpTime, offset); 
    11031107 
    11041108                            prevWp = curWp; 
    1105                             prevDateWp = curDateWp; 
     1109                            prevWpTime = curWpTime; 
    11061110 
    11071111                        } catch(ParseException e) { 
    1108                             System.err.println("Error while parsing date \"" + curDateWpStr + '"'); 
     1112                            System.err.println("Error while parsing date \"" + curWpTimeStr + '"'); 
    11091113                            e.printStackTrace(); 
    11101114                            prevWp = null; 
    1111                             prevDateWp = 0; 
     1115                            prevWpTime = 0; 
    11121116                        } 
    11131117                    } else { 
    11141118                        prevWp = null; 
    1115                         prevDateWp = 0; 
     1119                        prevWpTime = 0; 
    11161120                    } 
    11171121                } 
    … …  
    11211125    } 
    11221126 
    1123     private int matchPoints(ArrayList<ImageEntry> dateImgLst, WayPoint prevWp, long prevDateWp, 
    1124             WayPoint curWp, long curDateWp) { 
     1127    private int matchPoints(ArrayList<ImageEntry> images, WayPoint prevWp, long prevWpTime, 
     1128            WayPoint curWp, long curWpTime, long offset) { 
    11251129        // Time between the track point and the previous one, 5 sec if first point, i.e. photos take 
    11261130        // 5 sec before the first track point can be assumed to be take at the starting position 
    1127         long interval = prevDateWp > 0 ? ((int)Math.abs(curDateWp - prevDateWp)) : 5; 
     1131        long interval = prevWpTime > 0 ? ((long)Math.abs(curWpTime - prevWpTime)) : 5*1000; 
    11281132        int ret = 0; 
    11291133 
    11301134        // i is the index of the timewise last photo that has the same or earlier EXIF time 
    1131         int i = getLastIndexOfListBefore(dateImgLst, curDateWp); 
     1135        int i = getLastIndexOfListBefore(images, curWpTime); 
    11321136 
    11331137        // no photos match 
    … …  
    11421146            double distance = prevWp.getCoor().greatCircleDistance(curWp.getCoor()); 
    11431147            // This is in km/h, 3.6 * m/s 
    1144             if (curDateWp > prevDateWp) { 
    1145                 speed = 3.6 * distance / (curDateWp - prevDateWp); 
     1148            if (curWpTime > prevWpTime) { 
     1149                speed = 3.6 * distance / (curWpTime - prevWpTime); 
    11461150            } 
    11471151            try { 
    … …  
    11561160        // First trackpoint, then interval is set to five seconds, i.e. photos up to five seconds 
    11571161        // before the first point will be geotagged with the starting point 
    1158         if(prevDateWp == 0 || curDateWp <= prevDateWp) { 
    1159             while(i >= 0 && (dateImgLst.get(i).time.getTime()/1000) <= curDateWp 
    1160                     && (dateImgLst.get(i).time.getTime()/1000) >= (curDateWp - interval)) { 
    1161                 if(dateImgLst.get(i).tmp.getPos() == null) { 
    1162                     dateImgLst.get(i).tmp.setCoor(curWp.getCoor()); 
    1163                     dateImgLst.get(i).tmp.setSpeed(speed); 
    1164                     dateImgLst.get(i).tmp.setElevation(curElevation); 
     1162        if(prevWpTime == 0 || curWpTime <= prevWpTime) { 
     1163            while (true) { 
     1164                if (i < 0) 
     1165                    break; 
     1166                final ImageEntry curImg = images.get(i); 
     1167                if (curImg.getExifTime().getTime() > curWpTime 
     1168                    || curImg.getExifTime().getTime() < curWpTime - interval) 
     1169                        break; 
     1170                if(curImg.tmp.getPos() == null) { 
     1171                    curImg.tmp.setPos(curWp.getCoor()); 
     1172                    curImg.tmp.setSpeed(speed); 
     1173                    curImg.tmp.setElevation(curElevation); 
     1174                    curImg.tmp.setGpsTime(new Date(curImg.getExifTime().getTime() - offset)); 
    11651175                    ret++; 
    11661176                } 
    … …  
    11721182        // This code gives a simple linear interpolation of the coordinates between current and 
    11731183        // previous track point assuming a constant speed in between 
    1174         long imgDate; 
    1175         while(i >= 0 && (imgDate = dateImgLst.get(i).time.getTime()/1000) >= prevDateWp) { 
    1176  
    1177             if(dateImgLst.get(i).tmp.getPos() == null) { 
     1184        while (true) { 
     1185            if (i < 0) 
     1186                break; 
     1187            ImageEntry curImg = images.get(i); 
     1188            long imgTime = curImg.getExifTime().getTime(); 
     1189            if (imgTime < prevWpTime) 
     1190                break; 
     1191 
     1192            if(curImg.tmp.getPos() == null) { 
    11781193                // The values of timeDiff are between 0 and 1, it is not seconds but a dimensionless 
    11791194                // variable 
    1180                 double timeDiff = (double)(imgDate - prevDateWp) / interval; 
    1181                 dateImgLst.get(i).tmp.setCoor(prevWp.getCoor().interpolate(curWp.getCoor(), timeDiff)); 
    1182                 dateImgLst.get(i).tmp.setSpeed(speed); 
    1183  
     1195                double timeDiff = (double)(imgTime - prevWpTime) / interval; 
     1196                curImg.tmp.setPos(prevWp.getCoor().interpolate(curWp.getCoor(), timeDiff)); 
     1197                curImg.tmp.setSpeed(speed); 
    11841198                if (curElevation != null && prevElevation != null) { 
    1185                     dateImgLst.get(i).setElevation(prevElevation + (curElevation - prevElevation) * timeDiff); 
    1186                 } 
     1199                    curImg.setElevation(prevElevation + (curElevation - prevElevation) * timeDiff); 
     1200                } 
     1201                curImg.tmp.setGpsTime(new Date(curImg.getExifTime().getTime() - offset)); 
    11871202 
    11881203                ret++; 
    … …  
    11931208    } 
    11941209 
    1195     private int getLastIndexOfListBefore(ArrayList<ImageEntry> dateImgLst, long searchedDate) { 
    1196         int lstSize= dateImgLst.size(); 
     1210    private int getLastIndexOfListBefore(ArrayList<ImageEntry> images, long searchedTime) { 
     1211        int lstSize= images.size(); 
    11971212 
    11981213        // No photos or the first photo taken is later than the search period 
    1199         if(lstSize == 0 || searchedDate < dateImgLst.get(0).time.getTime()/1000) 
     1214        if(lstSize == 0 || searchedTime < images.get(0).getExifTime().getTime()) 
    12001215            return -1; 
    12011216 
    12021217        // The search period is later than the last photo 
    1203         if (searchedDate > dateImgLst.get(lstSize - 1).time.getTime() / 1000) 
     1218        if (searchedTime > images.get(lstSize - 1).getExifTime().getTime()) 
    12041219            return lstSize-1; 
    12051220 
    … …  
    12101225        while (endIndex - startIndex > 1) { 
    12111226            curIndex= (endIndex + startIndex) / 2; 
    1212             if (searchedDate > dateImgLst.get(curIndex).time.getTime()/1000) { 
     1227            if (searchedTime > images.get(curIndex).getExifTime().getTime()) { 
    12131228                startIndex= curIndex; 
    12141229            } else { 
    … …  
    12161231            } 
    12171232        } 
    1218         if (searchedDate < dateImgLst.get(endIndex).time.getTime()/1000) 
     1233        if (searchedTime < images.get(endIndex).getExifTime().getTime()) 
    12191234            return startIndex; 
    12201235 
    12211236        // This final loop is to check if photos with the exact same EXIF time follows 
    1222         while ((endIndex < (lstSize-1)) && (dateImgLst.get(endIndex).time.getTime() 
    1223                 == dateImgLst.get(endIndex + 1).time.getTime())) { 
     1237        while ((endIndex < (lstSize-1)) && (images.get(endIndex).getExifTime().getTime() 
     1238                == images.get(endIndex + 1).getExifTime().getTime())) { 
    12241239            endIndex++; 
    12251240        } 

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/GeoImageLayer.java

    r2909 r2931  
    146146 
    147147                try { 
    148                     e.time = ExifReader.readTime(f); 
     148                    e.setExifTime(ExifReader.readTime(f)); 
    149149                } catch (ParseException e1) { 
    150                     e.time = null; 
    151                 } 
    152                 e.file = f; 
     150                    e.setExifTime(null); 
     151                } 
     152                e.setFile(f); 
    153153                extractExif(e); 
    154154                data.add(e); 
    … …  
    313313        entries.add(new JSeparator()); 
    314314        entries.add(correlateItem); 
     315        if (!menuAdditions.isEmpty()) { 
     316            entries.add(new JSeparator()); 
     317        } 
    315318        for (LayerMenuAddition addition : menuAdditions) { 
    316319            entries.add(addition.getComponent(this)); 
    … …  
    365368            for (int i = data.size() - 2; i >= 0; i--) { 
    366369                cur = data.get(i); 
    367                 if (cur.file.equals(prev.file)) { 
     370                if (cur.getFile().equals(prev.getFile())) { 
    368371                    data.remove(i); 
    369372                } else { 
    … …  
    505508            double lon, lat; 
    506509 
    507             Metadata metadata = JpegMetadataReader.readMetadata(e.file); 
     510            Metadata metadata = JpegMetadataReader.readMetadata(e.getFile()); 
    508511            Directory dir = metadata.getDirectory(GpsDirectory.class); 
    509512 
    510513            // longitude 
    511514 
    512             Rational[] components = dir 
    513             .getRationalArray(GpsDirectory.TAG_GPS_LONGITUDE); 
     515            Rational[] components = dir.getRationalArray(GpsDirectory.TAG_GPS_LONGITUDE); 
    514516 
    515517            deg = components[0].intValue(); 
    … …  
    539541            // Store values 
    540542 
    541             e.setCoor(new LatLon(lat, lon)); 
    542             e.exifCoor = e.getPos(); 
     543            e.setExifCoor(new LatLon(lat, lon)); 
     544            e.setPos(e.getExifCoor()); 
    543545 
    544546        } catch (CompoundException p) { 
    545             e.exifCoor = null; 
     547            e.setExifCoor(null); 
    546548            e.setPos(null); 
    547549        } 
    … …  
    607609            .setButtonIcons(new String[] {"cancel.png", "dialogs/delete.png"}) 
    608610            .setContent(new JLabel(tr("<html><h3>Delete the file {0} from disk?<p>The image file will be permanently lost!</h3></html>" 
    609                     ,toDelete.file.getName()), ImageProvider.get("dialogs/geoimage/deletefromdisk"),SwingConstants.LEFT)) 
     611                    ,toDelete.getFile().getName()), ImageProvider.get("dialogs/geoimage/deletefromdisk"),SwingConstants.LEFT)) 
    610612                    .toggleEnable("geoimage.deleteimagefromdisk") 
    611613                    .setCancelButton(1) 
    … …  
    626628                } 
    627629 
    628                 if (toDelete.file.delete()) { 
    629                     System.out.println("File "+toDelete.file.toString()+" deleted. "); 
     630                if (toDelete.getFile().delete()) { 
     631                    System.out.println("File "+toDelete.getFile().toString()+" deleted. "); 
    630632                } else { 
    631633                    JOptionPane.showMessageDialog( 

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageEntry.java

    r2904 r2931  
    1818 
    1919final public class ImageEntry implements Comparable<ImageEntry>, Cloneable { 
    20     File file; 
    21     Date time; 
    22     LatLon exifCoor; 
     20    private File file; 
     21    private LatLon exifCoor; 
     22    private Date exifTime; 
     23    Image thumbnail; 
    2324 
     25    /** The following values are computed from the correlation with the gpx track */ 
    2426    private CachedLatLon pos; 
    2527    /** Speed in kilometer per second */ 
    … …  
    2729    /** Elevation (altitude) in meters */ 
    2830    private Double elevation; 
    29  
    30     Image thumbnail; 
     31    /** The time after correlation with a gpx track */ 
     32    private Date gpsTime; 
    3133 
    3234    /** 
    … …  
    4042    ImageEntry tmp; 
    4143 
     44    /** 
     45     * getter methods that refer to the temporary value 
     46     */ 
    4247    public CachedLatLon getPos() { 
    4348        if (tmp != null) 
    … …  
    5560        return elevation; 
    5661    } 
     62    public Date getGpsTime() { 
     63        if (tmp != null) 
     64            return tmp.gpsTime; 
     65        return gpsTime; 
     66    } 
     67 
     68    /** 
     69     * other getter methods 
     70     */ 
     71    public File getFile() { 
     72        return file; 
     73    } 
     74    public Date getExifTime() { 
     75        return exifTime; 
     76    } 
     77    LatLon getExifCoor() { 
     78        return exifCoor; 
     79    } 
     80    /** 
     81     * setter methods 
     82     */ 
    5783    public void setPos(CachedLatLon pos) { 
    5884        this.pos = pos; 
     85    } 
     86    public void setPos(LatLon pos) { 
     87        this.pos = new CachedLatLon(pos); 
    5988    } 
    6089    public void setSpeed(Double speed) { 
    … …  
    6493        this.elevation = elevation; 
    6594    } 
    66      
    67     public File getFile() { 
    68         return file; 
     95    void setFile(File file) { 
     96        this.file = file; 
     97    } 
     98    void setExifTime(Date exifTime) { 
     99        this.exifTime = exifTime; 
     100    } 
     101    void setGpsTime(Date gpsTime) { 
     102        this.gpsTime = gpsTime; 
     103    } 
     104    void setExifCoor(LatLon exifCoor) { 
     105        this.exifCoor = exifCoor; 
    69106    } 
    70107 
    … …  
    80117    } 
    81118 
    82     public void setCoor(LatLon latlon) 
    83     { 
    84         pos = new CachedLatLon(latlon); 
    85     } 
    86  
    87119    public int compareTo(ImageEntry image) { 
    88         if (time != null && image.time != null) 
    89             return time.compareTo(image.time); 
    90         else if (time == null && image.time == null) 
     120        if (exifTime != null && image.exifTime != null) 
     121            return exifTime.compareTo(image.exifTime); 
     122        else if (exifTime == null && image.exifTime == null) 
    91123            return 0; 
    92         else if (time == null) 
     124        else if (exifTime == null) 
    93125            return -1; 
    94126        else 
    … …  
    96128    } 
    97129 
    98     public void applyTmp() { 
    99         if (tmp != null) { 
    100             pos = tmp.pos; 
    101             speed = tmp.speed; 
    102             elevation = tmp.elevation; 
    103             tmp = null; 
    104         } 
    105     } 
     130    /** 
     131     * Make a fresh copy and save it in the temporary variable. 
     132     */ 
    106133    public void cleanTmp() { 
    107134        tmp = clone(); 
    … …  
    110137    } 
    111138 
     139    /** 
     140     * Copy the values from the temporary variable to the main instance. 
     141     */ 
     142    public void applyTmp() { 
     143        if (tmp != null) { 
     144            pos = tmp.pos; 
     145            speed = tmp.speed; 
     146            elevation = tmp.elevation; 
     147            gpsTime = tmp.gpsTime; 
     148            tmp = null; 
     149        } 
     150    } 
     151 
     152    /** 
     153     * If it has been tagged i.e. matched to a gpx track or retrieved lat/lon from exif 
     154     */ 
    112155    public boolean isTagged() { 
    113156        return pos != null; 
    … …  
    115158 
    116159    /** 
    117      * only partial info 
     160     * String representation. (only partial info) 
    118161     */ 
    119162    @Override 

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ImageViewerDialog.java

    r2909 r2931  
    1414import java.awt.GridBagLayout; 
    1515import java.awt.event.ActionEvent; 
    16 import java.awt.event.ActionListener; 
    1716import java.awt.event.KeyEvent; 
    1817import java.awt.event.WindowEvent; 
    … …  
    238237 
    239238        if (entry != null) { 
    240             imgDisplay.setImage(entry.file); 
    241             setTitle("Geotagged Images" + (entry.file != null ? " - " + entry.file.getName() : "")); 
    242             StringBuffer osd = new StringBuffer(entry.file != null ? entry.file.getName() : ""); 
     239            imgDisplay.setImage(entry.getFile()); 
     240            setTitle("Geotagged Images" + (entry.getFile() != null ? " - " + entry.getFile().getName() : "")); 
     241            StringBuffer osd = new StringBuffer(entry.getFile() != null ? entry.getFile().getName() : ""); 
    243242            if (entry.getElevation() != null) { 
    244243                osd.append(tr("\nAltitude: {0} m", entry.getElevation().longValue())); 
    … …  
    250249            //    osd.append(tr("\nlat: {0}, lon: {1}", Double.toString(entry.getPos().lat()), Double.toString(entry.getPos().lon()))); 
    251250            //} 
     251            //osd.append(tr("\nfile mtime: {0}", Long.toString(entry.getFile().lastModified()))); 
     252            //osd.append(tr("\nImage exif time: {0}", Long.toString(entry.getExifTime().getTime()))); 
     253            //if (entry.getGpsTime() != null) { 
     254            //    osd.append(tr("\nImage gps time: {0}", Long.toString(entry.getGpsTime().getTime()))); 
     255            //} 
     256             
    252257            imgDisplay.setOsdText(osd.toString()); 
    253258        } else { 

  • trunk/src/org/openstreetmap/josm/gui/layer/geoimage/ThumbsLoader.java

    r2662 r2931  
    7272 
    7373        private BufferedImage loadThumb(ImageEntry entry) { 
    74             final String cacheIdent = entry.file.toString()+":"+maxSize; 
     74            final String cacheIdent = entry.getFile().toString()+":"+maxSize; 
    7575 
    7676            if (!cacheOff) { 
    … …  
    8282            } 
    8383 
    84             Image img = Toolkit.getDefaultToolkit().createImage(entry.file.getPath()); 
     84            Image img = Toolkit.getDefaultToolkit().createImage(entry.getFile().getPath()); 
    8585            tracker.addImage(img, 0); 
    8686            try { 
Note: See TracChangeset for help on using the changeset viewer.