바운드 예외 예외 발생

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

가설적인 이웃이 범위를 벗어나는 경우에는 예외를 던져서는 안됩니다. 자바는 어쨌든 ArrayIndexOutOfBoundsException을 던집니다.

배열에 액세스하기 전에 경계를 확인해야합니다. x 또는 y이 범위를 벗어난 경우 어레이에 액세스하지 마십시오.

Answer 2

블록 추한 경우 귀하의 목록 (무뚝뚝하기) 위험. 대신 중첩 된 for 루프를 사용하지만 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으로 설정하고 있기 때문에 이것을 말하고 있지만 어디에도 저장하지 않습니다. 개인적으로, 나는 generation의 기본 클래스 무엇이든의 필드 neighbors을 정의하고 다음과 같습니다 많은 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++) { 
       if(! (dx == 0 && dy == 0) && isAlive(x+dx,y+dx)) { 
        generation[x][y].neighbors++; 
       } 
     } 
    } 
} 

내 관심사는 3이다 : 1. 실수를하는 것은 매우 쉽습니다. 2. if 안에 다른 코드를 추가하는 데 시간이 오래 걸리고 오류가 발생하기 쉽습니다. 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 
       } 
      } 
     } 
    } 
} 

, 을 권장), I는 함수

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++; 
       } 
      } 
     } 
    } 
} 

훨씬 더 좋습니다. 그리고, 주된 이유가 아닐지라도, 예외가있는 것보다 적은 코드와 복잡성 !!!