- 1 名前：デフォルトの名無しさん [2006/08/12(土) 01:56:11 ]
- 禁止ネタ（超既出）
・長い関数
・深いネスト
・グローバル変数
・goto
・memset
・malloc - free
・局所ブロック
・サンプルコードのtypo
・記述スタイル
・関数・変数名
過去スレ
その４： 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/
- 384 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 13:47:24 ]
- Delphiで、引数にconstつけて周る俺がきましたぉ
- 385 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 13:59:43 ]
- なんでconsomeはないの？
- 386 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 18:29:22 ]
- >>376 378
下手かどうか以前にバグってるぞ
- 387 名前：378 mailto:sage [2006/10/08(日) 21:01:10 ]
- >>386
どっかバグってた?
#関数名以外でw
- 388 名前：デフォルトの名無しさん [2006/10/08(日) 22:09:01 ]
- C++でクラスにする必要のない処理をわざわざクラス化してるとき
- 389 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:21:19 ]
- >>376 ++i を使ってる時点で俺的にｏｕｔなんだが
- 390 名前：デフォルトの名無しさん [2006/10/08(日) 22:23:22 ]
- *=ってなんでつかｗｗｗｗｗｗ
- 391 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:27:46 ]
- >>389-390
素人は帰れ。
- 392 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:35:47 ]
- resはどこで確保されてるか
- 393 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:37:08 ]
- >>389
C++の勉強をしましょう。
>>390
Cの勉強をしましょう。
- 394 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:37:52 ]
- >>392
>390が指摘している行の2行上。
- 395 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:44:24 ]
- はいはい、i++と++iは同じです。
- 396 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:50:20 ]
- >>395
C++の勉強をしましょう。
- 397 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 22:56:04 ]
- >>396
その話題飽きた
- 398 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:00:15 ]
- 勉強しても++iとi++は同じであることがわかるだけだけどな。
- 399 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:02:50 ]
- >>398
C++の勉強をしましょう。
- 400 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:03:40 ]
- 先に加算するか後で加算するかの違い
なんて関係ないことがほどんどだよな
- 401 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:05:07 ]
- C++の勉強をしましょう。
- 402 名前：デフォルトの名無しさん [2006/10/08(日) 23:06:31 ]
- >>376がＣ++で書いたとは限らない
- 403 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:23:53 ]
- >>402
C++ですが、なにか？
- 404 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:29:36 ]
- >>401
何が違うのか説明してみろ。
- 405 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:30:20 ]
- この場合は何もかわらんな
- 406 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:30:37 ]
- しょうがないな、正解を言うぞ、
i++; は tmp=i,++i,tmp; という命令、階乗は ((1 + n) * n) / 2 で求まる。
うはっＷＷＷオレ天才ＷＷ
- 407 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:33:43 ]
- 中学の数学の勉強をしましょう。
- 408 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:35:01 ]
- >>400
i++の式値はi
++iの式値はi+1
前とか後とかじゃない、式値が違うだけだ
- 409 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:37:01 ]
- >>408
で、>>376の文脈で違いはあるのか？
- 410 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:38:00 ]
- >>409は400か？
>>400では>>376に言及してないぞ？
急に文脈を無視して>>376に関連付けられても困る
- 411 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:39:06 ]
- >>410
流れ嫁
- 412 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:39:31 ]
- shine!
- 413 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:42:05 ]
- 0!が考慮されてないって意味じゃね？
あと++iとi++は戻り値が違う。
- 414 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:42:45 ]
- >>413
で、>>376の文脈で違いはあるのか？
- 415 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:43:12 ]
- >>414
無いね
それがどうしたの？
- 416 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:44:30 ]
- >>415
別に。
じゃ、この話題終了。
- 417 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:46:23 ]
- ここはぱっと見のコードの質を云々するスレなのだから、充分違うと思うが。
- 418 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:46:27 ]
- ところで int *iってして、*(++i++)はどんなのになるの？
文脈的に考えて
- 419 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:46:53 ]
- 何が違うの？
- 420 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:47:49 ]
- >>418
Cを勉強しましょう。
- 421 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:49:29 ]
- >>420
答えられないということですね。
- 422 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:50:33 ]
- 夏だなあ
- 423 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:50:33 ]
- 「〜しましょう」とか言ってる奴うぜー
- 424 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:51:25 ]
- 鹿児島商
- 425 名前：デフォルトの名無しさん mailto:sage [2006/10/08(日) 23:56:18 ]
- >>418
「i++が右辺値になり、それに前置++を使用しているのでコンパイルエラー」だと思う。
- 426 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 00:07:48 ]
- なんかこの話題最近見たきがするなぁ。
- 427 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 00:10:16 ]
- C++におけるpre/post incrementの知識を披露したいんでしょう。
- 428 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 00:10:45 ]
- このスレの80%はループでできて略
- 429 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 01:31:43 ]
- しかし無知が間違いと確信してるとこを指摘して
突っ込まれているところをニヤニヤしてる分には
ループでもいい
- 430 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 17:31:25 ]
- >>429
あんまり生産的なやり取りじゃないから
見てて気分よくないけどな。
- 431 名前：デフォルトの名無しさん mailto:sage [2006/10/09(月) 17:42:01 ]
- 馬鹿がうつるってのはあるけどな
- 432 名前：デフォルトの名無しさん mailto:sage [2006/10/10(火) 03:04:39 ]
- つまり良くねーんじゃねーか。
- 433 名前：あぼーん mailto:あぼーん [あぼーん]
- あぼーん
- 434 名前：349 mailto:sage [2006/10/11(水) 10:48:49 ]
- あぁ、ここに書き込んでたのか。
クラス名・変数名スレに書き込んだつもりだったんだけど
どこいったんだかわからなくなってた。
で、だ、これ、適当にしたほうがっていうのは
名前に関して２時間とか３時間とか１日とか悩んでも
結局変更することになるんだからほどほどにしとけって意味。
＃ ていうか、俺が悩んでつけた名前って言うのは
後から見るとピントはずれなことが多い。
リファクタリングをしないって意味じゃない。
Ｃだとリファクタリング大変だけど・・・。
- 435 名前：デフォルトの名無しさん mailto:sage [2006/10/11(水) 16:25:58 ]
- もういいって。
- 436 名前：デフォルトの名無しさん mailto:sage [2006/10/11(水) 22:28:55 ]
- >>434
＞俺が悩んでつけた名前って言うのは
名前で悩む前に設計がそれで適切なのかを考えてはどうか。
どの変数・関数がどんな役割を持つのかハッキリしないから
どんな名前つけて良いかが分からない。
役割がハッキリしてればその役割をそのまま名前に落とせば良い。
命名に詰まる時は、設計見直しのサインだよ。
- 437 名前：デフォルトの名無しさん mailto:sage [2006/10/12(木) 01:49:11 ]
- 他人が書いたコードを整理していると、しばしばネーミングに困る。
- 438 名前：デフォルトの名無しさん mailto:sage [2006/10/12(木) 02:47:24 ]
- >命名に詰まる時は、設計見直しのサインだよ
悩んだ末にすっきり名前が決まると、なんつーかこう、宿便が抜けたような爽快感。
それだけで仕事が終わった様な気持ちになれるな。
いや、冗談抜きで名前が決まった時点で仕事の何割かは完了してるわけなんだけどさ。
- 439 名前：デフォルトの名無しさん mailto:sage [2006/10/14(土) 21:34:09 ]
- lpszStrButtyakeBuririant0001->getSTRLFSPACETABCOMMA(index)->getSTRCONTROLSHIFTTAB(index_ind)->getChar(index_chr);
- 440 名前：デフォルトの名無しさん mailto:sage [2006/10/28(土) 15:46:28 ]
- うわー。。。
- 441 名前：デフォルトの名無しさん mailto:sage [2006/10/28(土) 18:35:19 ]
- malloc 関数で確保してるのに開放は delete 演算子。
2つのメンバ関数間でしか使われてないのでスタック割り当てで十分なはずだが、なぜかメンバ変数。
単体の画面アプリで外から呼ばれるはずもないのに、全てpublic関数。
const の検索結果0件。#DEFINEすら使ってない。何もかもリテラル。
default 句のない switch 文。default が不要な理由などどこにも書いてない。
JOIN すれば一発なのに、単体テーブルアクセスを内部で毎レコード繰り返してるDB参照処理。
どう見ても作り直しです。本当にありがとうございました。
- 442 名前：デフォルトの名無しさん mailto:sage [2006/10/28(土) 18:41:50 ]
- つ ラップしちゃえ！
- 443 名前：デフォルトの名無しさん mailto:sage [2006/10/30(月) 04:26:42 ]
- malloc(strlen(p)))
- 444 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 01:52:02 ]
- >>443
それだけ見ても、下手かどうか判断できないが。
例えばfgets()の直後に、改行文字を取り除いた文字列の複写を得たいのなら、妥当じゃないか。
- 445 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 13:04:04 ]
- >>444
その処理なら strdup でいいと思うが。
- 446 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 18:14:51 ]
- >>445
strdup がなかったら？
- 447 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 18:40:47 ]
- ↓ strdup無いなんてどんな環境だよ
↓ ○○環境にはないんだよボケ
↓ だったら自分でやりゃいいだろボケ
↓ ○○も知らないくせに下手なコードとか言うなボケ
↓ (グダグダな流れが続く)
という予感
- 448 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 19:39:16 ]
- >>446
＞strdup がなかったら？
malloc(strlen(p)+1) と strcpy だな。
- 449 名前：デフォルトの名無しさん mailto:sage [2006/10/31(火) 19:48:18 ]
- >>448
長さ覚えといてmemcpyの方がだいぶ速いことがあるよ。
- 450 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 22:02:11 ]
- for(;;){
if(judge){
break;
}
〜〜〜
〜〜〜
}
- 451 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 22:09:22 ]
- >>450
そういうのは終了条件（ if(judge){ break; } に相当するトコ）が
複雑になる可能性があるときに書くことあるなぁ。
- 452 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 22:13:26 ]
- >>451
理解の範囲を越えています。
- 453 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 22:21:05 ]
- >>443
コンパイルエラーじゃね？
- 454 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 22:53:55 ]
- >>453
その根拠は?
- 455 名前：デフォルトの名無しさん mailto:sage [2006/11/20(月) 23:53:49 ]
- 閉じ括弧が1つ多い。
- 456 名前：デフォルトの名無しさん mailto:sage [2006/11/21(火) 01:01:21 ]
- セミコロンがない
- 457 名前：デフォルトの名無しさん mailto:sage [2006/11/21(火) 11:19:53 ]
- セミコロン
は文の区切りです
- 458 名前：デフォルトの名無しさん mailto:sage [2006/11/21(火) 23:56:48 ]
- >450
↓だったら普通。また将来↓にする予定で>450と書くのも分からんではない
for(;;){
if(条件1){
break;
}
if(条件2){
break;
}
:
if(条件N){
break;
}
:
}
- 459 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 00:14:27 ]
- それは普通
for(;;){
if( is_must_to_break( ... ) ) break;
.
.
.
}
と書くだろ
- 460 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 00:16:25 ]
- >>458
while(is_proceding()) {
:
}
- 461 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 02:34:05 ]
- >>459
be動詞+助動詞+to+動詞
についてkwsk
- 462 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 02:38:51 ]
- All your bases are belong to us.
- 463 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 02:45:42 ]
- FIXME: bloken English
- 464 名前：デフォルトの名無しさん mailto:sage [2006/11/22(水) 22:49:57 ]
- >>461
火星語
- 465 名前：458 mailto:sage [2006/11/23(木) 00:11:50 ]
- >459,460
あれが一番常識的っつー意味で「普通」なのではなく
あーいう書き方をしてもそれほど変とも言い切れない、
という意味で「普通」です。
まぁかなり限定的だけど…
- 466 名前：デフォルトの名無しさん [2007/01/14(日) 15:04:58 ]
- このスレでまたコードの更正やっていいですか？ｗ
- 467 名前：デフォルトの名無しさん mailto:sage [2007/01/14(日) 20:30:42 ]
- ok
- 468 名前：デフォルトの名無しさん mailto:sage [2007/01/15(月) 14:01:00 ]
- while(*d++ = *s++);
- 469 名前：デフォルトの名無しさん mailto:sage [2007/01/17(水) 02:29:29 ]
- >>459
クソﾜﾛﾀ
- 470 名前：450 mailto:sage [2007/01/21(日) 18:48:43 ]
- #ifndef __HEADER_H__
#define __HEADER_H__
#define VOLTAGE0_STRING "voltage1"
#define VOLTAGE1_STRING "voltage2"
#define VOLTAGE2_STRING "voltage3"
#endif //__HEADER_H__
-------------------------------------
いろいろな意味で勘弁してよ〜
- 471 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 06:05:45 ]
- #define BASE \
unko_t **my_unko;\
int penis_count;\
struct SuperPenis {
BASE
...
}
struct OldCunt {
BASE
...
}
こういうの嫌い。ていうかマクロで変数定義済ませるコードはイヤ。
- 472 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 10:19:26 ]
- そんなヤツはおらんやろ〜
- 473 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 10:39:18 ]
- >>471
これは酷い。
- 474 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 16:23:19 ]
- >>471
うーん。本来
struct Base {
unko_t **my_unko;
int penis_count;
}
struct SuperPenis {
Base base;
...
}
struct OldCunt {
Base base;
...
}
であるべきなのを、
その様なベタ書きなデータ設計を強制されるなら
自分もやってしまいそうです
- 475 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 16:33:33 ]
- やるなよ。
- 476 名前：デフォルトの名無しさん mailto:sage [2007/02/09(金) 17:28:00 ]
- ところがこれが VC のコードで DECLARE_BASE とかいう名前だと
さほど違和感が無いあたり、漏れはMSに毒されてるな・・・
- 477 名前：デフォルトの名無しさん mailto:sage [2007/02/10(土) 05:53:40 ]
- オプソのライブラリとかでもよく使われてる。＞マクロによる一括変数定義
名前空間が特定のライブラリ色に染められるのが不愉快。
こんなだから、再利用性の低いコードばかり生まれるんだよな。
ちなみに、>>471 の方法は、Perlのコードに使われてたのを引っ張ってきた。
- 478 名前：デフォルトの名無しさん mailto:sage [2007/02/10(土) 11:52:37 ]
- それにしても例に使ってる変数名と構造体名はどうにかならなかったものか。
- 479 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 04:43:32 ]
- かたじけない。
- 480 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 11:49:31 ]
- >>471, >>476
最近の開発で、委譲ベースのMemoryPoolクラスを用意して、
#define MEMORYPOOL_DECLARE(CLS, NUM) ¥
typedef MemoryPool<CLS, NUM>::MyMemPool ¥
static void * operator new(size_t size) {return MyMemPool::alloc(size);} ¥
static void operator delete(void * p, size_t size) {MyMemPool::free(p, size);} ¥
みたいなのを作ったばかり・・・。
- 481 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 12:42:49 ]
- >>480
マクロを使わずにこんなんじゃ駄目だったの？
俺は多重継承でこういういmixin的な使用方法をわりとするけど。
template<typename CLS,int NUM>struct MemoryPool :{
...
...
static void * operator new(size_t size) {return alloc(size);}
static void operator delete(void * p, size_t size) {free(p, size);}
};
class CLS : public MemoryPool<CLS,100>{
...
};
- 482 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 13:28:54 ]
- >>481
出来ればそうしたかったんだけど、
MemoryPoolクラスの中身を↓な風に作ってて・・・。
template<typename T, size_t NUM>
struct MemoryPool {
unsigned char * buffer_[sizeof(T)];
...
static MemoryPool block_[NUM];
...
};
class Hoge : public MemoryPool<Hoge, 100> {
....
};
ってすると、sizeof(T)が確定しなくて、
コンパイルエラーになってしまう。
作りを大幅に変えずにエラーを回避する方法はないだろか・・・?
- 483 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 16:15:41 ]
- 開発環境は何使ってるんだ？
>>482 の方法で、Visual Studio 2005 Express だと問題なくコンパイルできるぞ。
ただ、
class Hoge: public MemoryPool<Hoge, 100>{
は、当然 Hoge の再定義になるから、
class Hoge1: public MemoryPool<Hoge, 100>{
... とする必要あるけど。
また、うまく動作するかは試してないけどな。
- 484 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 17:55:42 ]
- >>483
環境はVC++6.0です。
> class Hoge1 : public MemoryPool<Hoge, 100>
これだと新たにクラスを作らないといけないので・・・。
ところでさっきのコードは不正確でした。
実際はこんなかんじです。
Effective C++(だかModern C++ Design)を参考にしています。
template<typename T, size_t NUM>
struct MemoryPool {
union Chunk {
unsigned char buffer_[sizeof(T)];
Chunk * next_;
};
static Chunk block_[NUM];
static bool blockInitialized_;
static Chunk * head_;
static void * alloc(size_t size);
static void free(void * p, size_t size);
};
スレ違いのような気もするけど、自分がヘタレだと云う点では
間違っていないな・・・。
|
 |
