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.

[supergeoffrey]

C'est vendredi aprem! On peut se permettre un peu de relâchement.
Je peux t'assurer que le

Code :

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

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

, j'aurais jamais essayé de le simplifier!

[Népomucène]

C'est vendredi aprem! On peut se permettre un peu de relâchement.
Je peux t'assurer que le

Code :

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

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

, j'aurais jamais essayé de le simplifier!

ben, là, j'aurais quand même évité.
Je parie des dollars contre des gâteaux secs(*) que ce code a été écrit dans la plus grande urgence sous la menace d'un revolver sur la tempe.
A l'évidence il s'agit d'une codification présente dans la table produits d'un programme de gestion.
Quand dans un an ou deux, avec le nouveau DAF (**), on changera la codification , ça ne marchera plus et on cherchera en vain où on peut paramétrer le bouzingue avec la nouvelle codification !

(*) citation empruntée à Lucky Luke
(**) directeur administratif et financier -> dès qu'un nouveau arrive, il veut changer quelque chose dans le SI

[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

[tchize_]

Allez, c'est samedi matin.

Autre solution

Code :

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

if (Arrays.asList(new String[]{"05","27","28"}).contains(lavaleur))

Ca a au moins l'avantage d'être court et facile à lire, à défaut d'être performant.

[deathness]

Allez, c'est samedi matin.

Autre solution

Code :

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

if (Arrays.asList(new String[]{"05","27","28"}).contains(lavaleur))

Ca a au moins l'avantage d'être court et facile à lire, à défaut d'être performant.

Tout dépend si on préfère la perf à la maintenance.
Mais bon il faut différencier "long a lire" et complexité à comprendre.

[tchize_]

Allez, encore une, un peu plus élégante (java 7 requis) et performante

Code :

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

1
2
3
4
5
6
7
8
9
switch (codeSousProduit) {
   case "05":
   case "27":
   case "28":
      // ton code
      break:
   default:
      // ton else :)
}

[supergeoffrey]

Sympa pour la version java 7,
C'est vrai que le switch était limité avant

[deathness]

Allez, encore une, un peu plus élégante (java 7 requis) et performante

Code :

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

1
2
3
4
5
6
7
8
9
switch (codeSousProduit) {
   case "05":
   case "27":
   case "28":
      // ton code
      break:
   default:
      // ton else :)
}

Vu que les précédente version de java ne supporte pas, a utiliser avec beaucoup de prudence suivant ou le code doit etre execute.

[Népomucène]

Vu que les précédente version de java ne supporte pas, a utiliser avec beaucoup de prudence suivant ou le code doit etre execute.

ça sera plus facile de vérifier la version de la JVM
que d'être sûr que le client ne changera pas sa codification d'articles !

[pracede2005]

Vu que les précédente version de java ne supporte pas, a utiliser avec beaucoup de prudence suivant ou le code doit etre execute.

Cool je ne connaissais pas !

[adiGuba]

Autre solution

Code :

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

if (Arrays.asList(new String[]{"05","27","28"}).contains(lavaleur))

Ca a au moins l'avantage d'être court et facile à lire, à défaut d'être performant.

A noter que depuis Java 5.0 on peut l'écrire d'une manière bien plus lisible grace à l'ellipse :

Code :

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

if (Arrays.asList("05","27","28").contains(lavaleur))

[edit] Et si on veut de la performance, il suffit de passer par un Set...

a++