クソコードとは何か

**デフォルトの名無しさん** · 2021/01/30(土) 17:33:05.78

このスレはクソコードとは何かを考えるスレです。

・親クラスが子クラスに依存する処理を持つコード
例...社員クラスを継承した正社員クラスと派遣社員クラスがあり、社員クラスが正社員クラスの知識を持つ状況

・staticにするべきではないモデルにまでstaticにする人
例...社員クラスのメソッドを全てstaticにしたり、社員クラスにシングルトンパターンに相応するものを適用する人

等、クソコードを見た時に「あっ、これクソコードだ」って認識する根拠を挙げていきましょう。

**デフォルトの名無しさん** · 2021/01/30(土) 17:33:42.44

もうレスバの予感しかしない。
産まれたての子鹿のように震えてるよ。

**デフォルトの名無しさん** · 2021/01/31(日) 10:20:47.27

>>1
クソはお前の頭だよ

**デフォルトの名無しさん** · 2021/01/31(日) 11:28:04.42

iostream ωωω

**デフォルトの名無しさん** · 2021/01/31(日) 12:06:18.22

>>3
staticおじさん乙

**デフォルトの名無しさん** · 2021/01/31(日) 12:22:48.25

開幕から自分のコードに突き刺さって痛い

**デフォルトの名無しさん** · 2021/01/31(日) 12:46:25.58

．　／⌒ヽ
　 ∩ ＾ω＾）　な　ん　だ
　 |　　 ⊂ﾉ
　｜　　 _⊃
　し ⌒．　　　＿＿＿
　　 _／ ⌒ ⌒ ＼うん、うん、
　　/)) (●) (●) 　ヽそれで？
　｜∩　(_人_)　 |
　 / ノ､＿ヽノ＿ノ￣)
　/ /　　　　　 /ﾌ_/
　L＿／＼　　　＼(

**デフォルトの名無しさん** · 2021/01/31(日) 12:59:59.35

テストコードが無いのにテストしたとか言うプログラマーの提出するコード

**デフォルトの名無しさん** · 2021/01/31(日) 16:11:11.70

同じコードを何度も書く

**デフォルトの名無しさん** · 2021/01/31(日) 16:33:43.28

コードとコードつなぎ合わせて一つのプログラム作ったけどごちゃごちゃでクソコードになってたw

**デフォルトの名無しさん** · 2021/01/31(日) 16:59:37.31

function getPrice(id) {
const price = clothPrice.filter({ id }).first()
return price
}

はい、どこがクソか指摘してみて

**デフォルトの名無しさん** · 2021/01/31(日) 17:27:33.78

どこもクソじゃない

**デフォルトの名無しさん** · 2021/01/31(日) 17:28:15.94

>>11
getPrice関数いらなくね？
...で合ってる？

理由の指摘の仕方はどうしようかな

**デフォルトの名無しさん** · 2021/01/31(日) 17:29:51.90

>>13
理由を言えないならそれはレビューではない

**デフォルトの名無しさん** · 2021/01/31(日) 17:33:42.78

まず、このコードを見てクソだと気がつけない人相手だと、言い方が重要。
だから、説明に困ってる。
なぜ、気が付かないのかとか。
考えてるだけだよ。気がついた人は先に理由を述べてほしいが。

**デフォルトの名無しさん** · 2021/01/31(日) 17:35:04.85

>>15
相手のことはまず忘れろ。
理由を言え。

そうすればあとはその理由をどう言えば伝わるかという
次の話に行けるだろ

**デフォルトの名無しさん** · 2021/01/31(日) 17:40:45.07

getPrice(id) 関数とclothPrice.filter({ id }).first()
ってやってる事は一緒だよね？
責務も殆ど一緒。
些細な違いと言えばfirst以外を呼べなくしたことだが...
たぶん、インスタンスの名称からしてclothPriceをユーザー側に隠蔽する必要も無いのに隠蔽し、挙げ句に役割のかぶったほぼ無意味な関数を無駄に増やしてしまってる。

だから、クソ...だと思った。

**デフォルトの名無しさん** · 2021/01/31(日) 17:42:50.72

あと、一言で言えば...もしかしてドメインモデル貧血症？

**デフォルトの名無しさん** · 2021/01/31(日) 17:46:41.31

> getPrice(id) 関数とclothPrice.filter({ id }).first()
> ってやってる事は一緒だよね？

関数とその中のコードが
やってることが一緒になるのは当たり前です。

関数にする意味は、コードの可読性を上げるためです。

「clothPrice.filter({ id }).first()って何やってるの？」
「Priceをgetしてます！（ getPrice(id)）！」

という会話がコードレビューであるだろうなと少しでも思ったら
それは関数にすることで可読性が上がっているということです。

**デフォルトの名無しさん** · 2021/01/31(日) 17:57:42.08

>>19

> > getPrice(id) 関数とclothPrice.filter({ id }).first()
> > ってやってる事は一緒だよね？
>
> 関数とその中のコードが
> やってることが一緒になるのは当たり前です。

実装対象の仕様が1行程度のコードで住んでいる点をまず気にして。

> 関数にする意味は、コードの可読性を上げるためです。

可読性は上がったのだろうか？
だって、何のインスタンスのオブジェクトが返されるか名前から想像できますか？

**デフォルトの名無しさん** · 2021/01/31(日) 18:00:20.47

> だって、何のインスタンスのオブジェクトが返されるか名前から想像できますか？

逆。「何のインスタンスのオブジェクトが返されるか」を意識してはいけない。
それはかっこいい言葉で言うならデメテル法則に反している
実装の詳細を意識させるな

**デフォルトの名無しさん** · 2021/01/31(日) 18:02:13.86

>>11

clothPrice.filter({ id }).first()
だったら...clothPriceインスタンスのPriceが帰ってくるんだなーって一目で分かるけど

getPrice(id)
だったら...え？何が帰ってくるのこれ？
ってなりませんか？
仮にJavaみたいな型宣言が必要な言語だったとしても、何のインスタンスのPriceが得られるのかわからないですよね？

...って、これが本当の理由かっ！？
色々、問題が想像つくから、回答に迷ってる