鉱山周りの鉱山は半分の時間で動作します

Question

私はJavaで掃海艇クローンを書いています。周囲の爆弾数が計算される部分に問題がありました。何らかの理由で、一部の細胞が追加の爆弾を検出するか、爆弾が全く検出されません（奇妙なことに、一部の細胞は正常に機能します）。助けてもらえますか？ありがとう！

注：key[][] 9.

0として指定された爆弾の位置を記録int[][]アレイは、それぞれを取り囲む8個の細胞である爆弾の量を表す

int count空きスペースを表しますセル（私も8のtry-catchが境界セルを考慮するために、ループを持っている理由です）

PS：悪いフォーマット

EDITのため申し訳ありません：私は考え出し私の問題がどこにあったか（私は私に二度電話した）。どのように私はこのコードをより簡単で効率的にすることができるかについての提案はありますか？

private void numberSet() { 

    int count = 0; 
    for (int i = 0; i < key.length; i++) { 
     for (int a = 0; a < key[0].length; a++) { 

      if (key[i][a] == 0) { 

       try { 
        if (key[i + 1][a] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i - 1][a] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i][a + 1] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i][a - 1] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i + 1][a + 1] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i - 1][a + 1] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i + 1][a - 1] == 9) { 
         count++; 
        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       try { 
        if (key[i - 1][i - 1] == 9) { 
         count++; 

        } 

       } catch (java.lang.ArrayIndexOutOfBoundsException e) { 
        continue; 
       } 
       key[i][a] = count; 
      } 
      count = 0; 
     } 
    } 

} 

Answer 1

私は真剣にあなたがこれをやっている方法を放棄すると考えています。私が最初にコーディングを始めたとき、私も掃海艇のゲームを作って、あなたがしていたことをやった。いくつかの改訂の後、私はそれをすべていくつかのループに入れることができました。すべての行が何をしているかを伝えるために私のコードで十分にコメントを書こうとしました。このコードはテストされていないので、まったく正直なところですぐには機能しません。しかし、私はそれを数回見て、それは私にはうまくいくようです。私はあなたがこのコードをとり、それを動作させることをお勧めします。試してみる。

//For every Cell on the board 
for (int i = 0; i < key.length; i++){ 
    for (int a = 0; a < key[0].length; a++){ 

     //If it Is not a mine 
     if (key[i][a] == 0){ 
      int count = 0; 

      //Position of cells around key[i][a] relative to key[i][a] 
      for (int x = -1; x < 2; x++){ 
       for (int y = -1; y < 2; y++){ 

        //Storing x and y test points 
        int testX = i + x;//testX = i-1, i, and i+1 
        int testY = i + y;//testY = i-1, y, and y+1 

        //If the testX and testY values are within the range of the array 
        if ((testX >= 0 && testX < key[0].length) && testY >= 0 && testY < key.length){ 

         //If there is a mine 
         if (key[testX][testY] == 9){ 
          count++; 
         } 
        } 
       } 
      } 

      key[i][a] = count; 

      //count = 0; This is redundant. Line 7 count gets set to 0. 
     } 
    } 
} 

Answer 2

あなたがcontinue文を悪用されているように、それはforループバックの実行を移動し、したがって、countをインクリメントするために様々なチェックを完了しません見えます。

catchステートメントでは、何もしないでください。

Answer 3

if(key[i-1][i-1] == 9){ 

これは、iとaを持つ他のすべてのものとは異なります。これはiを2回使用します。多分それはあなたの問題ですか？