リーダブルコーディング技術スレ

**デフォルトの名無しさん** · 2013/10/11(金) 01:06:21.56

リーダブルコードとは

・読みやすさ、理解しやすさを追及したコードのことです
・読みやすいということはメンテナンス性も高いということです。
・理解しやすいコードはコードレビューも楽になります。
・理解しやすいコードはバグも少なくなります。
・短いほうが優れていることが多いですが、必ずしも短いほうがいいとは限りません。

俺ルール
・書き方の勝負であり、言語の勝負をしないでください。
・標準、非標準に限らずライブラリを使いましょう。
・なければ自分で関数を作りましょう。関数はなるべく汎用的に。

こういうコードを読みやすくするにはどうすればいいでしょうか？
というようなことを語るスレ。需要ありますかね？

**デフォルトの名無しさん** · 2014/12/12(金) 08:57:20.70

勉強になるわあ(´・ω・｀)

**デフォルトの名無しさん** · 2014/12/12(金) 09:27:52.69

メモリなどの制約があるからやるのであって、
それをするのが常識な業界であっても
その目的は「リーダブル」ではないことに注意ね。

**デフォルトの名無しさん** · 2014/12/12(金) 15:16:05.63

>>330
グローバル（スコープの話）と動的確保（メモリー領域の話）をごっちゃにするなよ...

**デフォルトの名無しさん** · 2014/12/12(金) 18:32:11.35

言語の話としては、スコープ(可視範囲)とエクステント(生存期間・寿命)な。
スタックに確保するかヒープに確保するかは、それらの実装の話。

**330** · 2014/12/12(金) 22:14:49.55

いやあ、結構可読性が変わってくるよ。
動的確保系の処理って、寿命管理にそれなりの設計というかコードを書く必要があるし、
未確保の場合の条件分け処理だとか。
グローバルにすればそういう系のコードが排除出来てすっきりするんで。

今、組み込みで動的確保バリバリのプロジェクトに関わってて。
本来大丈夫なはずの所まで、大量の未確保時の例外処理を入れまくって、
可読性？なにそれ？なソースを相手にしているところｗｗ

**デフォルトの名無しさん** · 2014/12/12(金) 23:24:59.05

>>335
> グローバルにすればそういう系のコードが排除出来てすっきりするんで。

ごめん、マジで言ってることがわからん

**デフォルトの名無しさん** · 2014/12/12(金) 23:30:21.77

>>336
C言語の問題になるが、例外というものがなく
エラーは戻り値で返ってくるので、
ちゃんとしたコードを書こうと思ったら原則的に
このように書かないといけない。

int ret;
ret = printf('Hello');
if (ret < 0) {
　return ret;
}

ret = printf('World');
if (ret < 0) {
　return ret;
}

**デフォルトの名無しさん** · 2014/12/12(金) 23:32:27.88

トンチンカンすぎんべ…

**デフォルトの名無しさん** · 2014/12/12(金) 23:35:05.15

>>337
だからグローバル使えばどうすっきりするのかと

**デフォルトの名無しさん** · 2014/12/12(金) 23:35:23.81

ポインタのヌルチェックとかな

**デフォルトの名無しさん** · 2014/12/12(金) 23:43:14.66

オブジェクト指向系言語だったらヌルオブジェクト使ってヌルチェックを不要に出来るし、
どのタイミングで動的確保されるかとかはクラスで隠蔽出来るんだが

**デフォルトの名無しさん** · 2014/12/12(金) 23:44:06.16

コードで示してくれない？
グローバルでもチェックが不要になる訳じゃないよね？

**デフォルトの名無しさん** · 2014/12/12(金) 23:46:39.06

？
ポインタをグローバルに持つとかじゃないよ？念のため。
普通に、ごく普通にプリミティブ型やら構造体変数をグローバルに持つだけなんだが。

**デフォルトの名無しさん** · 2014/12/12(金) 23:53:59.93

イメージ的には、様々な動的確保情報を全て確保した状態のヒープを、
最初から固定でグローバルに持っておく、という感じ。で伝わるかな。。

**デフォルトの名無しさん** · 2014/12/12(金) 23:54:55.17

（やべえ何度読み直しても分からねえ…何が分からないのかさえ分からねえ…）

**デフォルトの名無しさん** · 2014/12/12(金) 23:56:53.96

>>339
> だからグローバル使えばどうすっきりするのかと

グローバルを使えば、メモリ確保が失敗することはない。
つまり、すべてのメモリ確保エラーを
書かないで済む。

**デフォルトの名無しさん** · 2014/12/12(金) 23:58:05.91

>>343-344
ごめん、マジで伝わらん

**デフォルトの名無しさん** · 2014/12/12(金) 23:59:06.81

やり取りの内容がまだ理解できるレベルではないが
奥がふかいんですねえ
やっぱcだけじゃなくjavaあたりもやっといた方がいいのかな

**デフォルトの名無しさん** · 2014/12/13(土) 00:01:01.37

>>346
ヒープ ⇔ スタティックの話なの？

**デフォルトの名無しさん** · 2014/12/13(土) 00:02:55.00

おそらく最初から動的と静的の話なんでしょうよ

**デフォルトの名無しさん** · 2014/12/13(土) 00:05:24.55

そうなんだろうな...

**デフォルトの名無しさん** · 2014/12/13(土) 02:19:46.63

そうだな

**デフォルトの名無しさん** · 2014/12/13(土) 05:42:46.74

書いてみた
http://ideone.com/3l8aq6

**デフォルトの名無しさん** · 2014/12/13(土) 10:22:39.34

と思ったら終わってたのね、話

**デフォルトの名無しさん** · 2014/12/13(土) 10:25:53.22

>>353
え？？

そもそも ID:Znidw3Lc なん？

**デフォルトの名無しさん** · 2014/12/13(土) 10:31:24.66

>>355
ちがうよ
ログ見ながら書いてみただけ

**デフォルトの名無しさん** · 2014/12/13(土) 10:36:24.43

十年ROMれ

**デフォルトの名無しさん** · 2014/12/13(土) 10:40:39.82

あぅぅ

**名無しさん＠そうだ選挙に行こう** · 2014/12/13(土) 23:21:18.25

10年後に2chがなくなってなきゃいいが…

**デフォルトの名無しさん** · 2014/12/18(木) 20:59:16.69

ｃ言語である程度プログラムが大きくなってくると関数の数も増えて来るし
どの関数がどの関数を呼び出してるとか色々増えて来ると思うんだけど
こういうのってどういう管理が一般的なんだろか

**デフォルトの名無しさん** · 2014/12/18(木) 22:04:19.41

>>360
> どの関数がどの関数を呼び出してるとか色々増えて来ると思うんだけど

増えていかないようにモジュール化する

> こういうのってどういう管理が一般的なんだろか

増えちゃったとか不幸にもぐちゃぐちゃのソースを解析するなら Doxygen とか、金があるなら Imagix とかで見るとか

**デフォルトの名無しさん** · 2014/12/18(木) 23:37:34.74

関数がある程度増えたら、それを小さく分割するってことを
しないからどんどん増えるんだよね。

なんで過去のコードを修正しないのか？
簡単なうちにすぐ手を付けないから、あとから手遅れになるんだけど。

**デフォルトの名無しさん** · 2014/12/18(木) 23:47:41.58

あまり小さく分割すると見通しが悪くなる派

**デフォルトの名無しさん** · 2014/12/19(金) 14:45:53.53

>>363
それは分割の仕方が間違ってる派

正しいやり方は

長い関数があった時、その中で汎用的に使える所をシンプルな関数に
抜き出しながら全体を短くするというやり方。

間違ったやり方だと

長い関数があった時、その一部。例えば前半分と後ろ半分みたいに分けて
汎用的ではない関数（というよりもルーチン）を増やすやり方。

”見通しが悪くなる" という言葉がうまく表している。
間違ったやり方は、汎用的ではない関数ができるから"見ないといけない"
「処理があちこちにジャンプして見にくい」など言うのはまさにこれが原因

正しいやり方ならは、汎用的でシンプルだから（一度見るだけで）あとは見なくて良くなる。
見るべきコードが減って処理も分割されないので、あちこちにジャンプすることもなく、見通しが良くなる。

**デフォルトの名無しさん** · 2014/12/19(金) 23:26:05.82

>>364
以上、本ばかり読んでてコード書いたことのない自称できるプログラマーの意見でした

**デフォルトの名無しさん** · 2014/12/19(金) 23:29:01.15

>>365
なんにも言い返していない所がウケるｗ

**デフォルトの名無しさん** · 2014/12/19(金) 23:44:37.90

なかなか難しいところだ脳。
やっぱコード書く前にしっかり設計やってからやるべきなのかね。
趣味だけども

**デフォルトの名無しさん** · 2014/12/19(金) 23:57:59.54

>>366
正論だから反論はできないよ
無限の時間と労力かけてでもやれと言う主張だから、役に立たないけどな w

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

このスレの忘年会みたいな軽いノリで、リーダブルじゃない現場コードの晒し合いとか愚痴晒し合いとかしませんか
言語問わずなんでもありで

丁度上司のunreadableコードと戦っててもうね
出所バレない感じに整形したりしてるんですがもうね

**デフォルトの名無しさん** · 2014/12/20(土) 01:41:00.69

再帰関数使いたいけど無名関数じゃ再帰できないようだからしぶしぶprivateメソッド化して細切れになる現象

**デフォルトの名無しさん** · 2014/12/20(土) 05:18:20.39

>>364
ワンストップ・サービス(窓口)のFacadeはどうなる？

fA(){
fB();fC();fD();
}

>>367
Eclipseのリファクタリングの説明を読むといい

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

>>370つ Yコンビネータ
って全然リーダブルじゃねーか

**デフォルトの名無しさん** · 2014/12/20(土) 10:07:02.21

void isReadable()

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

new codeClone[num_of_func]

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

>>368
> 無限の時間と労力かけてでもやれと言う主張だから、役に立たないけどな w

なんで無限？ｗ
あなたが仕事すると無限の時間がかかるって話？ｗ

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

>>375
> 無限の時間と労力かけてでも
> 無限の時間と労力かかるけど

区別できないアホは絡んで来るなよ

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

>>371
> ワンストップ・サービス(窓口)のFacadeはどうなる？

その例から何を言いたいのかよくわからないよ。

何か俺が言ったことを勘違いしていそうだから、
勘違いしていそうなところの補足をしておく。

> 間違ったやり方だと
>
> 長い関数があった時、その一部。例えば前半分と後ろ半分みたいに分けて
> 汎用的ではない関数（というよりもルーチン）を増やすやり方。

この前半分と後ろ半分にわける間違ったやり方というのは、
例えば一つの工程をA、Bの二つに分けた時、
Bの内容（引数・戻り値じゃなくて"内容"）を見ないと
Aが最終的にどうなるかわからないようなもののこと。

Facadeみたいにそれぞれの独立した工程に処理を渡して
引数・戻り値といった最小限の情報だけでやり出来るものは関係ない。

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

>>376
区別できているかできていないかの前に
質問に答えてよ。

俺は「無限の時間と労力をかけてでもやれ」なんて一言も言っていない。
でもまあ、仕事（作業）するなら時間がかかるのはあたりまえだよ。

（その時間内で）適切なやり方をやれって言っているだけだが
どうして無限の時間をかけるって話になったのさ？

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

「適切なやり方をやれ」って言っただけなのに
それを「無限の時間と労力をかけてでもやれ」と勘違いしてしまうのは

「適切なやり方をやったら、俺は無限の時間がかかるんだ。適切なやり方＝無限の時間なんだ」って
>>368が言っているようにみえるのは仕方ないでしょｗ

**デフォルトの名無しさん** · 2014/12/20(土) 13:02:04.48

>>377
ようは暗黙にBの事前条件がAの事後条件となるのがダメということかね。
>>364の「汎用的」がどういうものを指しているのかいまいち意味がつかめなかったが、
条件を明確にするなら汎用/専用関係なくていいんじゃね？

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

>>396
java で何でかんでも定数定義して（カウンタ変数とか！）
フォーマッタでぐちゃぐちゃになったソースは見たことある

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

だから正論に反論なんてできないよ
「適切なやり方やればいい」
間違ったことは言ってないからな
でも、そんなことは小学生でも言えるわ w

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

>>382の上司「無限の時間をかけてでも適切なことをやれ！でも給料はいつも通り！」
って言われてるからここで愚痴ってるんでしょ

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

>>382
だから変なのは「無限の時間」って話が出てきた所だけだよ。
正論言ってるんだから、俺が言った内容に反論できないのはわかってるよ。
俺が言った内容ではない話が、どこからできたんだ？って言ってるだけ。

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

>>384
そんな時間じゃできません
と言われたらどう答えるんだ？

**デフォルトの名無しさん** · 2014/12/20(土) 23:25:50.20

>>385
誰が誰の立場でできませんっていうんだよ？
わけわからんわ。

時間内に出来るかどうかはケースバイケースなんだし、
出来るところまでやればいいだけだろ。

それに技術力を上げれば時間内にできることも多くなるぞ。
何も努力しないでコピペばかりするから技術力も上がらなくなる。

**デフォルトの名無しさん** · 2014/12/21(日) 00:56:33.95

>>385
上司「何言ってるんだ土日があるだろ。ただしそれはお前の能力不足が原因だから残業代は出ないからな。」

**デフォルトの名無しさん** · 2014/12/21(日) 11:05:47.64

>>386
> 誰が誰の立場でできませんっていうんだよ？

はあ？
指示を受けた人がその指示を実行する責任を負う立場でだよ。
常識だと思うが。

まあ、常識ないから
> 出来るところまでやればいいだけだろ。
なんてことを平気で言えるんだろうな。

>>387
先月すでに土日も使うことにあなたの指示で計画変更されてますが、お忘れでしょうか？ ...

**デフォルトの名無しさん** · 2014/12/21(日) 13:01:56.36

>>388
上司「俺は土日をつかえなんて指示はしていない。そんな計画を承認したはずもない。
ただお前が業務時間内に間に合わないなら土日や深夜や自宅時間も使わざる負えないだろうということだ。そんなこともわからんのか。」

**デフォルトの名無しさん** · 2014/12/21(日) 14:38:18.36

>>388
> 指示を受けた人がその指示を実行する責任を負う立場でだよ。

つまり俺か？なら出来ますって言えばいいだけじゃないか。

「お前このバグ直せる？」って聞かれた時
出来ますって答えるだろ？

**デフォルトの名無しさん** · 2014/12/21(日) 14:40:57.90

あ、もしかして俺なら（上級者なら）時間内にできる仕事を
どれだけ時間があっても出来ないって言いたい人なのかな？
俺にはいくら時間があっても仕事できないよって。

**デフォルトの名無しさん** · 2014/12/21(日) 14:50:49.21

まあ、ここいらでリーダブルコーディングの話に戻すか。

仕事が遅い人って見ていると、自分で自分を苦しめてるんだよね。

自分でぐちゃぐちゃなコードを書いて、
正しく動かない、バグが直せないって言ってる。
自分が書いたコードと戦っているわけさｗ

リーダブル、つまり読みやすいコードを書けるように
自分を鍛えれば、自分が書いたコードと戦うことは少なくなっていく。

でも仕事が遅い人って、自分が書いたコードと戦うことに精一杯。
時間がないを言い訳にしてレベルを上げようとしないから、
戦う敵を量産するだけで、更に時間をなくしてる。

コードは読むものなのだが、どうもレベルが低い人にとっては
解析する物になっているようだ。解析した後その謎を片付けないでそのままにして
さらにコードを追加するから、書けば書くほど謎が深まっていく
だから仕事をするばするほどかかる時間が増えていく。

あ、だから無限の時間って話になるのかｗ

**デフォルトの名無しさん** · 2014/12/21(日) 15:48:14.53

>>391-392
話の流れも読めてないバカ乙

**デフォルトの名無しさん** · 2014/12/22(月) 21:28:40.57

流れが読めないのはリーダブルでないのの典型

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

と責任転嫁ｗ

**デフォルトの名無しさん** · 2014/12/27(土) 23:32:28.76

自作のソフト、ネタ尽きてきたからコードの簡略化してたら５時間くらいたってた

**デフォルトの名無しさん** · 2015/10/14(水) 23:40:08.29

クリエイティブじゃないやつはリファクタリングして仕事した気になってる勘違いが多い

**デフォルトの名無しさん** · 2015/10/15(木) 01:26:25.23

ぎくっ

**デフォルトの名無しさん** · 2015/10/15(木) 12:52:24.82

はい

**デフォルトの名無しさん** · 2016/05/01(日) 16:37:41.94

匿名通信（Tor、i2p等）ができるファイル共有ソフトBitComet（ビットコメット）みたいな、
BitTorrentがオープンソースで開発されています

言語は何でも大丈夫だそうなので、P2P書きたい！って人居ませんか？

Covenantの作者（Lyrise）がそういう人と話したいそうなので、よろしければツイートお願いします
https://twitter.com/Lyrise_al

ちなみにオイラはCovenantの完成が待ち遠しいプログラミングできないアスペルガーｗ

The Covenant Project
概要

Covenantは、純粋P2Pのファイル共有ソフトです

目的

インターネットにおける権力による抑圧を排除することが最終的な目標です。そのためにCovenantでは、中央に依存しない、高効率で検索能力の高いファイル共有の機能をユーザーに提供します

特徴

Covenant = Bittorrent + Abstract Network + DHT + (Search = WoT + PoW)

接続は抽象化されているので、I2P, Tor, TCP, Proxy, その他を利用可能です
DHTにはKademlia + コネクションプールを使用します
UPnPによってポートを解放することができますが、Port0でも利用可能です(接続数は少なくなります)
検索リクエスト、アップロード、ダウンロードなどのすべての通信はDHT的に分散され、特定のサーバーに依存しません
１＾

**デフォルトの名無しさん** · 2016/11/11(金) 16:02:24.59

>>58
＞道具が使えるだけの奴は三流。
＞技術・知識も使いこなせて二流。
＞人も使いこなせてこそ一流。

医療系の管理者です。
今更ですが金言だと思いました。
その立場にならんとわからないですね。