do while looping forever

Question

У меня возникли проблемы с этим методом, я навсегда застрял на «Извините, вы должны указать Deeppan или тонкую основу, попробуйте еще раз:« даже если я нахожу jimmy, harry, deeppan, thin, Thin .. и т. д. После того, как я набрал либо глубокий, либо тонкий, я хочу, чтобы строка хранилась в переменной «размер» и возвращалась

Любые идеи, что мне не хватает?

public String Input(){ 


    String size; 

    Scanner sc = new Scanner(System.in); 
    System.out.println("thin or thick: "); 

    do { 
     size = scan.next(); 
     if (!size.equalsIgnoreCase("thick") || !size.equalsIgnoreCase("thin")) { 
      System.out.print("Sorry you must specify a thick or thin base, try again: "); 
     } else { 
      break; 
     } 
    } while (true); 

    return size; 
} 

Answer 1

Изменить || для & & в условии if. В настоящий момент условие будет истинным, если вы наберете «тонкий», потому что он не глубокий и наоборот.

Answer 2

while(true) сделает ваш цикл бесконечен, как его всегда истинной

причине вы делаете || операция

Answer 3

Причина:

!size.equalsIgnoreCase("Deeppan") || !size.equalsIgnoreCase("thin") 

size не может быть "Deeppan" и «тонкий "в то же время :-)

:

do { 
     size = scan.next(); 
     if (size.equalsIgnoreCase("Deeppan") || size.equalsIgnoreCase("thin")) { 
      break; 
     } 
     else { 
      System.out.print("Sorry you must specify a Deeppan or thin base, try again: "); 
     } 
    } while (true); 

пс. «размер» не является лучшим именем переменной ...

Answer 4

Заявление внутри вашего if всегда true.

Вы должны использовать либо

!size.equalsIgnoreCase("Deeppan") && !size.equalsIgnoreCase("thin") 

или

!(size.equalsIgnoreCase("Deeppan") || size.equalsIgnoreCase("thin")) 

Answer 5

Он всегда будет верным, и, следовательно, никогда не достигают else

Пример 1: Размер "Deeppan":
! size.equalsIgnoreCase ("Deeppan") || ! size.equalsIgnoreCase ("thin")
=! true || ! false
= false || правда
= истинный

Пример 2: Размер "Foo":
size.equalsIgnoreCase ("Deeppan") || ! size.equalsIgnoreCase ("thin")
=! false || ! false
= true || правда
= истинный

Answer 6

Нет причин использовать do-while, попробуйте while, как это, проверка состояние становится более простым:

while (true) { 
    size = scan.next(); 
    if (size.equalsIgnoreCase("deeppan") || size.equalsIgnoreCase("thin")) 
     return size; 
    System.out.println("Sorry you must specify a Deeppan or Thin base, try again:"); 
} 

Answer 7

Yuor заявление

если (size.equalsIgnoreCase ("! Deeppan ") ||! Size.equalsIgnoreCase (" тонкий "))

всегда прав.