■ソースコードレビューは、慎重に。

もうだいぶだいぶ時間が経ったので書き捨てていこうと思って画面を開いた。

あるシステムのソースコードレビューを頼まれ、仕事の合間に中身をみた。

モノは一応動いてて担当も喜んでるようだけど、自分からしたら何かとレベルの低い人が書いたプログラムと判断したので

思ったことを素直にかいて返事をした。

その後、幾度かレビューをしていったが、ある日、そのプログラマから

「レビューの表現が感情的かつ乱暴で傷つきます。配慮をお願いします」という旨のコメントがあった。

ハァ？と正直面食らったが、プログラムは汚いものは汚いし、設計思想もハチャメチャだし、

何より仕事なんだから、とこの要求を突っぱねた。

しばらくして、そのプログラマはココロの病で退職した。

自分のレビュー内容でひどくモチベーションを落としてそのまま…という感じだったらしい。

その話はあっという間に社内に広まった。自分を批判する人もいれば同情する人もいた。

会社からは誠心誠意謝罪をと求められ、先方に連絡をしたが受け入れてもらえなかった。

最も堪えたのは同じ職場の元婚約者の実家から婚約破棄をせまられたことだった。

法に触れるようなことをしでかす奴のとこに嫁にやれないという反応だった。

何とか誤解を解こうとしたが、

メンタルを壊すほどまでに追い込んだのは、君に原因があったのは事実じゃないか？

というのを覆すことはできなかった。

こういう時の負の情報網は収集つかないほどに速く広まってしまうのを身を以て体感した。

（２ｃｈ生活まとめ的にはザマアな展開なんだろうけど）

会社はその後すぐに退職してフリーで食いつないでいる。結婚はしていない。

コードレビューの機会もあるが、このときの反省を踏まえ、

ダメだ・クソコードだ・使えない・レベルが低い・設計が悪すぎる・など、相手に不快な思いをする表現になってないか、

こうすれば良いよという建設的な内容になるよう気をつけてはいる。

Permalink | 記事への反応(3) | 02:04

ツイートシェア