質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.60%

  • Linux

    3641questions

    Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

  • C++

    3337questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

  • シリアルポート

    43questions

    シリアルポートは一度に一ビットごと移行される物理的なインターフェイスです。一般的には、9ピンのd-subコネクタであるRS-232を指します。

c++によるXMODEMの受信関数を作っているが、dataの読み込みする前に終了する

解決済

回答 2

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 498

NEWBIEEBIEE

score 37

失礼します。
昨日ここのサイトでお世話になった者です。
引き続きよろしくお願いします。
こちらのコードですが、ここ(https://gist.github.com/zonque/0ae2dc8cedbcdbd9b933)のサイトのコードを元に、xmodem_receiveという関数を新たに実装し,XMODEMのデータ受信をしたいと思っております。
作成した関数なのですが、途中で何故か終了してしまいます。
受信側のxmodem_receiveは"Sending C ping..."でファイル送信の合図である'C'を送信側に対しておくることができており、その後、送信側はファイルの送信を行っているようなのですが
送信側
受信側のxmodem_receiveはそこで終了します。
受信側
データの受信を続けたいのですが、
肝心のデータ受信部分が動作しません。
データを受け取らずに終了してしまいます。
プログラムの構成がいけないのでしょうか
また、STKを
データから受信する際にprintfで出力したら
if文の(recSTX == X_STX)で通るにもかかわらず
’K’と出てくるのはなぜなのでしょうか。

ご教授お願いします

全体のコードはこちらの最下部にあります(http://stackoverflow.com/questions/41502457/my-implementation-of-xmodem-receiving-function-is-not-working)
わたしが実装した関数

static int xmodem_receive(int serial_fd, char* filename, int _crc){
    int skip = 0;
    uint8_t sdCRC = 'C';
    uint8_t sdACK = X_ACK;
    uint8_t eof = X_EOF;
    uint8_t sdNAK = X_NAK;
    uint8_t recSTX; // receive SOH from chunk
    uint8_t recBlk; // receive blk from chunk
    uint8_t recNegBlk; // receive blk negative from chunk
    uint8_t recData[1024]; // receive data from chunk
    uint16_t recChksum;
    FILE *fp;
    int ret, fd;
    struct stat stat;
    // If we want to receive, We have to send NAK to Sendor.



        /* バイナリ書き込み読み込みモードでファイルをオープン */
    fp = fopen(filename, "wb");
    //Send NAK still read SOH that header of one data chunks
        while(1){
            char status;
            printf("Waiting for sender ping ...");
            fflush(stdout);
            //
            do {
                    if(_crc){
                        printf("Send C ping....\n");
                        ret = write(serial_fd, &sdCRC, sizeof(sdCRC));
                    }else{
                        // send NAK before read SOH
                        printf("Send NAK ping....\n");
                        ret = write(serial_fd, &sdNAK, sizeof(sdNAK));
                    }    // after sending NAK,receiving SOH from chunk
                    fflush(stdout);
                    ret = read(serial_fd, &recSTX, sizeof(recSTX));


                    if(recSTX == X_STX){
                        printf("STX is %c\n", &recSTX);
                    }else{
                        printf("Garabage payload %c\n", &recSTX);
                    }
                    fflush(stdout);
                    if (ret != sizeof(recSTX)) {
                            printf("Not working");
                            fflush(stdout);
                            perror("read");
                            return -errno;
                    }
            } while (recSTX != X_STX);

            //So next, receiving blk
            ret = read(serial_fd, &recBlk, sizeof(recBlk));
            printf("Block Num is %d\n", recBlk);
            fflush(stdout);
            if (ret != sizeof(recBlk)) {
                perror("read");
                return -errno;
            }
            ret = read(serial_fd, &recNegBlk, sizeof(recNegBlk));
            printf("Negative Block Num is %d\n", recNegBlk);
            fflush(stdout);
            if (ret != sizeof(recNegBlk)) {
                perror("read");
                return -errno;
            }
            ret = read(serial_fd, (void *)&recData, sizeof(recData));
            printf("Data buffer is %c\n", &recData);
            fflush(stdout);
            if (ret != sizeof(recData)) {
                perror("read");
                return -errno;
            }
            ret = read(serial_fd, &recChksum, sizeof(recChksum));
            printf("Check sum is %c", &recChksum);
            fflush(stdout);
            if (ret != sizeof(recChksum)) {
                perror("read");
                return -errno;
            }
            // data block number check
            if(recBlk == 0xff - recNegBlk){
                perror("read");
                return -errno;
            }
            // data integrity check
            if(recChksum == swap16(crc16(recData, sizeof(recData)))){
                perror("read");
                return -errno;
            }
            // check of appended "0xff" from end of data to  end of chunk in chunk
            unsigned char *ptr = recData;
            ptr += sizeof(recData);
            while(*ptr == 0xff){
                ptr--;
            }
            fwrite(recData, (ptr - recData),1,fp); // write Datas bytes from our buffer
            // set skip flag or end connect
            ret = write(serial_fd, &eof, sizeof(uint8_t));
            if (ret != sizeof(eof) || ret != sizeof(X_STX)){
                return -errno;
            }else{
                if(ret == X_STX){
                    skip = 1;
                }else if(ret == EOF){
                    printf("EOF ...");
                    ret = write(serial_fd, &sdACK, sizeof(sdACK));
                    break;
                }else{
                    return -errno;
                }
            }

        }
    printf("done.\n");
    fclose(fp);
    return 0;

}


main関数

int main(int argc, char **argv)
{
        int a, ret, serial_fd;

        printf("STX: %c\n", X_STX);

        printf("ACK: %c\n", X_ACK);

        printf("NAK: %c\n", X_NAK);

        printf("EOF: %c\n", X_EOF);

        fflush(stdout);

        serial_fd = open_serial("/dev/ttyUSB0", 115200);
        if (serial_fd < 0)
                return -errno;


        if(std::string(argv[2]) == "0" || std::string(argv[2]) == "0"){
            ret = xymodem_send(serial_fd, argv[1], PROTOCOL_XMODEM, 1);
            if (ret < 0)
                return ret;
        }

        if(std::string(argv[2]) == "1"){
            ret = xmodem_receive(serial_fd, argv[1], 1);
            if (ret < 0)
                return ret;
        }


        return 0;
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • Chironian

    2017/01/07 15:25 編集

    受信側に問題がありそうですので、受信側の画面を載せた方が回答しやすいです。また、デバッグ用にprintf()する時は常にfflush()しておいた方が良いです。printf()だけですとバッファに溜まって画面にでるのが遅れますので。

    キャンセル

  • NEWBIEEBIEE

    2017/01/07 17:43

    以後気をつけます。

    キャンセル

回答 2

checkベストアンサー

0

途中で何故か終了してしまいます。 

stackoverflowでも指摘されていると思いますが・・・

受信側のプログラムコードの各関数の返り元が正常かどうか、どんな動きになっているかなどの情報が書かれていないので質問者さん自身のデバッグが充分に行われていないように見えます。例えばCRC/NAKをwriteした結果を変数に受けているだけでチェックしてませんし・・・

CRC/NAKのwriteが失敗するということを示唆しているわけではありません。それらも含めてコードの動作を充分に確認・把握していないように見えるという点を指摘しているつもりです。「途中で何故か終了してしまいます。 」ではなく最低限「このコードのここの関数の結果がこうなったためにプログラムが終了しているがそれがなぜそのような動きになるかわからない」という質問の仕方にすべきではないでしょうか?

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/07 17:43

    ご指摘のほどありがとうございます。

    キャンセル

  • 2017/01/07 18:21 編集

    多分こうした質問をする背景には「動くはずなのに何故動かないんだ」という気持ちがあるだろうと思います。自分もプログラマーの端くれなのでその気持ちはわかるのです。しかし最小限のデバッグをするということは知識の多い少ないにかかわらずできるはずです。その結果「なぜこうなるんだろう」となったときが難しいと思うんです。そのタイミングで質問するとよい質問になると思います。デバッグ頑張ってくださいね!

    キャンセル

  • 2017/01/08 14:42

    ありがとうございます。勉強にさせていただきます。

    キャンセル

0

こんにちは。

最初のSTX is Kは期待通りの動作でしょうか?
期待通りでないなら、それが何故なのか追跡しましょう。
どうしても分からない場合に、何故このように表示されるのか質問した方が良いと思いますよ。

皆さんポランティアですから、丸投げされると回答しづらいです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/01/08 14:41

    申し訳ございません。配慮の足りなさを憂いています。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.60%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

  • 解決済

    配列を関数に渡す

    c言語で「好きな言葉を入力すると、その言葉を一文字ずつ出力する(文字と文字の間は0.1秒)」というプログラムを作ろうと思い、以下のコードを書いてみました #include<s

  • 解決済

    cp -r コマンドのC言語による実装

    前提・実現したいこと ls -r コマンドのC言語のソースコードを参考にして cp -r コマンドを実装しているのですが どこを変えていいのかわかりません… 該当のソー

  • 受付中

    C言語でHTTP通信でHTMLファイルを取得し、保存することは可能でしょうか?C言語でクローリングが...

    初心者です。 よろしくお願いいいたします。 早速ですが、私はC言語でWebスクレイピング・Webクローリングをしたいと思っております。 WebスクレイピングやWebク

  • 解決済

    Rubyから実行ファイルを実行して、対話的に処理を行いたい。

    タイトルのとおりです。 実行ファイルを、端末からではなくRubyで実行して、入出力を対話的に行いたいです。 ただ、下のプログラムを動かすと、なにも出力せずとまります。 メッセ

  • 解決済

    [c]一つのデータでカラム数が変わるデータの読み込み

    質問失礼します。プログラム初心者です。 以下のような〜〜〜.datを fp1 = fopen(fname1,"r"); while((ret = fscanf(fp1,"%d %d

  • 受付中

    プログラムを見やすく改良したい

    正常に動くプルグラムを見やすく改良したい。 具体的に教えていただければありがたいです。セグメンテーションフォルトでベスト7まで表示して停止します。173行あたりだと思うのですが、よ

  • 解決済

    c言語 Segmentation faultについて

     前提・実現したいこと コンパイルは通るのですが、プログラムを実行すると Segmentation faultとなります。 調べてみるとmallocなどを使うべきなのかと思って

  • 受付中

    c言語のif文について

    c言語のif文の条件について 最近c言語の勉強をし始めたものです 以下のようなif文の条件の書き方を教えてください。 キーボード入力(key)から 自然数を入力→ ● a を入

同じタグがついた質問を見る

  • Linux

    3641questions

    Linuxは、Unixをベースにして開発されたオペレーティングシステムです。日本では「リナックス」と呼ばれています。 主にWebサーバやDNSサーバ、イントラネットなどのサーバ用OSとして利用されています。 上位500のスーパーコンピュータの90%以上はLinuxを使用しています。 携帯端末用のプラットフォームAndroidは、Linuxカーネル上に構築されています。

  • C++

    3337questions

    C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

  • シリアルポート

    43questions

    シリアルポートは一度に一ビットごと移行される物理的なインターフェイスです。一般的には、9ピンのd-subコネクタであるRS-232を指します。