Проверка сетки, исключение из запредельных исключений

Question

В настоящее время я пытаюсь проверить ячейки сетки для данных во всех ячейках вокруг нее. Вверх, влево, вниз, вправо и во всех диагоналях. Как я могу использовать бросок исключений, поэтому мне не нужно отдельно кодировать стороны и углы?

Это в настоящее время то, что у меня есть. isIsAlive() просто проверяет, активна ли ячейка. Соседи для ячейки включают в себя все активные клетки.

public void calcNeighbors() throws ArrayIndexOutOfBoundsException{ 

    int x =0; 

    int y =0; 
    int neighbors = 0; 



    while(x < 9){ 
     while(y < 9){ 

      if(generation[x+1][y+1].isIsAlive()){ 
       neighbors++; 

      } 
      if(generation[x+1][y].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x+1][y-1].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x][y-1].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x-1][y-1].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x-1][y].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x-1][y+1].isIsAlive()){ 
       neighbors++; 
      } 
       if(generation[x][y+1].isIsAlive()){ 
       neighbors++; 
      } 
      y++; 
     } 
     x++; 
     neighbors = 0; 
    } 
} 

Answer 1

Вы не должны бросать свое собственное исключение, если гипотетический сосед окажется вне пределов. Java все равно будет бросать ArrayIndexOutOfBoundsException.

Перед доступом к массиву вам необходимо проверить свои границы; не обращайтесь к массиву, если ваш x или y выходит за пределы допустимого диапазона.

Answer 2

Ваш список блоков if является уродливым (должен быть тупым) и опасным. Вместо этого используйте вложенные для циклов, но вычисляем верхнюю и нижнюю границы циклов for, учитывающих края.

for (int x = 0; x < MAX_X; x++) { 
    for (int y = 0; y < MAX_Y; y++) { 

    int minRow = Math.max(0, x - 1); 
    int maxRow = Math.min(MAX_X - 1, x + 1); 
    int minCol = Math.max(0, y - 1); 
    int maxCol = Math.min(MAX_Y - 1, y + 1); 

    for (int row = minRow; row <= maxRow; row++) { 
     for (int col = minCol; col <= maxCol; col++) { 
     if (row != x || col != y) { 
      if(generation[row][col].isIsAlive()){ 
      neighbors[x, y]++; 
      } 
     } 
     } 
    } 
    } 
} 

Answer 3

Не рекомендуется использовать исключения для этой цели. Но если вы настаиваете, вы можете сделать это следующим образом. Во-первых, определить метод

public boolean isAlive(int x,int y) { 
    try { 
     return this.generation[x][y].isIsAlive() ; 
    } catch(IndexOutOfBoundsException ex) { 
     return false ; // Or whatever you want to be the default 
    } 
} 

И затем использовать isAlive(x+1,y+1) вместо generation[x+1][y+1].isIsAlive() и так далее.

Кроме того, создается впечатление, что вы ошибочно объявляете локальную переменную int neighbors = 0;. Я говорю это, потому что вы продолжаете устанавливать его в 0 в конце, но вы его нигде не храните. Лично я хотел бы определить поле neighbors в какой бы это базовый класс generation и:

for(int x= 0 ; x < generation.length ; x++) { 
    for(int y= 0 ; y < generation[x].length ; y++) { 
     generation[x][y].neighbors= 0 ; 
     for(int dx= -1 ; dx <= 1 ; dx++) { 
      for(int dy= -1 ; dy <= 1 ; dy++) { 
       if(! (dx == 0 && dy == 0) && isAlive(x+dx,y+dx)) { 
        generation[x][y].neighbors++; 
       } 
     } 
    } 
} 

Меня беспокоит так много if с 3 являются: 1. Это очень легко сделать ошибку. 2. Для добавления любого другого кода во все if s потребуется много времени (и подвержено ошибкам). 3. Логику легче понять. Хотя вы также можете добавить комментарий, объясняющий, что вы собираетесь проверять соседей, и что соседи - это все 8 ячеек, где строка или столбец равна +1 или -1 текущей ячейке.

Кроме того, теперь, когда мы сократили число if с, мы могли бы также встраивать функцию выше и написать следующее:

for(int x= 0 ; x < generation.length ; x++) { 
    for(int y= 0 ; y < generation[x].length ; y++) { 
     generation[x][y].neighbors= 0 ; 
     for(int dx= -1 ; dx <= 1 ; dx++) { 
      for(int dy= -1 ; dy <= 1 ; dy++) { 
       try { 
        if(! (dx == 0 && dy == 0) && isAlive(x+dx,y+dx)) { 
         generation[x][y].neighbors++; 
        } 
       } catch(IndexOutOfBoundsException ex) { 
        // Do whatever you want in this case 
       } 
      } 
     } 
    } 
} 

Теперь без злоупотребляющих исключений (который на сегодняшний день является наиболее рекомендуется), я бы сказал, что добавить функцию

public boolean isValidNeighbor(int i,int j) { 
    return 0 <= i && i < generation.length && 0 <= j && j < generation[i].length ; 
} 

И ваш код становится:

for(int x= 0 ; x < generation.length ; x++) { 
    for(int y= 0 ; y < generation[x].length ; y++) { 
     generation[x][y].neighbors= 0 ; 
     for(int dx= -1 ; dx <= 1 ; dx++) { 
      for(int dy= -1 ; dy <= 1 ; dy++) { 
       if(! (dx == 0 && dy == 0) && isValidNeighbor(x+dx,y+dx) && isAlive(x+dx,y+dx)) { 
        generation[x][y].neighbors++; 
       } 
      } 
     } 
    } 
} 

Многое, намного лучше. И, даже если не главная причина, меньше кода и сложности, чем с исключениями !!!