- 1 名前：デフォルトの名無しさん mailto:sage [2007/02/04(日) 23:28:42 ]
- カッコの位置や変数名のつけ方から、何で規約なんて決めるのという話まで
コーディング規約・スタイルに関するスレ
前スレ
コーディング規約 第2条
pc10.2ch.net/test/read.cgi/tech/1068752664/
- 401 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 03:26:36 ]
- >>394
CodeWarriorというものがあってだな…
- 402 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 03:47:41 ]
- >>396
変更日・変更者コメントは、同時進行で進んでいるソース流用プロジェクトが、
流用元の最近の変更と自分たちの変更を区別するために欲しがっていたな。
- 403 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 03:53:56 ]
- >>401
組込のIDEには入力支援や型情報ポップアップがないって意味？
俺のとこではエディタとして別のIDE使ってたよ。組込の環境は使いにくいｗ
- 404 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 04:15:35 ]
- アセンブラだとメモリにどう入っているか分かるほうが楽
- 405 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 15:04:56 ]
- CodeWarrior のクソっぷりはなんとかならんか
- 406 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 17:25:27 ]
- >コードを見ればすぐ分かることを書くのは無駄。
アルゴリズムやデータ構造は知ってても、コードは読めんと言う相手と
連携しなきゃならん場合は必要だぞ。
具体的に例を挙げると、↓みたいなコードを、相手方にチェックしてもらうには
「読んで字のごとし」なコードもすごく重要。
・別言語で作ったアプリが吐いたデータを、整合性を崩さないように編集するコード
・ハードウェアを直接叩くコード
- 407 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 17:49:42 ]
- それは、「コードを見ればすぐ分かる」ことじゃない。
- 408 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 19:21:45 ]
- コードをみればすぐ分かるってのは、誰でも言語の知識だけでコードからすぐに読みとれること、と言い換えたら理解しやすいかもしれんな。
自分の知識ですぐ分かること、ではないぞｗ 言語以外の知識やその行付近以外の知識を必要とする時点ですでに「すぐ分かること」ではない。
例えばこういう感じ
*((volatile char*)0xF860) = 0; //&HF860番地にゼロを書く ←無駄なコメント
*((volatile char*)0xF860) = 0; //VDPリセット ←知識があれば分かることだが役にたつコメント
- 409 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 19:41:39 ]
- >>408
>*((volatile char*)0xF860) = 0; //VDPリセット ←知識があれば分かることだが役にたつコメント
これはコードをみればわかることじゃあないと思うな
0を入れるのか、リセットするのか、クリアするのかは、意味が少々違うからな
「文字」「文字値」「文字配列」「文字列」の違いみたいなものも結構気になる
- 410 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 20:10:39 ]
- >>409
コードを見れば分かる無駄なコメント に対する 役立つものの例として、という意味ね
「そこにゼロ書けばVDPがリセットされる」って知識がある人には、例の２つは同じ「コードをみればすぐ分かるコメント」に見えるかもしれない
でも、>>389で言っている「コードをみればすぐ分かるコメント」ってのは前者（ゼロを書く）であって、後者（リセット）のコメントの付け方ではないよ。
という説明のつもりだったｗ
- 411 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 21:15:41 ]
- #define VDP_ADDRESS (volatile char*)0xF860
#define RESET 0
*(VDP_ADDRESS) = RESET;
- 412 名前：409 mailto:sage [2009/02/21(土) 21:32:26 ]
- >>410
同意
>>411
そうそう、オレならそうする。
- 413 名前：デフォルトの名無しさん mailto:sage [2009/02/21(土) 22:23:07 ]
- #define VDP_ADDRESS ((volatile char*)0xF860)
#define RESET 0
#define RESET_VDP() *VDP_ADDRESS = RESET
RESET_VDP();
- 414 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 01:06:49 ]
- マクロ地獄だな
- 415 名前：409 mailto:sage [2009/02/22(日) 01:09:28 ]
- >>411は、レジスタのアクセスだけなのがわかるが、
>>413だと、いくつかのシーケンスが内蔵されているように見えるね
該当のレジスタに他の値を入れることがたびたびあって、この行でリセットをしたいなら、明示的な>>411
該当のレジスタには、他でアクセスすることがなく、ここでリセットするだけなら>>413だな
- 416 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 17:45:00 ]
- 皆さんのお知恵をお借りしにきますた
どっかの本かWEBで、ブーリアンの引数名にはNoとかNotはつけるな
ってあったんだけど、みなさんどう思いますか？
例えばbool NoScrollとか
- 417 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 17:57:32 ]
- >>416
賛成の反対で、これでいいのだｗ
- 418 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 18:08:30 ]
- >>416
本とか読んでる訳じゃないから、俺の感覚だと…
別にbool型なんだから
NoScrollにtrue入れるか、Scrollにfalse入れるかでしょ
それなら、コード中でどう使うかで使い分ければいいんじゃない
絶対に、NoScrollを主体で判断した方がいい部分で使われるのが分っているなら
NoScrollとすれば良い
それ以外で、Scrollという状態をtrue or falseで見る使い方なら
わざわざフラグを反転させる意味にしない方か良いと思う
- 419 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 19:02:30 ]
- >>418
どうもありがとうございます
その本だかによれば、ブーリアンは意味的に「〜する」の肯定で統一したほうが混乱が少ない、みたいなことを根拠にしてました
そうやって統一すると、何かを「する」場合はtrue、「しない」場合はfalseを指定すれば良くなる、と
マァ自分的にも個々の使い分けでいいと思いました
- 420 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 19:22:24 ]
- >>419
直感的に、True(真)なのにNo(否定)っていうのが誤りの元にならないかってことだよね。
"できるだけ"避けるべきだとは思うなぁ
ちょっとオレが変かもしれないけど
電気ポットの注ぎ口なんかよくハッとする
「赤」なのに「出る」のが気持ち悪い
- 421 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 19:37:12 ]
- 混在させるより統一した方がいいとは思うけどどっちでもいいと思えるほどの些細なことに見える。
- 422 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 19:49:32 ]
- >>420
赤＝危険という風に見ればいい
- 423 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 20:23:15 ]
- >>416
否定後禁止ルールは、isとかcanとか三人単数ルールとかと併用しないと意味不明になりがちだと思う。
その場合ならCanScrollとかScrollsとか。
- 424 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 20:25:59 ]
- 酷い日本語すみませぬ…。
×否定後 → ○否定語
×三人単数 → ○三人称単数
- 425 名前：420 mailto:sage [2009/02/22(日) 20:35:41 ]
- >>422
それはわかってるんだけどねｗ
出すことを前提に操作してるのに「赤」になるのがとてもアフォーざんす
- 426 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 22:22:51 ]
- なして？
- 427 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 22:55:00 ]
- 信号機だと赤は停止の色だから？
- 428 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 23:15:36 ]
- ・水は青、お湯は赤
・熱いから危険だから赤
- 429 名前：デフォルトの名無しさん mailto:sage [2009/02/22(日) 23:22:32 ]
- >425
そこは「お湯を注ごうとしてる人」とは違うアクターを想像するんだ。
例えば「ポットを運ぼうとしてる人」とか。
- 430 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 00:52:59 ]
- >>429
わかっちゃいるんだけどね、例えば、車で
アクセルを踏むと赤いランプが点くとしたら、、、気持ち悪いよね
そんな感じ
- 431 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 01:06:41 ]
- そのつまんねー自分語り、いつまで続けるつもり？
- 432 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 01:19:19 ]
- >>431
レスがある限りｗ
- 433 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 02:49:40 ]
- >>431
2ちゃん歴浅いんか？
嫌ならお前が別の話題を振ればいい。
- 434 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 09:35:12 ]
- >>433
2ちゃん歴浅いんか？
気に入らないレスならスルーしろ。
- 435 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 13:06:16 ]
- おまえここは初めてか？力抜けよ。
- 436 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 13:28:14 ]
- 420うぜーぞ
- 437 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 16:51:41 ]
- スルーしろといってる本人がスルーできてない
- 438 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 17:23:41 ]
- ｵﾏｴﾓﾅｰ
- 439 名前：デフォルトの名無しさん mailto:sage [2009/02/23(月) 20:53:40 ]
- ,;:'';, ,;:' ';
,;'':.:. ';,,,.,.,.,.,.,;:' ';
､:´: . ':,
,':.:..:. . ':, ま、お茶でも飲んで休憩すれ。
';:.:.:.:.. ／ ＼ ':,
';::.:.:.:. . ● ● ;'
';:::.::.:. （__人__） ;'
'':;,:.:.:.:.:. . ､:' （ (
,;:'':.:.:. '::,,.,.,.,.,,.,,_ ヽ ヽ
,;':::: :: `()＼; _＿O＿_ )ノ
,;' ;:.:. ;, .:.''_,.,,.,.,.,.,.,.,_,:'" ＼/=￣￣ =ソ,ﾌi!
,;' ;;:.:. ;, ;;'' {= = = = =イ .i!
;' ';;:.:. ;, ;:' ﾞ､_= = =_ノ i!
;, ';, ,; ;' ￣￣ ┌―‐┐〜
;, ﾞﾞ''''''" ; { 茶 }
:; ;'''"ﾞﾞ'';: :; ﾍニニﾉ
:; ; ;: ; ||￣￣￣￣￣￣￣||
':, ,; :; :; || ||
ﾞﾞ''''''''''" ﾞﾞ''''''''''"
- 440 名前：デフォルトの名無しさん mailto:sage [2009/02/24(火) 00:23:49 ]
- いや、俺はスルーしろなんて言ってないしなあ
命令しといて自分はやらない人につっこみ入れただけだしなあ。
- 441 名前：デフォルトの名無しさん mailto:sage [2009/02/24(火) 00:26:10 ]
- レス乞食
- 442 名前：デフォルトの名無しさん mailto:sage [2009/02/24(火) 00:30:51 ]
- 呼んだ？
- 443 名前：デフォルトの名無しさん mailto:sage [2009/03/02(月) 01:19:09 ]
- コメントに愚痴とかいいわけ書いてるやついるよなｗ
そういうヤツのコードは１関数１５００行とかスパゲッティな依存関係とかすごく汚いコードだったｗ
そんな事書くくらいならコードの可読性をあげるコメントを書けと
- 444 名前：デフォルトの名無しさん mailto:sage [2009/03/03(火) 01:33:42 ]
- パスカル記法ってただ頭を大文字にするだけ？
ハンガリアンみたいに詳細に説明してるサイトがないからそれしかわからん
- 445 名前：デフォルトの名無しさん mailto:sage [2009/03/03(火) 10:05:24 ]
- >>444
>パスカル記法ってただ頭を大文字にするだけ？
そんだけ。
- 446 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 12:19:31 ]
- おまいらtypedefどう思う？
いちいちtypedefしてんじゃねーよカスが。
- 447 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 12:50:46 ]
- なんでもかんでもtypedefするカス
typedefしなきゃ書けないコードを知らずに全否定するカス
- 448 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 15:23:25 ]
- 批判 [編集]
一部の人々は、typedefを広範に使用することに反対している。
ほとんどの議論は、typedefは単に変数の実際のデータ型を隠すだけであるという考えに集中する。
例えば、Linuxカーネルハッカーであり、ドキュメント作成を行っているGreg Kroah-Hartmanは、
関数プロトタイプ宣言を除いて、typedefの使用をやめさせようとしている。
彼は、typedefを使用することが、必要以上にコードを混乱させるだけでなく、
プログラマが巨大な構造体を単純な型と誤認識して使用してしまうことがあると主張している[1]。
- 449 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 17:34:56 ]
- >>448
どこからの引用なのか書けや
- 450 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 18:04:54 ]
- ここだな。
ttp://ja.wikipedia.org/wiki/Typedef#.E6.89.B9.E5.88.A4
- 451 名前：デフォルトの名無しさん mailto:sage [2009/03/14(土) 18:17:03 ]
- >>446
typedefはCとC++ではまったく事情が違う。
C++でテンプレートメタプログラミングするなら必須
- 452 名前：デフォルトの名無しさん mailto:sage [2009/04/19(日) 16:03:42 ]
- そうかなあ
- 453 名前：デフォルトの名無しさん mailto:sage [2009/04/19(日) 22:00:52 ]
- そうだよお
|
 |
