リファクタリングをただのコード修正と思ってる人へ

**デフォルトの名無しさん** · 2010/05/29(土) 17:25:56

テストも書かないでリファクタリングとかうけるｗ

まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。

書き換えるといっても、これなら同じ動きだろ？って推測でやってはいけない。

まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。

汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う

そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。

この手順と考え方を守ってないのはリファクタリングではない。

で？

**デフォルトの名無しさん** · 2017/02/09(木) 22:47:35.42

リファクタリングを行うタイミングを間違ってるんだよね

俺は何らかの機能修正とかでコードをいじるときに
関連するところを少しづつなおす
大抵が数行程度で10分もかからずに終わる

こまめに修正しないで、手遅れになってからやろうとするから
大規模になってしまって、別途工数が～とかいう話になるんだよ。

手遅れになってしまったものを修正する工数がないからと
手遅れなコードのままさらにコードを追加するから
さらに手遅れになっていくんだろうが

**デフォルトの名無しさん** · 2017/02/09(木) 22:59:39.83

>>806
多分解ってると思うけどその言い方は誤解を生むと思うぞ。
リファクタリングと機能追加は明確に分けるべきで、リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき

**デフォルトの名無しさん** · 2017/02/09(木) 23:07:28.39

>>805
他人の否定はしてるけど、自分では何も言ってないよ君

**デフォルトの名無しさん** · 2017/02/09(木) 23:07:33.91

リファクタリング=上腕二頭筋

**デフォルトの名無しさん** · 2017/02/09(木) 23:12:13.46

>>807
うん。わかってる

> リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき

どちらもあるよ。

機能追加してからリファクタリングする物の典型的な例がTDD
Red/Green/Refactor 通りの順番

**デフォルトの名無しさん** · 2017/02/10(金) 00:46:58.66

どんどん別の手法がまざっていく洗脳馬鹿

**デフォルトの名無しさん** · 2017/02/10(金) 01:18:18.70

別の手法ってなんの話だ？

**デフォルトの名無しさん** · 2017/02/10(金) 02:12:34.39

>>810 のことだよ。

**デフォルトの名無しさん** · 2017/02/10(金) 02:25:45.27

> Red/Green/Refactor 通りの順番

これのことか？

Refactorはリファクタリングのことだよ
別の手法じゃなくて全く同じもの

**デフォルトの名無しさん** · 2017/02/10(金) 12:06:41.01

1.機能追加前にテスト書く
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる

もしかしてこれもリファクタリング扱いしてるの？

**デフォルトの名無しさん** · 2017/02/10(金) 12:38:04.04

>>815
それはRed→Greenのところでしょ…

**デフォルトの名無しさん** · 2017/02/10(金) 21:26:25.38

　最近、特定のフラグ変数に関するif文の山をswitch構文に清書し直したんだけど、これってリファクタリング？

**デフォルトの名無しさん** · 2017/02/10(金) 21:34:25.32

>>816
それさ、リファクタリングのいち定義であってリファクタリングではないから。

**デフォルトの名無しさん** · 2017/02/10(金) 21:35:50.89

>>817
そのif文が変ならな。まともなif文なら書き換える必要がない。

**デフォルトの名無しさん** · 2017/02/10(金) 21:37:35.22

>>817
ただswitch文にすると仕様変更時にswitch文では対応できなくなる可能性があるので逆かもしれない。

**デフォルトの名無しさん** · 2017/02/10(金) 21:42:51.54

>>817
確かにswitchでかけるならばifよりswitchの方が少しだけ見やすくなるけど、
大抵はリファクタリングにはならないよ
巨大なswitchもリファクタリングすべき対象となる

何がダメなのかというと比較の回数がifのときと変わってないから。
比較が多い＝複雑なので、これは単なる書き方の違いでしかなくて
リファクタリングとまではいかない。重要なのは比較回数

じゃあどうすべきかというと、一番簡単な方法は
関数テーブル（ルックアップテーブル）を使う方法
大量のswitchでの比較が条件が揃えば0個にまで減る

**デフォルトの名無しさん** · 2017/02/10(金) 21:46:49.78

>>818
> それさ、リファクタリングのいち定義であって

> Red/Green/Refactor 通りの順番
がリファクタリングの定義に見えるんだ・・・

ここにはどこにもリファクタリングの定義なんてものは
書いてないんだが

>>816が言ってるのは、Red→Greenでやる内容は
リファクタリングじゃないよって言ってるだけでしょ

俺からも補足するが、リファクタリングは
動いているコードを、動いているコードに置き換えることだよ。

修正前と修正後のどちらかが動いていなければ
それはリファクタリングではない

**デフォルトの名無しさん** · 2017/02/10(金) 21:57:11.80

* Green
　　1.機能追加前にテスト書く
* Red
　　2.追加コード書く
　　3.テスト
　　4.ダメならテストに通るまでコードいじる
　　5.テストに通った。やったー完成だ！ ←ふざけんじゃないよ。なんだこのコピペだらけで無駄がばかりのクソコードは
* Green
　　6.リファクタリングする
　　7. 読みやすくなりました。←このコードならレビューする気になる。シンプルあとで読んだ人も理解しやすいだろう
　　8. 完成、1に戻る

**デフォルトの名無しさん** · 2017/02/10(金) 22:00:06.48

現実にはこの5. テストに通っただけの状態で
完成としてしまうやつが多いんだよな。

そういうコードで許される環境っていうのは
コードレビューが行われていない。
コードを見ずにテストに通ればOKとしてしまう環境

そして向かうはデスマーチｗ

**デフォルトの名無しさん** · 2017/02/10(金) 22:14:40.62

駄目だこりゃ。なんでそういう手順がリファクタリングなのか？

**デフォルトの名無しさん** · 2017/02/10(金) 22:20:09.64

>>825
お前が理解してないだけだよｗ

**デフォルトの名無しさん** · 2017/02/10(金) 22:33:35.95

Refactorとリファクタリングが同じ意味の言葉だってのが伝わってないのかも知れないな

**デフォルトの名無しさん** · 2017/02/10(金) 22:38:11.34

あ、はい間違えました

>　6.リファクタリングする

これじゃリファクタリンギングですねｗ

**デフォルトの名無しさん** · 2017/02/10(金) 22:59:58.18

このスレってそもそも特定のリファクタリング作業のスレッドか。話がかみ合わないわけだ。

**デフォルトの名無しさん** · 2017/02/10(金) 23:48:55.17

誰が特定のリファクタリング作業の話をしてるんだ？
誰がはどうでもいいや「特定の」ってどれのことだよ？

**デフォルトの名無しさん** · 2017/02/11(土) 00:00:12.78

TDDの例出したからTDDに限定した話だって勘違いしてるんでしょ

**デフォルトの名無しさん** · 2017/02/11(土) 00:04:14.10

>>829
「特定の」がなにか言うだけだよ。
お前がちゃんと考えて発言しているなら
簡単な質問なはずだが？

もっとも俺はお前が何も考えてないってことを
あぶり出すために「特定の」が何か聞いたんだけどなｗ

**デフォルトの名無しさん** · 2017/02/21(火) 23:44:51.93

お、あぶり出してるねえ

**デフォルトの名無しさん** · 2017/02/22(水) 01:09:25.60

このスレの存在忘れてたわｗ
なんだ逃げたのかｗ

**デフォルトの名無しさん** · 2017/02/22(水) 12:59:43.56

>>423 >>426 のやつNot Foundになってた

リファクタリングの誤用
https://bliki-ja.github.io/RefactoringMalapropism/

リファクタリングの境界線
[見当たらない]

インタフェースの変更はリファクタリングか
https://bliki-ja.github.io/IsChangingInterfacesRefactoring/

未知のバグフィックスはリファクタリングか？
https://bliki-ja.github.io/IsFixingAnUnknownBugRefactoring/

最適化はリファクタリングか？
https://bliki-ja.github.io/IsOptimizationRefactoring/

宣言の順序変更はリファクタリングか？
https://bliki-ja.github.io/IsDeclarationOrderingRefactoring/

**デフォルトの名無しさん** · 2017/02/23(木) 18:03:32.56

定義から定まってないからなリファクタリングって
いや、実在するのか？

**デフォルトの名無しさん** · 2017/02/23(木) 22:07:14.34

殆どの用語は定義なんか定まってないよ
数学でさえ、ある用語に対して数学ではこういう定義で
使いましょうと決めているだけ

**デフォルトの名無しさん** · 2017/02/26(日) 20:54:48.09

ソースを組んだときと今とでチンコのポジションを若干変更した
これがリファクタリングである

**デフォルトの名無しさん** · 2017/02/26(日) 22:44:41.65

特定の統合開発環境のリファクタリング機能をリファクタリングだと言ってるやつはいなくなったなw

**デフォルトの名無しさん** · 2017/02/27(月) 01:14:12.49

俺の中では変数名の変更=リファクタリング

Visual Studioの機能名がそうだったから

**デフォルトの名無しさん** · 2017/02/27(月) 01:49:20.76

俺の中では、マーチンファウラーのリファクタリング本（古い方）に
のっているのがリファクタリング
変数名の変更ももちろんのってる

後はそのリファクタリングをどれだけ簡単に
正確に行えるかの違い。

ローカルスコープ程度で済むものなら良いけど
スコープが広い部分のリファクタリングは手動でやるのは大変
それを自動的に間違いなく行える、静的型付け言語＋IDEの力は偉大

**デフォルトの名無しさん** · 2017/02/27(月) 02:49:09.61

あるスコープの中で外部から見た仕様を変えずに内部の設計を変えるのがリファクタリング

**デフォルトの名無しさん** · 2017/02/27(月) 17:22:25.25

>>842
それではチンコのポジションも

**デフォルトの名無しさん** · 2017/02/27(月) 22:10:17.46

チンポジ設計

**デフォルトの名無しさん** · 2018/05/23(水) 23:04:38.11

僕の知り合いの知り合いができたパソコン一台でお金持ちになれるやり方
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法　モニアレフヌノ』

K2FKH

**デフォルトの名無しさん** · 2018/07/04(水) 23:04:33.21

GKH