TrackRecordingService: cleanup.

This commit is contained in:
Dennis Guse
2020-04-13 16:51:57 +02:00
parent 2898e06246
commit 6a1fada22f
@@ -31,6 +31,7 @@ import de.dennisguse.opentracks.util.TrackPointUtils;
* Updater for {@link TrackStatistics}. * Updater for {@link TrackStatistics}.
* For updating track {@link TrackStatistics} as new {@link TrackPoint}s are added. * For updating track {@link TrackStatistics} as new {@link TrackPoint}s are added.
* NOTE: Some of the locations represent pause/resume separator. * NOTE: Some of the locations represent pause/resume separator.
* NOTE: Has still support for segments (at the moment unused).
* *
* @author Sandor Dornbush * @author Sandor Dornbush
* @author Rodrigo Damazio * @author Rodrigo Damazio
@@ -116,16 +117,16 @@ public class TrackStatisticsUpdater {
*/ */
public void addTrackPoint(TrackPoint trackPoint, int minRecordingDistance) { public void addTrackPoint(TrackPoint trackPoint, int minRecordingDistance) {
// Always update time // Always update time
updateTime(trackPoint.getLocation().getTime()); updateTime(trackPoint.getTime());
if (!LocationUtils.isValidLocation(trackPoint.getLocation())) { if (!LocationUtils.isValidLocation(trackPoint.getLocation())) {
// Either pause or resume marker // Either pause or resume marker
if (trackPoint.getLocation().getLatitude() == TrackPointsColumns.PAUSE_LATITUDE) { if (trackPoint.getLatitude() == TrackPointsColumns.PAUSE_LATITUDE) {
if (lastTrackPoint != null && lastMovingTrackPoint != null && lastTrackPoint != lastMovingTrackPoint) { if (lastTrackPoint != null && lastMovingTrackPoint != null && lastTrackPoint != lastMovingTrackPoint) {
currentSegment.addTotalDistance(lastMovingTrackPoint.distanceTo(lastTrackPoint)); currentSegment.addTotalDistance(lastMovingTrackPoint.distanceTo(lastTrackPoint));
} }
trackStatistics.merge(currentSegment); trackStatistics.merge(currentSegment);
} }
currentSegment = init(trackPoint.getLocation().getTime()); currentSegment = init(trackPoint.getTime());
lastTrackPoint = null; lastTrackPoint = null;
lastMovingTrackPoint = null; lastMovingTrackPoint = null;
elevationBuffer_m.reset(); elevationBuffer_m.reset();