テストも書かないでリファクタリングとかうけるw
まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。
書き換えるといっても、これなら同じ動きだろ?って推測でやってはいけない。
まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。
汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う
そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。
この手順と考え方を守ってないのはリファクタリングではない。
で?
探検
リファクタリングをただのコード修正と思ってる人へ
■ このスレッドは過去ログ倉庫に格納されています
2010/05/29(土) 17:25:56
825デフォルトの名無しさん
2017/02/10(金) 22:14:40.62ID:/WxwB06L 駄目だこりゃ。なんでそういう手順がリファクタリングなのか?
826デフォルトの名無しさん
2017/02/10(金) 22:20:09.64ID:+KYQgfiL >>825
お前が理解してないだけだよw
お前が理解してないだけだよw
827デフォルトの名無しさん
2017/02/10(金) 22:33:35.95ID:+HewTgrG Refactorとリファクタリングが同じ意味の言葉だってのが伝わってないのかも知れないな
828デフォルトの名無しさん
2017/02/10(金) 22:38:11.34ID:+KYQgfiL あ、はい間違えました
> 6.リファクタリングする
これじゃ リファクタリンギング ですねw
> 6.リファクタリングする
これじゃ リファクタリンギング ですねw
829デフォルトの名無しさん
2017/02/10(金) 22:59:58.18ID:/WxwB06L このスレってそもそも特定のリファクタリング作業のスレッドか。話がかみ合わないわけだ。
830デフォルトの名無しさん
2017/02/10(金) 23:48:55.17ID:+KYQgfiL 誰が特定のリファクタリング作業の話をしてるんだ?
誰がはどうでもいいや「特定の」ってどれのことだよ?
誰がはどうでもいいや「特定の」ってどれのことだよ?
831デフォルトの名無しさん
2017/02/11(土) 00:00:12.78ID:o1zrWG0U TDDの例出したからTDDに限定した話だって勘違いしてるんでしょ
832デフォルトの名無しさん
2017/02/11(土) 00:04:14.10ID:p/3UeWk3 >>829
「特定の」がなにか言うだけだよ。
お前がちゃんと考えて発言しているなら
簡単な質問なはずだが?
もっとも俺はお前が何も考えてないってことを
あぶり出すために「特定の」が何か聞いたんだけどなw
「特定の」がなにか言うだけだよ。
お前がちゃんと考えて発言しているなら
簡単な質問なはずだが?
もっとも俺はお前が何も考えてないってことを
あぶり出すために「特定の」が何か聞いたんだけどなw
833デフォルトの名無しさん
2017/02/21(火) 23:44:51.93ID:8I0Tfvzv お、あぶり出してるねえ
834デフォルトの名無しさん
2017/02/22(水) 01:09:25.60ID:TiP/fttU このスレの存在忘れてたわw
なんだ逃げたのかw
なんだ逃げたのかw
835デフォルトの名無しさん
2017/02/22(水) 12:59:43.56ID:zJ9IFSdf >>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/
リファクタリングの誤用
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/
836デフォルトの名無しさん
2017/02/23(木) 18:03:32.56ID:G3lxPXWh 定義から定まってないからなリファクタリングって
いや、実在するのか?
いや、実在するのか?
837デフォルトの名無しさん
2017/02/23(木) 22:07:14.34ID:Ka1UMSVA 殆どの用語は定義なんか定まってないよ
数学でさえ、ある用語に対して数学ではこういう定義で
使いましょうと決めているだけ
数学でさえ、ある用語に対して数学ではこういう定義で
使いましょうと決めているだけ
838デフォルトの名無しさん
2017/02/26(日) 20:54:48.09ID:wNjUkQs3 ソースを組んだときと今とでチンコのポジションを若干変更した
これがリファクタリングである
これがリファクタリングである
839デフォルトの名無しさん
2017/02/26(日) 22:44:41.65ID:zOBszuQK 特定の統合開発環境のリファクタリング機能をリファクタリングだと言ってるやつはいなくなったなw
840デフォルトの名無しさん
2017/02/27(月) 01:14:12.49ID:IXzsv4Rb 俺の中では変数名の変更=リファクタリング
Visual Studioの機能名がそうだったから
Visual Studioの機能名がそうだったから
841デフォルトの名無しさん
2017/02/27(月) 01:49:20.76ID:j4xHFZFw 俺の中では、マーチン ファウラーのリファクタリング本(古い方)に
のっているのがリファクタリング
変数名の変更ももちろんのってる
後はそのリファクタリングをどれだけ簡単に
正確に行えるかの違い。
ローカルスコープ程度で済むものなら良いけど
スコープが広い部分のリファクタリングは手動でやるのは大変
それを自動的に間違いなく行える、静的型付け言語+IDEの力は偉大
のっているのがリファクタリング
変数名の変更ももちろんのってる
後はそのリファクタリングをどれだけ簡単に
正確に行えるかの違い。
ローカルスコープ程度で済むものなら良いけど
スコープが広い部分のリファクタリングは手動でやるのは大変
それを自動的に間違いなく行える、静的型付け言語+IDEの力は偉大
842デフォルトの名無しさん
2017/02/27(月) 02:49:09.61ID:Ydy+ZWkb あるスコープの中で外部から見た仕様を変えずに内部の設計を変えるのがリファクタリング
843デフォルトの名無しさん
2017/02/27(月) 17:22:25.25ID:IXzsv4Rb >>842
それではチンコのポジションも
それではチンコのポジションも
844デフォルトの名無しさん
2017/02/27(月) 22:10:17.46ID:Ydy+ZWkb チンポジ設計
845デフォルトの名無しさん
2018/05/23(水) 23:04:38.11ID:Au5e7VGg 僕の知り合いの知り合いができたパソコン一台でお金持ちになれるやり方
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』
K2FKH
役に立つかもしれません
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』
K2FKH
846デフォルトの名無しさん
2018/07/04(水) 23:04:33.21ID:gFgZc5FG GKH
■ このスレッドは過去ログ倉庫に格納されています
ニュース
- ネット殺到「高市総理の責任」「完全に高市リスク」「負けるな」中国が水産物輸入停止→流石に総理批判の声も「どう責任取る?」 ★4 [樽悶★]
- 「“なり得る”って言っただけだから…」高市早苗“存立危機”答弁後に漏らした本音 ★3 [Hitzeschleier★]
- 歩道で93歳男性が女子大学生の自転車にはねられ意識不明 坂を下った先「気付いたときには目の前に」 [七波羅探題★]
- 中国外務省局長 「ポケットに手を入れていたのは寒いから」 日本との局長級会談で ★4 [お断り★]
- 中国が水産物の輸入停止、首相答弁撤回を要求…中国共産党機関紙「輸入停止は一つのシグナルにすぎない」 [ぐれ★]
- 【テレビ】佐野岳 TBS『最強スポーツ男子頂上決戦』収録で重傷… 右膝半月板損傷と靭帯断裂 全治8カ月~9カ月間 [冬月記者★]
- テレビ局各社が高市首相を一切批判せず中国批判を展開 安倍時代の報道完全復活 [633746646]
- この国の動物園、終わる… ゴリラ、キリン、ゾウ、人気動物が続々と消える… どこかに大きくてかっこいい大型獣いませんか? [452836546]
- 🍣にゃっはろ🌸~スシろ~🏡
- コメンテーター「山上がやった事はテロ、山上はテロリスト」「殺害せずSNSとかで発信するべだった」 [605029151]
- 海外大手メディア、高市が中国に宣戦布告したと次々報道し始める。どんどん外堀が埋められる [931948549]
- 中国人観光客のキャンセル率、ついに7割越えwwwwwwwwwwwwwwww [329329848]
