マーチン・ファウラーのプログラミングバンザイ [無断転載禁止]©2ch.net
■ このスレッドは過去ログ倉庫に格納されています
関数の適切な長さとは? マーチン・ファウラー氏は、長さより意図と実装の分離、そしてよい関数名が重要だと指摘
http://www.publickey1.jp/blog/17/post_262.html いくつかのガイドラインでは、1画面におさまらなくなったら、とあるし[1]、
それ以外にも再利用するかどうかを基本に、二度以上繰り返す処理であれば関数にすべきであり、
そうでなければそのままインラインにしておく、というのもある。 しかし私にとってもっとも理にかなっていると思われたのは、意図と実装の分離というものだ。
もしも、なにをしているのか理解するのに努力しなければならないコードの一部分があったとしたら、
そこを関数として切り出し、“what”に続いてその名前を付けてしまうのである。 こうすれば、もう一度そのコードを読んだときに、そのコードの目的はすぐさま
目に飛び込んでくるだろうし、その関数がやろうとしていること、
すなわち関数本体に注意を払う必要はなくなるだろう。
私はこの規律を採用してから短い関数を書くようなった、典型的には数行だ[2]。
コードが6行を超えると、もう私にとっては関数の匂いを感じ始めるし、
1行のコードからなる関数でさえ珍しくない[3]。 実のところコードのサイズが重要なのではない、という考えは、
ケント・ベックがオリジナルのSmalltalkシステムにおける実例に
よって示してくれたものだった。当時のSmalltalkはモノクロディスプレイで
動作していたので、テキストやグラフィックをハイライト表示したいときには、
画像を反転させることで実現する。そしてSmalltalkの画像のクラスには
この“Highlight”(ハイライト)と呼ばれるメソッドがあり、
この実装は単に“reverse”メソッドを呼び出すものだったのだ[4]。
この名称は実装よりも長いのたが、コードの意図とコードの
実装のあいだには大きな距離があったから、そのことは大きな問題にならなかった。 このように小さな関数は名前が良い場合にのみ機能するので、
名前付けには注意を払う必要がある。 これには練習が必要だけれど、
ひとたび上達したならばこの方法でコードを高度に自己文書化できる。
より大きいスケールで見れば、関数は物語のように読むことができ、
読むものにとって必要なときにどの関数を詳しく読み込んでいくか選択できるようになるのだ。 良いこと書いてるのに伸びないんだな
ちょっと抽象的すぎるのかな マジレスするとプログラマとしては実績のないコンサル野郎が言ったことを真に受けるほどプログラマはバカじゃない
6行とか非現実的、現場では100行の関数を即座に読み解く能力がないと戦力にならない
理想を描いて夢物語を語ってるだけの無能コンサルなんて誰も相手にしない Smalltalk をよくしらないとぴんときにくい話かと >>9
> 6行とか非現実的
それは能力がないからだって自覚はあるかい? あと、マーチン・ファウラーはコンサルではない。
お前なんか足元にも及ばない優秀な技術者だからね ほんとかよと思って Pharo Smalltalk http://pharo.org/
で調べてみたら 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} " 平均6行はトップクラスの技術者だろうけど
まあ平均10行ぐらいかな、これを普通だろと思う人と
ありえないと思う人で、プログラマと非プログラマに別れると思うわ
もちろん平均10行が普通っていうのは、
頑張ればできるけどさぁ、でもそうすると逆にわかりづらくなるしぃ
みたいな事を言ってる人は含めない。
平均10行を普通に書いてますっていう意味
もちろん俺も平均10行なら普通だな。
普通に書けば自然とそうなる。どうしようもないw >>13 の内訳も調べてみた。
| 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) " Smalltalkって、Smalltalkの環境そのものや遊びを除けば
どういう実案件で使われるの? >>15
ソースコード見てないけど
その長い行っていうのは条件分岐やループが少ない
平坦なコードになってると思う Smalltalkの案件や求人は国内外問わずたまにしか見かけないけど
逆にSmalltalkを使用言語に選べる案件であれば大抵のものは作れるよ
http://pharo.org/success >>18
ごめん、言いたいことが分からない
「その」は何を指しているの? >>12
で、実績は?
何の根拠もなく信奉してるだけだろ
目覚ませよ、Windows作ったゲイツの方が100倍優秀 業務では1000行のコードぼんぼん作れるくらいじゃないとやってけないぞ
10行とか遊びじゃねんだぞw入門書に載ってるコード打ち込んで金もらえるならいいけどな 業務ではシンプルなものを複雑にしないとやってけないぞ
といってるように思える。
なぜわざわざ複雑にするのか? >>22
ゲイツはWindowsなんか作ってないし
ジョブズがGUIやマウスを発明した!とか言いだしかねないリテラシーの低さだな >>14
行数短ければいいってもんでもない。
横長は見づらいし保守しにくいし
ケアレスミスの元。
例えば"{"を行の右の方に書くと見づらい。
また、処理1行ならカッコをはぶけるからといって
省いてバグ出したのはapple。 returnだけだからと言って横にreturn書いてるのとか
一貫性なくて見づらい。
長すぎるのは勿論ダメ。 無理に行数減らそうとしてコメント行や空行削ってメソッド数がメチャクチャ増えるのも嫌な感じだな この文脈で、無理に行数を減らすとか横に長いとかいう方向にしか頭が働かない時点で駄目だろ
ファウラーはそんなことは言っていないし、Smalltalk のコードもそんなふうには書かれていない >>28
ぽかーんとしか言いようがないなw
なんでお前、行数を減らすのに
そんな小手先のテクニックしか思いつかないの?
そんなの本質的な解決方法じゃないよね
普通にまともなコードを書けば行数は
このぐらいになると言う話をしているのに、
お前は、行数を短くする馬鹿なコードを見せてやる!
どうだ俺が書いた馬鹿なコードは馬鹿だろう?
と言ってるだけじゃねーか
> 省いてバグ出したのはapple。
あのバグが有るコードも長かったよねw だいたいうん百行ある長いメソッドは、何個かコメントで段落わけるようなことしてるから
それをくくりだすだけでもだいぶ見やすくなるよな Faulerは俺も読んでるけど、かなりの良書だぞ。
・単一責務を守る(最重要)
・正確な名前を付ける
・ドキュメンテーションコメント(JavaDoc)をきちんと書く
これに従えば、だいたい10行以下(徹底すれば5行以下)で収まるだろ。
メソッド長いヤツは、機能の分割と抽出ができてない。
長いのを処理できて一人前とか言ってるヤツいるけど、そんなのは向上心がないことに対する言い訳だからな。
昔行った現場(Java)で、
最大クラス:1万行
最長メソッド:700行
最深ネスト:12段階
最大循環的複雑度(メソッド):200
ってのがあったけど、
保守容易性指数5以下のクラス続出で、案の定炎上しまくってたからなwww >>34
ちゃんと実装してたから炎上ですんだんだろ
10行以下にしたら爆発してたぞ
欲を言えばメソッドが700行というのは短いな
実務では3000程度を目安にしたがいい お前らは経験が圧倒的に不足してるから有名な人が
言ったことを鵜呑みにして聖書のように大事にするだろ
そうやって中級者は必要以上に大量のメソッドを作りあげて
あっち見たりこっち見たりしなきゃいけなくなる
初級者よりも性質が悪い
福島原発の汚染廃棄物を全国にばら蒔いて大丈夫だ問題ないと言ってるようなもの
アンダーコントロールというのは集約によってなされるものだ
聖書を捨てて1000行以上のメソッドに神の存在を感じるんだ >>35
炎上って最悪の状況だぞ
お前にとってはいつものことなんだろうが。
いつもお前、長ったらしいコード書いてるだろw >>35
敢えてネタにマジレスするけどさぁ、お前、
循環的複雑度200のメソッドとか、
保守容易性指数3(笑)のクラスとか、
実際に見たことあんの?
誰一人として処理フローを脳内で追えないからなwww
見てそんなこと言ってるんなら、本当に頭沸いてるとしかいいようがないぞ。 かんたんな問題を複雑にして、
複雑なものをメンテできる俺ってすごくね?
と勘違いしているSI系プログラマ SIをバカにするのは大規模な開発をやったことないニワカだしなあ
入門書レベルのものなら何行でもいいだろうが
業務レベルのものなら1000行くらい書き下せないと
現実的に開発についてけないぞ 入門書読んでわかってきになってるニワカどものせいで
青い銀行は炎上してるんだぞ
お前らのせいなんだぞ >>40
だからさぁ、「現実的に開発についていけない」ってのが、勉強を放棄した証拠なんだよね。
「コードの質を高めて開発効率を上げる」というのが、本来あるべき姿ね。 >>42
営利企業での実践的な開発なんだから
必然的に開発効率は高いしコードの品質も高い
業務要件が複雑だから門外漢の人間が見てわかるものじゃないんだが
コードの書き方を変えればアホな自分にもわかるようになるんだ
自分は勉強が足りないんだと思ってる意識高い系のニワカが
一生大規模開発には携われないだろうな
ささいなバグで何百億が吹き飛ぶような世界だぞわかってんのか ファウラーの何がマズいかって日本の複雑な金融商品や
厳密な業務要件に関わる仕事をしたことがない雑魚だってこと
海外は適当でいい、バグだらけでもユーザはとやかく言わない
そういう文化でしか仕事したことのないなんちゃってコンサルの妄言を
真に受けてメソッドを6行にしようとか狂気の沙汰 >>43
はい?
1000行以上もあって「門外漢の人間が見てわかるものじゃない」のに、「コードの品質も高い」んですか?
メソッドの頭であなたが使ったローカル変数を、誰か別の人がメソッド末尾で触ってしまったら、あなたの作業は全て無意味ですよ。そのローカル変数が参照型の場合は特に。
そんなことだから収集つかなくなるんですよ、みずほさんみたいに(笑) >>45
ローカル変数のスコープは小さくするのが基本
批判しやすい状況を想定して批判するのはバカがすること
お前の脳ミソのキャパシティが疑われる 結局この程度のバカがハゲコンサルのお花畑詐欺に引っかかってるんだよ
水素水も飲んでそう、オブジェクト指向はすごくこれから人間を救っていくと思ってそう
ちげえから世の中そんなに甘くねえから、現実的なマジモンのプログラムっつーのは
泥臭いコードの中に存在するものだから 昨今の雑魚厨どもはコンパイラも書いたことがない分際でプログラムを語りやがるからいけねえわ
本格的なプログラムに片足の小指の先っちょでも突っ込んでから語ってほしいわ
オブジェクト詐欺のコンサルハゲに騙されるような貧弱プログラマで一生を終えたいのか 有名なファウラーが言ってたからそれに乗っかっておこうという
ミーハー根性丸出しでプログラマを名乗るのをやめていただきたい
プログラマなら自分で試して判断してもらいたい
ファウラーの戯言を正しいと仮定したときどういう結論が導かれるか
ファウラーのご託宣を間違いだと仮定したときどういう結論が導かれるか
実際に試して検証しないといけない
お前らがやってるのは小保方以下
ファウラーが言ってたやったと言ってるようなもの
まったく現実を見れてない
どっかの誰かが作ったなにかを盗んできて割烹着着て発表してるようなもの まあ俺は今は製薬系の大手企業でプログラマやってるから白衣着てプログラム書いてっけどな
SIにもいたことがあるが勉強になったぞ、お前らも食わず嫌い止めて
ハゲコンサルに騙されないだけの知恵を獲得したがいいぞ このスレで俺が一番声がでかい
俺が正しいってことなんだよ
体中から自信が溢れてくる >>49
有名なファウラーが言ってたからそれに反発する
俺クールっていう考え?w 違うね
ファウラーより俺の方が優れているという確信がある
自分の頭で考えない奴を俺はプログラマとは認めない > ファウラーより俺の方が優れているという確信がある
確信じゃなくて根拠を知りたいんだがw 俺の中にみなぎるこの自信が根拠だね
それ以上の説明がいるのかい >>56
バカかお前、頭の中にひなあられでも詰まってんのか
自信があるってことは自信を持つだけの人生経験を
積んできたってことは自明だろうが 自信に満ちた俺の熱いハートは外からみれば逆にクールに見えるだろうな
お前らに俺の余りあるクールさを分けてあげたいくらいだ
悪徳コンサルの手からお前らを救いたい 頭がクール
頭がクール
クール・クール・クルクルパー >59
ダニング=クルーガー効果って知っているかい? >>62
大規模開発もコンパイラの実装もやったことがない
雑魚がクソ禿げコンサルの戯言を金言として
無根拠に自分が優れたプログラマだと思い込むことでしょうか?わかりませんはい論破 1000行ひたすらループやら分岐やらが続いてるコードと
意味ある単位で、何してるのか自明な名前の関数にバラされてるコードだったら
考えるまでもなく後者のほうがわかりやすいな >>15
10000行超えててもコンパイルできるの?CompiledMethodの制限超えそう >>67
1000行だった。あとコメント抜いた行数? >>68
#linesOfCode のソースはこんな感じ。コメントは含むが空行は除く。
CompiledMethod >> linesOfCode
"An approximate measure of lines of code.
Includes comments, but excludes empty lines."
| lines |
lines := 0.
self getSource asString lineIndicesDo: [:start :endWithoutDelimiters :end |
endWithoutDelimiters > start ifTrue: [lines := lines+1]].
^lines >>69
念のためこの #linesOfCode というメソッド自身の linesOfCode は 8 を返す
(CompiledMethod >> #linesOfCode) linesOfCode "=> 8 " 自分をバカだと自覚していないバカが最も成長しない
熱意のあるバカは企業とって最も足手まといになる 英語できない日本人は、ちゃんとした名前の関数やメソッドを大量に作る心理的ハードル高いし
余計にダラダラ書いちゃいそう >>5の"highlight"メソッドの中身が"revese"というのは
ダイレクトにsmalltalkの流れを汲んでるObjective-Cなどをやってると
『クラスに命令(メッセージ)を送って動作させる』という思想が徹底してるから
ロボットに命令したら終わり(後は任せて追わなくていいように作れ)という
投げっぱなし前提思想と"人間が"何を自分で命令したか
後で明確にわかるように命令コマンド名を決めておけという
わかりやすい前提に基づいているのだけど、
まったく異なる思想でクラスや関数を捉えてる言語の側に住んでいると
「A=B=Cなら全部Aと書けばいいではないか!」で意図そのものが伝わらない。
そもそも自分がこいつに何をさせたか?という状態とコマンドの記述の話なので。 > まったく異なる思想でクラスや関数を捉えてる言語の側に住んでいると
どの言語のどの関数の話? smalltalkって見た目が最高にキモイな(褒め言葉) >>31
そもそも、>>1に長さが問題なんじゃないって書いてあるのにね。 長さが問題だっていうのなら単に関数を細切れにして
短くすれば問題解決になるはずだが
もちろんそんなことはありえないしな
ただ意図と実装の分離をちゃんと行えば
自然と関数は短くなる
短いからと言って良いと言い切れるわけではないが
長い関数であれば殆どの場合ダメと言っていいということ 古い言語だと短いけど訳がわからないコードもあるよね
昔のプログラマはオナニストばかりだったからコードが臭い >>80
たんに英語が読めないだけとかそういうことじゃなく? CやC++ができる人がObjective-Cのコード見て
「なにがなんだかわからん!変態だ!」って言ってたのといっしょじゃね?
そのまま永久に「Objective-Cは変態だかダメ!」って言い続けてたし。 長さが絶対的基準ではないだろうが長すぎるのはやはり問題。 Smalltalkのメッセージ式はお世辞にも短いとはいえないが、英語として読むにはぎり許容範囲
ただObjective-Cはそれに型情報も入れてさらに英語としても成り立たせようとする慣習があって長すぎ
それなのにXcodeにはつい最近まで補完がなかったってんだから信じられない >>82
Cしか知らないから、ぱっと見 >> と ^ がわからん
|a|は引数かローカル変数に見えた >>87
>> はそのメソッドが属するクラスを示すのに慣習的に用いられるものでメソッド定義時には使いません。
^ はリターンで Smalltalk 唯一の制御構文でもあります。
| a | がローカル変数宣言というのは合っています。
Smalltalk の式の文法は receiver message で、message は引数がないときはメソッド名そのまま、
引数があるときは、メソッド名に引数の数だけ含まれるコロンの場所で分断して、そこに引数を挿入する記法をとります。
たとえば、普通の言語(ただしメソッド名にコロンを含むことができると仮定)で obj.x:y:z:(a, b, c) という式は
obj x: a y: b z: c となります。ちなみに、Smalltalk ではメソッド名をメッセージセレクタ、あるいは短くセレクタとも呼びます。
メソッド定義は メッセージパーターン テンポラリ変数宣言 式 で構成されます。
メッセージパターンというのは、前述の式の message 部分と同じ記法でセレクタと仮引数を宣言するための部分です。
したがって、x: a y: b z: c | t1 t2 | t1 := a + b. t2 := c. ^t1 * t2 は、x:y:y: というメソッドを仮引数 a b c で定義したものになります。
このメソッドを定義を文字列で、たとえば compile: メソッドの引数としてクラスに送ってやることで、メソッドを定義できます。
Integer compile: 'x: a y: b z: c | t1 t2 | t1 := a + b. t2 := c. ^t1 * t2'.
1 x: 2 y: 3 z: 4 "=> 20 "
[:a :b | a + b] などというのはブロックと呼ばれる無名関数で、
引数はコロンを頭に置いたブロック引数宣言と処理内容を表わす式を | で区切って [ ] で括って記述します。
ブロックは第一級オブジェクトなので、代入やメソッドの引数にすることができます。 1000行の処理がザラとか言ってるやつ、registとか書いてそう。 誰でも簡単にパソコン1台で稼げる方法など
参考までに、
⇒ 『宮本のゴウリエセレレ』 というブログで見ることができるらしいです。
グーグル検索⇒『宮本のゴウリエセレレ』
CMRDMROXHN ☆ 日本の、改憲を行いましょう。現在、衆議員と参議院の
両院で、改憲議員が3分の2を超えております。
『憲法改正国民投票法』、でググってみてください。国会の発議は
すでに可能です。平和は勝ち取るものです。お願い致します。☆☆ ttps://martinfowler.com/articles/201803-refactoring-2nd-ed.html とても簡単な自宅で稼げる方法
参考までに書いておきます
グーグルで検索するといいかも『ネットで稼ぐ方法 モニアレフヌノ』
8GEWS ■ このスレッドは過去ログ倉庫に格納されています