クソコードとは何か

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

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

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

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

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

**デフォルトの名無しさん** · 2021/02/27(土) 10:28:16.32

>>765
いつ実行するかをあなたは話してますね、ふーんって思いました、なんでそんな話をしてくるんだろう

**デフォルトの名無しさん** · 2021/02/27(土) 10:28:52.51

>>766
テストコード＝クソコード
なのです

**デフォルトの名無しさん** · 2021/02/27(土) 10:30:34.90

>>768
理由は？
あと、俺が最初に挙げたテスト対象コードの依存の話は？

**デフォルトの名無しさん** · 2021/02/27(土) 10:30:45.78

あれを知ってるかこれを知ってるか、そんなことも知らないのか、民主党の国会質疑みたいですね

**デフォルトの名無しさん** · 2021/02/27(土) 10:32:30.36

>>769
テストコードを書くとテストコードを実行することやテストコードをきれいに書くことが目的なってしまうからです、このスレの議論を見ればよくわかりますね

**デフォルトの名無しさん** · 2021/02/27(土) 10:32:39.93

はあ、論理的に議論できない人プログラマーにもいるんですね（皮肉）

**デフォルトの名無しさん** · 2021/02/27(土) 10:35:35.35

言うに事欠いて僕の真似をするとは、良いでしょう、あなたがその言葉を発言することを許諾します

**デフォルトの名無しさん** · 2021/02/27(土) 10:36:03.25

許しました、僕は心が広いので

**デフォルトの名無しさん** · 2021/02/27(土) 10:54:25.52

NG推奨：テスト

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

>>742
クソコーダーはクソスレ建てるのがお上手ですね！

**デフォルトの名無しさん** · 2021/02/27(土) 14:21:58.44

まずこのスレがクソスレじゃん
どんなジャンルで悪いお手本をあげつらって技術磨くってのが成り立つんだよ
状況によって対応が細かく変わりうるプログラムってジャンルだと尚更ねーよ

**デフォルトの名無しさん** · 2021/02/27(土) 14:31:49.87

>>777
は？アンチパターン知らないのなら

**デフォルトの名無しさん** · 2021/02/27(土) 14:32:21.56

は？アンチパターンも知らないのなら引っ込んでろ底辺プログラマー。

**デフォルトの名無しさん** · 2021/02/27(土) 14:41:04.16

アンパンチなら知ってるけど

**デフォルトの名無しさん** · 2021/02/27(土) 14:42:05.95

僕はバイキンマンがおしゃれだと思うし好きですけど

**デフォルトの名無しさん** · 2021/02/27(土) 14:43:15.22

ドキンちゃん、メロンパンナちゃんは可愛いです

**デフォルトの名無しさん** · 2021/02/27(土) 14:45:08.51

天どんマンとカバオくんは渋くて好きです

**デフォルトの名無しさん** · 2021/02/27(土) 14:46:27.65

愛と勇気だけが友だちという歌詞も深いですよね、大人の心にも響きます

**デフォルトの名無しさん** · 2021/02/27(土) 14:50:27.61

>>777
スレ主がコンテキストを理解しない底辺プログラマーだから許してあげてね

**デフォルトの名無しさん** · 2021/02/27(土) 15:25:16.36

コンテキスト？
Device Contextとか、Android Activityとかのコンテキスト？

**デフォルトの名無しさん** · 2021/02/27(土) 15:47:29.01

多分コンテキストって言いたかっただけなんだろうと思う…

**デフォルトの名無しさん** · 2021/02/27(土) 16:37:52.70

>>786
文脈って意味ですよ

**デフォルトの名無しさん** · 2021/02/27(土) 16:55:56.55

俺は、あわしろ氏を信じる。

お前の言うことは信用しない。

**デフォルトの名無しさん** · 2021/02/28(日) 03:55:51.43

天ぷらだとうどんと蕎麦のどっちが好き？

**デフォルトの名無しさん** · 2021/02/28(日) 03:57:33.36

カレー

**デフォルトの名無しさん** · 2021/02/28(日) 03:59:46.09

>>791
天ぷら前提なんだが、レスがハエーから許す

**デフォルトの名無しさん** · 2021/02/28(日) 04:04:03.84

天ぷらカレー

**デフォルトの名無しさん** · 2021/02/28(日) 04:07:35.88

>>793
うどんか蕎麦で

**デフォルトの名無しさん** · 2021/02/28(日) 04:53:09.79

カレーどんぶり

**デフォルトの名無しさん** · 2021/02/28(日) 09:53:36.42

>>792
みなさんこれがクソコードです

**デフォルトの名無しさん** · 2021/02/28(日) 17:33:35.64

最初は有意義になるかなと思ったのにクソスレになったな

**デフォルトの名無しさん** · 2021/02/28(日) 17:44:13.70

>>797
そう思ってたのはスレ主だけだろw
スレタイ & 1スレでどっちみちクソスレになるのは確定してた

**デフォルトの名無しさん** · 2021/02/28(日) 22:04:12.82

5chに有意義なことを期待する時点で

**デフォルトの名無しさん** · 2021/02/28(日) 22:29:20.29

Qiitaでもクソコード云々言って炎上してる記事があったな

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

>>800
試しにクソコードでQiitaを検索してみたがポエムばっかりで面白いのなかったな
↓これが視点としては面白かったけど処理分割のアプローチ間違ってるから新たなクソコードが出来そう
https://qiita.com/teradonburi/items/6483e51d5f9d47f5c22a

**デフォルトの名無しさん** · 2021/02/28(日) 23:55:31.04

>>798
いや、スレ主じゃないけどアンチパターン研究スレは厶板に必要だとは思うよ
文脈から察せない人が暴れただけで

**デフォルトの名無しさん** · 2021/03/01(月) 00:40:02.48

>>802
アンチパターンを集めたかったんなら
スレタイと1スレをそうなるよう書けって話だろ
アンチパターンとクソコードは全く別物なんだよ
コンテキストを理解してないと言われてる理由と合わせて最低限パターン・ランゲージくらいは学んでこい

ちなみに「いや」から書き始めるレスはクソレス・パターンだからな

**デフォルトの名無しさん** · 2021/03/01(月) 02:16:03.17

>>803
君がコミュ障パターンなだけだろ

**デフォルトの名無しさん** · 2021/03/01(月) 02:47:59.87

>>804
そうやって自分でクソスレにしてんだよな
クソスレ主あってのクソスレ(ム板拡張版w)

**デフォルトの名無しさん** · 2021/03/01(月) 02:51:31.50

アンチパターンを取り上げてる本でオススメある？
SQLアンチパターンを読んで得るものが多かったので

**デフォルトの名無しさん** · 2021/03/01(月) 06:38:18.35

>>805
ほうクソの定義とは何かね

**デフォルトの名無しさん** · 2021/03/01(月) 06:39:15.47

マーチン・ファウラーのりファクタリングやろな

**デフォルトの名無しさん** · 2021/03/01(月) 06:52:02.27

>>805
おいお前話を広げろ、今のお前は他人のヘイトやってるだけのクソ野郎だからそうじゃないところを見せろ

**デフォルトの名無しさん** · 2021/03/01(月) 06:56:37.18

あ、間違ってたも
>>805は僕に同意してたわけね
でも僕は敏感な人間だから僕に言われてることだと思った、だけれども紛らわしい言い方した君が悪い、僕は絶対に悪くないからこの件は以後言及することを禁止します、以上

**デフォルトの名無しさん** · 2021/03/01(月) 07:06:26.34

上に出てるパターン・ランゲージってアレクザンダーの都市計画理論だろ、プログラミングの本じゃない、そんなたとえ話のようなものでわかった気になるのは危ない

Twitterでも建築の本読んでオブジェクト指向がどうとか言ってる人いるけどポストモダンなソーカル野郎としか思えないんだよなあ、関係のないことを関係づけてそこに気づいた自分が賢いと思いこんでしまうクルクルパー、地頭の良い人が陥りやすい穴のように見える

**デフォルトの名無しさん** · 2021/03/01(月) 08:09:59.68

40年近く前の本だしFORTRAN, PL/1が題材だから今の言語に合わない箇所もあるけど名著だと思う

プログラム書法第2版 / Brian W.カーニハン P.J.Plauger 著木村泉訳 | 共立出版
https://www.kyoritsu-pub.co.jp/bookdetail/9784320020856

**デフォルトの名無しさん** · 2021/03/01(月) 08:21:42.09

プログラミング作法の方が良いと思うよ

**デフォルトの名無しさん** · 2021/03/01(月) 11:23:05.55

中国行くと肛門PCR検査

**802** · 2021/03/01(月) 12:23:46.76

コミュ症の自覚がない>>803

**デフォルトの名無しさん** · 2021/03/01(月) 17:40:20.81

>>803
まーた、詭弁デメテルバカが発狂してるよ
一番勉強が足りてないお前が勉強してないっていうねw
頼むから、一生ROMってろ

**デフォルトの名無しさん** · 2021/03/01(月) 20:15:47.54

コードとはまた違うけど
javaのインターフェースとインプリメントはクソだなと思う
無駄にファイル数増えてクソうざい

**デフォルトの名無しさん** · 2021/03/01(月) 20:59:56.79

interfaceがクソってｗ
オブジェクト指向知らないのではｗ

**デフォルトの名無しさん** · 2021/03/01(月) 21:16:55.91

これだからJava脳はw
言語周りで幼稚な愚痴垂れ流してるやつのJava率の高さったらない

**デフォルトの名無しさん** · 2021/03/01(月) 22:39:30.52

>>811
GoFのデザインパターンも、ファウラーのリファクタリングも、エリックエバンスのDDDもみんなパターン・ランゲージ

それぞれのパターン・ランゲージに共通する要素は何なのか？
なぜパターンがソフトウェア開発に有用なのか？
この辺を学んで出直してくるといいんじゃね

**デフォルトの名無しさん** · 2021/03/01(月) 22:48:23.51

クソコードと言うのは単なるBad Practiceの一種であってアンチパターンとは違うもの

アンチパターンの意味を理解せずにアンチパターンの研究とかまあ無理よね

**デフォルトの名無しさん** · 2021/03/01(月) 23:16:44.33

>>818
abstractだけでいいって事では？

**デフォルトの名無しさん** · 2021/03/01(月) 23:43:38.69

C#やるとJavaのいろいろ足りてない部分はめちゃくちゃ実感する

**デフォルトの名無しさん** · 2021/03/02(火) 00:47:10.90

>>820
君が何を学んだかを教えてよ

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

いちいち2chで教えなくていいから

**デフォルトの名無しさん** · 2021/03/02(火) 10:54:48.23

Anti-pattern
https://en.wikipedia.org/wiki/Anti-pattern

**デフォルトの名無しさん** · 2021/03/02(火) 11:43:32.18

>>818
そのインターフェース本当に必要？ってのが大半
他クラスとは何も関係無い孤立したクラスなのにインターフェース実装されてるの見たことある？
javaやってる感出すためだけにファイル数が2倍になるゴミ

**デフォルトの名無しさん** · 2021/03/02(火) 12:24:38.24

肛門コード

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

>>827
ﾎﾟﾘﾓーﾌｨｽﾞﾑ

Storage storage

storage = new USBMemory()
or
storage = new SDCard()

以下、storageを使って読み出し

storage.read()

これができなくなるとか論外

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

１メソッド１インタフェイスのペアで作れば解決するじゃないか！

interface Writer{
void write(Data data);
}
interface Reader{
Data read();
}
interface Eraser{
void erase();
}

abstract class Storage implements Writer, Reader, Eraser{
abstract void write(Data data);
abstract Data read();
abstract void erase();
}

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

>>829
そうやって複数クラスで使う前提じゃなくて
自身にしか提供しないくせにインターフェースにするクソコード見ても同じこと言えんの？

その例で言うと
Interface USBMemoryと
Implements USBMemoryImplementsと
Interface SDCardと
Implements SDCardImplementsの4つファイルが作られるんだよ
もちろんStorageクラスなんて作られないからな、覚悟しとけ？

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

>>831
それはどうかと思うけど、それを根拠に>>817みたいな言語機能を批判しちゃだめだろって話
Java脳とか言い出す人が湧いてるし

まぁ、そのコードは確かに問題だな

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

脱線したが、無意味にインタフェースと実装部を分けるクソコードは俺も見たことがある
俺も初心者時代はやらかした事はあったな

無駄にコードが増えるだけでメリットが無いというね
初心者時代は、>>831のコードを書けば
class XXXImplement
から
class XXXImplement2
に乗り換える時が楽だなんて思ってたけど、実際はXXXImplement2なんて作らずにXXXImplementを直接編集してた

変な突っ掛かり方をして悪かったな

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

仕様と実装をどこで切り離しておくかという設計選択の話なのでコードだけでは判断できない
>>831のようにインターフェースに対して実装が1種類しかなくてもそれが望ましい状況もあれば望ましくない状況もある

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

>>834
望ましい状況って例えば何なんｗ
教えてプロjavaプログラマーさん

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

インターフェース作ってもいいけど同じファイルに押し込んで欲しい
こんなくだらない内容でファイル数が倍は勘弁してほしい

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

>>835
Storageの例と一緒だよ
共通した仕様を使いたいレイヤーと複数の実装を使いわけたいレイヤーがあって
それを分離することで結合度を下げときたい場合の選択肢の一つ

USBの例で言えばSDカードや他のストレージとは異なるUSB特有の仕様に依存したコードを書く必要がある場合に
実装が変更されても利用者側のプログラムを変更しなくてもいいようにしたかったり
複数の実装を実行時に切り替えて使えるようにしておきたい場合

面倒臭さと将来の柔軟性とのトレードオフ
切る場所が適切かどうかは要件次第なのでコードからは判断できない

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

>>835
実装を変えられるってことさ
オープンクローズプリンシパルを守ろうとしたらインタフェイス切るしかない

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

クラスは分けてなんぼ、理想を言えばメソッドごとにクラスを分けるべき

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

メソッドごとにクラスを分ける理由は？

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

同じ階層のものがたくさんあると
全体を把握しにくくなるんじゃないかなあ

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

クラス毎にファイルを分ける理由は？

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

Java脳のレスはインターフェースなくてもオブジェクト指向できるっつう話でしょ
引数に関数をとることでインターフェースの代替にできる時もあるしね

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

>>842
関連するものの単位が同じだからでは？

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

>>831
あの例だと
interface IStorageにread()などの必須メソッドを追加
class USBMemory implements IStorage
class SDCard implements IStorage
やろｗ
Storage抽象クラス又は親クラスで定義してoverrideでもいいけど
設計次第やな

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

>>845
違うんだよなぁｗ

Interface USBMemory
class USBMemoryImplements implements USBMemory
Interface SDCard
class SDCardImplements implements SDCard

こんなのが爆誕してるというかほぼ全クラスこんなので統一されてる絶望感分かる？

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

>>833
まあ仕様が悪いというより簡単に悪用される実態が問題なんだよね
そのパターンもあるあるやと思ってる
だけどもし仮に2に乗り換えたところで、何かあった時に戻れるようにって無印が残されて結局ファイル数は増える地獄なんだなぁw

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

>>846
テストしやすくて最高じゃないですか

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

ワクチン効かなくて人類オワタ

**デフォルトの名無しさん** · 2021/03/04(木) 00:08:07.08

親クラスが子クラスに依存する処理を持つコード例
https://stackoverflow.com/a/29907649/

Eric Lippertの書いてるコードがクソなのか？
それともその認識がクソなのか？

**デフォルトの名無しさん** · 2021/03/04(木) 09:40:02.79

ケース・バイ・ケース

**デフォルトの名無しさん** · 2021/03/04(木) 09:49:55.40

普通にクソってことでいいんじゃね？
クソだけど処理が小さく、まともにするには
過剰気味な設計が必要になるならそのままにするけど
良いとは思わないけど必要十分

**デフォルトの名無しさん** · 2021/03/05(金) 11:07:38.15

短絡的にクソコード認定したがるやつの脳みそがクソってことだよ
>>1のようなプログラマーは自分の見方がクソなのかもしれないと考えられるようにならない限り
いつまでたってもクソプログラマーのまま

**デフォルトの名無しさん** · 2021/03/05(金) 16:16:51.44

短絡的にクソプログラマー認定したがるやつの脳みそがクソってことだよ
>>853のようなプログラマーは自分の見方がクソなのかもしれないと考えられるようにならない限り
いつまでたってもクソコードのまま

**デフォルトの名無しさん** · 2021/03/05(金) 17:34:44.54

>>854
クソプログラマー認定も含めて自分の見方がクソかもしれないと考えてないようなら
そいつ自身がクソプログラマーの可能性はそれなりに高いだろうね

ただコードとプログラマーにはクソかどうか判断する基準に決定的違いがある
その決定的違いが分かってないから君はクソプログラマーなんだよ

**デフォルトの名無しさん** · 2021/03/05(金) 18:45:25.18

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

スレ主は短絡的にクソコード認定をしていないと思うけど
むしろ、なぜクソコードを見てクソコードだと感じたのか哲学するスレだろ

クソプログラマーの定義とかスレ違いだし、どうでもいいわ

**デフォルトの名無しさん** · 2021/03/05(金) 20:45:43.09

>>856
スレ主さんwちーすっ

**デフォルトの名無しさん** · 2021/03/05(金) 23:14:44.63

>>855
底辺に何言っても無駄
他人からは学べないから

**デフォルトの名無しさん** · 2021/03/06(土) 09:14:58.43

根本的な原理が分かってない人が意外なぐらい多い
情報の多重化、DRY原則からの逸脱を誘発するものがとにかく駄目
突き詰めればダメなものは間接的でもそこにつながっているから被害を受けるのだと分かる
いろんなデザインパターンなんて結局は
いろんな状況でDRYを遵守する手練手管が99%だろう
重複実装も必要な時があるが
どんなものをどんな風に重複させるかで
状況はかなり異なるので短絡的な判断は禁物

1が言ってる一つ目の例は
もっと文脈をはっきりさせないと
何がクソなのか断定出来ないし
これで伝わると思ってるなら難アリ

**デフォルトの名無しさん** · 2021/03/06(土) 11:40:20.85

そもそもデザインパターンみたいなのは、こうすれば良いのでは？みたいな物だし
正直そういう所から入った層よりは、ある程度組めるようになってから見て納得出来た方が良いかと
つまりは最初に読むものでは無いと

**デフォルトの名無しさん** · 2021/03/06(土) 12:21:11.27

バックグラウンドとデザパタへの反応

複雑でも無く大きくも無いプログラムで遊んでた人
→自分のコードに対してデザパタの概念が大きい
→無意味にデザパタ導入してコード無意味に膨らます
→あるいはデザパタは無価値だと騒ぎ出す

ある程度複雑で大きいものを作ろうとして糞の山量産した人
→自分のコードに対してデザパタの概念が小さい
→デザパタ導入して部分的に見通し良く場面が見える
→必要に応じてデザパタを無言で使用

**デフォルトの名無しさん** · 2021/03/06(土) 12:47:09.63

オブジェクト指向を理解するにはデザインパターンを理解するのが近道
言語の基本文法を学んだ後すぐデザインパターンを学ぶと上達がはやい
パターン熱にかかるのも早い方がいい

**デフォルトの名無しさん** · 2021/03/06(土) 14:22:35.38

デザインパターンとオブジェクト指向は全く別。

**デフォルトの名無しさん** · 2021/03/06(土) 14:29:20.30

>>863
GoFのデザインパターンのことだぞ

**デフォルトの名無しさん** · 2021/03/06(土) 19:16:45.22

このスレすげえ面白えな
>>11からの流れは思わず引き込まれてしまった
下手なラノベよりよっぽど読めるわ

**デフォルトの名無しさん** · 2021/03/06(土) 20:23:40.52

そう言えば「名前がクソ」というクソパターンは
DRYとあまり関係ないな
あまりに基本過ぎて盲点だった
使ってる言葉や概念がそもそもおかしいのは
あまりに基本的な問題だが一番有りがちかもしれない
特に日本では戯言過ぎて英訳なんか不可能なぐらいの
曖昧模糊とした言葉を話す人が多いので…
コードを書くずっと前から間違ってることが多い