プログラミング

コメントのいらないプログラムの書き方

こんにちは、ニュージーランド在住ブロガーのはっしー(@hassy_nz)です。

「プログラムには適切にコメントを書きましょう」
プログラミングの学習を始めると、ほぼ誰もが教わることです。
確かに、適切に書かれたコメントはコードを読みやすくし、理解を助けてくれます。

しかし、そもそもコメントって必要なのでしょうか?
もし、コメントがないと意味がわからないコードがあったとすれば、その書き方に問題があるのではないでしょうか?

コードの読みにくさをコメントで解決するのは、安易な方法に逃げているだけかもしれません。

実は、僕の働いている会社では 「原則としてコメントを書かない」のがルールになっており、それで困ったことはほとんどありません。(100人以上のデベロッパーがいる現場です)
もう入社して2年になりますが、ぶっちゃけコメントって書く必要がないなと思うようになりました。

この記事では、 誰でもコメントのいらないプログラムが書けるようになる方法を丁寧に紹介していきます!

実装する機能の例

例として、映画館のチケット料金を計算する関数を作ってみます。

要求仕様は、

  • 13歳以上はおとな料金で1800円
  • 13歳未満はこども料金で1200円
  • 毎月1日はサービスデー料金で一律1000円

とします。

レディースデーとかクーポン料金とか考えるともっと条件を複雑にできますけど、まずは単純な例からはじめましょう。

関数の名前を決める

まずは関数にどんな名前がふさわしいか考えましょう。
「動詞 + 目的語」の形に統一しておくと誰からみても何をする関数なのかわかる = コメントのいらないプログラムになります。
たまに「kansu…」とか「func…」とかいう命名を見かけることがありますが、「これは関数だよ!」と言われたところで何もうれしくないのでやめましょう。文字数の無駄です。
代表的な例を表にまとめたので参考にしてください。

機能 名前
値を代入する set…
値を取得する get…
計算する calculate…
変換する convert…to…
真偽値を返す is…

今回は「チケット価格を計算する関数」なので、「calculateTicketPrice」としてみましょう。
なお、コードは JavaScript をイメージして書いていきますね。

パラメータを決める

次に関数に渡すパラメータを決めます。
関数の名前で表現されている処理を実現するには、どれだけのパラメータがあればよいか? と考えてみましょう。
今回の例でいえば「お客さんの年齢」「日付」があれば、すべてのチケット価格が計算できます。
ということで、age と date の2つのパラメータを渡すことにします。

パラメータの名前も、なにを表しているかわかるようにしてくださいね。
くれぐれも「hensu」とか適当な名前をつけたり、同じ変数にぜんぜん違う値を繰り返し代入したりすることのないようにしましょう。

テストを書く

次にユニットテストを書きましょう。
テストは常に更新される仕様書です。
業務ロジックをテストに説明させておけば、関数の仕様をコメントにいちいち書く必要などありません。
関数のテストケースを見れば、それが仕様書になっているからです。
プログラムが仕様どおりに動いているかも常にチェックできるし利点だらけ。
もしユニットテストのない現場で働いているなら、今すぐ上司に直訴して導入してください(まじで)。

ここでは JavaScript のテストフレームワークである mocha と、アサーションフレームワークの chai を使うと仮定して書いてみます。
mocha では日本語でテストグループ名と個々のテスト名を書くことができるので、記述形式がかなり自由です。
が、対象となる条件をはっきりさせるためにも「(条件)なら(動作)」という形式でテスト名を統一するのが良いでしょう。
例としてはこんな感じです。

C# や Java などの言語のテストフレームワークでは、テスト関数の名称で条件を表さなければならない場合があります。
そんなときは、関数を「When (条件) Then (動作)」の形式で命名するようにしておくと、コメントなしでも条件が理解できるのでオススメです。
Java のテストフレームワークである JUnit 風に書くとこんな感じになります。

実装する

テストができたらいよいよ関数を実装していきます!
機能を少しずつ追加して、すべてのテストケースが正常終了するように作っていきましょう。
とりあえず何も考えずに実装すると、こんな感じになります。

これでも仕様どおりに動きますが、可読性を考えるとまだ改善の余地があります。
たとえば、
「なんで getDate が1だと特別なの?」
「age >=13 と age < 13 ってどうして場合わけされてるの?」
といった点がコードを読んだだけではわかりませんね。

コード内の「1」や「1000」のように、変数に代入されることなく突然出てくる数字はマジックナンバーと呼ばれています。
マジックナンバーは可読性や保守性を下げるので、原則として避けた方がよいです。
これらをもっと意味のある名前をもつ変数に代入すればさらに読みやすいコードになります。
改善した例がこちらです。

これを読めば、

  • チケットの価格を計算する関数である
  • 年齢と日付をパラメータにとる
  • 毎月1日はサービス料金で1000円
  • 13歳以上ならおとな料金で1800円、それ未満ならこども料金で1200円

ってことが誰でもわかりますよね。

どうです? コメントいらない気がしてきませんか?

(※ 関数の中で宣言されている変数は定数として外部に宣言することもできそうですが、とりあえず今回は関数内変数としました)

コードレビューに出す

コメントのいらないプログラムを書くため、最後に必要なステップがコードレビューです。
自分自身できれいなコードが書けたと思っていても、他人が見て理解できなければ意味がありません。
コメントなしで読めるかどうかを第三者に判断してもらいましょう。

レビューする側も「どんなコードが読みやすいのか」「自分だったらどのように書くか」といった視点をもってプログラムを読んでみると、自分自身のコーディング力の向上にもつながります。

コードレビュー専用のツールはいろいろあるので、現場にあったものを探してみてください。

参考▶コードレビューのツールについての調査 – Qiita

「コメントのいらないプログラムの書き方」のポイント3つ!

今回紹介した方法のポイントをまとめると、次の3つです。

  1. 関数や変数に適切な名前がつけられていれば、コメントはいらない
  2. 仕様がユニットテストですべて記述されていれば、コメントはいらない
  3. コメント無しで第三者にも理解できれば、コメントはいらない

最初は違和感があるかもしれませんが、この方法でコードを書いていくとコードそのものの読みやすさがどんどん改善されていくので、ぜひとも参考にしてみてください!

もっと勉強したい人にはこちらもオススメ▶ 現役プログラマがおすすめするプログラミングスクール3選!

ABOUT ME
はっしー
ニュージーランドで働くプログラマ。 日本のIT企業で月100時間超えの残業を経験して過労死しかけたことをきっかけに国外脱出、毎日定時帰りの生活と年収アップを実現させる。脱社畜、英語、海外移住などをテーマに情報発信中。Twitterフォロワーは1万人を超える。ニュージーランド永住権ホルダー。

POSTED COMMENT

  1. RC3 より:

    // 13歳以上 かつ 1日じゃないときは大人料金
    whenEqualsToOrOlderThanThirteenAndNotFirstDateOfMonthThenReturnAdultPrice
    // なぜかビルドエラー
    var priceAge14 = alculateTicketPrice(14, new Date(2018, 1, 2));

  2. imo_jo_chu より:

    >id:RC3
    ビルドエラー直した

  3. selvira1638 より:

    Thanks alot. Very good information.
    Agen Poker Online

  4. SciArtMusic より:

    小説家が小説を書くようにプログラムを書きたいですね…

COMMENT

メールアドレスが公開されることはありません。 * が付いている欄は必須項目です