クソコードとは何か

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

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

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

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

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

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

>>751
それただの人格攻撃ですね
はあ、論理的に議論できない人プログラマーにもいるんですね

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

>>753
1000回やる根拠がないので質問の意味がない、ナンセンスの塊

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

>>755
自分で実食を重ねると言っておきながら
実食の回数を否定するのか？ｗ

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

>>756
よく読んでください、1000回という数字に根拠がないと言ってます、また手段と目的を見誤ってますね

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

数字を増やすことが目的になっておられる

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

>>757
1000回という回数にこだわってるのはお前
試行錯誤の回数は多いほうが良いと言っている
より多く試行錯誤をするにはどうすればいいか
実食を重ねて試行錯誤していくに決まってる（笑）

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

こんなんだから日本はまともなソフトウェアを作れないんでしょうね

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

>>759
おやおやそれは無理がありすぎでしょうw
1000回という数字を出してそれを達成するにはどうすればよいかを聞いたのはあなたでしょう、他人のせいにしないでください

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

>>753であなたが言ってますよ

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

反論できなくなったらこれだから日本は～ってださすぎるだろ
そもそもIT先進国の方がテスト文化進んでるだろうに

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

日本からユーチューブやツイッターが生まれないのはユーザを見ずにコードを見てるからです

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

>>761
単体テストってビルドする度に実行するものだけど、理解できてます？
まさか、そんなことも理解せずに単体テスト批判してたの？
あと、クソコードスレなんで、是非、クソコード事例書いてみて
生産性のないお前の愚痴とか相手を挑発するだけで中身のない批判とかどうでもいいから

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

あと、クソコードの話と絡めることができないのなら、隔離スレ行ってくれませんか？

**デフォルトの名無しさん** · 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のようなプログラマーは自分の見方がクソなのかもしれないと考えられるようにならない限り
いつまでたってもクソプログラマーのまま