Discussion :

[grunk]

Salut à tous

J'ai une activity avec 4 boutons (nombre susceptible d'évoluer plus tard) tous ces boutons sont attachés à un évènement Onclick qui est le même excepté un paramètres qui diffère. En gros chaque bouton ouvre la même dialog mais avec un titre différent.

J'aimerais rendre le code du Onclick si possible plus générique afin de pouvoir ajouter d'autre bouton sans difficulté par la suite.

J'ai résolu le problème pour l'attribution de l'événement :

Code :

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

1
2
3
4
5
6
7
8
for(int i = 0; i < NBBTN ; i++)
{
	Button button = (Button) findViewById(
			getResources().getIdentifier("button"+(i+1), "id", this.getPackageName())
	);
 
	button.setOnClickListener(this);
}

mais pas pour le traitement de l'événement :

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
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
public void onClick(View v) 
{
	int camselected = -1;
 
	switch(v.getId())
	{
		case R.id.button1:
		{
			this.num = 0;
			this.test(this.num);
			break;
		}
 
		case R.id.button2:
		{
			this.num = 1;
			this.test(this.num);
			break;
		}
 
		case R.id.button3:
		{
			this.num = 2;
			this.test(this.num);
			break;
		}
 
		case R.id.button4:
		{
			this.num = 3;
			this.test(this.num);
			break;
		}
 
	}
}

qui pour le moment m'oblige à rajouter un case à chaque bouton.
Des astuces pour améliorer cette dernière partie ?

merci

[Simon MARQUIS]

La technique la plus simple est de créer deux tableaux :
- les identifiants
- les boutons

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
19
20
21
22
 
int [] ids = new int [] { R.id.bouton1, R.id.bouton2, ....};
Button [] boutons = new Button [ids.length];
 
...
for (int i = 0 ; i < ids.length ; i++ ){
   boutons[i] = (Button) findViewById(ids[i]);
   boutons[i].setOnClickListener(this);
}
 
...
onClick(View v){
 int id = v.getId();
 for (int i = 0 ; i < ids.length ; i++ ){
   if(id == ids[i]){
     this.num = 0;
     this.test(i);
     break;
 
    }
 }
}

[nicroman]

Dans le XML:

Code :

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

1
2
3
 
   android:onClick="onButtonClick"
   android:tag="MyText"

Dans le code:

Code :

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

1
2
3
4
5
 
   public void onButtonClick(View v)
   {
         test(v.getTag()); // <= v.getTag() sera la String "MyText" ....
   }

Meme pas besoin de toucher aux listeners

[grunk]

Merci pour vos réponses.

Je vais opter pour la solution de Simonmarky. Bien que plus simple la solution proposée par nicroman me gène un peu du fait que je vais me retrouver avec des évènements dans le code et dans le xml. Pas top pour la maintenance

[nicroman]

hummm... comme tu veux

Enfin, la solution d'avoir de l'UI dans le code est pire à mon humble avis
A chaque bouton ajouté, deux endroits à toucher (le XML pour le bouton, et le code pour la liste).

[grunk]

Enfin, la solution d'avoir de l'UI dans le code est pire à mon humble avis
A chaque bouton ajouté, deux endroits à toucher (le XML pour le bouton, et le code pour la liste).

De toute manière , il faudra à un moment donné faire du code au moins pour définir le contenu de OnClick. Donc je préfère tout regrouper au même endroit , un peu comme ce qu'on fait en javascript/html

[nicroman]

De toute manière , il faudra à un moment donné faire du code au moins pour définir le contenu de OnClick. Donc je préfère tout regrouper au même endroit , un peu comme ce qu'on fait en javascript/html

Ben c'est le même pour tous les boutons le code n'est fait qu'une fois, apres on touche plus au java....
Java:

Code :

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

1
2
3
4
5
 
public void onLaunchClick(View v)
{
    doLaunch(v.getTag());
}

Resources:

Code :

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

1
2
3
4
5
6
7
 
<layout>
   <button android:onClick="onLaunchClick" android:tag="bouton1"/>   
   <button android:onClick="onLaunchClick" android:tag="bouton2"/>
   <button android:onClick="onLaunchClick" android:tag="bouton3"/>
   <button android:onClick="onLaunchClick" android:tag="bouton4"/>
</layout>

[grunk]

Ben c'est le même pour tous les boutons le code

Oui oui , j'ai bien compris , mais ça c'est dans le cas idéal où il n'ya que ces boutons , ce qui n'est pas mon cas car j'ai d'autre actions qui elles sont totalement indépendantes.