高慢と偏見（5） そして戦いがはじまる

スキル | ワークスタイル | 人間関係

» 2010/12/13

　コードレビューという名の、三浦十字軍の暴走は続く。

　「......それから、210行めぐらいで部品マスタからデータ引っ張ってきてるけどね、ここで、PartsMasterGetterって使ってるね。これ何？」

　「部品マスタへのアクセスをまとめた共通ロジック？」富永さんは自信なさそうに答えた。

　「何それ？ 私に聞いてるのかね？」

　「共通ロジックです」富永さんは言い直した。

　「なんでわざわざ共通ロジックにしてあるの？ コードが追いにくいでしょう」

　――いや、コードをわざわざ追わなくてもいいように、切り出してあるんだって。

　富永さんは助けを求めるように平良さんを見た。つられて何人かが同じ方向を見たが、平良さんは自分の爪の状態が最重要の関心事になったらしく、右手を熱心に観察していた。

　「それは、その......」富永さんは疲れ切ったように視線をさまよわせた。「......ぼくが来たときにはすでにあったので、使ってるだけで......」

　「あ、そう。それを何の疑問もなく使ってるわけだ」

　「......」

　「1つの画面の処理は、そこで完結するようにコーディングしようとか思わなかったのかね」

　――いやいやいやいや、そりゃムダでしょう。そんなことしたら、スキーマが変わったときに、関連する全ソースを修正しなくちゃならなくなるって分かってますかい？

　「わざわざ別のクラスに分割しなくても、業務ロジックは書けるでしょう」

　――そりゃ書けるけど、誰かが悩んだことを工数かけて何度もやるのは効率が悪いでしょう。お金払うのはあなたの会社なんですけどね。

　「だったらわざわざクラスなんか作らないように。クラスなんか作らなくてもロジック関数にすればいいんだから」

　――だからロジック関数ってなんなのよ？

　「まったく、オブジェクト指向なんてものを無理やり使おうとするから、こういうことになるんだな」

　――違う。そりゃ、問題が全然違うでしょう。

　「私は自分でクラスなんか作ったこともないよ。それでも20年やってきたし、エンドユーザーからの信頼も厚いんだ」

　――ええ、そうでしょうね。エンドユーザーはソースまで見ないからね。

　「まあ、すでに作ってしまったものは仕方がないかな。もうこれで進んでる機能も多いだろうし。だがねひとこと言っておくとね」三浦マネージャのメガネがきらりと光った。「この業界で生き残りたかったら、私の経験に学んだ方がいい。このプロジェクトに参加できたことをきっと感謝するようになるから」

　もはや失笑するしかない。

　確かに三浦マネージャから学ぶものは多いかもしれない。アンチパターンの生き字引みたいなものなのだから。

　それにしても、「私の経験に学べ」という発言をどう捉えるべきなのだろう。

　「私の方針に従え」と取るべきなのか。

　「強制はしないが、私の方針に従った方が得だ」と取るべきなのか。

　Javaでクラスを作らないというのは、卵を割らずにオムレツを作れと言っているようなものだから、たぶん三浦マネージャの言う「クラス」というのは、さまざまな共通ロジックなどを指しているのだろう。三浦マネージャ方針に従うということは、今後、新たに共通ロジックを作成することは不可、という意味になる。

　その答えは、すぐに明らかになった。

　「以後、余計なクラスは作らないようにすること」三浦マネージャは明言した。「すでにできているクラスを使うのは仕方がないが、それも可能な限り避けること。Javaにはたくさんの標準クラスがすでにある。それを使えば、ほとんどの機能は作れるはずだからね」

　――おいおい。マジかよ。

　すでにこのプロジェクトではさまざまな共通ロジックが作成されている。常にメンテナンスされて機能の最適化を図っているし、日々新しく作成され続けている。私もいくつか作ってきた。

　例えば、部品コードは16桁か20桁の英数字だけど、画面には既定のフォーマットでハイフン編集されて表示される。桁数とコードの先頭のn文字によってフォーマットが微妙に変わるので、このロジックは、PartsCodeViewHelperというクラスに、getViewCode()というメソッドとしてまとめて記述されている。部品コードの表示は画面系／帳票系機能のほとんどで必要なので、このViewHelperクラスの恩恵は大きい。同じようなHelperクラスやGetterクラスなどの「便利クラス」はかなりの数になっていて、画面によっては、それらを並べるだけでできてしまうものもある。今後、新しいプロジェクトで使用できると思われるものも多く、平良さんはそれらを1つのjarファイルにまとめようとしていた。

　つまり、このプロジェクトから生み出される財産だ。

　三浦マネージャは、そういう恩恵を捨てろと言っているわけだ。「長い経験」に基づいて。こっちはいい迷惑だ。

◇　◇　◇　◇　◇　◇　◇　◇　◇　◇

　予定を大幅にオーバーして、　コードレビューは3時間後に終了した。

　全員が疲労困ぱいしていた。特に富永さんは、今にもぶっ倒れるんじゃないかと心配になるほど土気色の顔だ。なにしろ、平良さんが戦線離脱してしまったので、三浦マネージャからの集中砲火は富永さんが1人で浴びることになってしまったのだから。

　予定していたバッチ処理のレビューは、結局、延期になった。三浦マネージャはメンテナンス画面の、ほぼ全ソースについて、質問し、非難し、「長い経験」による改善案を提案（というより強制）し、これまで私たちが積み上げてきた小さな世界を否定することに終始することで、予定時間をすべて使ってしまったのだ。会議室に次の予約が入っていなければ、きっと何時間でも続けたに違いない。

　ゆっくりお茶でも飲みに行きたいところだったが、ムダにした3時間を取り戻さなければならない。私は自販機でミルクティーを買って開発室に戻ると、Eclipseを起動してため息をついた。

　一足先に戻っていたアツコさんが小声で話しかけてきた。

　「平良さんからメール来てるよ」

　「え？」

　私はメールを確認した。送信者は平良さん、あて先は私を含むメンバーの何人か。サブジェクトはない。

　変だと思ったのは、あて先が個別に指定されていることだった。メンバー間で連絡事項があるときは、専用のメーリングリストを使うのが普通なのに。

　本文を読むと理由が分かった。

　読んだら消してください。

　今日、来週の月曜日は通常の画面機能の作成を中止。作りかけのGetterクラス、Helperクラス、Checkerクラスの作成を完成させてください。

　三浦マネージャは今日は会議で戻りません。月曜日は出張です。

　短いメールだったが、平良さんの意図は明らかだった。平良さんは、三浦マネージャが使うことを（渋々）認めた「すでにできているクラス」を可能な限り作っておこうとしているわけだ。メールが送られたのは、少し時間のかかる機能を担当しているメンバーばかり。1日ぐらいの遅れはごまかせるということだ。全員の進ちょく状態を把握している平良さんならではの作戦だった。

　こうして私たちと三浦マネージャとの戦いがはじまった。大抵の争乱がそうであるように、この戦いの原因も実に下らないバカげたものだった。

　（続く）

　この物語はフィクションです。実在する団体名、個人とは一切関係ありません。また、特定の技術・製品の優位性などを主張するものではありません。