UPC 체크 코드 무한 루프를 중지하는 방법

Question

내 코드에 무한 루프가있어 문제를 해결할 수 없습니다. 또한 빈 줄을 삽입 할 때 색인 오류가 있다고합니다. 도와주세요!

유효한 UPC를 확인하는 첫 번째 단계는 홀수 위치 숫자를 더하고 3을 곱한 다음 그 숫자를 짝수 위치 숫자의 합에 더하는 것입니다. 그런 다음 나머지를 10으로 나눌 때 계산합니다. 나머지가 0이 아닌 경우이 나머지를 10에서 빼서 체크 숫자를 얻습니다. 나머지가 제로의 경우, 다음 검사 숫자 0

public static void main(String[] args) { 
      Scanner in = new Scanner (System.in); 
      System.out.println("Enter a UPC (enter a blank line to quit): "); 
      String enterUPC = in.nextLine(); 
      int length = enterUPC.length(); 
      int checkDigit=0; 

      char char1, char2, char3, char4, char5, char6, char7, char8, char9, char10, char11, char12; 
      int num1, num2, num3, num4, num5, num6, num7, num8, num9, num10, num11, num12; 


      char1 = enterUPC.charAt(0); 
      num1 = Character.getNumericValue(char1); 
      char2 = enterUPC.charAt(1); 
      num2 = Character.getNumericValue(char2); 
      char3 = enterUPC.charAt(2); 
      num3 = Character.getNumericValue(char3); 
      char4 = enterUPC.charAt(3); 
      num4 = Character.getNumericValue(char4); 
      char5 = enterUPC.charAt(4); 
      num5 = Character.getNumericValue(char5); 
      char6 = enterUPC.charAt(5); 
      num6 = Character.getNumericValue(char6); 
      char7 = enterUPC.charAt(6); 
      num7 = Character.getNumericValue(char7); 
      char8 = enterUPC.charAt(7); 
      num8 = Character.getNumericValue(char8); 
      char9 = enterUPC.charAt(8); 
      num9 = Character.getNumericValue(char9); 
      char10 = enterUPC.charAt(9); 
      num10 = Character.getNumericValue(char10); 
      char11 = enterUPC.charAt(10); 
      num11 = Character.getNumericValue(char11); 
      char12 = enterUPC.charAt(11); 
      num12 = Character.getNumericValue(char12); 



      while(length > 0) { 
       //Algorithm Step One and Algorithm Step Two 
       int stepOne = (num1 + num3 + num5 + num7 + num9 + num11) * 3; 
       int stepTwo = stepOne + (num2 + num4 + num6 + num8 + num10); 

       while (length == 12) { 
       //Algorithm Step Three 
       if(stepTwo%10!=0) { 
        checkDigit = 10 - (stepTwo%10); 
       } 
       else { 
        checkDigit = 0;} 

       System.out.println("Check digit should be : " + checkDigit); 
       System.out.println("Check digit is: " + num12); 

       } 
      } 
       if(checkDigit == num12) { 
        System.out.println("UPC is valid"); 
       } 
       else { 
        System.out.println("UPC is not valid"); 
       } 


       if (length == 0){ 
        System.out.println("ERROR! UPC MUST have exactly 12 digits"); 
       } 
       else{ 
        System.out.print("Goodbye!"); 
       } 

      } 
    } 

Answer 1

당신의 while 루프 변수 length을 확인하고 있습니다,하지만 당신은 루프 내에서이를 변경하지 않습니다해야합니다. 따라서 루프는 끝나지 않습니다.

스캐너에서 숫자를 반복해서 읽고 UPC 알고리즘을 수행하려는 경우 읽기, 숫자 나누기 및 길이 계산이 모두 루프 안에 있어야합니다. 이 작업을 한 번만하고 싶다면 루프가 없어야합니다.

모든 것이 내부로 이동했기 때문에 루프 상태에서 길이를 검사해서는 안됩니다. 이 루프 스캐너의 당신의 정의 후 바로해야한다는

while (true) { 

    System.out.println("Enter a UPC (enter a blank line to quit): "); 
    String enterUPC = in.nextLine(); 
    int length = enterUPC.length(); 

    if (length == 0) { 
     break; 
    } 

    // All the rest of the processing is done here. 

} 

참고 : 대신, DO-while 루프 또는 무한 While 루프 하나를 만들 수 있습니다. 그리고 그것이 끝나는 방법은 사용자가 들어간 후 길이를 검사하는 것입니다. 이 때문에 거기에 if 성명이 있습니다.

Answer 2

while 루프에서 길이를 줄여야합니다. 그렇지 않으면 length는 항상 12와 같거나 0보다 커야합니다. 또한 모든 선언을 입력하는 대신 int를 사용하는 것이 좋습니다. 그런 다음 temp char 변수를 사용하여 for 루프를 사용하여 루프를 반복 할 수 있으므로 그런 모든 것을 초기화 할 필요가 없습니다. int 배열을 통해 while 루프에서 다음

char[] arrayChars = new char[enterUPC.length()]; 
int[] arrayInts = new int[enterUPC.length()]; 
for(int i=0;i<enterUPC.length();i++) { 
    char temp=enterUPC.charAt(0); 
    num[i] = Character.getNumericValue(temp); 
} 

, 루프를 모든 0 + 모든 짝수 인덱스를 추가하고 3을 곱 모든 홀수 인덱스를 추가 : 대한 루프는 다음과 같이 할 수있다. 또한 회선이 비어 있는지 확인하지 않습니다.