このスレはクソコードとは何かを考えるスレです。
・親クラスが子クラスに依存する処理を持つコード
例...社員クラスを継承した正社員クラスと派遣社員クラスがあり、社員クラスが正社員クラスの知識を持つ状況
・staticにするべきではないモデルにまでstaticにする人
例...社員クラスのメソッドを全てstaticにしたり、社員クラスにシングルトンパターンに相応するものを適用する人
等、クソコードを見た時に「あっ、これクソコードだ」って認識する根拠を挙げていきましょう。
探検
クソコードとは何か
■ このスレッドは過去ログ倉庫に格納されています
2021/01/30(土) 17:33:05.78ID:BjNTZWUI
624デフォルトの名無しさん
2021/02/23(火) 21:16:18.20ID:5BmsWjYf ほーらやっぱり反論できなかったw
625デフォルトの名無しさん
2021/02/23(火) 21:16:58.40ID:H7g3tUGv626デフォルトの名無しさん
2021/02/23(火) 21:18:00.51ID:0iI/BmGY Xcodeとかでもテストコードカバレージ出るけどエラーになるかな・・
627デフォルトの名無しさん
2021/02/23(火) 21:20:06.97ID:0iI/BmGY628デフォルトの名無しさん
2021/02/23(火) 21:21:23.76ID:0iI/BmGY 自動テストは暇ならやればいいよ。
629デフォルトの名無しさん
2021/02/23(火) 21:23:33.72ID:5BmsWjYf630デフォルトの名無しさん
2021/02/23(火) 21:24:44.69ID:0iI/BmGY >>629
その意見はなんかズレてる気がするけど・・ジジイ?
その意見はなんかズレてる気がするけど・・ジジイ?
631デフォルトの名無しさん
2021/02/23(火) 21:26:29.19ID:luu761LB >>630
そいつ田中勇だよ、相手にするな
そいつ田中勇だよ、相手にするな
632デフォルトの名無しさん
2021/02/23(火) 21:26:51.74ID:H7g3tUGv > そのURLは自動テストはこうやったら良いですよ。っていう紹介ページだよ。
> 秋葉原のテスト専門会社に出したほうがトラブル少ないし安いし。
TDD知らないの?
> 秋葉原のテスト専門会社に出したほうがトラブル少ないし安いし。
TDD知らないの?
633デフォルトの名無しさん
2021/02/23(火) 21:28:40.06ID:2jti1hD2 時間ないから自動テスト作りながら開発するんじゃないの?
634デフォルトの名無しさん
2021/02/23(火) 21:28:52.26ID:zrvRgTjD635デフォルトの名無しさん
2021/02/23(火) 21:30:42.76ID:H7g3tUGv636デフォルトの名無しさん
2021/02/23(火) 21:37:27.80ID:ruTDgDI5 ほんとこいつまじうぜぇ
637デフォルトの名無しさん
2021/02/23(火) 21:55:00.29ID:zrvRgTjD >>602
で、結論が出てるので自動テストの話は終了するか自動テストスレを立てて下さい。
で、結論が出てるので自動テストの話は終了するか自動テストスレを立てて下さい。
638デフォルトの名無しさん
2021/02/24(水) 01:17:48.29ID:edAGB3A3 モジュール仕様書って作る?javadocみたいな
cだから開発環境では作ってくれなくて、自前で用意するしかなさそうな雰囲気なんだけど自動生成してくれるツールとかない?
cだから開発環境では作ってくれなくて、自前で用意するしかなさそうな雰囲気なんだけど自動生成してくれるツールとかない?
639デフォルトの名無しさん
2021/02/24(水) 01:27:10.83ID:cUgjNbyW コレとかは違う?というかダメか。
https://gist.github.com/nafuka11/9c031f74682b1f66ccf22e7fbca9ca4b
https://gist.github.com/nafuka11/9c031f74682b1f66ccf22e7fbca9ca4b
640デフォルトの名無しさん
2021/02/24(水) 05:41:22.24ID:IL+ryHZw641デフォルトの名無しさん
2021/02/24(水) 07:35:11.50ID:1rMe/08W 蒸し返さなくていいから二人揃って隔離病棟池
642デフォルトの名無しさん
2021/02/24(水) 07:36:31.14ID:7Ynmy8c9643デフォルトの名無しさん
2021/02/24(水) 07:50:22.37ID:cUgjNbyW >>640
おまえ死ねよ、しつこいよ
おまえ死ねよ、しつこいよ
644デフォルトの名無しさん
2021/02/24(水) 08:17:54.05ID:144B6M1D >>641,643
お前らも相手すんな
お前らも相手すんな
645デフォルトの名無しさん
2021/02/24(水) 08:53:02.20ID:edAGB3A3 >>642
知らなかった ありがとう
知らなかった ありがとう
646デフォルトの名無しさん
2021/02/24(水) 18:26:31.18ID:FKCMCStr 職場のクソコード
永続化する必要のない変数、全てをSQLightで管理
O/Rマッパー?何それ?
男は黙ってSQL文の文字列を埋め込め
データの取り出し方?
永続化する必要のない変数、全てをSQLightで管理
O/Rマッパー?何それ?
男は黙ってSQL文の文字列を埋め込め
データの取り出し方?
647デフォルトの名無しさん
2021/02/24(水) 18:30:34.85ID:FKCMCStr SELECTを書け
こうして、各種コードからSQLでアクセスするというグローバル変数共有化より恐ろしいクソコードができあがった
上司が勝手に外注に丸投げして作らせたコードが、プロジェクトメンバーでもない筈の俺の手元に何故か回ってきて発狂中
こうして、各種コードからSQLでアクセスするというグローバル変数共有化より恐ろしいクソコードができあがった
上司が勝手に外注に丸投げして作らせたコードが、プロジェクトメンバーでもない筈の俺の手元に何故か回ってきて発狂中
648デフォルトの名無しさん
2021/02/24(水) 18:35:50.36ID:FKCMCStr 今日の心がけ◆丸投げはやめましょう
一般社団法人クソコード研究所
一般社団法人クソコード研究所
649デフォルトの名無しさん
2021/02/25(木) 06:46:12.65ID:zWIoETxB コメントを無駄に装飾するのやめてほしいわ
2行以上の全コメントが枠で囲ってあんの
2行以上の全コメントが枠で囲ってあんの
650デフォルトの名無しさん
2021/02/25(木) 07:39:27.32ID:tKZpxD44 ポインタで挫折するってよく言われてるけど
アドレスの意味がわかってないだけ
int a = 1;
int* b= a;
a = 2;
int c = *b;
これでcに2が入ってないことがわからないとか論外だろ
アドレスの意味がわかってないだけ
int a = 1;
int* b= a;
a = 2;
int c = *b;
これでcに2が入ってないことがわからないとか論外だろ
651デフォルトの名無しさん
2021/02/25(木) 08:11:39.04ID:bxBNuN1v >>650
ごめん、わからんわ
ごめん、わからんわ
652デフォルトの名無しさん
2021/02/25(木) 09:08:33.19ID:2DtRWZ66 >>608
手順書がかりにあるとしてもという話だよアホが
手順書がかりにあるとしてもという話だよアホが
653デフォルトの名無しさん
2021/02/25(木) 09:12:18.10ID:bxBNuN1v654デフォルトの名無しさん
2021/02/25(木) 09:18:01.68ID:2DtRWZ66655デフォルトの名無しさん
2021/02/25(木) 09:18:59.00ID:2DtRWZ66 知ってるってことだけでマウンティングしようとしてるアホが君だよ、浅はかなのだよ
656デフォルトの名無しさん
2021/02/25(木) 09:21:13.40ID:2DtRWZ66 自動テストは底辺コーダの自己満と心に刻みなさい
657デフォルトの名無しさん
2021/02/25(木) 09:22:15.80ID:2DtRWZ66 テスト手順書という言葉は君の造語だと僕は思ってるけどね、一般的なものだとは思えない
658デフォルトの名無しさん
2021/02/25(木) 09:23:57.48ID:2DtRWZ66 自動テスト1万回回すより手動テスト一回の方が品質高い
659デフォルトの名無しさん
2021/02/25(木) 09:31:19.73ID:bxBNuN1v660デフォルトの名無しさん
2021/02/25(木) 09:35:33.88ID:2DtRWZ66 >>659
ははーん君は仮定が何かを知らないようだねえw
ははーん君は仮定が何かを知らないようだねえw
661デフォルトの名無しさん
2021/02/25(木) 09:37:40.49ID:2DtRWZ66 君は手順書wを知っていると言ってるだけの虚ろなテープレコーダーとなり下がった
662デフォルトの名無しさん
2021/02/25(木) 09:40:12.87ID:2DtRWZ66 テスト手順書という言葉自体一般社会では通用しない代物だから知らなくていいものだとは思ってるが仮にそのような下賤なものがあるとしても問題ないと言ってるのだよ、ここまで噛み砕かないとわからないかな
663デフォルトの名無しさん
2021/02/25(木) 09:42:43.67ID:bxBNuN1v 相変わらずキチのままだな
存在の仮定には反論してないぞ
そんなことも分かってないだろw
存在の仮定には反論してないぞ
そんなことも分かってないだろw
664デフォルトの名無しさん
2021/02/25(木) 10:09:07.67ID:R7WVwVFS 誰か>>650の相手してやれよ(2行目
665デフォルトの名無しさん
2021/02/25(木) 11:15:52.49ID:2DtRWZ66 >>663
反論してないのな
反論してないのな
666デフォルトの名無しさん
2021/02/25(木) 11:16:02.45ID:2DtRWZ66 >>664
アンドな
アンドな
667デフォルトの名無しさん
2021/02/25(木) 11:18:21.06ID:2DtRWZ66 >>664
あれ、でもこれ2が入ってないことと言ってるからアンドなくていんじゃね
あれ、でもこれ2が入ってないことと言ってるからアンドなくていんじゃね
668デフォルトの名無しさん
2021/02/25(木) 11:31:45.02ID:R7WVwVFS669デフォルトの名無しさん
2021/02/25(木) 11:35:19.84ID:bxBNuN1v >>664
C言語すら怪しいのにドヤってるやつの相手しろってか?w
C言語すら怪しいのにドヤってるやつの相手しろってか?w
670デフォルトの名無しさん
2021/02/25(木) 11:35:58.12ID:bxBNuN1v >>665
反論はしてない、お前のアホさを指摘してるだけw
反論はしてない、お前のアホさを指摘してるだけw
671デフォルトの名無しさん
2021/02/25(木) 12:13:25.25ID:2DtRWZ66 >>668
それはOSによるんじゃないかな
それはOSによるんじゃないかな
672デフォルトの名無しさん
2021/02/25(木) 12:13:36.12ID:2DtRWZ66 >>670
反論してないのな
反論してないのな
673デフォルトの名無しさん
2021/02/25(木) 12:14:48.54ID:2DtRWZ66 なんだ反論してなかったのか
僕の主張が全面的に正しくて反論の余地が一切なかったということか
僕の主張が全面的に正しくて反論の余地が一切なかったということか
674デフォルトの名無しさん
2021/02/25(木) 12:16:12.07ID:2DtRWZ66 2であることは保証されないよねって論旨だからOSによる動作の違いは些末なものさ
675デフォルトの名無しさん
2021/02/25(木) 12:18:43.21ID:2DtRWZ66 自動テストは底辺コーダの自己満です、システムの品質は一切上がりません、自動テストは80年代の手法
676デフォルトの名無しさん
2021/02/25(木) 12:46:09.79ID:R7WVwVFS 環境によるって話なら2が入る可能性もあるよw
677デフォルトの名無しさん
2021/02/25(木) 13:26:17.62ID:bxBNuN1v >>668
未定義動作だからそもそもcがないとかw
未定義動作だからそもそもcがないとかw
678デフォルトの名無しさん
2021/02/25(木) 13:26:53.04ID:bxBNuN1v >>673
アホさの指摘には触れられないってか?w
アホさの指摘には触れられないってか?w
679デフォルトの名無しさん
2021/02/25(木) 14:37:56.27ID:LJq279gj int a = 1;
int* b= a;
2行目の暗黙的変換がエラーにならないのって相当古い実装だよね。今でもあるのかな?
int* b= a;
2行目の暗黙的変換がエラーにならないのって相当古い実装だよね。今でもあるのかな?
680デフォルトの名無しさん
2021/02/25(木) 15:33:47.71ID:tKZpxD44 入ってることがわからない、だった
681デフォルトの名無しさん
2021/02/25(木) 15:54:22.42ID:2DtRWZ66 >>676
それは環境依存だから保証されないよねってことさ
それは環境依存だから保証されないよねってことさ
682デフォルトの名無しさん
2021/02/25(木) 15:57:22.21ID:2DtRWZ66 >>678
その言動がアホっぽいw
その言動がアホっぽいw
683デフォルトの名無しさん
2021/02/25(木) 15:58:24.50ID:2DtRWZ66 >>679
コード的にはなんの問題もないからなあ
コード的にはなんの問題もないからなあ
684デフォルトの名無しさん
2021/02/25(木) 16:00:03.24ID:2DtRWZ66 >>680
そこの君ぃそれはおかしいぞ君ぃ
そこの君ぃそれはおかしいぞ君ぃ
685デフォルトの名無しさん
2021/02/25(木) 16:35:21.09ID:bxBNuN1v686デフォルトの名無しさん
2021/02/25(木) 16:44:31.71ID:2DtRWZ66 >>685
そっくりそのまま返すわ
そっくりそのまま返すわ
687デフォルトの名無しさん
2021/02/25(木) 16:45:23.32ID:2DtRWZ66 僕は鏡、君はスケベ椅子
688デフォルトの名無しさん
2021/02/25(木) 16:47:45.65ID:cAbWQIRq689デフォルトの名無しさん
2021/02/25(木) 16:53:27.63ID:bxBNuN1v690デフォルトの名無しさん
2021/02/25(木) 16:53:58.85ID:R7WVwVFS >>680
int c = *b; の時、b = 1だから、1番他のint型の内容をcに入れてるよ
int c = *b; の時、b = 1だから、1番他のint型の内容をcに入れてるよ
691デフォルトの名無しさん
2021/02/25(木) 16:57:15.36ID:2DtRWZ66 >>689
はいはいまとめると反論ありませんってことな
はいはいまとめると反論ありませんってことな
692デフォルトの名無しさん
2021/02/25(木) 17:01:11.50ID:2DtRWZ66 僕の鉄のような正論の前で反論できなさすぎて悔しくて僕の人格を貶めようと頑張っておられるところ恐縮ですが反論ないのな
693デフォルトの名無しさん
2021/02/25(木) 17:05:53.09ID:0Aa2beUH あわしろ氏は、 >>692 は勉強しなおすべきと言ってたな。
694デフォルトの名無しさん
2021/02/25(木) 17:22:49.64ID:bxBNuN1v695デフォルトの名無しさん
2021/02/25(木) 18:01:57.08ID:2DtRWZ66 >>694
反論ないのな
反論ないのな
696デフォルトの名無しさん
2021/02/25(木) 18:02:38.68ID:2DtRWZ66 >>693
あわしろってLinux板で叩かれてるやつだろ
あわしろってLinux板で叩かれてるやつだろ
697デフォルトの名無しさん
2021/02/25(木) 18:59:36.92ID:bxBNuN1v698デフォルトの名無しさん
2021/02/25(木) 19:03:00.55ID:d2pfH4ce >>697
> バカはレスできなくなると同じ内容をひたすら繰り返すようになるのな
↓これなw
675 名前:デフォルトの名無しさん[sage] 投稿日:2021/02/25(木) 12:18:43.21 ID:2DtRWZ66 [17/27]
自動テストは底辺コーダの自己満です、システムの品質は一切上がりません、自動テストは80年代の手法
> バカはレスできなくなると同じ内容をひたすら繰り返すようになるのな
↓これなw
675 名前:デフォルトの名無しさん[sage] 投稿日:2021/02/25(木) 12:18:43.21 ID:2DtRWZ66 [17/27]
自動テストは底辺コーダの自己満です、システムの品質は一切上がりません、自動テストは80年代の手法
699デフォルトの名無しさん
2021/02/25(木) 19:29:01.20ID:NnkAM5s4 Googleのエンジニアに「自動テストは時間の無駄だから手動テストにしろ」って言えたら本物だよ
700デフォルトの名無しさん
2021/02/25(木) 20:08:29.75ID:2DtRWZ66 >>698
これ正しいな
これ正しいな
701デフォルトの名無しさん
2021/02/25(木) 20:15:40.28ID:GSBW3E39 自動テストサイクルが回ってるの見たことない
702デフォルトの名無しさん
2021/02/25(木) 20:16:42.74ID:d2pfH4ce 炎上の現場からお送りします。
「自動テストサイクルが回ってるの見たことない。テストする時間がない。」
「自動テストサイクルが回ってるの見たことない。テストする時間がない。」
703デフォルトの名無しさん
2021/02/25(木) 20:17:43.83ID:2DtRWZ66 >>699
権威にすがった時点で論理的に間違ってると認めたようなもんなんだよ
権威にすがった時点で論理的に間違ってると認めたようなもんなんだよ
704デフォルトの名無しさん
2021/02/25(木) 20:20:05.02ID:2DtRWZ66 >>702
僕はテストコードにこだわって破綻したプロジェクトを経験したことある、手段にこだわって目的を見失ったパターンですわ、それ以来僕はテストコードはにわかコーダの自己満だと思うようになったのだねえ
僕はテストコードにこだわって破綻したプロジェクトを経験したことある、手段にこだわって目的を見失ったパターンですわ、それ以来僕はテストコードはにわかコーダの自己満だと思うようになったのだねえ
705デフォルトの名無しさん
2021/02/25(木) 20:22:11.13ID:d2pfH4ce706デフォルトの名無しさん
2021/02/25(木) 20:25:43.66ID:GSBW3E39 一回のテストで確認してるものが多すぎるのだ
ひとつの結果をチェックって
DB更新するのにレコード全体がチェック対象になんねん
ひとつの結果をチェックって
DB更新するのにレコード全体がチェック対象になんねん
707デフォルトの名無しさん
2021/02/25(木) 20:27:32.84ID:d2pfH4ce 手動テストなら、レコード全体をチェックしなくても
動いてるからヨシでリリースできるのだ
動いてるからヨシでリリースできるのだ
708デフォルトの名無しさん
2021/02/25(木) 21:59:16.03ID:2DtRWZ66 >>707
良いこと言った、君は現場をよくわかってる
良いこと言った、君は現場をよくわかってる
709デフォルトの名無しさん
2021/02/25(木) 21:59:58.67ID:2DtRWZ66 自動テスト自己満コーダたちは現場を知らない
710デフォルトの名無しさん
2021/02/26(金) 01:10:16.93ID:HiCUAcQC おっ、まだやってる?
オラオラもっと罵り合えや!
オラオラもっと罵り合えや!
711デフォルトの名無しさん
2021/02/26(金) 01:28:41.17ID:nwGfe0iv >>710
うるせえ!今何時だと思ってんだ!馬鹿野郎が!
うるせえ!今何時だと思ってんだ!馬鹿野郎が!
712デフォルトの名無しさん
2021/02/26(金) 01:30:59.68ID:nqhzic13 >>708
やっぱりプロの仕事じゃねーなw
やっぱりプロの仕事じゃねーなw
713デフォルトの名無しさん
2021/02/26(金) 03:47:36.47ID:5d4w5oMJ テストコードなんか書かせてくれるエンドなんか
もう無いだろ、・・・
もう無いだろ、・・・
714デフォルトの名無しさん
2021/02/26(金) 03:49:44.09ID:lNpcN0RJ リクルートで10年アプリ作ってたけどテスト書いたことない
715デフォルトの名無しさん
2021/02/26(金) 06:13:35.29ID:sPHbC1so なんでユニットテストは品質が上がらないんだ?
716デフォルトの名無しさん
2021/02/26(金) 07:05:56.03ID:nwGfe0iv717デフォルトの名無しさん
2021/02/26(金) 07:13:20.98ID:nwGfe0iv システムを使うのは人間なんだから人間が最初から最後まで使ってみて品質を研ぎ澄ましていくんだよネジの造形を眺めて悦に入ってるようじゃダメだね
現代の開発環境では実装ミスなんて起こらないからそこを一生懸命テストしてどうするのってこと
現代の開発環境では実装ミスなんて起こらないからそこを一生懸命テストしてどうするのってこと
718デフォルトの名無しさん
2021/02/26(金) 07:46:11.61ID:nqhzic13 一所懸命テストしてるから実装ミスが起きてない
因果関係がおかしい
因果関係がおかしい
719デフォルトの名無しさん
2021/02/26(金) 08:20:21.32ID:wRESsqiS 実装ミスの定義がわからんw
720デフォルトの名無しさん
2021/02/26(金) 08:52:36.60ID:nqhzic13 仕様バグじゃなくて、実装上のバグのことだろ?
それ以外に考えられない
それ以外に考えられない
721デフォルトの名無しさん
2021/02/26(金) 09:18:10.55ID:5d4w5oMJ もう外食を禁止しちゃえばよくね?
722デフォルトの名無しさん
2021/02/26(金) 09:43:28.53ID:nwGfe0iv 外食を禁止します!
723デフォルトの名無しさん
2021/02/26(金) 12:12:42.54ID:xnYLStfr■ このスレッドは過去ログ倉庫に格納されています
