Discussion :

[kouesta]

Bonjour, j'aimerais optimiser un code.

J'ai un pour chaque composant d'un ArrayList à faire un traitement mais avec des conditions différentes.

Voici en clair ce que à donne codé de 2 façon :


Code 1
avantage : Pas de redondance de code
inconvénient : Perte de temps lorsque la dimension de array est improtante...en effet, on fait la vérification de la valeur type a chaque passage de la boucle alors qu'une seule fois suffit

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
 
private void myFunction(int type){
 
for(i=0;i<array.length;i++){
if(bidule && bidule2){
switch(type){
case 0:if(a<=b){...};break;
case 1:if(a>=b){...};break;
case 2:if(a!=b){...};break;
case 3:if(a==b){...};break;
}
}
 
}
}

Code 2
avantage : plus rapide lorsque la dimension de array est importante
inconvénient : redondance de code dans la boucle for

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
 
private void myFunction(int type){
 
 
switch(type){
case 0:{
for(i=0;i<array.length;i++){if(bidule && bidule2){if(a>=b){...}}}
;break;
}case 1:{
for(i=0;i<array.length;i++){if(bidule && bidule2){if(a<=b){...}}}
;break;
}case 2:{
for(i=0;i<array.length;i++){if(bidule && bidule2){if(a!=b){...}}}
;break;
}case 3:{
for(i=0;i<array.length;i++){if(bidule && bidule2){if(a==b){...}}}
;break;
}
}
 
}

Auriez vous une autre solution pour optimiser le code ?
Pour l'instant j'ai opté pour la solution 2

Le must serait que je puisse faire un truc du genre :

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
 
private void myFunction(int type){
 
Verificator ve;
switch(type){
case 0:ve='<=';break;
case 1:ve='>=';break;
case 2:ve='!=';break;
case 3:ve='==';break;
}
 
for(i=0;i<array.length;i++){
if(bidule && bidule2){
if(a ve b){...}
}
 
}
}

[the-gtm]

A moins que ta liste ne fasse des dizaines de milliers d'entrées tu ne verras pas trop la différence en temps d'exécution, si ça se trouve le compilateur est même assez malin pour voir que dans le premier cas, la condition ne change pas et optimiser la boucle.

Sinon tu peux faire une interface avec une seule méthode et implémentée par 4 classes qui font chacun des 4 cas :

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
private void myFunction(int type){
 
IBidule bidule = null;
switch(type){
case 0:if(a<=b){bidule = new Bidule1();};break;
case 1:if(a>=b){bidule = new Bidule2();};break;
case 2:if(a!=b){bidule = new Bidule3();};break;
case 3:if(a==b){bidule = new Bidule4();};break;
}
 
for(i=0;i<array.length;i++){
if(bidule && bidule2){
  bidule.faireBidule(...);
}
}

[Alkhan]

bonjour,

il est difficile de résoudre le problème si tu ne met pas toute les infos.
qu'est ce qui est fait dans les différents if (exemple : if(a<=b){...}), d'ou viennent a et b (et de quel type ils sont)?

[kouesta]

En gros, j'ai une fenetre de recherche ou il faut choisir la colonne et la condition de recherche dans le tableau.
a et b peuvent être des String ou des String convertis en int
-dans le cas de String j'ai une fonction qui test les equals matches...
-dans le cas de int j'ai une la fonction que j'ai montré qui test ==,!=,<=,>=
Cela peut aussi être une String qui reprèsente un fichier et dans ce cas je test l'existance du fichier.


Dans les 2 cas à l'intérieur du if, je rajoute l'entrée dans un array secondaire afin d'afficher la recherche en fonction de la demande...

La solution de créer des classes intermédiaires peut être intéressante, mais ça reviendrais à faire beaucoup de classe différentes (environ 20 dans mon cas)

Sinon le tableau peux aller au maxi à la 10 000 entrée je pense, très rarement au dessus...mais c'est simplement de faire à chaque fois le même test dans chaque entrée de la boucle for qui m'embete.

[Alkhan]

Ce serait bien si tu pouvais mettre le vrai code, car ton exemple n'est pas parlant !!