код, не выполняющийся в операторе switch

Question

Возможно, я пропустил что-то очень простое, но я новый, и попросить о помощи просто попросить. Это домашняя работа, поэтому вы можете дать подсказки, если хотите, но, пожалуйста, поймите, что я ужасен в java.

Ниже приведен код для адресной книги. Я хочу, чтобы пользователь входил в 1, чтобы просмотреть записи 2, чтобы добавить другую запись в книгу или 3, чтобы выйти. Я получил программу для работы, но она не зациклилась, чтобы спросить пользователя, что делать дальше. Затем я закодировал оператор switch, и когда пользователь выбирает 1, программа не запускает код, связанный с case 1: и он совпадает с случаем 2. Программа выполняет проверку моей записи (я написал валидатор в отдельном классе)

Я что-то пропустил при кодировании?

снова Я новый, поэтому не бить меня.

import java.util.Scanner; 

    public class AddressBookEntryApp 
    { 

public static void main(String[] args) 
{ 

    //create new scanner 
    Scanner ip = new Scanner(System.in); 
    //welcome user to the address book application 
    System.out.println("Welcome to the Address Book Application"); 
    System.out.println(); 
    int choice = 0; 
    boolean quit = false; 
    do 
    { 

     //have the user enter a menu number 
     System.out.println("1 - List entires"); 
     System.out.println("2 - Add entry"); 
     System.out.println("3 - Exit"); 


     System.out.println(); 
     int menuNumber = Validator.getInt(ip, "Enter menu number: ", 1, 3); 
     System.out.println(); 
     switch (choice) 
     { 
     case 1: 

       AddressBookIO GetEntryObject = new AddressBookIO(); 
       GetEntryObject.getEntriesString(); 
       System.out.println(AddressBookIO.getEntriesString()); 
       break; 

     case 2: 


       String name = Validator.getEntry(ip, "Enter name: "); 
       String email = Validator.getEntry(ip, "Enter email address"); 
       String phone = Validator.getEntry(ip, "Enter phone number: "); 
       AddressBookEntry newEntry = new AddressBookEntry(name, email, phone); 
       AddressBookIO.saveEntry(newEntry); 
       break; 


     } 
}while (!quit); 

    } 
    } 

Answer 1

Вы не меняете значение переменной choice. Поэтому он остается «0», как вы инициализировали в начале. Таким образом, ваш код никогда не попадает в утверждения дела. Это проблема . Попробуйте следующее:

choice = Validator.getInt(ip, "Enter menu number: ", 1, 3); 

Answer 2

Вы назначаете выбор с 0 (int choice = 0;) и никогда не меняется, поэтому в коммутаторе он не соответствует ни 1, ни 2.

Это должно вероятно быть

switch (menuNumber) 

Answer 3

choice установлен на 0. Возможно, вы включили menuNumber?

Answer 4

Да, вы назначили выходной сигнал валидатора на menuNumber, но вы делаете switch на choice.

Answer 5

В случае, если не эта линия

int menuNumber = Validator.getInt(ip, "Enter menu number: ", 1, 3); 

быть преобразованы к следующему:

choice = Validator.getInt(ip, "Enter menu number: ", 1, 3); 

Answer 6

Все в порядке, как указано ранее, "выбор" переменная не вход пользователя («menuNumber», я думаю). Это первый вопрос.

Теперь возникает вопрос: почему вы допустили эту ошибку? Потому что ваш код слишком сложный. Вы напишете гораздо лучший код, если будете читать советы от «чистого кода» Мартина Фаулера или других. Два простых совета:

• Сохраняйте небольшие функции, и будет намного легче идентифицировать ошибки.
• предпочитает «пока», чем «делать/пока», его гораздо легче читать.

Бюст самое главное, научиться использовать отладчик, с ним вы бы нашли ошибку в кратчайшие сроки.