スレを勃てるまでもな ..
[2ch|▼Menu]
321:デフォルトの名無しさん
07/10/11 23:39:22
入力パラメータが5つくらいあって、そのパラメータの条件でデータを
検索する関数を作らないといけないのですが、5つのパラメータ毎に
使用フラグがあって、フラグOFFの時は、検索条件になりません。
なので、検索条件としては、最低0個、最大5個になり、検索条件の
組み合わせもいろいろになってしまいます。
作り方としては、どういうやり方が簡単そうでしょうか?
パラメータの組み合わせがいろいろになるので、オーバーロードも
できないかなあ、と悩んでいます。

322:デフォルトの名無しさん
07/10/12 00:40:32
>>320
質問の答えにはならないが・・・。

そのWebページで紹介しているノウハウは、
下手な小細工
であるが故に、かえって遅くなったのだと思うよ。

無駄な一時オブジェクトの生成は、
コンパイラの最適化によって端折られるのであれば、
素直にコーディングするのがいいと思うよ。

323:322
07/10/12 00:41:14
文章が変だった。

無駄な一時オブジェクトの生成が、コンパイラの最適化によって端折られるのであれば、
素直にコーディングするのがいいと思うよ。

324:デフォルトの名無しさん
07/10/12 00:44:45
>>321
その「使用フラグ」を見ながら検索処理するのでは、ダメ?

325:デフォルトの名無しさん
07/10/12 01:10:18
>>324
ありがとうございます。
お風呂に入って考えてたけど、結局それが一番見通しがいいような
気がしてきた・・・


326:シロートです
07/10/12 01:41:53
C++言語を使っています。質問です。
4種類の連続した数値のデータがあります。4種類の測定時間や時間間隔は一緒です。
その4種類をひとまとまりとします。そのまとまりがいくつかあります。
それぞれのまとまりから一部分だけを取り出します。その一部分のデータの特徴を
どんどんと集めていき4種類のデータの特性を求めたいです。その方法がわかりません。
すいません。ホントシロートです。質問の意味がわからないかもしれません。
でも本当に困ってます。ヒントだけでもいいのでお願いします。

プログラムが違うのかもしれませんが、
「似たようなデータをどんどん記憶していくことによりそのデータ達の
特性を求める」ということかなと自分では考えたのですがその方法も分かりません。
方法を知っているとかこんなコマンドがあるなど本当に何でもいいのでよろしくお願いします。

327:デフォルトの名無しさん
07/10/12 01:55:59
エスパー以外お断りでなければもっと具体的に頼む

328:デフォルトの名無しさん
07/10/12 04:18:09
「特性」ってのをもっと具体的に説明しる

329:デフォルトの名無しさん
07/10/12 05:01:45
>>326
アルゴリズムの問題だね。
C/C++言語に特有の話ではないので、スレ違い。
さようなら。

330:デフォルトの名無しさん
07/10/12 05:07:00
      / ̄ ̄ ̄ ̄ ̄ ̄ ̄ ̄
     | 吉本いるか  
     \            
        ̄ ̄ ̄|/ ̄ ̄ ̄ ̄
                   ∧_∧      / ̄ ̄ ̄ ̄ ̄
         ∧_∧     ( ´Д`)    < 吉本いるか 
         ( ´Д` )   /⌒    ⌒ヽ    \_____
        /,  /   /_/|     へ \
       (ぃ9  |  (ぃ9 ./    /   \ \.∧_∧  / ̄ ̄ ̄ ̄ ̄
        /    /、    /    ./     ヽ ( ´Д` )< 吉本いるか 
       /   ∧_二つ (    /      ∪ ,  /   \_____
       /   /      \ .\\     (ぃ9  |     
      /    \       \ .\\    /    /  ,、  
     /  /~\ \        >  ) )  ./   ∧_二∃  
     /  /   >  )      / //   ./     ̄ ̄ ヽ    
   / ノ    / /      / / /  ._/  /~ ̄ ̄/ /   
  / ./     ( ヽ、     ( ヽ ヽ | /       ( ヽ、     
(   _)      \__つ    \__つ).し          \__つ   


331:デフォルトの名無しさん
07/10/12 05:33:44
いるよ

332:シロートです
07/10/12 08:49:30
みなさん、説明が下手ですいません。
4種類のデータはヘリコプタの制御に使う、スロットル、エルロン、エレベータ、ラダーです。
特性はホバリングをしている時の4種類のデータ入力の特徴を見つけたいです。
よろしくお願いします。

333:デフォルトの名無しさん
07/10/12 09:01:18
私らはヘリコプタの専門家ではないから、それらのデータの特徴と言われても
何をどうすればいいか判らんのだがね。
もっと具体的に説明はできないもんかね。

334:デフォルトの名無しさん
07/10/12 09:49:43
>>332
コンピュータは勝手に思考して適当によろしくやってはくれない。
処理の手順を1つずつ手取り足取りコーディングしてやる必要がある。

まずは、人間が手作業で計算して処理する手順を明らかにすること。
それくらいは自分でやってから、相談してね。

335:デフォルトの名無しさん
07/10/12 11:40:08
C言語で2次元変数a[4][5]として適当な文字を入れる場合、20文字入れないといけないんですよね?

336:デフォルトの名無しさん
07/10/12 11:53:19
何をしたいのかよくわからん。
こういうことか?
char a[4][5] = { "This", "is", "a", "pen" };

337:デフォルトの名無しさん
07/10/12 12:02:44
vector配列について質問です。
現在、配列に構造体を設定しているのですが、
その構造体の内容を書き換えるとき、以下のようなコードを書いています。

struct DATA data;

data=vec[i];
data.n=5;
vec.push_back(data);

できれば、このようなコードではなく、
vector[i]のところで構造体のアドレスを得るようにして、
メンバを変更したいと思っているのですが、
そのようなことは可能でしょうか。








338:デフォルトの名無しさん
07/10/12 12:08:00
>>335に追加で
#include<stdio.h>
void main(){
int i,j,a[4][5];
a[0][0]='た';a[0][1]='さ';a[0][2]='か';a[0][3]='あ';
a[1][0]='ち';a[1][1]='し';a[1][2]='き';a[1][3]='い';
a[2][0]='つ';a[2][1]='す';a[2][2]='く';a[2][3]='う';
a[3][0]='て';a[3][1]='せ';a[3][2]='け';a[3][3]='え';
a[4][0]='と';a[4][1]='そ';a[4][2]='こ';a[4][3]='お';
for(i=0;i<4;i++){
for(j=0;j<5;j++){
printf("%d",a[i][j]);
}
printf("\n");
}
}
と入力しても正しく入力できないのですがどうすればいいのでしょうか?

たさかあ
ちしきい
つすくう
てせけえ
とそこお

と表示させたいです。

339:デフォルトの名無しさん
07/10/12 12:09:08
>>336
そういうことです

340:デフォルトの名無しさん
07/10/12 12:12:17
>>338
ぱっと見、
printf("%d",a[i][j]);
これはおかしい。
%cだよ。

341:デフォルトの名無しさん
07/10/12 12:12:26
>>338
1) printfの書式について調べよう
2) charについて調べてみよう
3) 全角を表示するにはどうしたらいいか調べよう

342:デフォルトの名無しさん
07/10/12 12:14:33
>>340
無駄な指摘乙。

>>337
vec[i].n = 5ではいけないの?
安全にやるなら
vec.at(i).n = 5になるけど。

343:デフォルトの名無しさん
07/10/12 12:15:35
>>337
vec[i].n = 5;

344:デフォルトの名無しさん
07/10/12 12:16:03
そのコードは、
コンテナ内の既存の要素を変更しているのではなく、
コンテナ内の既存の要素をコピーし、少し変更し、新規の要素として末尾に追加しているぞ。

既存の要素を変更するなら、
vec[i].n=5 ;
でいけると思う。

あるいは、明示的に参照を使って、
DATA& data ;
data = vec[i] ;
data.n = 5;
これでもいいね。

345:デフォルトの名無しさん
07/10/12 12:16:53
>>340
>>341
ありがとうございます。
調べてみます

346:340
07/10/12 12:17:32
>>342
あー、%Cの間違い。

処理系によってはサポートしていないかもしれんが。

347:デフォルトの名無しさん
07/10/12 12:21:52
いや、
a[0][0]='た';a[0][1]='さ';a[0][2]='か';a[0][3]='あ';
とかも、
a[0][0]=L'た';a[0][1]=L'さ';a[0][2]=L'か';a[0][3]=L'あ';
にする必要あるな。

348:デフォルトの名無しさん
07/10/12 12:22:47
そもそも、int はマズいな。wchar_t だ。

とにかくツケ刃じゃダメだ。

349:デフォルトの名無しさん
07/10/12 12:39:49
>>347
>>348
ありがとうございます
試してみます

350:337
07/10/12 13:24:28
ご指摘ありがとうございます。
vec[i].nできれいにコードを書くことができましたが、
やはり構造体のアドレスを取得する方法が気になっています。
というのも、最終的に行いたいことは、
基底クラスを継承した複数の派生クラスをvetor配列に格納し、
その派生クラスのアドレスを基底クラスで受け取って、
仮想関数を呼び出すという処理をしたいからです。

CData *p;

//ここでvector配列と添字でCDataの派生クラスのアドレスをpで受け取る

p->Draw(); // 受け取った派生クラスの種類を意識せず関数を呼び出す

コードとしては、このようなものをイメージしているのですが、
実装は可能なのでしょうか。
申し訳ありませんが、よろしくお願いします。

351:デフォルトの名無しさん
07/10/12 13:25:49
p = &vec[i];

352:351
07/10/12 13:29:06
いや、まて。 なにかおかしい。

例えば
class Base{};
class Derived : public Base{};
だとして、
vector<Base> に Derived を入れることは出来ないぞ。
vector<Base*> とすることになるんじゃないか?
だったら
Base *p;
p = vec[i];
だと思うが。

353:デフォルトの名無しさん
07/10/12 13:33:12
的確すぎてワロタw

354:337
07/10/12 13:52:05
ご指摘ありがとうございます。
つまり、クラスのアドレスをvector配列に格納し、
実体そのものは別に管理するということでしょうか。


355:デフォルトの名無しさん
07/10/12 14:15:49
>>337
>実体そのものは別に管理するということでしょうか。
yes.
vector内でのコピーがポインタのコピーで済むから動作が速くなる(可能性もある)し。
余裕があれば(管理者が許せば)boost::shared_ptrとかを検討してみるのもあり。

356:337
07/10/12 14:39:49
vector<Base*>で上手くいきました。
ありがとうございました。

357:デフォルトの名無しさん
07/10/12 14:59:31
動画像からフレームを取り出してフーリエ変換したいのですが
取り出すプログラムが分かりません。
分かる方、教えて頂けると幸いです。

358:デフォルトの名無しさん
07/10/12 15:01:50
>357
動画像処理したいというなら、ライブラリが限られるけどDirectShowを使いたいのかな?
それならDirectShowフィルタを使うのが現実的で、次がサンプルグラバで1枚づつ画像を取り出すとかになると思う。
むつかしめだけどがんば。


359:357
07/10/12 15:23:15
>>358
レスありがとうございます。
秒間20フレーム程度で数秒の動画なのですが
簡単なプログラムで実装出来ないでしょうか?

先生に難しいプログラムでは無いと言われました。
その2つを見てみたのですがこの分野が苦手な自分には
少し難しいです;;

360:デフォルトの名無しさん
07/10/12 16:13:43
>>359
何のために学費払ってんだよ。
先生に聞け、先生に。

361:358
07/10/12 16:16:00
fft(高速フーリエ変換)自体は画像処理系の本ではかなりあたりまえにあるコードなので難しいものではないと思うんだけど
動画から画像を取り出すのはwindows標準でライブラリとか存在したかな...。
fftが難しくないのか、動画からフレームごとの画像を取り出すのも含めて難しくないのかちょっと聞いてみて欲しい。
あと、だれかできるだけ楽な、その方法知らないですか?>all

362:デフォルトの名無しさん
07/10/12 16:21:51
20フレームで数秒ってことは、50フレーム前後か。
そこいらの、動画を入力して、ビットマップ画像の羅列を出力するプログラムを使えばいいじゃん。

363:デフォルトの名無しさん
07/10/12 20:47:09
いろいろなデータファイルからデータ抽出して
アルゴリズムに適用して出力結果を出すプログラム作ってんだけど
微妙に値がおかしいんだよね。大方あってるんだけど。

これはデバックでなおるのかな?
150X30くらいのデータを配列にいれて1行ずつ処理してるんだけど
領域破壊とかなってるんでしょうかね。


364:デフォルトの名無しさん
07/10/12 20:57:03
>>363
さぁな、どんなバグがあるかは、デバッグしてみないとな。

365:デフォルトの名無しさん
07/10/12 20:58:47
>>363
テストしたの?

本屋に行って、
「基礎から学ぶソフトウェアテスト」
とかの本を買って読んでみ。

366:デフォルトの名無しさん
07/10/12 21:13:30
おまえらこんな抽象度の高い質問によく答えられるな

367:デフォルトの名無しさん
07/10/12 21:21:50
回答も抽象的だし

368:デフォルトの名無しさん
07/10/12 21:23:55
だってプログラムの概要をいってなんとかしてくれるのかい?
データファイル大量すぎて
説明しようがない

369:デフォルトの名無しさん
07/10/12 21:29:22
4725円のやつですか?
学校で買ったソフトウェア工学じゃ駄目?

370:デフォルトの名無しさん
07/10/12 22:00:39
学生さんか。
じゃぁ、>>365の本は向いてない。

ちゃんとしたレクチャーなんて、ここではできないから、インチキな方法を教えてお茶を濁そう。

メモリの不正アクセスは、検出するためのツールがある。高価なものから無償のものまで。
開発環境は何?

次に、プログラムが意図通り動いていないのであれば、まずはステップ実行して確認してみよう。
案外、あっさりと間違いを見つけられる。

それでもダメなら、すでに確認済のデータを入力し、正しいデータが出力されるか、確認してみよう。
おかしなデータが出力されるようなら、プログラムを部分ごとに個別に、テストデータを与えて、出力が正しいことを確認しよう。


371:デフォルトの名無しさん
07/10/12 23:16:09
cppunitとかunittest++とかboost.testとかライブラリから学ぶのがいいと思うんだぜ
中でもboost.testはbjamと組みあわせて複数のプロジェクトを一括テストとかできるし
make書くのが面倒臭い人にはおすすめ

372:デフォルトの名無しさん
07/10/12 23:28:44
んなものは、反復してテストするレベルに到達してからでいい。

すでにバグがあるらしいということが、わかっているのだから、まずはステップ実行だ。


373:デフォルトの名無しさん
07/10/13 01:49:15
データ処理において配列をたくさん使ってプログラムを作るのは
やめておいたほうがいいですか?
グリッドでの計算で二次元配列を2つ使って行っているんです。
30X150行を2つくらい。
領域を使いすぎてはいけないと聞いたんですけど。


374:デフォルトの名無しさん
07/10/13 07:53:56
>>373
たったそれだけ?
今時のPCなら問題ないでしょ。

375:デフォルトの名無しさん
07/10/13 09:19:25
まーその二次元配列のファイルを2500個処理するんだけどね

376:デフォルトの名無しさん
07/10/13 11:02:24
>>373,375
要は、そのときのメモリ使用量が使用環境において妥当かどうかだ。
30x150x2x2500と考えて、それぞれ4バイトデータとしたら、高々100MiB未満。
全部メモリ上に置いたって何にも問題ないだろ。
# 勿論、Linux@PS3やWinCE、組み込みってことなら話は別だが。

377:デフォルトの名無しさん
07/10/13 11:13:23
long l1 = 1.2;
long l2 = 3.4;
printf("%f\n", l1 + l2);
これで4.6または4.600のような値を表示したいのですが、どう直したらよいのでしょうか?

378:デフォルトの名無しさん
07/10/13 11:14:58
l1とl2の型をdoubleにする。

379:デフォルトの名無しさん
07/10/13 11:16:58
long l1 = 1.2;
long l2 = 3.4;
はもしかしたら優秀なコンパイラなら
long l1 = 1;
long l2 = 3;
として解釈するということでしょうか?

380:デフォルトの名無しさん
07/10/13 11:19:57
>>379
いいえ。優秀なコンパイラに限らず、気が違っていない限りそう解釈するはずです。

381:デフォルトの名無しさん
07/10/13 11:20:43
何が優秀なコンパイラなのかは知らないが、
規格上、浮動小数点数を整数に変換する際には、
小数点以下切捨てが行われることになっている。

382:デフォルトの名無しさん
07/10/13 11:21:14
URLリンク(kzk9.net)
の一番上のソースを見たんですが、clock_tはlongですよね。
なのに%10.30fとして小数以下を30桁出そうとしたこの人も
勘違いしているのでしょうか?

383:デフォルトの名無しさん
07/10/13 11:40:57
>>376
返答ありがとう。
よくよく考えてみると配列を150X30を30X150にしたり
無駄なことをしてるから
そこらへんを直してみるわ。

384:デフォルトの名無しさん
07/10/13 12:03:23
>>382
clock_tが実際に何型かは実装に依存する。
しかしprintfには、型が判明していなければデータを渡せない。
とりあえずdoubleにキャストすれば、大抵の環境では値が保持される。
俺ならlong longにキャストしたいところだが、long longはまだ標準にないからな…。

385:デフォルトの名無しさん
07/10/13 12:06:10
切捨てに関して便乗で

-4 -3 -2 -1 0 1 2 3 4 ・・・

-2 -2 -1 -1 0 0 1 1 2 ・・・
にしようと思ったのですが

int X=-1;
printf("%d\n",X/2);

が0になります
とりあえずXがマイナスのときを条件分けしましたが
EXCELのINT関数だとすんなりいくので不思議な感じがします
そういうものなんでしょうか?

386:デフォルトの名無しさん
07/10/13 12:07:12
>>382
いいえ、勘違いではなくclock_tがdoubleなどの実数型であるとでも思っているのでしょう。

387:デフォルトの名無しさん
07/10/13 12:08:20
>>382
もとが整数なのに小数で出力しようとするのはナンセンスだが、
doubleにキャストして変換しているので、間違いではないよ。

388:デフォルトの名無しさん
07/10/13 12:14:35
>>385
負数の丸めは実装依存ですが、一般的には0に近い方に丸められるようです。

389:デフォルトの名無しさん
07/10/13 12:22:17
>>388
浮動小数点数型から整数型への変換は、常に小数点部分を捨てた値で、
つまり0方向への丸めに決まっていたはずだが。

>>385
(int)floor(X / 2)

390:デフォルトの名無しさん
07/10/13 12:29:31
すまん、(int)floor(X / 2.0)とでもしないとだめだな。

浮動小数点数型から整数型への変換のことは、X3010:2003 6.3.1.4に書いてある。
手元のANSI C言語辞典の型変換の項目にもそう書いてあるから、
C89でも0方向への丸めで間違いないはず。

391:デフォルトの名無しさん
07/10/13 12:32:21
>もとが整数なのに小数で出力しようとするのはナンセンス
よしよし

392:デフォルトの名無しさん
07/10/13 13:09:33
>>389,390
すみません、どこに浮動小数点型が出てくるんですか? 元質はこれなんですが。
--
int X=-1;
printf("%d\n",X/2);


393:390=391
07/10/13 13:52:16
すまんorz

388の言うとおりだ。ただしC99では0方向への丸められることになった。
また、div関数などを使えば、C89でも必ず0方向への丸めになる。
丸め方を指定したければ、391のように浮動小数点数を使うという手もある。

394:385
07/10/13 14:44:59
>>388-390,393
整数型/整数型の丸めは0方向に
floorとかEXCELのINTとかは小数点部分を捨てた値にする関数っていう感じですかね

>>392
とりあえず(int)floor(X/2.0)=-1でできますね
たしかにわざわざ実数で割るのもなんかすっきりしませんが、そういうものだと思うことにします

ありがとうございました

395:デフォルトの名無しさん
07/10/13 14:52:37
floorとExcelのINTは負の無限大方向への丸め。
小数部分を捨てるというと、
Cのやる0.5→0、-1.5→1のような字面的に整数部分だけの値にすることを連想する。

396:デフォルトの名無しさん
07/10/13 16:22:59
(int)floor(X / 2.0)

これってアリなの?

Xが整数型だから、
X / 2
になっちゃわないか?

(int)floor((double)X / 2.0)
などとして、Xを実数にしないと。

397:デフォルトの名無しさん
07/10/13 16:24:37
>>396
2.0

398:デフォルトの名無しさん
07/10/13 16:31:49
doubleで割ってるからdoubleになる
大丈夫

399:デフォルトの名無しさん
07/10/13 16:44:39
左項に合わせるのだと思い込んでたよ。


400:デフォルトの名無しさん
07/10/13 18:10:02
コンパイルしたら警告が600個あります
今のところバグは無いのですが、何か他に弊害はありますか?

401:デフォルトの名無しさん
07/10/13 18:11:19
まず間違いなくバグだらけだろうな

402:デフォルトの名無しさん
07/10/13 18:32:22
>>400
そんなに大量に警告が出ていたら、無視してはいけない警告が出ても、気がつかないだろうな。
それが、大量の警告を放置することによる害。

きちんと精査した上で警告を無視すると決めたら、
当該部分だけに限定して、pragmaとかで警告を抑制すべし。

403:デフォルトの名無しさん
07/10/13 18:53:41
GNUのソースは警告0ですか?

404:デフォルトの名無しさん
07/10/13 19:38:20
>>400
多分ライブラリが嫌らしいコーディングしてるんだろうがな

405:デフォルトの名無しさん
07/10/13 20:03:57
Wallにしたらstlでも凄い量の警告出るぜ

406:デフォルトの名無しさん
07/10/13 20:08:08
>>404
wow
調べてみたらけっこうライブラリというか同じ部分の警告でかぶってました

407:デフォルトの名無しさん
07/10/13 21:59:34
3桁ごとにカンマが入ってる数字文字列を
int型に変換する関数ってないんですかね

408:デフォルトの名無しさん
07/10/13 22:09:50
標準ではありません。大したもんじゃないから自作しましょう。

409:デフォルトの名無しさん
07/10/13 22:16:07
そうですか

lexical_castで通ると思ってたらエラー出るし…

自分で書くことにします
すみません

410:デフォルトの名無しさん
07/10/13 22:22:18
boost使うならやりようは色々あるかと

当該スレで聞いてみては?

411:デフォルトの名無しさん
07/10/15 20:13:04
C++なんですが、AfxsetResuseで
明示的にリソースを設定した場合
リソースの解放も明示的に行う必要が
あるんでしょうか?

412:デフォルトの名無しさん
07/10/15 20:44:31
>>411
C++標準には、そのような識別子は定義されていないが?

413:デフォルトの名無しさん
07/10/16 04:05:24
質問です。

以下のように書いたとき、
arrayの各要素は10で初期化されると思っていたんですが、
コンパイルして実行してみるとarray[0]のみ10で
残りは0で初期化されてました。
配列の各要素をすべて同じ値で初期化するにはどうすれば良いんでしたっけ?
よろしくお願いします。

#include <iostream>

int main(int argc, char** argv)
{
int array[10] = {10};

for (int k = 0; k < 10; ++k) {
std::cout << array[k] << std::endl;
}

return (0);
}

414:デフォルトの名無しさん
07/10/16 04:33:10
int array[10] = {10,10,10,10,10,10,10,10,10,10};

0以外の数字で全て同じ数値で初期化するには、いちいち全部書かなきゃダメ。

めんどくさいなら for(i=0; i<10; i++) array[i] = 10; みたいにループで初期化する。

415:デフォルトの名無しさん
07/10/16 04:50:04
int array[10] = { FILL(10,10) };
みたいにプリプロセッサで書けないかなぁ…

416:デフォルトの名無しさん
07/10/16 05:02:13
C++ならクラスにしてコンストラクタで初期化させればええやん

417:デフォルトの名無しさん
07/10/16 07:03:56
std::vector<int> array(10, 10);

418:デフォルトの名無しさん
07/10/16 08:47:11
>>414
0以外のときは駄目だったんですね。
全然知りませんでした。

ありがとうございました。


419:デフォルトの名無しさん
07/10/16 12:25:23
あれ?自動変数は0に初期化されるんでしたっけ?

420:デフォルトの名無しさん
07/10/16 12:43:56
いいえ、普通の自動変数は初期化されません。
但し、構造体や配列の初期化指定子の数が足りない場合は0が補われます。
従って、all bit 0が数値0を表わさない実数型であったとしても、数値0が保証されます。

421:デフォルトの名無しさん
07/10/16 13:34:07
プログラム的にmain関数内でforループの中で配列を宣言するのはどう思いま
すか?配列の領域がループ回数分かわるのでforループの中にいれたいのですが。


422:デフォルトの名無しさん
07/10/16 13:36:43
>配列の領域がループ回数分かわる
ここが意味不明なので何とも言えない。

423:デフォルトの名無しさん
07/10/16 13:41:17
forループの中で1ループごとにファイルを1つ読み込んで
そのファイルの行数を配列の領域としたいんです。
だからファイルをかえるとファイルの行数もかわり、領域もかわるというわけ
です。わかりにくいですか

424:デフォルトの名無しさん
07/10/16 13:44:09
あーなるほど、言いたいことは判った。

配列って言うか、動的メモリ領域の話ね。
ループ外で全く使わないなら、ループ内で定義していいんじゃない?
つーか、そうすべきだと思うけど。
# 但し、殊にCではアナクロ的に関数の先頭で全てのローカル変数を宣言しないと気が済まない人もいるから注意。

425:デフォルトの名無しさん
07/10/16 13:55:16
ただそのときに
その配列の前にファイル読み込んで行数確認して
配列の領域確保して
またファイル読み込んでそこの配列に代入というプログラムにしてるんです。
無駄にファイル読み込んでるとかいわれそうだけど。

最初に全てのファイルの領域を確保する
配列を定義してもいんですが、その行数をまた違う配列でも使うので、
それは避けたいんです。

426:デフォルトの名無しさん
07/10/16 14:04:46
ファイルの二回読み自体は仕様的に問題ないなら構わないと思う。
しかし、寧ろそれだけ複雑ならループ内を全部別関数にするべきじゃないのか?
main()はあくまでもコマンドラインとのI/Fだと思うんだが。

427:デフォルトの名無しさん
07/10/16 14:06:45
STLなら自動拡張する

428:デフォルトの名無しさん
07/10/16 14:20:06
まあそうなんですけど。
今つくってるのは1つのファイル(これを入力)に対して
forループ回違うファイルを読み込んで処理して
それぞれファイルでの値を出して一番高いものを出す。
その一番高いものが入力と合ってたら正解みたいなプログラムなんですけど。
これをmain関数内でやってるわけなんですが、
入力が複数として、この複数の入力の中で正解がいくつあるかって最終的には
したいんです。

今のmain関数も別関数として
入力をいれる処理をmainとして、元のmain関数において正解のものを
returnさせて正解率を出すべきですよね?


429:デフォルトの名無しさん
07/10/16 14:27:02
抽象的に過ぎてよく判らんw

こんなもんじゃないのかな?
main()
{
問題処理();
}

問題処理()
{
統括ファイル読み込み();
for () {
個別ファイル処理();
if (正解判定()) {
正解処理();
}
}
}

430:デフォルトの名無しさん
07/10/16 14:32:14
んなもんですね。。ありがとうございます。
mainから問題処理のところを手作業でやってもいんですけど
入力データが100くらいあるから手作業だとだるすぎるので。
今求められてるのは正解率ですから。しかし何かの機械の実装って本当にめん
どくさいね。まだ社会人じゃないから実情は知らないけど。


431:デフォルトの名無しさん
07/10/16 14:33:04
オジサンの弱った頭には、ややこしくて理解できん。
話を整理してもらえないか。

432:デフォルトの名無しさん
07/10/16 14:36:26
宿題スレじゃないんだし、エスパーごっこはそろそろ終わりでいいんじゃないか?
>430だって>429までの流れを踏まえて少しは自分で頭を使いたいだろうし。

433:デフォルトの名無しさん
07/10/16 14:44:07
もうforループの中で配列宣言つかっていいてのと
ファイル2回読みはOKかどうかもわかったので失礼します

434:デフォルトの名無しさん
07/10/16 17:55:42
まあ、ファイルのクローズと、領域の解放を忘れなければどうでも良いんじゃね

435:デフォルトの名無しさん
07/10/16 18:48:47
若い人は記憶力まかせにしても破綻しにくいからのぉ。

わしも小学生の頃は、
恐ろしくスパゲッティなコードを書いていたが、
ちゃんと把握できていたので何ら問題なかったな。

いまではもう無理。

436:デフォルトの名無しさん
07/10/16 18:51:02
若いのにRAIIとかエディタの支援機能に頼りまくりで記憶力低下中だけど
元からそんなに記憶力良い方じゃないから問題ないか

437:デフォルトの名無しさん
07/10/16 18:51:59
ここは自分のダメな所を書くスレになりました

438:デフォルトの名無しさん
07/10/16 18:57:10
若くても、頭脳を有効に使うために、スマートなコードを書くべき。

439:デフォルトの名無しさん
07/10/16 20:59:21
Cの配列宣言って要素数に定数じゃなくて変数って使えないんじゃ?
動的な領域確保したい時はCはmallocに頼るしかないはずだと記憶していたのですが?
覚え間違いかな…?

># 但し、殊にCではアナクロ的に関数の先頭で全てのローカル変数を宣言しないと気が済まない人もいるから注意。

これはブロック({}で囲まれた部分のこと、正式な呼び方知らない)の先頭でしか変数の宣言を許さないCの制限のせいでしょう
C++ではこの制限がないからbetterCとしてC++を使っているなら関係ない話ですが

440:デフォルトの名無しさん
07/10/16 21:47:30
>>439
その制限は関係ないよ。

関数の先頭でしか変数宣言すべきではないという石頭の連中は、
制限がないC++においても、同様のルールを主張していたりするから。


441:デフォルトの名無しさん
07/10/16 22:16:45
>>439プ

442:デフォルトの名無しさん
07/10/16 22:23:16
>>439
>Cの配列宣言って要素数に定数じゃなくて変数って使えないんじゃ?
C99なら出来る。C89はムリ

443:デフォルトの名無しさん
07/10/17 11:22:34
C言語の入門書を読んでたら、getsやscanfは使うなとか、fgetsを使えとか
ややこしいですよね。デリミタで区切られたデータ列を読み込む必要がある
んですが、呼び側のコードをシンプルにするため、こんな関数を作ってみま
した。下記のように使うのですが、何かマズいところはないでしょうか?
今のところ一応動いているのですが・・・

 char s[128];
 FILE *p = fopen("hoge.txt", "rb");
 if (ReadData(p, s) == 0) break;

/*********************************************************************
 ☆ファイルから1単位読み込んで文字列として返す
  デリミタで挟まれた文字列を1単位とする。
  int IsDelimiter(int c)はデリミタなら1、そうでなければ0を返す
  空のフィールドは存在しないものとする。
*********************************************************************/


444:デフォルトの名無しさん
07/10/17 11:23:47
int ReadData(FILE *p, char *s)
{
int ex, c, ip;
/* 非デリミタ文字が現れるまで読み飛ばす */
ex = 0;
while (ex == 0){
c = fgetc(p); if (c == EOF) break;
if (IsDelimiter(c) == 0) ex = 1;
}
if (ex == 0) {
s[0] = '\0'; return 0;
}
else {
s[0] = (char) c;
}

/* デリミタが現れるまで,文字列を積み上げる */
ip = 1;
ex = 0;
while (ex == 0){
c = fgetc(p); if (c == EOF) break;
if (IsDelimiter(c) == 1) {
ex = 1;
}
else {
s[ip] = (char) c; ip++;
}
}
s[ip] = '\0';
return 1;
}


445:デフォルトの名無しさん
07/10/17 12:31:33
128文字以上を入力するとマズい

getsを使うなってのは、読み込む最大文字数を指定できないからで、入力ファイルに予想より長い文字列が現れたら致命的
代わりにfgetsを使えってのは、fgetsは最大文字数を指定できるので、どんな入力が来ても死ぬことはない
>>444 の関数はそれと同じ問題を孕んでいる

ユーザに127文字までしか入力してはいけないといくら言い聞かせてみたところで、禁止されるとやりたくなるのが人というものだし、
そうでなくてもついうっかり文字数制限を忘れてしまうこともあるだろうし、プログラムはどんな入力が来ても安全なように作っておくべき

446:デフォルトの名無しさん
07/10/17 12:56:45
>>444
ReadDataにsの要素を渡して文字数のチェックをちゃんとすればおk
てか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位取ってごまかしちゃうつもりでしたが、
言われてみればその通りですね。変数ipをチェックすればいいので、
バッファがあふれたらエラーを返すコードを追加しようと思います。

>>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
あ、すまん、所々ミスってる上にそもそも動作が違うなw

これでどうだ

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の戻り値を勘違いしてた上に、
そもそも動作を全く勘違いしてたw

これならわざわざ書き直す必要なかったですね……
偉そうにでしゃばってすみません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
コンパイルに5分かかるプログラムを作ったら一人前ですか

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
勘弁してやれw 勘違いだろw

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

489:デフォルトの名無しさん
07/10/21 20:31:08
>>475
それだとABC(data);って書いたら駄目だわなw
#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
やべ、俺リロードしなさすぎw
吊ってくるww

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っていう数がでてくる。
これってどういうこと?


次ページ
最新レス表示
スレッドの検索
類似スレ一覧
話題のニュース
おまかせリスト
▼オプションを表示
暇つぶし2ch

4716日前に更新/260 KB
担当:undef