java.lang.nullpointerexception error in array

[zahra abaszade]

with this code I get a java.lang.NullPointerException:

public class TicTacToe
{
    private int count=0;
    private String letter;
    private int[][]myArray;
 public TicTacToe()
    {
 
       int[][] myArray = new int[3][3];
       for(int i=0;i>3;i++){
           for(int j=0;j>3;j++){
           myArray[i][j]=0;
        }
        }
 
    }
  public void nextMove(int s1,int s2){
        myArray[s1][s2]=1;//error is here
        count++;
 
        if(count==1||count==3||count==5||count==7||count==9){
            letter="O";
 
        }
        if(count==2||count==4||count==6||count==8){
            letter="X";
 
        }
 
       System.out.println("button"+s1+","+s2+":"+letter);
 
 
    }
}

[Frinavale]

In your constructor code for-loop, on line 10 you have:

for(int i=0;i>3;i++)

This won't work...i will never be greater than 3.
You have similar code on line 11...

Change the check for i greater than 3 to i less than 3.

Edit: the reason you're getting the error is because your array isn't be initialized properly due to the problem in your loops.

-Frinny

[neeraj0708]

You need to declare array properly....
in your nextMove method it is not initialized properly... check it.

    public class TicTacToe
    {
        private int count=0;
        private String letter;
        private int[][]myArray;
private  int[][] myArray = new int[3][3];
//declare array here so that it will be visible in your //whole class
 
     public TicTacToe()
        {
 
           int[][] myArray = new int[3][3];//remove it
           for(int i=0;i>3;i++){
               for(int j=0;j>3;j++){
               myArray[i][j]=0;
            }
            }
 
        }
      public void nextMove(int s1,int s2){
            myArray[s1][s2]=1;//error is here
            count++;
 
            if(count==1||count==3||count==5||count==7||count==9){
                letter="O";
 
            }
            if(count==2||count==4||count==6||count==8){
                letter="X";
 
            }
 
           System.out.println("button"+s1+","+s2+":"+letter);
 
 
        }
    }
 
 

[zahra abaszade]

thank you very much for your best answer!!

[zahra abaszade]

thank you frinavale!!