スレを勃てるまでもないC/C++の質問はここで 3

スレを勃てるまでもな ..

446:デフォルトの名無しさん
07/10/17 12:56:45
>>444
ReadDataにsの要素を渡して文字数のチェックをちゃんとすればおｋ
てかex変数が無駄に見える……

447:デフォルトの名無しさん
07/10/17 12:57:32
間違えた、sの要素数を、ね

448:デフォルトの名無しさん
07/10/17 13:52:11
ofstreamはdeleteした際には自動的にcloseされるますか？

449:デフォルトの名無しさん
07/10/17 16:56:00
yes yes yes

450:444
07/10/17 21:22:36
レスありがとうございます。

>>445
>128文字以上を入力するとマズい

実装時には、バッファ長を1kB位取ってごまかしちゃうつもりでしたが、
言われてみればその通りですね。変数ｉｐをチェックすればいいので、
バッファがあふれたらエラーを返すコードを追加しようと思います。

>>446
>ex変数が無駄に見える……

やっぱり？
ループの脱出条件をコーディングしながら決められるので、ついつい
やっちゃいます。こんな場合は、

　　while ( (c=fgetc(p) != EOF ) {

とか書くもんなんでしょうか？

451:デフォルトの名無しさん
07/10/17 22:24:27
>>450
exが無駄って言ったのは、
ループ中途脱出（break）を判別するためだけに使ってるように見えたから。
あれならwhileループ後のif文をそのままwhile内にぶちこめば、
ex使わないでいけそうだったし。
まあ、分かりやすく書けば良いと思うので、聞き流して結構です

do {
if( (c=fgetc(p)) == EOF ) {
s[0] = '\0';
return 0;
}
} while (IsDelimiter(c));
ループ終了条件を>>450みたくデリミタでなくEOFにすると、
この場合はreturnするために別にまたif文書かないといけないから、無駄そう

452:デフォルトの名無しさん
07/10/17 22:50:46
C言語なんだから、関数ばしばし作ろうよ。

・デリミタもしくはEOFまで読み飛ばし、最後に読み飛ばした文字を返す
・デミリタまで文字列を格納する
という2つの関数を作ればスッキリするよ。

453:444
07/10/17 22:59:06
>>451
>whileループ後のif文をそのままwhile内に

む。なるほど。
こちらの方がシンプルですね。

プログラムをできるだけ愚直に書こうと努力しているつもりですが、
スマートな解を示されると感動しちゃいます。勉強になりました。

>>452
>関数ばしばし

実装フェーズでは、そうなっちゃうかもしれません。
ただ、データ読み出しの関数なので、恐らく何万回も呼ばれる
ことになると思います。
実のところ、こんな冗長な関数を作るべきか、まだ悩んで
いるんです・・・

454:デフォルトの名無しさん
07/10/17 23:09:09
何万回程度の関数呼び出しコストなんて屁でもないぞ

455:デフォルトの名無しさん
07/10/17 23:15:46
というかこれでいいと思う

int ReadData(FILE *p, char *s){
int c, ip=0;

while( (c=fgetc(c)) != EOF)
if (IsDelimiter(c))
s[ip++] = (char) c;

s[ip] = '\0';
return !!ip;
}

456:デフォルトの名無しさん
07/10/17 23:21:41
いいのか、ほんとに。

457:デフォルトの名無しさん
07/10/17 23:29:45
なんでいいのか判らん・・・

458:455
07/10/17 23:32:29
あ、すまん、所々ミスってる上にそもそも動作が違うなｗ

これでどうだ

int ReadData(FILE *p, char *s){
int c , ip=0 , flag=0;

while( (c=fgetc(p)) != EOF ){
if (flag){
if (!IsDelimiter(c)) s[ip++] = (char) c;
else break;
} else if (IsDelimiter(c)) flag=1;
}

s[ip] = '\0';
return !!ip;
}

うーん、あんまりスマートじゃない……

459:455
07/10/17 23:34:58
>>456-457
いや、IsDelimiterの戻り値を勘違いしてた上に、
そもそも動作を全く勘違いしてたｗ

これならわざわざ書き直す必要なかったですね……
偉そうにでしゃばってすみませんorz

460:444
07/10/17 23:44:46
各位殿

色々ご指導ありがとうございました。
とりあえず仮決めですが、下記のようにすることにしました。
色々いじったら、仕様が変わっちゃった・・・

>>454
ちょっと安心しました。

>>455
読み込み長を返すアイデア頂きました。
ありがとうございました。

461:444
07/10/17 23:45:24
int ReadData(FILE *p, char *s)
{
int c, ip;

/* 非デリミタ文字が現れるまで読み飛ばす */
do {
if ( (c = fgetc(p)) == EOF ) {
s[0] = '\0';
return 0;
}
} while (IsDelimiter(c) == 1);
s[0] = (char) c;

/* デリミタが現れるまで，文字列を積み上げる */
ip = 1;
while (1) {
if ( (c = fgetc(p)) == EOF ) break;
if ( IsDelimiter(c) == 1 ) break;

s[ip] = (char) c;
ip++;
if (ip >= MAXBUFSIZE) {
ip--;
s[ip] = '\0';
return -ip;
}
}
s[ip] = '\0';
return ip;
}

462:デフォルトの名無しさん
07/10/17 23:58:02
自分なら、↓こんな感じかなぁ。

int ReadData(FILE* pFileIn, char* strOut) {
*s = '\0' ;
int countDelimiter = 0 ;
int countOutput = 0 ;

while(1) {
int c ;

c = fgetc(pFileIn) ;
if (c == EOF) break ;

if (IsIsDelimiter(c)) {
++countDelimiter ;
if (countDelimiter ==2) break ;
} else if (countDelimiter == 1) {
*strOut = (char)c ;
++strOut ;
}
}

return countOutput ;
}

463:462
07/10/17 23:58:45
おっと、間違った。

int ReadData(FILE* pFileIn, char* strOut) {
int countDelimiter = 0 ;
int countOutput = 0 ;

while(1) {
int c ;

c = fgetc(pFileIn) ;
if (c == EOF) break ;

if (IsIsDelimiter(c)) {
++countDelimiter ;
if (countDelimiter ==2) break ;
} else if (countDelimiter == 1) {
*strOut = (char)c ;
++strOut ;
}
}

*strOut = '\0' ;
return countOutput ;
}

464:デフォルトの名無しさん
07/10/18 00:14:00
countDelimiterで制御するのですね。
ちょっとトリッキーかな・・・

465:462
07/10/18 00:22:08
やりたい処理を素直に書いただけなんだけどなぁ・・・。

466:デフォルトの名無しさん
07/10/18 00:35:46
文化が違うのがオモロイなぁ・・・

467:デフォルトの名無しさん
07/10/18 01:16:12
int ReadData(FILE *fp , char *out , int n){
int c , i=0;

while(i<n-1){
c = getchar(fp);
if (c == EOF || (IsDelimiter(c) && i != 0)) break;
if (!IsDelimiter(c)) out[i++] = c;
}
out[i] = '\0';
return i;
}

468:デフォルトの名無しさん
07/10/20 22:47:48
エラーメッセージを出力するところなんですが、

#define ABC "ABCの時のエラーメッセージ"

などと定義しておいて、エラーメッセージの出力関数には、
エラーコードの「ABC」のみを引数で渡すようになっています。
ただの文字列に変換する場合はこれでよかったんですが、
エラーの時に出た数字もエラーメッセージに含める場合も追加されました。
その数字は、書式指定（%dとか%lfとか）なので、同じようにマクロで
文字列を生成しようとして、つまづいてしまいました。

こんな感じで書いてみたんですが、当たり前だけどstrをどこで定義すれば
よいのかがわかりません。

#define ABC(gStr) sprintf( str, "ABCの時のメッセージ( %g ).", gStr )

引数なしの時と同じように、文字列（str）をそのまま生成するには
どう書いたらいいですか？

469:デフォルトの名無しさん
07/10/20 23:28:42
>>468
> エラーコードの「ABC」のみを引数で渡す

あなたは自分がやっていることを理解してない。
ABCはエラーコードなんてものではなく、文字列リテラルへのポインタ。
なお、複数の同じ内容の文字列リテラルが1つにまとめられることは保証されていない。

そして、マクロが何者なのか、理解していない。
いまのコンパイラはプリプロセッサが統合されているので、わかりにくいが、
ソースコードは、プリプロセッサによってマクロが展開された後に、コンパイルされる。
マクロは「定義」するものだが、その定義とは、日本語の定義とは意味が異なる。

> strをどこで定義すればよいのかがわかりません。

エラーはネストするの?
シングルスレッドとマルチスレッド、どっち?

470:469
07/10/20 23:35:01
ああぁ

呼び出し側の関数にエラーコードとして返したりせず、
エラーメッセージを出力する関数にしか渡さないのなら、
適当に自動変数で持てばいいと思うよ。

CとC++どっち?

471:デフォルトの名無しさん
07/10/21 00:00:21
>>468
#define ABC(gStr) "ABCの時のメッセージ("#gStr")"

ってgStrが変数なのか
知らね

472:デフォルトの名無しさん
07/10/21 00:21:50
基本的なことが理解できてない人に、
場当たり的に教えるのは、
効率が悪すぎる。

473:デフォルトの名無しさん
07/10/21 00:42:22
468です。いくつかのご回答、ありがとうございました。
>>469
自分のしていることを、自分で理解していないのかもしれません。
> エラーコードの「ABC」のみを引数で渡す
は間違いで、エラー出力関数への引数には、文字列になります。
エラーはネストしません。また、シングルスレッドです。

>>470
C++です。
「適当に自動変数」というのは、文字列自動変数をエラーコード
毎にその場でベタにつくって、エラー出力関数に渡す、という感じでしょうか？

>>472
効率よく教わるためには、どの基本的なことを勉強すればよいでしょうか。

474:デフォルトの名無しさん
07/10/21 00:56:30
>>473
ネストせず、シングルスレッドなら、str をグローバル変数にしてしまうのも可能。
簡単だけど、あんまり良いやり方ではないが、とりあえず最小の変更で済むかな。
strなんて短くてどこにでもありそうな名前は、変えるべきだよ。

> 「適当に自動変数」というのは、文字列自動変数をエラーコード
> 毎にその場でベタにつくって、エラー出力関数に渡す、という感じでしょうか？

そう。

475:wolf ◆8VH3XAqjlU
07/10/21 11:00:52
>>468
>>473
C++なら普通polymorphismを使いますが兎も角動かすには
#define ABC(i_code) { char str[256]; sprintf(str, "ABCの時のメッセージ( %d ).", i_code); printf("%s\n",str); }
ブロックについては　URLﾘﾝｸ(www.geocities.jp)

476:デフォルトの名無しさん
07/10/21 11:47:16
>>475
ありがとうございます。
実環境で確認してみます。

477:デフォルトの名無しさん
07/10/21 16:45:38
おま・・・。
せめてdo-while

#define ABC(i_code) do{ char str[256]; sprintf(str, "ABCの時のメッセージ( %d ).", i_code); printf("%s\n",str); }while(0)

で、ソレは俺も質問見たとき考えたけど、マクロの意味というか役割というかが変わってるから、
「なんでエラーメッセージをマクロにしたいのか」が分からないと使えるかどうか分からなかった。

個人的には、ストリングテーブル的に↓とかで十分かな、と思う。

#define ABC "ABCの時のメッセージ( %s )."
(共通な文字列リテラルの切り出し。日本語版やら英語版やらを作るときに使う)

478:デフォルトの名無しさん
07/10/21 16:56:29
>>477
do whileハックを使わないといけないのは、古いコンパイラ。

479:デフォルトの名無しさん
07/10/21 17:33:37
>>478
それはすまない。俺の勉強不足のようだ。
ついては↓のコードがエラー無くコンパイル通るような
「新しいCコンパイラ」は何かを教えてくれないか？

#include <stdio.h>
#define macro(x) {puts(x);}

int main()
{
if(1) macro("str1");
else macro("str2");
return 0;
}

480:デフォルトの名無しさん
07/10/21 17:56:42
>>479
そういう{}のないif文は、コーディング規約で禁止。

481:デフォルトの名無しさん
07/10/21 18:00:58
マクロには;をつけないっていう選択肢はなし？

482:デフォルトの名無しさん
07/10/21 18:32:53
どう考えてもバグの原因になるだろ。

483:デフォルトの名無しさん
07/10/21 18:58:42
小細工禁止!

マクロによる関数もどきはダメ。
インライン関数やテンプレート関数を使うべし。

484:デフォルトの名無しさん
07/10/21 19:06:06
コンパイルに５分かかるプログラムを作ったら一人前ですか

485:デフォルトの名無しさん
07/10/21 19:07:23
そうだねえ。
再帰テンプレートとか作ると簡単に5分超えるね。
練習にはいいんじゃない？

486:デフォルトの名無しさん
07/10/21 19:10:36
で、規約やら小細工やら良いんだけど

＞do whileハックを使わないといけないのは、古いコンパイラ。
誰かコレを解説して貰えんか？
コンパイラの新旧が、マクロを括るブロックにdo-whileを使うことと
どう関係してんだ？

487:デフォルトの名無しさん
07/10/21 19:11:40
勘弁してやれｗ　勘違いだろｗ

488:デフォルトの名無しさん
07/10/21 19:42:09
制御文なしに中括弧だけを書くとエラーになるような古いコンパイラがあったんじゃないか。

489:デフォルトの名無しさん
07/10/21 20:31:08
>>475
それだとABC(data);って書いたら駄目だわなｗ
#define ABC(i_code) do{ char str[256]; sprintf(str, "ABCの時のメッセージ( %d ).", i_code); printf("%s\n",str); }while(0)
こうだな
まあ受け売りだが

てか一回sprintf経由する必要はあるの？
#define ABC(i_code) printf("ABCの時のメッセージ( %d ).\n", i_code)
これじゃ駄目なんかいな

490:489
07/10/21 20:33:43
やべ、俺リロードしなさすぎｗ
吊ってくるｗｗ

491:デフォルトの名無しさん
07/10/21 20:34:54
エラーメッセージを生成するのと、表示するのは、分離したほうがいいと思う。

492:デフォルトの名無しさん
07/10/22 10:50:35
つーか、元質はデバッグ出力関数に渡す引き数を生成したいんじゃないかな?
だとしたら、>475以下はダメだろう。

493:デフォルトの名無しさん
07/10/22 19:39:51
C言語に関して質問です。
構造体の各要素に添え字を使ってアクセスすることはできないのでしょうか？

例えば
struct OBJ{
int x;
int y;
char c;
char str[10];
};

main()
{
struct OBJ tmp;
tmp.[0] ← xのこと
tmp.[1] ← yのこと
tmp.[2] ← cのこと
tmp.[3] ← str[]のこと
}

といった感じのことがしたいのです。実現可能な方法があれば是非お願いします。

494:デフォルトの名無しさん
07/10/22 19:43:42
unionを使えば出来ないことはないけど、正直お勧めしない。

495:デフォルトの名無しさん
07/10/22 19:44:02
C++なら・・・無理か。
型もバラバラだしな。
添え字に変数を使って

tmp.[i]

ってしたとき、何型で扱うつもりなんだ

496:デフォルトの名無しさん
07/10/22 19:44:13
>>493
限定的にはunionを使えばできる。
ややこしくはなるがポインタをキャストしてもできる。
どちらにしろ、サイズの違う型を統一的には扱えない。

497:デフォルトの名無しさん
07/10/22 20:19:14
>>493
なんで、そんな変なことをしたいの?

498:デフォルトの名無しさん
07/10/22 20:22:08
テンプレートクラスでtemplate　<Type T>のTにクラスを入れることが可能なのですか？
int型などを入れるメリットは分かるのですが具体的にどんなクラスを入れれば役に立つのか教えていただけないでしょうか？

499:493
07/10/22 20:29:47
回答して下さった方々有難う御座います。
for文のインデックスを利用出来たらと思いました。
しかし型などが違うのでその都度場合わけをしないといけないので出来たとしても結局のところ手間は増えますね。

500:デフォルトの名無しさん
07/10/22 20:33:51
>>498
可能です。
STLのvector<T> はTの配列と置き換えられる動的な再配置も可能なクラスですが
当然組み込み型以外でもvectorを使うことが出来ます。

501:デフォルトの名無しさん
07/10/22 21:15:25
float a;
a=1;
print("%d",a);
を実行すると
1072693248っていう数がでてくる。
これってどういうこと？

502:デフォルトの名無しさん
07/10/22 21:24:52
>>501
言語およびコンパイラは何？
"%f" を使うべきである事は知っていて、なぜそうなるのかが知りたいということ？

503:デフォルトの名無しさん
07/10/22 22:53:01
>>501
printfの引数の2つ目以降の型って決まってないよね。
だから、printfにとっては、どんな型の値を引数に渡されたのか、わからない。
そのため、書式文字列に%dとあれば、intだと思って処理するわけ。

504:デフォルトの名無しさん
07/10/22 22:54:09
>>498
可能。

int型を入れるのと同じメリットが、stringクラスを入れるのにもあると思うのだが。
stringの配列なんて使う機会ないか？ないなら、まぁ仕方ない。

505:デフォルトの名無しさん
07/10/22 22:58:36
list<vector<string> >
とかな。

506:デフォルトの名無しさん
07/10/22 23:28:17
>>505
そんなこと、したことないぞ。

まず、stringのように重いものを、ポインタや参照ではなく値として持つvectorに格納するのが、乱暴だ。

次に、vector<string>を直にlistに格納するのも、乱暴だ。
vector<string>に何か意味があるのであれば、それなりのクラスにすべきだ。

507:デフォルトの名無しさん
07/10/22 23:35:45
乱暴ってのが何を意味してるかわからんが、
別にstringは重くないしvector<string>も十分に速いぞ

508:デフォルトの名無しさん
07/10/22 23:36:17
あほすぎる・・・

509:デフォルトの名無しさん
07/10/22 23:46:20
>>506が何を言いたいのかﾜｶﾝﾈ
乱暴ってどういう意味？
vector<string>を使うなって言ってるの？

510:デフォルトの名無しさん
07/10/22 23:57:05
vector<string>を内包したオブジェクトつくって
インターフェースをきちんとしたオブジェクト指向な作り方しろってことか…。

C++だからって必ずオブジェクト指向で作らんでもいいだろ…

511:デフォルトの名無しさん
07/10/23 00:00:22
>>506は「そんなこと、したことないぞ」と言っている通り、思い込みで書き込んでいるんだよ。

512:デフォルトの名無しさん
07/10/23 00:02:01
vector<string> に何か意味をもたせるとして、所詮 vector 程度なら変数名で十分示せると思う。
vector<string> はオブジェクト指向だよ。

513:デフォルトの名無しさん
07/10/23 00:27:02
>>507
それは実装依存だな。

どーでもいい場所なら乱暴なコーディングをしても構わないし、すべきだろうが、
重くないとか十分に速いとか言ってるようじゃ、C++使う意味がないんじゃないか。

>>509
文字どおりさ。
動けば何でもいい、そういうやりかた。

>>510
何らかの意味があるのだからtypedefすべきだし、
大抵の場合は何か付随したデータがあるわけで、
それと本体のvector<string>を別管理するのは、
不適切だと思う。

>>512
オブジェクト志向と言いながら、
BASICやスクリプト言語のように、型にルーズでどうしようもない代物を作る人を、見てきたわけだが。
彼らは万能ナイフを作ることに喜びを感じているようだが。

514:デフォルトの名無しさん
07/10/23 08:58:30
ふ～ん。君はそういう事に喜びを感じているんだね。

515:デフォルトの名無しさん
07/10/23 09:02:19
そこでboost::ptr_vector, ptr_list
内部ではvoid*として保持されているためコードサイズ膨張対策にもなる
重いオブジェクトのコンテナならこれで決まり☆

516:デフォルトの名無しさん
07/10/23 10:21:21
>>513 のやり方を推し進めたら、プログラム中に出てくる概念全部を
typedefなりクラス化なりすることになるな
アホとしか思えん

いちいち型を定義してたら型がむやみに大量発生するし、
そのせいでtypedefされた型の元の型がわからなくなって
どういう操作ができるのかいちいち調べる羽目になる

名前の一覧は vector<string> names; で十分
過度の抽象化は毒であると知れ

517:デフォルトの名無しさん
07/10/23 10:44:09
>>516の書き込みをみて
VC++が頭をよぎった。
ふとよぎった。

518:デフォルトの名無しさん
07/10/23 10:50:52
>>513は早すぎる最適化をやっちゃうタイプだな

519:デフォルトの名無しさん
07/10/23 10:51:54
>>516
> 名前の一覧は vector<string> names; で十分

名前だけで単独なんだ・・・どういう場合?

520:デフォルトの名無しさん
07/10/23 15:36:35
スイッチを押すと外部割り込みでInterrupt関数が呼び出されて、スイッチを押した回数を数えて変数countに保存し、
タイマ割り込みで一定時間ごとにTimer関数を呼んで押した回数ごとに行動を変えるプログラムを作りたいです。
グローバル変数を使う以外に二つの関数の間でcount変数の値を渡す方法はありますか？

521:デフォルトの名無しさん
07/10/23 16:04:40
つまり、二つの割り込み関数からそれぞれ設定と参照を行いたいわけだな。

グローバル変数の代わりに、それらの関数をメンバとして持つクラスを作ってそのメンバとしてcountを持てばいい。
勿論、そのクラスの生存期間に注意が必要。

522:520
07/10/23 16:23:07
せっかく回答してくれたのにすみません。
C++ではなくCで行う方法を教えてください。

523:デフォルトの名無しさん
07/10/23 16:37:47
>>520
設定も参照もどちらも割り込みルーチンから行なうのなら、
グローバル変数にしておくのが無難だがなぁ。
グローバル変数にしたくない理由は?

524:デフォルトの名無しさん
07/10/23 16:44:44
メインで変数作ってポインタ渡せば?
グローバルと同じような物だけど

525:520
07/10/23 17:49:50
>>523
やはりグローバル変数がいいですか・・・。
プログラム作ってる途中でよくわからなくなることが多くて個人的にあまり好きじゃないだけです。

526:デフォルトの名無しさん
07/10/23 18:12:44
>>520
gccなら関数内関数で外側のローカル変数を参照出来るはず。
グローバル変数以上に問題のあるコードかもしれんが。

527:デフォルトの名無しさん
07/10/23 19:13:25
>>524
割り込み処理だと渡しようがないと思うんだ。

528:wolf ◆8VH3XAqjlU
07/10/23 21:52:40
>>520
>>525
１．割り込みマスク外した後ならsignal関数では？
２．共通の割り込みハンドラで割り込みを１箇所で受けられるかな？

529:デフォルトの名無しさん
07/10/24 07:25:37
ここで聞いてよいのかわからないのですが・・・
CPadてVistaでも使えるものなんでしょうか？

とりあえず実行すると～.exeが見つかりません～てなエラーが出てきます。
これはOSとCPadとの相性が悪いのか、自分が何かミスを犯しているのか、どちらなのでしょう？

530:デフォルトの名無しさん
07/10/24 07:29:32
調べてみたら自己解決できそうなので、自分で調べてみます。

531:デフォルトの名無しさん
07/10/24 07:32:45
解決、スレ消費申し訳ありませんでした。

532:デフォルトの名無しさん
07/10/25 16:20:24
VCでコントロール(仮にオリジナルのペイントソフト)を作成して
それをVB.NETから呼び出して、VBで作成したインターフェース内に埋め込みたいのですが
VCで作ったものをDLLにまとめることくらいしか現在わかっていません。
何に対応(例えば継承させるクラス)させればよいのかなどおおまかに教えていただけないでしょうか？

533:デフォルトの名無しさん
07/10/25 16:23:09
VBのスレで聞いたら？

534:デフォルトの名無しさん
07/10/25 18:26:14
>>532
> 何に対応させればよいのか
↓
> コントロール

自分で書いていて気がつかないとは。

535:デフォルトの名無しさん
07/10/25 19:31:24
ABのどっちが好き？

#include <iostream.h>
void f(unsigned int u){
　printf("%u", u);
}
main(int argc, char **argv){
　unsigned int u = 0;
　if(argc > 1){
A　　u = static_cast<unsigned int>(atoi(argv[1]));
B　　sscanf(static_cast<const char *>(argv[1]), "%u", &u);
　}
　f(u);
}

536:デフォルトの名無しさん
07/10/25 19:32:55
lexical_cast<>

537:デフォルトの名無しさん
07/10/25 19:55:07
qa3457974

538:デフォルトの名無しさん
07/10/26 01:33:48
>>535
エラー処理してない点は同じなのでどちらも却下。

だ　が　敢　え　て　言　わ　せ　て　も　ら　う

if(1 < argc){

だと！

539:デフォルトの名無しさん
07/10/26 06:44:04
>>538
if(argc > 1)
より
if(1 < argc)
の方がいい理由も添えて書いてくれ。

540:デフォルトの名無しさん
07/10/26 11:33:39
いまどきiostream.hって・・・

541:デフォルトの名無しさん
07/10/26 13:57:17
.h

542:デフォルトの名無しさん
07/10/26 14:13:27
ｂｆ

543:538
07/10/26 17:15:12
>>539
半分冗談だったんだけどね（＾＾；　適当に流して欲しかったんだけど。
単に俺のスタイルなだけ。

例えば、『aが１よりおおきくて、かつ１０より小さい』という命題について

if(1 < a && a < 10)

と書くと、常に右が大きくなるでしょ。
完全に主観だけど、あとでソースを見返すとき分かりやすいんですよ。
（数学の）実数線も右が大きいし、なんとなくそういうクセをつけました。

それだけ。

544:デフォルトの名無しさん
07/10/26 17:36:45
>>543
俺もそう書く
理由も同じ

545:デフォルトの名無しさん
07/10/26 21:27:52
vectorは「動的配列」という事なのですが、
これは、

void init_vct()
{
　　vector< vector<int> > v;
　　v.resize(3);
　　for(int i=0; i<v.size(); ++i) v[i].resize(5);

　　int array[] = { 0, 1, 2, 3, 4};
}

int main()
{
　　init_vct();

　　return 0;
}

としたときに、init_vct()関数を抜けても、
int型配列 arrayと違って、vector< vector<int> >型配列 vは、
メモリ上から消されないということですか？

//初歩的な質問で申し訳ありません。

546:デフォルトの名無しさん
07/10/26 21:33:34
消えるだろ　ふつう

547:デフォルトの名無しさん
07/10/26 21:37:01
>>545
関数中のvはスコープから抜ける時デストラクタが呼ばれ、
確保された領域は開放される。

v.push_back(i)などで、どんどん追加でき、
サイズが動的に拡張されていく配列。

548:デフォルトの名無しさん
07/10/26 21:57:37
>>543
if( x==0xFF ) などと比較したいのに、誤って代入文 if( x=0xFF ) と
書いてしまう凡ミスを防ぐために変数を右に　つまりif( 0xFF==x ) と
書くと良い　みたいなことがドコカの本で書かれていた記憶がある

549:デフォルトの名無しさん
07/10/26 22:00:36
>>548
ぱっと見て「ヘタだなぁ」と思うコード　その６
ｽﾚﾘﾝｸ(tech板)l50
ここで今、思いっきりその話になってます。

550:デフォルトの名無しさん
07/10/26 22:33:42
>>543
俺は『aが１よりおおき』いなら、

if(a > 1)

と書くな。『1がa未満』なら逆に書くけど。

俺も一時期>>543と同じ書き方してたけど、
仕事でプログラム書き始めてからは仕様書通りに書く癖が付いた。

551:デフォルトの名無しさん
07/10/26 23:35:50
sscanf()はバッファオーバーフローがあるから
使ってはいけないんじゃなかったっけ？
普通atoi()だろ。

552:デフォルトの名無しさん
07/10/26 23:39:02
どちらでも仕様書どおりだよ。

553:デフォルトの名無しさん
07/10/26 23:50:21
>>545です。
>>546さんありがとうございます。

以下のようなプログラムを書いたのですが、
正しく表示されるのはたまたまメモリに残っているせい？

#include<iostream>
#include<vector>
using namespace std;

class VEC{
　 private:
　 public:
　　 vector< vector<int> > vec;
　　 VEC(int num);
　　 VEC(){};
};

VEC::VEC(int num)
{
　 vec.resize(3);
　 for(int i=0; i<3; ++i) vec[i].resize(5);

　 for(int i=0; i<vec.size(); ++i){
　　 for(int j=0; j<vec[i].size(); ++j){
　　　 vec[i][j] = num+j;
　　 }
　　 num+=vec[i].size();
　 }
}

554:デフォルトの名無しさん
07/10/26 23:50:58
class A{
　 private:
　 public:
　　 A();
　　 void Show();
　　 vector<VEC> v;
};

A::A()
{
　 int j=0;
　 for(int i=0; i<10; ++i){
　　 v.push_back(VEC(j));
　　 j+=10;
　 }
}

555:デフォルトの名無しさん
07/10/26 23:51:55
void A::Show()
{
　 for(int i=0; i<v.size(); ++i){
　　 for(int j=0; j<(v[i].vec.size()); ++j){
　　　 for(int k=0; k<(v[i].vec[j].size()); ++k){
　　　　 cout << v[i].vec[j][k] << " ";
　　　 }
　　　 cout << endl;
　　 }
　　 cout << endl;
　 }
}

int main()
{
　 A aa;
　 aa.Show();

　 return 0;
}

556:デフォルトの名無しさん
07/10/26 23:52:44
>>553
関数抜けた時点で解放され、別の値に変わる可能性はある
ゴミ箱の中身消したけど復活できるかどうかと同じ事だ

557:デフォルトの名無しさん
07/10/27 00:04:16
>>553-555
別にどこもおかしなようには見えないが、どこに問題があると思ってるの？

558:デフォルトの名無しさん
07/10/27 00:49:51
>>545です。
>>546
>>547
>>556
さんのご指摘もありますが、
newで確保されていないvector配列の中身（vector< vector<int> > vec）が、
最後まで保持されていること・・・
についてです。

vecはクラスの中で宣言はされているものの、
VEC()で代入される中身は局所的なもので、
VEC()を出たらvec配列の中身は、開放されるのかなぁと。

サイトで「vectorは"動的"配列」というのを目にしたのですが、
これは配列のサイズを"動的"に確保するだけで、
newのメモリの"動的"確保とは別物なのではないか？
と思ったので、試しています。

559:デフォルトの名無しさん
07/10/27 00:51:53
>>545です。
かなりトンチンカンなこと言ってると思いますが、
ご指導お願いいたします。m(_ _)m

560:デフォルトの名無しさん
07/10/27 00:57:53
>>559
クラス内の関数は、抜けても値は保持される　常識だろう　vectorとか関係なし

561:デフォルトの名無しさん
07/10/27 00:59:15
>>559
基本的なことが、いくつも、理解できていない気配がする。
泥縄的に教えてもしょーがないので、ちゃんとC++の基本を勉強したほうがいいよ、としか言えない。

562:デフォルトの名無しさん
07/10/27 00:59:20
オブジェクト指向っていうのは、データに対してあれこれ作業をするって事だ
あれこれ作業するごとにデータが消えていたら駄目だろ

563:デフォルトの名無しさん
07/10/27 01:00:16
v.push_back(VEC(j));
の
VEC(j)はどこいっちゃったの？

ってこと？

564:デフォルトの名無しさん
07/10/27 01:20:13
>>545です。
みなさん、レスありがとうございます。

クラスの中でサイズを指定しないと配列の宣言てできませんよね。
もしコンストラクタの引数の値を配列のサイズにしたいときには、
データメンバに、
int a*;
を入れて、
コンストラクタの中で、
a=new int[コンストラクタの引数の値];
と（私は）します。

だから、私はvector配列をクラスのデータメンバとするときも、
配列のサイズを指定しないから、
どこかでnewすることでvector配列の領域を確保しなければ
ダメなのかぁと思ってしまったわけです。
newをせずに、ただ値を代入しただけでは、
局所な値を代入しているに過ぎないなどと思ってしまいました。

色々考えているうちに、頭の中がごちゃごちゃになってしまいました。
己の力の無さを改めて自覚しました。

565:デフォルトの名無しさん
07/10/27 01:31:07
>>564
vectorの中でnew/deleteしてる。

566:デフォルトの名無しさん
07/10/27 01:32:02
コンストラクタとデストラクタで何か表示するクラス作って
vectorにpush_backしてると何か閃くかもしれない

567:デフォルトの名無しさん
07/10/27 01:38:24
クソッタレな本に引っかかってしまった可哀想な犠牲者なんだろうな。
きちんとした本を読んでいれば間違わないような誤解してるんだもの。

568:デフォルトの名無しさん
07/10/27 01:50:13
これからプログラムを勉強する俺に是非教えてほしい。

ちょっとしたアプリケーションを作りたくて
なんとなく、適当に参考書見て、高校の頃少し勉強した、デルファイに似てるような気がしたから
VBの参考書買って、MSからVS2008bata2をダウンロードしてサンプルプログラムを作っては
自分なりに書き換えて違う結果を出力させたりと、少しずつ理解してきてる・・つもり。

けど、VBとC++はどのスレ見てもVBは糞ってよく見るんだけど、それはなぜですか？
自分は結構VBがなじんできてるしこの頃。。
実際、プログラムの組みやすさ、軽さ、ほか色々。
ここで聞くのはどうかと思うけど、できれば教えてほしい

もし、VBはやめとけ！と言うならばお勧めの言語を教えていただきたい。

569:デフォルトの名無しさん
07/10/27 01:54:18
すいません、教えていただけないでしょうか。

Ｃ言語で、２つのスレッドが交互にグローバル変数を参照→インクリメントして、
決められた値になったら終了する。という処理を作りました。
ここからグローバル変数を用いずにスレッド間でデータを渡しあうことで同様の処理を
実現したいのですが、どうすればいいでしょうか。

自分で調べたところ、スレッド間ではお互いのメッセージキューへ格納→読み出しを行えば
データを渡せることがわかったのですが、見当違いな考えかもしれません…。
どのような方法・関数を用いればいいか、ご教授のほど宜しくお願いします。

570:デフォルトの名無しさん
07/10/27 01:59:25
OSぐらい書けよ

571:デフォルトの名無しさん
07/10/27 02:02:14
>>570
申し訳ありませんでしたorz
作成環境は以下のとおりです。

ＯＳ：Windows XP
コンパイラ：Microsoft Visual C++ 2005 Express Edition

572:デフォルトの名無しさん
07/10/27 02:03:22
VBが糞と言われる理由

1. VB自体ではなく、VB使いが糞であるため
2. VBでプログラミングを勉強すると糞になるため
3. VBが適切な手段の場合には良いが、そうではないのにVBを使う糞がたくさんいるから
4. VB.NETではないVBは既に終わっており、なおかつ、VB.NETをやるならC#をやるべきだから

573:デフォルトの名無しさん
07/10/27 02:05:35
>>569
質問が抽象的すぎる。
もっと具体的に詳しく。

また、なぜ、グローバル変数をやめようと考えているのか、その理由も。

574:デフォルトの名無しさん
07/10/27 02:16:09
　　∧＿∧ ｸﾜｯ!
　∩｀iWi´∩　
　ヽ　|m| .ノ
　｜.￣｜
　　Ｕ⌒Ｕ

575:デフォルトの名無しさん
07/10/27 02:17:26
>>574
それ何か可愛いな。

576:デフォルトの名無しさん
07/10/27 02:25:48
>>573
色々と足りない部分が多くて申し訳ありません。
初心者の為、処理が的確ではないかもしれませんが現在の処理は以下のとおりです。

unsigned __stdcall thread0(void *lpx)
{
　/*排他制御にミューテックスを使っています*/
　HANDLE hM;
　hM = *(HANDLE*)lpx;

　while ( 100 > nCnt )
　{
　　WaitForSingleObject( hM, INFINITE );
　　nCnt++;
　　printf("nCnt = %d", nCnt);
　　ReleaseMutex( hM );
　}
　printf("thread0 END");
　return 0;
}

これと同様の処理を行うスレッドがもう一つありまして、
交互にグローバル変数nCntをインクリメントして、100になったら終了させるようにします。
このnCntをグローバル変数ではなく、ローカル変数にしてスレッド間で値を渡し合うことで、
同様の処理ができるようにしてみたいのです。

グローバル変数を使わない理由ですが、現在Ｃ言語を勉強中で、
様々な処理方法を試してみたいと思ったからです。

577:デフォルトの名無しさん
07/10/27 02:33:47
そのスレッドに渡すlpxでミューテックスと一緒に共有したいデータを渡すとか。

struct thread_data {
HANDLE hM;
int nCnt;
};
unsigned __stdcall thread0(void *lpx)
{
struct thread_data *data = lpx;
　/*排他制御にミューテックスを使っています*/
　HANDLE hM;
　hM = data->hM;

while ( 100 > data->nCnt )
以下略

578:573
07/10/27 03:03:46
>>576
なるほど。文字どおりの処理をしていたのね。ごめん。

まず、そのプログラムには重大な問題がありそう。

それはともかく、
スレッド毎にメッセージ・キューを持たせて、相互にメッセージを送り合う
ということだと、まず、最初の選択として、
Windowsが提供してくれるメッセージ・キューを使うのか、
それ以外のメッセージ・キューを使うのか、
というのがあるが、とりあえず前者の方向で。

Win32APIのPostThreadMessageを使う。
まずは、MSDNライブラリ等でPostThreadMessageの解説を見よう。

579:デフォルトの名無しさん
07/10/27 03:04:25
>>577
ご助言ありがとうございます。
ただ、私が今回試してみたいのはスレッド間通信？というものになりそうなんですが、
このコードがそういった処理になるのでしょうか。
サンプルコードが分かり辛いと思うので、自信がありません…。

580:デフォルトの名無しさん
07/10/27 03:07:12
>>578
ありがとうございます。
エラー処理や組み方自体についても、学ぶべきことが多いですorz

PostThreadMessageについて、早速見てみます。

581:573
07/10/27 03:14:26
ついでに>>576のプログラムの問題点と解決方法を。

まず、コンパイラの最適化の問題。
変数nCntの宣言にvolatileを付けていますか? 付けていなければ、付けること。

次に、100 > nCnt の比較が、排他制御の外にあること。
nCntに触っていいのは、排他制御の内側だけ。

582:デフォルトの名無しさん
07/10/27 03:28:55
>>581
たしかにこれでは…色々と参考になりました。

MSDNライブラリで調べたところ、
PostThreadMessage関数でデータをポストして、
GetMessage関数でポストされたデータを取得することで、
スレッド間でデータをやりとりできそうなので、早速試してみます。

ご助言いただいた方々、本当に有難うございましたm(_ _)m

583:デフォルトの名無しさん
07/10/27 16:47:47
>>545です。
>>554
のA()で、

A::A()
{
　 int j=0;
　 for(int i=0; i<10; ++i){
　　 v.push_back(VEC(j));
　　 j+=10;
　 }
}

としているのですが、ここで作られるVEC型オブジェクトは、
局所的なもので、寿命はfor 1回分の間だけですよね？
コンストラクタが呼ばれた直後にデストラクタが呼ばれるので、

vector<VEC*> v;
と宣言を直して、
A()では、
v.push_back(new VEC(j));
としなければダメですか？

何度も申し訳ないです。

584:デフォルトの名無しさん
07/10/27 16:55:04
VEC(j)のコピーがvector内に追加される

585:デフォルトの名無しさん
07/10/27 16:59:02
> コンストラクタが呼ばれた直後にデストラクタが呼ばれるので、
ただし、Aのvector<VEC> v;は、v.push_backの内部で、
引数を使ったコピーコンストラクタを使い、勝手に保持する。

vector<string> v;でv.push_back("ababa");と渡しても、
const char *pを受け取って内部で string(p) のようにして保持する。

> vector<VEC*> v;
> v.push_back(new VEC(j));

のようなことをするのなら、Aのデストラクタで、
push_backの所でnewしたぶんをdeleteしてやる必要がでてくる。

586:デフォルトの名無しさん
07/10/27 16:59:51
かぶったorz

587:デフォルトの名無しさん
07/10/27 18:34:43
>>583
基本ができてなさすぎ。
ちゃんと勉強して出直せ。

588:デフォルトの名無しさん
07/10/27 20:26:59
>>583
いろいろごっちゃになっているようだが、
・関数内で宣言されたオブジェクトは関数が終わったら破棄される
と言われて、
関数が終わったらvectorの中身が壊れるのでは、と疑問を持ったみたいだけど、
この場合は関数へ渡す引数として宣言した「一時オブジェクト」なので問題ない。
そうでなくても、vectorは要素を「コピー」して保持するので、
元のオブジェクトが破棄されても何ら問題はない。

589:588
07/10/27 20:31:56
間違えた、俺がごっちゃになってるorz
書き直すと、

関数内で作ったオブジェクトをvectorに入れたら、
関数を抜けた時にそのオブジェクトが破棄されてvectorの要素が無効になるのでは、と疑問を持ったみたいだけど、
この場合はvectorへ渡す要素として宣言した「一時オブジェクト」なので問題ない。
そうでなくても、vectorは要素を「コピー」して保持するので、
元のオブジェクトが破棄されても何ら問題はない。

590:588
07/10/27 20:34:03
ああ、いや、一時オブジェクトは関係ないか、結局破棄されるんだから

とにかくvectorはコピーコンストラクタ呼び出してコピーしてるので、
元のオブジェクト破棄しても問題ない

591:588
07/10/27 20:42:08

592:デフォルトの名無しさん
07/10/27 20:47:52
ややこしく考え杉。

int g_c ;

void funcA(int& a) {
int b = a ;
g_c = a ;
}

void testA(void) {
int x = 3 ;
funcA(x) ;
}

これと同じよ

593:デフォルトの名無しさん
07/10/27 21:33:41
ﾌﾌﾌﾌﾌﾌﾊﾊﾊﾊﾊﾊ

594:デフォルトの名無しさん
07/10/27 23:07:18
int ary[2] = { 1000 , 2000 };
int *po;

po = &ary[0];
printf("po\t = %x\n" , po);
printf("*po++\t = %d\n" , *po);
po++;
printf("po++\t = %x\n" , po);
printf("*po++\t = %d\n" , *po);
*po++;
printf("po++\t = %x\n" , po);
printf("*po++\t = %d\n" , *po);

これで最後の方で*poに1を加えてるつもりなのですが、変な値になって
*po+=1だと大丈夫です

*poはポインタの中身だと今まで思ってたのですが、、、、なんでなのでしょうか

595:デフォルトの名無しさん
07/10/27 23:09:12
(*po)++;

釣りなんだろうけど

596:デフォルトの名無しさん
07/10/27 23:21:37
理由のところ、教えてくれませんか

なんで（）つけないとダメなのか。
というか中身にアクセスするときは（）つけた方がいいんでしょうか

597:デフォルトの名無しさん
07/10/27 23:28:44
>>596
演算子の優先度

598:デフォルトの名無しさん
07/10/28 04:23:38
*po++ は *(po++) という意味だから、だな

599:デフォルトの名無しさん
07/10/28 04:48:37
俺は演算子の優先度の表を暗記するだけの頭脳がないので、
決して、
*po++ ;
なんてコードは書かない。
他人が書いたのを読まないといけないときは、優先度の表を見て確認する。

そもそも、後ろに++を付けるのは特別なときだけだし・・・。

600:デフォルトの名無しさん
07/10/28 13:39:19
*++p

601:デフォルトの名無しさん
07/10/28 13:40:46
++(*po);

602:デフォルトの名無しさん
07/10/28 20:56:27
>>601
括弧イラネ

603:デフォルトの名無しさん
07/10/28 21:02:43
優先順位覚えてないから付けといて

604:デフォルトの名無しさん
07/10/28 21:33:35
左側に単項演算子二つで優先順位も糞もないだろ

605:デフォルトの名無しさん
07/10/28 21:52:24
むしろ全ての演算子を単項演算子にすれば頭で思い浮かべたものをキャレットを一々前後させずに一気に書ける

606:デフォルトの名無しさん
07/10/28 21:55:25
>>605
それって何て逆ポーランド記法？

607:デフォルトの名無しさん
07/10/28 23:15:58
★改行が多すぎと言われたので、何度かにわけます。

構造体Sのvectorがあって、SのメンバdoubleDataに関してfindをしようとしています。
（#include <algorithm>しています）

struct S{
string strA;
int intData;
double doubleData;
};

findを使っているところは、こんな感じです。
vector<S>::iterator it;
it = find(VecTest.begin(), VecTest.end(), 1.7);
cout << (*it).doubleData << endl;

608:デフォルトの名無しさん
07/10/28 23:16:34
★続き

Boland55で、以下のようなエラーが出ます。

エラー E2094 c:\Borland\Bcc55\include\algorith.cc 72: != 演算子が使われたがクラ
ス S では double 型のための定義が存在しない(関数 find<S *,double>(S *,S *,const
double &) )
警告 W8057 c:\Borland\Bcc55\include\algorith.cc 75: パラメータ 'last' は一度も使
用されない(関数 find<S *,double>(S *,S *,const double &) )
警告 W8057 c:\Borland\Bcc55\include\algorith.cc 75: パラメータ 'value' は一度も
使用されない(関数 find<S *,double>(S *,S *,const double &) )
*** 1 errors in Compile ***

609:デフォルトの名無しさん
07/10/28 23:17:24
★続き

ちなみに、以下のような演算子オーバーロードをしているのですが、
何がよくないのでしょうか。
bool operator == ( const S& left, const S& right )
{
return left.doubleData == right.doubleData;
}

bool operator != ( const S& left, const S& right )
{
return left.doubleData != right.doubleData;
}

610:デフォルトの名無しさん
07/10/28 23:32:46
find には 1.7 じゃなくて S を渡さんといかんのでは？

611:デフォルトの名無しさん
07/10/28 23:34:03
struct has_double
{
bool operator()(const S& s, double d) const
{
return s.doubleData == d;
}
};

find_if(VecTest.begin(), VecTest.end(), has_double(1.7));

612:デフォルトの名無しさん
07/10/28 23:36:44
まちがった

struct has_double
{
double d;

has_double(double d)
:d(d)
{}

bool operator()(const S& s) const
{
return s.doubleData == d;
}
};

613:デフォルトの名無しさん
07/10/28 23:44:17
>> 612さん

コンパイル、通りました！すごい！！
でも全く解読できましぇん。
一語一句の意味をこれから勉強します。。。

614:デフォルトの名無しさん
07/10/28 23:55:56
>>612
テンプレート使って、

template<double value>
struct has_double
{
bool operator()(const S& s) const
{
return s.doubleData == value;
}
};

find_if(VecTest.begin(), VecTest.end(), has_double<1.7>);

ってのではダメ?

615:デフォルトの名無しさん
07/10/29 00:00:45
はは
定数だからね

616:デフォルトの名無しさん
07/10/29 00:13:48
もっと簡潔・明瞭に書く方法ないのかなぁ。

617:デフォルトの名無しさん
07/10/29 09:53:08
C++勉強しはじめてリファレンスを知って、
ポインタ使うよりずっといいじゃないですかって気分なんですが、
関数の引数としてリファレンス使うときに、

void hoge(int* out1, int* out2); //out2の出力が必要ない場合はout2としてNULLを渡す

のような仕様はリファレンスでは実現できないってことなんでしょうか？
毎回ダミー変数宣言して渡すのも馬鹿らしいし。

618:デフォルトの名無しさん
07/10/29 12:44:05
>>617
どうしてもやりたかったらこういう手はどうだろう。
--
int dummyOut;

void hoge(int & out1, int & out2 = dummyOut)
{
if (& out2 != & dummyOut) {
out2 = someOutput;
}
}
--
ダミーは必要だけど、毎回じゃないし使わないときは出力処理を回避することもできる。

次ページ