为什么这不起作用？ 比较两个二维数组中的元素

Question

For some reason this will always return 0? Why? I compare two 2d arrays of Space objects, which have a specific colour on them. I have made sure the two 2d arrays I am comparing are different but the method won't work?

public int compareArray(Space[][] test1, Space[][] test2)
{
    for(int row = 0;row< test1.length;row++)
    {
        for(int column = 0; column<test1.length;column++)
        {
            if(!((test1[row][column]).getColour()).equals(test2[row]    [column].getColour()))
            {
                System.out.println("Found a non match");
                return 1;
            }

        }
    }
    return 0;
}

I call this method here, the evenNumber method, changes test, and makes it different to test2, it's for the project I'm doing

public void testMethods()
   {
       Space[][] test = new Space[4][4];
   Space[][] test2= new Space[4][4];

   for(int i = 0; i < test.length; i++){
         for(int j = 0; j < test.length; j++){
              test[i][j] =  new Space(spaceColour.Null); 
              test2[i][j] =  new Space(spaceColour.Null);
         }
    }


   test[0][1].setColour(spaceColour.White); 
   test[1][1].setColour(spaceColour.White);  
   test[2][1].setColour(spaceColour.Black); 
   test[2][2].setColour(spaceColour.Black); 

   test2[0][1].setColour(spaceColour.White); 
   test2[1][1].setColour(spaceColour.White);  
   test2[2][1].setColour(spaceColour.Black); 
   test2[2][2].setColour(spaceColour.Black); 

  System.out.println(compareArray(test, test2));


   evenNumber(test);
   drawBoard(test);
   System.out.println("");
   drawBoard(test2);

 }

Answer 1

test[0][1].setColour(spaceColour.White); 
test[1][1].setColour(spaceColour.White);  
test[2][1].setColour(spaceColour.Black); 
test[2][2].setColour(spaceColour.Black); 

test2[0][1].setColour(spaceColour.White); 
test2[1][1].setColour(spaceColour.White);  
test2[2][1].setColour(spaceColour.Black); 
test2[2][2].setColour(spaceColour.Black); 

The colors of test1 and test2 are the same. So your if statements evaluates to false all the time, as you are checking if they are not the same.

Try with these values:

test[0][1].setColour(spaceColour.White); 
test[1][1].setColour(spaceColour.White);  
test[2][1].setColour(spaceColour.Black); 
test[2][2].setColour(spaceColour.Black); 

test2[0][1].setColour(spaceColour.Black); 
test2[1][1].setColour(spaceColour.Black);  
test2[2][1].setColour(spaceColour.White); 
test2[2][2].setColour(spaceColour.White); 

Answer 2

I think you need to change the second for loop from

for(int column = 0; column<test1.length;column++)

to

for(int column = 0; column<test1[0].length;column++)

Answer 3

You need to do a little more

if (test1.length != test2.length) {
  return 1;
}
for(int row = 0;row< test1.length;row++)
{
  if (test1[row].length != test2[row].length) {
    return 1;
  }
  // back to your code
  for(int column = 0; column<test1[row].length;column++)
  {
    if(!((test1[row][column]).getColour()).equals(test2[row][column].getColour()))
    {
      System.out.println("Found a non match");
      return 1;
    }
  }