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

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

リーダブルコードとは

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

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

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

**デフォルトの名無しさん** · 2014/11/21(金) 14:12:52.15

変数名をresultではなく、 doneやcompleteのように状態を表す言葉にすればいい。
if (done) { }

**デフォルトの名無しさん** · 2014/11/21(金) 14:14:08.05

if (expr == TRUE) とか書くのは馬鹿げている、とcomp.lang.c FAQの回答にもある。
http://c-faq.com/bool/bool2.html 単に if (expr) と書くべき。

**デフォルトの名無しさん** · 2014/11/22(土) 03:29:01.87

真偽値の場合、doneかisDoneにするかで迷う時がある。

**デフォルトの名無しさん** · 2014/11/22(土) 04:07:44.39

結局どーんすんの

**デフォルトの名無しさん** · 2014/11/22(土) 05:06:36.83

たまにいずどーんにすんの

**デフォルトの名無しさん** · 2014/11/22(土) 11:35:45.69

いやexprは関係演算子以外から導くべきではない、って話だろ

**デフォルトの名無しさん** · 2014/11/22(土) 16:46:09.25

if(variable == undefined)

**デフォルトの名無しさん** · 2014/11/22(土) 19:28:06.50

職場のスパゲティソースのメトリクス値がやばすぎるんで

今度大幅に直そうという話はあがったものの
全く手がついてないんだけど

どういうふうに課員を指導したらいいんだろう

**デフォルトの名無しさん** · 2014/11/22(土) 22:08:19.55

同値変形のやり方を練習させろよ

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

それは、どうちて？

**デフォルトの名無しさん** · 2014/11/23(日) 08:18:15.22

>>220
こーいう人ってなんなん？

分かりやすいだけがルールじゃないし、
分かりやすい定義が曖昧だから、ルール作って
こーいう意識もってやりましょうってのがルールなのになんで勝手なことするの

**デフォルトの名無しさん** · 2014/11/23(日) 08:46:26.24

そういう人が言う「分かりやすいコード」って
基本的に誤解しやすいコードが多い

**デフォルトの名無しさん** · 2014/11/23(日) 08:59:26.38

>>200がプロだって前提で言うと、
>>200みたいな意見は研究所出身の人に多いね。

で、そういう人の書いたコードは、実際にモノに組み込んで
テストすると大量のバグが露見する(机上、単体テストはOKて言ってたにもかかわらず！)

**デフォルトの名無しさん** · 2014/11/23(日) 09:00:06.86

失礼、>>200ではなく>>220でした。

**デフォルトの名無しさん** · 2014/11/23(日) 09:16:30.23

>>242
それはルールを守らずに，独自のわかりやすい方法で書いたからってこと？

**デフォルトの名無しさん** · 2014/11/23(日) 09:34:54.26

一貫性の欠いたフレームワークなんて作られた日には使えたもんじゃないだろ

**デフォルトの名無しさん** · 2014/11/23(日) 11:55:45.10

独自のわかりやすいコードっていうのを見てみたいわｗ

わかりやすいコードっていうのは、独自じゃない。
多くが○○スタイルみたいな言い方で呼ばれてる。
もちろんいろんなスタイルがあって、わかりやすいスタイルというのは
一つではないが、わかりやすいという点では共通してる。

わかりやすいのに、バグがでまくるとか矛盾してるってｗ

**デフォルトの名無しさん** · 2014/11/23(日) 12:50:11.85

C言語で
#ifdef DEBUG
printf(var);
#endif
が1つの関数に頻出するコードは非常に読みにくい

今やらされてる組織内独自のフレームワークのコード

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

>>246
プリプロの記法はもうちょっとなんとかならないものか？#if/#ifdef/#endif を一行で書ければいいのだが‥

**デフォルトの名無しさん** · 2014/11/23(日) 13:20:52.44

>>245
本人はわかりやすいつもりで書いてるコードが該当するんじゃね？
省略できるものは全部省略しちゃうとか。

**デフォルトの名無しさん** · 2014/11/23(日) 15:02:18.53

こうだろ
DBGP(var);

**デフォルトの名無しさん** · 2014/11/23(日) 21:25:37.37

>>248
> 本人はわかりやすいつもりで書いてるコードが該当するんじゃね？
> 省略できるものは全部省略しちゃうとか。

だからそれがどんなのかってこと。
例えば、単語を省略して書くのは
わかりにくいとされている。

**デフォルトの名無しさん** · 2014/11/23(日) 21:27:09.08

>>246
それ、明らかにわかりにくいコードだよね。

わかりやすいコードとして考えられたのではなく、
単にやっつけ仕事でifdef DEBUGを入れたのだろう。

一般的な仕組み、例えばsyslog当たりを使えば
分かりやすくなるよね。

**片山博文MZ次期CEO** ◆T6xkBnTXz7B0 · 2014/11/23(日) 21:27:34.82

#ifdef NDEBUG
#define DEBUG_ONLY(x)
#define RELEASE_ONLY(x) x
#else
#define DEBUG_ONLY(x) x
#define RELEASE_ONLY(x)
#endif

**241** · 2014/11/24(月) 00:06:43.37

な～んかID:c84onc2Pの癇に障ったみたいなので一応補足すると、
別に>>220みたいな意見を持つ人全員がバグ量産機だなんて言うつもりはないよ。

ただ、プロとして仕事する以上、コーディング規約に沿ったものでなければ
納品できない開発なんてざらにあるんだよね(有名所だとIEC61508とか)。

釈迦に説法かもしれんが、分かり易いとテストし易いは、似てるようで別物。
技術力が充分でないのに「分かり易ければOK」ていう人は往々にして
テスト漏れ仕込むってことを愚痴りたかっただけ。>>241は完全に個人的な経験からくる偏見だな。ごめんね。

**デフォルトの名無しさん** · 2014/11/24(月) 00:10:37.50

プロとか言い訳すんなやｗ

「コーディング規約に沿ったものでなければ納品できない」
というのが、理由であって

わかりやすくするためではないだろ。

クライアントが望むなら、けばけばしいデザインのサイトだって作るさ。
だがそれは、クライアントの指示でだめなことをしているのであって
それが正しいと思ってはいけない。

今ここに、クソなクライアントはいないんだぜ？

**デフォルトの名無しさん** · 2014/11/24(月) 00:14:44.32

いや、言い訳じゃなくて、契約の問題ですし。。。常識でしょ？

**デフォルトの名無しさん** · 2014/11/24(月) 00:19:33.68

スレタイ読んでね。リーダブルコーディング技術スレ。

リーダブルなコードのための話であって、

契約で～とかいうのが理由ならそれは無関係だろ。

**デフォルトの名無しさん** · 2014/11/24(月) 00:22:07.34

プロか否か、常識か否か、
これを早々に持ち出す姿勢ｗ

**デフォルトの名無しさん** · 2014/11/24(月) 00:25:05.80

>>249
そう書いてあればまだ読みやすいのにひどいコードだよ全く
あくまでメンテナンス性最悪なコードの例として出したんだけどね

>>253
横からだけど一言多いなんじゃないかな

**デフォルトの名無しさん** · 2014/11/24(月) 00:25:55.45

プロとか常識とか言い出す奴は
信用してないわｗ

だってそれ、単なる虎の威を借る狐だもの。
それに、虎ですらないかもしれないしな

俺の信じる、虎さまはこうおっしゃってる。
だから信じなさい。みたいなｗ

俺が定義するプロはこういうものなんだ。
お前はプロじゃないのか！みたいで
どっかの宗教と同じ。お前は（俺らの）神を信じないのか！

**デフォルトの名無しさん** · 2014/11/24(月) 00:27:15.22

プロとか常識とかクライアントが決めたこことか無視しようぜ

わかりやすいコードとはなにか？
これだけで考えよう。

**デフォルトの名無しさん** · 2014/11/24(月) 00:27:26.67

う～ん、何か余計に癇に障ったみたいだね。
不覚にもスレ違いな話題に誘導されちゃったから、しばらく黙ってるは。

以下↓本来のリーダブルコーディング技術スレ

**デフォルトの名無しさん** · 2014/11/24(月) 00:30:38.62

まあ、俺は最初から、わかりやすいコードの
話しかしてないんけどねｗ

○な書き方、△な書き方、どっちがわかりやすい？
の答えに、どちらでもわかりやすい。という答えがあるのは事実

だから複数のコーディング規約が存在するわけで。

書き方を（自分の好きな書き方に）統一すると、気持ちいかも知れないが
わかりやすさの点から見れば、どちらでもいいことが多い。

**デフォルトの名無しさん** · 2014/11/24(月) 00:34:18.09

>>262
>>254 >>259

**デフォルトの名無しさん** · 2014/11/24(月) 00:35:36.40

>>263
全部俺が言ったことだなｗ

**デフォルトの名無しさん** · 2014/11/24(月) 00:36:13.26

あ、まとめてくれてありがとよって
いうところかｗ

**デフォルトの名無しさん** · 2014/11/24(月) 00:50:24.64

>>246
エディタの置換コマンドで
一行に置換してしまえばいいんじゃないか？

**デフォルトの名無しさん** · 2014/11/24(月) 00:59:19.90

>>266
読むぶんには読めるけど書き換えはさせてもらえない
なのにコード読まないと処理が理解できないというコード

リーダブルと真逆なものを上げることで参考になればいいなと思って

**デフォルトの名無しさん** · 2014/11/24(月) 01:04:11.30

> 読むぶんには読めるけど書き換えはさせてもらえない
可愛そうだよね。まあ、社内規則だからｗ

で、ここはリーダブルなのかはどういうことかって話すスレだから
社内規則で変えられないとかはどうでもよくて。

じゃあリーダブルなのはどういうコードかってことで、
意見を集めましょう。

ってもうみんな書いたか？俺はsyslogって書いた。

デバッグ出力をするために再コンパイルが必要っていうのも
変な話なんだよね。そもそもオプションで指定すればいいと。
デバッグ出力機能を持っている。という所まで仕様。
で、こういう仕様を考える人、そこまで頭が回る人が、いないんだろうね。

**デフォルトの名無しさん** · 2014/11/24(月) 01:05:10.00

只のデバッグプリントなら読み飛ばせばいいだけじゃん

リーダブルの真逆てコードの意図が伝わってこないものを言うんじゃない？

**デフォルトの名無しさん** · 2014/11/24(月) 01:06:14.43

どうでもいいけど、syslogでデバッグって相当頭悪いか、ぬるい開発だな

**デフォルトの名無しさん** · 2014/11/24(月) 01:10:43.53

おいおい、syslogでやるのはデバッグプリントであって
デバッグそのものじゃないぞｗ

syslogにあるだろ。プライオリティdebugのことだよ。

http://www.atmarkit.co.jp/ait/articles/0209/07/news002_2.html
> プライオリティ（priority）
>
> priorityは、プログラムが出力するメッセージのうち、ログのレベルの重要度を設定します。
>
> priority 内容
> debug デバッグ情報
> info 情報
> notice 通知
> warn 警告
> err 一般的なエラー
> crit 致命的なエラー
> alert 緊急に対処すべきエラー
> emerg システムが落ちるような状態
> 下になるほど重要度が高い

なんでこんな基礎的を説明せにゃならんのだｗ

**デフォルトの名無しさん** · 2014/11/24(月) 01:11:55.52

既に世の中にあるものを使わずに（知らずに）
少ない知識でもやれるぜって考えで
オレオレで実装するのも、リーダブルにならない原因の一つだよね。

**デフォルトの名無しさん** · 2014/11/24(月) 06:48:20.80

なぜ、人はアンリーダブルを求めるのでしょうか？
人は生きて、死ぬだけの存在でありながら、
存在するということに意味を持たせ、生と死をより複雑なものにしようとしました。

本来、人は無駄を好む存在です。
オートメーション、自動化、リーダブルなどというものは、
将来、オートメーションにより追いやられた人々により破壊されることが目に見えているのですから、
破壊されるために効率化を豪語し、効率化に勤しむ人々がいることに、
私は含み笑いを抑えることができません。

**デフォルトの名無しさん** · 2014/11/24(月) 07:00:20.21

リーダプルは、時間的にどの程度のスパンで考えるのかな？

**デフォルトの名無しさん** · 2014/11/24(月) 07:09:48.66

読みやすさは主観であり、人により異なる。
全員に対して読みやすさを一定量担保するために、コーディング規約を作る。

デバッグプリント関係は、『CODE COMPLETE』第八章防御的プログラミングがお勧め。

**デフォルトの名無しさん** · 2014/11/24(月) 07:30:10.66

> 読みやすさは主観であり、人により異なる。

つまり、人によっては、Aという書き方が読みやすく、
別の人は、Bと言う書き方、Cと言う書き方が一番読みやすいということ

> 全員に対して読みやすさを一定量担保するために、コーディング規約を作る。

たとえば規約でBという書き方に統一する。
さて、統一することによって、Aという書き方は読みにくということになるのか？
Bという書き方は読みにくいということになるのか？

もちろん違う。なぜなら、
> 読みやすさは主観であり、人により異なる。
と書いてあるように、人によって、AやCが読みやすいということに変わりはないからだ。

つまりなりが良いたいかというと、
AでもBでもCでも読みやすいことには変わりないということ。

統一するのは、読みにくい書き方を無くすのが目的であり、
読みやすい書き方は複数あるということは真実だし、
読みやすい書き方ならどれでもいいことにかわりはないのである。

だからこそAとCは自分のに取っては一番ではないが
それでも読みやすいならばどれでもいいから、その規約に従うのである。

**デフォルトの名無しさん** · 2014/11/24(月) 07:35:54.08

>>220
＞× 書き方を統一しようとする
＞○ わかりやすいように書く

世間に受け入れられているコーディング規約に従うのが一番わかりやすいんだから
それ対立軸じゃなくて両方○になるか両方×になるかだろ

**デフォルトの名無しさん** · 2014/11/24(月) 07:42:27.18

> 世間に受け入れられているコーディング規約

が複数あることを忘れてない？

わかりやすい書き方は一つじゃないんだよ。

**デフォルトの名無しさん** · 2014/11/24(月) 07:55:36.62

＞が複数あることを忘れてない？

忘れてない

＞わかりやすい書き方は一つじゃないんだよ。

もちろん

で？

**デフォルトの名無しさん** · 2014/11/24(月) 07:55:49.98

>>271
当たり前なことをわざわざ引用付きで語んなよｗ
お前がぬるい開発しかしてないことはよく分かったから

**デフォルトの名無しさん** · 2014/11/24(月) 08:01:29.70

>>280
お前、「お前は語るな」と「俺はそう思ってる」の
2つのことしか言ってないぞ。

何か言い返せよｗ

**デフォルトの名無しさん** · 2014/11/24(月) 08:03:05.74

>>279
で？と言われてもな。

最初から言っているように、

わかりやすいコードを書くことが目的であって
書き方を統一することは目的じゃないんだよって話
言い換えると、わかりやすければ統一する理由はない。

本当の目的を忘れるなって話。

**デフォルトの名無しさん** · 2014/11/24(月) 08:06:01.78

＞言い換えると、わかりやすければ統一する理由はない。

いや、ＡとＢの両方わかりやすくても、一つの製品にＡとＢが混在してたら駄目だろ

**デフォルトの名無しさん** · 2014/11/24(月) 08:28:02.98

デバッグ出力でsyslogって発想は無いなぁ

最初に批判されてるが
#ifdef DEBUG
は普通のイディオムだから小さなプログラムなら悪くはないし
もう少し大きな規模になってロギングフレームワークを検討しようというなら
もっと抽象度の高いレベルでちゃんとしたフレームワークを採用するだろ
（C++ならglogとか。言語ごとにあるだろ）

**デフォルトの名無しさん** · 2014/11/24(月) 08:38:23.76

>>282
君のコードは分かりにくい。はい。論破。

だから、ルールを決めるんだよ。お分かり？

**デフォルトの名無しさん** · 2014/11/24(月) 09:04:52.85

>>283
> いや、ＡとＢの両方わかりやすくても、一つの製品にＡとＢが混在してたら駄目だろ
なんで？それで問題が起きたことなんて無いよ。

どうせコーディング規約なんて誰かが我慢すれば済むようなもんでしょ？
一つに統一することで、他の人が我慢するという横暴（笑）が成り立つんだから。
全員が我慢するか、一部の人だけ除いて我慢するかの違い。

>>285
なんで、世の中で普及してあるコーディング規約の
一つを使ってるのにわかりにくいんだよｗ
馬鹿じゃないのか？

**デフォルトの名無しさん** · 2014/11/24(月) 09:05:43.57

>>284
> デバッグ出力でsyslogって発想は無いなぁ
無いなぁと言われてもね。

ほとんどの有名どころのオープンソースはsyslog
でデバッグ出力している。
それが事実じゃね？

**デフォルトの名無しさん** · 2014/11/24(月) 09:13:38.29

＞なんで？それで問題が起きたことなんて無いよ。

すまんが、2ch で匿名でここから議論する気はないので退散。論破と思うならそれで構わん。
教育しても俺に見返りないし。

**デフォルトの名無しさん** · 2014/11/24(月) 09:16:25.55

はい、論破と思いますよ？
別にわざわざいなくてもいいのに。

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

このページの「デバッグライト」とか
http://www.pro.or.jp/~fuji/mybooks/cdiag/cdiag.6.3.html

ここでは、DebugPrintというマクロを用意しているわけだが、一度これを用意したら全員がDebugPrintを使わなきゃ意味ない
DebugPrintを使ってるコードと#ifdef DEBUGを生で使っているコードが両方存在するのは最悪で。

DebugPrintを用意することで、その定義を変えればデバッグ先をsyslogにしたりできるだろ
ところが、生で#ifdef DEBUGされてるコードが残ってたらそこで変更漏れのバグになる
DebugPrintなど無くて全員生で書いていれば、変更時には漏れがないように注意するからまだマシ
DebugPrintを用意したのに使ってないというのは本当に最悪

これがまず、統一するってことな。

**デフォルトの名無しさん** · 2014/11/24(月) 09:27:04.25

もう一つは、
そもそもこういうDebugPrintみたいなのは車輪の再発明なので
今時こんな「オレオレフレームワーク」を作るのは間抜け
（元ページはCopyright 1996）
あるものを使うのが普通

**デフォルトの名無しさん** · 2014/11/24(月) 09:44:09.94

>>286
同じものは同じように見えるようにしておかないと、コードレビューしづらい。
コードに手をいれるときにいちいちスタイルどうするか迷うのも無駄。

・・・コードレビューしないなら問題は起こらない。
まわりと明らかにスタイルの違うコードを混入させて気にしないなら問題は起こらない。
つまりそういうことなんだろうと思ってる。

**デフォルトの名無しさん** · 2014/11/24(月) 10:02:33.90

>>291
> あるものを使うのが普通

だからsyslogだろ？

もしかして、syslog（ライブラリ）を使っていれば
標準出力や任意のファイルに出力できるって知らない？

**デフォルトの名無しさん** · 2014/11/24(月) 10:06:55.47

>>287
> ほとんどの有名どころのオープンソースはsyslog
> でデバッグ出力している。

サーバーソフトだけじゃね？
クライアントソフトの例を 10個ぐらいあげてみて

**デフォルトの名無しさん** · 2014/11/24(月) 10:28:26.69

ログは異常管理にdebugレベルを含めて使うけど、
開発時のデバッグには使わないかな。

**デフォルトの名無しさん** · 2014/11/24(月) 10:41:04.30

>>294
サーバプログラムだってsyslog決め打ちなんて少ないと思うが。ログ出力先の選択肢の一つとしてsyslogがあるだけだろ。

Apache httpd (http://httpd.apache.org/docs/current/logs.html)
エラーログは普通はファイルに書かれます (通常 Unix システムでは error_log、Windows と OS/2 では error.log)。 Unix システムではエラーを syslog やパイプでプログラムに送ることができます。

Postgresql (https://www.postgresql.jp/document/9.3/html/runtime-config-logging.html)
PostgreSQLは、stderr、csvlogおよびsyslogを含めて、サーバメッセージのログ取得に対し数種類の方法を提供します。

Tomcat (http://tomcat.apache.org/tomcat-6.0-doc/logging.html)
Logging in Apache Tomcat is implemented with the help of Apache Commons Logging library.

有名どころのオープンソースでログ機能がsyslog決め打ちって何かあるの？

**デフォルトの名無しさん** · 2014/11/24(月) 10:57:11.56

>>295
「デバッガを使え」みたいな方法論と、デバッグ出力をどうするかは別だと思うが、そういうこと？どうしてる？

**デフォルトの名無しさん** · 2014/11/24(月) 11:23:35.39

>>297
システム運用中に異常処理を行った場合、管理者にその旨を通報する用途でログを使う。
あくまで、システムの一部なので、リリース後でも取り除かれない。

開発時にだけ有効で、リリース時に無効にする用途には使わないかなって話。

**デフォルトの名無しさん** · 2014/11/24(月) 11:56:52.74

開発時のみで使うものだったら
ifdefで残したりしねーよ？

そういう残してあるコードが
じゃまだってのが、話の始まりだろ？

**デフォルトの名無しさん** · 2014/11/24(月) 12:01:55.48

>>298
ああ、理解した
デバッグエイドとログを完全に別の概念として使い分けるということか
プリプロセッサ命令のある言語だと確かにそれがスマートだな、なるほど

**デフォルトの名無しさん** · 2014/11/24(月) 12:05:06.98

>>299
> 開発時のみで使うものだったら
> ifdefで残したりしねーよ？

え゛？

**デフォルトの名無しさん** · 2014/11/24(月) 12:55:47.70

ifdefでソースコードに書いて
それをずっと保守し続けてるってことは
残す理由があるからそうしてる。
ここまでOK？

その残されたコードがじゃまで
リーダブルにならくなって困っている。
ここまではOK？

ならばifdefなんか使わずに
もっとわかりやすいコードにしろって話。

**デフォルトの名無しさん** · 2014/11/24(月) 15:12:20.28

お前はいい加減周りのことも考えて行動や発言できるようになれよ
コーディングはそれから

**デフォルトの名無しさん** · 2014/11/24(月) 17:13:58.44

>>302
で、具体的な提案もなく、文句言うだけ？

**デフォルトの名無しさん** · 2014/11/24(月) 17:49:29.24

プリプロの改良だね‥

**デフォルトの名無しさん** · 2014/11/24(月) 18:31:46.57

で、そこに#ifdefの方がわかりやすい派が現れるんですねわかります

**片山博文MZ次期CEO** ◆T6xkBnTXz7B0 · 2014/11/24(月) 18:39:55.49

#ifdefをインデントするのってアリ？

**デフォルトの名無しさん** · 2014/11/24(月) 19:32:19.00

中身はインデントするかも

**デフォルトの名無しさん** · 2014/11/24(月) 19:42:14.05

ソースとプリプロセッサのインデントが混じって見辛くなるから、俺はしない

**デフォルトの名無しさん** · 2014/11/24(月) 20:41:48.81

半角スペースでインデントはよく見かける

**デフォルトの名無しさん** · 2014/11/24(月) 22:10:52.63

プリプロセッサのみの部分（define値の切り替えとかerrorとか）だったらインデントしたほうが見やすいと思いインデントしてる
ソース内にかかる部分だと309と同じ意見でしてない

基本は#ifがネストしないようにしたいと思ってるけどね

**デフォルトの名無しさん** · 2014/11/24(月) 22:23:12.31

プリプロ行はスペースで、コード行はタブでインデントしてる。
見やすくはなるけど、読みやすさに繋がるかどうかはわからない。

**デフォルトの名無しさん** · 2014/11/24(月) 22:28:30.02

#if XXX
　aaa
　bbb
#endif

よりも

#if XXX
void func() {
　aaa
　bbb
}
#else
void func() {}
#endif

func()
ってやっちゃうなあ。これならfunc呼び出し部分についてのインデントに悩まなくてもすむじゃん？
しかも条件XXXで切り替わる部分が複数に成った場合でも、定義を一カ所にまとめておける。

**デフォルトの名無しさん** · 2014/11/24(月) 22:34:36.62

目から瓢箪が独楽です

**デフォルトの名無しさん** · 2014/11/24(月) 22:38:27.07

関数呼び出しじゃなくてマクロにした方が良くない？
リーダブルの観点ではないけれど、関数呼び出しのネストが深くなるのは良くないと思うんだ。

**デフォルトの名無しさん** · 2014/11/24(月) 22:39:45.84

>>313
func の内容が数行ならいいけど、もっと増えると同じ内容を両方に書かないといけなくなる
func の型とか引数が変更された時に片方の変更を忘れる

とかあるので、俺はやらない

**デフォルトの名無しさん** · 2014/11/25(火) 01:15:46.45

>>316
> func の内容が数行ならいいけど、もっと増えると同じ内容を両方に書かないといけなくなる
> func の型とか引数が変更された時に片方の変更を忘れる

だからこそ>>313のように書くんでしょ？

これなら変更が一箇所にまとまる。

もしこれを、funcを使う場所ごとにに
ifで加工としたら、funcを100箇所使っていたら、
100箇所、同じ変更を繰り返さないといけないよね？

**デフォルトの名無しさん** · 2014/11/25(火) 02:33:45.34

目から箪笥です

**デフォルトの名無しさん** · 2014/11/25(火) 03:16:47.66

私も目から箪笥です

**デフォルトの名無しさん** · 2014/11/25(火) 03:53:07.08

同じじゃない部分だけ分ければいいだけじゃん

**デフォルトの名無しさん** · 2014/11/25(火) 06:54:59.12

>>317
ああ、すまん、勘違いしてたわ
変更部分を別関数に切り出すと言うことか
> 　aaa
> 　bbb
が元々関数に切り出すような意味のある単位ならわかるけど、処理の途中の一部を切り出したような場合だと関数名も付け辛いし分かりにくくなる
あと、単に aaa, bbb って書いてるから違和感ないけど、実際には aaa, bbb で必要とする情報や、逆に変更した情報等のやり取りを引数でやる必要もあるので、結構面倒に思うんだけど

**デフォルトの名無しさん** · 2014/11/25(火) 07:46:51.85

確かOpenSSLも各プラットフォーム毎に
#ifdefの嵐でで手が付けられなくなったらしいけど、
こういうのはTemplate Methodパターンみたいなものを使えばいいんだよ。

デバッグ用の関数、またはクラスと
リリース用の関数、またはクラスを作って
それを使うコードはどちらを使っているか意識する必要がないようにする

これぐらい基本だと思うんだが、OpenSSLを見る限り
世の中のレベルってこの程度のものなのかもしれんな。

**デフォルトの名無しさん** · 2014/11/29(土) 12:47:04.96

>>322
こう？関数fにdebug用とrelease用をラップさせんの？

f_debug() { }
f_release() { }
f() { f_debug(); }

**デフォルトの名無しさん** · 2014/11/29(土) 14:50:29.22

デバッグ用出力の話なら
>>249 で十分テンプレートメソッドパターンと言えるでしょ

**デフォルトの名無しさん** · 2014/11/30(日) 21:01:48.36

呼び出し元のスコープを維持したいからマクロを使うんであって、関数やクラスに分けられるものをマクロで書く奴はいないだろ

**デフォルトの名無しさん** · 2014/11/30(日) 22:41:03.33

『レガシーコード改善ガイド』を読むと、どこにテスト用のフックを
仕掛けるかみたいな話が出てくるからお勧めしとく。

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

c言語でなるべくグローバル変数を使わない方がいいという事で
グローバル変数無しで書き直したら、関数の引数がやたら多くなってしまった。
(´・ω・｀)

**デフォルトの名無しさん** · 2014/12/12(金) 00:31:23.89

こぞーよ、こーぞーたいつかいやがれ