Code plus générique pour OnClick

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:

---

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:

---

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

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

Code:

---

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;   } } }

---

Dans le XML:

Code:

---

1 2 3

android:onClick="onButtonClick" android:tag="MyText"

---

Dans le code:

Code:

---

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 ;)

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 :(

hummm... comme tu veux ;)

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

Citation:

---

Envoyé par nicroman

Enfin, la solution d'avoir de l'UI dans le code est pire à mon humble avis :mouarf:
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 ;)

Citation:

---

Envoyé par grunk

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:

---

1 2 3 4 5

public void onLaunchClick(View v) { doLaunch(v.getTag()); }

---

Resources:

Code:

---

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>

---

Citation:

---

Envoyé par nicroman

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.