テストも書かないでリファクタリングとかうけるw
まずな、リファクタリングでは機能追加・修正は行わない。
動作はまったく同じでコードをきれいに書き換えること。
書き換えるといっても、これなら同じ動きだろ?って推測でやってはいけない。
まずテストを書く。ユニットテストをできるように、
単一のクラスでインスタンスを作る。
汚いコードなのだからたいていは依存関係のせいで単一ではクラスが生成できない
それを生成するために、クラスの動作を書き換える。
といっても元のコードは修正しない。継承やプリプロセッサを使って
依存関係を断ち切るために既存のコードを上書きする。
どうしてもそれが不可能な場合には、決められた手順で最小のコード修正を行う
そうやって既存のクラスのユニットテストを行う。
それでやっとリファクタリングが行える。
既存のクラスのユニットテストを通るように新たなコードに修正、
もしくは新規作成して置き換える。
この手順と考え方を守ってないのはリファクタリングではない。
で?
リファクタリングをただのコード修正と思ってる人へ
■ このスレッドは過去ログ倉庫に格納されています
2010/05/29(土) 17:25:56
749デフォルトの名無しさん
2017/02/08(水) 22:05:08.20ID:EqksEKaR >>746
> リファクタリング前提の開発なんて小規模開発以外はやるものじゃないね
設計に問題があるときどうしてるの?
問題があるっていうのは、最初の時点で間違っていた場合の他
機能追加などで最初の想定と変わってしまった場合
リファクタリングしないで、問題ある設計のまま
開発を続行するの?
大きい開発ほど最初に想定しなかった事態が起こるはずだけど?
> リファクタリング前提の開発なんて小規模開発以外はやるものじゃないね
設計に問題があるときどうしてるの?
問題があるっていうのは、最初の時点で間違っていた場合の他
機能追加などで最初の想定と変わってしまった場合
リファクタリングしないで、問題ある設計のまま
開発を続行するの?
大きい開発ほど最初に想定しなかった事態が起こるはずだけど?
750デフォルトの名無しさん
2017/02/08(水) 22:42:04.56ID:E3hlYujb751デフォルトの名無しさん
2017/02/08(水) 22:59:10.86ID:EqksEKaR >>750
> ただの上流への出戻り、やり直し、糞開発じゃないの
上流へ出戻りするのは良いんだが、
今のコードはどうするんだよ?捨てるのか?
リファクタリングというのは今のコードをよく知られている
手順を使って安全に(バグを入れることなく)変化させていくことをいう
リファクタリングの勉強をちゃんとした人なら、知っているはずだが
リファクタリングしないことを選択すべき理由として
作り直したほうが速い場合っていうのがある。
作り直したほうが早い場合はリファクタリングの哲学によって作りなおすが、
コードを安全に変化させたほうが早い場合は当然リファクタリングをする。
で最初の質問に戻るが、お前コードを修正したほうが早い場合でも
リファクタリングしないで、コードを捨てて作り直すって言ってんの?
それとも行き当たりばったりなただの修正を行ってバグ発生させるの?
> ただの上流への出戻り、やり直し、糞開発じゃないの
上流へ出戻りするのは良いんだが、
今のコードはどうするんだよ?捨てるのか?
リファクタリングというのは今のコードをよく知られている
手順を使って安全に(バグを入れることなく)変化させていくことをいう
リファクタリングの勉強をちゃんとした人なら、知っているはずだが
リファクタリングしないことを選択すべき理由として
作り直したほうが速い場合っていうのがある。
作り直したほうが早い場合はリファクタリングの哲学によって作りなおすが、
コードを安全に変化させたほうが早い場合は当然リファクタリングをする。
で最初の質問に戻るが、お前コードを修正したほうが早い場合でも
リファクタリングしないで、コードを捨てて作り直すって言ってんの?
それとも行き当たりばったりなただの修正を行ってバグ発生させるの?
752デフォルトの名無しさん
2017/02/08(水) 23:15:25.00ID:/T6I+uKy 現状それなりに動いているけど
将来性を考慮しての改善するのがリファクタリングでしょ
そもそもマトモに動いてないコードは問題外
将来性を考慮しての改善するのがリファクタリングでしょ
そもそもマトモに動いてないコードは問題外
753デフォルトの名無しさん
2017/02/08(水) 23:18:57.65ID:EqksEKaR > 将来性を考慮しての改善するのがリファクタリングでしょ
YAGNIに反するからそれは違う。
その将来が絶対来るというのなら話は別だけど
YAGNIに反するからそれは違う。
その将来が絶対来るというのなら話は別だけど
754デフォルトの名無しさん
2017/02/08(水) 23:22:55.16ID:E3hlYujb >>751
うーんw
いずれにしてもコードを捨てる捨てないに関わらず設計に問題があるものをその開発中に直すことをおれの界隈ではリファクタリングとは言わないね
おまえがそれをリファクタリングと言うのは自由なので好きにしてくれw
うーんw
いずれにしてもコードを捨てる捨てないに関わらず設計に問題があるものをその開発中に直すことをおれの界隈ではリファクタリングとは言わないね
おまえがそれをリファクタリングと言うのは自由なので好きにしてくれw
755デフォルトの名無しさん
2017/02/08(水) 23:27:19.72ID:EqksEKaR >>754
俺が言ってるんじゃなくて世間一般の定義。
っていうか開発中とそうでないのが別れてる時点で
お前、前時代的だしなぁ。
お前なら一日に何十回もデプロイするような
ウェブアプリをどうやって変化させていくんだ?
「俺にはできません。」ですかな?w
俺が言ってるんじゃなくて世間一般の定義。
っていうか開発中とそうでないのが別れてる時点で
お前、前時代的だしなぁ。
お前なら一日に何十回もデプロイするような
ウェブアプリをどうやって変化させていくんだ?
「俺にはできません。」ですかな?w
756デフォルトの名無しさん
2017/02/08(水) 23:31:13.83ID:EqksEKaR ちなみに、TDDでは最初にテストコードを書いて
それに通る最小限の実装を書く、その後リファクタリングを
行うというのを繰り返して開発するものので、
http://www.atmarkit.co.jp/ait/articles/1403/05/news035.html#025
> 開発中に直すことをおれの界隈ではリファクタリングとは言わないね
> 開発中に直すことをおれの界隈ではリファクタリングとは言わないね
というのはもぐり、無知レベルでしかないよ
それに通る最小限の実装を書く、その後リファクタリングを
行うというのを繰り返して開発するものので、
http://www.atmarkit.co.jp/ait/articles/1403/05/news035.html#025
> 開発中に直すことをおれの界隈ではリファクタリングとは言わないね
> 開発中に直すことをおれの界隈ではリファクタリングとは言わないね
というのはもぐり、無知レベルでしかないよ
757デフォルトの名無しさん
2017/02/08(水) 23:31:50.65ID:EqksEKaR758デフォルトの名無しさん
2017/02/08(水) 23:36:21.44ID:E3hlYujb うーんw
自ら話を反らしていくスタイルw
自ら話を反らしていくスタイルw
759デフォルトの名無しさん
2017/02/08(水) 23:38:20.99ID:EqksEKaR それてないじゃん?w
それてるっていうのなら、戻していいよ。君の手で
それてるっていうのなら、戻していいよ。君の手で
760デフォルトの名無しさん
2017/02/08(水) 23:53:10.75ID:E3hlYujb うーんww
そもそも少人数短期イテレーション開発におけるリファクタリングの話なんて始めからしてないのだがw
毎日デプロイするwebアプリ?そんな小さな話も全くしてないw
おまえが数億円契約の大規模開発をやったことないのはよくわかったw
そもそも少人数短期イテレーション開発におけるリファクタリングの話なんて始めからしてないのだがw
毎日デプロイするwebアプリ?そんな小さな話も全くしてないw
おまえが数億円契約の大規模開発をやったことないのはよくわかったw
761デフォルトの名無しさん
2017/02/09(木) 01:52:04.00ID:CNUBJX7I >>760
大規模だからって何の自慢になると思ってるんだ?
大規模は別にどうでもいいよ
どういった開発をしているのかが重要だろ。
いくら金がかかっているからって
ろくでもないやつばっかり集めて人海戦術で
ひたすら効率の悪い開発方法をしている?
だとしたら自慢どころか恥だよね。
もっと規模とか金以外に自慢できることないの?
大規模だからって何の自慢になると思ってるんだ?
大規模は別にどうでもいいよ
どういった開発をしているのかが重要だろ。
いくら金がかかっているからって
ろくでもないやつばっかり集めて人海戦術で
ひたすら効率の悪い開発方法をしている?
だとしたら自慢どころか恥だよね。
もっと規模とか金以外に自慢できることないの?
762デフォルトの名無しさん
2017/02/09(木) 04:01:02.59ID:VpXdxNl0763デフォルトの名無しさん
2017/02/09(木) 06:44:32.92ID:IOSaScxB リファクタリングに開発規模は関係ない
レガシーコードの山に頭抱えながら仕事するのも自由だし、好きにしてくれ
レガシーコードの山に頭抱えながら仕事するのも自由だし、好きにしてくれ
764デフォルトの名無しさん
2017/02/09(木) 07:38:20.57ID:W9c8fNbT 規模が大きい開発はクソコーダーがクソコードを量産してくるから俺様が随時コードを書き換えてやる
ってのが ID:E3hlYujb の主張
こういうリファクタオナニストに好きにさせてはいけない
ってのが ID:E3hlYujb の主張
こういうリファクタオナニストに好きにさせてはいけない
765デフォルトの名無しさん
2017/02/09(木) 08:20:04.14ID:2NAFiD3Z リファクタリングに必要な見積りを出してくれ
766デフォルトの名無しさん
2017/02/09(木) 11:43:47.05ID:qhoyYE05 >>765
これな
これな
767デフォルトの名無しさん
2017/02/09(木) 11:48:27.92ID:qhoyYE05768デフォルトの名無しさん
2017/02/09(木) 12:39:15.13ID:Ra4XvV1b 本当に大規模開発やったことあるのか?
新規で作って作りっぱなしだったのか?
新規で作って作りっぱなしだったのか?
769デフォルトの名無しさん
2017/02/09(木) 12:49:33.29ID:2OmAEiex770デフォルトの名無しさん
2017/02/09(木) 13:07:11.30ID:/WGpXuP3771デフォルトの名無しさん
2017/02/09(木) 13:11:02.24ID:/WGpXuP3772デフォルトの名無しさん
2017/02/09(木) 14:01:01.97ID:Ra4XvV1b じゃあ何のことをリファクタリングと呼んでるの?
773デフォルトの名無しさん
2017/02/09(木) 14:27:10.00ID:VpXdxNl0 そもそもファクタリングとは?
それをやり直すってことでしょ
それをやり直すってことでしょ
774デフォルトの名無しさん
2017/02/09(木) 15:01:34.48ID:Lyr2GEHi じゃあ逆に聞くけど設計から変えてUTのコードがそのまま使えるケースってどんなの?w
テストコードの書き換えが必要な改修は開発スキームとして事前にスケジュールに組み込まれるリファクタリングとは全く別ものだろww
テストコードの書き換えが必要な改修は開発スキームとして事前にスケジュールに組み込まれるリファクタリングとは全く別ものだろww
775デフォルトの名無しさん
2017/02/09(木) 15:14:26.88ID:lnTHGhne 別にリファクタリングは個別のユニットテストに影響ないレベルとは限らないでしょ。
リファクタリング対象によってそこは取捨選択すべき。
リファクタリング対象によってそこは取捨選択すべき。
776デフォルトの名無しさん
2017/02/09(木) 17:19:48.49ID:EPpoXydB 結論から言えば概念としてのリファクタリングは否定しないが、
リファクタリングのための工数なんて実際は取れないし、
そんな余計なリソースがあるなら他の事に回した方がいい
所詮暇を持て余した学生の戯言だよ
リファクタリングのための工数なんて実際は取れないし、
そんな余計なリソースがあるなら他の事に回した方がいい
所詮暇を持て余した学生の戯言だよ
777デフォルトの名無しさん
2017/02/09(木) 19:13:25.68ID:Lyr2GEHi778デフォルトの名無しさん
2017/02/09(木) 19:41:42.20ID:2NAFiD3Z プロジェクトの開始時にリファクタリングの見積も出してね
オーバーした分はお金出せないからね
オーバーした分はお金出せないからね
779デフォルトの名無しさん
2017/02/09(木) 20:10:08.12ID:VpXdxNl0 だからまずファクタリングの定義からしようか
それをやり直すってだけの話でしょ
それをやり直すってだけの話でしょ
780デフォルトの名無しさん
2017/02/09(木) 20:29:21.19ID:Ra4XvV1b 機能要件を満たすため
非機能要件を満たすため
オナニーするため
目的が何であろうとリファクタリングはリファクタリングですよ
非機能要件を満たすため
オナニーするため
目的が何であろうとリファクタリングはリファクタリングですよ
781デフォルトの名無しさん
2017/02/09(木) 21:09:00.44ID:DXVOdvHd オブジェクト指向でシステム開発するなら、リファクタリングは必須です。
クラスの設計、インターフェースの抽出・設計といったプログラミング前に
既に必要です。
クラスの設計、インターフェースの抽出・設計といったプログラミング前に
既に必要です。
782デフォルトの名無しさん
2017/02/09(木) 21:09:16.18ID:W9c8fNbT783デフォルトの名無しさん
2017/02/09(木) 21:29:45.22ID:l61bzEJu とりあえず動くものを新規で作って書き換える場合か、すでに動いているもので、修正が入りまくって汚くなったものを整理するくらいしかない。
自由にいじくりまわすのをリファクタリングとは言わない。
自由にいじくりまわすのをリファクタリングとは言わない。
784デフォルトの名無しさん
2017/02/09(木) 21:35:33.02ID:CNUBJX7I >>762
> while文よりfor文の方がわかりやすいね
> って書き直す事のどこが機能追加なの?
while文よりfor文の方がわかりやすいねとか
俺は言ってない。お前が言ったことだ。
お前は、機能追加じゃない例を自分で出して
自分で自分にツッコミを入れてるだけなんだが、
何がしたいの?
> while文よりfor文の方がわかりやすいね
> って書き直す事のどこが機能追加なの?
while文よりfor文の方がわかりやすいねとか
俺は言ってない。お前が言ったことだ。
お前は、機能追加じゃない例を自分で出して
自分で自分にツッコミを入れてるだけなんだが、
何がしたいの?
785デフォルトの名無しさん
2017/02/09(木) 21:39:19.62ID:lnTHGhne786デフォルトの名無しさん
2017/02/09(木) 21:42:53.34ID:CNUBJX7I >>777
> いやだから話の前提が>>749だから選択も糞もないくやらないといけない状況でそれをリファクタリングとはいわねーって
お前根本的に間違ってるんだわ。
まず先に最初の話のツッコミ漏れを補完しておくとだな
>>749で設計に問題があるときはどうするのか聞いた。
これは言い換えると「やり直し以外の選択の余地がない例」として上げたんだよ。
それなのに、>>750で
> ただの上流への出戻り、やり直し、糞開発じゃないの
やり直しすればいいじゃんって、お前アスペか?
やり直し以外の選択肢の余地がないことにたいして、
「やり直ししろ」ってそれは反論になってない
俺が意図してる答えだ
でな、リファクタリングかどうかっていうのは、手段なの
「糞もないくやらないといけない状況」のときに、
リファクタリングを使って、修正を行うのか
リファクタリングを使わずに、修正を行うかの話
ここまで言えば、お前が根本的に間違ってることが理解できるだろ?
お前はリファクタリングが手段であることをわかってない。どんな状況かはどうでもいい。
修正をするときに、どうやって修正するかの話だ。
お前は、リファクタリングしないで、行き当たりばったりで修正してバグを入れ込むんだろう?
俺はリファクタリング(わかりやすく言えばリファクタリングの本に書いてあるテクニック)を使って修正するんだよ
> いやだから話の前提が>>749だから選択も糞もないくやらないといけない状況でそれをリファクタリングとはいわねーって
お前根本的に間違ってるんだわ。
まず先に最初の話のツッコミ漏れを補完しておくとだな
>>749で設計に問題があるときはどうするのか聞いた。
これは言い換えると「やり直し以外の選択の余地がない例」として上げたんだよ。
それなのに、>>750で
> ただの上流への出戻り、やり直し、糞開発じゃないの
やり直しすればいいじゃんって、お前アスペか?
やり直し以外の選択肢の余地がないことにたいして、
「やり直ししろ」ってそれは反論になってない
俺が意図してる答えだ
でな、リファクタリングかどうかっていうのは、手段なの
「糞もないくやらないといけない状況」のときに、
リファクタリングを使って、修正を行うのか
リファクタリングを使わずに、修正を行うかの話
ここまで言えば、お前が根本的に間違ってることが理解できるだろ?
お前はリファクタリングが手段であることをわかってない。どんな状況かはどうでもいい。
修正をするときに、どうやって修正するかの話だ。
お前は、リファクタリングしないで、行き当たりばったりで修正してバグを入れ込むんだろう?
俺はリファクタリング(わかりやすく言えばリファクタリングの本に書いてあるテクニック)を使って修正するんだよ
787デフォルトの名無しさん
2017/02/09(木) 21:45:29.54ID:CNUBJX7I >>778
> プロジェクトの開始時にリファクタリングの見積も出してね
リファクタリングが手段であることを知れば、
「リファクタリング(手段)の見積もり」がおかしい言い方だとわかるだろう。
正しく言うならば「プロジェクトの開始時に修正の見積もだしてね」だ
修正にはバグだけじゃなくて仕様の追加や間違いによる修正も含まれる。
あとは修正するしかない状況で、行き当たりばったりで修正するか
リファクタリングで修正するかの違いなだけだ
> プロジェクトの開始時にリファクタリングの見積も出してね
リファクタリングが手段であることを知れば、
「リファクタリング(手段)の見積もり」がおかしい言い方だとわかるだろう。
正しく言うならば「プロジェクトの開始時に修正の見積もだしてね」だ
修正にはバグだけじゃなくて仕様の追加や間違いによる修正も含まれる。
あとは修正するしかない状況で、行き当たりばったりで修正するか
リファクタリングで修正するかの違いなだけだ
788デフォルトの名無しさん
2017/02/09(木) 21:46:27.58ID:CNUBJX7I789デフォルトの名無しさん
2017/02/09(木) 21:52:20.21ID:lnTHGhne790762
2017/02/09(木) 21:55:31.20ID:VpXdxNl0 >>784
ほらな
きちんと書いてないと
こうやって誤解されるわけだ
言いたかったのは
「現状while文で処理されていて問題ないけど
このケースはfor文の方がわかりやすいから書き直そう!」
どこに機能追加される余地があんの?
ほらな
きちんと書いてないと
こうやって誤解されるわけだ
言いたかったのは
「現状while文で処理されていて問題ないけど
このケースはfor文の方がわかりやすいから書き直そう!」
どこに機能追加される余地があんの?
791デフォルトの名無しさん
2017/02/09(木) 21:57:10.56ID:CNUBJX7I793デフォルトの名無しさん
2017/02/09(木) 21:58:44.70ID:CNUBJX7I >>753には while とか for の話は
明らかに書いてないけど?
明らかに書いてないけど?
794デフォルトの名無しさん
2017/02/09(木) 21:59:52.90ID:VpXdxNl0795デフォルトの名無しさん
2017/02/09(木) 22:01:15.92ID:CNUBJX7I YAGNIの話?
753 自分:デフォルトの名無しさん[sage] 投稿日:2017/02/08(水) 23:18:57.65 ID:EqksEKaR [3/7]
> 将来性を考慮しての改善するのがリファクタリングでしょ
YAGNIに反するからそれは違う。
これがどうfor やwhileと関係すんの?
753 自分:デフォルトの名無しさん[sage] 投稿日:2017/02/08(水) 23:18:57.65 ID:EqksEKaR [3/7]
> 将来性を考慮しての改善するのがリファクタリングでしょ
YAGNIに反するからそれは違う。
これがどうfor やwhileと関係すんの?
796デフォルトの名無しさん
2017/02/09(木) 22:02:47.10ID:lnTHGhne ほんまやね。レス辿って明らかにおかしいのを見つけたがら言ったけどそれforのレスにかすりもしなかったわ。すまん。
797デフォルトの名無しさん
2017/02/09(木) 22:06:33.30ID:Lyr2GEHi >>786
おまえ日本人か?w
おまえ日本人か?w
798デフォルトの名無しさん
2017/02/09(木) 22:08:07.52ID:CNUBJX7I 下手に勘違いされると困るから補足しておくと、
汚いから修正するってだけならYAGNIでやるべきではないが、
その部分を修正していたり読む必要があるならば、
いつ必要しているの?今でしょ?ってことで
今すぐ必要としていることなわけだから、
そこを修正するのはYAGNIではない
汚いから修正するってだけならYAGNIでやるべきではないが、
その部分を修正していたり読む必要があるならば、
いつ必要しているの?今でしょ?ってことで
今すぐ必要としていることなわけだから、
そこを修正するのはYAGNIではない
799デフォルトの名無しさん
2017/02/09(木) 22:25:26.99ID:lnTHGhne 読まなきゃ汚いと思わないんだから、矛盾してるような気がするけど。
必要ないのに汚いコードを読み始める酔狂な奴が居るとしたら別だけど。
必要ないのに汚いコードを読み始める酔狂な奴が居るとしたら別だけど。
800デフォルトの名無しさん
2017/02/09(木) 22:29:02.28ID:CNUBJX7I > 読まなきゃ汚いと思わないんだから
いや、コードメトリクスツールなどをつかえば
読まなくても、客観的に汚いとわかる
いや、コードメトリクスツールなどをつかえば
読まなくても、客観的に汚いとわかる
801デフォルトの名無しさん
2017/02/09(木) 22:29:15.76ID:Ra4XvV1b >>797
いつになったら何のことをリファクタリングと呼んでるか書いてくれるの?
いつになったら何のことをリファクタリングと呼んでるか書いてくれるの?
802デフォルトの名無しさん
2017/02/09(木) 22:34:05.44ID:Ra4XvV1b 汚いから直すってのもリファクタリングと呼ぶと思うけど、
汚いから直すってのは実際の開発現場じゃなかなか難しいと思うよ
その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね
汚いから直すってのは実際の開発現場じゃなかなか難しいと思うよ
その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね
803デフォルトの名無しさん
2017/02/09(木) 22:41:25.83ID:CNUBJX7I >>802
> その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね
だから俺は聞いたんだよ?
そういうどうしても直さなければいけないときに
リファクタリングで安全に修正するのか?
それとも行き当たりばったりに修正してバグを入れるのか?って
> その汚さが原因で性能が出ないとか、機能追加できない、し辛いって明確な理由があれば別だけどね
だから俺は聞いたんだよ?
そういうどうしても直さなければいけないときに
リファクタリングで安全に修正するのか?
それとも行き当たりばったりに修正してバグを入れるのか?って
804デフォルトの名無しさん
2017/02/09(木) 22:43:53.46ID:Ra4XvV1b いや、俺に噛みつかれましても
805デフォルトの名無しさん
2017/02/09(木) 22:45:29.16ID:Lyr2GEHi806デフォルトの名無しさん
2017/02/09(木) 22:47:35.42ID:CNUBJX7I リファクタリングを行うタイミングを間違ってるんだよね
俺は何らかの機能修正とかでコードをいじるときに
関連するところを少しづつなおす
大抵が数行程度で10分もかからずに終わる
こまめに修正しないで、手遅れになってからやろうとするから
大規模になってしまって、別途工数が〜とかいう話になるんだよ。
手遅れになってしまったものを修正する工数がないからと
手遅れなコードのままさらにコードを追加するから
さらに手遅れになっていくんだろうが
俺は何らかの機能修正とかでコードをいじるときに
関連するところを少しづつなおす
大抵が数行程度で10分もかからずに終わる
こまめに修正しないで、手遅れになってからやろうとするから
大規模になってしまって、別途工数が〜とかいう話になるんだよ。
手遅れになってしまったものを修正する工数がないからと
手遅れなコードのままさらにコードを追加するから
さらに手遅れになっていくんだろうが
807デフォルトの名無しさん
2017/02/09(木) 22:59:39.83ID:lnTHGhne >>806
多分解ってると思うけどその言い方は誤解を生むと思うぞ。
リファクタリングと機能追加は明確に分けるべきで、リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき
多分解ってると思うけどその言い方は誤解を生むと思うぞ。
リファクタリングと機能追加は明確に分けるべきで、リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき
808デフォルトの名無しさん
2017/02/09(木) 23:07:28.39ID:Ra4XvV1b >>805
他人の否定はしてるけど、自分では何も言ってないよ君
他人の否定はしてるけど、自分では何も言ってないよ君
809デフォルトの名無しさん
2017/02/09(木) 23:07:33.91ID:2NAFiD3Z リファクタリング=上腕二頭筋
810デフォルトの名無しさん
2017/02/09(木) 23:12:13.46ID:CNUBJX7I >>807
うん。わかってる
> リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき
どちらもあるよ。
機能追加してからリファクタリングする物の典型的な例がTDD
Red/Green/Refactor 通りの順番
うん。わかってる
> リファクタリングして、テスト走らせて振る舞いが変わってない事を確認してから機能追加に入るべき
どちらもあるよ。
機能追加してからリファクタリングする物の典型的な例がTDD
Red/Green/Refactor 通りの順番
811デフォルトの名無しさん
2017/02/10(金) 00:46:58.66ID:/WxwB06L どんどん別の手法がまざっていく洗脳馬鹿
812デフォルトの名無しさん
2017/02/10(金) 01:18:18.70ID:+KYQgfiL 別の手法ってなんの話だ?
813デフォルトの名無しさん
2017/02/10(金) 02:12:34.39ID:/WxwB06L >>810 のことだよ。
814デフォルトの名無しさん
2017/02/10(金) 02:25:45.27ID:+KYQgfiL > Red/Green/Refactor 通りの順番
これのことか?
Refactorはリファクタリングのことだよ
別の手法じゃなくて全く同じもの
これのことか?
Refactorはリファクタリングのことだよ
別の手法じゃなくて全く同じもの
815デフォルトの名無しさん
2017/02/10(金) 12:06:41.01ID:QRGeJWes 1.機能追加前にテスト書く
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる
もしかしてこれもリファクタリング扱いしてるの?
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる
もしかしてこれもリファクタリング扱いしてるの?
816デフォルトの名無しさん
2017/02/10(金) 12:38:04.04ID:+HewTgrG >>815
それはRed→Greenのところでしょ…
それはRed→Greenのところでしょ…
817デフォルトの名無しさん
2017/02/10(金) 21:26:25.38ID:1oBD+cZp 最近、特定のフラグ変数に関するif文の山をswitch構文に清書し直したんだけど、これってリファクタリング?
818デフォルトの名無しさん
2017/02/10(金) 21:34:25.32ID:/WxwB06L >>816
それさ、リファクタリングのいち定義であってリファクタリングではないから。
それさ、リファクタリングのいち定義であってリファクタリングではないから。
819デフォルトの名無しさん
2017/02/10(金) 21:35:50.89ID:/WxwB06L >>817
そのif文が変ならな。まともなif文なら書き換える必要がない。
そのif文が変ならな。まともなif文なら書き換える必要がない。
820デフォルトの名無しさん
2017/02/10(金) 21:37:35.22ID:/WxwB06L >>817
ただswitch文にすると仕様変更時にswitch文では対応できなくなる可能性があるので逆かもしれない。
ただswitch文にすると仕様変更時にswitch文では対応できなくなる可能性があるので逆かもしれない。
821デフォルトの名無しさん
2017/02/10(金) 21:42:51.54ID:+KYQgfiL >>817
確かにswitchでかけるならばifよりswitchの方が少しだけ見やすくなるけど、
大抵はリファクタリングにはならないよ
巨大なswitchもリファクタリングすべき対象となる
何がダメなのかというと比較の回数がifのときと変わってないから。
比較が多い=複雑なので、これは単なる書き方の違いでしかなくて
リファクタリングとまではいかない。重要なのは比較回数
じゃあどうすべきかというと、一番簡単な方法は
関数テーブル(ルックアップテーブル)を使う方法
大量のswitchでの比較が条件が揃えば0個にまで減る
確かにswitchでかけるならばifよりswitchの方が少しだけ見やすくなるけど、
大抵はリファクタリングにはならないよ
巨大なswitchもリファクタリングすべき対象となる
何がダメなのかというと比較の回数がifのときと変わってないから。
比較が多い=複雑なので、これは単なる書き方の違いでしかなくて
リファクタリングとまではいかない。重要なのは比較回数
じゃあどうすべきかというと、一番簡単な方法は
関数テーブル(ルックアップテーブル)を使う方法
大量のswitchでの比較が条件が揃えば0個にまで減る
822デフォルトの名無しさん
2017/02/10(金) 21:46:49.78ID:+KYQgfiL823デフォルトの名無しさん
2017/02/10(金) 21:57:11.80ID:+KYQgfiL * Green
1.機能追加前にテスト書く
* Red
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる
5.テストに通った。やったー完成だ! ←ふざけんじゃないよ。なんだこのコピペだらけで無駄がばかりのクソコードは
* Green
6.リファクタリングする
7. 読みやすくなりました。←このコードならレビューする気になる。シンプルあとで読んだ人も理解しやすいだろう
8. 完成、1に戻る
1.機能追加前にテスト書く
* Red
2.追加コード書く
3.テスト
4.ダメならテストに通るまでコードいじる
5.テストに通った。やったー完成だ! ←ふざけんじゃないよ。なんだこのコピペだらけで無駄がばかりのクソコードは
* Green
6.リファクタリングする
7. 読みやすくなりました。←このコードならレビューする気になる。シンプルあとで読んだ人も理解しやすいだろう
8. 完成、1に戻る
824デフォルトの名無しさん
2017/02/10(金) 22:00:06.48ID:+KYQgfiL 現実にはこの5. テストに通っただけの状態で
完成としてしまうやつが多いんだよな。
そういうコードで許される環境っていうのは
コードレビューが行われていない。
コードを見ずにテストに通ればOKとしてしまう環境
そして向かうはデスマーチw
完成としてしまうやつが多いんだよな。
そういうコードで許される環境っていうのは
コードレビューが行われていない。
コードを見ずにテストに通ればOKとしてしまう環境
そして向かうはデスマーチw
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
■ このスレッドは過去ログ倉庫に格納されています
ニュース
- 【サッカー】U-17日本代表、激闘PK戦制す 北朝鮮撃破で6大会ぶり8強入り U17W杯 [久太郎★]
- 日本行き空路49万件キャンセル 中国自粛呼びかけ 日本行きチケット予約の約32%に相当 ★3 [ぐれ★]
- 【サッカー】日本代表、ボリビアに3発快勝 森保監督通算100試合目を飾る…鎌田、町野、中村がゴール [久太郎★]
- XやChatGPTで広範囲の通信障害 投稿や閲覧できず [蚤の市★]
- 【芸能】日中関係悪化でエンタメ業界に大ダメージ… JO1の中国でのイベント中止、邦画は公開延期、STARTOアイドルへの影響も [冬月記者★]
- 【インバウンド】中国人観光客の日本での消費額は年間約2兆円超…中国政府は公務員の出張取り消し [1ゲットロボ★]
- 青銅聖闘士のパンチは音速←わかる 白銀聖闘士はその数倍←まぁわかる 黄金聖闘士は光速←は?
- 4時だから窓から4回ちんこ出した
- クマどもが冬眠拒否
- さわやかって
- 生活保護を受けている私だけど、おはようございます。
- 【朗報】ローソン「Мサイズのカップを購入してLサイズのコーヒーを入れてくださいね」 [455031798]
