analyze to fix statistics for all firmwares

Author: SliDeeN

app/src/main/java/com/mg4/control/model/StatsRecords.kt

@@ -1,6 +1,7 @@
 package com.mg4.control.model
 
 import kotlin.math.abs
+import kotlin.math.floor
 import kotlin.math.max
 import kotlin.math.roundToInt
 
@@ -174,9 +175,19 @@ data class Trip(
      * Nommée « moteur » et non « traction » : la MG4 est une propulsion, et la XPower une quatre
      * roues motrices. Null tant qu'aucun poste n'est connu, sinon on présenterait le total comme
      * une mesure séparée qui n'existe pas.
+     *
+     * Sur un trajet à l'énergie intégrée, le seul poste connu est [auxiliaryKwh] et le calcul est
+     * celui du camembert de l'écran d'origine : « Driving power » = total − « AC and others ». Ce
+     * compteur tronque au kWh, le vrai poste vaut donc jusqu'à 1 kWh de plus : le reste est rendu
+     * au kWh entier **par défaut**, seule valeur ronde qui tombe toujours dans l'intervalle
+     * possible. À présenter comme un ordre de grandeur, au même titre que [auxiliaryKwh].
      */
     val motorKwh: Float?
         get() {
+            if (energyIntegrated) {
+                val autres = auxiliaryKwh ?: return null
+                return floor(energyKwh - autres).coerceAtLeast(0f)
+            }
             if (climateKwh == null && accessoriesKwh == null) return null
             // Sur le BRUT : moteur + climatisation + accessoires − régénération = énergie nette.
             val reste = energyKwh - (climateKwh ?: 0f) - (accessoriesKwh ?: 0f)

app/src/main/java/com/mg4/control/ui/StatsFragment.kt

@@ -321,8 +321,11 @@ class StatsFragment : Fragment() {
         if (trip.energyIntegrated)
             getString(R.string.stats_detail_origin) to getString(R.string.stats_detail_origin_integrated)
         else null,
-        // L'énergie du moteur n'est pas publiée : c'est le total moins les postes annexes.
-        trip.motorKwh?.let { getString(R.string.stats_detail_motor) to kwh(it) },
+        // L'énergie du moteur n'est pas publiée : c'est le total moins les postes annexes. Sur un
+        // trajet calculé, le poste annexe est au kWh entier : le moteur ne peut pas être plus fin.
+        trip.motorKwh?.let {
+            getString(R.string.stats_detail_motor) to (if (trip.energyIntegrated) kwhEntier(it) else kwh(it))
+        },
         trip.climateKwh?.let { getString(R.string.stats_detail_climate) to kwh(it) },
         trip.accessoriesKwh?.let { getString(R.string.stats_detail_accessories) to kwh(it) },
         trip.auxiliaryKwh?.let { getString(R.string.stats_detail_auxiliary) to kwhEntier(it) },
@@ -738,9 +741,9 @@ class StatsFragment : Fragment() {
     private fun kwh(value: Float): String = "${fmt(value)} kWh"
 
     /**
-     * Valeur d'un compteur qui n'avance que par kWh entier (celui de l'écran d'origine) : un ordre
-     * de grandeur, annoncé comme tel — et « moins d'un » plutôt qu'un zéro qui passerait pour une
-     * mesure.
+     * Valeur connue au kWh entier seulement — le compteur de l'écran d'origine, et le moteur qui
+     * s'en déduit : un ordre de grandeur, annoncé comme tel — et « moins d'un » plutôt qu'un zéro
+     * qui passerait pour une mesure.
      */
     private fun kwhEntier(value: Float): String =
         if (value < 1f) "< 1 kWh" else "≈ ${value.roundToInt()} kWh"

app/src/test/java/com/mg4/control/model/TripTest.kt

@@ -191,4 +191,23 @@ class TripTest {
         assertTrue(trip(km = 12, kwh = 1.8f, integre = 12.3f).energyKnown)
         assertTrue(trip(km = 0, kwh = 0.1f, integre = 0.4f).energyKnown)
     }
+
+    @Test
+    fun `trajet calcule, le moteur est le total moins climatisation et autres`() {
+        // Le camembert de l'écran d'origine : « Driving power » = total − « AC and others ».
+        // Le compteur d'origine tronque au kWh : le vrai poste vaut 2 à 3, le moteur 15,6 à 16,6.
+        val t = trip(km = 120, kwh = 18.6f).copy(energyIntegrated = true, auxiliaryKwh = 2f)
+        assertEquals("au kWh entier, par défaut", 16f, t.motorKwh!!, 0.001f)
+    }
+
+    @Test
+    fun `petit trajet calcule, le moteur reste sous le kWh`() {
+        val t = trip(km = 3, kwh = 0.6f).copy(energyIntegrated = true, auxiliaryKwh = 0f)
+        assertEquals(0f, t.motorKwh!!, 0.001f)
+    }
+
+    @Test
+    fun `trajet calcule sans compteur d'origine, pas de moteur`() {
+        assertNull(trip(km = 120, kwh = 18.6f).copy(energyIntegrated = true).motorKwh)
+    }
 }