Utilisez une chaîne dans un switch case en Java

Question

J'ai besoin de changer les if suivants en une structure switch-case tout en vérifiant une String, afin d'améliorer la complexité cyclomatique.

String value = some methodx;
switch (value) {
    case "apple":
        method1;
        break;
    case "carrot":
        method2;
        break;
    case "mango":
        method3;
        break;
    case "orange":
        method4;
        break;
}

Mais je ne suis pas sûr de la valeur que je vais obtenir.

Answer 1

Java (avant la version 7) ne prend pas en charge String dans switch/case. Mais vous pouvez obtenir le résultat souhaité en utilisant une énumération.

private enum Fruit {
    apple, carrot, mango, orange;
}

String value; // supposer une entrée
Fruit fruit = Fruit.valueOf(value); // entourer de try/catch

switch(fruit) {
    case apple:
        method1;
        break;
    case carrot:
        method2;
        break;
    // etc...
}

Answer 2

Apprenez à utiliser else.

Étant donné que valeur ne sera jamais égale à deux chaînes inégales en même temps, il n'y a que 5 résultats possibles -- un pour chaque valeur qui vous intéresse, plus un pour "aucun des cas ci-dessus". Mais parce que votre code n'élimine pas les tests qui ne peuvent pas réussir, il y a 16 chemins "possibles" (2 ^ le nombre de tests), dont la plupart ne seront jamais suivis.

Avec else, seuls les 5 chemins qui peuvent réellement se produire existent.

String value = some methodx;
if ("apple".equals(value )) {
    method1;
}
else if ("carrot".equals(value )) {
    method2;
}
else if ("mango".equals(value )) {
    method3;
}
else if ("orance".equals(value )) {
    method4;
}

Ou commencez à utiliser JDK 7, qui inclut la capacité d'utiliser des chaînes dans une instruction switch. Bien sûr, Java va simplement compiler le switch en une structure du type if/else de toute façon...

Answer 3

Pour réduire la complexité cyclomatique, utilisez une map :

Map> map = new HashMap<>();
map.put("apple", new Callable() { public Object call() { method1(); return null; } });
...
map.get(x).call();

ou le polymorphisme

Answer 4

Juste pour concrétiser la réponse d'emory, le code exécutable est le suivant :

  Map> map = new HashMap>();
  map.put( "test" , new Callable () { public User call (){ return fillUser("test" ); }} ) ;
  map.put( "admin" , new Callable () { public Utente call (){  return fillUser("admin" ); }} ) ;

où l'utilisateur est un POJO, et ensuite

  User user = map.get(USERNAME).call();

enfin la méthode appelée est quelque part :

 private User fillUser(String x){        
        User user = new User();
        // set something in User
        return user;
}

Answer 5

Java ne prend pas en charge Switch-case avec String. Je suppose que ce lien peut vous aider. :)