Discussion :

[grunk]

Salut à tous,

je refactor une app dans laquel j'essai d'alléger un peu les activities.
Mon but est de ne garder dans les activity que ce qui concerne le graphique (apparition/disparition de vue , changement de texte, animation, etc ...) et d'avoir à coté un controller qui va venir gérer toute la logique liée à la vue. J'ai ensuite tout un tas de classe métier que je considère comme les modèles.

Activity :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public class MainActivity extends AppCompatActivity
        implements NavigationView.OnNavigationItemSelectedListener {
 
    private MainController mController;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        mController = new MainController(this);
 
    }
 
    public void showToast() {
        Toast.makeText(this, "message", Toast.LENGHT_SHORT).show();
    }
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public class MainController {
 
    private final WeakReference<MainActivity> mActivityRef;
 
    public MainController(MainActivity activity) {
        mActivityRef = new WeakReference<>(activity);
    }
 
	public void doSomething()
	{
		//Traitement
		if(mActivityRef != null) {
			final LoginActivity activity = mActivityRef.get();
			activity.showToast();
		}
	}
 
}

- Que pensez vous de cet implémentation ? On est pas dans le MVC "pure tradition" mais l'idée me plait bien , c'est assez léger à mettre en place.
- J'ai opté pour une weakreference de l'activity dans le controller , mais je ne suis pas certains que ce soit nécessaire. Avis ?

J’accroche pas aux architecture MVVM ou MVP qui rajoute trop de classe/code à mon gout.

[Turvy]

Bonjour,

Je ne connais pas vraiment les références, mais quelle serait la différence (niveau mémoire, gain de temps...) entre ton code et

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
 private final MainActivity activity;
 
    public MainController(MainActivity activity) {
        this.activity= activity;
    }
....

On essaie de mettre en place ce genre de pratique chez nous, mais parfois on accumule les

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
currentActivity.runOnUiThread(new Runnable() {
		@Override
		public void run() {
			// modification d'une vue
		}
	});

dans la couche métier

[grunk]

Je ne connais pas vraiment les références, mais quelle serait la différence (niveau mémoire, gain de temps...) entre ton code et

Y'a pas de différence si ce n'est que dans mon cas la référence est faible (weak). Ce qui veux dire que si pour une raison ou une autre mon controller est coincé sur un traitement , il n'empèchera pas l'activity de se détruire et d'être ramassée par le GC

mais parfois on accumule les

Dans mon cas je ne l'utilise jamais puisque le controller va appeler une fonction de l'activity pour ce genre de chose.

[MasterMbg]

Salut grunk,

Salut à tous,

je refactor une app dans laquel j'essai d'alléger un peu les activities.
Mon but est de ne garder dans les activity que ce qui concerne le graphique (apparition/disparition de vue , changement de texte, animation, etc ...) et d'avoir à coté un controller qui va venir gérer toute la logique liée à la vue. J'ai ensuite tout un tas de classe métier que je considère comme les modèles.

Activity :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public class MainActivity extends AppCompatActivity
        implements NavigationView.OnNavigationItemSelectedListener {
 
    private MainController mController;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        mController = new MainController(this);
 
    }
 
    public void showToast() {
        Toast.makeText(this, "message", Toast.LENGHT_SHORT).show();
    }
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
public class MainController {
 
    private final WeakReference<MainActivity> mActivityRef;
 
    public MainController(MainActivity activity) {
        mActivityRef = new WeakReference<>(activity);
    }
 
	public void doSomething()
	{
		//Traitement
		if(mActivityRef != null) {
			final LoginActivity activity = mActivityRef.get();
			activity.showToast();
		}
	}
 
}

- Que pensez vous de cet implémentation ? On est pas dans le MVC "pure tradition" mais l'idée me plait bien , c'est assez léger à mettre en place.
- J'ai opté pour une weakreference de l'activity dans le controller , mais je ne suis pas certains que ce soit nécessaire. Avis ?

J’accroche pas aux architecture MVVM ou MVP qui rajoute trop de classe/code à mon gout.

Je trouve ton idée bien osée et en même temps intéressante. Je suis entrain de scruter et voir sur quel mesure intervenir avec un plus et quelques remarques constructives.

Seulement, concernant le WeakReference, je ne pense pas que cela soit nécessaire dans ce contexte du fait que l'activity à laquelle on fera référence ne sera pas gardée en tant que membre static par le garbage collector.

Du reste, je continue à analyser ton idée.

Merci

Christian Djo,