クソコードとは何か

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

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

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

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

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

**デフォルトの名無しさん** · 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を知っていると言ってるだけの虚ろなテープレコーダーとなり下がった

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

テスト手順書という言葉自体一般社会では通用しない代物だから知らなくていいものだとは思ってるが仮にそのような下賤なものがあるとしても問題ないと言ってるのだよ、ここまで噛み砕かないとわからないかな

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

相変わらずキチのままだな
存在の仮定には反論してないぞ
そんなことも分かってないだろw

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

誰か>>650の相手してやれよ（2行目

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

>>663
反論してないのな

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

>>664
アンドな

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

>>664
あれ、でもこれ2が入ってないことと言ってるからアンドなくていんじゃね

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

>>667
もちろんそうなんだけど、そもそもexceptionじゃね？
とかw

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

>>664
C言語すら怪しいのにドヤってるやつの相手しろってか？w

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

>>665
反論はしてない、お前のアホさを指摘してるだけw

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

>>668
それはOSによるんじゃないかな

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

>>670
反論してないのな

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

なんだ反論してなかったのか
僕の主張が全面的に正しくて反論の余地が一切なかったということか

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

2であることは保証されないよねって論旨だからOSによる動作の違いは些末なものさ

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

自動テストは底辺コーダの自己満です、システムの品質は一切上がりません、自動テストは80年代の手法

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

環境によるって話なら2が入る可能性もあるよw

**デフォルトの名無しさん** · 2021/02/25(木) 13:26:17.62

>>668
未定義動作だからそもそもcがないとかw

**デフォルトの名無しさん** · 2021/02/25(木) 13:26:53.04

>>673
アホさの指摘には触れられないってか？w

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

int a = 1;
int* b= a;

2行目の暗黙的変換がエラーにならないのって相当古い実装だよね。今でもあるのかな？

**デフォルトの名無しさん** · 2021/02/25(木) 15:33:47.71

入ってることがわからない、だった

**デフォルトの名無しさん** · 2021/02/25(木) 15:54:22.42

>>676
それは環境依存だから保証されないよねってことさ

**デフォルトの名無しさん** · 2021/02/25(木) 15:57:22.21

>>678
その言動がアホっぽいw

**デフォルトの名無しさん** · 2021/02/25(木) 15:58:24.50

>>679
コード的にはなんの問題もないからなあ

**デフォルトの名無しさん** · 2021/02/25(木) 16:00:03.24

>>680
そこの君ぃそれはおかしいぞ君ぃ

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

>>682
>>682
まあ、もうそう言うレスしかできないんだろうけど…
可哀想にw

**デフォルトの名無しさん** · 2021/02/25(木) 16:44:31.71

>>685
そっくりそのまま返すわ

**デフォルトの名無しさん** · 2021/02/25(木) 16:45:23.32

僕は鏡、君はスケベ椅子

**デフォルトの名無しさん** · 2021/02/25(木) 16:47:45.65

>>679
constraint violationなので実装依存
標準に準拠してれば最低限警告は出す

**デフォルトの名無しさん** · 2021/02/25(木) 16:53:27.63

>>686
返してどうするw
指摘に対してなにも言えないのは君ね
ちなみに反論はできないんじゃなくてする必要がないからしてないだけだから
>>663にその理由も書いてあるけど理解できてないでしょ？

**デフォルトの名無しさん** · 2021/02/25(木) 16:53:58.85

>>680
int c = *b; の時、b = 1だから、1番他のint型の内容をcに入れてるよ

**デフォルトの名無しさん** · 2021/02/25(木) 16:57:15.36

>>689
はいはいまとめると反論ありませんってことな

**デフォルトの名無しさん** · 2021/02/25(木) 17:01:11.50

僕の鉄のような正論の前で反論できなさすぎて悔しくて僕の人格を貶めようと頑張っておられるところ恐縮ですが反論ないのな

**デフォルトの名無しさん** · 2021/02/25(木) 17:05:53.09

あわしろ氏は、 >>692 は勉強しなおすべきと言ってたな。

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

>>691-692
する必要のない反論はなくて
アホさの指摘にはぐぅの音も出なくてクヤチー
ってことなw

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

>>694
反論ないのな

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

>>693
あわしろってLinux板で叩かれてるやつだろ

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

>>695
バカはレスできなくなると同じ内容をひたすら繰り返すようになるのな
もちろん反論は（必要ないから）ないし、アホと言う指摘に触れられたくない必死のアホがいるという事実も変えられないw

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

>>697
> バカはレスできなくなると同じ内容をひたすら繰り返すようになるのな

↓これなｗ

675 名前：デフォルトの名無しさん[sage] 投稿日：2021/02/25(木) 12:18:43.21 ID:2DtRWZ66 [17/27]
自動テストは底辺コーダの自己満です、システムの品質は一切上がりません、自動テストは80年代の手法

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

Googleのエンジニアに「自動テストは時間の無駄だから手動テストにしろ」って言えたら本物だよ

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

>>698
これ正しいな

**デフォルトの名無しさん** · 2021/02/25(木) 20:15:40.28

自動テストサイクルが回ってるの見たことない

**デフォルトの名無しさん** · 2021/02/25(木) 20:16:42.74

炎上の現場からお送りします。

「自動テストサイクルが回ってるの見たことない。テストする時間がない。」

**デフォルトの名無しさん** · 2021/02/25(木) 20:17:43.83

>>699
権威にすがった時点で論理的に間違ってると認めたようなもんなんだよ

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

>>702
僕はテストコードにこだわって破綻したプロジェクトを経験したことある、手段にこだわって目的を見失ったパターンですわ、それ以来僕はテストコードはにわかコーダの自己満だと思うようになったのだねえ

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

>>704
そのプロジェクトは権威がある所のプロジェクトですか？
それとも権威がない所のプロジェクトですか？

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

一回のテストで確認してるものが多すぎるのだ
ひとつの結果をチェックって
ＤＢ更新するのにレコード全体がチェック対象になんねん