0001仕様書無しさん
2017/01/30(月) 23:33:00.67http://www.publickey1.jp/blog/17/post_262.html
探検
マーチン・ファウラーのプログラミングバンザイ [無断転載禁止]©2ch.net
■ このスレッドは過去ログ倉庫に格納されています
http://www.publickey1.jp/blog/17/post_262.html
0002仕様書無しさん
2017/01/30(月) 23:34:25.38それ以外にも再利用するかどうかを基本に、二度以上繰り返す処理であれば関数にすべきであり、
そうでなければそのままインラインにしておく、というのもある。
0003仕様書無しさん
2017/01/30(月) 23:35:08.45もしも、なにをしているのか理解するのに努力しなければならないコードの一部分があったとしたら、
そこを関数として切り出し、“what”に続いてその名前を付けてしまうのである。
0004仕様書無しさん
2017/01/30(月) 23:35:31.30目に飛び込んでくるだろうし、その関数がやろうとしていること、
すなわち関数本体に注意を払う必要はなくなるだろう。
私はこの規律を採用してから短い関数を書くようなった、典型的には数行だ[2]。
コードが6行を超えると、もう私にとっては関数の匂いを感じ始めるし、
1行のコードからなる関数でさえ珍しくない[3]。
0005仕様書無しさん
2017/01/30(月) 23:36:10.28ケント・ベックがオリジナルのSmalltalkシステムにおける実例に
よって示してくれたものだった。当時のSmalltalkはモノクロディスプレイで
動作していたので、テキストやグラフィックをハイライト表示したいときには、
画像を反転させることで実現する。そしてSmalltalkの画像のクラスには
この“Highlight”（ハイライト）と呼ばれるメソッドがあり、
この実装は単に“reverse”メソッドを呼び出すものだったのだ[4]。
この名称は実装よりも長いのたが、コードの意図とコードの
実装のあいだには大きな距離があったから、そのことは大きな問題にならなかった。
0006仕様書無しさん
2017/01/30(月) 23:36:44.15名前付けには注意を払う必要がある。 これには練習が必要だけれど、
ひとたび上達したならばこの方法でコードを高度に自己文書化できる。
より大きいスケールで見れば、関数は物語のように読むことができ、
読むものにとって必要なときにどの関数を詳しく読み込んでいくか選択できるようになるのだ。
0007仕様書無しさん
2017/01/31(火) 19:54:01.09ちょっと抽象的すぎるのかな
0008仕様書無しさん
2017/01/31(火) 20:44:01.220009仕様書無しさん
2017/01/31(火) 21:50:05.136行とか非現実的、現場では100行の関数を即座に読み解く能力がないと戦力にならない
理想を描いて夢物語を語ってるだけの無能コンサルなんて誰も相手にしない
0010仕様書無しさん
2017/01/31(火) 22:32:51.470011仕様書無しさん
2017/02/01(水) 05:42:36.03> 6行とか非現実的
それは能力がないからだって自覚はあるかい？
0012仕様書無しさん
2017/02/01(水) 05:43:02.65お前なんか足元にも及ばない優秀な技術者だからね
0013仕様書無しさん
2017/02/01(水) 09:08:48.09で調べてみたら 9万個のメソッドが 56万行で書かれていて確かに平均 6行程度だった
| linesOfCodes |
linesOfCodes := CompiledMethod allInstances collect: #linesOfCode.
^{#count->linesOfCodes size. #total->linesOfCodes sum. #average->linesOfCodes average asFloat}
"=> {#count->93474. #total->562876. #average->6.021738665297302} "
0014仕様書無しさん
2017/02/01(水) 09:42:37.06まあ平均10行ぐらいかな、これを普通だろと思う人と
ありえないと思う人で、プログラマと非プログラマに別れると思うわ
もちろん平均10行が普通っていうのは、
頑張ればできるけどさぁ、でもそうすると逆にわかりづらくなるしぃ
みたいな事を言ってる人は含めない。
平均10行を普通に書いてますっていう意味
もちろん俺も平均10行なら普通だな。
普通に書けば自然とそうなる。どうしようもないｗ
0015仕様書無しさん
2017/02/01(水) 11:03:51.90| roundedLinesOfCodes |
roundedLinesOfCodes := ((CompiledMethod allInstances collect: #linesOfCode) roundTo: 10) asBag.
^roundedLinesOfCodes size -> roundedLinesOfCodes sortedCounts
"=> 93468→{56695→0. 29798→10. 4712→20. 1257→30. 474→40.
218→50. 125→60. 53→70. 49→80. 18→90. 12→100. 11→110.
10→140. 6→120. 5→130. 4→150. 4→180. 3→200. 2→210. 2→300.
1→160. 1→170. 1→190. 1→230. 1→290. 1→400. 1→410. 1→510. 1→1300. 1→1700} "
6割くらいのメソッドが 4行以下、
3割くらいが 5行〜14行程度、
以降、20行程度、30行程度、40行程度、50行程度、…と減っていく
ちなみに 1000行を超えるメソッド 2つは、いずれもコンスタントメソッドだった
CompiledMethod allInstances select: [:meth | meth linesOfCode > 1000] thenCollect: #selector
"=> #(#iso3166Codes #tigerPoints) "
0016仕様書無しさん
2017/02/01(水) 13:50:16.230017仕様書無しさん
2017/02/01(水) 19:43:18.10どういう実案件で使われるの？
0018仕様書無しさん
2017/02/01(水) 22:10:35.80ソースコード見てないけど
その長い行っていうのは条件分岐やループが少ない
平坦なコードになってると思う
0019仕様書無しさん
2017/02/01(水) 22:32:53.49逆にSmalltalkを使用言語に選べる案件であれば大抵のものは作れるよ
http://pharo.org/success
0020仕様書無しさん
2017/02/01(水) 22:47:20.91ごめん、言いたいことが分からない
「その」は何を指しているの？
0021仕様書無しさん
2017/02/02(木) 00:56:39.480022仕様書無しさん
2017/02/02(木) 00:58:29.44で、実績は？
何の根拠もなく信奉してるだけだろ
目覚ませよ、Windows作ったゲイツの方が100倍優秀
0023仕様書無しさん
2017/02/02(木) 01:00:53.0210行とか遊びじゃねんだぞｗ入門書に載ってるコード打ち込んで金もらえるならいいけどな
0024仕様書無しさん
2017/02/02(木) 01:43:15.05といってるように思える。
なぜわざわざ複雑にするのか？
0025仕様書無しさん
2017/02/02(木) 07:10:56.30何行以上だと「長い」の？
0026仕様書無しさん
2017/02/02(木) 07:35:11.71そんな餌に俺様が、クマ〜
0027仕様書無しさん
2017/02/02(木) 07:39:52.45ゲイツはWindowsなんか作ってないし
ジョブズがGUIやマウスを発明した！とか言いだしかねないリテラシーの低さだな
0028仕様書無しさん
2017/02/02(木) 07:49:35.03行数短ければいいってもんでもない。
横長は見づらいし保守しにくいし
ケアレスミスの元。
例えば"{"を行の右の方に書くと見づらい。
また、処理1行ならカッコをはぶけるからといって
省いてバグ出したのはapple。
0029仕様書無しさん
2017/02/02(木) 07:51:50.68一貫性なくて見づらい。
長すぎるのは勿論ダメ。
0030仕様書無しさん
2017/02/02(木) 08:10:15.400031仕様書無しさん
2017/02/02(木) 08:37:54.84ファウラーはそんなことは言っていないし、Smalltalk のコードもそんなふうには書かれていない
0032仕様書無しさん
2017/02/02(木) 09:38:36.82ぽかーんとしか言いようがないなｗ
なんでお前、行数を減らすのに
そんな小手先のテクニックしか思いつかないの？
そんなの本質的な解決方法じゃないよね
普通にまともなコードを書けば行数は
このぐらいになると言う話をしているのに、
お前は、行数を短くする馬鹿なコードを見せてやる！
どうだ俺が書いた馬鹿なコードは馬鹿だろう？
と言ってるだけじゃねーか
> 省いてバグ出したのはapple。
あのバグが有るコードも長かったよねｗ
0033仕様書無しさん
2017/02/02(木) 20:02:07.38それをくくりだすだけでもだいぶ見やすくなるよな
0034仕様書無しさん
2017/02/03(金) 01:12:36.41・単一責務を守る(最重要)
・正確な名前を付ける
・ドキュメンテーションコメント(JavaDoc)をきちんと書く
これに従えば、だいたい10行以下(徹底すれば5行以下)で収まるだろ。
メソッド長いヤツは、機能の分割と抽出ができてない。
長いのを処理できて一人前とか言ってるヤツいるけど、そんなのは向上心がないことに対する言い訳だからな。
昔行った現場(Java)で、
最大クラス:1万行
最長メソッド:700行
最深ネスト:12段階
最大循環的複雑度(メソッド):200
ってのがあったけど、
保守容易性指数5以下のクラス続出で、案の定炎上しまくってたからなｗｗｗ
0035仕様書無しさん
2017/02/03(金) 19:42:42.33ちゃんと実装してたから炎上ですんだんだろ
10行以下にしたら爆発してたぞ
欲を言えばメソッドが700行というのは短いな
実務では3000程度を目安にしたがいい
0036仕様書無しさん
2017/02/03(金) 19:49:23.10言ったことを鵜呑みにして聖書のように大事にするだろ
そうやって中級者は必要以上に大量のメソッドを作りあげて
あっち見たりこっち見たりしなきゃいけなくなる
初級者よりも性質が悪い
福島原発の汚染廃棄物を全国にばら蒔いて大丈夫だ問題ないと言ってるようなもの
アンダーコントロールというのは集約によってなされるものだ
聖書を捨てて1000行以上のメソッドに神の存在を感じるんだ
0037仕様書無しさん
2017/02/03(金) 21:03:46.83炎上って最悪の状況だぞ
お前にとってはいつものことなんだろうが。
いつもお前、長ったらしいコード書いてるだろｗ
0038>>34
2017/02/03(金) 23:54:21.53敢えてネタにマジレスするけどさぁ、お前、
循環的複雑度200のメソッドとか、
保守容易性指数3(笑)のクラスとか、
実際に見たことあんの？
誰一人として処理フローを脳内で追えないからなｗｗｗ
見てそんなこと言ってるんなら、本当に頭沸いてるとしかいいようがないぞ。
0039仕様書無しさん
2017/02/03(金) 23:58:39.72複雑なものをメンテできる俺ってすごくね？
と勘違いしているSI系プログラマ
0040仕様書無しさん
2017/02/04(土) 00:43:56.27入門書レベルのものなら何行でもいいだろうが
業務レベルのものなら1000行くらい書き下せないと
現実的に開発についてけないぞ
0041仕様書無しさん
2017/02/04(土) 00:44:37.45青い銀行は炎上してるんだぞ
お前らのせいなんだぞ
0042>>34
2017/02/04(土) 00:56:48.60だからさぁ、「現実的に開発についていけない」ってのが、勉強を放棄した証拠なんだよね。
「コードの質を高めて開発効率を上げる」というのが、本来あるべき姿ね。
0043仕様書無しさん
2017/02/04(土) 01:02:02.47営利企業での実践的な開発なんだから
必然的に開発効率は高いしコードの品質も高い
業務要件が複雑だから門外漢の人間が見てわかるものじゃないんだが
コードの書き方を変えればアホな自分にもわかるようになるんだ
自分は勉強が足りないんだと思ってる意識高い系のニワカが
一生大規模開発には携われないだろうな
ささいなバグで何百億が吹き飛ぶような世界だぞわかってんのか
0044仕様書無しさん
2017/02/04(土) 01:07:50.33厳密な業務要件に関わる仕事をしたことがない雑魚だってこと
海外は適当でいい、バグだらけでもユーザはとやかく言わない
そういう文化でしか仕事したことのないなんちゃってコンサルの妄言を
真に受けてメソッドを6行にしようとか狂気の沙汰
0045>>34
2017/02/04(土) 01:12:45.62はい？
1000行以上もあって「門外漢の人間が見てわかるものじゃない」のに、「コードの品質も高い」んですか？
メソッドの頭であなたが使ったローカル変数を、誰か別の人がメソッド末尾で触ってしまったら、あなたの作業は全て無意味ですよ。そのローカル変数が参照型の場合は特に。
そんなことだから収集つかなくなるんですよ、みずほさんみたいに(笑)
0046仕様書無しさん
2017/02/04(土) 01:18:14.50ローカル変数のスコープは小さくするのが基本
批判しやすい状況を想定して批判するのはバカがすること
お前の脳ミソのキャパシティが疑われる
0047仕様書無しさん
2017/02/04(土) 01:24:08.48水素水も飲んでそう、オブジェクト指向はすごくこれから人間を救っていくと思ってそう
ちげえから世の中そんなに甘くねえから、現実的なマジモンのプログラムっつーのは
泥臭いコードの中に存在するものだから
0048仕様書無しさん
2017/02/04(土) 01:29:22.91本格的なプログラムに片足の小指の先っちょでも突っ込んでから語ってほしいわ
オブジェクト詐欺のコンサルハゲに騙されるような貧弱プログラマで一生を終えたいのか
0049仕様書無しさん
2017/02/04(土) 01:44:21.91ミーハー根性丸出しでプログラマを名乗るのをやめていただきたい
プログラマなら自分で試して判断してもらいたい
ファウラーの戯言を正しいと仮定したときどういう結論が導かれるか
ファウラーのご託宣を間違いだと仮定したときどういう結論が導かれるか
実際に試して検証しないといけない
お前らがやってるのは小保方以下
ファウラーが言ってたやったと言ってるようなもの
まったく現実を見れてない
どっかの誰かが作ったなにかを盗んできて割烹着着て発表してるようなもの
0050仕様書無しさん
2017/02/04(土) 01:56:15.97SIにもいたことがあるが勉強になったぞ、お前らも食わず嫌い止めて
ハゲコンサルに騙されないだけの知恵を獲得したがいいぞ
0051仕様書無しさん
2017/02/04(土) 02:24:17.96俺が正しいってことなんだよ
体中から自信が溢れてくる
0052仕様書無しさん
2017/02/04(土) 03:18:18.72有名なファウラーが言ってたからそれに反発する
俺クールっていう考え？ｗ
0053仕様書無しさん
2017/02/04(土) 03:43:47.21ファウラーより俺の方が優れているという確信がある
自分の頭で考えない奴を俺はプログラマとは認めない
0054仕様書無しさん
2017/02/04(土) 03:59:13.81確信じゃなくて根拠を知りたいんだがｗ
0055仕様書無しさん
2017/02/04(土) 04:07:26.54それ以上の説明がいるのかい
0056仕様書無しさん
2017/02/04(土) 04:17:56.550057仕様書無しさん
2017/02/04(土) 04:37:29.600058仕様書無しさん
2017/02/04(土) 04:39:46.01違うけど、俺がクールなのは否定できない
0059仕様書無しさん
2017/02/04(土) 04:43:40.57バカかお前、頭の中にひなあられでも詰まってんのか
自信があるってことは自信を持つだけの人生経験を
積んできたってことは自明だろうが
0060仕様書無しさん
2017/02/04(土) 04:48:02.91お前らに俺の余りあるクールさを分けてあげたいくらいだ
悪徳コンサルの手からお前らを救いたい
0061仕様書無しさん
2017/02/04(土) 05:01:57.71頭がクール
クール・クール・クルクルパー
0062仕様書無しさん
2017/02/04(土) 12:13:28.24ダニング=クルーガー効果って知っているかい？
■ このスレッドは過去ログ倉庫に格納されています
ニュース
- 【サッカー】UEFA-CL決勝戦 ドルトムント×レアル・マドリード [久太郎★]
- 「異性を家に入れるな」男性をアパートの部屋に入れた女性（20代）に大家の男（79）が立腹 剪定ばさみで脅迫した疑いで逮捕 [モフモフちゃん★]
- 「食べるものに困り…」スマホひったくり ベトナム人逮捕 東京・足立区 [モフモフちゃん★]
- どなたであろうと、この中国の人を捕えて引き渡してくださった方に賞金500万円をその場でお渡しします [モフモフちゃん★]
- 粗品 宮迫博之からの「テレビ出てた時の俺を超えてから言え！」「お前、アメトークみたいな番組作ったっけ？」との挑発を一蹴★2 [冬月記者★]
- 交番横のコンビニで強盗未遂 18歳の男逮捕 [モフモフちゃん★]
- 【WOWOW】UEFAチャンピオンズリーグ・ヨーロッパリーグ 決勝トーナメント ★38
- 【ATP】テニス総合実況スレ2024 Part 97【WTA】
- 巨専】
- はません
- とらせん もうええやろ
- かもめせん な60
- 🏡
- CDショップ店主「CDは終わりを感じる」ついに1週間で客ゼロの大惨事 [462275543]
- なんで猛虎弁つかうと怒られるの？
- 【急募】ネット「早口で喋る人は頭が良い？！、、、せやッ！」こういう奴最近多くね？ [859851824]
- このプラモデルの塗装済み完成品ヤフオクで買うとしたらいくらまで出せる？
- 小学生さん、作文でスマートフォンになりたいと書いてしまうｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗｗ