クソコードとは何か

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

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

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

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

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

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

>>375
その点テストコードは、バージョン管理するから
動いていたものが動かなくなった原因までしっかり記録される

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

>>374
は？それが正しいかどうかは仕様書見ないとわからないだろ？
仕様書とテストコードとコード見るんやな
なんか無駄じゃねぇの？
仕様書とコード見るだけでよくね？

なんでテストコード書く必要あったんだっけ？

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

そもそも仕様書とコードを見比べてどうして正しいかわからないんだっけ？

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

>>377
> 「普通、一つのコミットの中にテストコードと実装コードが含まれる」間違い、0点です。

ん？証拠を持ってくればいいわけ？

例なんていくらでもあるけど、一つのコミットの中にテストコードと実装コードが含まれてます。
https://github.com/facebook/react/pull/20813/files

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

>>381
ウィルス

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

>>379
いい加減「正しい」の意味をすり替えるのやめろ

「書いてあるとおりに正しくテストした」ことが
自動テストならわかるって言ってるだろ

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

>>382
そうだよね。そうやって嘘を付くしかないもんねｗ

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

>>383
え？それは仕様書通りって意味でないとビジネスで役に立たないよね？

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

自動テストでどうやって仕様書通りかどうかわかるの？

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

お前のテストコードがクソコードだったら終わりじゃん

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

さっきからなんの正しい説明にもなってないじゃん
自動テストも手動テストも仕様書通りかどうかテストすんだろ
何度も言わせるなクソが

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

>>381
js系の人ですか（笑）reactの場合、ver17.0.0になってもissueのOpen数が500あります。明らかにデグレを
生んでいますが、issueのClose数も1万近くです。テストが完璧だとどうやって担保しているでしょうか？（笑）
リリース数は数えたら137回です

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

>>381
ん？ショーコですか？誰もそんな事言ってないと思いますが？ｗ
参考までにそのコミットの1つ前にはテストコードが含まれてませんが？ｗ2つ前も含まれてませんw
対象コードはCIのコードですがｗよくあるFix typo も多くはテストコードとやらは含まれませんね。

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

>>387
”「書いてあるとおりに正しくテストしたことが自動テストならわかる」って言ってるだろ ”
上の方を見ても1回もそんな事はいってませんね・・・それと、日本語の区切りがおかしいので
直しておきました。「書いてあるとおりに正しくテストした」これを一言で言えばオナニー猿です。

逃げてしまったのか

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

>>327
はいはい、どうしようもない現実を知らないとか幸福な人生で何よりですw

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

テストの内容（文章 or コード）が、仕様書通りかは、手動テストも自動テストもわからない
この勝負はドロー（引き分け）

テストの内容（文章 or コード）が仕様書通り正しいと仮定する
手動テストは、テストの内容を正しく実行したとは限らないが
自動テストであれば、テストの内容を正しく実行したことが保証される
なぜならコードは書いたとおりに動くから

この勝負は自動テストの勝ち

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

>>393
だからテストコードのテストを行わなければ完結しない

ということだね

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

>>394
テストコードのテストというのは意味不明
お前の理屈だと、テストのテストが必要になるだろ
テストのテストのテストも必要になるだろ
アホらしい

テストコードに必要なのはコードレビュー
テストコードのレビューが必要なように
テスト実行手順書のレビューも必要

この勝負はドロー

そしてテストコードのレビューがOKだった場合
書いてあるテストコードの通りテストは行われるが

テスト実行手順書のレビューがOKでも
手動でテストすると、そのとおりにテストが行われるとは限らない

ということで、手動テストの負けになる

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

>>308

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

俺が折角>>305で流れ戻してやろうと思ったのに馬鹿を刺激すんな
もう手遅れだけど

馬鹿は詭弁を並べて相手にならねーから会話するな
馬鹿と会話しようとするから疲れる
テキトーに流しとけ

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

推奨するやつの頭が悪いから
テストの自動化自体も悪く見えるパターンだなこれ

ルビきちパターン

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

>>397
馬鹿筆頭の自覚無しw

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

>>399
煽り耐性無さ過ぎ...

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

スレの趣旨的にはむしろ>>305が正しい

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

>>1や>>305の中身がペラいから欲しい反応を貰えないだけやろ

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

>>320の"再利用性の無いクラスは糞"でトライしてみたんだろうけどペラすぎて役に立たないから誰も反応しない

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

そう？
むしろさり気なくanemic domainの話をしてる奴らがいたから、良スレの予感したのに一人の馬鹿が荒らした挙げ句に単体テストの必要性の有無とかいう低レベルスレになってがっかりしてる俺がここにいるけど？

内容がペラいってか、本当の経験者なら察せるし、一言二言から深堀すればいいのに

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

単体テストの話で流れるせいで誰も手が出せん
何なら、隔離スレいる？

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

ペラいとか言うけど、そもそも>>320の言ってることわかる？
他人を批判する前に貴方の実力チェックがしたいのだが、なんで再利用できないクラスがクソなのかわかる？

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

>>406
>なんで再利用できないクラスがクソなのかわかる？

その考え方がペラいって言ってんのにわかんないやつだな
これ全部同一人物じゃなきゃありえんな

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

>>407
その考えとは？わからないから教えてくれや

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

>>407
まさか再利用できないクラスが普通だとか思ってる？流石にそれはレベル低すぎてその発想はなかったけど、そういうこと？

たのむから伝わるように文章を書いてくれ
お前の文章こそ情報量がない
あるのは相手にマウントを取ろうとする意気込みだけ

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

>>408
一般常識を持たず
自分の半径5cmの出来事の一側面だけを見て
コンテキストを無視して一般化しようとする考え方

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

>>410
一般常識？ID変えながら、こんな時間に2chやる俺らが？w
発言元からコンテキスト境界を読み取れずに他人を批判するお前こそ薄ペラ野郎だわ

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

まず再利用性とは何かをまるで理解していない
アプリ全体の一箇所でしかインスタンス化されないからといって再利用性が無いとは言わない
再利用には様々な形があるにもかかわらずインスタンス化される箇所の数という著しく狭い一つの側面だけしか見えていない

そもそもインスタンス化される箇所の数と、そのクラスがクソかどうかとは何の関係も無い
ないない尽くし

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

クラスの再利用性の度合いが低ければそのクラスがクソかと言えば全くそんなことは無い
クソがどうかは再利用性の度合いとは関係ない

クラスベースのOO言語を使ってるならどんなアプリでそのアプリ特有の処理はそのアプリの用途に特化したクラスで実現する
当然、一般的に言うところのクラスとしての再利用性は無い

それだけでクソだと言い出したらGoogleが作るものでもMSが作るものでも世の中のアプリはほぼ全てクソの塊

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

ジョブがやたら多いシステムきつい

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

>>413
> クラスベースのOO言語を使ってるならどんなアプリでそのアプリ特有の処理はそのアプリの用途に特化したクラスで実現する
> 当然、一般的に言うところのクラスとしての再利用性は無い

そんなの当たり前じゃん
殺人事件が起きて殺人は駄目だと訴える遺族の前で死刑囚は殺してはいけないと言うのか？とか言い出すようなもの

それくらい察してやれ

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

って、アプリ特有ってまさか業務特有の処理とかじゃないよな？
フレームワーク特有だったらわかるけど

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

組込してるけど、USBクラス、SDカードクラスのインスタンス化が封じられていて、しかもUSBクラスの中にMCUのポートに依存する処理が書かれてたらクソだと思う

なんで、回路構成変わるたびにいちいちUSBのロジックを実装しなきゃなんねーんだよクソがって言いたい

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

>>417
なんで修正しないの？
staticなメソッドとメンバーのみを使うとしても必要なところを入れ替えるように作れるでしょ

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

トイレ行っていい？

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

>>418
文面だと伝わりづらいと思うからコード書くけど
USBポートが2ポートある場合

USB usb1 = new USB（port1）
USB usb2 = new USB（port2）

で済むのに、インスタンス化を封じられたら

USB1 usb1 = USB1.getSingleton（）
USB2 usb2 = USB2.getSingleton（）

みたいにUSB1クラス、USB2クラスを自分で定義しないといけなくなるからやめた方がいいよって話
なぜかというと、USB1と2に同じコードを書かないといけないし、もし、USBの処理に仕様変更が生じると、今後はUSB1と2を変更しないと駄目だから
面倒でしょ？2個程度なら...という油断は命取り

>>419
ご自由に

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

>>395
それで？
仕様書通りであると説明できたん？

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

>>421
何度も言ってるが
仕様書通りのテストコード（or テスト手順書）になってるかはレビューで確認する
その点ではテストコードも手動テストの手順書も差はない

差があるのは
テストコードの通りにコンピュータが実行したか（実行したと保証される）
テスト手順書のとおりに人間が実行したか（実行したと保証されない）
この点で、手動テストは劣っている

>>421
なにか言い返す言葉があるならどうぞ

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

>>420
インスタンス化しとるやーん

USBに共通する処理をコンポジションで切り出してれば1箇所の修正で済むよね

シングルトンにするメリットがある前提ならそのコードの形だけでクソコードだとは思わないな

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

>>422
え？コードレビューの他にテストコードのレビューやるの？
何がよかったんだっけ？

**320** · 2021/02/18(木) 22:15:10.24

反応してくれるやつがおっただけでも嬉しいわ（ヽ´ω｀）

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

>>412
ワイはそれをあると言っている
インスタンス化された個数こそがそのクラスの価値だと言っている
抽象クラスやインタフェースについてはこの主張では扱わない
アホらしいと思うのなら聞き流してくれていい

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

>>404
単体テストがどうのこうのってもう心底どうでもいいよなｗ
俺もそれの結論について興味が持てない

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

>>420
インスタンス化を封じて苦しくなってくるのあるよなｗ
最初のうちはそれでよさそうに思えるんだけどね

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

>>426
インスタンス化された個数と
インスタンス化される箇所の個数は全然違うと思うんだが？

コード上では1箇所でしかインスタンス化されないがリクエスト毎にインスタンス生成されるクラスとか

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

ごめんね下のほうの意味が本来言いたかったこと
実行時のことは考えず単にコード上での評価

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

>>430
そうすると何か特殊な前提を置いてるかインスタンス化する箇所という言葉の定義が全く違うかだな

インスタンス化がnewしたりファクトリメソッドを呼ぶ事だとしたら
Webでもモバイルでもデスクトップでもコード上の1箇所でしかインスタンス化されないクラスが結構な割合で存在するほうが普通

わかりやすいところで言えばコントローラクラスやアプリケーションクラス

**デフォルトの名無しさん** · 2021/02/19(金) 02:02:43.04

>>420
USB io = USB.create(port1)で物理的なUSBが重複する可能性があり、排他制御が必要ならそうすると思うよ。
USB io = new USB(port1)で個別にインスタンス作って、マルチスレッド・マルチプロセスで動くなら必要ないけど
自然的なコンストラクタでメンバー変数は初期化されることに意味があるだけで、それ以外の構築初期化を
しようとしたらファクトリーメソッドが必要になる。もちろんnewの後にメンバーメソッドを毎回自分で
呼び出して初期化しても良いわけだけど、ま、言いたいことは確かに安易な設計でシングルインスタンスは
避けるべきだけど、近代的なクラス型の言語によるメンバーの隠蔽とファクトリーメソッドは無関係

**デフォルトの名無しさん** · 2021/02/19(金) 02:35:10.05

>>424
> え？コードレビューの他にテストコードのレビューやるの？

当たり前だろ
お前コードレビューをするのは当然として、
お前、テスト手順書のレビューしないのか？
だからお前がやったテストはいつも抜けがあるんだろうがｗ
すいません、テストが漏れてましたじゃねーよ、いっつもいっつも

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

>>433
じゃあテストコードのテストもいるよね？

**デフォルトの名無しさん** · 2021/02/19(金) 02:59:07.17

>>434
お前の真似をすると
テスト手順書のテストもいるよね？

当然、答えは「はい」になるよねｗ

**デフォルトの名無しさん** · 2021/02/19(金) 06:52:05.98

>>434
いらない、レビューで足りる
理由は>>255に書いてある

**デフォルトの名無しさん** · 2021/02/19(金) 07:43:49.70

>>435
え？設計書のテストが？
テストコードってもんにしたから特有の手順が増えちゃってるんだろ

**デフォルトの名無しさん** · 2021/02/19(金) 07:48:43.11

設計書
→コード→テスト
→テスト手順書→レビュー
で済むはずが
設計書
→コード→テスト
→テストコード
→レビュー
→テスト
になってるんだろ
テストコードがソースコードである以上テストしないわけにはいかんだろ

**420** · 2021/02/19(金) 07:59:08.25

>>423
伝わりにくくてすまん
紛らわしい言い方だったか

>>428
分かってくれてありがとう

**デフォルトの名無しさん** · 2021/02/19(金) 08:24:00.57

クソ言語とは
PHP、Swift、Python
JavaScriptも昔はかなりクソだったが完全に復活した
Obj-CはSwiftに取って代わられることは無かった、コード効率が良すぎる

**デフォルトの名無しさん** · 2021/02/19(金) 08:51:26.62

>>438
レビューのあとの「テスト」ってビルドボタンを押す度に走るテストのこと？
それとと、gitとかにcommitする度に走るテストのこと？

作業をカテゴリ分けすると増えているように見えるだけで、作業量は減っているように見えるが

途中からこのスレ来たから、流れがわかっていない

**デフォルトの名無しさん** · 2021/02/19(金) 08:52:42.54

>>441
おｋ
半年Romれ

**デフォルトの名無しさん** · 2021/02/19(金) 08:54:08.49

>>438

あと、よく見ると手順が間違ってる
> 設計書
→コード
→テストコード
→レビュー（？）
→テスト（自動化）
if 自動化のテストにすべて合格したら
→結合テスト以降のテスト
だよ

**デフォルトの名無しさん** · 2021/02/19(金) 09:01:26.16

>>442
あっ（察し）
失礼しました（退場）

**デフォルトの名無しさん** · 2021/02/19(金) 10:36:32.08

>>438
> 設計書
> →コード→テスト
> →テスト手順書→レビュー
もうこの時点で意味不明w
普通は
1) 設計書 ⇒ コード（⇒ レビュー）
2) 設計書 ⇒ テスト手順書 ⇒ レビュー
3) 手動テスト（工数大）
だろ
で、自動テストなら
1) 設計書 ⇒ コード
2) 設計書 ⇒ テストコード ⇒ レビュー
3) 自動テスト（工数ほぼ "0"）
要するにテストコードは計算機に対するテスト手順書だから手間は変わらんよ
むしろプログラマーだと仕様書よりコード書く方が楽と思う奴もいっぱいいるしw
あとどっちもテストと書いてるけど手動テストと自動テストでは工数が全く違うから同じ「テスト」という言葉でごまかすなよ
なおテストコードはデバッグが必要と主張するなら>>255にちゃんと反論しろ

**デフォルトの名無しさん** · 2021/02/19(金) 10:53:49.57

テストコードは暇なら書くレベルの価値しかない。
テスト仕様書に漏れがない事が最も重要。
実データで実動作で検証するのが納品条件。

**デフォルトの名無しさん** · 2021/02/19(金) 11:02:12.72

実機繋がってないと無意味なテストにしかならんの多いのよな

**デフォルトの名無しさん** · 2021/02/19(金) 11:29:00.52

>>438
> テストコードがソースコードである以上テストしないわけにはいかんだろ
だからいらない
馬鹿なのかな？
テストコードのテストなんてお前したことないだろ？
世の中の誰もしたことないわｗ
そんなものありはしないんだから

> 設計書
> →コード→テスト
> →テスト手順書→レビュー
> で済むはずが

だめでしょ？レビューが正しいかをテストしなきゃｗｗｗ

**デフォルトの名無しさん** · 2021/02/19(金) 11:45:25.31

>>448
まて、そのレビューは本当に大丈夫なのか？
レビューが適切かどうかも検証しよう（名案）

**デフォルトの名無しさん** · 2021/02/19(金) 11:47:16.24

間違えた
レビューが正しいかのテストが適切かどうかもテストしよう
だった

**デフォルトの名無しさん** · 2021/02/19(金) 11:50:29.27

レビューした結果もちゃんと記録しないといけないからな
レビューの結果をドキュメントとして残すなら
そのドキュメントのレビューも必要になる
当然の帰結だ

**デフォルトの名無しさん** · 2021/02/19(金) 11:52:44.73

単体テストの範囲でこんな凝った仕組みいらんよ
一番やりたい結合は今度セッティングのが時間かかるんで

**デフォルトの名無しさん** · 2021/02/19(金) 12:21:07.58

「結合テストはすごく時間がかかる」

え？なんで？

「単体テストでやるべきことも
全部結合テストで手動でテストやってるからさ！」

↑馬鹿じゃね？

**デフォルトの名無しさん** · 2021/02/19(金) 12:30:35.44

敢えて皆と同じくらいテストコードの書き方を知ってる自分が弁護するけど

class クソクラス
　頻繁に仕様が変わるメソッド
　　クソコード
　不具合が生じると仕様が変わるメソッド
　　クソコード
　思考停止でプライベート変数のgetter
　　思考停止の変数リターン
　思考停止でプライベート変数のsetter
　　思考停止の変数代入

みたいなクラスを単体テストするところを考えてみよう

他の皆の無駄じゃない主張は疑いもなく事実だろうけど、テストコードは無駄という主張はある意味では事実なのだと思う

これは...扱うコードの質の違いが生んだ悲劇
そう、事故だったんだ

テストコード厨呼ばわりについては...うん（諦め）

**デフォルトの名無しさん** · 2021/02/19(金) 12:34:28.76

>>454
テストコードが無駄なケースでは
手動テストも無駄になる

頻繁に仕様が変わるメソッドがあったとして
手動テストをしたら時間の節約になるとでも？
ああ、テストしないってことね（笑）

プロの仕事じゃねーよ

**デフォルトの名無しさん** · 2021/02/19(金) 12:43:07.19

>>455
> ああ、テストしないってことね（笑）
たぶん、彼の言う手動テストは俺らの言う「テストしない」のレベルだと思う
ややこしいのは主張している本人は「テストしたつもり」になっている点

**デフォルトの名無しさん** · 2021/02/19(金) 12:54:53.31

>>455
TDD前提で語ってた
クソコードのテストコードを書こうとしたら、直ぐに設計は是正されないとだぞ

**デフォルトの名無しさん** · 2021/02/19(金) 13:02:30.29

まず、クラス設計する際にテストコードみたいなコードを書いたりしない？
クラス利用者から見たクラスの使い勝手を確かめる意味で

で、何度も試行錯誤して、素晴らしいクラスモデルができたらテストコードを書いて、その後、実装をして...
この作業プロセスができていないから単体テストは無駄だという主張が生まれたと思っていた

**デフォルトの名無しさん** · 2021/02/19(金) 13:19:31.43

>>458
abstractクラス<T>から頭の中で書いてるよ、それで十分
テストコードが必要なケースは、mockableAPIから全ケース流す時くらい、実データだとめんどくさいから

**デフォルトの名無しさん** · 2021/02/19(金) 14:03:37.42

>>456
そうなんだよな。テストしたつもりになってるんだろうなって思うよ

「手動テストでテストできる！
リリース前に、人を大量に集めて人海戦術でテストするんだ！」

なんて言われたら、バグ出たらどうするの？って聞きたくなる

「バグを直して（そこだけ）テストするだけじゃないか！」っていうだろうな
たぶん、そこだけしかテストして無くて全体の再テストをしないだろう
最後にもう一回テストすればいい？　いやいやバグはなくならないんだから
最後なんてありえないだろ

時間がないときは小さいバグは運用でカバーとかいってリリースするんだろ？
そしてあとから修正するんだろ？再テストするだろ？
何回小さいバグを修正する？そのたびに人を大量に集めて人海戦術でテストする？
どんだけ時間とコストがかかるんだよ？

たぶんその答えは
「小さいバグの修正なんだから、たぶん他に影響してないはずだ。動いていればヨシっ」
なんだろ？

プロの仕事じゃねーよ

自動テストしてるプロジェクトでは、一日数回のリリースと全テストを行うことだってある
このスピードに手動テストでは追いつけない

**デフォルトの名無しさん** · 2021/02/19(金) 14:39:19.79

自動テストじゃほとんど何もできない。
日本語/フォント間違えてるとか色違うとかカクカク動くとか。

**デフォルトの名無しさん** · 2021/02/19(金) 14:45:40.28

しばらくぶりに見たプロジェクトの自動テストが通らないことぐらい
開発現場じゃ常識なんだよ!（パプリカ：DCmini）

**デフォルトの名無しさん** · 2021/02/19(金) 14:48:09.77

>>461
> 日本語/フォント間違えてるとか色違う

それはクリティカルなバグじゃないですねｗ
正しくデータが処理されることをテストしたことありますか？
どうやってやりましたか？

**デフォルトの名無しさん** · 2021/02/19(金) 14:49:14.42

>>462
> しばらくぶりに見たプロジェクトの自動テストが通らないことぐらい
それを手動テストして動いたら面白いなｗ

まあよくあるよね。本当はバグってるのに
表面上は動いてるように見えちゃう
だから手動テストは駄目なんだよ

**デフォルトの名無しさん** · 2021/02/19(金) 14:55:40.06

>>463
直接変数に値入れればいいじゃん
コード変わったらまたやる
毎回テスト回すとか無駄

**デフォルトの名無しさん** · 2021/02/19(金) 15:01:38.27

直接変数に値入れるとは？
まさかデバッガ経由でやってんの？
デバッガを使うと挙動が変わるからテストにならない

**デフォルトの名無しさん** · 2021/02/19(金) 15:01:59.28

テストは本番用ビルドでやるのが鉄則

**デフォルトの名無しさん** · 2021/02/19(金) 15:03:42.70

>>466
mockかデバッガーで十分です

**デフォルトの名無しさん** · 2021/02/19(金) 15:06:22.55

客を自動テストして欲しいわ、言うことコロコロ変わるからw

**デフォルトの名無しさん** · 2021/02/19(金) 15:20:06.23

判定させたら99.99%がクソレスだった件

**デフォルトの名無しさん** · 2021/02/19(金) 16:24:05.46

>>468
mockやデバッガーでどうやって
何千ものテスト項目を実行するの？ｗ

**デフォルトの名無しさん** · 2021/02/19(金) 17:47:03.42

要件を確定させずレスバってプログラマどころじゃないな
しかも平日の昼間から真っ赤ってまともじゃないの自己紹介しているようなもの

**デフォルトの名無しさん** · 2021/02/19(金) 17:47:13.51

自分はアプリ開発と組み込み開発の経験があるけど、Web開発って単体テストやらんの？

Webは、Python+Django、node、古いPHPの入門書を読んだ程度にしか触れないからリアル開発事情に興味がある（開発経験が無い）

**デフォルトの名無しさん** · 2021/02/19(金) 18:57:54.50

web開発ってユーザーが入力欄に何を入れてくるかの組み合わせが
膨大になるから、テストも困難になるな。

**デフォルトの名無しさん** · 2021/02/19(金) 18:58:00.55

単体テストをやらないのはSIerだよ

ああ、いや、エクセルに単体テスト報告書を書くとか
そういう意味でなら単体テストをやってるよｗ
テストコードを書かない手動テストのことをSIerは単体テストって言ってる

**デフォルトの名無しさん** · 2021/02/19(金) 19:01:29.71

ああ、わかりやすいのが見つかった。これがSIerのいう単体テスト（手動テスト）

【Web系最高って言うけと゛本当なの？】siの５次請けから離脱したエンシ゛ニアか゛話してみた

https://www.slideshare.net/yuukinakajima794/websi-67526868#8
8. てすとほうほう winshot、エクセル、人間スクショ一枚取り忘れただけでも
テストがやり直しになるプレッシャーと進捗表.xls との戦いテスト結果.xlsが
不要なのでめんどくささがない開発者が目で見ておｋ

https://www.slideshare.net/yuukinakajima794/websi-67526868#12

12. ソース修正とかテストバグが出た！→単体テスト障害報告書.xlsを書く→
必要に応じて単体テストケース.xlsを直す→上司にお話して単体テスト障害報告
書を確認して貰う→テストをするバグが出た！→直す→必要に応じてテストパターンを
増やす→テストするあまりに、あまりにめんどくさい、進捗表.xlsには障害数も書くのでバグゼロは困るらしい

**デフォルトの名無しさん** · 2021/02/19(金) 19:04:03.28

田中勇←口だけテストコード大好き変態老人
技術力はゼロwww

**デフォルトの名無しさん** · 2021/02/19(金) 19:19:27.73

>>476
まあそういうテストが必要なフェイズは確かにあるが、その前に自動テスト入れるべきではある。