■無職転生プログラマが誤解していたこと

無職が転生してプログラマになったんですよ。運よく雇ってくれる会社が見つかった。

それまで一人でしかプログラムを書いたことがなかったから、コードレビューをしてもらえるということにとても期待していたし、それで安心して仕事ができると思っていた。

しかし、コードレビューは俺が期待していたものとは違うらしいと気づくまで、ものすごく時間がかかった。そんな話。

無職転生してプログラマになって、もちろん新規開発にアサインされるわけがなく既存のコードの保守が仕事となった。そのコードはなかなか大規模なもので、その全てを把握するなんて到底無理なものだった。

うん、ここですでに結論が出てるんだけど、当時の俺はそれに気がつかなった。

んで、割れ窓を直していくのが仕事なんだけど、俺がデグレが起こるようなことをしようとしていたら先輩が気づいて止めてくれる。それがコードレビューをしてもらえるってことだと思っていたわけだ。

でも、全然そうじゃなかった。

先輩がしてくれるコードレビューは追加されるコードの品質を見るというもので、デグレの可能性に気づいてくれるなんてものじゃなかった。

そりゃそうだ。コードの全貌を把握している人間はいないのだから。

だけど俺はそれに気づかず、何度かデグレによるバグを出した。

その辺りでようやくうっすら気づいてきた。コードレビューはデグレが起きないようにしてくれるものじゃないって。コードにより引き起こされるバグの責任はレビュアーじゃなくて俺にあるんだって。

レビュアーもこのコードの全貌を把握していない以上、当然の帰結だった。

それはとても怖いことだったけど、でももう乗ってしまった馬車だ。俺は俺なりに仕事をするしかなかった。

そうして仕事をしているうちに、そこそこデカい不具合を出した。

不具合の原因は、影響範囲がそこそこ広い割れ窓の修正手段をとってしまったことだった。そのそこそこ広い影響範囲の中に、作られたのが昔すぎてドキュメントも残っていないのに今だにユーザーが使っている機能があって、そこがデグレた。しかもそのデグレによって出た損失は、回復不可能なものだった。

もちろん、レビュアーがそれに気づく事はなかった。

そこで俺のメンタルは限界を迎えて、会社を辞めた。

本当に俺はどうしたらこの結末を回避できたのだろうか。

辞めてから結構時間が経った今でもわからない。

増田にいるつよつよプログラマ諸氏に、俺はどうするべきだったのかアドバイスして欲しい。

Permalink | 記事への反応(6) | 09:31

ツイートシェア