왜 checkforWin (File f)이 true 일 때 true를 반환하지 않습니까?

Question

** 

이 이동하고 추가 이동 후 checkForWin의 운전 방식을이기는 이상이 고모 쿠 게임 제작에 여분의 이동 원인은이 직후 checkForWin 방법이어야 승리를 감지 방법하지만 입니다 해당 makeMove 메서드입니다.

**

import java.io.File; 
boolean hasWinner = false; 
File gameFile = new File("src/centralGameFile.txt"); 

do{ 
    //player 1 
    makeMove(gameFile); 
    // check for win 
    if (checkForWin(gameFile)){ 
     hasWinner = true; 
     break; 
    } 
    // player 2 
    makeMove(gameFile); 
    // check for win 
    if (checkForWin(gameFile)){ 
     hasWinner = true; 
     break; 
    } 
}while(hasWinner == false); 

System.out.println("somebody has won the game"); 

/*this method is located in another class in the same package and is 
    called from an instance of the class using the access operator */ 

protected boolean checkForWin(File f){ 
//return true if the file has a winner in it using scanner to look for it 
//this method works correctly when tested with just a file in a test class 
} 

// 간결

/* makeMove(File f) method copies the text from f and over writes 
it adding another character; in context this is a gomoku/tic-tac-toe 
style game but on a bigger board. 
*/ 

Answer 1

checkForWin works correctly when tested with just a file in a test class 

코드의 일부 생략/catch 블록 시도 :

do{ 
    //player 1 
    makeMove(gameFile); 
    // check for win 
    if (checkForWin(gameFile)){ 
     hasWinner = true; 
     break; 
    } 
    // player 2 
    makeMove(gameFile); 
    // check for win 
    if (checkForWin(gameFile)){ 
     hasWinner = true; 
     break; 
    } 
}while(hasWinner == false); 

System.out.println("somebody has won the game"); 

을하는 경우를은 true을 반환하며 방법은 makeMove(gameFile)에 매달려 있어야합니다. 이것은 아마도 무한 루프에서 멈출 수 있습니다.

Answer 2

나는 귀하의 문제의 원인은 checkForWin이 사실 이 아니며,이 아니라는 것을 제안합니다. 이있을 수 있기 때문에 :

그 방법 당신의 테스트가 불충분
• , 또는
• 파일을 업데이트하는 코드 (예를 들어,이 파일을 플러시/종료되지 않음) 옳은 일을하고 있지 않거나
• 잘못된 파일을 checkForWin (으)로 호출하고 있습니다.

어느 쪽이든, 여기에 실제로 무엇이 일어나고 있는지에 대한 충분한 정보가 없습니다. 최소한 checkForWin 메소드의 코드를 볼 필요가 있으며 아마도 파일을 업데이트하는 코드를 볼 필요가있을 것입니다.

---

내가 관심을 가지고 있지만 ...이 코드에서 사소한 오류의 커플 ...하지만 당신에 대해 묻는 문제를 야기하기에 충분하지 :

이

1. 귀하의 hasWinner 플래그가를 이중화는 그것을 설정하고 테스트하는 코드와 동일합니다. 루프를 작성한 방법은 두 개의 break 문 중 하나를 실행 한 경우 루프가 끝난 후 명령문을 얻을 수있는 유일한 방법입니다.

2. 이 (다른 상황에서) 나쁜 스타일 ... 잠재적 위험 :

   ... while (!hasWinner); 
   

   는 첫째, 더 읽을로

   ... while (hasWinner == false); 
   

   이 작성되어야한다. 모든 Java 프로그래머는 ! 연산자가 무엇을 의미하는지 알고 있어야하며 !을 사용하면이를 표현하는 관용적 인 방법입니다.

   둘째, 오류가 발생하기 쉽습니다. 이것을 고려하십시오 : 당신이 실수로 = 대신 ==를 작성한 다음

   ... while (hasWinner = false); 
   

   . 유감스럽게도 = 양식은 합법적 인 Java이며 ... 과 다른 의미로을 의미합니다. 관용적 인 버전을 사용하면이 실수를 할 수 없습니다.