C言語 画像をバイナリファイルで読み込んで、メッセージの送受信をする

Question

前提・実現したいこと

C言語初心者で勉強している者です。できれば今日中に解決したいです。
画像データをバイナリファイルで読み込んで、メッセージの送受信を行いたいです。
（別プログラムでのメッセージの送受信）

発生している問題・エラーメッセージ

送信側、受信側ともにsend_sizeがundeclated(first use this function)となり
機能していないようですが原因がわかりません。
他にもプログラム上に問題があれば、ソースコードで教えていただけますと幸いです。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <errno.h>
 
#define KEY 2020
 
typedef struct{
         long mtype;
         int send_size;
         char image[1024];
 }DATA;
 
 int main()
 {
         DATA    data;
         data.mtype=1;
         data.send_size=0;
         char mtext[256];
 
         int msgid;
         FILE *fp;
         extern int errno;
 
 
         if((msgid=msgget(KEY,0666|IPC_CREAT))==-1){
             printf("MsgGet Error=%d\n",errno);
             exit(EXIT_FAILURE);
         }
 
         if((fp=fopen("gazou.png","rb"))==NULL){
             printf("ファイルが開けませんでした");
         exit(EXIT_FAILURE);
         }
 
 
 
         while(feof(fp)==0){
             data.send_size=fread(&data.image,1,1024,fp);
             printf("%d\n",data.send_size);
             printf("%ld\n",sizeof(data)-sizeof(long));
             printf("msgid:%d\n",msgid);
             if(msgsnd(msgid,&data,data.send_size+sizeof(int),0)==-1){
                 printf("errno=%d\n",errno);
                 perror(mtext);
                 fputs(mtext,stdout);
                 exit(EXIT_FAILURE);
             }
    
         }
         fclose(fp);
         exit(EXIT_FAILURE);
    
 }

#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <sys/types.h>
#include <sys/ipc.h>
#include <sys/msg.h>
#include <errno.h>
 
#define KEY 2020
 
typedef struct{
         long mtype;
         int send_size;
         char image[1024];
         }DATA;
 
 
 int main(int argc,char* argv[])
 {
     int status=EXIT_FAILURE;
     int msgid;
     extern int errno;
 
     DATA data;
 
     FILE *fp;
     char mmtext[256];
 
 
    
     data.send_size=0;
 
     if((fp=fopen("gazou2.png","wb"))==NULL){
         printf("ファイルが開けませんでした");
         exit(EXIT_FAILURE);
     }
 
 
     if((msgid=msgget(KEY,0666|IPC_CREAT))==-1){
         printf("MsgGet Error=%d\n",errno);
         exit(status);
     }
 
 
         while(1){
                 if(msgrcv(msgid,&data,sizeof(data)-sizeof(long),0,0)==-1){
                         printf("recieve error=%d\n",errno);
                         perror(mmtext);
                         exit(EXIT_FAILURE);
                }
                  fwrite(&data.image,1,data.send_size,fp);
                  printf("data.send_size=%d\n",data.send_size);
    
                 if(data.send_size<1024){
                     break;
                 }
         }
             printf("書込み終了\n");
             fclose(fp);
　            exit(EXIT_SUCCESS);
   }   

Linuxのubuntu上で開発しています。

Comments

[etsuhisa]

1つ目でgazouというファイルを1024バイトずつ読み取ってメッセージキューで2つ目に送り、2つ目でキューから読み取ったデータをgazouというファイルに書き込む、ということが実現したいことですか？

[nekoatsume]

コメントありがとうございます。
実現したいことはほぼご認識のとおりなのですが、
本文に記載漏れしていたところを追記します。

1バイトを1024個ずつ読み込んで、
データが1024より小さいことを確認して終了するような状態にしたいです。
（送信側も1バイトずつ送信して受信側が0になったときにfcloseする）
また、書き込んだファイル名は別名にしようと思っています。
本文に書いておらず申し訳ございません。

実行結果の確認方法についてもよくわかっていないので教えていただけたら幸いです、、。
質問が多く申し訳ございませんがよろしくお願いいたします。

Answer 1

2つ目のソースコードから、今回の実装とは直接関係のない処理を削除してスリム化したほうがよいです。

直接関係ないので削除。

     　    struct msgbuff{
          long mtype;
          char mtext[80];
          }b;

コマンドの第1引数は処理に影響を与えるものではない。他の「printf("Recieve～」もすべて削除。

          if(argc<2){
              printf("Argument required.\n");
              exit(status);
          }

          printf("Recieve %d Start.\n",no=atoi(argv[1]));

直接関係ないので削除。

                      printf("Receive%2d:",no);
                      fputs(b.mtext,stdout);

追加位置が違う。whileの閉じ括弧(})の直前。

                data.send_size=fwrite(&data,1,data.send_size,fp);    

1024バイト未満で処理を終了する要件なら、whileではなくdo～whileのほうがよいと思う。

   do {
       ：
   } while(data.send_size < 1024);

他にも、fopenエラー時のfcloseは不要、どちらのソースコードもmainの終了前にfcloseをいれる、
などの修正が必要です。

Comments

[nekoatsume]

上記、ご教授いただきありがとうございました。
ご指摘いただいた点をもとにコードを完成させることができました。（更新しました）
私の理解が甘く、不躾な質問してしまい申し訳ございませんでした。
今後は質問したい内容を整理して記載するよう努めたいと思います。

[etsuhisa]

完成できてなによりです。こちらも十分に対応できなかったにも関わらずBAに選んでいただきありがとうございました。今後ともよろしくお願いします。

Answer 2

1つ目のほうは、構造体dataに関する点の修正が必要だと思います。

              DATA    data;
              data.mtype=0; //構造体dataのメンバ
 　　         //data send_size; //不要な宣言

    while(feof(fp)==0){
        data.send_size=fread(&data.data,1,1024,fp); //send_sizeはdataのメンバにする
        if(msgsnd(msgid,&data,data.send_size+sizeof(int),0)==-1){ //構造体のメンバからsizeof(int)では？

2つ目は、msgrcvの後に、fwriteだと思います。
その他にも不要な定義を取り除いたりと、だいぶ直さないとならないと思います。

Answer 3

コンパイラのエラーの通りsend_size(とmtype)が宣言されていません。
data.mtypeやdata.send_sizeの誤りでしょう。
あといくつか全角スペースが入っています。

---

Recieve 0:　（ここにバイナリの文字が3文字ランダムで表示されている状態です）

fputs(b.mtext,stdout);
で初期化されていないb.mtextを出力しているためです。

---

• 受信側

etsuhisaさんも指摘されているようにデータを受け取る前にfwriteを呼んでしまっています。
ほかにぱっと見で気づいた点。
・data.send_sizeに修正したとしても、初期化されていないままwhileの条件式に使用している。
（追記）コードが更新されましたが、0初期化した後1024と比較しているため一度もループ実行されません。
・fwriteの書き込みサイズが受信したdata.send_sizeを使用せず1024固定になっている。
・条件式が代入文になっています。

-if(errno=ENOMSG)
+if(errno==ENOMSG)

Comments

[nekoatsume]

コメントありがとうございます。
上記の対応でエラーは消えたのですが、
実行結果がどのようになっていれば正解かを
理解できておらず、、プログラムが正常かどうか確認の方法がわかっておりません。
ちなみに受信側の結果は以下のように表示されます。
結果の確認方法について教えていただけないでしょうか。
Recieve 0 Start.
Recieve 0:　（ここにバイナリの文字が3文字ランダムで表示されている状態です）

また、何回か実行していたらMsgsnd Error=22となってしまい
ipcrmでキューを削除しみましたが上記の表示が消えません。。

[jimbe]

> 実行結果がどのようになっていれば正解かを理解できておらず
それでは「自分が何を作っているのか分からない」ということです.
なぜこのプログラムを書いているのでしょうか.