0

ゲーム TicTacToe を実行するプログラムを作成しようとしています。すべてのメソッドの作成が完了し、あとはドライバー プログラムを作成するだけです。ドライバプログラムを作成する前に、ボードを文字と一緒に印刷しようとしましたが、私の方法は正しくないと思います。私のエラーは次のようになります。

java.lang.ArrayIndexOutOfBoundsException: 3
at TicTacToeBoard.move(TicTacToeBoard.java:75)
at TicTacToe.main(TicTacToe.java:24)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at edu.rice.cs.drjava.model.compiler.JavacCompiler.runCommand(JavacCompiler.java:271)

ここに私の2つのプログラムがあります:

これは私のドライバー プログラムですが、まだ完成していないようです。最後に表示されるのはテンプレートです。これにより、各プログラムがどのように機能するかがわかります。

class TicTacToe
{
 public static void main(String [] args)
 { 
  //System.out.println("Welcome! Tic-Tac-Toe is a two player game.");
  //System.out.println("Enter player one's name: ");

  TicTacToeBoard game = new TicTacToeBoard();
  System.out.println(game.toString());

  //int count = 0;


  game.move('x', 1, 3); 
  // game.move('o', 1, 1);

  /* while (game.gameWon() !true || count != 9)
     {
      System.out.print(game.move()); 
      System.out.print(game.isEmpty()); 
      }*/   
     }
    }

これがすべてのメソッドの場所です......

class TicTacToeBoard
{
 private char [][] board = new char[3][3];
 String b;

 // This a new constructor that creates a tic-tac-toe board
 public TicTacToeBoard()
 {
  for (int rows = 0; rows < board.length; rows++)// creates rows
  {
   for (int columns = 0; columns <board[rows].length;columns++)// creates columns
   {
    //System.out.println("| ");
     board[rows][columns] = ' ';
    //System.out.println(" |\n" );
    }
   }
 }

  // creates a string form of the tic-tac-toe board and allows the user
  // to access it during the game.
  public String toString()
  {
   String b = "";

   // creates a vertical bar at the beginning and the end of each row
   for (int rows = 0; rows < board.length; rows++)
   {
     b += "| ";
     // adds a space for each row and column character in tic-tac-toe board.
     for (int columns = 0; columns < board[rows].length; columns++)
     {
      b += board[rows][columns] + " ";  
      }
       b += "|\n";// prints a | space space space | and breaks off to create two new lines.
      }
       return b; // prints the tic-tac-toe board to be accessed by the user.
     }

String move(char x, int rows, int columns)
{ 
 String b = "";

 // creates a vertical bar at the beginning and the end of each row
 for (int r = 0; r < board.length; r++)
 {
   b += "| ";

  for (int c = 0; c < board[r].length; c++)
  {
   b += board[r][c] + " ";  //prints 3 spaces on each line. 
    // prints string character from user input if row and column not equal to zero 
   if (board[rows - 1][columns - 1] >= 0 && board[rows - 1][columns - 1] <= 2 )
   {
    board[rows - 1][columns - 1] = x;// prints character in the specified index from user input 
    b += board[rows - 1][columns - 1];// prints out the board and the new character in specified space.
     } 
      else if (board[rows - 1][columns - 1] < 0)   // makes user pick another choice
      return "ILLEGAL MOVE, TRY AGAIN!"; 
      // adds a space for each row and column character in tic-tac-toe board.
     }
      b += "|\n";// prints a | space space space | and breaks off to create two new lines.
    }  
     return b; // prints the tic-tac-toe board to be accessed by the user.

   }

  // checks if a space character is empty
  void isEmpty(char x, int row, int col)
  {
   if (board [row - 1][col - 1] == ' ')
    board[row - 1][col - 1] = x;
   else // makes user pick another row and column if space character is not empty
   System.out.println("ILLEGAL CHOICE, PICK AGAIN!");
   }
  // checks if game is won   
 public boolean gameWon(int row, int col)
 {  
  if ((board[2][0] == board[1][1]) && (board[2][0] == board[0][2]))
   return true;
  else if ((board[2][0] != board[1][1]) && (board[2][0] != board[0][2]))
   return false;   

  if ((board[2][2] == board[1][1])&& (board[2][2] == board[0][0]))
   return true;
  else if ((board[2][2] != board[1][1])&& (board[2][2] != board[0][0]))
   return false;   

  if ((board[0][0] == board[1][0]) && (board[0][0] == board[2][0]))
   return true;
  else if ((board[0][0] != board[1][0]) && (board[0][0] != board[2][0]))
   return false;        

  if ((board[0][1] == board[1][1]) && (board[0][1] == board[2][1]))
   return true;
  else if ((board[0][1] != board[1][1]) && (board[0][1] != board[2][1]))
   return false;        

  if ((board[0][2] == board[1][2]) && (board[0][2] == board[2][2]))
   return true;
  else if ((board[0][2] != board[1][2]) && (board[0][2] != board[2][2]))
   return false;   

  if ((board[0][0] == board[0][1]) && (board[0][0] == board[0][2]))
   return true;
  else if ((board[0][0] != board[0][1]) && (board[0][0] != board[0][2]))
   return false;   

  if ((board[1][0] == board[1][1]) && (board[1][0] == board[1][2]))
   return true;
  else if ((board[1][0] != board[1][1]) && (board[1][0] != board[1][2]))
   return false;     

  if ((board[2][0] == board[2][1]) && (board[2][0] == board[2][2]))
   return true;
  else 
   return false;
 }
}

これが全体のテンプレートです!!!!!

class TicTacToe
{
  public static void main (String [] args)
  {
       TicTacToeBoard b = new TicTacToeBoard();

       while (game not over)
       {
            swtich player
            increment turn counter

            until user enters a valid move    
            {
                  prompt for move
             }

            make move
            b.makeMove (player, row, col);

            print board    
            System.out.println(b);
       }

       print outcome
  }
 }


 class TicTacToeBoard
 {
   private char [][] board = ...;

   public TicTacToeBoard()
   {
      initialize board with spaces
    }

   public void makeMove (char c, int row, int col)
   {
        store symbol in specified position
    }

   public boolean isEmpty(int row, int col)
   {
        return true if square is unfilled
    }

   public boolean gameWon()
   {
        check board for a win
    }

   public String toString ()
   {
        return String representation of board
    }

 }
4

1 に答える 1

4

プログラミング言語はエラーに対して容赦がなく、厳密さと注意を私たちに強います。

あなたのコードは私たちが読むのが非常に難しいため、私たちにとってもあなたにとっても、いたるところにあるインデントから始めてデバッグする必要がありますが、特に次のような不注意なエラーもあります。

for (int r = 0; r < board.length; rows++)

ここで何が悪いのか分かりますか?rは と同じではなく、rows一方をループのインデックスとして使用してから他方をインクリメントすることはできません。ループ内でこれらの変数の両方を使用しています。コードには他にもいくつかの不注意なエラーがあります。

最初からやり直すことをお勧めしますが、コードにはもっと注意を払い、特にインデントには注意してください。中括弧を正しく並べないと、あるコード ブロックがいつ終了し、別のコード ブロックが開始するかがわかりません (私たちも同じです)。

ああ、次回は、エラーの原因となっているコードの行をお知らせください。この情報を推測する必要がなければ、お手伝いがずっと簡単になります。

編集
新しいコードのインデントはいくらか改善されていますが、まだオフです。これはあなたが持っているものです:

String move(char x, int rows, int columns)
{ 
 String b = "";

 // creates a vertical bar at the beginning and the end of each row
 for (int r = 0; r < board.length; r++)
 {
   b += "| ";

  for (int c = 0; c < board[r].length; c++)
  {     
   b += board[rows][columns] + " ";  

これは私が推奨するものです:

String move(char x, int rows, int columns) {
  String b = "";

  // creates a vertical bar at the beginning and the end of each row
  for (int r = 0; r < board.length; r++) {
     b += "| ";

     for (int c = 0; c < board[r].length; c++) {

        // let's check to see what the variables hold!
        System.out.printf("rows: %d, columns %d, r: %d, c: %d%n", rows, columns, r, c); 
        b += board[rows][columns] + " "; // **** the offending line ****

さらに重要なことは、printf ステートメントの直後に例外が続く結果に注意してください。

rows: 1, columns 3, r: 0, c: 0
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 3

したがって、ここでは、値 3 を保持し、配列を爆発させるのは列変数です。このメソッドを呼び出す方法と、列パラメーターに 3 を渡す理由を確認するには、遡って調べる必要があります。

編集 2
最新の投稿を再確認すると、列パラメーターとして 3 を受け入れるように move メソッドをハードコーディングしています。

game.move('x', 1, 3);

何よりもまずそれを修正してください。そのパラメーターを 3 にすることはできません。

于 2012-08-19T03:39:54.080 に答える