深いネストは忌むべき - 超ウィザード級ハッカーのたのしみ

Zen of PythonはPython以外の言語であっても私がコードを書くときの指針である。

その一つに、

Flat is better than nested.

という文句がある。深いネストはコードの可読性を著しく落とす。スクリーンの右のほうに文字が寄っているコードは大変読みづらい。特にJavaはネストが深くなりがちなので要注意だ。

public class Foo {
    static private class Bar {
        int var;
        void method_spam() {
            synchronized (this) {
block:          {
                    if (var > 0) {
                        try {
                            for (int i = 0; i < 10; i++) {
                                if (method_egg()) {
                                    break block;
                                }
                            }
                        } catch (Exception e) {
                        }
                    }
                }
            }
        }
    }
}

このような}が数多く並ぶようなコードは読みたくない。人間は深いネストを見て、何処のブロックにいるか把握できるほど賢くない。汚いコードをバグなく書ける人はいない。

上のコードを綺麗にするとしたら、以下のようにするだろう。

public class Foo {

}

class Bar {
    int var;
    synchronized void spam() {
        if (var <= 0) {
            return;
        }
        for (int i = 0; i < 10; i++) {
            boolean b = false;
            try {
                b = egg();
            } catch (Exception e) {
                return;
            }
        }
    }
}