Использовать строку в корпусе переключателя в java

Мне нужно изменить следующий if на switch - case, проверяя для String, чтобы улучшить циклическую сложность.

String value = some methodx;
if ("apple".equals(value)) {
    method1;
}

if ("carrot".equals(value)) {
    method2;
}

if ("mango".equals(value)) {
    method3;
}

if ("orange".equals(value)) {
    method4;
}

Но я не уверен, какую ценность я получу.

Ответ 1

Java (до версии 7) не поддерживает String в switch/case. Но вы можете достичь желаемого результата с помощью перечисления.

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

String value; // assume input
Fruit fruit = Fruit.valueOf(value); // surround with try/catch

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

Ответ 2

Научитесь использовать else.

Так как value никогда не будет равным двум неравным строкам сразу, есть только 5 возможных результатов - по одному для каждого значения, о котором вы заботитесь, плюс один для "ни одного из вышеперечисленных". Но поскольку ваш код не устраняет те тесты, которые не могут пройти, он имеет 16 "возможных" путей (2 ^ количество тестов), из которых большинство никогда не будут соблюдаться.

С else единственными путями, которые существуют, являются 5, которые действительно могут произойти.

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;
}

Или начните использовать JDK 7, который включает в себя возможность использования строк в инструкции switch. Конечно, Java просто компилирует switch в конструкцию типа if/else в любом случае...

Ответ 3

Чтобы уменьшить циклическую сложность, используйте карту:

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

или полиморфизм

Ответ 4

Теперь все используют Java 7, верно? Вот ответ на оригинальную проблему:

String myString = getFruitString();

switch (myString) {

    case "apple":
        method1();
        break;

    case "carrot":
        method2();
        break;

    case "mango":
        method3();
        break;

    case "orange":
        method4();
        break;
}

Примечания

  • Операторы case эквивалентны использованию String.equals.
  • Как обычно, соответствие строк зависит от регистра.
  • В соответствии с docs, это, как правило, быстрее, чем использование цепочечных операторов if - else (как в cHao answer).

Ответ 5

Чтобы сделать конкретный ответ emory, исполняемый код следующий:

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

где пользователь является POJO, а затем

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

наконец, вызываемый метод где-то:

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

Ответ 6

Java не поддерживает Switch-case со String. Я думаю, эта ссылка может вам помочь.:)

Ответ 7

Вот возможный способ до 1.7, который я не могу рекомендовать:

public class PoorSwitch
{
    final static public int poorHash (String s) {
        long l = 0L;
        for (char c: s.toCharArray ()) {
            l = 97*l + c;
        }
        return (int) l;
    }

    public static void main (String args[])
    {
        String param = "foo";
        if (args.length == 1)
        {
            param = args[0];
        }
        // uncomment these lines, to evaluate your hash
        // test ("foo");
        // test ("bar");
        switch (poorHash (param)) {
            // this doesn't work, since you need a literal constant
            // so we have to evaluate our hash beforehand:
            // case poorHash ("foo"): {
            case 970596: {
                System.out.println ("Foo!");
                break;
            }
            // case poorHash ("bar"): {
            case 931605: {
                System.out.println ("Bar!");
                break;
            }
            default: {
                System.out.println ("unknown\t" + param);
                break;
            }
        }
    }

    public static void test (String s)
    {
        System.out.println ("Hash:\t " + s + " =\t" + poorHash (s));
    }
}

Возможно, вы могли бы работать с таким трюком в сгенерированном коде. Иначе я не могу это порекомендовать. Не так много, что возможность столкновения хэшей заставляет меня беспокоиться, но если что-то перепутано (вырезать и вставить), трудно найти ошибку. 931605 не является хорошей документацией.

Возьмите его как доказательство концепции, как любопытство.

Ответ 8

Мы можем применять Switch только для совместимого типа данных: short, Shor, byte, Byte, int, Integer, char, Character или enum type.

Ответ 9

Оценка переменных String с помощью оператора switch реализована в Java SE 7, и, следовательно, она работает только в java 7. Вы можете также посмотрите, как эта новая функция реализована в JDK 7.

Ответ 10

Не очень красиво, но вот еще один способ:

String runFct = 
        queryType.equals("eq") ? "method1":
        queryType.equals("L_L")? "method2":
        queryType.equals("L_R")? "method3":
        queryType.equals("L_LR")? "method4":
            "method5";
Method m = this.getClass().getMethod(runFct);
m.invoke(this);

Ответ 11

String value = someMethod();
switch(0) {
default:
    if ("apple".equals(value)) {
        method1();
        break;
    }
    if ("carrot".equals(value)) {
        method2();
        break;
    }
    if ("mango".equals(value)) {
        method3();
        break;
    }
    if ("orance".equals(value)) {
        method4();
        break;
    }
}