Discussion :

[grunk]

Salut à tous,

Je débute avec android et enchaine donc les tutos. Je me suis donc atteler à une simple calculatrice.

Je suis étonnament surpris par la lourdeur de la gestion des événements , du coup je me dis que j'ai peut être raté quelques subtilité.

Voici ce que je fait :

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
37
38
39
40
41
42
43
public void onClick(View v)
{
	switch(v.getId())
	{
		case R.id.button0 : 
		case R.id.button1 :
		case R.id.button2 :
		case R.id.button3 :
		case R.id.button4 :
		case R.id.button5 :
		case R.id.button6 :
		case R.id.button7 :
		case R.id.button8 :
		case R.id.button9 :
		{
			String value 	= ((Button)v).getText().toString();
			if(update)
				update = false;
			else
				value = ecran.getText() + value;
 
			ecran.setText(value);
		}
	}
}
 
public void onCreate(Bundle savedInstanceState) 
{
	super.onCreate(savedInstanceState);
	setContentView(R.layout.main);
 
	ecran = (EditText) findViewById(R.id.editResult);
	findViewById(R.id.button0).setOnClickListener(this);
	findViewById(R.id.button1).setOnClickListener(this);
	findViewById(R.id.button2).setOnClickListener(this);
	findViewById(R.id.button3).setOnClickListener(this);
	findViewById(R.id.button4).setOnClickListener(this);
	findViewById(R.id.button5).setOnClickListener(this);
	findViewById(R.id.button6).setOnClickListener(this);
	findViewById(R.id.button7).setOnClickListener(this);
	findViewById(R.id.button8).setOnClickListener(this);
	findViewById(R.id.button9).setOnClickListener(this);
}

C'est surtout la partie dans le onCreate qui me chagrine un peu. Si j'ai 200 boutons , je vais avoir une tartines de code complètement imbuvable juste pour m'attacher à ma méthode gérant le onclick.

N'y a t'il pas plus simple ?
(je précise que mon expérience JAVA se limite à du non graphique , donc patapé si je fait des erreurs grossières )

[MrDuChnok]

Ton code pourrait être "simplifier" comme ça :

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
 
 
public OnClickListener btClickListener = new OnClickListener() {
  public void onClick(View v) {
    String value 	= ((Button)v).getText().toString();
    if(update) {
      update = false;
    } else {
      value = ecran.getText() + value;
    }
 
    ecran.setText(value);	
  }
}
 
 
public void onCreate(Bundle savedInstanceState) 
{
	super.onCreate(savedInstanceState);
	setContentView(R.layout.main);
 
	ecran = (EditText) findViewById(R.id.editResult);
	findViewById(R.id.button0).setOnClickListener(btClickListener);
	findViewById(R.id.button1).setOnClickListener(btClickListener);
	findViewById(R.id.button2).setOnClickListener(btClickListener);
	findViewById(R.id.button3).setOnClickListener(btClickListener);
	findViewById(R.id.button4).setOnClickListener(btClickListener);
	findViewById(R.id.button5).setOnClickListener(btClickListener);
	findViewById(R.id.button6).setOnClickListener(btClickListener);
	findViewById(R.id.button7).setOnClickListener(btClickListener);
	findViewById(R.id.button8).setOnClickListener(btClickListener);
	findViewById(R.id.button9).setOnClickListener(btClickListener);
}

Après je suis d'accord que la gestion évènementielle "à la Swing" est un peu lourde par moment...

Si j'ai 200 boutons , je vais avoir une tartines de code complètement imbuvable juste pour m'attacher à ma méthode gérant le onclick.

Je partirais plutôt dans la raisonnement inverse :
Est-ce normal d'avoir plus de 200 boutons sur un même interface ?
Après il y a certaines techniques d'optimisation qui te feront gagner des lignes de code, mais il faut que le code soit bien fait, et soit suffisamment générique.

[grunk]

Pourquoi est ce mieux de passer une instance de OnClickListener plutôt que mon objet courant qui de toute manière implémente OnClickListener ?

Je partirais plutôt dans la raisonnement inverse :
Est-ce normal d'avoir plus de 200 boutons sur un même interface ?

On est d'accord , la calculette me semble déjà être un exemple costaud niveau nombre de bouton mais c'était pour l'exemple

[Feanorin]

Bonjour,

Il me semble qu'il y avait déjà eu une discussion sur ce sujet , et la solution donnée pour éviter cette surcharge dans le OnCreate était de l'implémenter directement dans le xml

http://developer.android.com/referen...ndroid:onClick

android:onClick
Since: API Level

Name of the method in this View's context to invoke when the view is clicked. This name must correspond to a public method that takes exactly one parameter of type View. For instance, if you specify android:onClick="sayHello", you must declare a public void sayHello(View v) method of your context (typically, your Activity).

Must be a string value, using '\\;' to escape characters such as '\\n' or '\\uxxxx' for a unicode character.

This may also be a reference to a resource (in the form "@[package:]type:name") or theme attribute (in the form "?[package:][type:]name") containing a value of this type.

This corresponds to the global attribute resource symbol onClick.
Related Methods

Après c'est la même charge de code sauf que tu n'auras pas la masse de déclaration dans le onCreate .