エラーが前提のコーディングに関して

Question

マインスイーパをCUIで作っていまして開けた場所が0だった場合その周りを開ける。その中に0があったらまた、周りを開けていくメソッドを作りました。
li :前回開けたx,y座標が半角ｽﾍﾟｰｽ区切りで入ります。
map_open : 開いたところは1が入ります.
ans_map : 爆弾や周囲の爆弾の数が入ります。

public void open_arround(ArrayList<String>li) {
        ArrayList<String>new_li=new ArrayList<>();
        for(String s:li){
            String st[] =s.split(" ");
            int x = Integer.parseInt(st[0]);
            int y = Integer.parseInt(st[1]);
            for(int i = -1;i<2;i++){
                for(int j = -1;j<2;j++){
                    try{//<<<<<<========疑問
                        if(map_open[x+i][y+j]!=1&&ans_map[x+i][y+j].equals("0")){//端っこのほうは範囲外を参照しようとしてｴﾗｰが出るよ
                            int a = x+i;
                            int b = y+j;
                            new_li.add(a+" "+b);
                            map_open[x+i][y+j]=1;
                        }else if(map_open[x+i][y+j]!=1){
                            int a = x+i;
                            int b = y+j;
                            map_open[x+i][y+j]=1;
                        } 
                    }
                    catch(ArrayIndexOutOfBoundsException e){}
                }
            }
        }
    if(new_li.size()>0){open_arround(new_li);}
    else{return;}
    }
コード

こういった書き方はやめたほうがいいのでしょうか？

Answer 1

むかしのDelphiで、TCPソケットコンポーネントでの状態遷移の通知に例外を使っていた覚えがあります。
まあ、それでもそれ専用の例外を定義してそれを発行させるようなことですんで、今回の例とはちと違います。

まあ、こういう正常系の通知、というか、処理が終わったからそこから抜けるために例外を使うというのは１０歩ぐらい譲ってアリ（メリットがそれなりにあるならやってもいいかなぐらいでｗ）だとは思いますが、それでもそれ用の例外を定義して使うべきものですね。

今回の質問のようにあえてエラーを発生させるというのは、他の方の回答にもあるように、これはまずいですね。

＃まー、とにかく動きゃええ、と言うならきちんと動くんだろうけど

Answer 2

例外が発生する前提でのコーディングはよっぽどの理由がない限り控えた方が良いです。
(個人的にはほぼ禁止で、対処療法として使用する程度だと思っています。)
理由は、コードの可読性の低下を招くためです。

今回のように例外前提のコーディングをしたくなる場合は下記が該当すると思います。
・判定処理を行うのが面倒
・例外をcatchすれば、判定処理と同等の事ができる

しかし、階層の深いところで例外を投げてしまった場合、それがどのような処理(判定)が原因で投げられたものなのか
致命的ななのか、それとも単なる判定処理を例外処理で同等の事をしているのか分からなくなります。
(今回のような小さな処理であれば終えますが、大きな処理・チーム等で作成する場合は顕著になります。)

質問者様はコーディングについての良し悪しの質問をされているので
きちんと分かりやすい判定処理のメソッド名を付け、それを実行する事で例外を投げる必要もなく読みやすいコーディングを行われる方が良いかと考えます。

Answer 3

場合によりけりと思いますが、個人的には避けたいです。

理由は・・・

論理の上でArrayIndexOutOfBoundsExceptionが発生するケースは無視してもよいことが確実なら問題ないですけど、何かの拍子に論理を変更したような場合「本来ArrayIndexOutOfBoundsExceptionが起きる可能性がないと思っていた箇所で誤ってそれを起こすようなヘマ」をしてしまうと、この例外が無条件に握りつぶされているために、プログラムを動かしてもすぐにそれと気づけないからです。

---

余談ですが・・・
Javaアプリのデバッグ実行時に「任意の例外が発生したときに中断」という条件で動かしてしまうとJVMの起動時に何度もNPEやArrayIndexOutOfRangeで止まってしまいます。Javaの標準ライブラリーの中に「ある種の例外を事前にチェックするのではなくあえて握りつぶしている」ような論理があるためだと思います。最適化、あるいは実装の都合であえてそういう方法を選んだのでしょうが、デバッグしずらくてめんどくさいなぁなんて感じることがあります。