この記事は、翔泳社で実施する独習ゼミの研修を紙上で再現したものです。

使用したプログラミング言語はJavaです。プログラミング経験の有無から「初級コース」と「中級コース」に分かれて、それぞれ4問ずつプログラミングの課題に取り組んでいただきました。講師を務めた私の役目は、赤ペンでソースコードを添削することでした。提出されたプログラムは、どれもきちんと動作しましたが、誰にも共通した問題がありました。それは「プロらしくない」ということです。

実際に内定者さん（学生さん）から提出していただいたソースコードと添削結果を掲載しますので、皆さんも一緒にプロらしさとは何かを考えてみてください。また、指導する立場の人は、連載で指摘していく項目を意識しながら、後輩指導に取り組んでみるのもよいでしょう。今回は、中級コース第1回の課題を紹介します。なお、提出者の名前は匿名であり、ソースコードも一部を抜粋したものを掲載しています。

1 :// 課題1
2 :import java.io.*;
3 :
4 :class Numberquiz {
5 :  public static void main(String args[]) {
6 :    double ans1 = Math.random() * 100  +1 ;
7 :    int ans2 = (int) Math.ceil(ans1) ;
8 :
9 :    try {
10:      InputStreamReader isr = new InputStreamReader(System.in);
11:      BufferedReader br = new BufferedReader(isr);
12:
13:      for( int i = 1 ; ; i++ ) {
14:        System.out.print("いくつだと思いますか？") ;
15:        int yourans = Integer.parseInt(br.readLine()) ;
16:        if( ans2 == yourans ) {
17:          System.out.println("正解です！あなたは" + i + "回で当てました。") ;
18:          break ;
19:        }else
20:          System.out.println( (ans2 < yourans) ? "もっと小さい" : "もっと大きい" ) ;
21:      }
22:    }
23:    catch(Exception e) {
24:      System.out.println("Exception: " + e);
25:    }
26:  }
27:}

もっと多くのコメントを書きましょう
　プロになると、誰か他の人にソースコードを引き継ぐ場合があります。できるだけ多くのコメントを付けて、内容を読みやすくしてください。とはいえ、コメントが多すぎてもいけません。どの程度が適切なのかは、自分が引き継ぐ立場だったらどうだろう？ と考えて判断してください。
プログラムの先頭には、プログラムの内容、作者、レビジョン（改訂履歴）、作成年月日などを示すとよいでしょう。これらは必須のコメントです。クラスには、何のためのクラスなのかを示すとよいでしょう。また、メンバ（フィールドとメソッド）ごとに機能の説明を書きましょう。メソッドの処理の流れの中には、フローチャートで1つの図記号になる単位でコメントを付けるとよいでしょう。

繰返しはwhile文を使うとよいでしょう
　これは個人的な好みの問題であるとも言えますが、回数をカウントした繰り返しならfor文、回数が未定で条件に応じた繰返しならwhile文を使うと、誰にでもわかりやすいプログラムとなります。以下に例を示します。

// for 文を使った繰り返し
for (int i = 1; ; i++) {
    ・・・
}

// while文を使った繰り返し
do {
    ・・・
 } while (ans2 != yourans);

誰にでも理解できるシンプルな表現を心がけましょう
13行目のfor (int i = 1;;i++) { ・・・ } という書式や、20行目の三項演算子（？：）を使った部分は、トリッキーでカッコイイのですが、誰にでも理解しやすいとは言えません。あなたがベテランプログラマになった状況を思い浮かべてください。もしも「あなたのコードは、新人プログラマには難し過ぎる部分がある。このシステムの改造は、新人プログラマに任せようと思ったが、引き続きあなたにやってもらうことにしよう」などという事態になったら困りますね。いつまでたっても、同じシステムのメンテナンスから離れられないからです。
誰でも理解できるシンプルな表現を心がけましょう。たとえば、三項演算子ではなくif〜else文を使えばよいのです。

// 三項演算子を使った場合
System.out.println( (ans2 < yourans) ? "もっと小さい" : "もっと大きい" ) ;

// if〜else文を使った場合
if (ans2 < yourans) {
    System.out.println("もっと小さい" ) ;
} else {
    System.out.println("もっと大きい" ) ;
}

もちろん、コードの実行速度が要求される場面であれば、どんなにトリッキーなコーディングをしても構いません。ただし、この課題の内容では、コードの実行速度は要求されないので、見やすいコードとなることを優先してください。

この調子で引き続き学習を続けていってください。　がんばってくださいね！