akaimo / akaimo-sezemi-2015-readable-code Goto Github PK

実際のコード

処理をしたらすぐに検査をして、異常があればすぐに失敗しているのがいい。

    fname = fopen(argv[1], "r");

    if (fname == NULL) {
        printf("%sが開けません\n", argv[1]);
        exit(1);
    }

どうしてリーダブルだと思っているか？

fname の変数名のスコープの中で最短距離でエラーの検査をしています。読んでいて思考がジャンプしなくても何をしているか理解できるし、他の処理が挟まらないうちに exit で失敗してくれるのでファイルの open 失敗による他の影響を考える負担から解放され、リーダブルだと思いました。

この書き方の一言説明

早めの検査、早めの失敗

見つけた経緯

NULL チェックの == != の違いによるリーダブルなコードを探そうと見ていて、どうしてここで NULL をチェックしているんだっけ？と考えたときにこのコードはリーダブルだと気付きました。