search

kmzbrnoI / hJOPandroidDriver

hJOP Engine Driver for Android
https://hjop.kmz-brno.cz/hJOPdriver
Apache License 2.0
4 stars 4 forks source link

Resume aktivity řízení loko zobrazí nekonzistentní funkce s vybranou lokomotivou #39

Closed horacekj closed 3 years ago

horacekj commented 3 years ago

Standardní řízení: (převzaty také další lokomotivy): Screenshot_20201204-170303

Aplikace přesunuta na pozadí a znovu otevřena. Stav:

Screenshot_20201204-170310

horacekj commented 3 years ago

Chová se stejně i při přetočení displaye (funkce se změní na funkce jiné lokomotivy)

kudlav commented 3 years ago

Tohle už jsem taky řešil, přesně takového chování eleminuje použítí ViewModel (View, ViewModel, Model), [Google], [Medium]. Třída aktivity by neměla uchovávat stav aplikace.

Určitě to půjde opravit i jinak, je to na zvážení, protože je to doporučenou architekturou a nové aplikace nemá cenu psát jinak, ale tady by to znamenalo úpravy.

Dřív nebo později (spíš později) se také budeme muset vypořádat s deprecated AsyncTask, buď přechodem na Kotlin (kam Google vývojáře směřuje) nebo na práci s procesy.

kudlav commented 3 years ago

Nakonec to bylo způsobeno přepsáním zvoleného vlaku při každém updateHW, problém by se projevoval nejen při obnovení obrazovky. Opraveno v 80c0394deb83df7087af8602d14fa8d2247f3bab