2ちゃんねる スマホ用 ■掲示板に戻る■ 全部 1- 最新50    

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

1 :デフォルトの名無しさん:2010/05/29(土) 17:25:56 .net
テストも書かないでリファクタリングとかうけるw

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

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

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

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

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

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

で?

796 :デフォルトの名無しさん:2017/02/09(木) 22:06:33.30 ID:Lyr2GEHi.net
>>785
おまえ日本人か?w

797 :デフォルトの名無しさん:2017/02/09(木) 22:08:07.52 ID:CNUBJX7I.net
下手に勘違いされると困るから補足しておくと、

汚いから修正するってだけならYAGNIでやるべきではないが、

その部分を修正していたり読む必要があるならば、
いつ必要しているの?今でしょ?ってことで
今すぐ必要としていることなわけだから、
そこを修正するのはYAGNIではない

798 :デフォルトの名無しさん:2017/02/09(木) 22:25:26.99 ID:lnTHGhne.net
読まなきゃ汚いと思わないんだから、矛盾してるような気がするけど。
必要ないのに汚いコードを読み始める酔狂な奴が居るとしたら別だけど。

799 :デフォルトの名無しさん:2017/02/09(木) 22:29:02.28 ID:CNUBJX7I.net
> 読まなきゃ汚いと思わないんだから
いや、コードメトリクスツールなどをつかえば
読まなくても、客観的に汚いとわかる

800 :デフォルトの名無しさん:2017/02/09(木) 22:29:15.76 ID:Ra4XvV1b.net
>>796
いつになったら何のことをリファクタリングと呼んでるか書いてくれるの?

801 :デフォルトの名無しさん:2017/02/09(木) 22:34:05.44 ID:Ra4XvV1b.net
汚いから直すってのもリファクタリングと呼ぶと思うけど、
汚いから直すってのは実際の開発現場じゃなかなか難しいと思うよ

その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね

802 :デフォルトの名無しさん:2017/02/09(木) 22:41:25.83 ID:CNUBJX7I.net
>>801
> その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね

だから俺は聞いたんだよ?

そういうどうしても直さなければいけないときに
リファクタリングで安全に修正するのか?
それとも行き当たりばったりに修正してバグを入れるのか?って

803 :デフォルトの名無しさん:2017/02/09(木) 22:43:53.46 ID:Ra4XvV1b.net
いや、俺に噛みつかれましても

804 :デフォルトの名無しさん:2017/02/09(木) 22:45:29.16 ID:Lyr2GEHi.net
>>800
理解できる頭がないのに質問するなよw
何度も言ってるだろうがww

805 :デフォルトの名無しさん:2017/02/09(木) 22:47:35.42 ID:CNUBJX7I.net
リファクタリングを行うタイミングを間違ってるんだよね

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


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

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

806 :デフォルトの名無しさん:2017/02/09(木) 22:59:39.83 ID:lnTHGhne.net
>>805
多分解ってると思うけどその言い方は誤解を生むと思うぞ。
リファクタリングと機能追加は明確に分けるべきで、リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき

807 :デフォルトの名無しさん:2017/02/09(木) 23:07:28.39 ID:Ra4XvV1b.net
>>804
他人の否定はしてるけど、自分では何も言ってないよ君

808 :デフォルトの名無しさん:2017/02/09(木) 23:07:33.91 ID:2NAFiD3Z.net
リファクタリング=上腕二頭筋

809 :デフォルトの名無しさん:2017/02/09(木) 23:12:13.46 ID:CNUBJX7I.net
>>806
うん。わかってる

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

どちらもあるよ。

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

810 :デフォルトの名無しさん:2017/02/10(金) 00:46:58.66 ID:/WxwB06L.net
どんどん別の手法がまざっていく洗脳馬鹿

811 :デフォルトの名無しさん:2017/02/10(金) 01:18:18.70 ID:+KYQgfiL.net
別の手法ってなんの話だ?

812 :デフォルトの名無しさん:2017/02/10(金) 02:12:34.39 ID:/WxwB06L.net
>>809 のことだよ。

813 :デフォルトの名無しさん:2017/02/10(金) 02:25:45.27 ID:+KYQgfiL.net
> Red/Green/Refactor 通りの順番

これのことか?

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

814 :デフォルトの名無しさん:2017/02/10(金) 12:06:41.01 ID:QRGeJWes.net
1.機能追加前にテスト書く
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる

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

815 :デフォルトの名無しさん:2017/02/10(金) 12:38:04.04 ID:+HewTgrG.net
>>814
それはRed→Greenのところでしょ…

816 :デフォルトの名無しさん:2017/02/10(金) 21:26:25.38 ID:1oBD+cZp.net
 最近、特定のフラグ変数に関するif文の山をswitch構文に清書し直したんだけど、これってリファクタリング?

817 :デフォルトの名無しさん:2017/02/10(金) 21:34:25.32 ID:/WxwB06L.net
>>815
それさ、リファクタリングのいち定義であってリファクタリングではないから。

818 :デフォルトの名無しさん:2017/02/10(金) 21:35:50.89 ID:/WxwB06L.net
>>816
そのif文が変ならな。まともなif文なら書き換える必要がない。

819 :デフォルトの名無しさん:2017/02/10(金) 21:37:35.22 ID:/WxwB06L.net
>>816
ただswitch文にすると仕様変更時にswitch文では対応できなくなる可能性があるので逆かもしれない。

820 :デフォルトの名無しさん:2017/02/10(金) 21:42:51.54 ID:+KYQgfiL.net
>>816
確かにswitchでかけるならばifよりswitchの方が少しだけ見やすくなるけど、
大抵はリファクタリングにはならないよ
巨大なswitchもリファクタリングすべき対象となる

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

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

821 :デフォルトの名無しさん:2017/02/10(金) 21:46:49.78 ID:+KYQgfiL.net
>>817
> それさ、リファクタリングのいち定義であって


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

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

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

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

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

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

823 :デフォルトの名無しさん:2017/02/10(金) 22:00:06.48 ID:+KYQgfiL.net
現実にはこの5. テストに通っただけの状態で
完成としてしまうやつが多いんだよな。

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

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

824 :デフォルトの名無しさん:2017/02/10(金) 22:14:40.62 ID:/WxwB06L.net
駄目だこりゃ。なんでそういう手順がリファクタリングなのか?

825 :デフォルトの名無しさん:2017/02/10(金) 22:20:09.64 ID:+KYQgfiL.net
>>824
お前が理解してないだけだよw

826 :デフォルトの名無しさん:2017/02/10(金) 22:33:35.95 ID:+HewTgrG.net
Refactorとリファクタリングが同じ意味の言葉だってのが伝わってないのかも知れないな

827 :デフォルトの名無しさん:2017/02/10(金) 22:38:11.34 ID:+KYQgfiL.net
あ、はい間違えました

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

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

828 :デフォルトの名無しさん:2017/02/10(金) 22:59:58.18 ID:/WxwB06L.net
このスレってそもそも特定のリファクタリング作業のスレッドか。話がかみ合わないわけだ。

829 :デフォルトの名無しさん:2017/02/10(金) 23:48:55.17 ID:+KYQgfiL.net
誰が特定のリファクタリング作業の話をしてるんだ?
誰がはどうでもいいや「特定の」ってどれのことだよ?

830 :デフォルトの名無しさん:2017/02/11(土) 00:00:12.78 ID:o1zrWG0U.net
TDDの例出したからTDDに限定した話だって勘違いしてるんでしょ

831 :デフォルトの名無しさん:2017/02/11(土) 00:04:14.10 ID:p/3UeWk3.net
>>828
「特定の」がなにか言うだけだよ。
お前がちゃんと考えて発言しているなら
簡単な質問なはずだが?

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

832 :デフォルトの名無しさん:2017/02/21(火) 23:44:51.93 ID:8I0Tfvzv.net
お、あぶり出してるねえ

833 :デフォルトの名無しさん:2017/02/22(水) 01:09:25.60 ID:TiP/fttU.net
このスレの存在忘れてたわw
なんだ逃げたのかw

834 :デフォルトの名無しさん:2017/02/22(水) 12:59:43.56 ID:zJ9IFSdf.net
>>422 >>425 のやつ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/

835 :デフォルトの名無しさん:2017/02/23(木) 18:03:32.56 ID:G3lxPXWh.net
定義から定まってないからなリファクタリングって
いや、実在するのか?

836 :デフォルトの名無しさん:2017/02/23(木) 22:07:14.34 ID:Ka1UMSVA.net
殆どの用語は定義なんか定まってないよ
数学でさえ、ある用語に対して数学ではこういう定義で
使いましょうと決めているだけ

837 :デフォルトの名無しさん:2017/02/26(日) 20:54:48.09 ID:wNjUkQs3.net
ソースを組んだときと今とでチンコのポジションを若干変更した
これがリファクタリングである

838 :デフォルトの名無しさん:2017/02/26(日) 22:44:41.65 ID:zOBszuQK.net
特定の統合開発環境のリファクタリング機能をリファクタリングだと言ってるやつはいなくなったなw

839 :デフォルトの名無しさん:2017/02/27(月) 01:14:12.49 ID:IXzsv4Rb.net
俺の中では変数名の変更=リファクタリング

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

840 :デフォルトの名無しさん:2017/02/27(月) 01:49:20.76 ID:j4xHFZFw.net
俺の中では、マーチン ファウラーのリファクタリング本(古い方)に
のっているのがリファクタリング
変数名の変更ももちろんのってる

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

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

841 :デフォルトの名無しさん:2017/02/27(月) 02:49:09.61 ID:Ydy+ZWkb.net
あるスコープの中で外部から見た仕様を変えずに内部の設計を変えるのがリファクタリング

842 :デフォルトの名無しさん:2017/02/27(月) 17:22:25.25 ID:IXzsv4Rb.net
>>841
それではチンコのポジションも

843 :デフォルトの名無しさん:2017/02/27(月) 22:10:17.46 ID:Ydy+ZWkb.net
チンポジ設計

844 :デフォルトの名無しさん:2018/05/23(水) 23:04:38.11 ID:Au5e7VGg.net
僕の知り合いの知り合いができたパソコン一台でお金持ちになれるやり方
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』

K2FKH

845 :デフォルトの名無しさん:2018/07/04(水) 23:04:33.21 ID:gFgZc5FG.net
GKH

252 KB
新着レスの表示

掲示板に戻る 全部 前100 次100 最新50
名前: E-mail (省略可) :

read.cgi ver 2014.07.20.01.SC 2014/07/20 D ★