Вызов метода абсолютно ничего не делает?

Question

Так что прямо сейчас я конвертирую программу массива в связанный список для моего окончательного назначения. Однако я столкнулся с проблемой при попытке вызвать один из моих методов. Когда пользователь выбирает параметр 3 в ProcessChoice (также есть метод getChoice()), он возвращается в главное меню. Это методы, связанные с этим выбором. DispItem() и FindItem() находятся в другом классе. ProcessChoice() находится в пользовательском классе.

void DispItem() { 
    ItemNode current = head; 
    current = FindItem(); 
    if (current == null) { 
     System.out.println("\nThe item was not found.\n"); 
    } else current.DispItem(); 
} 

ItemNode FindItem() { 
    ItemNode current = head; 
    System.out.println("\nPlease enter the ID of the item you are looking for.\n"); 
    int ID = keyboard.nextInt(); 
    while (current != null) { 
     if (ID == current.GetItemID()) { 
      current.DispItem(); 
     } else System.out.println("Error."); 
    } 
    return (current); 
} 

public static void ProcessChoice(int qChoice, InvenLL qMyAcct) { 
    if (qChoice == 1) { 
     qMyAcct.DispItems(); 
    } else if (qChoice == 2) { 
     qMyAcct.AddItem(); 
    } else if (qChoice == 3) { 
     qMyAcct.DispItem(); 
    } 
    /*else if (qChoice == 4) 
       { 
        qMyAcct.ModifyItem(); 
       }*/ 
    else if (qChoice == 5) { 
     System.out.println("\nYou have exited the program. The item data will now be saved. Good bye!\n"); 
    } 
} 

Answer 1

Ваш цикл while внутри FindItem() никогда не переназначает текущий, похоже, что он будет работать навсегда. Возможно, вы запускаете это внутри блока try-catch?

Answer 2

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

void dispItem(){ 
    ItemNode current = head; 
    current = findItem(); 
     if(current == null){ 
      System.out.println("\nThe item was not found.\n"); 
     } 
     else current.dispItem(); 
} 

ItemNode findItem(){ 
    ItemNode current = head; 
    System.out.println("\nPlease enter the ID of the item you are looking for.\n"); 
    int ID = keyboard.nextInt(); 
    while(current != null){ 
     if(ID == current.getItemID()){ 
      current.dispItem(); 
     } 
     else System.out.println("Error."); 
    } 
    return (current); 

} 

public static void processChoice(int qChoice, InvenLL qMyAcct){ 
    switch(qChoice){ 
    case 1: 
     qMyAcct.dispItems(); 
     break; 
    case 2: 
     qMyAcct.addItem(); 
     break; 
    case 3: 
     qMyAcct.dispItem(); 
     break; 
    case 4: 
     qMyAcct.modifyItem(); 
     break; 
    case 5: 
     System.out.println("\nYou have exited the program. The item data will now be saved. Good bye!\n"); 
     break; 
    } 
} 

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

while(current != null){ 
    if(ID == current.getItemID()){ 
     current.dispItem(); 
    } 
    else System.out.println("Error."); 
} 

Как написано, этот цикл никогда не кончится, потому что current никогда не обновляется внутри цикла.

Вы, вероятно, имел в виду сделать что-то вроде этого:

while(current != null){ 
    if(ID == current.getItemID()){ 
     current.dispItem(); 
    } 
    else System.out.println("Error."); 

    current = current.next(); //or whatever function returns the next node. 
} 

Однако это все не имеет смысла. findItem не должен печатать ошибки или отображать содержимое списка; он должен найти пункт и ничего больше. Это результат смешивания вашей бизнес-логики с логикой приложения.

Правильная реализация такого рода вещей должна быть полностью отделена от подсказок и т. Д. Фактически, в реальном мире не будет реализовываться связанный список вообще - можно использовать java.util.LinkedList. Я подозреваю, что это не вариант, поскольку это школьное задание. Однако вы можете сделать следующее самое лучшее, и используйте java.util.AbstractSequentialList. Если это не разрешено, то внесите java.util.List самостоятельно или, по крайней мере, создайте реализацию связанного списка на этом интерфейсе. Помимо этого, SMOP должен был реализовать необходимую структуру меню.