Средство проверки блоков судоку Java

Существует 9 возможных значений, поэтому ваш массив должен иметь размер 9, а не 3.

Считать нужно только в том случае, если ячейка уже заполнена. Если это так, верните false. Если вы дойдете до конца, не вернув false, верните true.

++ увеличивает возвращаемое значение, чтобы вы могли его сравнить. Если это 2, то ячейка уже была увеличена.

Код:

public class BlockTest {
  public static boolean checkBlock (int col0to2, int row0to2)
  {
    int [] tempArray = new int[9];
    for (int x=0; x<3; x++)
    {
      for (int y=0; y<3;y++)
      {
        int tempVal = board [col0to2+x][row0to2+y];
        if (tempVal < 1 || tempVal > 9) return false;
        if (++tempArray[tempVal - 1] == 2) return false;
      } 
    }
    return true;
  }

  public static int[][] board = new int[9][9];
  public static void main(String[] args) {
    for(int i = 0; i < 3; i++) {
      for(int j = 0; j < 3; j++) {
        board[i][j] = i*3 + j + 1;
      }
    }
    System.out.println(checkBlock(0,0));
    board[0][0] = 7;
    System.out.println(checkBlock(0,0));
  }
}

Выход:

true
false

if ((board [col0to2+x][row0to2+y]) > 0 && (board [col0to2+x][row0to2+y]) < 10) {
      int tempVal = board [col0to2+x][row0to2+y];
      tempArray [tempVal - 1] = tempVal; // i think this line is giving me the run error

}

Ваша проверка границ для tempval заключается в том, что она должна быть от 1 до 9 включительно. Предположим, что это 9. Затем вы назначаете tempArray[8] = 9. Здесь вы сталкиваетесь с проблемой, поскольку tempArray был инициализирован как массив, содержащий 3 элемента.

Я думаю, вы хотите что-то вроде:

tempArray = new int[9]

и задание будет

tempArray[x*3 + y] = tempVal;

Я не проверял это, но в основном он заполняет массив каждым найденным числом.

Поскольку вам нужно иметь дело с числами 1-9, вам нужно место для 9 целых чисел в tempArray, но вы объявляете, что оно будет длиной всего 3. Кроме того, вы на самом деле не проверяете, используются ли все номера (и ни один из них не дублируется).

public boolean checkBlock (int col0to2, int row0to2) {
   int [] tempArray = new int [9];
   for (int x=0; x<3; x++) {
      for (int y=0; y<3;y++) {
         int tempVal = board [col0to2+x][row0to2+y];
         if (tempVal > 0 && tempVal < 10) {
            if (tempArray[tempVal - 1] != 0) {
               return false;
            }
            tempArray [tempVal - 1] = tempVal;
         }
      } 
   }
   return true;
}

Я понимаю, откуда вы это взяли, но, как говорили другие люди, ваш tempArray должен содержать 9 значений. Раньше я писал средство проверки судоку (однозначное назначение, проверка решения была правильной, преодолел отметку 90% для всего приложения судоку), и я предпочитал иметь массив с 9 значениями, каждое из которых представляет 1-9 цифр, но вместо того, чтобы установить значение tempVal, я увеличил его. Облегчает проверку того, что вы получили только одно из каждого числа.

int [] boxCounts = {0, 0, 0, 0, 0, 0, 0, 0, 0};

// X loop
    // y loop
        // get value
        boxCounts[value - 1]++;
    // end y loop
// end x loop

// check we found one of each
for(int j = 0; j < 9; j++) {
    if(boxCounts[j] != 1 )
        return false;
    }
}