From ce21fe356967710aee3b8f3e4ba52f8aa7ecc89b Mon Sep 17 00:00:00 2001 From: Dennis Guse Date: Wed, 15 Nov 2023 18:41:13 +0100 Subject: [PATCH] Cleanup: renamed SensorData* to Aggregator*. --- .../io/file/importer/ExportImportTest.java | 21 +++++++------- .../sensorData/SensorDataCyclingTest.java | 16 +++++------ .../TrackRecordingServiceRecordingTest.java | 8 +++--- .../BluetoothHandlerBarometricPressure.java | 4 +-- .../BluetoothHandlerCyclingCadence.java | 6 ++-- .../BluetoothHandlerCyclingDistanceSpeed.java | 6 ++-- .../BluetoothHandlerManagerCyclingPower.java | 6 ++-- .../BluetoothHandlerManagerHeartRate.java | 6 ++-- ...luetoothHandlerRunningSpeedAndCadence.java | 6 ++-- .../opentracks/sensors/SensorManager.java | 10 +++---- .../{SensorData.java => Aggregator.java} | 6 ++-- ...nce.java => AggregatorCyclingCadence.java} | 8 +++--- ...va => AggregatorCyclingDistanceSpeed.java} | 8 +++--- ...Power.java => AggregatorCyclingPower.java} | 6 ++-- ...eartRate.java => AggregatorHeartRate.java} | 6 ++-- ...ataRunning.java => AggregatorRunning.java} | 8 +++--- .../sensors/sensorData/SensorDataSet.java | 28 +++++++++---------- .../sensorData/SensorHandlerInterface.java | 2 +- 18 files changed, 81 insertions(+), 80 deletions(-) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorData.java => Aggregator.java} (91%) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorDataCyclingCadence.java => AggregatorCyclingCadence.java} (87%) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorDataCyclingDistanceSpeed.java => AggregatorCyclingDistanceSpeed.java} (89%) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorDataCyclingPower.java => AggregatorCyclingPower.java} (70%) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorDataHeartRate.java => AggregatorHeartRate.java} (72%) rename src/main/java/de/dennisguse/opentracks/sensors/sensorData/{SensorDataRunning.java => AggregatorRunning.java} (84%) diff --git a/src/androidTest/java/de/dennisguse/opentracks/io/file/importer/ExportImportTest.java b/src/androidTest/java/de/dennisguse/opentracks/io/file/importer/ExportImportTest.java index 48af105fb3..3d8da5ae02 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/io/file/importer/ExportImportTest.java +++ b/src/androidTest/java/de/dennisguse/opentracks/io/file/importer/ExportImportTest.java @@ -59,10 +59,11 @@ import de.dennisguse.opentracks.io.file.TrackFileFormat; import de.dennisguse.opentracks.io.file.exporter.TrackExporter; import de.dennisguse.opentracks.sensors.AltitudeSumManager; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingCadence; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingDistanceSpeed; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingPower; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataHeartRate; +import de.dennisguse.opentracks.sensors.sensorData.Aggregator; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingCadence; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingDistanceSpeed; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingPower; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorHeartRate; import de.dennisguse.opentracks.sensors.sensorData.SensorDataSet; import de.dennisguse.opentracks.services.TrackRecordingService; import de.dennisguse.opentracks.services.handlers.TrackPointCreator; @@ -71,7 +72,7 @@ /** * Export a track to {@link TrackFileFormat} and verify that the import is identical. *

- * Note: those tests are affected by {@link de.dennisguse.opentracks.sensors.sensorData.SensorData}.isRecent(). + * Note: those tests are affected by {@link Aggregator}.isRecent(). * If the test device is too slow (like in a CI) these are likely to fail as the sensor data will be omitted from actual. */ @RunWith(AndroidJUnit4.class) @@ -549,22 +550,22 @@ private void mockBLESensorData(TrackPointCreator trackPointCreator, Float speed, SensorDataSet sensorDataSet = new SensorDataSet(); - sensorDataSet.cyclingPower = Mockito.mock(SensorDataCyclingPower.class); + sensorDataSet.cyclingPower = Mockito.mock(AggregatorCyclingPower.class); Mockito.when( sensorDataSet.cyclingPower.hasValue()).thenReturn(true); Mockito.when(sensorDataSet.cyclingPower.getValue()).thenReturn(Power.of(power)); - sensorDataSet.heartRate = Mockito.mock(SensorDataHeartRate.class); + sensorDataSet.heartRate = Mockito.mock(AggregatorHeartRate.class); Mockito.when(sensorDataSet.heartRate.getValue()).thenReturn(HeartRate.of(heartRate)); - SensorDataCyclingCadence cyclingCadence = Mockito.mock(SensorDataCyclingCadence.class); + AggregatorCyclingCadence cyclingCadence = Mockito.mock(AggregatorCyclingCadence.class); Mockito.when(cyclingCadence.hasValue()).thenReturn(true); Mockito.when(cyclingCadence.getValue()).thenReturn(Cadence.of(cadence)); sensorDataSet.cyclingCadence = cyclingCadence; if (distance != null && speed != null) { - SensorDataCyclingDistanceSpeed distanceSpeed = Mockito.mock(SensorDataCyclingDistanceSpeed.class); + AggregatorCyclingDistanceSpeed distanceSpeed = Mockito.mock(AggregatorCyclingDistanceSpeed.class); Mockito.when(distanceSpeed.hasValue()).thenReturn(true); - Mockito.when(distanceSpeed.getValue()).thenReturn(new SensorDataCyclingDistanceSpeed.Data(null, distance, Speed.of(speed))); + Mockito.when(distanceSpeed.getValue()).thenReturn(new AggregatorCyclingDistanceSpeed.Data(null, distance, Speed.of(speed))); sensorDataSet.cyclingDistanceSpeed = distanceSpeed; } diff --git a/src/androidTest/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingTest.java b/src/androidTest/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingTest.java index ac98399e59..ca3e18949f 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingTest.java +++ b/src/androidTest/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingTest.java @@ -20,7 +20,7 @@ public class SensorDataCyclingTest { @Test public void compute_cadence_1() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(1, 1024))); @@ -32,7 +32,7 @@ public void compute_cadence_1() { @Test public void compute_cadence_2() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(1, 6184))); @@ -44,7 +44,7 @@ public void compute_cadence_2() { @Test public void compute_cadence_sameCount() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(1, 1024))); @@ -57,7 +57,7 @@ public void compute_cadence_sameCount() { @Test public void compute_cadence_sameTime() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(1, 1024))); @@ -69,7 +69,7 @@ public void compute_cadence_sameTime() { @Test public void compute_cadence_rollOverTime() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(1, UintUtils.UINT16_MAX - 1024))); @@ -82,7 +82,7 @@ public void compute_cadence_rollOverTime() { @Test @Deprecated public void compute_cadence_rollOverCount() { - SensorDataCyclingCadence current = new SensorDataCyclingCadence("", ""); + AggregatorCyclingCadence current = new AggregatorCyclingCadence("", ""); // when current.add(new Raw<>(new BluetoothHandlerCyclingCadence.CrankData(UintUtils.UINT32_MAX - 1, 1024))); @@ -96,7 +96,7 @@ public void compute_cadence_rollOverCount() { @Test public void compute_speed() { - SensorDataCyclingDistanceSpeed current = new SensorDataCyclingDistanceSpeed("", ""); + AggregatorCyclingDistanceSpeed current = new AggregatorCyclingDistanceSpeed("", ""); current.setWheelCircumference(Distance.ofMM(2150)); // when @@ -111,7 +111,7 @@ public void compute_speed() { @Test @Deprecated public void compute_speed_rollOverCount() { - SensorDataCyclingDistanceSpeed current = new SensorDataCyclingDistanceSpeed("", ""); + AggregatorCyclingDistanceSpeed current = new AggregatorCyclingDistanceSpeed("", ""); current.setWheelCircumference(Distance.ofMM(2000)); // when diff --git a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceRecordingTest.java b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceRecordingTest.java index 9d7a50106a..6a0b045a71 100644 --- a/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceRecordingTest.java +++ b/src/androidTest/java/de/dennisguse/opentracks/services/TrackRecordingServiceRecordingTest.java @@ -37,9 +37,9 @@ import de.dennisguse.opentracks.sensors.AltitudeSumManager; import de.dennisguse.opentracks.sensors.BluetoothHandlerRunningSpeedAndCadence; import de.dennisguse.opentracks.sensors.SensorManager; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorHeartRate; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorRunning; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataHeartRate; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataRunning; import de.dennisguse.opentracks.services.handlers.TrackPointCreator; import de.dennisguse.opentracks.settings.PreferencesUtils; import de.dennisguse.opentracks.stats.TrackStatistics; @@ -255,7 +255,7 @@ public void testRecording_blesensor_only_no_distance() { Track.Id trackId = service.startNewTrack(); trackPointCreator.getSensorManager().setAltitudeSumManager(altitudeSumManager); SensorManager sensorManager = trackPointCreator.getSensorManager(); - sensorManager.sensorDataSet.add(new SensorDataHeartRate("", "")); + sensorManager.sensorDataSet.add(new AggregatorHeartRate("", "")); // when String sensor1 = "2020-02-02T02:02:03Z"; trackPointCreator.setClock(sensor1); @@ -592,7 +592,7 @@ public void testRecording_gpsAndSensor_gpsIdleMoving_sensorMoving() { Track.Id trackId = service.startNewTrack(); trackPointCreator.getSensorManager().setAltitudeSumManager(altitudeSumManager); SensorManager sensorManager = trackPointCreator.getSensorManager(); - sensorManager.sensorDataSet.add(new SensorDataRunning("", "")); + sensorManager.sensorDataSet.add(new AggregatorRunning("", "")); // when String sensor1 = "2020-02-02T02:02:03Z"; diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerBarometricPressure.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerBarometricPressure.java index 0ba7811bc8..44adebbc95 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerBarometricPressure.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerBarometricPressure.java @@ -6,7 +6,7 @@ import java.util.UUID; import de.dennisguse.opentracks.data.models.AtmosphericPressure; -import de.dennisguse.opentracks.sensors.sensorData.SensorData; +import de.dennisguse.opentracks.sensors.sensorData.Aggregator; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerBarometricPressure implements SensorHandlerInterface { @@ -22,7 +22,7 @@ public List getServices() { } @Override - public SensorData createEmptySensorData(String address) { + public Aggregator createEmptySensorData(String address) { return null; //TODO } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingCadence.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingCadence.java index c0757e1ac9..74ad56d337 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingCadence.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingCadence.java @@ -6,8 +6,8 @@ import java.util.List; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingCadence; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingCadence; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerCyclingCadence implements SensorHandlerInterface { @@ -24,8 +24,8 @@ public List getServices() { } @Override - public SensorDataCyclingCadence createEmptySensorData(String address) { - return new SensorDataCyclingCadence(address); + public AggregatorCyclingCadence createEmptySensorData(String address) { + return new AggregatorCyclingCadence(address); } @Override diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingDistanceSpeed.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingDistanceSpeed.java index f9c64a1e18..a0cb87a483 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingDistanceSpeed.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerCyclingDistanceSpeed.java @@ -9,8 +9,8 @@ import java.util.List; import java.util.UUID; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingDistanceSpeed; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingDistanceSpeed; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerCyclingDistanceSpeed implements SensorHandlerInterface { @@ -26,8 +26,8 @@ public List getServices() { } @Override - public SensorDataCyclingDistanceSpeed createEmptySensorData(String address) { - return new SensorDataCyclingDistanceSpeed(address); + public AggregatorCyclingDistanceSpeed createEmptySensorData(String address) { + return new AggregatorCyclingDistanceSpeed(address); } @Override diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerCyclingPower.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerCyclingPower.java index 1605dec081..149ec92074 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerCyclingPower.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerCyclingPower.java @@ -9,8 +9,8 @@ import java.util.UUID; import de.dennisguse.opentracks.data.models.Power; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorCyclingPower; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataCyclingPower; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerManagerCyclingPower implements SensorHandlerInterface { @@ -26,8 +26,8 @@ public List getServices() { } @Override - public SensorDataCyclingPower createEmptySensorData(String address) { - return new SensorDataCyclingPower(address); + public AggregatorCyclingPower createEmptySensorData(String address) { + return new AggregatorCyclingPower(address); } @Override diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerHeartRate.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerHeartRate.java index 1af0d581fe..10ace7bc0a 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerHeartRate.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerManagerHeartRate.java @@ -9,8 +9,8 @@ import java.util.UUID; import de.dennisguse.opentracks.data.models.HeartRate; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorHeartRate; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataHeartRate; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerManagerHeartRate implements SensorHandlerInterface { @@ -36,8 +36,8 @@ public List getServices() { } @Override - public SensorDataHeartRate createEmptySensorData(String address) { - return new SensorDataHeartRate(address); + public AggregatorHeartRate createEmptySensorData(String address) { + return new AggregatorHeartRate(address); } @Override diff --git a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerRunningSpeedAndCadence.java b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerRunningSpeedAndCadence.java index ffc293ca5e..1a7f8a6f4b 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerRunningSpeedAndCadence.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/BluetoothHandlerRunningSpeedAndCadence.java @@ -11,8 +11,8 @@ import de.dennisguse.opentracks.data.models.Cadence; import de.dennisguse.opentracks.data.models.Distance; import de.dennisguse.opentracks.data.models.Speed; +import de.dennisguse.opentracks.sensors.sensorData.AggregatorRunning; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorDataRunning; import de.dennisguse.opentracks.sensors.sensorData.SensorHandlerInterface; public class BluetoothHandlerRunningSpeedAndCadence implements SensorHandlerInterface { @@ -29,8 +29,8 @@ public List getServices() { } @Override - public SensorDataRunning createEmptySensorData(String address) { - return new SensorDataRunning(address); + public AggregatorRunning createEmptySensorData(String address) { + return new AggregatorRunning(address); } @Override diff --git a/src/main/java/de/dennisguse/opentracks/sensors/SensorManager.java b/src/main/java/de/dennisguse/opentracks/sensors/SensorManager.java index 6fd6a768fc..1f1535a6fa 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/SensorManager.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/SensorManager.java @@ -9,8 +9,8 @@ import androidx.annotation.VisibleForTesting; import de.dennisguse.opentracks.data.models.TrackPoint; +import de.dennisguse.opentracks.sensors.sensorData.Aggregator; import de.dennisguse.opentracks.sensors.sensorData.Raw; -import de.dennisguse.opentracks.sensors.sensorData.SensorData; import de.dennisguse.opentracks.sensors.sensorData.SensorDataSet; import de.dennisguse.opentracks.services.handlers.GPSManager; import de.dennisguse.opentracks.services.handlers.TrackPointCreator; @@ -28,7 +28,7 @@ public class SensorManager implements SharedPreferences.OnSharedPreferenceChange private final SensorDataChangedObserver listener = new SensorDataChangedObserver() { @Override - public void onConnect(SensorData sensorData) { + public void onConnect(Aggregator sensorData) { sensorDataSet.add(sensorData); observer.onChange(new SensorDataSet(sensorDataSet)); } @@ -40,7 +40,7 @@ public void onChange(Raw sensorData) { } @Override - public void onDisconnect(SensorData sensorData) { + public void onDisconnect(Aggregator sensorData) { sensorDataSet.remove(sensorData); observer.onChange(new SensorDataSet(sensorDataSet)); } @@ -128,9 +128,9 @@ public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, @Null public interface SensorDataChangedObserver { - void onConnect(SensorData sensorData); + void onConnect(Aggregator sensorData); void onChange(Raw sensorData); - void onDisconnect(SensorData sensorData); + void onDisconnect(Aggregator sensorData); } } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorData.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/Aggregator.java similarity index 91% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorData.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/Aggregator.java index aa59c24d78..82ae79c2fe 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorData.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/Aggregator.java @@ -6,7 +6,7 @@ import de.dennisguse.opentracks.sensors.BluetoothRemoteSensorManager; -public abstract class SensorData { +public abstract class Aggregator { protected Raw previous; @@ -15,11 +15,11 @@ public abstract class SensorData { private final String sensorAddress; private final String sensorName; - SensorData(String sensorAddress) { + Aggregator(String sensorAddress) { this(sensorAddress, null); } - SensorData(String sensorAddress, String sensorName) { + Aggregator(String sensorAddress, String sensorName) { this.sensorAddress = sensorAddress; this.sensorName = sensorName; } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingCadence.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingCadence.java similarity index 87% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingCadence.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingCadence.java index a1ab8c4ba5..61a3483549 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingCadence.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingCadence.java @@ -10,15 +10,15 @@ import de.dennisguse.opentracks.sensors.BluetoothHandlerCyclingCadence; import de.dennisguse.opentracks.sensors.UintUtils; -public class SensorDataCyclingCadence extends SensorData { +public class AggregatorCyclingCadence extends Aggregator { - private final String TAG = SensorDataCyclingCadence.class.getSimpleName(); + private final String TAG = AggregatorCyclingCadence.class.getSimpleName(); - public SensorDataCyclingCadence(String sensorAddress) { + public AggregatorCyclingCadence(String sensorAddress) { super(sensorAddress); } - public SensorDataCyclingCadence(String sensorAddress, String sensorName) { + public AggregatorCyclingCadence(String sensorAddress, String sensorName) { super(sensorAddress, sensorName); } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingDistanceSpeed.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingDistanceSpeed.java similarity index 89% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingDistanceSpeed.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingDistanceSpeed.java index 271323e006..71374c0192 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingDistanceSpeed.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingDistanceSpeed.java @@ -11,17 +11,17 @@ import de.dennisguse.opentracks.sensors.BluetoothHandlerCyclingDistanceSpeed; import de.dennisguse.opentracks.sensors.UintUtils; -public class SensorDataCyclingDistanceSpeed extends SensorData { +public class AggregatorCyclingDistanceSpeed extends Aggregator { - private final String TAG = SensorDataCyclingDistanceSpeed.class.getSimpleName(); + private final String TAG = AggregatorCyclingDistanceSpeed.class.getSimpleName(); private Distance wheelCircumference; - public SensorDataCyclingDistanceSpeed(String sensorAddress) { + public AggregatorCyclingDistanceSpeed(String sensorAddress) { super(sensorAddress); } - public SensorDataCyclingDistanceSpeed(String sensorAddress, String sensorName) { + public AggregatorCyclingDistanceSpeed(String sensorAddress, String sensorName) { super(sensorAddress, sensorName); } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingPower.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingPower.java similarity index 70% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingPower.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingPower.java index 5a6fa125db..e60e08ee6d 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataCyclingPower.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorCyclingPower.java @@ -4,13 +4,13 @@ import de.dennisguse.opentracks.data.models.Power; -public class SensorDataCyclingPower extends SensorData { +public class AggregatorCyclingPower extends Aggregator { - public SensorDataCyclingPower(String address) { + public AggregatorCyclingPower(String address) { super(address); } - public SensorDataCyclingPower(String name, String address) { + public AggregatorCyclingPower(String name, String address) { super(name, address); } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataHeartRate.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorHeartRate.java similarity index 72% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataHeartRate.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorHeartRate.java index 903c722322..68879cb590 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataHeartRate.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorHeartRate.java @@ -4,13 +4,13 @@ import de.dennisguse.opentracks.data.models.HeartRate; -public class SensorDataHeartRate extends SensorData { +public class AggregatorHeartRate extends Aggregator { - public SensorDataHeartRate(String address) { + public AggregatorHeartRate(String address) { super(address); } - public SensorDataHeartRate(String name, String address) { + public AggregatorHeartRate(String name, String address) { super(name, address); } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataRunning.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorRunning.java similarity index 84% rename from src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataRunning.java rename to src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorRunning.java index 0025ea8671..425f20705c 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataRunning.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/AggregatorRunning.java @@ -10,15 +10,15 @@ /** * Provides cadence in rpm and speed in milliseconds from Bluetooth LE Running Speed and Cadence sensors. */ -public final class SensorDataRunning extends SensorData { +public final class AggregatorRunning extends Aggregator { - private static final String TAG = SensorDataRunning.class.getSimpleName(); + private static final String TAG = AggregatorRunning.class.getSimpleName(); - public SensorDataRunning(String sensorAddress) { + public AggregatorRunning(String sensorAddress) { super(sensorAddress); } - public SensorDataRunning(String sensorAddress, String sensorName) { + public AggregatorRunning(String sensorAddress, String sensorName) { super(sensorAddress, sensorName); } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataSet.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataSet.java index e426a19635..ae7cc254b9 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataSet.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorDataSet.java @@ -22,19 +22,19 @@ public final class SensorDataSet { private static final String TAG = SensorDataSet.class.getSimpleName(); @VisibleForTesting - public SensorDataHeartRate heartRate; + public AggregatorHeartRate heartRate; @VisibleForTesting - public SensorDataCyclingCadence cyclingCadence; + public AggregatorCyclingCadence cyclingCadence; @VisibleForTesting - public SensorDataCyclingDistanceSpeed cyclingDistanceSpeed; + public AggregatorCyclingDistanceSpeed cyclingDistanceSpeed; @VisibleForTesting - public SensorDataCyclingPower cyclingPower; + public AggregatorCyclingPower cyclingPower; @VisibleForTesting - public SensorDataRunning runningDistanceSpeedCadence; + public AggregatorRunning runningDistanceSpeedCadence; public SensorDataSet() { } @@ -79,11 +79,11 @@ public Pair getSpeed() { return null; } - public SensorDataCyclingPower getCyclingPower() { + public AggregatorCyclingPower getCyclingPower() { return cyclingPower; } - public void add(@NonNull SensorData data) { + public void add(@NonNull Aggregator data) { set(data, data); } @@ -117,7 +117,7 @@ public void update(@NonNull Raw data) { throw new UnsupportedOperationException(data.getClass().getCanonicalName()); } - public void remove(@NonNull SensorData type) { + public void remove(@NonNull Aggregator type) { set(type, null); } @@ -163,24 +163,24 @@ public void reset() { if (runningDistanceSpeedCadence != null) runningDistanceSpeedCadence.reset(); } - private void set(@NonNull SensorData type, @Nullable SensorData sensorData) { - if (type instanceof SensorDataHeartRate hr) { + private void set(@NonNull Aggregator type, @Nullable Aggregator sensorData) { + if (type instanceof AggregatorHeartRate hr) { heartRate = hr; return; } - if (type instanceof SensorDataCyclingCadence cc) { + if (type instanceof AggregatorCyclingCadence cc) { cyclingCadence = cc; return; } - if (type instanceof SensorDataCyclingDistanceSpeed ds) { + if (type instanceof AggregatorCyclingDistanceSpeed ds) { cyclingDistanceSpeed = ds; return; } - if (type instanceof SensorDataCyclingPower cp) { + if (type instanceof AggregatorCyclingPower cp) { cyclingPower = cp; return; } - if (type instanceof SensorDataRunning rr) { + if (type instanceof AggregatorRunning rr) { runningDistanceSpeedCadence = rr; return; } diff --git a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorHandlerInterface.java b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorHandlerInterface.java index 9893ae556d..89e409f5a1 100644 --- a/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorHandlerInterface.java +++ b/src/main/java/de/dennisguse/opentracks/sensors/sensorData/SensorHandlerInterface.java @@ -11,7 +11,7 @@ public interface SensorHandlerInterface { List getServices(); - SensorData createEmptySensorData(String address); + Aggregator createEmptySensorData(String address); void handlePayload(SensorManager.SensorDataChangedObserver observer, ServiceMeasurementUUID serviceMeasurementUUID, String sensorName, String address, BluetoothGattCharacteristic characteristic); }