fopen_sを使った座標のセーブ、ロード

Question

前提・実現したいこと

dxライブラリを使いマウスで線を引くようなプログラムを作っています。今回はその書いた線をfwrite,freadを使って保存、読み込みをしたいと考えています

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

保存の方はできているらしいのですが読み込みの時がうまくいきません(例外スロー)。自分が思うにlineCnt,buf_cntあたりがおかしいのかなとは思うのですがどこをどう直せばいいのか分からない状況です

該当のソースコード

#include "DxLib.h"
#include <stdlib.h>
#include<stdio.h>
const int WIN_WIDTH = 920; //ウィンドウ横幅
const int WIN_HEIGHT = 600;//ウィンドウ縦幅

//モードの構造体
enum PTYPE
{
    Non,
    Pen,
    pen2,
    Eraser,
};

//線の構造体(座標、色)
typedef struct line
{
    float BlackLineX;
    float BlackLineY;
    int color;

    PTYPE Ptype;
}line;


int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance, LPSTR lpCmdLine, int nCmdShow)
{
    //初期化前の設定
    ChangeWindowMode(TRUE);
    SetGraphMode(WIN_WIDTH, WIN_HEIGHT, 32);

    //DXライブラリ初期化
    if (DxLib_Init() == -1)return -1;

    //背景色設定
    SetDrawScreen(DX_SCREEN_BACK);
    SetBackgroundColor(50, 50, 50);
    float speed = 3;
    int lineMaX = 1000;
    //カウント用の変数
    float len = 0;
    int lineCnt = 0;
    //セーブ、ロード用の変数
    float buf_x[1000];
    float buf_y[1000];
    float buf_x2[1000];
    float buf_y2[1000];
    int buf_cnt = 0;
    int buf_cnt2 = 0;

    FILE* fp;
    FILE* fp2;
    FILE* fp3;
    
    //malloc
    line* bline;
    bline = (line*)malloc(lineMaX * sizeof(line));
   
    if (bline == NULL) {
        return 0;
    }

    //マウス情報用変数
    int posX, posY;
    bool Drag = false, penflag = true;
    int type = 0;
    //キー入力用変数
    char KeyBuf[256];

    int save=0;
    int load = 0;
    //メインループ
    while (!ProcessMessage())
    {
        ClearDrawScreen();
        //キー入力１
        GetHitKeyStateAll(KeyBuf);

        if (KeyBuf[KEY_INPUT_1] == 1) { penflag = true; }
        if (KeyBuf[KEY_INPUT_2] == 1) { penflag = false; Drag = false; }
        if (KeyBuf[KEY_INPUT_0] == 1) { lineCnt = 0; }

        

        //色の切り替え
        if (KeyBuf[KEY_INPUT_A] == 1) { type = 0; }
        if (KeyBuf[KEY_INPUT_R] == 1) { type = 1; }
        if (KeyBuf[KEY_INPUT_G] == 1) { type = 2; }
        if (KeyBuf[KEY_INPUT_B] == 1) { type = 3; }
        if (KeyBuf[KEY_INPUT_5] == 1) { bline[lineCnt].Ptype = pen2; }
        //マウス座標取得
        GetMousePoint(&posX, &posY);

        //マウス入力処理
        if (GetMouseInput() & MOUSE_INPUT_LEFT) {//mouseが押されたとき

            //ドラッグ中は座標を追加
            bline[lineCnt].BlackLineX = posX;
            bline[lineCnt].BlackLineY = posY;

            //ドラッグ中は線を引く
            if (penflag) {
                if (Drag) {
                    bline[lineCnt].Ptype = Pen;//modeをペンに
                }

                //ドラッグ開始 drag中ではない
                else {
                    Drag = true;    //ドラッグフラグをオンにする
                    bline[lineCnt].Ptype = Non;
                }
            }
            //penflagがfalseの時typeを消しゴムに
            else {
                bline[lineCnt].Ptype = Eraser;
            }

            //mouse座標に毎フレーム座標を追加している
            lineCnt++;
            //メモリが足りなくなったら拡大
            //relloc
            if (lineCnt == lineMaX) {//最大数なったら
                lineMaX += 1000;//1000増やす
                line* tmp = (line*)realloc(bline, lineMaX * sizeof(line));
                if (tmp == NULL) {
                    free(bline);
                    return -1;
                }
                bline = tmp;
            }
        }

        else {//マウスのキーは押されていない
            if (Drag)Drag = false;  //ドラッグフラグをオフにする
        }

        //色の切り替え
        if (type == 0) { bline[lineCnt].color = GetColor(255, 255, 255); } else if (type == 1) { bline[lineCnt].color = GetColor(255, 0, 0); } else if (type == 2) { bline[lineCnt].color = GetColor(0, 255, 0); } else if (type == 3) { bline[lineCnt].color = GetColor(0, 0, 255); }

        //bline配列の情報を描画
        for (int i = 0; i < lineCnt; i++) {
            switch (bline[i].Ptype)
            {
            //pen書くとき
            case Pen:
                //一つ前の座標から今の座標へつなぐ線
                DrawLine(bline[i - 1].BlackLineX, bline[i - 1].BlackLineY, bline[i].BlackLineX, bline[i].BlackLineY, bline[i].color,6);

                break;
                //消しゴム描くとき
            case Eraser:
                DrawBox(bline[i].BlackLineX - 20, bline[i].BlackLineY - 20, bline[i].BlackLineX + 20, bline[i].BlackLineY + 20, GetColor(5,5,5), TRUE);
                break;
            }
        }

        //セーブ
        if (KeyBuf[KEY_INPUT_S] == 1) {
            save = 1;
        }
        //ロード
        if (KeyBuf[KEY_INPUT_L] == 1) {
            load = 1;
        }

        //線のセーブ部分(現在使われてる配列の要素数、線分のx座標、線分のy座標)
        if (save == 1) {
            buf_cnt = lineCnt;
            for (int i = 0; i < lineCnt; i++) {
                buf_x[i] = bline[i].BlackLineX;
                buf_y[i] = bline[i].BlackLineY;
            }

            //使われた配列の要素数の保存
            fopen_s(&fp3, "test.dat", "wb");
            if (fp3 == NULL) {
                DrawFormatString(150, 300, GetColor(255, 255, 255), "セーブできません\n");
            }              
            else {
                DrawFormatString(150, 300, GetColor(255, 255, 255), "buf_Cntセーブ成功:%d\n",buf_cnt);
                fwrite(&buf_cnt, sizeof(int), 1, fp3);
                fclose(fp3);
            }
            
            //x座標の保存
            fopen_s(&fp, "test.dat", "wb");
            if (fp == NULL) {
                DrawFormatString(150, 340, GetColor(255, 255, 255), "セーブできません\n");
            }
            else {
                DrawFormatString(150, 340, GetColor(255, 255, 255), "x座標セーブ成功\n");
                //buf_cnt分書き出し
                fwrite(&buf_x, sizeof(float),buf_cnt, fp);
                fclose(fp);
            }

            //y座標の保存
            fopen_s(&fp2, "test.dat", "wb");
            if (fp2 == NULL) {
                DrawFormatString(150, 380, GetColor(255, 255, 255), "セーブできません\n");
            }
            else {
                DrawFormatString(150, 380, GetColor(255, 255, 255), "Y座標セーブ成功\n");
                //buf_cnt分書き込み
                fwrite(&buf_y, sizeof(float),buf_cnt, fp2);
                fclose(fp2);
            }
        }
        //線のロード
        if (load == 1) {
            //使われた配列の要素数の読み込み
            fopen_s(&fp3, "test.dat", "rb");
            if(fp3==NULL){
                DrawFormatString(0, 300, GetColor(255, 255, 255), "ロードできません\n");
            }
            else {
                DrawFormatString(0, 300, GetColor(255, 255, 255), "ロード成功\n");
                fread(&buf_cnt2, sizeof(int), 1, fp3);
                fclose(fp3);
                lineCnt = buf_cnt2;
            }

            //x座標の保存
            fopen_s(&fp, "test.dat", "rb");
            if (fp == NULL) {
                DrawFormatString(0, 340, GetColor(255, 255, 255), "ロードできません\n");
            }            
            else {
                DrawFormatString(0, 340, GetColor(255, 255, 255), "ロード成功\n");
                //buf_cnt分読み出し
                fread(&buf_x2, sizeof(float),buf_cnt, fp);
                fclose(fp);
                for (int i = 0; i < buf_cnt; i++) {
                    bline[i].BlackLineX = buf_x2[i];
                }
            }

            //y座標の保存
            fopen_s(&fp2, "test.dat", "rb");
            if (fp2 == NULL) {
                DrawFormatString(0, 380, GetColor(255, 255, 255), "ロードできません\n");
            }
            else {
                DrawFormatString(0, 380, GetColor(255, 255, 255), "ロード成功\n");
                //buf_cnt分読み出し
                fread(&buf_y2, sizeof(float), buf_cnt, fp2);
                fclose(fp2);
                for (int i = 0; i < buf_cnt; i++) {
                    bline[i].BlackLineY = buf_y2[i];
                }
            }
           }
         
        //文字等描画
        {
            DrawString(0, 0, "キー操作：0=全削除 1=ペン 2=消しゴム", 0x0);
            DrawFormatString(0, 32, GetColor(0, 0, 0), "現在の配列使用数:%d(確保数:%d)", lineCnt, lineMaX);
            DrawFormatString(0, 64, GetColor(0, 255, 0), "線のセーブ:S");
            DrawFormatString(0, 96, GetColor(0, 255, 0), "線のロード:L");
            if (penflag) {
                DrawString(0, 16, "現在のモード：ペン", 0x0);
                DrawLine(posX, posY - 10, posX, posY + 10, 0x0);
                DrawLine(posX - 10, posY, posX + 10, posY, 0x0);
            } else {
                DrawString(0, 16, "現在のモード：白塗り", 0x0)
                    ;
                DrawBox(posX - 20, posY - 20, posX + 20, posY + 20, GetColor(255, 255, 255), TRUE);
                DrawBox(posX - 20, posY - 20, posX + 20, posY + 20, GetColor(255, 255, 255), FALSE);
            }
        }
        
        //表画面に転送
        ScreenFlip();
    }

    free(bline);
    DxLib_End();

    return 0;
}

試したこと

for文の中身とかを変えてみましたがそれでもうまく動いてくれませんでした

補足情報（FW/ツールのバージョンなど）

現在詰まっている所としてはセーブしてからロードする際、セーブ時に書いた線を画面に表示させたいのですが今のところだと何も表示されない状態です

Comments

[itagagaki]

開発は何で行っていますか？

[matsu0o]

Dxライブラリの方を使わせてもらっています

[itagagaki]

お聞きしたのは、OSは何か、コンパイラは何かです

[matsu0o]

申し訳ありません。OSの方はwindows　コンパイラの方はMicrosoft visual C++ですね

[itagagaki]

Visual Studioでしたらデバッグ実行して例外が起こったときに呼び出し履歴を見てみるなどしてみると良いと思いますが、それはやってみましたか？

[matsu0o]

呼び出し履歴や出力ウィンドウは見ました　これに関しては自分の知識不足なんですが書き込みアクセス違反くらいしか分らず他は正直何を書いているのかが全く分かりませんでした

Answer 1

まず、fopen_sのヘルプを検索して見つけます。

fopen_s, _wfopen_s

このページを開いて、第三引数に"w"を渡したときに何が起きるのかを調べます。

"w" 書き込み用に空のファイルを開きます。 指定したファイルが既に存在すると、そのファイルの内容は破棄されます。

これを踏まえて質問のプログラムを見てみましょう。

            //使われた配列の要素数の保存
            fopen_s(&fp3, "test.dat", "wb");
            if (fp3 == NULL) {
                DrawFormatString(150, 300, GetColor(255, 255, 255), "セーブできません\n");
            }              
            else {
                DrawFormatString(150, 300, GetColor(255, 255, 255), "buf_Cntセーブ成功:%d\n",buf_cnt);
                fwrite(&buf_cnt, sizeof(int), 1, fp3);
                fclose(fp3);
            }

            //x座標の保存
            fopen_s(&fp, "test.dat", "wb");
            if (fp == NULL) {
                DrawFormatString(150, 340, GetColor(255, 255, 255), "セーブできません\n");
            }
            else {
                DrawFormatString(150, 340, GetColor(255, 255, 255), "x座標セーブ成功\n");
                //buf_cnt分書き出し
                fwrite(&buf_x, sizeof(float),buf_cnt, fp);
                fclose(fp);
            }

            //y座標の保存
            fopen_s(&fp2, "test.dat", "wb");
            if (fp2 == NULL) {
                DrawFormatString(150, 380, GetColor(255, 255, 255), "セーブできません\n");
            }
            else {
                DrawFormatString(150, 380, GetColor(255, 255, 255), "Y座標セーブ成功\n");
                //buf_cnt分書き込み
                fwrite(&buf_y, sizeof(float),buf_cnt, fp2);
                fclose(fp2);
            }

最初にfopen_s(&fp3, "test.dat", "wb");でファイルを開いてbuf_cntを書き込みます。
次にfopen_s(&fp, "test.dat", "wb");で同じファイルを開きますが、指定したファイルが既に存在するので、そのファイルの内容は破棄されます。 つまり、直前に書き込んだbuf_cntの内容を破棄したうえでbuf_xを書き込みます。
最後にfopen_s(&fp2, "test.dat", "wb");で同じファイルを開きますが、指定したファイルが既に存在するので、そのファイルの内容は破棄されます。 つまり、直前に書き込んだbuf_xの内容を破棄したうえでbuf_yを書き込みます。

これで、何が起こっているか理解していただけたでしょうか。

1つのファイルに複数の情報を書き込みたいのであれば、1回だけfopen_sしたうえで、fwriteを必要な回数だけ繰り返すようにしてください。freadの側も同様です。

Comments

[matsu0o]

なるほど　何回もfopenしてしまうと前に読み込んだ情報が消えてしまうということですか、、知りませんでした　ありがとうございます
コードとしてはこのような書き方で問題ないのでしょうか(書いた線の保存はまだですが 一応エラーの方は消えました)
if (save == 1) {
buf_cnt = lineCnt;
for (int i = 0; i < buf_cnt; i++) {
buf_x[i] = bline[i].BlackLineX;
buf_y[i] = bline[i].BlackLineY;
}

//使われた配列の要素数の保存
fopen_s(&fp, "test.dat", "wb");
if (fp == NULL) {
DrawFormatString(150, 300, GetColor(255, 255, 255), "セーブできません\n");
} else {
DrawFormatString(150, 300, GetColor(255, 255, 255), "buf_Cntセーブ成功:%d\n", buf_cnt);
for (int i = 0; i < buf_cnt; i++) {
DrawFormatString(600,i*21, GetColor(255, 255, 255), "buf_xセーブ成功:%f\n", buf_x[i]);
}

fwrite(&buf_cnt, sizeof(int), 1, fp);
fwrite(&buf_x, sizeof(float), buf_cnt, fp);
fwrite(&buf_y, sizeof(float), buf_cnt, fp);

fclose(fp);
}

}
if (load == 1) {
//x座標の保存
fopen_s(&fp, "test.dat", "rb");
if (fp == NULL) {
DrawFormatString(0, 340, GetColor(255, 255, 255), "ロードできません\n");
} else {
DrawFormatString(0, 340, GetColor(255, 255, 255), "ロード成功\n");

fread(&buf_cnt2, sizeof(int), 1, fp);//buf_cnt分読み出し
fread(&buf_x2, sizeof(float), buf_cnt, fp);
fread(&buf_y2, sizeof(float), buf_cnt, fp);
fclose(fp);
lineCnt = buf_cnt2;
for (int i = 0; i < buf_cnt; i++) {
bline[i].BlackLineX = buf_x2[i];
bline[i].BlackLineY = buf_y2[i];
DrawFormatString(300, i * 21, GetColor(255, 255, 255), "buf_xセーブ成功:%f\n", bline[i].BlackLineX);
}
}

}

[actorbug]

手元に環境が無いので目視のみの確認となりますが、問題はなさそうです。

[matsu0o]

非常に分かりやすい説明でした。ありがとうございます　追加の質問で申し訳ないのですが今の自分のコードだと描いた線が保存されずにロードしても画面に何も映りません。このコードだと、また別にセーブ、ロードする物を追加する必要があるのでしょうか、、？ 個人的には
for(int i=0;i<lineCnt;i++){DrawLine(bline[i - 1].BlackLineX, bline[i - 1].BlackLineY, bline[i].BlackLineX, bline[i].BlackLineY, bline[i].color,6);}
これで線を描画しているので lineCnt,bline[i]BlackLineX,bline[i]BlackLineY この3つをセーブ、ロードすればできると思ったんですが、、、

[actorbug]

申し訳ないですが、これ以上のことは私ではわかりかねます。
私より詳しい他の方に回答してもらってください。