クソコードとは何か

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

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

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

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

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

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

>>560
お前がストレス解消のために書いてるってことはわかったよｗ

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

暇ならテスト書いていいんじゃない

**デフォルトの名無しさん** · 2021/02/22(月) 07:10:09.10

暇だからという理由で手動テストをしてたのか？
手動テストをやめれば暇になるぞｗｗｗ

**デフォルトの名無しさん** · 2021/02/22(月) 07:10:31.54

開発工数30日とかテストなんか書けるかハゲ

**デフォルトの名無しさん** · 2021/02/22(月) 07:11:40.21

>>565
だからテストしないと？
プロの仕事じゃないな

**デフォルトの名無しさん** · 2021/02/22(月) 07:11:58.72

巷には短納期かつトラブル案件だらけ

**デフォルトの名無しさん** · 2021/02/22(月) 07:12:33.00

そして短納期かつトラブル案件は自動テストがない

なるほどなｗ

**デフォルトの名無しさん** · 2021/02/22(月) 07:12:50.79

悠々とテストコード書ける案件やりたいな

**デフォルトの名無しさん** · 2021/02/22(月) 07:13:46.16

テストコード書く時間がないなら
それよりももっと時間がかかる手動テストやれるわけがないので
テストサボってるだけなんだろうな

**デフォルトの名無しさん** · 2021/02/22(月) 07:14:21.10

テストコード工数を認めてもらえる案件やりたいな
もう単体テストはエンドでやる、みたいなのばっかり

**デフォルトの名無しさん** · 2021/02/22(月) 07:15:19.40

そして仕様変更が来る

**デフォルトの名無しさん** · 2021/02/22(月) 07:16:26.70

エージェントが20%とか抜いてるからエンドがコストにシビアすぎる

**デフォルトの名無しさん** · 2021/02/22(月) 07:18:04.08

自動テストがないプロジェクトは短納期かつトラブル案件らしいｗ

**デフォルトの名無しさん** · 2021/02/22(月) 07:18:22.62

もう60日以上の案件久しくやってないな

**デフォルトの名無しさん** · 2021/02/22(月) 07:18:33.25

プロジェクトの炎上具合を測れるな
自動テストがないプロジェクトは短納期かつトラブル案件

**デフォルトの名無しさん** · 2021/02/22(月) 07:19:07.55

自動テストとか暇な案件が羨ましいわ

**デフォルトの名無しさん** · 2021/02/22(月) 07:19:23.92

自動テストを書いてないやつは、60日以上の案件をやらせてもらえないらしい

**デフォルトの名無しさん** · 2021/02/22(月) 07:20:34.85

自動テストあるのにメンテしてなくて炎上するケースが50%くらいある
ずっと自動テスト工数割いてくれよ

**デフォルトの名無しさん** · 2021/02/22(月) 07:21:38.82

60日以上の案件がほぼ無い

**デフォルトの名無しさん** · 2021/02/22(月) 07:22:34.66

120日半年くらいあればなぁ

**デフォルトの名無しさん** · 2021/02/22(月) 07:22:48.08

>>533
最近の単体テストツールは影響がないテストコードを実行しないものが多くある

**デフォルトの名無しさん** · 2021/02/22(月) 07:23:43.26

他人の書いたテストコードのメンテが辛いのは分かる

**デフォルトの名無しさん** · 2021/02/22(月) 07:27:46.33

メソッドを今流にメンテしたらテストコード通らないからギリギリまでママにされていて大規模改修とかやめてくれ

**デフォルトの名無しさん** · 2021/02/22(月) 07:29:23.41

テストを直すのがめんどくさい時が多々ある
やめてくれ

**デフォルトの名無しさん** · 2021/02/22(月) 07:52:25.48

テストコードとか書かないに限るな

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

平日にID真っ赤にする手動テストおじさんはNGしとけ
無視して俺らと会話しようぜ
仕事してるからレスは少なくなるけど

**デフォルトの名無しさん** · 2021/02/22(月) 18:49:35.65

本日の職場で見たクソコード

class MainActivity implement 機能Aインタフェース, 機能B,インタフェース機能Cインタフェース...機能Mインタフェース{

}

なんで12個のインタフェース継承してるの？
円卓の騎士かな？
Androidで発生するイベント全部、1個のアクティビティに継承して実装するのやめてください実装者が死んでしまいます

**デフォルトの名無しさん** · 2021/02/22(月) 19:43:16.58

それはクソコードと言うよりクソ設計だろ

**デフォルトの名無しさん** · 2021/02/22(月) 22:40:13.49

クソコード兼クソ設計だな

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

>>587
なんか話せよ

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

>>559
そのテストコード書くのにかかった時間は仕様変更によりすべて無駄になる世界があるんよ、君はまだ世界の広さを知らない

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

>>592
仕様変更によりテスト手順書が全て無駄になる世界はないのか？w

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

こいついっつもテストする時間のことを無視するよなｗ

テストコードを書く vs テストコードを書かないでしか比べてない
それよりも重要な何度も行うテストの時間を持ち出されたら困るのだろうな
手動で数回テストをするだけであっという間に開発コストが数倍に跳ね上がる

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

テスト書くのめんどくさいんで俺が抜けた後のやつに任せるわ

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

テストコード1回書く間に手動テスト5回はできそうなんだが？
んで1回テスト通したら次の変更までやんねーよな？

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

ここまでをまとめると、テストコードはSWAGGERとかで自動生成される部分のみでおk？

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

>>596
それ、テスト手順書書く時間入れてなくね？

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

>>593
テスト手順書なんて作るん？

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

>>598
テスト手順書って何？

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

仕様変更があるのは手順書作る前だから問題ないのさ、でもテストコードは無駄の極み底辺コーダーの自己満なのさ

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

>>597
オーケー

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

ユニットテスト100回回すより総合テスト1回の方が品質高まる、実装ミスなんて現代のプログラミング環境では無視できる

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

>>596
> テストコード1回書く間に手動テスト5回はできそうなんだが？
たった5回って少なすぎだろｗ

バグ5個見つかったらそれで終わりじゃん

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

flutterとかJITでガンガンテスト→修正→リファクタリングを繰り返すからテストコード要らないんだよね。

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

>>605
Flutter標準のやり方でテストしてるってことかな？
どの方法を使ってるの？

Testing Flutter apps
https://flutter.dev/docs/testing

Automated testing falls into a few categories:

- A unit test tests a single function, method, or class.
- A widget test (in other UI frameworks referred to as component test) tests a single widget.
- An integration test tests a complete app or a large part of an app.

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

>>606のページに書いてあった
　　　　　　　　　 Unit　　Widget　　Integration
Confidence　　　　Low　　Higher　　Highest
Maintenance cost　Low　　Higher　　Highest
Dependencies　　　Few　　More　　Most
Execution speed　　Quick　　Quick　　Slow

俺が言っていたことと同じだけど翻訳すると

・信頼性
　ユニットテスト：低い　ウィジット：高い　統合テスト：最高
・メンテナンスコスト
　ユニットテスト：低い　ウィジット：多い　統合テスト：最も多い
・依存関係
　ユニットテスト：低い　ウィジット：少し依存する　統合テスト：たくさん依存する
・実行速度
　ユニットテスト：速い　ウィジット：速い　統合テスト：遅い

統合テストはメンテナンスコストが悪くて遅いんだよね

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

>>599-600
テスト手順書知らない体なのに>>601で手順書の前とか意味不明
テスト語る前に自分の頭のデバッグしろよw

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

テストコードは暇なら書くレベルの価値しかない
AppleもGoogleも特に推奨してないし

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

はいはい、議論の負けたから続きじゃなくて
前の話に戻して逃げるとｗ

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

>>610
それ言うなら「議論に」かな？
日本語を勉強しよう！

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

やっぱりどうでもいい指摘

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

Flutterが自動テストを推奨していることをどう思うのか？

Testing Flutter apps
https://flutter.dev/docs/testing

Automated testing falls into a few categories:

- A unit test tests a single function, method, or class.
- A widget test (in other UI frameworks referred to as component test) tests a single widget.
- An integration test tests a complete app or a large part of an app.

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

テストコード自動生成ツールとかあったな・・
今どこいったんだろ・・

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

>>614
使えないものは消えるだけの話
自動テストに、テストコード自動生成とかいう眉唾ものは必要ない

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

>>613
どこで推奨してる？五万とあるプラグインのほとんどにテストコード無いけど。

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

https://developer.android.com/training/testing/fundamentals?hl=ja

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

>>616
一番最初に書いてある

The more features your app has, the harder it is to test manually.
Automated tests help ensure that your app performs correctly
before you publish it, while retaining your feature and bug fix velocity.

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

>>617
推奨はしてないね。

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

>>619
手動テストやれって書いてある？
書いてないね

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

テストコード書け、ってwarningでも見たことないし

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

>>621
カバレッジがそれに相当する
テストされてる範囲が低ければエラーになったりするし

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

つまり暇なら書くレベルなんだよ。
「テストコード工数が嵩んで納期を守れません」
なんてPJ無いし。

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

ほーらやっぱり反論できなかったｗ

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

>>623
根拠は？
俺はURL示したけど
あと、Developerサイトに書いてあることが推奨じゃなかったらなんなの？必須？

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

Xcodeとかでもテストコードカバレージ出るけどエラーになるかな・・

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

>>625
そのURLは自動テストはこうやったら良いですよ。っていう紹介ページだよ。
秋葉原のテスト専門会社に出したほうがトラブル少ないし安いし。

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

自動テストは暇ならやればいいよ。

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

>>627
手動テストはこうやったらいいよっていう紹介はないんですね
やっぱり紹介するに値しないものだからなんでしょうね

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

>>629
その意見はなんかズレてる気がするけど・・ジジイ？

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

>>630
そいつ田中勇だよ、相手にするな

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

> そのURLは自動テストはこうやったら良いですよ。っていう紹介ページだよ。
> 秋葉原のテスト専門会社に出したほうがトラブル少ないし安いし。

TDD知らないの？

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

時間ないから自動テスト作りながら開発するんじゃないの？

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

>>632
あ、もういいです。テストコードは基本書きません。
暇なら適宜書きます。

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

>>634
お前はそうなのだろうけど、他所スレでやってくれない？
ここのスレタイ読める？

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

ほんとこいつまじうぜぇ

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

>>602
で、結論が出てるので自動テストの話は終了するか自動テストスレを立てて下さい。

**デフォルトの名無しさん** · 2021/02/24(水) 01:17:48.29

モジュール仕様書って作る？javadocみたいな
cだから開発環境では作ってくれなくて、自前で用意するしかなさそうな雰囲気なんだけど自動生成してくれるツールとかない？

**デフォルトの名無しさん** · 2021/02/24(水) 01:27:10.83

コレとかは違う？というかダメか。
https://gist.github.com/nafuka11/9c031f74682b1f66ccf22e7fbca9ca4b

**デフォルトの名無しさん** · 2021/02/24(水) 05:41:22.24

>>634
テストコードを書いてないあんたが忙しいというのはよくわかった
自動テストをしてないプロジェクトが炎上するのはよく聞く話だ
やっぱり忙しくなるんだね

**デフォルトの名無しさん** · 2021/02/24(水) 07:35:11.50

蒸し返さなくていいから二人揃って隔離病棟池

**デフォルトの名無しさん** · 2021/02/24(水) 07:36:31.14

>>638
とりあえずdoxygenはCにも対応してるはず
http://www.doxygen.jp/

**デフォルトの名無しさん** · 2021/02/24(水) 07:50:22.37

>>640
おまえ死ねよ、しつこいよ

**デフォルトの名無しさん** · 2021/02/24(水) 08:17:54.05

>>641,643
お前らも相手すんな

**デフォルトの名無しさん** · 2021/02/24(水) 08:53:02.20

>>642
知らなかったありがとう

**デフォルトの名無しさん** · 2021/02/24(水) 18:26:31.18

職場のクソコード

永続化する必要のない変数、全てをSQLightで管理
O/Rマッパー？何それ？
男は黙ってSQL文の文字列を埋め込め

データの取り出し方？

**デフォルトの名無しさん** · 2021/02/24(水) 18:30:34.85

SELECTを書け
こうして、各種コードからSQLでアクセスするというグローバル変数共有化より恐ろしいクソコードができあがった

上司が勝手に外注に丸投げして作らせたコードが、プロジェクトメンバーでもない筈の俺の手元に何故か回ってきて発狂中

**デフォルトの名無しさん** · 2021/02/24(水) 18:35:50.36

今日の心がけ◆丸投げはやめましょう

一般社団法人クソコード研究所

**デフォルトの名無しさん** · 2021/02/25(木) 06:46:12.65

コメントを無駄に装飾するのやめてほしいわ
2行以上の全コメントが枠で囲ってあんの

**デフォルトの名無しさん** · 2021/02/25(木) 07:39:27.32

ポインタで挫折するってよく言われてるけど
アドレスの意味がわかってないだけ
int a = 1;
int* b= a;
a = 2;
int c = *b;
これでcに2が入ってないことがわからないとか論外だろ

**デフォルトの名無しさん** · 2021/02/25(木) 08:11:39.04

>>650
ごめん、わからんわ

**デフォルトの名無しさん** · 2021/02/25(木) 09:08:33.19

>>608
手順書がかりにあるとしてもという話だよアホが

**デフォルトの名無しさん** · 2021/02/25(木) 09:12:18.10

>>652
手順書が何かを知らないのに仮にあるとしてとかアホすぎ
恥の上塗りかよw

**デフォルトの名無しさん** · 2021/02/25(木) 09:18:01.68

>>653
何か知らなくても仮定することできるよ
ブラックホールがあるとしても僕たちが吸い込まれることを恐れることはない、のように

**デフォルトの名無しさん** · 2021/02/25(木) 09:18:59.00

知ってるってことだけでマウンティングしようとしてるアホが君だよ、浅はかなのだよ

**デフォルトの名無しさん** · 2021/02/25(木) 09:21:13.40

自動テストは底辺コーダの自己満と心に刻みなさい

**デフォルトの名無しさん** · 2021/02/25(木) 09:22:15.80

テスト手順書という言葉は君の造語だと僕は思ってるけどね、一般的なものだとは思えない

**デフォルトの名無しさん** · 2021/02/25(木) 09:23:57.48

自動テスト1万回回すより手動テスト一回の方が品質高い

**デフォルトの名無しさん** · 2021/02/25(木) 09:31:19.73

>>654
手順書が何かも知らないなら前とか分からんだろw
論理的思考能力 "0" の無能乙

**デフォルトの名無しさん** · 2021/02/25(木) 09:35:33.88

>>659
ははーん君は仮定が何かを知らないようだねえw

**デフォルトの名無しさん** · 2021/02/25(木) 09:37:40.49

君は手順書wを知っていると言ってるだけの虚ろなテープレコーダーとなり下がった