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

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

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

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

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

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

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

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

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

で？

**デフォルトの名無しさん** · 2014/09/17(水) 12:15:48.80

>>511
うわあ読解力ﾊﾟﾈｪ

**デフォルトの名無しさん** · 2014/09/17(水) 21:24:55.43

>>511
> 独りよがりの文章しか書けないんだろうな。
独りよがりって、自分よがりと同じ意味じゃね？

**デフォルトの名無しさん** · 2014/09/17(水) 23:24:58.57

君達は本当に頭が悪いんだな

**デフォルトの名無しさん** · 2014/09/18(木) 00:20:18.22

うむ

http://www.weblio.jp/content/%E8%87%AA%E5%88%86%E3%82%88%E3%81%8C%E3%82%8A

自分よがり
読み方：じぶんよがり
別表記：自分善がり

ひとりよがり（独り善がり）に同じ。周囲を顧慮せずに自分の都合や損得のみで物事を考えること。

**デフォルトの名無しさん** · 2014/09/18(木) 19:33:06.94

>>500
まぁほぼ同意だな

**デフォルトの名無しさん** · 2014/09/18(木) 20:26:58.62

>>516
自作自演しなくていいよ。
そんなコメント普通しないからｗ

**デフォルトの名無しさん** · 2014/09/18(木) 22:19:08.95

要求仕様の変更に追随するために設計を変えていくのがリファクタだ

撒き散らしたクソコード直すのはリファクタとは言わん
テストの前に個々人レベルでやる仕事だ

**デフォルトの名無しさん** · 2014/09/18(木) 22:31:11.52

他人が作ったものをメンテする時は
まず他人にやらせろってこと？

**デフォルトの名無しさん** · 2014/09/18(木) 23:29:47.48

クソコード直すのがリファクタリングでいいと思うお

**デフォルトの名無しさん** · 2014/09/18(木) 23:52:12.25

結局言葉遊びの域を出なかったね

**デフォルトの名無しさん** · 2014/09/18(木) 23:53:02.19

つーかリファクタリングとか言いにくいから修正でいいじゃん

**デフォルトの名無しさん** · 2014/09/19(金) 00:33:41.85

リファクタリングを定義しようぜ
とりあえずQA形式にして質問と回答は最近の書き込みから適当に埋めた

＜＜リファクタリングとは＞＞
Q 何時やるの？ A やりたい時。糞コードを見つけたら。今でしょ
Q 具体的には(工程では)いつ？ A 回答なし
Q やる必要あるの？ A 回答なし
Q 誰がやるの？A 自称上級者
Q 上級者の定義は？A 回答なし
Q その上級者には何かメリットが？A 回答なし
Q リファクタリングってお金もらえるの？A ボランティア
Q どういう効果が期待できるの？ A 将来的なリスクを回避する、または逆にリスクを抱える

**デフォルトの名無しさん** · 2014/09/19(金) 17:39:57.70

>>500に対して>>503の返しするということは、>>500の話が論が正しいことを>>503が認めているということ。

つまるところ、会社の構成員の実力によって不可能になるということ。
をあんたの会社では～。の一文で認めている。

ゆえにその切り返しをするなら、リファクタリングは議論する価値がないということ

**デフォルトの名無しさん** · 2014/09/19(金) 17:51:51.67

ちょっと何言ってるのかわかりませんわ

**デフォルトの名無しさん** · 2014/09/19(金) 18:08:18.14

誰でもできることでないなら、実現は夢物語ということだ

**デフォルトの名無しさん** · 2014/09/19(金) 18:36:04.63

工程を確保して
下がやりたいと言ったら邪魔するなというだけの話かもしれない

**デフォルトの名無しさん** · 2014/09/19(金) 18:47:45.84

>>524
行き詰まるところ先人達が考えて考えた結果

　新言語の開発

にしか答えがないんだよなww
どんな素人でも言語のルールには逆らえない。つまり従わざるを得ない。
理論だけで劇的な変化があったのなんて

GOTOレス

だけだろ。

わかりやすくステップ数で言うけどjavaで10ステップで組まれていてごちゃごちゃなプログラムがある。
仕様変更するのに1万ステップの変更が必要。これを、新言語おまんこ女学院言語で開発すると、

main()
おまんちょ（）

って書くだけで同じ機能が実現され、

仕様変更には

use ここか？ここがええんか？
main
おまんちょ（伝説の第４９手目（３箇所、同時、左手だけちょうどいい微速））

って直せば実現できる言語が生まれれば、これ以上の可読性の向上はないわけで。
馬鹿でも簡単に、馬鹿でも可読性が高く、馬鹿でもスピーディーに・・・・を求めて新言語というのは生まれている

**デフォルトの名無しさん** · 2014/09/19(金) 18:48:57.11

>javaで10ステップ
を
>javaで10万ステップ
に訂正

**デフォルトの名無しさん** · 2014/09/19(金) 19:42:02.22

>>485
パッケージ製品のメジャーバージョンアップとか？
一般的なIT土方の工事現場にはまず縁ないわな

**デフォルトの名無しさん** · 2014/09/19(金) 20:44:14.36

リファクタリングしない人は
修正する時、テストしないの？

テストするならリファクタリングしても
問題ないよね。

**デフォルトの名無しさん** · 2014/09/19(金) 20:59:36.15

全部埋めてやったぞ

＜＜リファクタリングとは＞＞
Q 何時やるの？ A やりたい時。糞コードを見つけたら。今でしょ
Q 具体的には(工程では)いつ？ A アジャイルですから。工程て何？
Q やる必要あるの？ A やらなきゃバグでるよ？いつか破綻するよ？
Q 誰がやるの？A 自称上級者
Q 上級者の定義は？A 糞コードを発見する嗅覚を持つこと
Q その上級者には何かメリットが？A コードが綺麗な方が気持ちいいだろ
Q リファクタリングってお金もらえるの？A ボランティア
Q どういう効果が期待できるの？ A 将来的なリスクを回避する、または逆にリスクを抱える

**デフォルトの名無しさん** · 2014/09/19(金) 21:28:04.06

＜＜リファクタリングとは＞＞
Q 何時やるの？ A テスト駆動開発においては、テスト記述→コードを書く→リファクタリングの順番で開発する
Q 具体的には(工程では)いつ？ A コードを書いた直後。工程で言えば開発工程
Q やる必要あるの？ A 義務だろうね。小説で言えば清書する必要あるの？ぐらいのレベル。
Q 誰がやるの？A 客観的指標（McCab等）で差があるコードを見比べて、優れている方を言えるレベルの人全員
Q 上級者の定義は？A 客観的指標（McCab等）で計測して良いとされるコードを書ける人
Q その上級者には何かメリットが？A 上級者であっても、可読性やメンテナンス性が高いコードの方が早く機能追加・修正・不具合対応ができる。
Q リファクタリングってお金もらえるの？A 開発の一部であり、清書みたいなものなので当然お金はもらえる。
Q どういう効果が期待できるの？ A 期待ではなく事実としてメリットがある。メリットは二つ上に書いたとおり。

**デフォルトの名無しさん** · 2014/09/19(金) 21:57:34.29

当然お金はもらえるってなんだよワロタ

**デフォルトの名無しさん** · 2014/09/19(金) 22:04:08.26

>>534
たとえばさ、誤字脱字、これはあっても読めれば別に問題ないんだよ。
でも見つけたら直すだろう？

そうやって誤字脱字の見直し、修正が例え1時間で終わったとしても
業務である以上、それは給料に変わるんだよ。

**デフォルトの名無しさん** · 2014/09/19(金) 22:35:47.59

問題ないのに直すのは明らかに無駄な労力だろ
リファクタリングてそういう事なのか？

**デフォルトの名無しさん** · 2014/09/20(土) 00:28:50.80

そこらに地雷が埋まってても踏まなきゃ平気なんだよ

**デフォルトの名無しさん** · 2014/09/20(土) 00:35:33.60

>>531
そもそもテストは各工程に含まれてるよね？
一度コードに手を付けたらリファクタリングだろうが何だろうがテストはやり直しだよ
工程をこなしてる間つまりリリースし終わって暇にでもならないと
リファクタリングなんか入り込む余地なんて無いぞ
要するに無駄ってことじゃないの

**デフォルトの名無しさん** · 2014/09/20(土) 01:43:02.74

>>536
「ソースコードを読むのに時間が掛かる」は
一番重要な問題だよ。

それが開発工数の大半を占めているんだから

**デフォルトの名無しさん** · 2014/09/20(土) 10:04:40.32

まあ理論的に完璧なリファクタリング手法を使っても
無関係であるはずの部分でコンパイラやＯＳのバグを踏んだらそれまでだから
実テストにやたら工数のかかってたらその後はさわれないだろうね

**デフォルトの名無しさん** · 2014/09/20(土) 10:58:44.27

テストやりました。バグが見つかりました。

ソースコードが汚くて、コードの意味がわかりません。
ソースコードが汚くて、バグの原因がわかりません。
あちこちにコピペされてて、バグの修正が大変です。
バグを修正したら、別のバグが発生しました。
バグを修正しましたが、処理が複数のモジュールに
分散していたので多くの再テストが必要です。

こういうのを解決するのがリファクタリング。

**デフォルトの名無しさん** · 2014/09/20(土) 11:27:24.57

違うね。

> テストやりました。バグが見つかりました。
> （以下略）

それはデバッグや。

**デフォルトの名無しさん** · 2014/09/20(土) 11:43:01.81

>>542
いや、それは前置きだから。

リファクタリングが解決するのは、
デバッグまでの過程で発生する問題。

同じデバッグという目的であっても、
その目的を達成するまでの時間は違うんだよ。

**デフォルトの名無しさん** · 2014/09/20(土) 11:48:06.25

ちょっと意味が分からない

**デフォルトの名無しさん** · 2014/09/20(土) 12:22:36.52

開発段階ですでにスケジュールオーバーしてる現場だらけなのに机上の空論w
新言語開発しろw

**デフォルトの名無しさん** · 2014/09/20(土) 13:18:35.72

>>544
意味がわからないレベルの人っているんだよねｗ

**デフォルトの名無しさん** · 2014/09/20(土) 14:46:45.86

テストってコードを書いた本人がやるべき？

**デフォルトの名無しさん** · 2014/09/20(土) 15:02:25.23

リファクタリングすら必要ないほどの腕のいい奴を最初から揃えれば良くね？
というのと同じくらいリファクタリング理論は破綻してる

**デフォルトの名無しさん** · 2014/09/20(土) 15:33:26.65

他人のソースコードの解読は新規作成より時間がかかるケースが多々。
仕様がわかってるなら、そんなチェックをお願いしないといけない奴のソースレビューをするより、最初から自分でやったほうが早いし確実
つまり、リファクタリングの行き着く先は、無脳者を開発に入れなければ良いに行き着く。
少数精鋭方式。

**デフォルトの名無しさん** · 2014/09/20(土) 15:48:28.37

>>548-549
一理あるが理想論
だから現実解としてリファクタリングが出てきたんじゃないのか？

**デフォルトの名無しさん** · 2014/09/20(土) 16:04:59.48

仕様変更っていうのは避けられないものだし、
機能追加や新しく出来た○○に対応するとか
どうしても変化は避けられないわけで
その変化にたいしてコードも変化させていかないといけないんだよ。

それにさ、人っていうのは成長するのは当たり前なのだから、
一年前の自分より、もっと優れたものを書けるようになっているはず。

一年後はさらに優れたものを書けるようになるはずだが、
それは今をきちんとやっていてできること。
過去の自分の真似ばかりしていても、成長はできないよ。

**デフォルトの名無しさん** · 2014/09/20(土) 16:20:19.19

何を意味の分からん事を
と思ったら、またお前かｗ

**デフォルトの名無しさん** · 2014/09/20(土) 18:11:49.33

>>552
言い返したいことがあるのなら
なにか言い返していいんだぜ

**デフォルトの名無しさん** · 2014/09/20(土) 20:30:21.92

>>550
でもgotoレスと違って誰も実践しないだろ？
そういうことだ。
誰が好き好んで人の尻拭いをやるというのだ

**デフォルトの名無しさん** · 2014/09/20(土) 20:40:59.03

リファクタリングは有能が無能の尻を拭く作業ではなく自分の尻を拭く作業だ

最初は速度など無視してとりあえず動くように作る
動いたら次に速く動くようにする
速く動いたらそれを読みやすく整える
読みやすくなったら今度は新機能を楽々追加する

これで立派なリファクタリングだ

**デフォルトの名無しさん** · 2014/09/20(土) 21:06:59.81

速く動くようにするのは一番後でいいだろ

**デフォルトの名無しさん** · 2014/09/20(土) 21:21:43.45

>>556
ボトルネックが出来て、そこを解決すれば全体に速度が上がるようなものなら
後からでもどうにかなるが、全体的に微妙に遅い、微妙にリソース食っている、
それで全体的に遅くなってるだと後からじゃどうにもならない。

**デフォルトの名無しさん** · 2014/09/20(土) 21:27:33.19

>>555
実際誰もやっていない
それが、この理論の決定的な答え
誰もやらない＝やって評価されるどころか、失敗した場合の責任を問われるだけ
メリット一切無い

**デフォルトの名無しさん** · 2014/09/20(土) 21:36:13.07

>>555
そもそも、そんなものはリファクタリングではない

リファクタリングは「現時点『運用中』の動いているシステムプログラム」に勝手に手を加えること。
「まがいなりにも動いているシステムのプログラムを勝手にいじるな」
というのが世界の常識。でこの常識を変えようとした思想がリファクタリング。
ただ、結局は、誰もやらんってこと。

理由の大半は「仕様がわからん」ということ。

**デフォルトの名無しさん** · 2014/09/20(土) 21:44:36.36

エクストリームプログラミングから全否定かよ。
カオスだな。

**デフォルトの名無しさん** · 2014/09/20(土) 21:54:03.46

バグを仕込んでしまう可能性があるのが致命的
あとは評価制度の問題だな
長期的に絶対メリットはあるんだがそんなん誰も見ちゃいねえし

**デフォルトの名無しさん** · 2014/09/20(土) 22:00:15.71

他人の尻を拭くハメになったとき、リファクタリングが可能である唯一の条件は、
引継ぎ時点でリファクタリングされているソースコードであること、だ
そうでなかったらご愁傷様あきらめた方がいい

**デフォルトの名無しさん** · 2014/09/20(土) 22:07:58.53

>>559
> リファクタリングは「現時点『運用中』の動いているシステムプログラム」に勝手に手を加えること。

なんでウソつくの？

本当と言い返せるものなら言い返していいんだぜ？ｗ
ただし、そんなことをしたら、ソースを要求するので
準備しておくこと。

**デフォルトの名無しさん** · 2014/09/20(土) 22:53:25.35

恐怖駆動開発だな。
http://www.hanselman.com/blog/FearDrivenDevelopmentFDD.aspx

レガシーコードでテストが無い and/or コードに対する理解が無い
↓
エンバグ、ビルドを壊すことへの恐怖

**デフォルトの名無しさん** · 2014/09/20(土) 23:06:54.25

>>557
新しいマシン買えばいいだけじゃないか
開発費とハードウェア費用どっちが高くつくと思ってるんだ

**デフォルトの名無しさん** · 2014/09/21(日) 01:19:15.97

りファクタリングって言うのは、オブジェクト指向の言語（これが大前提）で、
開発段階にあって、試作品が第一段階で問題なく動いて、
あとは性能やコードの見直しやｶｲｾﾞﾝだよねって段階で設計へフィードバックするプロセスですよ。
その後に顧客側のチェックと納品ですよね。

運用中のシステムに手を入れるという話では無いですよ。
ｶｲｾﾞﾝを全否定ならトヨタはどうなるんでしょうか？

**デフォルトの名無しさん** · 2014/09/21(日) 01:26:54.31

一度客に納入してさあ、次に改修の仕事がきたときにリファクタリングしていたから工数が減っていたとしてもなんのメリットもないんだよな。むしろ工数が少なきゃ金が取れない。リファクタリングした分取ったら水増し請求だ。

機能やバグについて同等で、メンテしやすさだけ向上させる工程なんて日本の一般的な開発ではインセンティブゼロだよ。趣味ならともかくプロの仕事なら。

最初からリファクタリングの作業も込みで工数見積もりなり、費用請求しろ、ってそんな見積もり見て納得する客いるわけないじゃん。最初から手直しいらない納品物にして下さい、と言われてオチ。

出来上がってテスト済みのソースをあれこれいじるなんてお遊びが入り込む余地はない。

**デフォルトの名無しさん** · 2014/09/21(日) 01:42:02.59

それと、テストという概念は個人がスクリプトやツール類を書く場合には絶対に生まれてこない発想で、
多人数で、複数の技能レベルの人が、仕様書をガイドに組織的に開発に関わる条件下で必然的に生じる仕事上の手続き。

個人がコード書く場合は、ソースコード全体、アルゴリズム、関数やクラスを全て把握しているので
テストコードをいちいち書く必要がない。だからデバックという手順はあっても、関数やクラスのテストコードを
いちいち書くという発想にはならない。コード修正も容易、エラーが発生してもその場で即対応できる個人レベルの作業としては正しい。

組織で開発する部門では他人が必要なコードを仕様化した関数やクラスの動作条件を定めているから、
それを大前提としてモジュール動作をテストする必要がある。一見して馬鹿げている行為に思えるのは、
アルゴリズムを提示せずトップダウン的に関数のIN/OUTのみで仕様を提示するためにモジュールのテストが必須になる。
チェックデータの側面についていえば情報管理上テストデータが必ず必須になる。それは本物と試験用のデータが違うということ。

テストコードは本人が書くと、世界は如何にばかげた行為に時間を費やしているのかと思うだけだが、
本来は書く側は最終コードが動作すると確認した上で、２重チェックとして仕様を作成した側がテストするためのコードなり手段を作るべきだな。
人が居ないというなら、テストコード書くのでなく、統合テスト的に動作チェックすべき。

**デフォルトの名無しさん** · 2014/09/21(日) 01:52:41.82

リファクタリングは工数水増しの手段ではない。

保守とは違うので、日本の開発やってる業界のインセンティブなど関係がない。
リファクタリングとは関係がないな。

見積もりの段階でリファクタリングなどの話は出てこない。
手直しが要るといって費用請求している段階で、それはリファクタリングの問題ではなく
失敗したプロジェクトや案件だということだろう。それを誤魔化す為に誰かが嘘として
リファクタリングと言ってるだけだろう。

**デフォルトの名無しさん** · 2014/09/21(日) 02:21:17.93

もし>>567が客側の意見であれば、失敗したプロジェクトや案件とは縁を切って
再出発したほうが開発側も、客側も気持ち的にはしあわせになれますよ。

**デフォルトの名無しさん** · 2014/09/21(日) 06:26:59.02

>>563

>http://ja.wikipedia.org/wiki/%E3%83%AA%E3%83%95%E3%82%A1%E3%82%AF%E3%82%BF%E3%83%AA%E3%83%B3%E3%82%B0_(%E3%83%97%E3%83%AD%E3%82%B0%E3%83%A9%E3%83%9F%E3%83%B3%E3%82%B0)

>リファクタリング登場の経緯と目的[編集]
>リファクタリングが登場する以前は、一度正常な動作をしたプログラムは二度と手を触れるべきではないと言われていた。
>下手に手を加えて動作が変わってしまうと、それに伴って関連する部分にも修正が加えられ、やがて修正はプロジェクト
>全体に波及し対処しきれなくなるかも知れない。またソフトウェアテストを十分に行い正常な動作が確認されたとしても、
>そのプログラムをわずかでも改変すれば、バグ (欠陥) が見つかったときに改変があったプログラムを疑わなければならない。

**デフォルトの名無しさん** · 2014/09/21(日) 06:35:53.53

>>566
>りファクタリングって言うのは、オブジェクト指向の言語（これが大前提）

は？wwwwwwwwwwwwwwwwwwww
オブジェクト指向とそうではない言語での違いを説明しろよ
オブジェクト指向という言葉になんか宗教的妄信でもしてるのか？

**デフォルトの名無しさん** · 2014/09/21(日) 09:08:44.80

必要なのはコードのリファクタリングではなく
チームメンバのリファクタリングである

**デフォルトの名無しさん** · 2014/09/21(日) 09:29:51.06

言い換えると、ソースコードをリファクタリング出来るように
チームメンバーの技術力を高めることである。

**デフォルトの名無しさん** · 2014/09/21(日) 09:47:59.75

> チームメンバのリファクタリングである
リファクタリングて振舞いを変えてはいかんのでは？

**デフォルトの名無しさん** · 2014/09/21(日) 12:25:58.75

例えばバグが多く、しかも非常にメンテしにくい糞コードがあったとする
この場合
1. バグを含んだままリファクタリングして、その後バグを直す
2. 先にバグを直して正しいテストを通る様にしてから、リファクタリングに着手する
どっち？

**デフォルトの名無しさん** · 2014/09/21(日) 12:33:35.65

>>576
原則としては2番

ただし、今既に十分なテストがあるのなら、
リファクタリングしてもよい。

ようするに「バグが有ることに気づいていなかった」と考える。

バグが有るといえども、その状態で使われていたのであれば、
問題ない範囲（把握している範囲）で使われていたわけで、
使われていた部分の動きが変わっていなければ何も問題ない。

適切に動いていないバグの部分の動きが変わった所で
誰も困らない。そもそもバグは仕様ではないのだから。

**デフォルトの名無しさん** · 2014/09/21(日) 13:51:42.60

>>576
2. 一択。
1. は死亡フラグ。

**デフォルトの名無しさん** · 2014/09/21(日) 15:34:48.90

ホント現実をみてないな
クソコードの影にクソ仕様書あり

仕様が網羅されていないクソ仕様書、そしてあってるのかあってないのかもわからんクソソース
そして、仕様を完全把握しているものは不在（いない）

コードを読みやすくする？直す？どーやってなおすんだよ正しい仕様不明なのに
手を入れなければ責任問題は自分にはかからない
正式に仕事として認可されるまでは、さわらずほっとく。こんなのが大量なんだ現実は

まともな仕様書を作るやつは、まともなプログラムも書いている。でもそれは修正等々なんて必要のないソース

リファクタリングの対象となるソースは、前者のクソコード、クソ仕様書のほうだ。正解が不明なのにどうやって直すんだ

**デフォルトの名無しさん** · 2014/09/21(日) 16:39:44.45

>>579
具体的に、クソコードになってしまう
クソ仕様の例を言ってみて。
思いつかないんだよね。

**デフォルトの名無しさん** · 2014/09/21(日) 16:40:37.43

> コードを読みやすくする？直す？どーやってなおすんだよ正しい仕様不明なのに

それは、正しい仕様がなければ、
現在のコードの動きを仕様と考えればいいだけ。

**デフォルトの名無しさん** · 2014/09/21(日) 16:53:43.27

>>581
クソソース1万ステップもよみとーない
句読点のない文章みたいに処理の区切りがまったくないような文体で書かれたソースなんぞ誰が手をつけるか

**デフォルトの名無しさん** · 2014/09/21(日) 16:54:11.17

開発手法が根本からおかしい
リファクタリング以前の問題

**デフォルトの名無しさん** · 2014/09/21(日) 16:55:08.58

>>580
クソ仕様って誰が言った？
クソ仕様「書」って言ってるの。馬鹿なの？あほなの？無能なの？
おまえマ板の無職windowsさんでしょ

**デフォルトの名無しさん** · 2014/09/21(日) 17:12:48.09

>>584
じゃあ、そのクソ仕様書の具体例を
言ってみて。

**デフォルトの名無しさん** · 2014/09/21(日) 17:14:26.03

>>582
> 句読点のない文章みたいに処理の区切りがまったくないような文体で書かれたソースなんぞ誰が手をつけるか

でも、バグがあったら直さないといけないし、
仕様変更、機能追加があったら、手を付けないといけないよね？

**デフォルトの名無しさん** · 2014/09/21(日) 17:24:56.25

うちのソースコードはほとんどが綺麗だから
句読点のない文章みたいに処理の区切りがまったくないような文体で書かれたソース
なんてほとんど無いよ。

そんな汚いソースがたくさんある所は、
手をつける箇所＝リファクタリングするべき場所でしょう。

**デフォルトの名無しさん** · 2014/09/21(日) 17:25:48.01

この板はガセネタをもとにした煽り合いのスレしかないみたいだな

・××と言われているが本当は○○
・こんなことも知らないの？

これが２ちゃんクヲリチーだな

**デフォルトの名無しさん** · 2014/09/21(日) 19:17:45.62

>>585
作ってあげるよ、クソ仕様書

機能：
　俺の心の中にあるいい塩梅で、業務を遂行する処理

以上

これは極端に書いたが、冗談抜きで口頭で仕様をやりとりしたのか
いっさい、仕様書に、当該機能部分の記載がない。

**デフォルトの名無しさん** · 2014/09/21(日) 20:05:10.80

嫌なら辞めれば良いのに
って思う

**デフォルトの名無しさん** · 2014/09/21(日) 20:10:42.85

>>590
どっちにたいして言ってるのかわからんよ、その文では

クソシステムが嫌と言ってる人に言ってるのか、
リファクタリングｗをしないやつはだめだって言ってるやつに言ってるのか

どっちにも解釈できる

**デフォルトの名無しさん** · 2014/09/21(日) 21:44:29.99

クソソース1万ステップ読まなくていいようにするためにリファクタリングするんじゃないか

**デフォルトの名無しさん** · 2014/09/21(日) 21:45:48.38

マーチン・フラウアーのリファクタリング本てruby版とjava版て言語が違うだけで内容は重複してるの？
それとも別の内容が多いから両方買った方がいい？

**デフォルトの名無しさん** · 2014/09/21(日) 22:48:11.16

>>589
それがソースコードとなんの関係があるの？

その曖昧な仕様によって、作られる機能が
曖昧になるだろう。

だが、機能が曖昧になるだけで、
それを実現するコードの読みやすさは関係ないはずだ。

**デフォルトの名無しさん** · 2014/09/21(日) 23:01:02.38

>>593

>>454ででてるリンク先の一覧が内容の比較として使えるよ。
多くは重複しているが、それぞれ片方にしかないものもある。

と思ったが、Java版、新装版がでたんだったか。
旧版、新装版、両方持っているが新装版の方はまだ見てないやｗ

ぱっとみ片方にしかないものは、言語の特徴によるものっぽいから
重複率は同じか少し高くなるだけだろうね。

それでも俺はそのうちRuby版かうけどねｗ

**デフォルトの名無しさん** · 2014/09/22(月) 01:14:38.68

リファクタリング真理教か原理主義かしらんがウゼエ

**デフォルトの名無しさん** · 2014/09/22(月) 06:10:50.31

>>596
リファクタリングっつって真理教とか妄想するお前が一番オームくさい。

リファクタリング大勝利！！！

**デフォルトの名無しさん** · 2014/09/22(月) 06:22:02.06

Aさんがリファクタリングしました。
Bさんが読みましたAさんのリファクタリングがクソです。Bさん直しました。
Cさん見ましたBさんのソースコードクソです。Cさん直しました。
永遠と、、、、、

個人の力量、感性によって結果が大きく作用してしまうものは、学術的工学的な基本がないからどうにもならない。

**デフォルトの名無しさん** · 2014/09/22(月) 08:09:32.93

普通は、

Aさんがリファクタリングしました。
Bさんが読みましたAさんのリファクタリングがクソです。Bさん直しました。
Cさん見ましたBさんのソースコードクソです。Cさん直しました。

Cさんリファクタリング大勝利！！！・・・でしょ？

**デフォルトの名無しさん** · 2014/09/22(月) 08:34:50.21

Class レイザラモンFoo Extends Object
Public サルinit(void);
Public　クエリFoo(String SQL文はクソ);
Protect ないですFoo(void);
Destractor システム緊急停止オペ呼んでくさいFoo(void);
End Class

こんな仕様のクラスを業務システムのスーパークラスにしてあとは全て継承
という仕様は誰だって疑問を持つと思うんだが、そこを直すのがリファクタリング。

リファクタリング大勝利！！！

**デフォルトの名無しさん** · 2014/09/22(月) 11:41:29.79

>>598
それリファクタリング関係ないしｗ

なぜなら、俺が作ったコードを誰かが編集した時に、
変な修正の仕方をされることがあるから。

それを直すのがリファクタリングなわけ。

**デフォルトの名無しさん** · 2014/09/22(月) 12:15:28.88

>>599
再びAさんが担当した時、Cさんのコードを読めず仕事になりませんでした。
Cさんリファクタリング大勝利！！！

**デフォルトの名無しさん** · 2014/09/22(月) 13:03:42.75

優れたの方コードを採用すればいいだけだろ。

**デフォルトの名無しさん** · 2014/09/22(月) 13:12:39.94

個人の知識の違いによってソースの見え方が違うからな
アホが天才のコードを見ると何じゃこりゃってなるし、逆もまた然り

**デフォルトの名無しさん** · 2014/09/22(月) 13:51:20.75

知識って・・・そんなの一機能5分もあれば
知らない状態から理解できるだろ。

どんだけ素人なんだ？

つーか知らない時点で、判断できないわけで
議論からはお役御免なわけだが

**デフォルトの名無しさん** · 2014/09/22(月) 13:53:03.85

それに、知らないなら何をやっているか
理解できないわけで、リファクタリング出来るわけがないな。

**デフォルトの名無しさん** · 2014/09/22(月) 17:49:10.78

おれはやらな～いw

これでリファクタリングは崩壊するwww

**デフォルトの名無しさん** · 2014/09/22(月) 19:42:14.76

アホみたいな話だが結局>>598が真理だよな。
リファクタリングする人は、自分のコードもいずれリファクタリングされる事を受けいれねばならん。

**デフォルトの名無しさん** · 2014/09/22(月) 20:32:19.28

リファクタリングはただのコード修正と思ってもらってかまわない

**デフォルトの名無しさん** · 2014/09/22(月) 20:44:04.47

リファクタリングするよ派はここでなにしてんの？しないよ派とじゃれあってるのが楽しいの？
やり合ってる本人は議論してるつもりかもしれないけど、外から見てるとしょうもなさすぎてｗｗ

**デフォルトの名無しさん** · 2014/09/22(月) 21:11:07.07

>>607
それは単に、プログラム？なにそれおいしいの？
動けばいいでしょ？って言ってるだけでしょう？
そんな会社はリファクタリング以前に
仕事が終わってる。