- 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/
- 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);
};
スレ違いのような気もするけど、自分がヘタレだと云う点では
間違っていないな・・・。
- 485 名前：481 mailto:sage [2007/02/11(日) 18:32:39 ]
- >>484
んーそういう場合は実体化のタイミングをずらせばOK。
..なんだけどVC6ってこれ大丈夫だっけ。VC6はtemplate絡みのバグ多すぎでイケるか自信ない...
template<typename T>
union Chunk{
unsigned char buffer_[sizeof(T)];
Chunk * next_;
};
template<typename T, size_t NUM>
struct MemoryPool {
static Chunk<T>*block(){static Chunk<T>block_[NUM];return block_;}
static bool blockInitialized_;
static Chunk<T>* head_;
static void * alloc(size_t size);
static void free(void * p, size_t size);
};
- 486 名前：デフォルトの名無しさん mailto:sage [2007/02/11(日) 20:03:46 ]
- >>485
おお。試してみたところ期待通りの結果(継承に置き換え可能)になりました。
(そのままでもOKでしたが、ChunkはMemoryPoolのインナークラスにしました)
なるほど実体化の遅延ですか。勉強になりました。
- 487 名前：デフォルトの名無しさん mailto:sage [2007/02/12(月) 03:29:09 ]
- 480です。しまった、ちゃんとお礼を云ってなかった。
>>481さん、>>483さん、ありがとうございました。
スレ汚しすみません。
- 488 名前：デフォルトの名無しさん mailto:sage [2007/03/17(土) 20:25:00 ]
- chokin = chokin != 0 ? chokin : 0;
- 489 名前：デフォルトの名無しさん mailto:sage [2007/03/17(土) 20:27:06 ]
- 最適化で消滅しそうだな
- 490 名前：デフォルトの名無しさん mailto:sage [2007/05/10(木) 16:49:30 ]
- ム板のその４の次スレ、ここでいいの？
- 491 名前：デフォルトの名無しさん mailto:sage [2007/05/10(木) 16:50:16 ]
- ↑
×ム
○マ
- 492 名前：デフォルトの名無しさん [2007/05/21(月) 18:57:37 ]
- age。
- 493 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 00:09:59 ]
- 今やってるプロジェクト、Cで作っているけど動的なメモリ確保は禁止なんだぜ？
リークの原因になるからだって。
- 494 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 00:30:24 ]
- >>493
一昔前のゲームプログラムとか（携帯機なら今でも）、組み込みだと当たり前では？
- 495 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 01:33:15 ]
- どこの「当たり前」だよ
- 496 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 01:48:40 ]
- そもそも動的に割り当てるほどのメモリが無いというならともかく、
リークするからとmallocを禁止するのは当たり前とは言えないだろうな。
- 497 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 08:08:12 ]
- mallocがクソな環境もあるんだよ
- 498 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 10:02:54 ]
- 普通に使っていてリークするようなクソmallocがあるのか、
そりゃたいへんだな。
- 499 名前：デフォルトの名無しさん mailto:sage [2007/06/01(金) 12:14:13 ]
- リンクオプションで領域確保しておいて、その中で自前で管理するってプロジェクトはあったな。
- 500 名前：デフォルトの名無しさん mailto:sage [2007/06/02(土) 12:22:58 ]
- リークするからと言う理由だけじゃなくて、そもそも必ず確保できるわけじゃない
可能性があるから malloc() 禁止のところは組み込みなら普通にある。
misra malloc とかでぐぐってみ。
- 501 名前：デフォルトの名無しさん mailto:sage [2007/06/02(土) 13:26:43 ]
- > 確保できるわけじゃない可能性があるから malloc() 禁止のところは組み込みなら普通にある。
そんなことは誰でもしってます
- 502 名前：デフォルトの名無しさん mailto:sage [2007/06/02(土) 13:36:13 ]
- > そんなことは誰でもしってます
とりあえず知らない人間はいないことをどうやって証明したのかｋｗｓｋ
- 503 名前：デフォルトの名無しさん mailto:sage [2007/06/03(日) 03:28:59 ]
- マロックさんをわるくいうんじゃない！！
- 504 名前：デフォルトの名無しさん mailto:sage [2007/06/04(月) 11:10:03 ]
- >>502
誰もが最初は知らなかった筈ですから、
「誰でも」が「遍く」ではないことは想像できることです。
では「誰でも」とはどういった範囲の人たちを指しているのでしょう？
それは >>500 のみぞ汁。
- 505 名前：>>500 mailto:sage [2007/06/04(月) 22:57:36 ]
- > 「誰でも」が「遍く」ではないことは想像できることです。
言い訳乙。
少なくとも俺は、唐突に「誰でも」って書かれたら、全世界の人間を想定する。
- 506 名前：デフォルトの名無しさん mailto:sage [2007/06/04(月) 23:02:35 ]
- お前が宇宙人を差別することは分かった。
- 507 名前：デフォルトの名無しさん mailto:sage [2007/06/05(火) 06:16:20 ]
- 自分が悪い時に、素直に間違いを認めず「だけど、〜」って関係ない事象を語って
煙にまこうとする奴を見た時は「ヘタだなぁ」と思うな。
コードじゃないけど。
- 508 名前：デフォルトの名無しさん mailto:sage [2007/06/05(火) 06:53:36 ]
- なんか>>504から臭ってこない？
- 509 名前：デフォルトの名無しさん mailto:sage [2007/06/05(火) 09:26:57 ]
- 組込みやらないやつはsbrkとか自分たちで実装する
環境なんて思いもよらないんだろうな。。。
組込みやらないやつは
void main
を見て
int main
じゃないから下手だ、とか言うんだろうな。
|
 |
