Egalité de contenu d'un String parmi plusieurs valeurs

Version imprimable

14/09/2012, 13h00
LeBabouin

Egalité de contenu d'un String parmi plusieurs valeurs

Bonjour!

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

Code:

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

Merci d'avance!
14/09/2012, 13h48
pracede2005

Ben malheuresement Non ! ça se saurait si ça existait.
14/09/2012, 14h00
LeBabouin

Citation:

Envoyé par pracede2005

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

Dingue! Ok merci pracede!

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

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 }

Autre possibilité :
Code:

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.

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

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:

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

14/09/2012, 15h27
thelvin
Citation:
Envoyé par supergeoffrey

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

Code:

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:

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.
14/09/2012, 15h45
supergeoffrey

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

Code:

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

, j'aurais jamais essayé de le simplifier!
14/09/2012, 16h04
Népomucène

Citation:

Envoyé par supergeoffrey

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

Code:

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
14/09/2012, 16h31
LeBabouin

Merci tout le monde!

Citation:

Envoyé par Népomucène

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

**edit** ouaip, bien vu Népomucène :ccool:
14/09/2012, 16h40
Népomucène

Citation:

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

:mouarf2:
15/09/2012, 00h35
tchize_

Allez, c'est samedi matin.

Autre solution

Code:

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.
17/09/2012, 10h42
deathness

Citation:

Envoyé par tchize_

Allez, c'est samedi matin.

Autre solution

Code:

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.

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

Code:

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

17/09/2012, 12h20
supergeoffrey

Sympa pour la version java 7,
C'est vrai que le switch était limité avant
17/09/2012, 13h31
deathness
Citation:
Envoyé par tchize_

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

Code:

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.
17/09/2012, 15h30
Népomucène

Citation:

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 !
17/09/2012, 15h43
pracede2005

Citation:

Envoyé par deathness

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 !
17/09/2012, 17h53
adiGuba

Citation:

Envoyé par tchize_

Autre solution

Code:

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:

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

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

a++