プログラマー１年目で先輩が書いたコードをレビューしてるんだが

[0001 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:55:32.051 ID:uZbzHSqV0]

悪文すぎて酷い

switch文の中にswitch文　しかも絶対に通らないケースがある
深いif文のネスト　綺麗な逆ピラミッド
まるでコピペしたかのように同じ処理を連続で書いてる（関数にして抽象化してくれ）
クソ多いフィールド
何をしてるのかわからないクラスと関数、変数

助けてくれ
これ単体テストしないといけないらしい

[0002 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:56:56.390 ID:3nsv7cqR0]

全部書き出して理解するところからスタートだな

[0003 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:57:14.488 ID:lFYghw2Y0]

それ将来のお前だよ

[0004 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:57:28.953 ID:DMDc12Rxd]

レビューってそういうことじゃないから

[0005 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:58:51.251 ID:zylMjqKg0]

規約が悪いんだろ

[0006 警備員[Lv.13(前14)][苗] 2024/04/19(金) 13:59:01.303 ID:ZGSL+Bwo0]

本人に言って修正させればいいじゃんニートじゃないんだからさあ

[0007 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 13:59:44.003 ID:rDbOdl/x0]

レビューするなら指摘して直してもらうんじゃないのか…

[0008 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:01:13.051 ID:uZbzHSqV0]

>>6
先輩だから言いづらい
>>7
すまんじゃあレビューじゃない

[0009 以下、5ちゃんねるからVIPがお送りします🐙 2024/04/19(金) 14:01:26.700 ID:vU7gTIHG0]

抽象化しすぎると何をやってるのか分かるなくなるから

[0010 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:02:26.507 ID:+u6PcEN60]

言いづらいとか言ってたら仕事にならん
駄目なものは駄目だと、上司だろうが社長だから言えるようになるのが社会人としての第一歩だ

[0011 かさのぼん ◆KSNVONyPAM 警備員[Lv.10][苗]:0.00041245 2024/04/19(金) 14:03:09.091 ID:t3xvlQ9L0]

ChatGPTに丸投げ

[0012 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:04:19.380 ID:JoAplRp10]

コードレビューってそんなことしないだろ
無から書いたヤツじゃなくて改修とかでしょ
単体テストなら改修した所だけすればいいし下手に抽象化とか関数化して直したら既存の動作保証できなくて面倒になるやつ

[0013 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:04:29.799 ID:rDbOdl/x0]

コードから単体テストケース実行用のコード書こうとしてる？

[0014 以下、5ちゃんねるからVIPがお送りします🐙 2024/04/19(金) 14:04:54.545 ID:vU7gTIHG0]

コードなんて書かなくて良いよ
生きてても意味ないんだから

[0015 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:04:58.375 ID:kV2bn9X70]

>>1
それを指摘しろよ

[0016 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:06:11.820 ID:61wV3lhw0]

単体テストまで書かせろよ

[0017 警備員[Lv.27][苗] 2024/04/19(金) 14:06:25.331 ID:jP3dbrOU0]

見にくい程度だったらほっとけ、言い合ってる時間が無駄
バグや悪影響出そうなのだけ指摘
そして経歴積んだら転職でさっさとオサラバ

[0018 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:08:32.555 ID:uZbzHSqV0]

>>16
まじで仕様もよくわかってないのに俺にぶん投げるのよくないよな
おかげでvipするしかねえよ

[0019 以下、5ちゃんねるからVIPがお送りします🐙 2024/04/19(金) 14:09:27.679 ID:vU7gTIHG0]

金もらいながらVIPはたいそうなご身分やな

[0020 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:10:02.866 ID:gCT+uj5K0]

Nintendo社長だったイワッチが現役時代
「いまあるものを活かしながら
　手直ししていく方法だと２年かかります。
　イチからつくり直していいのであれば、
　半年でやります」
て答えた時も似たようなコード渡されたんでしょうな

[0021 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:11:49.962 ID:o5chhy3y0]

それを書いた奴に指摘しないとお前の評価だけが下がるよ

本人に言えないならプロマネ辺りに「どうやってもよくわからない返答が来る」みたいに相談して自分の技能では探し当てられないがなんかおかしいみたいにして上にジャッジさせな

[0022 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:13:12.144 ID:bKhZpx6C0]

短いコードを関数化すると却って見にくくなる

[0023 以下、5ちゃんねるからVIPがお送りします 2024/04/19(金) 14:16:45.140 ID:CXYVPtFJ0]

レビュー（糞だなぁと評論する作業）でワロタ