Move conversion to/from absolute/relative weight to base class

android_app/app/src/main/java/com/health/openscale/gui/views/FatMeasurementView.java

@@ -47,21 +47,18 @@ public class FatMeasurementView extends FloatMeasurementView {
     }
 
     @Override
-    protected float getMeasurementValue(ScaleMeasurement measurement) {
+    protected boolean shouldConvertPercentageToAbsoluteWeight() {
-        if (percentageEnable) {
+        return !percentageEnable;
-            return measurement.getFat();
     }
 
-        return measurement.getConvertedWeight(getScaleUser().getScaleUnit()) / 100.0f * measurement.getFat();
+    @Override
+    protected float getMeasurementValue(ScaleMeasurement measurement) {
+        return measurement.getFat();
     }
 
     @Override
     protected void setMeasurementValue(float value, ScaleMeasurement measurement) {
-        if (percentageEnable) {
         measurement.setFat(value);
-        } else {
-            measurement.setFat(100.0f / measurement.getConvertedWeight(getScaleUser().getScaleUnit()) * value);
-        }
     }
 
     @Override
@@ -75,11 +72,7 @@ public class FatMeasurementView extends FloatMeasurementView {
 
     @Override
     protected float getMaxValue() {
-        if (percentageEnable) {
+        return maybeConvertPercentageToAbsolute(80);
-            return 80;
-        }
-
-        return 300;
     }
 
     @Override

android_app/app/src/main/java/com/health/openscale/gui/views/FloatMeasurementView.java

@@ -54,6 +54,7 @@ public abstract class FloatMeasurementView extends MeasurementView {
     private Date dateTime;
     private float value = NO_VALUE;
     private float previousValue = NO_VALUE;
+    private float userConvertedWeight;
     private EvaluationResult evaluationResult;
 
     private String nameText;
@@ -227,6 +228,38 @@ public abstract class FloatMeasurementView extends MeasurementView {
         return isEstimationEnabled() && getMeasurementMode() == MeasurementViewMode.ADD;
     }
 
+    protected boolean shouldConvertPercentageToAbsoluteWeight() {
+        return false;
+    }
+
+    private float makeAbsoluteWeight(float percentage) {
+        return userConvertedWeight / 100.0f * percentage;
+    }
+
+    private float makeRelativeWeight(float absolute) {
+        return 100.0f / userConvertedWeight * absolute;
+    }
+
+    protected float maybeConvertPercentageToAbsolute(float value) {
+        if (shouldConvertPercentageToAbsoluteWeight()) {
+            return makeAbsoluteWeight(value);
+        }
+
+        return value;
+    }
+
+    private void updateUserConvertedWeight(ScaleMeasurement measurement) {
+        if (shouldConvertPercentageToAbsoluteWeight()) {
+            // Make sure weight is never 0 to avoid division by 0
+            userConvertedWeight = Math.max(1.0f,
+                    measurement.getConvertedWeight(getScaleUser().getScaleUnit()));
+        }
+        else {
+            // Only valid when shouldConvertPercentageToAbsoluteWeight() returns true
+            userConvertedWeight = -1.0f;
+        }
+    }
+
     @Override
     public void loadFrom(ScaleMeasurement measurement, ScaleMeasurement previousMeasurement) {
         dateTime = measurement.getDateTime();
@@ -235,9 +268,21 @@ public abstract class FloatMeasurementView extends MeasurementView {
         float newPreviousValue = NO_VALUE;
 
         if (!useAutoValue()) {
-            newValue = clampValue(getMeasurementValue(measurement));
+            updateUserConvertedWeight(measurement);
+
+            newValue = getMeasurementValue(measurement);
+            newValue = maybeConvertPercentageToAbsolute(newValue);
+            newValue = clampValue(newValue);
+
             if (previousMeasurement != null) {
-                newPreviousValue = clampValue(getMeasurementValue(previousMeasurement));
+                float saveUserConvertedWeight = userConvertedWeight;
+                updateUserConvertedWeight(previousMeasurement);
+
+                newPreviousValue = getMeasurementValue(previousMeasurement);
+                newPreviousValue = maybeConvertPercentageToAbsolute(newPreviousValue);
+                newPreviousValue = clampValue(newPreviousValue);
+
+                userConvertedWeight = saveUserConvertedWeight;
             }
         }
 
@@ -247,9 +292,16 @@ public abstract class FloatMeasurementView extends MeasurementView {
     @Override
     public void saveTo(ScaleMeasurement measurement) {
         if (!useAutoValue()) {
+            if (shouldConvertPercentageToAbsoluteWeight()) {
+                // Make sure to use the current weight to get a correct percentage
+                updateUserConvertedWeight(measurement);
+                setMeasurementValue(makeRelativeWeight(value), measurement);
+            }
+            else {
                 setMeasurementValue(value, measurement);
             }
         }
+    }
 
     @Override
     public void restoreState(Bundle state) {

android_app/app/src/main/java/com/health/openscale/gui/views/MuscleMeasurementView.java

@@ -45,21 +45,18 @@ public class MuscleMeasurementView extends FloatMeasurementView {
     }
 
     @Override
-    protected float getMeasurementValue(ScaleMeasurement measurement) {
+    protected boolean shouldConvertPercentageToAbsoluteWeight() {
-        if (percentageEnable) {
+        return !percentageEnable;
-            return measurement.getMuscle();
     }
 
-        return measurement.getConvertedWeight(getScaleUser().getScaleUnit()) / 100.0f * measurement.getMuscle();
+    @Override
+    protected float getMeasurementValue(ScaleMeasurement measurement) {
+        return measurement.getMuscle();
     }
 
     @Override
     protected void setMeasurementValue(float value, ScaleMeasurement measurement) {
-        if (percentageEnable) {
         measurement.setMuscle(value);
-        } else {
-            measurement.setMuscle(100.0f / measurement.getConvertedWeight(getScaleUser().getScaleUnit()) * value);
-        }
     }
 
     @Override
@@ -73,11 +70,7 @@ public class MuscleMeasurementView extends FloatMeasurementView {
 
     @Override
     protected float getMaxValue() {
-        if (percentageEnable) {
+        return maybeConvertPercentageToAbsolute(80);
-            return 80;
-        }
-
-        return 300;
     }
 
     @Override

android_app/app/src/main/java/com/health/openscale/gui/views/WaterMeasurementView.java

@@ -47,21 +47,18 @@ public class WaterMeasurementView extends FloatMeasurementView {
     }
 
     @Override
-    protected float getMeasurementValue(ScaleMeasurement measurement) {
+    protected boolean shouldConvertPercentageToAbsoluteWeight() {
-        if (percentageEnable) {
+        return !percentageEnable;
-            return measurement.getWater();
     }
 
-        return measurement.getConvertedWeight(getScaleUser().getScaleUnit()) / 100.0f * measurement.getWater();
+    @Override
+    protected float getMeasurementValue(ScaleMeasurement measurement) {
+        return measurement.getWater();
     }
 
     @Override
     protected void setMeasurementValue(float value, ScaleMeasurement measurement) {
-        if (percentageEnable) {
         measurement.setWater(value);
-        } else {
-            measurement.setWater(100.0f / measurement.getConvertedWeight(getScaleUser().getScaleUnit()) * value);
-        }
     }
 
     @Override
@@ -75,11 +72,7 @@ public class WaterMeasurementView extends FloatMeasurementView {
 
     @Override
     protected float getMaxValue() {
-        if (percentageEnable) {
+        return maybeConvertPercentageToAbsolute(80);
-            return 80;
-        }
-
-        return 300;
     }
 
     @Override