クソコードとは何か

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

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

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

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

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

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

問題文に「命名規則は関係ない」って書いてあるのになぁ

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

一文字変数は命名規則ではない

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

脳がバグってるのか？
どうみても...ビジ...面白いから黙っておこう

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

>>76
hとwを高々保持するのが目的のようなやつをクラスにする必要あるやろかとは思う
あとプライベートにしてアクセッサを用意するのはいかにもなんだけど
これは言うほど嬉しくも無いと結論づけてもういいんじゃないかな
こーいうデータ保持クラスおまえらも山ほど書いてきたと思うけど
嬉しかったことあるかこれ？
結局これを受け入れる側のクラスライブラリとべったりの未来しか見えないし

**76（移動中）** · 2021/01/31(日) 22:30:49.34

>>85
素晴らしい！正解！！
理由説明も、その通り！

まぁ、解決するとしたら、そもそもこんなクラスを作るなorドメインロジックをしっかり記述しましょうってところか。

オブジェクト指向（キリッ）とか言いつつ、完全にオブジェクト指向の本質を無視したプログラムでした。

**デフォルトの名無しさん** · 2021/01/31(日) 23:07:03.11

でも教科書にはこういう例しか載っていないんだよな・・・

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

たしかに

Box box = new Box();

box.setHeight(10);
box.getHeight();
box.setWidth(7);

こんなことやってても虚しいなw
せめて

Box box1 = new Box(10, 7);
Box box2 = new Box(10, 5);

if( box1.area() < box2.area() )｛｝

くらいのことはできてほしいな
あと、コンストラクタで変な値（マイナスとか）を入れたら例外を出すとか

教科書はまだ未開拓なところがあるからしゃーない
最近の学生がどんな教科書を読んでるかは知らないけど

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

実装ありきでそれに対するアクセッサという順にしか考えないし
実装ではなくてインタフェースに対してプログラミングするとかも分かってない
インタフェースを設計する上で見るべきところに思考がフォーカスされてない
C#はプロパティがあって簡潔で嬉しい！これでアクセッサ書き放題ぐへへってなもん

89 · 2021/01/31(日) 23:29:48.84

あ、流れ読んでなかった
>>89はどれに対するレスじゃなくて雑感ね

**デフォルトの名無しさん** · 2021/01/31(日) 23:32:30.66

ふう。やれやれ

アクセッサにするのは将来の拡張性のためだろ
Javaであればプロパティが存在しないから
そうしないとインターフェース互換性を保ったまま
継承することができない。

言われてコードを書いてるだけで、その理由を知らないんだな

**デフォルトの名無しさん** · 2021/01/31(日) 23:37:56.79

自分か継承をしたことがないからって
クソコードというのはコードの話をしてない
自分の経験の話になっている

**デフォルトの名無しさん** · 2021/01/31(日) 23:39:23.18

大丈夫？ID真っ赤だぞ

**デフォルトの名無しさん** · 2021/01/31(日) 23:45:54.87

IDが赤くなるのはレスが多いからなだけだぞ？
それ以外に意味はない

**デフォルトの名無しさん** · 2021/02/02(火) 07:34:00.26

クラス利用者の事を考えずに実装したクラスは全部クソコード説

**デフォルトの名無しさん** · 2021/02/02(火) 14:23:08.30

お前の書いたのは全てクソコード説

**デフォルトの名無しさん** · 2021/02/02(火) 14:38:58.03

てめえのもな

**デフォルトの名無しさん** · 2021/02/02(火) 15:05:56.95

>>96
このスレで語られている内容を理解できない96はクソコードしか書けないニート説

**デフォルトの名無しさん** · 2021/02/02(火) 15:18:13.00

>>11
コードの良し悪しは用途と仕様が不明確ならまともな判断はできないから
どちらもわからない状態じゃコーディングプラクティスの指摘だけになる

- clothPriceはどっからやってくるの？それでいいの？
- idを指定して複数の値が返されるようなデータ構造を使うべきなの？
- filter(id).first()よりfind(id)のほうがいいのでは？
- clothPriceが1000万件になっても問題ない？
- 俺の知ってるJavaScriptでは({ id })とは書けないけど？
- .first()はどこにどういう仕様で定義されてるの？
- idに対応する値がclothPriceに存在しなかった場合にどうなるの？
- テストコードも一緒に出してね

**デフォルトの名無しさん** · 2021/02/02(火) 16:23:17.93

>>99
書いてないことをいちいち質問したらきりがない
タブの数は2でいいの？とかまででてくるだろ

それらの質問は「問題ない」として扱えば
クソコードではないという答えが正しい

**デフォルトの名無しさん** · 2021/02/02(火) 18:40:58.39

お前ら（>>26 >>99）、国語の問題で「告白された○○君はなぜ顔を赤らめたのですか」と問われたら、「そんなのどこにも書いて無い！！もしかしたら唐辛子ガムを噛んでいたら予想以上に辛くて顔を真っ赤にしただけかもしれないだろ！！！」とか真顔で言い出しそうだな。

なんなんだこのスレは

**デフォルトの名無しさん** · 2021/02/02(火) 18:53:57.65

>>101
国語の問題と科学の問題は分けて考えないといけない
実際、顔を赤らめた理由は風邪だったかもしれないわけだ

**デフォルトの名無しさん** · 2021/02/02(火) 19:17:56.91

ここまで詭弁が病気を疑うレベル
論理学が発展しそうだ

**デフォルトの名無しさん** · 2021/02/02(火) 19:18:09.01

ここまで詭弁が酷いと病気を疑うレベル
論理学が発展しそうだ

**デフォルトの名無しさん** · 2021/02/02(火) 19:20:12.00

>>103
そんな滅茶苦茶な日本語を書くほど興奮してるのか？何が導火線に火を着けたんだw

**デフォルトの名無しさん** · 2021/02/02(火) 19:27:02.71

>>102
アスペ死ね

**デフォルトの名無しさん** · 2021/02/02(火) 19:33:33.22

さあ盛り上がって参りました

**デフォルトの名無しさん** · 2021/02/02(火) 20:57:02.78

脳にコロナが沢山います

**デフォルトの名無しさん** · 2021/02/02(火) 21:30:20.78

>>11はせっかくお題を出してくれたんだから建設的に行こうや
あら探ししてても成長が無いぞ

> getPrice(id)

IDから価格を引くようなことするんなら
DBMS側で全部管理しておいて
コード上にはDM操作API(※)の操作のみがあるほうがスッキリすると思う
※例えばJavaで言ったらJDBC呼び出し

> clothPrice

あとこれ
fooPrice, barPriceってどんどん増えてきそうだけど大丈夫か？って思う
データの中身を区別したあとのものを変数に入れるのが不気味
クエリのパラメータ側にあったほうが自然に思える

なるべく細かくないものから指摘した

**デフォルトの名無しさん** · 2021/02/02(火) 21:31:10.75

訂正
×DM操作API
○DB操作API

**デフォルトの名無しさん** · 2021/02/02(火) 22:08:39.45

>>109
毎回DB読まなくてもいいようにマスタデータをメモリ上にロードしてるのかもしれない

const makeCloth = function(repository) {
const clothPrice = repository.load(…)
return {
getPrice(id) {
const price = clothPrice.find(id)
return price
}
}
}

const Cloth = makeCloth(repository)
Cloth.getPrice(id)

**デフォルトの名無しさん** · 2021/02/03(水) 00:29:01.19

明示的にポインタ型で変数宣言したらauto&にしろって指摘受けたけど型推論にメリットってあるのか
聞くのも面倒だから言う通りに修正したけど

**デフォルトの名無しさん** · 2021/02/03(水) 07:53:29.40

自分の場合、型推論は
var x = 何か
var y = x
var z = y
var a = z
...
「何か」の部分に扱う型の決定を委ねたい場合に使う
でも、「何か」に任せたら不具合になりかねない場合は使わない

**デフォルトの名無しさん** · 2021/02/03(水) 17:47:38.43

>>111
流石にそれは問題文の課題解釈

**デフォルトの名無しさん** · 2021/02/04(木) 13:52:50.53

クソコードは存在しない。
クソプログラマがいるだけだ。

**デフォルトの名無しさん** · 2021/02/04(木) 15:37:08.26

前件否定
詭弁スレかなここは

**デフォルトの名無しさん** · 2021/02/04(木) 15:40:28.24

クソプログラマが書いたコードがクソコード

**デフォルトの名無しさん** · 2021/02/04(木) 16:11:03.81

普通のコードを解釈と言う名の詭弁で棄損している
ケチを付けるためだけに延々と持論を展開したりな
すると普通のコードがゴミというレッテルを貼られる

そりゃあそうだ、そいつはアラを見つけて如何にしてレッテルを貼り付けるかだけに終始してるからだ
だからそいつには「ゴミ化」の手法がある

**デフォルトの名無しさん** · 2021/02/04(木) 17:51:50.18

すごいドヤ顔してそう

**デフォルトの名無しさん** · 2021/02/04(木) 18:10:05.67

>>118が何を言ってるのか理解できない

**デフォルトの名無しさん** · 2021/02/04(木) 18:17:16.95

詭弁の特徴のガイドライン（ム板拡張版）
・理解できる事を理解できないフリをして論点をずらす
例）sin関数を使って計算してください→sinって何？w罪？w

・議題を否定して論点をずらす
例）アンパンマンは何故、バイキンマンに勝つのか→アンパンマンなんて存在しねーよバーカバーカ

・直感と感覚を否定し、理由が言えなければ事実ではないと決めつける
例）地震が怖かった→なんで地震が怖かったの？理由が言えないのなら怖いなんて嘘だ

このスレみてたら、こんなのが思い浮かんだ
議論をもとに戻す前に、二度とこんな書き込みすんなと警告しとく

**デフォルトの名無しさん** · 2021/02/04(木) 18:29:41.03

詭弁て>>115や>>118みたいなやつだろ

**デフォルトの名無しさん** · 2021/02/04(木) 18:30:42.54

>>121
> 詭弁の特徴のガイドライン（ム板拡張版）

自分で勝手にオレオレルール作るなよｗ

**デフォルトの名無しさん** · 2021/02/04(木) 18:32:23.89

>>120
俺も理解できん

**デフォルトの名無しさん** · 2021/02/04(木) 18:34:52.95

クソスレ上げんなカス

**デフォルトの名無しさん** · 2021/02/04(木) 18:46:14.74

>>121
sin関数のくだり(>>43)は質問の仕方が悪かっただけでしょ

**デフォルトの名無しさん** · 2021/02/04(木) 18:52:47.96

ム板拡張版www

**デフォルトの名無しさん** · 2021/02/04(木) 19:04:30.45

sin関数のくだりは、明らかにMathオブジェクトだとわかるものを持ってきて
それと同一視させようとしてるから、>>43がアホなだけ

**デフォルトの名無しさん** · 2021/02/04(木) 19:56:09.47

ム板拡張版ってw
まぁ、時々、該当しそうな人を見かけるけどさw

それはさておき、酷いと感じるコードって

・仕様変更の度に膨大なプログラム変更の工数がかかるコード（人件費が無駄にかかる）
・全く品質の保証されないコード（保証するためのテストにかかる人件費が非現実的な価格になる）

これを満たしているコード全てじゃない？
これを満たすコードを分析していけば、答えが見つかりそう

**デフォルトの名無しさん** · 2021/02/05(金) 18:13:08.64

class Rest{
ログインメソッド
ユーザー情報取得メソッド
位置情報送信メソッド
位置情報送信成功通知メソッド
位置情報送信失敗通知メソッド
位置情報以外の情報送信成功通知メソッド
位置情報以外の情報送信失敗通知メソッド
HTTPエラー通知メソッド
private変数のgetterメソッド※全て定義
}

転職前の会社で見つけたAndroidクソコード
こんなコードを渡された時は転職を決意した

**デフォルトの名無しさん** · 2021/02/05(金) 20:09:09.32

位置情報送信メソッド
と
位置情報送信成功通知メソッド
は
どういう関係なの？後者は前者に渡されるコールバック関数？

**デフォルトの名無しさん** · 2021/02/05(金) 20:30:02.05

位置情報とか必要な時だけは渡すが、終わったら絶対にアプリを落とすようにしてる

**デフォルトの名無しさん** · 2021/02/05(金) 20:59:57.01

>>130
1つのクラスでバックエンドAPIのスタブをまとめてるだけなら別にクソコードじゃない

**130** · 2021/02/06(土) 00:28:38.26

>>131
後者は前者を呼んだ後、HTTPレスポンスが返ると引数に渡された関数型が呼ばれる

前者の結果を非同期で受け取るイメージ

**デフォルトの名無しさん** · 2021/02/06(土) 13:51:42.40

クソコードというよりウンコードだろ

**デフォルトの名無しさん** · 2021/02/06(土) 13:59:02.77

unicode(ウンコード)

**デフォルトの名無しさん** · 2021/02/09(火) 17:43:42.15

httpレスポンス（Json）に1個、新しい項目を追加したらプログラムの書き直しが20箇所近くで発生してワロタ
無駄なラッパーによる地獄の変更作業

**デフォルトの名無しさん** · 2021/02/10(水) 03:51:43.35

ほほwww

**デフォルトの名無しさん** · 2021/02/10(水) 23:05:34.63

1万行のメソッド
巻物のような一本モノシーケンスで途中の幾多のエラーチェックの度にいろんなフラグを立てまくり、最終的に最後の行まで到達してから関数冒頭で行なったエラーチェック結果を参照、結局エラーでしたで終了
しかもユーティリティクラスのメソッド

**デフォルトの名無しさん** · 2021/02/11(木) 12:29:17.94

頭の悪い人がMVVM

**デフォルトの名無しさん** · 2021/02/11(木) 16:13:15.21

皆、通る道は同じなんだな

**デフォルトの名無しさん** · 2021/02/12(金) 00:11:29.55

テストコードって必要？

**デフォルトの名無しさん** · 2021/02/12(金) 00:17:10.35

テストコードがないプロジェクトなんて見たことないレベル
まともにメンテナンスが続いてるソフトで探してみ

**デフォルトの名無しさん** · 2021/02/12(金) 01:50:17.30

会社コードならテスト無いの山ほどある

**デフォルトの名無しさん** · 2021/02/12(金) 07:51:11.85

自社開発のパッケージ製品作ってんだけどテストコードって書いたことないんだよね
自分自身の学習コストはともかくメンバーの学習コストが怖くてね

**デフォルトの名無しさん** · 2021/02/12(金) 08:29:33.79

メンバーのせいにするな。お前より優秀だろ。

**デフォルトの名無しさん** · 2021/02/12(金) 12:30:09.75

>>142
ちょっと前までテストコードの無い会社で働いてたけど、地獄だったよ

詳細設計の妥当性確認ができない点がヤバイ
上司やリーダーに単体テストをしていないことの危険性を説明したが...テストコード書いてもどうせ無駄になる的な事を言ってた

そもそも、そのテストコードが些細な変更で無駄になるような設計をしている事がマズイのだが...そこには触れてほしくないみたいな感じだったよ

地獄の住人は地獄しか知らない
逆に、テストコードを書く会社の人達も地獄を知らない

**デフォルトの名無しさん** · 2021/02/12(金) 22:53:23.25

でもなー単体テストの品質はどこで保証されるん？
テストコードもコードレビューするんけ？

**デフォルトの名無しさん** · 2021/02/12(金) 23:11:03.49

>>148
当たり前だろ。

**デフォルトの名無しさん** · 2021/02/12(金) 23:12:24.97

業務によらね？
頻繁に仕様が変わっちまうのに変えるのが悪いって言われてもね
それで金もらってんだし悪いもクソもないんだよ3日後には変更履歴がすだれみたいに色付いてるのに
のんびりテストコードなんて書いてたって無意味は無意味だろ

**デフォルトの名無しさん** · 2021/02/12(金) 23:13:45.01

だいたいテストコードに限らずレビューするだろ

手動のテストでもレビューしなければ

お前何のテストした？
これらのテストをしました。スクショがテストした証拠です
いや、テストしたかどうかじゃなくて、そのテスト内容は問題ないのか？
しりません。テストしました。信じてください

ってなるやろが？

**デフォルトの名無しさん** · 2021/02/12(金) 23:14:36.71

>>150
仕様が頻繁に変わるからテストしてません
でリリースするつもり？
アホなの？

**デフォルトの名無しさん** · 2021/02/12(金) 23:15:27.50

ひょっとしてテスト項目レビューとかしたことないのか？

**デフォルトの名無しさん** · 2021/02/12(金) 23:15:50.20

テストコード書いたからOKとはならんよな
重要なのは仕様から見たテストコードが適切かどうかであって書きゃいいってもんじゃない

**デフォルトの名無しさん** · 2021/02/12(金) 23:17:20.91

>>152
いや別にテストコードなんか時間かかるやん
デバッガの計算後の値をエクセルにコピペするだけでええやん

**デフォルトの名無しさん** · 2021/02/12(金) 23:19:19.72

>>152
まあ、あるんじゃね？
うちも大学の研究室に納めるやつとかはバグあったらごめんね、連絡くれたら直すから
なんて契約のものもあるし
まあ向こうに直せるぐらいの能力持ってる人いるけど

**デフォルトの名無しさん** · 2021/02/12(金) 23:20:27.11

>>154
当たり前じゃん。ただしテストコードは実際に動くから、それでテストしたということがはっきり記録される

手動だと、テストしたつもりだけどなぁ。もう一回やったらうまくいきません。
テストのやり方を間違えてたかもテヘペロってなる。これが問題

ひどい場合だと、テストの手順に漏れがあったけど
もう一回やるのが面倒だからってキャプチャを作ってごまかしたりできる
テスト項目一覧とかはそれが正しくてもそのとおりにやったことに記録にはならないから

まあ動画にでも取ってりゃ実際に何をやったかは記録できるが
その記録ムービーを全部見なければ意味がないその時間もない

**デフォルトの名無しさん** · 2021/02/12(金) 23:21:27.80

大抵書いた直後の一回だけやん必要なのって
そこでテストできていれば
とりあえずそれでええやん
コードにして引き釣り回す必要ってどこにあるん？

**デフォルトの名無しさん** · 2021/02/12(金) 23:21:48.54

>>155
> いや別にテストコードなんか時間かかるやん

手動のテストのほうが時間かかるだろ？
俺が個人で作ってるツールなんか1000を超えるテストを
数秒で終わらせることができるから

コードを修正するたびに全テストを実行できる
それと同じことをやってみ？できるんか？

**デフォルトの名無しさん** · 2021/02/12(金) 23:22:36.21

>>158
仕様が頻繁に変わるってことは、そのたびに全部のテストが必要ってことだぞ

**デフォルトの名無しさん** · 2021/02/12(金) 23:23:15.89

>>159
いや、テストコードの作成コストヤバイって
対象コードよりヤバイときのが大半やん

**デフォルトの名無しさん** · 2021/02/12(金) 23:24:15.62

>>160
だからデバッガの値のエクセル貼り付けで駄目な理由って何なん？

**デフォルトの名無しさん** · 2021/02/12(金) 23:26:11.51

>>162
エクセルに貼り付ける時にミスする可能性がある
エクセルに記録されるのは結果だけで
その手順が記録されない

**デフォルトの名無しさん** · 2021/02/12(金) 23:26:33.56

>>161
> 対象コードよりヤバイときのが大半やん
当たり前だろ。なんでその事がわからないのさ？

**デフォルトの名無しさん** · 2021/02/12(金) 23:31:57.56

てか、君が思ってるほど
無駄にコストかけられないんだよｗ

まあ、そこでやらなくても
結合とかシステムテストとかあるからさｗ
そんなに気張ってもらってもどうせ結合で出るようなの取れないし？

**デフォルトの名無しさん** · 2021/02/12(金) 23:33:28.70

それとその段階だと仕様でまずい部分も出るだろうし
あんまりガッツリテストやるよりさっさと出してもらったほうが嬉しいっつーか？

**デフォルトの名無しさん** · 2021/02/12(金) 23:36:39.78

>>165
無駄にコストをかけられないというのなら
手動テストを何回やればOKか見積もり立てられるの？

コードを修正したら全部テストいないといけないわけだ
修正した後にまたバグが見つかるかもしれないし
別のバグを入れてしまうかもしれない
そしたらまた全部テストやり直しだ

お前が言うようにコードよりもテストのほうが多くなるぐらいなのに
そのテストに時間がかかる方法を使っていたら
何度もテストできねーだろ

**デフォルトの名無しさん** · 2021/02/12(金) 23:37:47.45

>>166
リリース間近の話をしてますが？

**デフォルトの名無しさん** · 2021/02/12(金) 23:38:14.73

あと一旦リリースした後のバージョンアップとか

**デフォルトの名無しさん** · 2021/02/12(金) 23:58:17.35

>>168
え？システムテストをテストコードでできないでしょ

**デフォルトの名無しさん** · 2021/02/12(金) 23:59:41.90

>>169
あ、いや、お金にならなかったら今回で終わりなんで

**デフォルトの名無しさん** · 2021/02/13(土) 00:03:54.30

>>170
うーん、お前馬鹿なのかな？

自動化できない所が1個あるからって
全部、手動でやるとか馬鹿だよね

手動でやるのが大変だからそれを減らすって話を
してるってその時点で理解できてないの？

**デフォルトの名無しさん** · 2021/02/13(土) 00:04:38.64

>>171
じゃあもうお前の仕事なくなるじゃんｗ
自分でお金にならないような仕組みにしておいて
仕事なくすとかアホだなぁ

**デフォルトの名無しさん** · 2021/02/13(土) 00:16:07.02

>>172
うーん？聞いたことないなぁ？
んなとこまでテストコードなんて役に立たないでしょ

**デフォルトの名無しさん** · 2021/02/13(土) 00:18:40.61

>>174
本気で頭が悪そうｗ

「そんな所まで役に立たない」＝「そんな所以外では役に立つ」

ってお前はいいました。

**デフォルトの名無しさん** · 2021/02/13(土) 00:20:25.46

自分で「そんな所以外では役に立つ」って言ってるのに
気づいてなさそうなんだよなｗ

**デフォルトの名無しさん** · 2021/02/13(土) 00:20:59.63

あー、いや、作るコストまで含めたときはんなもんいらねーわ
どう考えてもテスト対象のコードより時間かけてテストコード作るってのは無意味だろ

**デフォルトの名無しさん** · 2021/02/13(土) 00:22:17.38

お前は実際にテスト対象のコードより時間かけて手動テストしてるだろーが
何を言ってるんだだこいつは？

**デフォルトの名無しさん** · 2021/02/13(土) 00:22:26.90

デバッガ機能で値をエクセルに貼るので十分だって

**デフォルトの名無しさん** · 2021/02/13(土) 00:24:07.56

まさかテストコードを書く時間しかみてなくて
テストコードがなければ、テストする時間はゼロになるとでも思ってるのか？

テストする時間の話をしてるんだが？

テストコードを書けば限りなくテストする時間は短くなる
手動テストするとテストする時間が膨大になる
テストする回数は数回程度じゃ終わらない

**デフォルトの名無しさん** · 2021/02/13(土) 00:24:17.75

>>178
アホか
ステップ実行して値のプレビューウィンドウをエビデンスに貼るだけだわ
それ以上のもんいらんし

**デフォルトの名無しさん** · 2021/02/13(土) 00:25:20.87

>>180
いや、めっちゃ時間かかるやん
いらんわお前