既存コードの甘い匂い (悪意なきチグハグコードの誕生) - jfluteの日記

まえがき 前提として、しっかりとコーディング規約やコーディング手順などが整備されている現場ではあまり関係ないかもしれません。 そういうのを整備して実践していことが難しい現場、スタートアップからインクリメンタル開発を経て、成長していくサービスを長期間作り上げていく事業会社が主なターゲットの話かもしれません。 既存コードに if 文がありました さて、あなたが開発現場に途中から参画しました。すでに A と B という別の人が作った画面があります。 あなたは C を作ります。C は A と B と似ています。 「さあ、作ってください」 と言われました。どうしますか？ ... まあ、普通に考えたら、A と B を参考に作りますよね。ここに甘い匂いがします。 A と B には、とある定型の if 文による制御がありました。C では一見、要らなそうに見えますが複雑でよくわからない。 C でも if 文

[sds-page]

後々まで面倒見る気あるならこれでもいいけどスポットで入って既存コードと全然違う書き方してやり逃げするのはメンテナンスコスト跳ね上がるからやめてほしい

[Shinji_041]

変なコードに合わせても、わからないコードは入れないな

[snowcrush]

ああ、これは自分も陥りがちな罠だ。ダメなコードだけど全体として統一感かまある場合合わせるべきかすごい迷う。

[koogawa]

“「既存コードに合わせて終了」の姿勢は、 プロフェッショナルとして、どうでしょう？” 耳が痛い

[n314]

プログラムではやらなくてもCSSでやっちゃう。

[narusase]

その複雑でわかりにくいifを関数化して外だしにする or 意図をきちんとドキュメンテーションする(コメント書く)してない時点で、技術的に甘いんじゃーと思ってしまうのは自分だけ？

[t2y-1979]

スキルが足りないのは仕方ないけど、自分でも考えないといけない

[onionskin]

プログラマじゃなくてもある話。設計でも運用でも、企画でさえも。

[osnq]

ぐああ

[nippondanji]

"「自分でも説明のできないコードを提出する」というのは価値の高い仕事でしょうか？"