関数の長さ 上級者10行後、中級者100行前後

1 :仕様書無しさん:2014/10/12(日) 04:11:10.31 .net
初心者、数百行〜数千行

(コメント行、空行、括弧だけの行除く)

これ、だいたいあってる

2 :仕様書無しさん:2014/10/12(日) 05:58:02.93 .net

マジか?初心者が数百行も書けるのか

133 :仕様書無しさん:2015/04/23(木) 00:43:58.09 .net

どんどん分割して短くしていくっていうのはプログラミングの理想ではあるけど、クラスやメソッドの名前付けが下手な人がやるとすごい残念なことになる
まあ俺のことだけど\(^o^)/

というかエクササイズ何度かトライしてみたけど毎回パッケージ名どうすりゃいいんだろってとこで詰まる
閉鎖性共通の原則とか意識するとどんどんネストが深くなったりするし

23 :仕様書無しさん:2014/10/13(月) 15:03:07.54 .net

正しい形になっているか、というチェックでやろうとすると深くなるんだよな
全体から正しくない形を除いて行って、残ったのは正しい形というチェック方法にするだけで、浅く。しかも漏れなく作れるのに。

178 :仕様書無しさん:2016/04/18(月) 18:09:50.59 .net

更に言うと実務ではルール通りいかないことも多々あるが、練習くらいはルールを意識して書けと書いてた

165 :仕様書無しさん:2016/02/14(日) 22:06:31.53 .net

>>163
は?画面遷移じゃなくて、アプリケーションの状態の管理をコントローラーがやってんの?

60 :仕様書無しさん:2014/10/18(土) 10:29:06.61 .net

ひねれないのは初心者
ひねって有害なのは中級者
ひねって有益なのは上級者

19 :仕様書無しさん:2014/10/13(月) 02:16:29.52 .net

>>18
ひどすぎワロタw

まず読む前に改行を直すな。

174 :仕様書無しさん:2016/04/09(土) 20:42:13.40 .net

アクセサ禁止で要素ひとつになるまでクラスにするって不可能じゃない?

14 :仕様書無しさん:2014/10/12(日) 17:59:06.68 .net

コードの長さっていうか複雑度のほうが重要かな。

長くても条件分岐やループがなければ全然構わない。
処理が上から下に流れただけだから。

大量の条件分岐などがはいるとだめ。

192 :仕様書無しさん:2016/05/31(火) 07:37:29.05 .net

lgbtは男女に分類されるべきでlgbtという性別はないのではないか問題

38 :仕様書無しさん:2014/10/13(月) 22:52:53.87 .net

何の分野でも何の業界でも一部の有能しか扱えないものなんて一部の有能しか使ってないやろ
世の中、何の分野でも何の業界でも無能が8割以上で構成されてんだから

202 :仕様書無しさん:2017/07/06(木) 17:14:29.43 .net

だから熱い想いは分かったから統計的なデータはよ

118 :仕様書無しさん:2014/11/19(水) 14:36:01.63 .net

2chは上級者のフリをした初級者ばかり

40 :仕様書無しさん:2014/10/13(月) 22:56:39.96 .net

お前らみんな有能そう

186 :仕様書無しさん:2016/05/30(月) 13:24:46.31 .net

>>183
enumにしたところでメンバー変数2つ以上なんだよなあ

96 :仕様書無しさん:2014/11/17(月) 04:49:08.64 .net

>>84 >>89
GJ

これ面白いね

既に会得してるものだと、読んでる最中に「あ〜これ、知らない間に俺やってるわw」ってなる
また、その概念に名前付けして整理できるからなお嬉しい

非言語依存なのもいい

原書読んでみようかしら

160 :仕様書無しさん:2015/11/21(土) 22:07:35.44 .net

>>103
これでelseの代わりになる
if (a)
foo;
if (!a)
bar;

122 :仕様書無しさん:2014/11/24(月) 19:16:46.70 .net

Rubyには一関数5行って規約があったりもする

121 :仕様書無しさん:2014/11/22(土) 13:44:45.24 .net

こういうスレがあったんやな。関数=メソッドの長さはクラス構成と密接に関わるしな。

http://kanae.2ch.net/test/read.cgi/prog/1351405185/l50

172 :仕様書無しさん:2016/04/03(日) 22:34:52.91 .net

メソッドの行数が増えることに気をとられて無駄に分割する奴もまたぺーぺーである

21 :仕様書無しさん:2014/10/13(月) 08:31:52.59 .net

>>18
だれに習ったんだろう?
この改行センスはw

でも、ifも深いわぁ・・・俺の知っているプログラムで最高の深度は23w
肯定ifしか書けない人だったわぁ・・・
否定で考えればスッキリするのに・・・。。

139 :仕様書無しさん:2015/04/26(日) 09:08:33.31 .net

処理の単位を考慮せずやみくもに関数ばかり増やすのはgoto使ってるのと変わんないもんな

79 :仕様書無しさん:2014/11/02(日) 23:58:14.70 .net

>>73 コード以外のドキュメント書かせても誰も読まないと思う。

42 :仕様書無しさん:2014/10/13(月) 22:59:13.98 .net

>>41
複雑じゃないか。

そんな機能を学習する時間のほうがかかる。

26 :仕様書無しさん:2014/10/13(月) 17:51:51.49 .net

>>25
ない。

それは速度を優先してる気になっているだけのコードだろう。
つまりベンチマーク(比較)すらしてない。

61 :仕様書無しさん:2014/10/18(土) 15:10:14.85 .net

>>58
C言語をdefineで別の言語風に仕立て上げてて笑った
それはCOBOL風なのかな?

120 :仕様書無しさん:2014/11/20(木) 09:25:35.81 .net

>>118
じゃあお手本見せて

191 :仕様書無しさん:2016/05/31(火) 02:28:31.09 .net

そのISOってどれくらい知られてるの?

135 :仕様書無しさん:2015/04/25(土) 14:24:46.35 .net

関数が長いという理由で、手続きを関数化するのは悪手
c#ならregionとかで囲む

100 :仕様書無しさん:2014/11/18(火) 00:05:18.46 .net

>>84
このエクササイズ程、やった事ない人とやった人の間で評価がわかるものはないな。
前者はいろいろdisりまくり、後者は2度と以前のやり方に戻れないと言う。

100 :仕様書無しさん:2014/11/18(火) 00:05:18.46 .net

>>84
このエクササイズ程、やった事ない人とやった人の間で評価がわかるものはないな。
前者はいろいろdisりまくり、後者は2度と以前のやり方に戻れないと言う。

149 :仕様書無しさん:2015/04/26(日) 14:43:22.23 .net

あらあら、心に闇を抱えているようですね

183 :仕様書無しさん:2016/05/30(月) 00:29:39.77 .net

>>181
isMaleって書いちゃうセンスが既に禁止事項な件・・・
その勢いだと、isFemale、isTransgenderとかも作るのか?

131 :仕様書無しさん:2015/04/05(日) 00:25:42.17 .net

行数じゃなくて、複雑度、
具体的に言えば、ifやループなんかの
数を制限した方がいいだろうな。
ツールを使わないと数えるのが面倒になるけど。

156 :仕様書無しさん:2015/05/19(火) 00:02:36.32 .net

>>155
うるせえ死ね

48 :仕様書無しさん:2014/10/13(月) 23:03:18.63 .net

体感的にまともなコンビニ店員とか1割もいない気がするが

126 :仕様書無しさん:2015/04/03(金) 06:56:01.67 .net

>>39
いやまて、だめだろそれは

171 :仕様書無しさん:2016/04/03(日) 20:33:14.93 .net

>>169
PG自体が技術者のぺーぺーだろ

32 :仕様書無しさん:2014/10/13(月) 22:40:18.83 .net

○ 無名クラス使いまくると破綻するよ、馬鹿がやると

まあ、馬鹿がやれば何使っても
破綻してますわw

114 :仕様書無しさん:2014/11/19(水) 06:57:17.97 .net

>>111
それだと、ラップしたクラスの中で結局getter使わないといけなくない?

103 :仕様書無しさん:2014/11/18(火) 06:49:27.52 .net

else禁止ってエラー処理はどうすんの?

52 :仕様書無しさん:2014/10/13(月) 23:10:42.97 .net

ネットが普及した今は簡単に色々調べられるけど
ネット普及前のプログラミング世代は大変だったろうな

11 :仕様書無しさん:2014/10/12(日) 13:15:52.44 ID:VzZdInX2x

分からんなぁ
ひと続きの処理が長いと1000行とか2000行とかになる
行を減らす目的で関数に切り分けると、
あの変数も使うから引数に追加して、これも使うから引数に追加して、ってなって
結局ここは分けない方が楽、って結論になる
だいたいメイン処理の一部を関数にしても一回しか使わないし、
処理があっちこっち飛ぶと追いかけるのがめんどくなる

99 :仕様書無しさん:2014/11/17(月) 09:15:48.90 .net

>>95
参考までに調べてみた。

Smalltalk(の特に、VisualWorks とか Squeak/Pharo などの Smalltalk-80 直系子孫)は
35年近く蓄積されたコードで動いているんだけど、メソッドの多くは実際に短く書かれていて
たとえば、手元の Pharo Smalltalk http://pharo.org/ の最新リリース
(のIDEを記述した全ソース)について集計した場合、

(CompiledMethod allInstances collect: [:method | method getSource lineCount]) average asFloat
“=> 7.12162415603901 ”

…とコメントを込みでも平均で7行程度しかなかった。

155 :仕様書無しさん:2015/05/05(火) 04:39:45.23 .net

ここの人はみんな、保守系底辺派遣PGなのかなぁw?
頑張って綺麗なコード書いて!

33 :仕様書無しさん:2014/10/13(月) 22:40:55.55 .net

>>31
お前が、使い方をしらんだけ。

188 :仕様書無しさん:2016/05/30(月) 22:51:34.30 .net

性別をboolにするとかセンス無いというか
常識知らないというか、知識ないというか。

ISO 5218に従えばいいだけだろ
自分で考えるなや

ISO 5218とは、言語に依存しない1桁のコードによるヒトの性別の表記に関する国際規格。
https://ja.wikipedia.org/wiki/ISO_5218

9 :仕様書無しさん:2014/10/12(日) 09:54:18.85 .net

 かなりOOを極めないと10行ぐらいにはならない感じ。
で、相当優秀な人でもOOをマスターしている人はそんなにいない(言語やジャンルによる?)
ちなみにJDKやAndroidのソースコードもメソッド200行とかあるよ。

49 :仕様書無しさん:2014/10/13(月) 23:04:09.64 .net

類は友を呼ぶってやつだな。

90 :仕様書無しさん:2014/11/17(月) 01:28:34.99 .net

> 1 行につきドットは1 つまでにすること

これも一時変数量産するだけで突破できちゃうな

こりゃちゃんと本に書いてある解説を読まず
このルール項目だけを見てエクササイズするととんでもないことになりそうだな

参考になったらSNSでシェアお願いします!

レスを投稿する(名前省略可)

この記事を読んだ方へのおすすめ

  • 任天堂・富士通・NEC・日立、転職ならどれ?

    元スレ 1 :仕様書無しさん:2009/05/19(火) 11:31:57 .net さて。お前らSEはその大半が「転職潜在層」「転職経験者」ということだろう。 いくらITドカタなんて罵倒しても強制労働じゃない、自分で…

  • 無能ってすぐ人に答えを求めるよな

    元スレ 1 :仕様書無しさん:2018/10/30(火) 16:40:52.55 .net 二言目には教えてくれ こっちは人間グーグルじゃねえんだよ 30 :仕様書無しさん:2018/12/03(月) 12:27:03.…

  • 【強欲】これでも中国に仕事を出すのか?【異常】

    元スレ 1 :仕様書無しさん:2010/09/25(土) 23:51:47 .net 売国企業認定だな。 38 :仕様書無しさん:2010/10/22(金) 22:53:54 .net どうぞどうぞ 121 :仕様書無し…

  • 14歳、発達障害の登校拒否ですが…1

    元スレ 1 :仕様書無しさん:2017/06/17(土) 15:18:13.19 .net 冷やかしネカマはじめました。 将来プログラマーになれますかね? 皆様のご助言を求めたいです。 424 :仕様書無しさん:2018…

  • 損保ジャパンがCOBOL一掃を決意

    元スレ 1 :仕様書無しさん:2015/10/15(木) 12:24:41.33 .net 随分前から「もう古い」と言われ続けてきたCOBOL言語。それでも、COBOLで書かれた業務アプリケーションは、今も多くの企業で利…

  • 会社でjQuery使ったら、無能な上司がブチ切れたw

    元スレ 1 :仕様書無しさん:2014/12/29(月) 14:26:29.40 .net いわく 「変なものを使うな」 「俺はそんなもの知らん」 「実績がない」 「他人のことなんか関係ない」 「試してみるつもりもない」…

  • 競技プログラミングにハマるプログラマのスレ 18

    元スレ 1 :仕様書無しさん:2019/01/28(月) 00:11:47.31 .net プログラミングコンテスト(プロコン)やオンラインジャッジや競技プログラミング(競プロ)やCTFなどを楽しんでる競技プログラマ(競…

  • プログラマーはアニメをみよう! 26クール

    元スレ 1 :仕様書無しさん:2018/06/28(木) 17:57:42.82 .net プログラマーはアニメをみよう! 25クール https://medaka.5ch.net/test/read.cgi/prog/…

  • ゴミスペックな30男だけど主にアプリ作って副業したい

    元スレ 1 :仕様書無しさん:2017/02/22(水) 18:38:10.52 .net あとLINEスタンプとかつくりたい 61 : ◆wLeG5ArZiJeS :2017/03/24(金) 14:20:28.75 …

  • 【INW】インターネットウェア 01【TCS】

    元スレ 1 :仕様書無しさん:2008/07/06(日) 01:15:44 .net 東京コンピューターサービスの関連です。 何でも知ってること教えてください。 昔スレあったみたいだけどリトライ! 注意事項 ・個人名・住…

  • SIerってどうしてバカにされてるんですか?

    元スレ 1 :仕様書無しさん:2018/11/14(水) 22:03:02.10 .net 私が来年入る会社がSIerの会社らしいのですが 心配です 3 :仕様書無しさん:2018/11/14(水) 22:17:43.1…

  • 起業しようぜ17

    元スレ 1 :仕様書無しさん:2018/02/12(月) 00:04:37.25 .net 前スレ 起業しようぜ16 http://medaka.2ch.net/test/read.cgi/prog/1486759150…

最近のコメント

匿名 : 合同会社グラウンドステップ 被害者の会 part2
 吉川利幸さんが社長を務めるランサーズでの評価 すご... (5/05)
匿名 : Opera12(Presto) Part8
 Dragonflyが使えなくなったorg (6/25)
匿名 : あえてコマンドライン禁止のLinux
 いまではほとんどのディストリのインストで日本語環境までコマン... (6/15)
ページTOPへ↑