可変長バイナリを読み込みたい

質問

＜プログラム環境＞
Windows XP,VC++6.0,MFC AppWizard(exe),ダイアログベース

＜目的＞
COMポートから可変長のバイナリデータを受信する

＜質問＞
受信するバイナリの長さは受信した内容で分かるようになっています。
例えば、
先頭が"0x0b"であれば11個のバイナリが続いて送られてくるという感じです。
この例ように、送られてくる内容（"0x0b"）を確認して、それに応じて
11個データを読み込む、というプログラムにするには、どのようにしたら良いでしょうか？

今は以下のソースで、バイナリの長さを指定して受信しています。
//////////オーバーラップ構造体の初期化//////////
OVERLAPPED old;
ZeroMemory( &old, sizeof(old) );
old.Offset = 0;
old.OffsetHigh = 0;
old.hEvent = hEvent;

//////////データ受信////////////////////////////
unsigned char rdBuf[10];
unsigned char* prdBuf;
DWORD dwCount;
DWORD dwRead;

prdBuf = &rdBuf[0];
dwCount = 10;

if(ReadFile(hCom,prdBuf,dwCount,&dwRead,&old) == 0){
if(ERROR_IO_PENDING == GetLastError()){
if(WaitForSingleObject(hEvent,INFINITE) == WAIT_OBJECT_0){
if(GetOverlappedResult(hCom,&old,&dwCount,TRUE)){
//"データ受信完了"
}}}
else //エラー
}
宜しければ、ご指摘の程よろしくお願いします。

回答No.1

nda23 · ベストアンサー

考え方としては
(1)１バイト読み込む
(2)読み込んだデータ＝０なら終了
(3)読み込んだデータを入力バイト数に指定してデータを読み込む
となりましょうか。
尚、ReadFileの後、無期限でWaitするくらいならオーバーラップする必要は
ないと思いますが、特別な理由でもあるのでしょうか？

unsigned char rdBuf[256];
unsigned char ucLen;
DWORD dwCount;
DWORD dwRead;

if(ReadFile(hCom,&ucLen,1,&dwRead,NULL)){
　　if( ucLen == 0 ) return;
　　dwCount = ucLen;
　　if(ReadFile(hComrdBuf,dwCount,&dwRead,NULL)){
　　　　//"データ受信完了"
　　　　return;
　　}
}
//エラー

補足

的確なアドバイス有難うございます！
教えて頂いた方法でできました。
そして、無期限のwaitをやめて、100ミリセカンドだけ待つようにしました。

dwReadには読み込んだバイト数が格納されると思うのですが、
１バイト読み込んだ後に、
char Buf[256];
wsprintf(Buf,"dwRead = %d",dwRead);
MessageBox(Buf);
とすると、"dwRead = 0"が表示されるのは何故でしょうか？
確実に１バイトは読み込まれているハズなので、"dwRead = 1"となって欲しいです・・。

ご指摘の程よろしくお願いします。

回答No.8

nda23

>set_time_limit()という関数を使うことなのでしょうか？
お得意のReadFileを使うんです。100mSでタイムアウトになるヤツ
ハンドルが標準入力になります。でも、SleepでOKなら、その方が
ベターだと思います。標準入力はコンソールアプリケーションで
ないと取得できませんから。

お礼

nda23様のおかげで、私がやりたいプログラムにかなり近づきました！とてもご親切に回答下さり有難うございました。

回答No.7

nda23

(1)自動リセットですね。問題ありません。
(2)ここだけの話ではハッキリとした原因は特定できません。
　空のメッセージボックスの代わりにSleep(100)を入れてみますか…
　あるいは標準入力からタイムリミット付きの入力を入れてみるとか…
　後者のような気がしますけど、推測の域を出ません。

補足

有難うございます。
Sleep();
を使えば、できました。
77ミリ秒あたりが閾値で、成功したり失敗したりします。
原因が分からないうちは100ミリ秒あたりを指定しておこうと思います。

「標準入力からタイムリミット付きの入力」とは
set_time_limit()という関数を使うことなのでしょうか？

よろしくお願いします。

回答No.6

nda23

(1)イベントハンドラを初期化していないのでは？
　つまり、最初からシグナル状態だったり、自動リセットでないハンドルをResetしないで使い回すとか…
　CreateEventの指定値を公開してください。
(2)error C2664: 'wsprintfA' : 1 番目の引数を 'unsigned char *' から 'char *' に～
　ヘッダファイルの中にwsprintfの定義があり、第1引数は(char*)と指定されているためです。
　charとunsigned charは最上位ビットを符号と見るかどうかの差だけで、
　バイト単位のデータであることに変わりはありません。こういう場合はキャストします。
　wsprintf( (char *)prdBuf,～

補足

nda23様有難うございます。
（１）
// イベントオブジェクトを作成する
hEvent = CreateEvent( NULL, FALSE, FALSE, NULL );
if( hEvent == NULL ) MessageBox("イベント作成エラー",NULL,MB_ICONSTOP);
です。
自動リセットなので、シグナル状態のままになる事はありませんよね？

（２）
ご指摘の通りに修正してエラーは無くなりました。

回答ANo.4の補足にある、
wsprintf(Buf,"dwRead = %d",dwRead);
MessageBox(Buf);
の部分で"dwRead = 0"と出力される事に関して一つ気づきました。
ReadFile()の前に、MessageBox("")すると"dwRead = 1"のように正しい値が出力され、ReadFile()までに一回もMessageBoxを使わないと"dwRead = 0"と出力されます。
この現象の原因は何かご存じでしょうか？

よろしくお願いします。

回答No.5

nda23

WaitForSingleObjectの戻り値はDWORDなので、BOOL bLError で受けるのは…
戻り値はWAIT_TIMEOUTでなければ成功と見てよいでしょう。
http://msdn.microsoft.com/ja-jp/library/cc429427.aspx

wsprintf(Buf,"dwRead = %d",dwRead);
MessageBox(Buf);
この結果は"dwRead = 1"ですか？
尚、Bufはchar Buf[16]とか定義してありますか？
※スタックは4バイト境界に取られるので、[13]としても節約になりません。

受信長にかかわらず10バイトを16進で表示していますが、自動変数は初期化されて
いないので、何らかの表示はされますが、受信したものかどうかは怪しい。

prdBuf = Buf; // &Buf[0] としなくても良い。
prdBuf += wsprintf(prdBuf,"%s","文字 =");
for ( int i = 0 ; i < dwRead ; i++ ) {
　　prdBuf += wsprintf(prdBuf," %02X",rdBuf[i]);
}
MessageBox(Buf);
上記のようにしてください。
ただし、Bufは十分な大きさを確保しておいてください。

マルチスレッドを制御できると、処理時間を半分くらいに短縮できるようになります。
頑張って習得してくださいね。

補足

nda23様、有難うございます！！
＞WaitForSingleObjectの戻り値はDWORDなので、BOOL bLError で受けるのは…
宣言をDWORD型に変更しました。

＞戻り値はWAIT_TIMEOUTでなければ成功と見てよいでしょう。
else if(bLError == WAIT_TIMEOUT)のメッセージが出なかったので成功ですね＾＾

＞wsprintf(Buf,"dwRead = %d",dwRead);
＞MessageBox(Buf);
＞この結果は"dwRead = 1"ですか？
この結果が"dwRead = 0"なんです・・。本当に何故か分かりません。
しかし、最後に
wsprintf(Buf,"dwRead = %d",dwRead);
MessageBox(Buf);
を書いて実行すると"dwRead = 6"となりました。こちらは正しい値が出るので、ますます分かりません・・。

＞尚、Bufはchar Buf[16]とか定義してありますか？
失礼しました。Bufの定義はchar Buf[256];としています。

＞受信長にかかわらず10バイトを16進で表示していますが、自動変数は初期化されて
＞いないので、何らかの表示はされますが、受信したものかどうかは怪しい。
はい、初期化していないので、「02 80 9a 00 b1 56 cc cc cc cc」と表示されています。因みに先頭は「06」ですので、６個受信してます。
初期化の仕方は宣言の際にchar Buf[256]="";でいいのでしょうか？こうしても、「02 80 9a 00 b1 56 cc cc cc cc」が表示されました。

＞prdBuf = Buf; // &Buf[0] としなくても良い。
かなりスッキリしました。

＞prdBuf += wsprintf(prdBuf,"%s","文字 =");
＞for ( int i = 0 ; i < dwRead ; i++ ) {
＞　　prdBuf += wsprintf(prdBuf," %02X",rdBuf[i]);
＞}
＞MessageBox(Buf);
＞上記のようにしてください。
「error C2664: 'wsprintfA' : 1 番目の引数を 'unsigned char *' から 'char *' に変換できません。」というエラーが出るのですが、なぜでしょうか？

＞ただし、Bufは十分な大きさを確保しておいてください。
Bufのサイズは256で十分なのでしょうか・・？

何度も質問を繰り返して申し訳ありませんが、よろしくお願いします。

回答No.4

nda23

>確実に１バイトは読み込まれているハズなので、
本当ですか？
オーバラップしたんでしょ。読み込み完了前に制御は戻ってきますよね。
読み込みが完了する前なら、dwReadの内容は0もありえる値です。
100ミリ待てば入力が完了する保証はあるのでしょうか？
失礼ですが、マルチスレッドを扱える技術がおありでしょうか？
シングルスレッドで、オーバラップした以上、dwRead=0は入力タイムアウトで、
エラー処理になるべきです。

>unsigned char rdBuf[dwCount];
Basicではないので、こんな方法では領域を確保できません。
例え、1バイトでもメモリを取得する場合はGlobalAlloc等の関数を使います。
http://msdn.microsoft.com/ja-jp/library/cc430065.aspx

でも、符号無し8ビット整数の最大値(256)くらいなら、こういう関数を使う程ではありません。
サンプルでもそうなっていますね。但し、読み込んだデータを蓄積していく場合では
動的なメモリの確保が必要になってきます。

補足

ご回答有難うございます！！
ご指摘の通りマルチスレッドの知識は全くありません・・。
これからマルチスレッドを学習して、データ送信と受信を同時に行うプログラムを作る予定です。

以下に今のソースを示します。
コメントアウト部分はMessageBox等で表示する処理をしています。
実行すると、タイムアウトなどのエラーに関する表示は無く、先頭のバイナリの情報で得た長さ分のデータを受信し、その内容が正しく表示されました。
なので、受信は出来ていると思っているのですが、やはり受信できていないでしょうか？

//////////先頭の１バイトだけ受信する/////////////////////////////////////////
BOOL bLError;
unsigned char ucLen;
DWORD dwCount = 1;
DWORD dwRead;

if(ReadFile(hCom,&ucLen,dwCount,&dwRead,&old) == 0){
if(ERROR_IO_PENDING == GetLastError()){
bLError = WaitForSingleObject(hEvent,100);
if(bLError == WAIT_OBJECT_0){
if(GetOverlappedResult(hCom,&old,&dwCount,TRUE));//データ受信完了
}
else if(bLError == WAIT_FAILED);//受信スリープエラー
else if(bLError == WAIT_TIMEOUT);//受信タイムアウト
}
else ;//受信エラー
}
//////////メッセージボックスでdwReadの値を表示//////////////////////////////
wsprintf(Buf,"dwRead = %d",dwRead);
MessageBox(Buf);

//////////先頭データの情報から可変長で読み込む//////////////////////////////
unsigned char rdBuf[256];
unsigned char* prdBuf;

prdBuf = &rdBuf[0];
dwCount = ucLen;

if(ReadFile(hCom,prdBuf,dwCount,&dwRead,&old) == 0){
if(ERROR_IO_PENDING == GetLastError()){
bLError = WaitForSingleObject(hEvent,100);
if(bLError == WAIT_OBJECT_0){
if(GetOverlappedResult(hCom,&old,&dwCount,TRUE))));//データ受信完了
}
else if(bLError == WAIT_FAILED);//受信スリープエラー
else if(bLError == WAIT_TIMEOUT);//受信タイムアウト
}
else ;//受信エラー
}

//////////受信したデータ10バイト分をMessageBoxで表示//////////////////////
wsprintf(Buf,"文字 = %02x %02x %02x %02x %02x %02x %02x %02x %02x %02x",rdBuf[0],rdBuf[1],rdBuf[2],rdBuf[3],rdBuf[4],rdBuf[5],rdBuf[6],rdBuf[7],rdBuf[8],rdBuf[9]);
MessageBox(Buf);

GlobalAllocは今抱えているトラブルを解決したら、試してみます。

回答No.3

jx-word

指定されたバイト数を"読み込む"と考えると面倒なので、適当なバイト数をまとめて読み込んで、そこから必要な情報を取り出してはどうでしょうか。

unsigned char rdBuf[256];
int len;

if( ReadFile( hCom, rdBuf, sizeof( rdBuf), &dwRead, NULL)){
　len = rdBuf[0];

　memcpy( コピー先, &rdBuf[1], len);
}else{
　エラー処理
}

↑は1回のReadFileで全部読めることを前提として単純に書いてますが、途中で切れたことを考えるとループで回してバッファに詰める処理が必要です。

回答No.2

shimix

1バイト読む→データのサイズを確認する→そのバイト数分読み込む　でいいと思いますけど。

Cなら最初の1バイトを読んでからデータ用のメモリーを確保してもいいのでは？

補足

有難うございます。
dwCount=6となったので、6バイト分のメモリを確保しようと、
unsigned char rdBuf[dwCount];
としたのですが、以下３つのエラーが出ました。
定数式が必要です。
サイズが 0 の配列を割当てまたは宣言しようとしました。
'rdBuf' : サイズが不明です。

dwCountの値を使って、メモリーを確保するにはどうしたら良いのでしょうか？

ご指摘の程よろしくお願いします。