重要なのは優れたコードではなく上が納得するコード

下請けCOBOLERとして生きていた頃のノウハウ Advent Calendar 2016　二日目の記事です。

　自分がいた現場ではソースはすべてコメントをつけてバックアップしていました。

例えば

sample.cob

select a,
       b,
       c
 from hoge;

とあった場合に、ｄもセレクト対象となったとしましょう。

その場合は以下のように書かなければなりません。
*はコメントという意味です。

sample.cob

select a,
       b,
*** 2016/12/3 IRN7777対応 [派遣先会社名]誰々  MOD START ****
*      c
       c,
       d
*** 2016/12/3 IRN7777対応 [派遣先会社名]誰々  MOD   END ****
 from hoge;

こうなります、ある日、エンジニアのCさんは気を利かせてこう書きました。

sample.cob

select a,
       b,
       c
*** 2016/12/3 IRN7777対応 [派遣先会社名]誰々  MOD START ****
      ,d
*** 2016/12/3 IRN7777対応 [派遣先会社名]誰々  MOD   END ****
 from hoge;

こうすれば修正する行数も少なくて済むし、今後eやfが入ってきたときもコメントがごちゃごちゃしませんよね？

しかし、これが論争となりました。

上の人曰く。「いきなり書き方を変えられては困る。どうして書き方を変えたのか説明できない」というのです。

エンジニアの方の言い分としては修正行数が少なくて済む、今後追加されたときにも備えられると何度も言っているのですが、聞く耳なし

結果的にエンジニアさんが折れて前の書き方に習った対応をしていました。

どちらが正しいのかといえば、断然前者だと思います。
しかし、30分以上も同じ言い争いをして、二人合わせて工数1Hを失い、しかも職場も雰囲気が悪くなるし自分の主張は認められない。
そうであるなら後者が断然正解になります。

select文の,を前につけるか、後につけるかそんなことも変えられない現場なのでとにかく前に習え以外のすべての行動が禁止されます。

余計なことは考えず、前に習えの姿勢で行きましょう
もし仮に、リーダーが替えた方がいいじゃんと言って来たら愛想笑いを浮かべながらそうですねと言って言われた通りに直しましょう。
(不思議なことに元から前に[,]が付いていたらそれに従うのが正解となります。)