Refactor bluetooth reading

reunu / unustasis

An open-source app for the Unu Scooter Pro

Apache License 2.0

27 stars 7 forks source link

Refactor bluetooth reading #24

Closed DSIW closed 3 months ago

DSIW commented 5 months ago

Extract ScooterReader
Extract BatteryReader
Extract CharacteristicRepository
Remove unused CBB capacity reading

QA checklist

[x] Read state (state & power state)
[x] Read primary battery SOC
[x] Read primary battery cycles
[x] Read secondary battery SOC
[x] Read secondary battery cycles
[x] Read seat closed
[x] Read handlebar
[x] Read AUX SOC
[x] Read CBB SOC
[x] Read CBB charging state
[x] Update ping
[ ] SOCs are cached (after app restart)
[x] Commands can be sent (locking & hibernation)

TheFreal commented 4 months ago

Gerade mal ein bisschen durch geguckt...ich weiß, clean code und so, aber fühlt sich für mich persönlich irgendwie etwas arg verschachtelt an? Brauchen sowas wie CacheManager oder StateOfChargeReader wirklich eigene Klassen und Dateien?

DSIW commented 4 months ago

Kein Problem, ich schaue mir die Abstraktionen nochmals kritisch an und werde es hoffentlich noch weiter vereinfachen können.

Grundsätzlich bin ich ein Freund von eher kleineren Klassen, um sie leichter testen zu können.

Den CacheManager hatte ich hinzugefügt, um das Schreiben und Auslesen an einer Stelle zu haben.

Den StateOfChargeReader finde ich gerade sinnvoll, weil er den Ping und das Caching beinhaltet. Ggf. brauchen wir den extra CycleReader nicht und kann es in einen allgemeineren BatteryReader umfunktionieren.

DSIW commented 4 months ago

Ich habe nun einige Komponenten zu dem BatteryReader hinzugefügt inkl. Unit-Tests.

TheFreal commented 3 months ago

Might make sense to rebase your branch first to make sure all changes make it over