Naughts and Crosses (Tic Tac Toe), 서브 루틴 쿼리

Question

저는 제 Noughts and Crosses 프로젝트의 마지막 단계에 있습니다. 나는 꽤 꼼꼼히 갇혀 있습니다. 나는 이동 검증 서브 루틴과 서브 루틴을 전적으로 수행했습니다 상자의 빈 공간을 "X"또는 "O"로 변경하는 것에 기인하여 내 코드가 내 코드의 일부가 현재 컨텍스트에 존재하지 않고 완전히 당황한 것으로 알리는 것 같습니다.

코드는 다음과 같습니다 내가 갖고있는 것 같다

namespace ConsoleApplication1 
{ 
class Program 
{ 
    static void Main(string[] args) 
    { 
     string[,] grid = new string[3, 3] {{" "," "," "}, 
              {" "," "," "}, 
              {" "," "," "}}; 
     string board = System.IO.File.ReadAllText("E:\\BoardGame.txt"); 
     Console.WriteLine(board); 
     int player = 0; 
     var XCoordinate = 0; 
     var YCoordinate = 0; 
     int x, y; 
     GetMoveCoordinates(ref XCoordinate, ref YCoordinate); 
     if (player == 0) 
     {     
      grid[XCoordinate, YCoordinate] = " X "; 
      player++; 
     } 
     else 
     { 
      grid[XCoordinate, YCoordinate] = " O "; 
      player--; 
     }    
     UpdateGrid(grid, box); 
     if (player == 1) 
     { 
     } 

    } 


    public static void GetMoveCoordinates(ref int XCoordinate, ref int YCoordinate) 
    { 
     int CommaLocation; 
     bool GameHasBeenWon = false; 
     string CoordinatesInput; 
     string XChar, YChar; 
     while (GameHasBeenWon == false) 
     { 
      try 
      { 
       Console.Write("Enter your coordinates: (x,y) "); 
       CoordinatesInput = Console.ReadLine(); 
       CommaLocation = CoordinatesInput.IndexOf(",".ToString()); 
       XChar = CoordinatesInput.Substring(CommaLocation - 1, CommaLocation); 
       YChar = CoordinatesInput.Substring(CommaLocation + 1); 
       XCoordinate = int.Parse(XChar); 
       YCoordinate = int.Parse(YChar); 
      } 
      catch 
      { 
       Console.WriteLine("Invalid Input- Please Try Again"); 
      } 
     } 
    } 


    public static bool CheckValidMove(int XCoordinate, int YCoordinate, string[,] Board) 
    { 
     if ((XCoordinate >= 1) || (XCoordinate <= 3)) 
     { 
      if ((YCoordinate >= 1) || (YCoordinate <= 3)) 
      { 
       if ((Board[XCoordinate, YCoordinate]) == " ") 
       { 
        return true; 
       } 
       else return false; 
      } 
      else return false; 
     } 
     else return false; 
    } 


    public static void UpdateGrid(string[,] grid, string box) 
    { 
     Console.Clear(); 
     for (int x = 0; x < grid.GetLength(0); x++) 
     { 
      for (int y = 0; y < grid.GetLength(1); y++) 
      { 
       box = box.Replace((x + 1) + "," + (y + 1), grid[y, x]); 
      } 
     } 
     // In the case not required as clearning the console default the cursor back to 0,0, but left in 
     // as an example 
     Console.SetCursorPosition(0, 0); 
     Console.WriteLine(box); 
    } 
} 
} 

그러나 문제는 코드가 나에게 UPDAT에서 해당 상자를 알 것 같다 if 문에서, 메인 아래 e (grid, box)는 현재 컨텍스트에 존재하지 않지만 마지막 서브 루틴에서 수행해야합니까? 나는 진술서로 그것을하기로되어 있나, 아니면 뭔가 빠져 있니? 또한 코드를 정리하는 방법에 대한 조언이 있으면 기꺼이 감사하겠습니다 (예 : 매개 변수를 추가하지만 먼저 기호를 그립니다).

이 그리드는이 모습입니다 : 몇 가지 오류가 여기에있다

+---+---+---+ 
| | | | 
+---+---+---+ 
| | | | 
+---+---+---+ 
| | | | 
+---+---+---+ 

Answer 1

. 먼저, 다음은 컴파일되지 않습니다 :

UpdateGrid(grid, box); 

을 앤드류 휘태커는 코멘트에 나타난 바와 같이, (당신이 선언되지하거나 초기화 않음) 주 법에는 "상자"변수가 없습니다. 변수가 무엇인지 정의하고 올바르게 초기화하면 다시 컴파일됩니다.

다음에 빠른 문체 참고 다음

while (GameHasBeenWon == false) 

명시 적으로 참과 거짓을 비교하지 마십시오 -이

while (!GameHasBeenWon) 

다음 줄 언급하는 것입니다 할 수있는 올바른 방법을 on에는 몇 가지 버그가 있습니다.

(XCoordinate >= 1) || (XCoordinate <= 3) 

즉, XCoordinate> = 1 또는 그 이하입니다. 3까지는 당신이 의미하는 바가 전혀 없습니다. 실제로,이 논리에 의해 정수가 유효합니다. 1보다 크거나, 1보다 작거나, 3보다 작거나, 3과 같기 때문에 정수입니다. (생각해보십시오. 어떤 정수가 false 일 수 있습니까?) 또한 3은 구체적으로 이 아니며 유효한 인덱스이지만입니다. 배열은 인덱스가 0입니다. 따라서이 실제로 경우 문의 측면에서

(XCoordinate >= 0) && (XCoordinate < 3) 

을해야한다 :

if ((Board[XCoordinate, YCoordinate]) == " ") 
      { 
       return true; 
      } 
      else return false; 

반환 true 그 진술이 거짓 정확히 (Board[XCoordinate, YCoordinate]) == " "이 false 사실과 정확히. 너는 할 수있다

return (Board[XCoordinate, YCoordinate]) == " "; 

사실, "if"문 전체에 대해 그렇게 할 수있다. (IDE 바로 앞에 앉아 있지 않기 때문에 구문이 완벽하지 않다면 사과드립니다.)

return ((XCoordinate >= 0) && (XCoordinate < 3) && 
     ((YCoordinate >= 0) && (YCoordinate < 3)) && 
     ((Board[XCoordinate, YCoordinate]) == " "));