クソコードを見る度に怖くなる

業務で結構な量のコードレビューを毎日してるんだけど 最近マジでクソコードが多い 適当に書き殴ったコードなんじゃなくて とにかく思い付いたところからコーディングして 実際に動作させたら思い通りに行かないから継ぎ接ぎで修正して 最終的に機能を満たしたから完成、PR作成、レビューよろしく、みたいなのが本当に多い 無駄な処理が多数含まれているのなんて当たり前だし 機能を満たせてるように見えるコードも境界値的なところでバグだらけだったり そういうコードが特に最近増えている 問題なのはレビューで指摘した部分が実は今回のPRではなくて既に業務システムに組み込まれてる、とかいうのも多々あって めちゃくちゃヒヤリとするようなコードも多い レビューは数人でやってるんだけど、こういうコードを通してしまう人物に2，3人心当たりがあるし とはいえ人材不足で仕方ないんだろうな、という気がしている 多分だけどソフトウェ

[verda]

ウォーターフォール時代は何百人でピラミッドを作っていたので玉石混淆な前提でガチガチな開発をしてたけど 近年省力化してアジャイルにシフトしようとした結果 なぜか玉石混淆で柔軟な開発をしているように感じる

[shikiarai]

コーディング規約を渡すとありがとうございますって言って大事にデスクトップに保管してるんだよね。コーディング規約読んだ？って聞くとデスクトップに置いてます！って返事をくれるんだ。読んだ？って聞いてるのに

[junglejungle]

Cプログラミング診断室の改善前レベルのクソコードを未だに書く人は居る。副作用の概念すら知らないでフラグ管理しまくりとか、あちこち即値だらけとか。勉強する気が無いんだろうな。

[turanukimaru]

大元はアジャイルというか、ミニウォーターフォールから設計書を抜いてアジャイルと呼んでるのが問題なことがある。スクラムは多分、何を作らなければいけないかプログラマに理解させるって工程が抜けてる。

[chikoshoot]

体感的には、根本原因は仕様・設計が煮詰まっていないからってパターンも多いかな。実装前の制約（仕様、アーキ、コーディング規約、テスト観点とか）で縛ってても "バグだらけ" って状況になるなら確かに怖いかも。

[poliphilus]

多分ちゃんとしたコードを書く人が珍しい。オープンソースとかやってるのがその「珍しい」人たちで、クソコード生産者こそがデフォルトなのだ。

[triggerhappysundaymorning]

だからレビュアーって仕事が存在するのでは。

[ustam]

プログラミングなんて書いてりゃそのうち成長するよ。まあ、現在進行形で被害を被っている君には関係ない話かもしれんが。

[y_as]

リジェクト一択なのでは？

[strow0343]

その手の奴はプロセスと規約でガチガチに縛るか設計書もどきからコード吐くツール使って直接コード書かせないとかで対策されている印象

[zyzy]

仕様がゆるゆる過ぎればそりゃそうなるだろうし、設計詰める人が駄目なのでは？

[tfurukaw]

アジャイルの名の下に仕様の詰めを後回しにして(アジャイルでも手戻りは避けるべきなのだが)、いい加減にコーディングし、それもアジャイルの名の下に有耶無耶にされ、結局納期に追われそのままリリースのパターン

[kiyo_hiko]

現場からはいいスタイルはあまり真似されないのにハンガリアンやヨーダ条件式がよく真似されてるのを見るが何だろう。自然言語に反する記法が逆にツウっぽく見えてるのかな高級言語は大抵自然言語に寄せたものなのに

[cl-gaku]

なにせ優秀なプログラマはガチャを作るのに忙しいので

[solidstatesociety]

こういう時直すだけで人に教えないと無限地獄

[nemoba]

昔そうだった。でも、動けばよいがゴールだと思ってる人も多いんだよね。だからそこをゴールにしないでくれって話すようになった。すぐコードは綺麗になるわけじゃないけど、気持ちは全然変わったよ

[brightsoda]

苦土酵素

[atsushieno]

こういう主観的な議論は、仕事が増えすぎてレビューされなくなったとか逆にレビューが多くなった結果「見る側も以前は気づかなかっただけ」とか、見る側のスキルの変化とか、いろいろ変数がありすぎてスルーでいい

[hiro_curry]

AIが書いたコードも増えてるらしいし、品質より( リリースの)スピードを重視する潮流なんだろうなあ

[yamadadadada2]

ちょっとわかるな。ここまで酷くないけど「動けばいい」と思ってる人同士で実装、レビュー、マージまでやられてると後から直す人間のモチベーションが死ぬ

[xlc]

多重請負では無能がプログラマになるので必然。その一方で本来であればIT業界に居られなかった無能に機会を与えているという側面もある。日本のIT業界の闇は深い。

[als_uz]

しんどいね

[toaruR]

上手くリードできれば評価≒ギャラあがるよー

[beresford]

そういう奴は上に上がれないだろうし(上がる気もなさそう)別にいいんじゃないの。知らんけど。

[yfukuda827]

設計書作って欲しいですよね アジャイルやってるなら単体テスト仕様書に変わるのはcode coverage になるのかな

[opnihc]

危ないコード見る機会は増えてる。派遣で人集めるくらいならChatGPTのほうがマシレベル。

[kobito19]

俺の周りの感覚では逆で、昔よりずいぶんましになった。転職しよう

[JULY]

そもそも、IT 土木の世界では、下請けはとりあえず書けますの寄せ集めで、SIer はコードを読めないし、それでなんとかなっていたのは、一部の優秀なエンジニアのおかげ。

[dekaino]

効率や美しさを捨てて misra-Cのような不毛かつ柔軟性に乏しいコーディングルールを強要するしかないのだろうか? それで死ぬ人が減らせるなら…

[WildWideWeb]

お疲れ様です。考えられる回答の例：「走りながら考える」って上の人間が言ってるのに、どうして俺たちが心配しなきゃならないの？スピード感が大事なんでしょ？まずは動くところからでしょ？

[nakamura-kenichi]

それでもロジックレベルなだけならまだマシで、結構書き方レベルで無茶苦茶やからなｗ。特にFW使ってたら銘銘が設計お構いなしに書き散らかしてリファクタも放置で、事業成長の先に破綻しか無いがどこも常態やでｗ。

[Sephy]

最近という事はChatGPTで生成したクソコードではなかろうか。あれが生成するものを動くように対症療法的に手直ししていったらジェンガみたいなやべーコードになる気がする。リーダブルコードでぶん殴れ