Discussion :

[LeBabouin]

Bonjour!

Y aurait-il une méthode plus élégante d'effectuer le test suivant ?

Code :

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

if(codeSousProduit.equals("05")||codeSousProduit.equals("27")||codeSousProduit.equals("28"))

Merci d'avance!

[pracede2005]

Ben malheuresement Non ! ça se saurait si ça existait.

[LeBabouin]

Ben malheuresement Non ! ça se saurait si ça existait.

Dingue! Ok merci pracede!

[AlexRNL]

Si tu as beaucoup de valeurs à tester, tu peux mettre tes String dans un set. Par exemple :

Code :

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

1
2
3
4
5
6
7
8
Set<String> set = new HashSet<>();
set.add("05");
set.add("28");
set.add("27");
 
if (set.contains(codeSousProduit)) {
	// traitement
}

[Népomucène]

Autre possibilité :

Code :

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

1
2
3
4
5
6
7
 
        String valeursOk = "[05][27][28]";
        String codeSousProduit = "27";
 
        if (valeursOk.contains("[" + codeSousProduit + "]")){
            System.out.println("gagné");
        }

Au delà de l'élégance, l'intérêt de cette solution, ainsi que celle d'AlexRNL,
c'est de pouvoir construire dynamiquement ta condition.
La liste "05","27","28" va certainement évoluer et si tu restes sur le code présenté, il te faudra reprendre ton application.

[supergeoffrey]

Si ton code est sur deux caractère tu peux faire ça

Code :

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

1
2
3
 
int x = Integer.parseInt(codeSousProduit);
if ((x-5)*(x-27)*(x-28)==0)

et si développe pour le plaisir un truc du genre

Code :

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

1
2
3
 
int x = Integer.parseInt(codeSousProduit);
if (x*x*x-70*x*x+1031*x==-3780)

[thelvin]

Si ton code est sur deux caractère tu peux faire ça

Code :

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

1
2
3
 
int x = Integer.parseInt(codeSousProduit);
if ((x-5)*(x-27)*(x-28)==0)

et si développe pour le plaisir un truc du genre

Code :

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

1
2
3
 
int x = Integer.parseInt(codeSousProduit);
if (x*x*x-70*x*x+1031*x==-3780)

Joli bricolage très geek, mais on a parlé d'élégance, ce qui en général est plutôt question de maintenabilité et de faire les choses de sorte qu'elles soient immédiatement évidentes.

Certes c'est pas des maths très compliqués, mais ils ne se justifient pas .
C'est certes très très efficace. Mais une table de hachage l'est suffisamment, sans dépendre du contenu des String, et pour être honnête String.contains() est suffisant aussi.
Et, plus important, if(x == 5 || x == 27 || x == 28) est encore plus efficace, et est essentiellement la même chose en plus clair.

[LeBabouin]

Merci tout le monde!

Autre possibilité :

La liste "05","27","28" va certainement évoluer et si tu restes sur le code présenté, il te faudra reprendre ton application.

Je suis bien conscient de ce problème, hélas c'est une modif ultra urgente et on m'a garanti que la liste de ces valeurs ne changerait pas de si tôt, comme d'habitude

**edit** ouaip, bien vu Népomucène

[Népomucène]

on m'a garanti que la liste de ces valeurs ne changerait pas de si tôt