Kompose icon indicating copy to clipboard operation
Kompose copied to clipboard

Published 20 hours ago • verified publisher icon jtouzy

UseCase + ViewModel

Open StephaneBg opened this issue 5 years ago • 3 comments
trafficstars

StephaneBg avatar Feb 04 '20 07:02 StephaneBg

Il y a un peu trop de modifications au niveau du common, ça change trop l'archi. Le UseCase et l'Interactor ?

jtouzy avatar Feb 06 '20 07:02 jtouzy

C'est juste du renommage. On trouve souvent ces notations en Clean Architecture côté Android. Cela évite en plus le suffixe Impl.

StephaneBg avatar Feb 06 '20 07:02 StephaneBg

Justement, l'architecture commence à un peu trop pencher côté Android. Elle est prévue pour être générique pour multiplateforme et on se retrouve à devoir changer pour faire des spécifications particulières pour Android, notamment concernant le problème de rotation, juste pour éviter un re-draw de l'écran.

Je pense qu'il va falloir chercher une autre solution. En plus, la PR impacte plusieurs choses : les tests d'un côté et cette implémentation de VM de l'autre côté, et j'ai du mal à voir les impacts de chaque ajout.

jtouzy avatar Feb 06 '20 09:02 jtouzy