nDiki : コードレビュー

インスペクション ウォークスルー コーディング規約 単体テスト メモ 社長 社内 Perl 勉強会 開発 ゴールデンウィーク Perl 開発ガイドライン 暗号化 ペアプログラミング ソフトウェア開発 エディタ プリント 初めてのPerl 第3版 Perl モジュール バージョン管理 ドキュメンテーションコメント ソフトウェアテスト ポイント ポインタ 難読化 勉強会 プリティプリンタ テープ チェックリスト perltidy バグトラッキングシステム

■ 第2回 社内 Perl 勉強会

リャマ本を使用した社内 Perl 勉強会の2回目を開催。 今回は7人。直前に社長につかまって約20分遅れのスタート (スミマセン)。

進行方法はほぼ前回と同じで、事前に練習問題を解いておき、勉強会では互いにコードレビューをしながら進行役の私がポイントを説明していく形式。

今日は「初めてのPerl 第3版」第3章が範囲で、リストと配列がテーマ。

@ 今回の変更点と反省点

• 解答のソースコードは全員 perltidy で色付き HTML に変換してプリントアウトして持ってくるようにしてもらった。前回より見易さアップ。
• 前回は1人1解答につき1枚プリントして持参してもらったが、今回は各自出席者分プリントアウトし持ち寄ってもらうことにした。これによって、それぞれ配られたプログラムをゆっくりレビューできる。書き込みもできるし、持ち帰ることもできるので効果あり。
  • 反省点はプリントの枚数が多くなって混乱したこと(「第2問の○○さんの解答プリントはどれだっけ?」)。次回は、冒頭で全員のものを集めて問題番号ごとに整理してステープラーでまとめるという形にしよう。
• ソースコードの先頭にコメントとして問題番号と作成者名を入れてもらうようにしたのだが、それだけだと誰のものか区別しにくいとの意見。
  • perltidy のオプションで HTML 化した時にタイトルをいじれればいいのだが、そのような機能はなさそうだ。
  • それこそ誰か、練習として H1 要素を書き換えるスクリプトとか書いてくれないかなぁ。

今のところ、全員ついてこれている様子。 スカラーコンテキストとリストコンテキストは、Perl 初心者にとってとっつきにくい部分の一つだと思うので、これが理解できれば結構いけそうな気もする。

今回はトータル80分ぐらい。7人だと1時間半ぐらい確保しておいた方がいいかもしれないな。

次回はゴールデンウィークを挟んで2週間後。

• 第1回 社内 Perl 勉強会 (2006-04-21)
• Perl プリティプリンタの定番 perltidy (2006-04-23)
• Perl 用の doxygen のようなツールはないのかな (2006-07-25)
• Emacs でお好きな言語のソースコードを色付き HTML テキストに変換 (2006-04-21)
• 第4回 社内 Perl 勉強会 (2006-05-22)

■ 久しぶりに C 言語プログラムのコードレビュー

開発が順調ではないプロジェクトについて、Perl モジュールと C 言語によるプログラムのソースコードをレビューした。

非公式なコードレビューだが、ぽこぽこ問題点が見つかる。 C 言語なんて久しぶりだけれど、ぱっと見てポインタの計算が間違えていることに気がついたりして、自分もまだ捨てたもんじゃないなと。

(自分も含めて)コーディングする人は「ま、いっか」「後で」で書いてしまいがち。 で、往々にして「後で」はやってこない。

やはり他人に見てもらうことが重要。

• 第2回 社内 Perl 勉強会 (2006-04-28)
• TortoiseCVS 1.8.0-RC4 (2004-08-26)
• 今日のさえずり - 上げ潮特大号 (2008-09-18)
• 第1回 社内 Perl 勉強会 (2006-04-21)
• Algorithm::RabinKarp でソースコード中の重複を発見 (2006-08-04)