Discussion :

[krilas]

Bonjour à tous,


Voilà, je travaille sur une application qui a été développée par un ancien programmeur. Cette application a été développée avec Jdev 10g sous XP.

Donc, je veux changer un IF d'une forme trop longue comme:

Code :

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

1
2
3
4
5
6
7
8
 
If (type = 1 ou type = 2 ou type = 3  ou type = 4 ou type = 5
     ou type = 6 ou type = 7 ou type = 8 ou type = 9 ou type = 10
     ou type = 11 ou type = 11 ou type = 12 ou type = 13 ou type = 14
     ou type = 15 ou type = 16 ou type = 17 ou type = 18 ou type = 19 ...){
 
 ...
}

Comment puis je remplacer ce IF par un autre plus conviviable ?


Merci d'avance de votre aide

[adiGuba]

Salut,


Ben tout simplement :

Code :

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

1
2
3
if ( 0 < type && type < 20 ) {
 
}

a++

[GanYoshi]

Bonjour,
Je sais pas si "type" dans son exemple est un int ou si c'est juste pour l'exemple. ^^'

Dans le doute, j'aurais mis :

Code :

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

1
2
3
4
5
6
 
 
List<Type> listType = getListType();
if(listType.contains(type){
...
}

Edit : Grillé comme un poulet

[Jimmy_]

Moi je dis méfiance, il faudrait être un peu naïf pour coder comme ça,
peut-être qu'il n'a pas le choix.
Faut voir le contexte moi je dis.

[GanYoshi]

Je vois pas de cas où ma liste marche pas.

[Air P-E]

un switch case peut-être ?
Mais comme l'a dit GanYoshi, ça dépend du type...

[Rei Ichido]

Dans ce cas, il est plus sécurisé de faire une méthode monTestSurUnEntier, qui va elle même utiliser la liste, en private static final ; comme ça on évite des modification intempestives sur cette liste dans le code (ce qui n'est pas censé arriver au vu du code).

[Jimmy_]

Je vois pas de cas où ma liste marche pas.

Si ca marche, mais des fois il suffit de tester l'inverse qui est en fait très court et de faire un not dessus pour le même effet.
Enfin faut voir le vrai code.