機能追加や変更した時に、全部テストしてるでしょ?
いつもやってることじゃん
リファクタリングすると全部テストしろと言ってくるやつの矛盾
■ このスレッドは過去ログ倉庫に格納されています
2018/04/15(日) 13:13:44.63
2018/04/15(日) 13:16:31.37
それなw
あと、リファクタリングするとお前責任取れるのかとか言ってくるが
機能修正でバグった時の責任はリファクタリングじゃないんだから
全部あんたがとれよって言うと顔真っ赤になるw
あと、リファクタリングするとお前責任取れるのかとか言ってくるが
機能修正でバグった時の責任はリファクタリングじゃないんだから
全部あんたがとれよって言うと顔真っ赤になるw
2018/04/15(日) 13:23:46.40
半角カナを使う奴まだいたのか
2018/04/15(日) 13:29:25.23
スレタイ長くて入らないからね
2018/04/15(日) 13:53:39.75
簡潔にまとめられない奴はプログラミング向いてない
6仕様書無しさん
2018/04/15(日) 14:07:43.60 そんなことない
なんでもかんでも一緒にするな
なんでもかんでも一緒にするな
2018/04/15(日) 14:43:15.84
リファクタリングはテスト必須で
論理的におかしくならないと判明している
決められたやり方でやるので
リスクは低いしテストは自動化されています。
その前提ぐらい知らないんですか?
論理的におかしくならないと判明している
決められたやり方でやるので
リスクは低いしテストは自動化されています。
その前提ぐらい知らないんですか?
11仕様書無しさん
2018/04/15(日) 14:47:26.22 > そもそもクラスまで作り変えてるならテスト免除どころか設計書のレビューからだよね
そうですね? なんでレビューせずに
リファクタリングするなんて思い込んでんの?
そうですね? なんでレビューせずに
リファクタリングするなんて思い込んでんの?
12仕様書無しさん
2018/04/15(日) 15:05:33.14 > リファクタリングはテスト必須で
って書いてあるのに
↓
9 返信:仕様書無しさん[sage] 投稿日:2018/04/15(日) 14:45:55.12
>>8
だからテストやらなくていいって?
これもう頭が悪いレベルだろw
って書いてあるのに
↓
9 返信:仕様書無しさん[sage] 投稿日:2018/04/15(日) 14:45:55.12
>>8
だからテストやらなくていいって?
これもう頭が悪いレベルだろw
13仕様書無しさん
2018/04/15(日) 15:06:06.29 リファクタリングの存在を否定できないと
困る人がいるんですよw
困る人がいるんですよw
14仕様書無しさん
2018/04/15(日) 15:30:21.76 設計書から書き直すならただの改修作業なんだよね?
15仕様書無しさん
2018/04/15(日) 15:32:49.9017仕様書無しさん
2018/04/15(日) 19:48:05.09 >>15
TDDでリファクタリングした場合、
1、初歩的なミスにすぐ気付ける
2、作業者が精神的に安心できる
なお、開発者目線の開発体制の優先順位は
バージョン管理 > コードレビュー > 自動テスト > リファクタリング
なので、必ずリファクタリングとTDDはセットになる
TDDでリファクタリングした場合、
1、初歩的なミスにすぐ気付ける
2、作業者が精神的に安心できる
なお、開発者目線の開発体制の優先順位は
バージョン管理 > コードレビュー > 自動テスト > リファクタリング
なので、必ずリファクタリングとTDDはセットになる
18仕様書無しさん
2018/04/15(日) 19:48:54.60 >>16
リファクタリングの手法では先に自動化されたテストコードを書く
もしくはすでに存在していなければならない
だから、リファクタリングしたらテストしろよっていうのは意味不明。
リファクタリングの中にテストすることが条件として含まれており
修正前と修正後の両方、それどころか修正のたびにテストすると言ってるのに
なんでそんな当たり前ことを、それも最後だけテストするような質問をするんだ?と
疑問になる
おそらく修正とテストが分離した考え方しか持って無くて
修正後に手動でテストを行うというやり方しか知らないのだろう
そんな再現性がなくてテスト回数もすくないんじゃ、
バグがなかなか取れずにデスマになるのは必死だろう
リファクタリングの手法では先に自動化されたテストコードを書く
もしくはすでに存在していなければならない
だから、リファクタリングしたらテストしろよっていうのは意味不明。
リファクタリングの中にテストすることが条件として含まれており
修正前と修正後の両方、それどころか修正のたびにテストすると言ってるのに
なんでそんな当たり前ことを、それも最後だけテストするような質問をするんだ?と
疑問になる
おそらく修正とテストが分離した考え方しか持って無くて
修正後に手動でテストを行うというやり方しか知らないのだろう
そんな再現性がなくてテスト回数もすくないんじゃ、
バグがなかなか取れずにデスマになるのは必死だろう
20仕様書無しさん
2018/04/15(日) 21:17:10.48 なにをゆうちょるんじゃあ主は
21仕様書無しさん
2018/04/15(日) 21:37:54.42 リファクタリング厨って受け売りでしか語らないから面白くない
22仕様書無しさん
2018/04/15(日) 23:16:16.00 ゴミスレ
23仕様書無しさん
2018/04/16(月) 05:06:44.38 オープニングから自演w
24仕様書無しさん
2018/04/16(月) 05:08:34.0026仕様書無しさん
2018/04/16(月) 12:37:10.95 >>19
そこらへんの話でいうと、
売上や品質が上がった客観的なデータが示せりゃ、どの会社でも自動テストやリファクタリングは採用されるんよ
そこまでしてないなら、その議論は語るに値しない
んで、殆どの会社はそこを検証していない
そこらへんの話でいうと、
売上や品質が上がった客観的なデータが示せりゃ、どの会社でも自動テストやリファクタリングは採用されるんよ
そこまでしてないなら、その議論は語るに値しない
んで、殆どの会社はそこを検証していない
27仕様書無しさん
2018/04/16(月) 12:45:59.3529仕様書無しさん
2018/04/16(月) 13:36:47.19 >>27
仕事を規定の時間内に許容できる品質のものを上げるのが実装者に求める要求で、それに付随する手段は手段でしかない。
で、手段は自由にやればいいし、基本介入しないし、結果しか見る気は無い。
だからやりたければやれば?としか自分は言わない
工数増えるのは論外
仕事を規定の時間内に許容できる品質のものを上げるのが実装者に求める要求で、それに付随する手段は手段でしかない。
で、手段は自由にやればいいし、基本介入しないし、結果しか見る気は無い。
だからやりたければやれば?としか自分は言わない
工数増えるのは論外
30仕様書無しさん
2018/04/16(月) 14:37:58.22 >>26
> そこまでしてないなら、その議論は語るに値しない
> んで、殆どの会社はそこを検証していない
検証しろっていうのはわかるが、それは世間一般には
すでに検証されていてすでに品質が上がったというデータは出てるんだよな
それも検証するコストも出せないような会社がやるような中途半端な
方法じゃなくて、もっと大規模で信頼できる方法で検証した結果がすでにある。
で、それを参考にすりゃいいのに、参考にしないんだろう。
つまり他人を信用してない。だから他人がなにやってもどんなデータを出しても
そういう会社は信用しないんだろう。
> そこまでしてないなら、その議論は語るに値しない
> んで、殆どの会社はそこを検証していない
検証しろっていうのはわかるが、それは世間一般には
すでに検証されていてすでに品質が上がったというデータは出てるんだよな
それも検証するコストも出せないような会社がやるような中途半端な
方法じゃなくて、もっと大規模で信頼できる方法で検証した結果がすでにある。
で、それを参考にすりゃいいのに、参考にしないんだろう。
つまり他人を信用してない。だから他人がなにやってもどんなデータを出しても
そういう会社は信用しないんだろう。
31仕様書無しさん
2018/04/16(月) 14:41:43.64 リファクタリングが有効かどうか?
検証してやるよ。コスト?そりゃ当然かかるよ
ここで嫌がる会社が多い
重要なのは、失敗するかもしれないがやってみよう
という考えが通じる会社かどうかなんだろうな
検証してやるよ。コスト?そりゃ当然かかるよ
ここで嫌がる会社が多い
重要なのは、失敗するかもしれないがやってみよう
という考えが通じる会社かどうかなんだろうな
32仕様書無しさん
2018/04/16(月) 14:52:53.46 体力がなくて単純工すらできないごみクズがIT技術者(笑)になる日本では
コストメリットどころかリスクしかない。
コストメリットどころかリスクしかない。
33KAC
2018/04/16(月) 17:54:10.17 「リファクタリングしたら工数減ります」
「リファクタリングの為の工数ください」
うん。意味不明だね。
「リファクタリングの為の工数ください」
うん。意味不明だね。
34仕様書無しさん
2018/04/16(月) 18:19:38.72 > 「リファクタリングの為の工数ください」
誰もそんなこと言ってないけど?
>>31が言ってること勘違いした?
リファクタリングが有効かどうかの検証の話だよ。
検証するには、同じことをリファクタリグをするのと
しないので2回やらないといけない。
検証にはコストがかかる。
誰もそんなこと言ってないけど?
>>31が言ってること勘違いした?
リファクタリングが有効かどうかの検証の話だよ。
検証するには、同じことをリファクタリグをするのと
しないので2回やらないといけない。
検証にはコストがかかる。
36仕様書無しさん
2018/04/16(月) 19:33:16.6737仕様書無しさん
2018/04/16(月) 19:50:39.52 リファクタリングで増減する工数って、内部設計クラスレベル〜クラステストまででしょ
詳細設計省かずやる巨大案件でも無い限り、実装者に委ねられる部分やん。
ともすれば、やりたきゃやれって回答にしかならん。
詳細設計省かずやる巨大案件でも無い限り、実装者に委ねられる部分やん。
ともすれば、やりたきゃやれって回答にしかならん。
38仕様書無しさん
2018/04/16(月) 20:03:20.01 >>36
> いや、リファクタリングで総工数が減るなら勝手にやればいいだけじゃね?
俺はやってるよ。リファクタリングで工数減るから
そもそも修正作業の中に自動的に組み込まれるものだから
あえて "リファクタリング単体で" 工数を取るなんてことはしない
だからリファクタリングのために工数くださいっていう発想が
そもそも間違ってるわけ。修正に伴う追加作業じゃない。
修正作業そのものなんだから
> いや、リファクタリングで総工数が減るなら勝手にやればいいだけじゃね?
俺はやってるよ。リファクタリングで工数減るから
そもそも修正作業の中に自動的に組み込まれるものだから
あえて "リファクタリング単体で" 工数を取るなんてことはしない
だからリファクタリングのために工数くださいっていう発想が
そもそも間違ってるわけ。修正に伴う追加作業じゃない。
修正作業そのものなんだから
39仕様書無しさん
2018/04/16(月) 20:08:32.88 仕組みが悪いからって正常に動いてる箇所まで手を入れるってねーな
40仕様書無しさん
2018/04/16(月) 20:37:02.97 リファクタ自体は否定しないが、>>1は確実に技術に溺れて盲信してるアホだから許可出さない
41仕様書無しさん
2018/04/16(月) 21:02:09.3044仕様書無しさん
2018/04/16(月) 21:24:48.73 修正ってなんだよ、バグってんのか
機能追加だろうがなんでも修正って言う奴は素人でしょ
機能追加だろうがなんでも修正って言う奴は素人でしょ
45仕様書無しさん
2018/04/16(月) 21:26:39.53 ただのレッテル張りじゃねえかw
46仕様書無しさん
2018/04/16(月) 21:27:14.47 素人というやつは童貞だ
イカ臭いプログラム書いてろ
と言ってるようなもの
イカ臭いプログラム書いてろ
と言ってるようなもの
47仕様書無しさん
2018/04/16(月) 21:30:14.08 >>43
なんどもリファクタリングは修正の一部だって言ってるじゃん
一部ってことはそれ以外も有るってことだよ
複雑なコードを読み解いて、リファクタリングしないで複雑なまま修正するのと
複雑なコードを読み解いて、リファクタリングしてから修正するのと
どっちがリスクが少ないかって話だ
前者だと、複雑なコードを読み解いた人(修正担当者)がいるにもかかわらず
別の人が、また大変な思いをして読み解いてレビューしなければいけない
そして、次同じところを修正する時、また複雑なコード読み解かなければいけない
なんどもリファクタリングは修正の一部だって言ってるじゃん
一部ってことはそれ以外も有るってことだよ
複雑なコードを読み解いて、リファクタリングしないで複雑なまま修正するのと
複雑なコードを読み解いて、リファクタリングしてから修正するのと
どっちがリスクが少ないかって話だ
前者だと、複雑なコードを読み解いた人(修正担当者)がいるにもかかわらず
別の人が、また大変な思いをして読み解いてレビューしなければいけない
そして、次同じところを修正する時、また複雑なコード読み解かなければいけない
48仕様書無しさん
2018/04/16(月) 21:32:16.33 みんな実装するだけで精一杯なんよ
49仕様書無しさん
2018/04/16(月) 21:33:31.81 リファクタリングは神の領域に達するものにのみ許される技。
素人はおとなしくExcel方眼紙でセル結合の練習でもしておけ。
素人はおとなしくExcel方眼紙でセル結合の練習でもしておけ。
52仕様書無しさん
2018/04/16(月) 21:48:55.54 入門レベルの奴でも上から目線になれるから張り切ってんのか
56仕様書無しさん
2018/04/16(月) 22:21:54.62 リファクタリングしてるのでNG
58仕様書無しさん
2018/04/16(月) 23:48:14.82 テストしづらいとか、クラスが肥大化しすぎて開発に支障があるとかならまだしも
見通しが悪いなんて理由でうごいてるもんをほいほい変えていいもんじゃろか
見通しが悪いなんて理由でうごいてるもんをほいほい変えていいもんじゃろか
59仕様書無しさん
2018/04/17(火) 01:28:22.88 リファクタリングは超プロの領域
このスレには本当のリファクタリングが出来る奴はいない
elseなしに書き換えるのが精一杯のFラン説教野郎みたいなのがいるだけ
このスレには本当のリファクタリングが出来る奴はいない
elseなしに書き換えるのが精一杯のFラン説教野郎みたいなのがいるだけ
60仕様書無しさん
2018/04/17(火) 01:37:33.43 大規模リファクタリングの秘孔をついた
お前のプロジェクトはもう死んでいる
お前のプロジェクトはもう死んでいる
61仕様書無しさん
2018/04/17(火) 02:47:02.14 >>55
> リファクタリングするときのレビューとかどうしてる?
リファクタリングではないときはどうしてる?
それと同じなんだが。
まあそういう質問するってことはリファクタリングに限らず
そもそも設計もコードもレビューしてないってことなんだろうけど
まずはそこからやねw
> リファクタリングするときのレビューとかどうしてる?
リファクタリングではないときはどうしてる?
それと同じなんだが。
まあそういう質問するってことはリファクタリングに限らず
そもそも設計もコードもレビューしてないってことなんだろうけど
まずはそこからやねw
62仕様書無しさん
2018/04/17(火) 02:49:20.05 >>57
> 自動試験だけで完璧なものができるとか信じてる?
だからレビューするんでしょ?
自動試験はバグがないことを証明する手段
その手段が正しいかどうかはレビューでみる。
人手でレビューして、テストした内容も記録されて
後なにがあれば完璧なものができると思います?
> 自動試験だけで完璧なものができるとか信じてる?
だからレビューするんでしょ?
自動試験はバグがないことを証明する手段
その手段が正しいかどうかはレビューでみる。
人手でレビューして、テストした内容も記録されて
後なにがあれば完璧なものができると思います?
63仕様書無しさん
2018/04/17(火) 02:51:40.76 >>58
> テストしづらいとか、クラスが肥大化しすぎて開発に支障があるとかならまだしも
> 見通しが悪いなんて理由でうごいてるもんをほいほい変えていいもんじゃろか
機能追加とか修正するという前提なんだから、どちらにしろ変更するしかないでしょw
> テストしづらいとか、クラスが肥大化しすぎて開発に支障があるとかならまだしも
> 見通しが悪いなんて理由でうごいてるもんをほいほい変えていいもんじゃろか
機能追加とか修正するという前提なんだから、どちらにしろ変更するしかないでしょw
65仕様書無しさん
2018/04/17(火) 08:35:24.00 入門書第1章に書いてあることの受け売りなら長文で語るけど、具体的な話からは煽りで逃げるし、
体験から語ってると思えるような発言もないし、こいつ多分そんなにノウハウないよ
体験から語ってると思えるような発言もないし、こいつ多分そんなにノウハウないよ
66仕様書無しさん
2018/04/17(火) 10:12:11.71 >>64
レビューもするし自動以外のテストもする
そこは普通の機能変更のときとなにも変わらない
ただその時のやり方が、複雑なコードを頑張って解析して
無理やりパッチを当てるような、あとから見た時に
なんでこんな面倒なことしてんの?と思われるような変更をするやり方から、
次回は複雑なコードを読まなくて良いように、複雑なコードを
シンプルに修正しつつ、まるで0からコードを書いた時のように
こういう機能を実現するなら、普通こう書くよねという
当たり前の形にするやり方に変わるだけ
変更するたびに複雑になっていくやり方をやめて
変更しても複雑にならない、むしろシンプルになるような
やり方をしましょうってこと
その後にやるテストうんぬんは、もともと機能変更でやる予定だったんだろ?
なら最初からやるって決まってることじゃないか
レビューもするし自動以外のテストもする
そこは普通の機能変更のときとなにも変わらない
ただその時のやり方が、複雑なコードを頑張って解析して
無理やりパッチを当てるような、あとから見た時に
なんでこんな面倒なことしてんの?と思われるような変更をするやり方から、
次回は複雑なコードを読まなくて良いように、複雑なコードを
シンプルに修正しつつ、まるで0からコードを書いた時のように
こういう機能を実現するなら、普通こう書くよねという
当たり前の形にするやり方に変わるだけ
変更するたびに複雑になっていくやり方をやめて
変更しても複雑にならない、むしろシンプルになるような
やり方をしましょうってこと
その後にやるテストうんぬんは、もともと機能変更でやる予定だったんだろ?
なら最初からやるって決まってることじゃないか
68仕様書無しさん
2018/04/17(火) 10:13:52.53 レビューってさ
もう儀式になってない?
もう儀式になってない?
■ このスレッドは過去ログ倉庫に格納されています
ニュース
- 青森 緊急地震速報 [ぐれ★]
- 「偽サッチャー」「自滅的」「時代遅れ」 高市首相の経済政策を海外メディアが酷評 ★3 [蚤の市★]
- ミス・ユニバース フィンランド代表の「つり目」写真が波紋… 本人釈明も批判やまず 協会謝罪「徹底的に検証」へ★2 [冬月記者★]
- 【速報】衆院議員定数削減法案、自民・維新が今国会成立見送りで調整 [Hitzeschleier★]
- 東京都「都民の税金1.5兆円が国に奪われている」「全国に分配されている」に地方民ブチギレ ★2 [Hitzeschleier★]
- 高市首相の答弁書に「台湾有事答えない」と明記 存立危機発言当時 ★4 [蚤の市★]
- そんなことより自民党(献金追及より議員定数削減)、定数削減先送りへwwwww [577451214]
- 地蔵 [268244553]
- 日本、高市が辞任しても日中関係を改善させられそうな首相候補がいなくて詰む [329271814]
- 地震ある度に「気持ち悪い揺れだった」って言う奴いるよなw
- 「ごま油」👈何に使うんだこれ [394133584]
- お昼休みなので>>2のキャラをかいてあそぶ
