まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。
書き換えるといっても、これなら同じ動きだろ？って推測でやってはいけない。
まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。
汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う
そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。
この手順と考え方を守ってないのはリファクタリングではない。
で？
プロセスだ。やり方だ。
コードがきれいになったからリファクタリングをしたと
思うのは間違いだ。
バグを入れないのは当たり前だが、コードの動作を変えずに
決められたた手順をもちい、クラス間の依存関係を減らすことで
ユニットテストできるようにする作業がリファクタリングだ。
テストコードが増えてなければ、それはコードを書き直しただけで
リファクタリングとは呼ばない。
リファクタリングブラウザとは、このリファクタリング作業を簡単にするための
ツールであり、リファクタリングブラウザの機能を使った作業（名前変更など）そのものは
リファクタリングではない。
新たな設計方針にたどり着ける。
テストしやすい設計。
クラスを単独で生成できるように
依存関係（クラスが別のクラスを使用するなど）が
少ない設計。
publicメソッドの引数には、なるべくクラスを使わない。
クラス内部で別のクラスを生成しない（必要な場合は外部から渡す）
全部main(String [] arg)で作るんですねわかりま…
こういう考えの流れになっていないとだめ
リファクタリングしたい
↓
リファクタリングするにはユニットテストコードが必要
↓
既存のコードを（なるべく）修正せずにテストするにはどうするか。
↓
（依存関係が無ければ簡単にテストコードを書けるが、たいていはそのままではテストコードかけないので）
既存のコード・クラスをどうやって他のコード・クラスとの依存関係から分離させるか。
この分離が一番大変な作業。
分離させることに成功したらあとは簡単。
ユニットテストを書いてコードを修正すればよい。
名前変更（リファクタリングブラウザの機能）というのは、
例えて言うのなら小説を書く（リファクタリング）ときの文房具（リファクタリングブラウザ）と
同じ関係だよ。
文房具を使って文字を書いたり、消しゴムで消したりという作業は、
小説を書くための道具を使っているわけだが、
本質的には小説を書く行為そのものではない。
その通りです。
何か嫌がらせされてる気がする。
ソフトウェアテストで良い本ありませんか。
レガシーコード改善ガイド
言語訓練のために立てたものです。
アイと研究員とのやり取りに利用するスレッドなので、
関係者以外は書きこまないで下さい。
京都大学霊長類研究所
だと思った
>>1の一行目で分かった
コーダの思いがこもってるんだからあ
釘宮風にたのむ
ありがとうございます。
本見てみます。
チルノのパーフェクト算数教室？
これは、コード修正じゃなくてりふぁくたりんぐなんだから！
ちゃんと勉強してから出直しなさい！
なによ……わざわざしてあげるんだから…何か言いなさいよ！
あっ、べ、別にアンタのためにやるんじゃないんだから、
勘違いしないでよね！ばーかばーか！
目的に即してれば、細かいことは違ったって問題ないのに、それは手順としてあーだーこーだ・・・みっともない。
遅レスですまん
ファウラーの「リファクタリング」は必読
「パターン指向リファクタリング入門」はオススメ
あと、関連してデザインパターン関連の本を読んでおくといいかも
そもそもリファクタリングするために必要なユニットテスト技術も未熟だからか。
日本は技術レベルが低いよ。
愚痴はここまでにして、データベースのフィールドってリファクタリングするのに
厄介だよね。ただの文字列だと名前変更に追尾できない。
それはO/Rマッパー使ってもそうだけど、データベース関連はリファクタリングを念頭に置くと、
SQLとデータベースからクラスとして自動生成したほうがいいのかもしれない。
このクラス（仮にDBCLASSとする)ではデータベースのフィールドが一メソッド（プロパティ）になる。
これはビジネスロジックを書くいわゆるモデルではなく、データベースの構造をあらわすだけのクラス。
DBCLASSのプロパティを変更しても、DBCLASS内部でアクセスしているデータベースのフィールド名は変わらない。
逆にアクセス先のデータベースのフィールド名を変えると、DBCLASS内部でアクセスする
データベースのフィールド名も自動的に変わる。
フォームのクエリー名にも当てはまるんだよな。
要はデータではなく、意味がある名前であるキーを
ただの文字列として扱うのはやめようということ。
フォームのクエリー名を変更したら、
自動的にコードの名前に反映させたい。（逆もあり）
そのためのコードの生成機能が必要だよ。
いまだにフォームの設定かいたら、その部分のコードが自動生成されないの？
みんなふつうにやってるとおもったんだが気のせいであったか。
何の話？
自動生成されたとしても、それを文字列で扱っていたらだめだよ。
ようはリファクタリングブラウザで、それがキーとして認識されるような形になってほしい。
リファクタリングだけじゃなくて、コード補完もできるようになるから間違いが減るしね。
コンパイル時点でのエラー検出もおこなえるようになる。
いつも波風が立つのはなぜだろう
理解してないからだろ
こういう成功例を見せながら地道に説得していくしかないんじゃないの？
ttp://ascii.jp/elem/000/000/497/497905/
いい記事だった。
そこの社長ぐらい、うちんとこの社長も柔軟な頭だったらいいんだがなあ
例えば平屋を拡張して2階、3階と増やしていったら、
1階にも手に入れないと、いつか倒壊しますよね?とか。
やってみたら意外と倒壊しないんじゃないの？、とか
だから目先のコスト増に怯えて将来の飛躍の芽を摘んでしまう。
理解のない人にはリファクタリングしているとはあえて言わないのも手だと
Ruby版リファクタ本に書いてあって目から鱗だったわ
どうもそいつはリファクタリングしている=暇そうにしていると思っているっぽい。
http://www.infoq.com/jp/news/2010/06/decision-to-refactor
>リファクタリングしている=暇そうにしていると思っているっぽい。
正解だから困る
組み込み向け「軽量Ruby」と「Rubyチップ」、福岡県が経産省の事業で開発へ
ttp://itpro.nikkeibp.co.jp/article/NEWS/20100628/349693/?ST=oss
的外れな提示だったらすまそ。
組み込み機器のほうの話なのか
いちおうLuaはヤマハのルータなんかにも使われているぽい。
http://www.infoq.com/jp/news/2010/09/large-scale-refactoring
Ａ〜Ｚまであるクラスの共通処理をＰクラスとして纏めたとするわな
したら、ＰクラスをいじるたびにＡ〜Ｚまでの影響範囲を調べなあかんのやで
仮にＡにしか関係しないＰに入ってしまった共通処理を弄るのだって
一度結合したらはがしてＡクラス特有の処理としてＰクラスからＡクラスを避ける処理なんて
結構、邪魔な処理になるで
それでもＰとしてまとめてしまったらＡ〜Ｚまでサポートしていかなあかんのやで
リファクタリング、結構、だが、誰がこの未来を予測できる？
Ａ〜Ｚまでの共通処理が走ってるＰクラスを蹴飛ばしてＰＡクラスを新しく作って
Ａ−ＰＡ、Ｂ〜Ｚ−Ｐの関係を新しくぶった切るような切込みはできんやろ？
せいぜいＰの処理にcase Aを入れるような修正が精一杯やないか？
今後を考えてリファクタリングなんてする意味があるんか？
自動テストが伴わないものはリファクタリングとは呼ばない。
いや、上で書いたのはあくまで例だよ
上の例は処理をまとめることによって明らかに構造が汚くなってるよね？
この先もまとめたＰクラスにＡ〜Ｚへのどれかの追加が入るたびに
case B、case C・・・と追加されていくだろうね
そのときＰクラスを消滅する選択肢は誰か選べるんだろか？
こーゆーのってさ、その時点ででは確定しなくね？
じゃ、リファクタリングってなんのためにするの？
って根本的なところに目を向けたときに次の開発をしやすくするためでしょ？
したら、その仕様書もなしでリファクタリングをするってことは
仕様書もないのにプログラムの構造を「俺が好き」なだけの構造にただ意味もなく
書き換えてるだけのマスターベーションなんだよ
case B、case Cとなるようなまとめ方はアフォがやるものだろ。
それはリファクタリング以前の問題。
ほうほう、それじゃ上記の例でＰの共通部分にＡのみに変更がかかったらどう修正する気？
Aが自前で実装すればいいんじゃね
それか、Pが取る引数を工夫して柔軟な対応ができるようにする
Aが自前で実装するように戻す。
あとそういうのは例えばStrategyパターンを使うべきで、共通化するものじゃない。
それすら分からないなら幼房。
>Aが自前で実装
だが、現実は絶対にcase Aを入れる
自分のソース見てみろ
今回は俺が逃げ道塞いだから自前で実装って言っただけだろ？ｗ
正気に戻ればＰクラスなんてつくらねーんだけどな
ガキみたいなこと言うなよ池沼
ほかに影響に与えずに拡張なんていくらでもやりようがあると思うよ
いや、その場面でcaseとか、既にそういう実装だった場合を除いてしないと思うが…
違うな
大抵の場合、外部とのやりとりをＡ〜Ｚまでまとめない一心でＰクラスで作ってしまうから
Ａだけ切り離してもインターフェースがＰを通しているのでＰにＡへの変更を吸収してもらうほかないって感じになってるはず
じゃないとまとめたうまみないしねｗ
おまえは統失か。
1+1は？と聞かれてみんなが2と答えたら、田んぼの田だやーい間違った間違ったと
はしゃぐ鼻水垂らしたガキにしか見えないんだが。
せめて反論しろよ(笑)
>変更を吸収してもらうほかないって感じになってるはず
想像と主観だけ。
でFA
自分が何が分かってないかすらわかってない
具体的な反論よろしく
>>49=>>68が具体的なコード出せ。そうしたら反論してやる。
まあ、コーダー未満の池沼だから出せないだろうが。
わかってないのは君だけじゃないの？
無駄だってわかりつつやるとお金もらえるからみんなやってるだけだよ
「なんのためにするリファクタリングなのか？」
って考えたらすぐに自分のやってることの無意味さに気がつくはず
現状の設計とマッチしてないから合わせるならわかるけど
それは開発段階で気がつくただの不具合だよね？
将来どうするのか？もわからないのにどうしてコードに手を入れられるの？
仕様書無しでコードに手をいれるのはダメだダメだあれほど言ってたのになんでこういうーは好きにやっちゃうの？
目的がわからないのに最適（？）なコードなんてわかるわけないじゃない
なんのためのリファクタリングなの？
すべてがおかしい、でもお金になるし偉い人もやってるからやる
ただ、それだけのもんだよこれは
やあ幼房。>>49を読んで意味が分かるのは池沼のおまえだけだ。
うっとおしいからやめろよ
変な言葉も聞きなれない横文字も使ってないし誰でもわかる内容だ
内容の意味がわからないのは経験が浅いからだろ
日本語でOK。
プログラム中で使用する文字列をすべてヘッダで#define切っちゃうのとかよくいる
使うその場に直接書いたほうがいいことに気がつくのはいつのことだろうか？
聞きかじりで行儀の良いとされる作りにしようとして
多重定義したり実際の文字列探すまでが長かったりする
くそみたいな所が多すぎるからそう感じるんだろ
根拠は？
意味がねぇからさ
ソースをみた瞬間に文字列が直接書いてあればその時点で意味がわかる
それを#defineきったら今度は命名した奴のセンスに丸投げすることになる
それが必ずしもわかりいい名前をつけてくれるとは限らない
また、複数個所でこの文字列を使うことがあったとして・・・いや・・・ないだろ？ｗ
あるならなにかおかしいんだｗ
文字列の場合、数値とはちょっと違う事情があってこういう定数を#defineを切るのはあまり意味ない
また、数値の場合も俺は#defineを切る意味がないと思ってる
こんなこというとすぐに頭に血を上らせてソースでいきなり出てきた数値の意味が〜云々で怒るやつがいるが
俺はそうじゃなくてソースを読んで出てきた数値がわかるようなソースを書くべきだと思ってる
ちゃんとインスタンスの生成箇所をしっかり制御すればそもそもプログラム全域に一つの値が
複数使いまわされることなどほとんどない（はず）
数値計算（3.1415・・・）ですら特定のファイルにまとまる
しかし、プロジェクトにグローバルインスタンスホルダー（大域変数セット）などあると俺の言ってることは全く理解不能だと思う
責務やカプセル化の話と、可読性とはイコールではないだろーに。
自分の説の論理的な飛躍に気づけないのか？
「ちゃんとインスタンスの制御をしっかりすれば複数箇所に出てくることはほとんどない」
まともな名前すら付けられん人間が「しっかりとしたインスタンスの制御」なんてやると思うかい？
インスタンスの管理なんてやる必要があるのはグローバル変数使ってるからだろ？
だとすれば
＞ちゃんとインスタンスの生成箇所をしっかり制御すればそもそもプログラム全域に一つの値が
＞複数使いまわされることなどほとんどない（はず）
ってのはグローバル変数使ってる前提なのか
なんでそうなるの？
いままでグローバル変数にばっかたよってるからそういう脳みそしかないんでしょ？
管理が要らないのか要るのか、どっちなのさ
それとグローバル変数がどうして関係あると思ったの？
それは>>82に言ってよ
なにが「だとすれば」なの？
馬鹿だからグローバル使うしか方法わからないだけじゃない？
いままでプログラム組んでてこの程度ってすごくみじめ
何故にインスタンスの制御とかの話になったの？
要らないんでしょ？
グローバル変数使わなければね
俺の言ってることなにか矛盾してる？
「何故」と理由を聞いてるのだが
あんまり気にしなくていいよ
「ちゃんとインスタンスの生成箇所をしっかり制御すれば」＝「グローバル変数なんて使うほどお馬鹿じゃなければ」
って意味だからｗ
ちょっと待て、たかがグローバル変数を使わないだけのことを
「ちゃんとインスタンスの生成箇所をしっかり制御」
なんて仰々しく書いてたのか
仰々しく書いたつもりはないけど
君にはできないことだよねｗ
何故そう思った？
たとえば、多角形を表示するプログラムがあったとする。
それは実際は三角形を表示するプログラムを反復実行してるとする。
三角形を表示するプログラムは、線を表示するプログラムを反復実行してるとする。
線を表示するプログラムは、点を表示するプログラムを反復実行してるとする。
すると、
点を表示するプログラムを、高速に書き換えれば、
これに依存した全てのプログラムが、その内容を変更しなくとも、高速化する。
ってのがリファクタリング。
これのポイントは、修正したのは点を表示するプログラムだけで、それ以外は一切変更していないのに、他も高速化できること。
このためには、インターフェースを極力保ったままで、中身だけを変更することが重要になる。
これがリファクタリングと、普通のコード修正との違い。
TOP カテ一覧 スレ一覧 2ch元 削除依頼 ▲
Pythonについて（アンチ専用） (781)
Microsoft Silverlight その9 (239)
CLDC+MIDP+携帯電話用Javaスレッド part 9 (919)
C言語で２ｃｈに書き込みたいのですが (204)
オブジェクト指向は本当に正しいのか？ (234)
OpenGLスレ Part17 (925)
--log9.info------------------
コミPo!スレ＠同人版３ (351)
【青田】175専用スレ【買い】52匹目 (297)
【アザゼル・サブマス】柴犬【牛歩レポ漫画】 (413)
【OH MY GOD!!】模造クリスタル 総合スレ27 (351)
【絵柄乗っ取り】クロヤギ出版・午前4時/4415★3【アンソロゴロ】 (933)
【犯人逃亡】高天原 傷害事件注意喚起スレパート3【木刀殴打】 (544)
少女病 Part4 (897)
【winny疑惑】土山にうのデスクトップ45【AtaraxiA】 (791)
【TB・下手・再生他】TEC・中村トウヤ【トレス常習】6 (829)
絡みスレ423＠同人板 (533)
デジ同人ピコ29 (553)
【くらえ！】逆転15【異議あり！】 (833)
【要注意】LEGO!/中川わか/ぱんだきの子/ぱぷこ【歌王子】 (253)
忍法帖テストスレその5 (481)
【公式トレパク】季刊SS絵師ばびりぃ★2【ポケモン・ポップン】 (315)
ロックマン同人総合【雑談】 (168)
--log55.com------------------
ハトホルの書 新・ハトホルの書
スピチュアルな音楽と動画を貼っていくスレ♪
引き寄せの法則について考えていたら論理的思考を極めていた
雑談スレッド in Spiritual板
チャネリングスレ
【スピリチュアルｗｗ】もうすぐ自Rるけど【オーラｗｗｗ】
【願望達成】塩祈願
【スピリチュアル】江原まさひろスレ【詐欺師】
