Supprimer la redondance

Version imprimable

Bonjour,

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

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

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:

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

04/12/2012, 09h57
vangoeth

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

Citation:

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

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

04/12/2012, 10h20
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?
04/12/2012, 10h30
Hephaistos007

Citation:

Envoyé par vangoeth

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 ? :aie:
04/12/2012, 10h34
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 ^^
04/12/2012, 10h52
Feanorin

Citation:

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 ...
04/12/2012, 11h27
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