Discussion :

[vangoeth]

Bonjour,

Je viens vers vous pour savoir comment je pourrais supprimer la redondance dans ce type de déclaration:

Code :

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

1
2
3
4
5
new AsyncDownloadCol1().execute(url, "choix", mVariable, "date", "2007");
new AsyncDownloadCol2().execute(url, "choix", mVariable,"date", "2009");
new AsyncDownloadCol3().execute(url, "choix", mVariable,"date", "2010");
new AsyncDownloadCol4().execute(url, "choix", mVariable,"date", "2011");
new AsyncDownloadCol5().execute(url, "choix", mVariable,"date", "2012");

Mes profs me disent "tableau de variable" mais ça me parait pas vraiment faisable ou j'ai zappé qlqch :s

[Feanorin]

Bonjour,

La méthode par réflection, mais c'est pas propre (j'ai pas testé) par contre ca te fait changer la déclaration de la méthode execue qui passerait par la classe Type

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
 
Class Type {
String url;
String choix;
String variable;
String textdate;
String date;
 
 public Type (String myurl,String mychoix,String myvariable,String mytextdate,String mydate)
 {
        url =myurl;
        choix = mychoix;
        variable = myvariable;
        textdate = mytextdate;
        date = mydate;
    }
};
 
 
protected void Call(string classname, Type type){
 
Class theClass  = Class.forName(classname);
Object objet= theClass.newInstance();
Method m = objet.getMethod("execute", Type.class);
m.invoke(null,type);
}
 
String[] date= {
"2007",
"2009", 
"2010",
"2011",
"2012"};
 
// A faire au lieu des tes 5 appels 
for (int i=0; i < date.length(); i++){
   Type type = new Class(url,"choix",mVariable,"date",date[i]);
   Call("AsyncDownloadCol" + Integer.toString(i+1),type );
}

Après peux être qu'au lieu d'avoir n Classe il serait plus simple de centraliser tout dans la même classe avec un selector

[vangoeth]

ça me parait fort fastidieux comme façon de faire ^^

[Feanorin]

ça me parait fort fastidieux comme façon de faire ^^

C'est pour cela que je te proposais de centraliser tes actions dans une seule classe avec un selector .

Du style

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
String[] date= {
"2007",
"2009", 
"2010",
"2011",
"2012"};
 
// Tu n'utilises qu'une seule classe AsyncDownloadCol
// A faire au lieu des tes 5 appels 
for (int i=0; i < date.length(); i++){
   new AsyncDownloadCol().execute(url, "choix", mVariable, "date", date[i],i); // i te servira comme switch pour savoir quelle méthode tu dois appliquer.
}

[vangoeth]

C'est vrai que vu comme ça c'est déjà plus simple :p

Merci je vais voir ce que je peux faire ^^
Par contre on m'a confirmé que android est reconnu pour être redondant comme code.
Je sais pas ce que tu en penses et à quel niveau?

[Hephaistos007]

Par contre on m'a confirmé que android est reconnu pour être redondant comme code.

Qui t'as confirmé cela ? on pourrais avoir une discussion avec cette personne ?

[vangoeth]

Bah franchement au niveau des listener sur les boutons je vois pas du tout pq on commencerait à jouer avec des boucles pour éviter justement d'avoir bcp de lignes de codes ^^

[Feanorin]

Bah franchement au niveau des listener sur les boutons je vois pas du tout pq on commencerait à jouer avec des boucles pour éviter justement d'avoir bcp de lignes de codes ^^

Si tu implémentes correctement tes listeners normalement tu peux factoriser ton code, après c'est sûr qui si on programme sans prendre du temps, le code peut devenir redondant comme tout langage ...

[MrDuChnok]

Après il y a encore des solutions en utilisant les annotations pour alléger la vue du code.

Mais bon comme l'a dit Feanorin, une bonne conception permet déjà de se simplifier énormément :
- les temps de développement
- les temps de débogage
- la complexité du code