ぱっと見て「ヘタだなぁ」と思うコード　その６

1 名前：デフォルトの名無しさん [2007/10/25(木) 03:35:55 ] 禁止ネタ（超既出） ・長い関数 ・深いネスト ・グローバル変数 ・goto ・memset ・malloc - free ・局所ブロック ・サンプルコードのtypo ・記述スタイル ・関数・変数名 過去スレ その５： pc11.2ch.net/test/read.cgi/tech/1155315371/ その４： pc8.2ch.net/test/read.cgi/tech/1153312202/ その３： pc8.2ch.net/test/read.cgi/tech/1149986051/ その２： pc8.2ch.net/test/read.cgi/tech/1142741989/ 初代　： pc8.2ch.net/test/read.cgi/tech/1141867015/ 260 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 07:38:41 ] 腹減った 261 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 07:43:11 ] >>257 >暇つぶしの友 他所でやれ脳足りん。 262 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 07:49:35 ] if (!strcmp(str1, str2)) {} は綺麗 *dst++ = *src++ と勝るとも劣らないくらい綺麗 263 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 07:50:41 ] while(*dst++ = *src++); 264 名前：デフォルトの名無しさん [2007/10/27(土) 08:24:17 ] 英語ができると、if (!strcmp を、if not string compare と読んでしまうので 等値性をテストしているとは認識しにくいな。 !をnotと読まない人は気にならないんだろうけど。 265 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 08:55:17 ] if (!strcmp(str1, str2)) {} の何がいけないのか分からない 266 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 08:56:01 ] if(str1 == str2) {} だろ上官 267 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 08:59:28 ] >>266 その発想は無かった 268 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 09:06:07 ] ・・・なんでこんなことを議論しているのかがよく判らない。 269 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 09:14:44 ] おまいらおはよう。未だに左定数の話をしていて正直がっかりだ。 >>169 賛同ありがとう。「簡単なことをこんなに難しく書くやつがいるんだぜ」「頭 痛いよなぁ」で済むと思ったら、予想外の展開になって別の意味で頭が痛かっ たよ。 >>171 皮肉? いや、いたってマジだぞ? だってソース見たとき本気で頭かかえたもの。 ちなみにこういうコードへの対策は、「10行以上書いたらレビューしろ。毎日 レビューしろ。動いてからレビューするな」以外に知らないんだ。(そういう 意味でペアプログラミングはすげー有用だと思うんだが、まだ実現できたこと がない) >>189 > の中で重要な処理をしているかもしれない。 それだったら「本質の抜粋」にならないでしょーが。 for文の中は、変数名と定数値以外、「そのまんま」だ。 違う点は、オリジナルはC++じゃなくてJavaなこと、関数(メソッド)のシグネチャ、 書いた処理の前にもコードがあること、最後のif文のブロックの中身。 270 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 09:39:00 ] >>269 補足しておくと、ここで言ってる「本質」とは、 「b-N＜a≦b (a, b, Nは整数) を判定する、ただそれだけのために、大小比較じゃなくてループを使ってる」 ってことだと書けば理解してもらえるか?>>189 271 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 09:58:28 ] ぱっと見て「ヘタだなぁ」と思うスレ 272 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:06:30 ] くだらねー議論してんなぁ。 定数を左に置くか否かなんてとっくの昔に結論でてるだろうが。 >>196のような例外を除いて定数を左において可読性落としたり、 スクロールしないと全部が見渡せないような条件式なんか書くな。 コンパイラが警告出さない時の数十年前に編み出された小手先を 未だに神の一手が如く崇拝してんじゃねぇよ、ボケ。 これを知ってる俺って技術力たけぇなんて思ってたかもしれんが、 ド低脳の証明だ。玉石混交の石のほうだ、カス。 273 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:14:14 ] おまえら、何度も同じことをくりかえすならfor文の中にでも入れとけよ 274 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:18:04 ] >271 禿同 275 名前：デフォルトの名無しさん [2007/10/27(土) 10:18:23 ] おはよう。しつこいがもう一度爆弾投下！ test.cpp 　if (a = 0) printf("x"); D:>bcc32 test.cpp Borland C++ 5.5.1 for Win32 Copyright (c) 1993, 2000 Borland test.cpp: 警告 W8060 test.cpp 7: おそらく不正な代入(関数 main() )　←←←【ここ注目！】 Turbo Incremental Link 5.00 Copyright (c) 1997, 2000 Borland >>80 わざわざひねくれて if (0 == a) としなくても、ちゃんと警告出してくれるから。 ミスを防ぐため、というのは全然理由にならない。 「定数は左」って書き方って、１５年以上も前に「Ｃマガジン」って雑誌が流行らせたんだよね。 当時の古いコンパイラは上のような警告は出していなかったかもしれない。 でも今は無料のタダのコンパイラでさえ警告出すようになって、「定数左」は意味の無いものになった。 １５年も前に決着が付いて廃れたことを、いまだに「神の福音」のように信じ込んでいるヤツがいるんだな。 おまいは隠れキリシタンかｗ 276 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:21:53 ] >275 if文の中身間違えてない？ 277 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:30:11 ] >>269 あまりにもスレタイトルに沿ったコードだったからレスつけるまでもないと思った。 278 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 10:50:13 ] >>275 コンパイラによっても違うな。さすがゴミクズ撒き散らすマイクロソフトって感じだ。 C:>bcc32 test.cpp　　【borlandのコンパイラ】 Borland C++ 5.5.1 for Win32 Copyright (c) 1993, 2000 Borland test.cpp: 警告 W8060 test.cpp 5: おそらく不正な代入(関数 main() )　　【警告が出る】 Turbo Incremental Link 5.00 Copyright (c) 1997, 2000 Borland C:>cl test.cpp　　【マイクロソフトのコンパイラ】 Microsoft(R) 32-bit C/C++ Optimizing Compiler Version 14.00.50727.42 for 80x86 Copyright (C) Microsoft Corporation. All rights reserved. test.cpp Microsoft (R) Incremental Linker Version 8.00.50727.42 Copyright (C) Microsoft Corporation. All rights reserved. 　　　　　　　　　　　　　　　　　　　【警告は出ない】 /out:test.exe test.obj 279 名前：276 mailto:sage [2007/10/27(土) 10:55:07 ] 間違った；ごめん 280 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:02:33 ] >>278 cl は /W4 で使うもんだ。 gcc は最低でも -Wall で使うもんだ。 281 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:11:33 ] おれ定数左派だけど定数を左におく場合の欠点って何？ 可読性落ちる？ でも落ちるか？ 慣れてないとえっ？って思うかもしれないけど別にそれぐらい大丈夫でしょ。 javaでも "hoge".equal(var); ってイディオムあるぐらいだし。 lintなんて毎回通さないし、コンパイラの警告も丸カッコついてるとスルーされるし。 (論理式で繋がってたり、マクロが絡んだりすると丸カッコつく場合があるよね) あとたまたま警告を見過ごしてオブジェクトファイルになってしまうと、次にファイル更新しないと警告でないよね。 おれ注意散漫だから1年に2、3度ぐらいは==と=の間違いやってしまうけど定数左のおかげで早期発見できてるよ。 もちろん定数右でもコンパイラの警告で見つかってるかもしれないけど 定数左にして損はなしと思うから昔から今までずっとそのスタイル。 282 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:16:37 BE:789120656-2BP(125)] ソースファイル内でそのスタイルで統一されてるなら、別に定数が右だろうが 左だろうが、個人の好みだからいいと思うけどなあ。 会社とかで、規約がちゃんとある場合は別で。 283 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:22:03 ] C:\test>cl /W4 test.c Microsoft(R) 32-bit C/C++ Optimizing Compiler Version 14.00.50727.42 for 80x86 Copyright (C) Microsoft Corporation. All rights reserved. test.c c:\test\test.c(7) : warning C4706: 条件式の比較値は、代入の結果になっています。 Microsoft (R) Incremental Linker Version 8.00.50727.42 Copyright (C) Microsoft Corporation. All rights reserved. /out:test.exe test.obj >>278 clでもちゃんと警告出るじゃねぇか 284 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:25:11 BE:1473024487-2BP(125)] gccも-Wallつけないと警告でないね。 まあ、普通は付けてると思うけど。 285 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:37:51 ] >>282 スタイルには単に好みの問題と、実際にメリットのあるものとがある。 前者は ifの条件式の後に中カッコつけるか、改行してからつけるか、など。 後者はif のあとには必ず中カッコをつける、など。 定数左は後者だと思う。(可読性に問題はないと思うので) 286 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 11:52:10 ] >>282 「統一されてればいい」ってことなら、あえて少数派のスタイルを選ぶことはないんじゃないの? >>72 287 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 12:10:59 BE:526080454-2BP(125)] >>285 定数左は確かに代入によるバグを防げるときもあるけど、 その反面、慣れないとぱっと見てわかりづらいよね？ 例えば、代入文は x = 0; などと書くわけで、大抵の文は左から右に読んでいく。 そして、定数は右にある。これへの慣れが強いので、 if(0 == x)などと書かれると、ここで思考が停止してしまう。 ソースコードの理解しやすさ(可読性？）も考えると、トレードオフで どちらを選ぶかは個人の趣味レベルになるのかなあと。 上で議論されてるように、コンパイラとかで容易にチェックできるわけだし。 288 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 12:12:39 ] コーディングスタイルに拘りすぎる奴はヘタクソ コーディングスタイル論争に突入する奴はさらにヘタクソ 289 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 12:15:18 BE:1262592386-2BP(125)] >>286 少数派ではあるけれども、>>285の言うようにチェッカとかを使わなくても ミスに気づくことができるという利点があるので選ぶんだろうね。 ちなみに漏れは定数は右派。 右におくソースがほとんどだし、慣れてるし。 漏れが見たオープンソースなソフトウェアのソースは定数が右にあった。 もし、定数が左にあることにもっと優位な点があるなら、とっくに ほとんどのソースが定数左になってると思うんだけど、いかがだろう。 290 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 12:49:32 ] とヘタクソ厨がもうしております 291 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 12:57:03 ] >>281 主語と目的語の区別がつかないアホには 自然言語からやり直せといいたい。 292 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 13:05:06 ] 読む分にはどっちでもいい。何の違いもない。 書くときはsubject == compareの方が流れ的に書きやすい。 293 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 13:50:50 ] 288の書いたコードは汚そうだな。 294 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 14:23:51 ] おまえら…。 【審議中】 　　　 ∧,,∧　 ∧,,∧ 　∧ (´・ω・) (・ω・｀) ∧∧ (　´・ω)　Ｕ) ( つと ﾉ(ω・｀ ) ｜　Ｕ (　　´・) (・｀　　)　と ﾉ 　ｕ-ｕ （l 　　 ) (　　　ﾉu-ｕ 　　　　 `ｕ-ｕ'.　`ｕ-u' 【結果発表】 ﾊﾟｯ　　　ﾊﾟｯ　　　ﾊﾟｯ　　　　ﾊﾟｯ　　　ﾊﾟｯ　　　　ﾊﾟｯ 　[チラ]　 [シの]　 [裏に]　 [書き]　　[やが]　 [れ！] 　　‖∧,,∧ ‖∧,,∧ ‖∧,,∧ ‖∧,,∧ ‖∧,,∧ ‖∧,,∧ 　　∩・ω・｀) ∩・ω・｀) ∩・ω・｀) ∩・ω・｀) ∩・ω・｀) ∩・ω・｀) 　　 （　　　 ）. （　　　 ）. （　　　 ） （　　　 ） （　　　 ）　（　　　 ） 　　　｀ｕ-ｕ´　 ｀ｕ-ｕ´ 　 ｀ｕ-ｕ´　 ｀ｕ-ｕ´ 　｀ｕ-ｕ´　 ｀ｕ-ｕ 295 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 14:31:10 ] if(*dst++ = *src++){ hogehoge } みたいなとき警告でないようにしたいのですが 296 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 14:33:15 ] if((*dst++ = *src++) != 0){ 297 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 15:11:01 ] もうね、コーディングスタイルスレでも立ててそこでやれと 298 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 15:40:13 ] しかし、無茶苦茶な低能が居たものだな... if (!strcmp(str1, str2)){} は、知らない者に対して、文字列が等価で無いこと比較しているような誤解を与えることがあるので好ましくは無いだろう では、 if (strcmp(str1, str2) == 0){} これはどうだろうか? この場合、!strcmp()よりも悪化していると言える 本質的な問題を解決していないからである しかも、余分に入力した上で、知らない者に対し更なる誤解の種をまいていると言えるからである 故に、C FAQでは、 #define Streq(s1, s2) (strcmp((s1), (s2)) == 0) と言うマクロを用意することで if (Streq(str1, str2)){} と記述させる事により、文字が等価であることを比較している事を判りやすくしましょうと書いているのだ つまり、定数を左に置く正当の理由とは言えない その上、彼の主張の一つである、定数を左に置くことで、何と比較しているのか判りやすくなると言う弁からすると "文字列が等価では無いことを調べている"ように強調されてしまった駄目なコードの例と化しているのである 彼は、これを大まじめに、こちらの方が判りやすいと言っているが、どう考えても、判りにくなっていると思われる 299 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 15:41:35 ] わかったから月曜になったらちゃんとハロワ池よ 300 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 15:54:07 ] ;｀;:ﾞ;｀(;ﾟ;ж;ﾟ; )ﾌﾞﾌｫｯ！ 301 名前：デフォルトの名無しさん [2007/10/27(土) 16:04:59 ] strcmpでの定数の右左に関して言えば、圧倒的に右だろう。 if (strcmp(str1, str2) < 0) を読むとき、不等号が見たままの形として利用できる。 つまり、 str1 < str2 として直感的に理解することができる。 同様の条件を右定数で書くと、 if (0 > strcmp(str1, str2)) となって、見たままの直感と反してしまう。 そのつどリファレンスを引いたりしたくないし、 機械的に右へ書いてしまう習慣は役に立つ。 302 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 16:05:07 ] できる奴ほどスタイルにこだわる 303 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 16:14:56 ] 大きいほうを右に配置したいし、常にそうするから、 結果的に不等号の「>」は不要だと思うんだよなぁ。 勿論、流儀の問題でぶつかったりする心配はあるけど、 そうじゃなくて自分ひとりで「>」無しでやっていくうえで、 どうしてもでてくる不都合ってあるのかな？ 「<」じゃ困ってしまうような事態があるのかな？ チラシの裏にでも書いておけって？　すまんね。 304 名前：デフォルトの名無しさん [2007/10/27(土) 16:31:17 ] だれか、288を日本語に翻訳してくれ。 で、実際のところ、定数は左に置けとか頑張ってるのって、一人だけだろ？ 305 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 17:26:57 ] >>301 それは「定数を左辺に」と主張している派の本質と話題をそらせて釣りだと思うけど 306 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 17:28:27 ] ぱっと見て「バカだなぁ」と思うコード 307 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 18:09:07 ] スレスピードがきもいよお前ら 308 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 18:14:17 ] いまム板で一番勢いがあるスレだぜここは 309 名前：デフォルトの名無しさん mailto:sage [2007/10/27(土) 18:15:40 ] 昨夜から今朝にかけての速度はもう、ある種の祭りだった。

---

---

[ 新着レスの取得/表示 (agate) ] / [ 携帯版 ]

read.cgi ver5.27 [feat.BBS2 +1.6] / e.0.2 (02/09/03) / eucaly.net products.
担当:undef