- 1 名前：デフォルトの名無しさん mailto:sage [2010/05/29(土) 17:25:56 .net]
- テストも書かないでリファクタリングとかうけるｗ
まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。
書き換えるといっても、これなら同じ動きだろ？って推測でやってはいけない。
まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。
汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う
そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。
この手順と考え方を守ってないのはリファクタリングではない。
で？
- 660 名前：デフォルトの名無しさん mailto:sage [2014/11/28(金) 12:44:59.78 ID:X4M4SbHd.net]
- >>659
それは再利用じゃないよ
機能を小さく保つ事で、本来の目的である一次利用出来る範囲が広がるだけだ
それを便宜的に「再利用しやすい」などと表現してる訳だ
ソフトウェア本来の目的とは異なる本当の意味での再利用にはコストがかかるんだよ
- 661 名前：デフォルトの名無しさん mailto:sage [2014/11/28(金) 17:41:36.95 ID:03S1h8mH.net]
- 本当の意味での再利用？
そんなもんに興味持つ必要はなさそうだが？
>>659
それな。
本当にそれ。
- 662 名前：デフォルトの名無しさん mailto:sage [2014/11/29(土) 01:59:23.82 ID:TbFyYdBX.net]
- >>660
> 機能を小さく保つ事で、本来の目的である一次利用出来る範囲が広がるだけだ
一次利用が出来る範囲が広がる＝再利用だろｗ
- 663 名前：デフォルトの名無しさん mailto:sage [2014/11/29(土) 13:13:20.77 ID:U5ivpV2C.net]
- 再利用じゃなくて流用と言えばいいのに
- 664 名前：デフォルトの名無しさん mailto:sage [2014/11/29(土) 14:17:24.97 ID:v2v5Wnkr.net]
- 全く修正しないで使うのが再利用。
コピーして修正して使うのが流用
流用をすると、AとA'というコードが生まれなんで二つに分かれてるの？
違いはなんんあの？と結局両方を読まないといけなくなる。
流用するたびに読むべきコードがどんどん増えていく。
はてに一つを修正するともう片方の修正を忘れるとか。
なので全く修正しないで使う「再利用」でないと意味が無い。
全く修正しないで使えるのだからコピーする必要はなくなる
- 665 名前：デフォルトの名無しさん mailto:sage [2014/11/29(土) 14:46:23.56 ID:A4nuaoXO.net]
- 流用元から流用先がどこにどれだけあるかは基本的に分からない。
流用元に何かバグがあって修正する必要があった場合に論理的に考えると
流用先も直しておいた方がいいor直す必要がある。 ただ流用先を全て特定
することは難しく、特に流用先で動くように変数名とか変えてあると確実に
漏れる。バグじゃなくても何か修正追加したいと思った時の事を考えると
コピペをばらまくより関数で再利用するほうがいいよねってなる。
ただ人間は欲深いもので1つの関数にあれもこれも詰め込んで破綻させる
奴が出てきたりする。こういうのを見て、関数なんて作らなくてコピペでいい
だろって奴が出てきて無限ループする。
- 666 名前：デフォルトの名無しさん mailto:sage [2014/11/29(土) 22:16:12.25 ID:UEas1JmU.net]
- 頑張って同じ機能がすでにあるか探して、
あったらあまりいいコードじゃなくても正しく仕様を満たすからそれ使って
散々気をつけてんのに誰かが変更しやがる
- 667 名前：デフォルトの名無しさん mailto:sage [2014/11/30(日) 12:24:33.02 ID:NxMP0P1A.net]
- リファクタリングと言う名の正義
- 668 名前：デフォルトの名無しさん mailto:sage [2014/12/05(金) 19:01:54.53 ID:cszTTiF/.net]
- >>667
同意､正義を振りかざす曲者
- 669 名前：デフォルトの名無しさん mailto:sage [2014/12/21(日) 11:54:37.04 ID:LOCxTeYe.net]
- >>660,660
>ソフトウェア本来の目的とは異なる本当の意味での再利用にはコストがかかるんだよ
アプリ内でのモジュールの共通化（＝アプリに特化したフレームワーク、ライブラリ
の抽出）と、どんなプロジェクトでも使えるようにフレームワークやライブラリーを
汎用化するのは似て非なるものだと思う。
- 670 名前：デフォルトの名無しさん mailto:sage [2015/02/15(日) 22:12:39.01 ID:wyXhgSeE.net]
- >>1-1000
テスト
- 671 名前：デフォルトの名無しさん mailto:sage [2016/03/27(日) 21:55:32.43 ID:N7IGtcj3.net]
- どのタイミングでやったら工数に影響を与えずにできるの?
|
 |
