ﾘﾌｧｸﾀﾘﾝｸﾞすると全部ﾃｽﾄしろと言ってくる奴の矛盾3
いつもやってることじゃん
よくある応答のパターン
1. 必要もないのにリファクタリングを勝手にやるな
⇒ 客などから修正を要求するときに行うので、勝手にやることはありません
2. 修正の指示は出したがリファクタリングの指示は出してない
⇒ 修正工程の一つに含まれる作業なので修正と分離することは不可能です
3. リファクタリングすると動きが変わるだろ
⇒ 動きを変えないのがリファクタリングです
4. リファクタリングしたら全部テストしなきゃならないだろ
⇒ 修正しろと客などから言われたんだから修正するしか無い
どちらにしろ全部テストするんだろう？
前スレ
ﾘﾌｧｸﾀﾘﾝｸﾞすると全部ﾃｽﾄしろと言ってくる奴の矛盾2
https://medaka.2ch.sc/test/read.cgi/prog/1526720757/
共通ルーチン？
なんで分けて改修しないんですか？
テスト全部やり直してくださいよ
もう一回やったら訴えますよ
これが現実
震えて眠れ
> なんで分けて改修しないんですか？
分けて二箇所改修する意味は？
テストはした箇所は治っていました。ですが似たような箇所で同じような計算をしていて
そちらは治っていませんでした。軽く検査した所、同じような計算が10箇所ありましたので
その改修とテストに今回の10倍の時間と費用がかります。
片方は問題ねーから
それはね。あなたから開発費をせしめるためですよ
共通ルーチン使っていて、片方は問題ないなんてありえないよ
どういう場合にそうなるんだよｗ
違う処理であるべきなのに、同じ処理にしてしまったんだから
え？想像力がたりなさすぎない？
一つは精度が必要なときとかfloatで十分、doubleでないとダメ
条件が特定のルーチンしか通らないとか
いくらでもあるぜ
お前一つも浮かばなかったの？
プログラマやめちゃえよ
設計なんか語るレベルにないじゃん
修正内容が「精度を上げたい」なら共通関数を変更するべきだが、
「一箇所だけ精度を上げたい」ならば、そのときに分ければいいの
おかしいでしょ？精度を上げたい箇所を明確にしないまま作業するなんて
まず先に精度を上げたい場所がどこかが決まる。
そしてそれはリファクタリングではなくて単なる仕様変更
一箇所だけ精度をえるという仕様変更
その違いもわからないなら、プログラマやめたほうがいいよ
> 条件が特定のルーチンしか通らないとか
設計が破綻してる
func() { A処理 and B処理 } があって？
一つは、A処理 B処理 両方通って
もう一つが A処理 のみしか通らないなら
func 内部で条件フラグとかで分けるんじゃなくて
呼び出し側で、A処理を行うA() と B処理を行うB() を呼び出す
もう一つは、A処理を行うA() のみを呼び出すの
クソコードによくある。長い関数でグダグダやっていて
特定の場合だけ処理を分けたいから、フラグを追加してフラグみて
片方だけ実行とかね。ほんとコレ、クソコード
> 条件が特定のルーチンしか通らないとか
そもそも言ってることがおかしくない？
func() { Aルーチン and Bルーチン } があって条件が特定のルーチンしか通らないんでしょ？
片方は Aルーチン と Bルーチンの 両方を通る。もう片方はAルーチンのみを通るということだ
修正箇所がBルーチンなら片方は修正されるし、もう片方はそもそも通らないのだから関係ない話
だからAルーチンが修正箇所ということになるが、
「条件が特定のルーチンしか通らない」とはどういうことだ？
両方共、修正箇所のAルーチンを通ってるではないか？
AルーチンとかBルーチンのようなエリアができてる時点でクソ
この関数は上げましたよ？でも他の部分がfloatのままなので
他に渡すときにfloatに変換されますねぇ
他の部分を変更しろとは言われてないしーｗｗｗ
それな
これに尽きる
単にfloat版とdouble版の関数を作るだけだろ
例えばこんな感じに
https://linuxjm.osdn.jp/html/LDP_man-pages/man3/clog10.3.html
> double complex clog10(double complex z);
> float complex clog10f(float complex z);
> long double complex clog10l(long double complex z);
>
> 他の関数は同じ機能を持つ float 版と long double 版である。
何かを修正するのと、新しく精度違う版が必要になった（機能追加）のを
ごっちゃにするんじゃねーよ
じゃあ、そういう修正が入っても既存は一切いじらないって主張でいいんだよね？
まず「そういう修正」の中身を明らかにしようか？
floatをdoubleに変えるのは"仕様変更"である
リファクタリングは仕様を変えないものなのだから
仕様を変えると言っている以上、これはリファクタリングではない
リファクタリングなら当然既存を修正する
繰り返すが、既存を修正する
お前が言ってる「そういう修正」の中身は
「他の処理は変えずに一部分だけ精度を上げるという仕様変更のための修正」だ
リファクタリングもしたからバグったって？
いやいや、リファクタリングをは動きを変えないものだから
それでバグるわけ無いでしょ？
改修内容を間違えたんですよね。一部分だけ修正しろと言ったのに
全部修正したんですよね？リファクタリングのせいにしないでください
もう一回うそついたら訴えますよ
これが現実
震えて眠れ
まあ、人間である以上いじったらバグる可能性は残るんだよ
君は信用できない
って普通に言われるだろ
仕事はお前だけが回してるわけじゃない
小さい会社でばっかり組んでるとヤバイな
修正する以上、テストするだろ？
リファクタリングは修正に関する部分しかしないんだから
テストの工数はすでに計上済み
> まあ、人間である以上いじったらバグる可能性は残るんだよ
だから修正範囲のテストをするんですよね？
それとも修正をするなと？
客から言われてるのに？
え？だからお前のはいらんとこに手を広げて関係ない機能まで修正入るんだろ？
共通処理使ってるからって理由だけで
何度も言うが、リファクタリングは動きを変えないものなんだから
共通処理を変えた所で動きは変わらない
修正内容が共通処理部分なんだろ？
ならリファクタリングしなかったとしても共通処理を修正するだろ
ということでここからはリファクタリングとは関係ない話だぞ
なら共通処理を修正するなら、テストもするわけだ
共通処理をコピーしたとしても、テストの数は変わらない
俺はあたり前のことしか言ってないぞ
関係ない話ばっかりしてるんだもんなぁｗ
あきれるよ。
単に共通処理を修正（仕様変更）する時、
コピーして修正（仕様変更）するか
コピーせずに修正（仕様変更）するの話じゃんか
動き同じでも触ったらテストするんだよ
おｋ？
ちゃんと共通処理のテストをしたのか？
共通処理の中は、エリアが２つに分かれていて、
今回の修正で影響がある部分はテストしました
どういう意味だ？
今回のテストは共通処理の中のエリアBを通りますが、
他の部分からエリアBを通る部分はないはずです。
だから他の部分は影響がないはずです。
だから共通処理を修正しましたが、共通処理自体のテストはしてません
今回の修正内容に関する部分だけテストしました。
テストやり直しな
> 動き同じでも触ったらテストするんだよ
そこは今回の修正対象ですよね？
なら最初からテストするに決まってるじゃないですかｗ
動きが同じってどこまで？
メモリの使い方も同じなの？
いいや
お前のはやらなくていい範囲まで手を出してるよね？
やらなくて良い範囲は手を出しませんよ？
何いってんの？
今回の修正する範囲、テストする範囲しか
リファクタリングしませんｗｗｗ
まさか、共通処理を修正しているくせに
共通処理の一部分は通らないはずだから
テストしなくていいとか言ってんの？ｗｗｗ
> メモリの使い方も同じなの？
メモリの使い方が仕様に明記されてるならその通りでしょうね
（明記されてないなら何が正しいかも決められないわけで、
正しいことをテストできない）
だから俺はしない
分離しちゃう
リファクタリングなんてソースの修正範囲を広げる愚行などもってのほかだ
動作は変えないんじゃなかったのか？
> だから俺はしない
> 分離しちゃう
だからリファクタリングの話関係ないじゃん
バグ修正の場合も分離するの？
あちこちでバグ起きてるんだけど？
> 動作は変えないんじゃなかったのか？
動きを変えないということは、
ソースコードをコピーしたりしてもだめだっていうのはわかってる？
× 動きを変えないということは、
○ メモリの使い方を変えないということは
いかなる修正を行っても、メモリの使い方は
変わるから・・・お前すべてのテストをしろって言いたいの？
リファクタリングとは関係ない話だけど
ソースコードコピーしたら、その分テスト工数が増えるじゃん
まさか関数否定派だったとはｗ
共通処理禁止、コードをコピペして使え
バグが見つかったら、コピペした物すべて
探し出して修正しろ。そして全てテストしろ
リファクタリングなんてまともな職場なら許可なんて下りるわけない
やってないところはCOBOLに関係がある分野ぐらいか・・・
まあ、あそこは文化が違うからな
「修正」の中に含まれてるもんだからね
は？
理由もないのに内部の造りまで変えたソースなんて品証通るわけねーだろ
コードレビューナメてるのか？
テストしてるから通るよ
ってか、なんのためにテストしてるの？
解読する作業なんです。
チームの仲間が暗号コードを作成し
別の人が暗号コードを解読する。
それがコードレビューなのです。
そういう不毛なことをして喜んでいるんです。
通る？
なぜ？
テメーの不必要な変更で増えた分のお客の受け入れテストの金と時間はどこから出るの？
そのままおっさんになってるようなのは痛い
だすもなにも、逆にコスト減るからなぁ
クソ汚いコードを何日もかけてコードレビューするのにくらべたら
綺麗なコードはほんの数分で終わってしまうこともざらにある
このコードが正しいかどうか見てください！って言われても
コード汚いとそいつが作業した時間以上にかかるんだよ
コードレビューやるのにどれだけ時間がかかると思うのか
自由な社風って素敵やん
テストはいまあるテストコードをそのまま流せばいいだけ
だからリファクタリングのコードレビューは、
単に自動テストが変更されていないかを確かめるだけでよい
必要なのは本質的な仕様変更に対する修正
この修正がリファクタリングされてないと複雑な修正になってしまう
リファクタリング後だと簡単な修正で済む
コードレビューも楽になる
仕事を任せられるっていうのはそういうことやで
許可無いと仕事させてもらえないなら、
お前はその程度ってことだ
煽りたくて必死って感じかな？
そういう奴にはコレを言ってやれ
お前はその程度ってことだ
使い捨てにされるってのは所詮そこまでの奴
俺ならそいつにこう言ってやるね
お前はその程度ってことだ
草ばっか生やしちゃってさ・・・
がっかりだよ
言い返せることがあるなら言い返せばいいんだぞ
話ならいくらでも続けてやる
仕様変更というリファクタリングとは関係ない話に
すり替えようとしたのは何故だ？
そしてテストする箇所が増えるのに
コードをコピーするのは何故だ？
無駄に改行
とにかく草
これがﾘﾌｧｸﾀ厨
そこで働いているのはひたすら奴隷よ
替えのきく奴隷
話すりかえてすぐ逃げる
リファクタリングなどの技術力って基本的にポータブルなスキルだから、会社がどうこうは関係ないよね。
>>69
意味もなく煽る
お前のことだよ
極端すぎるからお前の意見なんてまともなやつなら同調しねーよｗ
意味がないに同調してればいいんだよ
あんなのは労働基準も社内ルールも全く違う国の夢物語
いまだ縦社会の日本じゃ到底実現不可なのが現状だよ
大手ゼネコンの多重受けみたいな場合でもなければ融通は効くでしょう
環境のせいではなく不勉強で過去の踏襲しかしないタイプのリーダーしか居ないことが本当の問題
彼らは比較的自由にできるチャンスがある案件でも日本風のやり方しかできない
絶対に体外的な仕様に影響がでる。
この事実を理解しいいないバカがリファクタリングすると大変なことになる。
> 絶対に体外的な仕様に影響がでる。
でるわけないじゃんｗ
リファクタリングで使用するテクニックは
動きが変わらないと保証されてるものばかり
数学の式の変形とやってることは変わらん
いじったとこの影響範囲全部
基本的には元々あるテストを流すだけ
ベンチマーク追加して、リファクタリングの有効性を示すことはあるけど
元々あるテストを流すだけってなんや？
何十人も動員して、エクセルに書いたテスト仕様書の
通りに実行してテストするんやで
テスト仕様書が曖昧で、毎回これどういう意味ですか？って聞かれるは、
人によってやり方が微妙に違うわで、正しくテストしているか不安だから
同じテストを何人も使って行うんやで
どれだけ金がかかってると思う？
コストがかかるから一年に数回しかできない
その数回で確実なテストを行うんだ。
これが真面目な仕事ぶりっていうもんや
根性見せろや
だよね
明らかに作るより金かけてるもんね
テスト流すだけって頭おかしいわ
はいキチガイ
社会に出ろよボーヤ
え、ポチッとボタン押すだけのことがそんなに難しいの？
それ単体だけだろ？
結合とシステムテストは？
ｗｗｗ
ニートおつ
どうやら30年前の世界に生きてるやつがいるようだ
50代スレに戻ってどうぞ
> 結合とシステムテストは？
どういう意味？
結合とシステムテストでバグが一つでも見つかったら、
修正しなきゃいけないってことで、
修正するならば、全部テストやり直しになるので
結合とシステムテストでは原則バグは見つからないという前提
それでもバグは見つかるので、
全部テストするコスト×数回分がテスト全体のコストになる
どれだけ高額になるかわかるだろう？
全部テストっていうのはそれだけのことを年数回やっている
信頼性も高いテストになりますけど？
じゃあなんで動いてるコードいじったの？バカ？
100〜のスレッドの続きを読む