Failed текст проверки

Question

У меня есть фрагмент кода, который последовательно дает мне ошибку:

do { 
    System.out.println("Choose Role: (Manager, Developer, QA) "); 
    role = scan.nextLine(); 
    // For testing:   /////////////////////// 
    System.out.println("role is: " + role); 
    //////////////////////////////////////////////// 
    if (is_numeric(role)) { 
     System.out.println("Invalid Input."); 
     continue; 
    } else if (!role.equalsIgnoreCase("MANAGER") || !role.equalsIgnoreCase("DEVELOPER") || !role.equalsIgnoreCase("QA")) { 
     System.out.println("Invalid Role"); 
     continue; 
    } else { 
     break; 
    } 
} while (true); 

Я добавил «Для тестирования» блок просто увидеть, если по какой-то причине есть что-то происходит с переменной роли , но его нет. Независимо от того, как я пишу менеджер/разработчик/qa (будь то в шапках, маленьких буквах и т. Д.), Запускается «Неверная роль», и цикл переходит снова.

Любые предложения?

Answer 1

Логически, этот тест является неправильным

(!role.equalsIgnoreCase("MANAGER") || 
!role.equalsIgnoreCase("DEVELOPER") || 
!role.equalsIgnoreCase("QA")) 

Почему? Потому что, если роль = «МЕНЕДЖЕР», она не равна «РАЗРАБОТЧИК» (или «ОК») и наоборот. Я думаю, что вы хотели

(!role.equalsIgnoreCase("MANAGER") && 
!role.equalsIgnoreCase("DEVELOPER") && 
!role.equalsIgnoreCase("QA")) 

Answer 2

!role.equalsIgnoreCase("MANAGER") || !role.equalsIgnoreCase("DEVELOPER") || 
       !role.equalsIgnoreCase("QA") 

Не то, что вы хотите, замените || с &&.

В своем коде вы говорите: Если role не «МЕНЕДЖЕР» ИЛИ если role не «РАЗРАБОТЧИК» ИЛИ если role не «ОК».

Благодаря Short-circuit evaluation, если первое условие true, остальные не будут оценены, потому что true || anything всегда true.

Answer 3

Java, если тестовое короткое замыкание для булевых выражений.

Изменение || в &&

Answer 4

Вопреки тому, что все остальные говорят, если то, что вы пытаетесь модели «не действует», то не распространять негатив по каждому условию и использовать &&. Это не правильное моделирование. Модель «действует», поставить скобки вокруг него и свести на нет всю вещь:

if (!(role.equalsIgnoreCase("MANAGER") || role.equalsIgnoreCase("DEVELOPER") || 
      role.equalsIgnoreCase("QA"))) { 
    //... 
} 

Но это тот случай, когда вспомогательный метод улучшит ясности кода в значительной степени. Первая модель, что вы хотите модель, а не обратное:

public boolean validRole(String role) { 
    return role.equalsIgnoreCase("MANAGER") || 
      role.equalsIgnoreCase("DEVELOPER") || 
      role.equalsIgnoreCase("QA"); 
} 

Тогда ваше заявление, если трудно получить неправильно и сам документы:

if (!validRole(role)) { 
    //... 
}