クソコードとは何か

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

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

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

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

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

**デフォルトの名無しさん** · 2021/02/21(日) 19:53:59.50

誰か突っ込んでやれよ

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

再利用できないコードの特徴
・凝集度が低い
・結合度が高い

この2つがセットだと、自分が使いたい機能に対して無関心な知識まで要求される
そういうのがクソコードの筆頭だと思う

Stringは、上手く実装されている例だな

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

もしかして文字列出力が多いソフトウェアだと文字列クラスの利用が多くなるんじゃないか？

そうでなくとも5000個の文字列が各所に散らばってるんだろ
どうやって文章を管理して辻褄合わせるんだよ

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

まずさ
別に出来がいいから使ってるわけじゃ無いだろ？
クラスの造りが優秀なのとソースで使用回数が多いことに関連がそんなねぇよ
業務アプリだったらLogってのが簡単にStringをぶっちぎるのなんて簡単に想像できる

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

あれ？手動テストの話終わったの？
もっと罵り合えよつまんねーな

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

最後はこれでおしまい

>>507
複雑なのはたくさん値があってその組み合わせが膨大だから
手動テストも、その組み合わせをテストしないといけない
時間がかかって現実的だからテストしないって言ってる？
手動テストはテストをしないこと・・・意味がわからんｗ

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

いや、流石にハードの動作知らんやつが設定値7に対して6.997で止まることを想定できんやろ
どんな組み合わせをやっても無理なはずやで
テキトーなことを言うな

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

>>524
ハードの動作知らんやつが手動でテストしても
何の解決もしないけど？頭大丈夫か？

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

ここまでをまとめると、自動テストは暇なら書く程度の価値
って事でおk？

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

手動テストに膨大な時間がかかるので
自動化してもっと価値が高いことに時間を使いましょうって話だよ

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

手動テストに膨大な時間なんかかからないと思うんだが
そもそも
その時間まで積んだうえの開発コストだと思うんだが

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

コロナウイルス→鳥インフルH5N8→人

もうダメだな、鶏肉も食えない

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

>>528
俺が作ってる1万行程度のツールはテスト項目数が1000個以上ある
この項目数は多いというほどのものでもないだろう
このテストを全て自動的に実行する時間はに30秒程度

もし手動で実行するなら1つあたり10秒で終わらせられたとしても
1万秒、300倍以上もの時間がかかっている
1万秒といったら、2時間半以上にもなる
開発しながらテストとかやってられないレベル

大きなシステムではテスト項目数は数万、数十万にもなるから
テストに数日かかる。コードを修正するたびにそんな事やってられないから
手動テストはテストをサボることになる

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

>>530
毎回やる必要のないテストばかりだろ？

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

> その時間まで積んだうえの開発コストだと思うんだが

開発コストは低い方がいいだろ？
そうすりゃ請求金額の多くを儲けにできる

1ヶ月かかる仕事を効率化して1週間でできるようにしたから
販売金額も1/4でいいですよなんて、技術料ゼロの儲けが出ない
ビジネスモデルで仕事してるなら可哀想としか言うしかないがなｗ

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

>>531
毎回やらなくていいとはどうやって判断したのか？
いつも想定してない所でバグが出るもんだ
コミットのたびにCIで全テストをするのは当たり前

ローカルでは一部のテストだけを行えばいいが
CIで自動的に全テストを行うのは普通の話だ
そこまでの高い信頼性を手動テストでやってるか？

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

自動テストで完結しないのに、議論に値しないだろ

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

>>534
論点はどれだけ時間を減らせるかだ
手動でしかできない部分だけ手動でやる
自動でできる部分も含めて全部手動でやる
どちらが時間がかかるか言うまでもないな

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

自動テストはヒマなら書くレベルの価値しかない

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

>>536
話し終わってねーだろｗ
手動テストが時間がかかるという話が出た途端
なぜそれまでの話をなかった事にするんだ？ｗｗｗ

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

自動テスト書くくらいなら他の仕事やれよ
単価テスターの6倍とかなんだから

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

>>538
ならテスターが自動テスト書けばいいだろｗｗｗ

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

>>539
それはアリ、でもそんな現場は無い

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

>>540
類は友を呼ぶってことだよｗ
自動テストがあるプロジェクトはたくさんある

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

プログラム分からないほうがテスターとして価値ある
時々すごい操作するし

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

メンテしなくなる自動テストのエレジー

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

もう丸1年くらいテストコード見てない

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

はい、手動テストの方が時間がかかるって
結論をそらし始めましたよｗ

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

ヒマなら書いたほうが良いのは認めてる

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

社員じゃないんで書きたくないというのもある

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

自動テスト書いた方が早いことは自動テスト書きながら開発してるやつなら分かると思うんだ

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

エンドが書けば良いと思うんだが・・オナニーと大して変わらないんだから

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

SwaggerとかでAPI自動生成される場合は書いたほうが良いな

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

自動テストを書く時間まで積んだうえの開発コストだと思うんだが？ｗ
手動でテストをすると、その開発コストが大きくかかる

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

手動テスト要らないプロジェクトを知らない

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

手動テスト要らないなんて一言も言ってないよな
自動テストを書く時間まで積んだうえの開発コスト
全部手動でテストをすると、その開発コストが大きく跳ね上がる

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

>>553
そんな事ない、テスト観点の洗い出しが変なだけ

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

>>554
テスト観点の洗い出しが良くなれば
同じ件数のテストを手動で早く実行できるという
理由を言ってみて

手動テスト要らないなんて一言も言ってないよな
自動テストを書く時間まで積んだうえの開発コスト
全部手動でテストをすると、その開発コストが大きく跳ね上がる

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

テストコードの保守工数より手動テストしたほうが確実

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

>>555
自分で考えろ

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

やっぱり話をすり替えようとしていただけだったかｗ

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

具体例

俺が作ってる1万行程度のツールはテスト項目数が1000個以上ある
この項目数は多いというほどのものでもないだろう
このテストを全て自動的に実行する時間はに30秒程度

もし手動で実行するなら1つあたり10秒で終わらせられたとしても
1万秒、300倍以上もの時間がかかっている
1万秒といったら、2時間半以上にもなる
開発しながらテストとかやってられないレベル

大きなシステムではテスト項目数は数万、数十万にもなるから
テストに数日かかる。コードを修正するたびにそんな事やってられないから
手動テストはテストをサボることになる

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

いちいち理由を聞くな、ここは自分考えだけを言って
ストレス解消する便所の落書きだ

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

まあ手動のテストで1つあたり10秒って書いたけど
普通そんな時間で終わらせられないよなｗ

事前準備の手順があって結果のデータを見比べて
画面を注意深く観察してスクショとって
チェック表にチェック入れるんだろ？
1分でも短いかな？

**デフォルトの名無しさん** · 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