호출하는 메소드는 절대적으로 아무것도하지 않습니까?

Question

그래서 지금은 배열 프로그램을 최종 할당을위한 링크 된 목록으로 변환하고 있습니다. 그러나, 내 방법 중 하나를 호출하려고 할 때 문제가 발생했습니다. 사용자가 ProcessChoice에서 옵션 3을 선택하면 (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

FindItem() 내부의 while 루프는 현재를 재 할당하지 않으며 영원히 반복하지 않는 것처럼 보입니다. 어쩌면 이것을 try-catch 블록 내에서 실행하고 있습니까?

Answer 2

코드의 첫 번째 문제는 스타일입니다. 여기가 자바의 코딩 규칙을 사용하여 어떻게 보일지입니다 : 문체의 문제와

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은 오류를 인쇄하거나 목록 내용을 표시하면 안됩니다. 그것은 이고 다른 것은입니다. 이는 비즈니스 논리를 응용 프로그램 논리와 혼합 한 결과입니다.

이 종류의 것을 올바르게 구현하면 linkedlist이 메뉴 프롬프트와 다른 것들로부터 완전히 분리됩니다. 실제로 실제 세계에서는 연결된 목록을 전혀 구현하지 않을 것입니다. 하나는 java.util.LinkedList입니다. 나는 이것이 학교 임무이기 때문에 이것이 선택 사항이 아니라고 생각한다. 그러나 가장 좋은 방법은 java.util.AbstractSequentialList입니다. 허용되지 않는 경우 직접 java.util.List을 구현하거나 해당 인터페이스에서 연결된 목록 구현을 패턴 화하십시오. 그 외에도 필요한 메뉴 구조를 구현하는 것은 SMOP 일 것입니다.