２次元配列のデータをテキストファイルのデータに更新

[北見里]

２次元配列のデータをテキストファイルのデータに更新したいのですがうまくできません
最初のint hantei[15][20] = {
{ 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
{ 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,1,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,1,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,1,0,0,0,1,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,1,1,1,0,0,0,1,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,1,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
{ 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
{ 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
};
を　smpl.txtに記入している
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1
1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,0,0,0,0,0,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,0,0,0,0,0,1,0,0,0,0,0,0,0,1,1
1,1,1,1,1,0,0,0,0,0,1,0,0,0,1,0,0,0,1,1
1,1,1,1,1,0,0,0,1,1,1,0,0,0,1,0,0,0,1,1
1,1,1,1,1,0,0,0,0,0,0,0,0,0,1,0,0,0,1,1
1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1
1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1
に変更したいです

コード:

 
#include "DxLib.h"
 #include <stdio.h>
#include <stdlib.h>


typedef struct{
        int x,y,img,muki,walking_flag;
}ch_t;
 
int hantei[15][20] = {
        { 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
        { 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,1,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,1,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,1,0,0,0,1,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,1,1,1,0,0,0,1,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,1,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,0,0,0,0,0,0,0,0,0,0,0,0,0,1,1 },
        { 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
        { 1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1,1 },
};
 
int IsAbleToGo(int x,int y,int muki){//進めるかを判定する
        if(muki==0)//上向きなら
                if(hantei[y/32-1][x/32]==1)//進めるか判定
                        return 1;//エラー
        if(muki==1)//左向きなら
                if(hantei[y/32][x/32-1]==1)
                        return 1;
        if(muki==2)//下向きなら
                if(hantei[y/32+1][x/32]==1)
                        return 1;
        if(muki==3)//右向きなら
                if(hantei[y/32][x/32+1]==1)
                        return 1;
        return 0;//正常
}
 int loadscript(void)
{
	FILE *fp;	/* (1)ファイルポインタの宣言 */
	char s[256];
	int iNum;
	/* (2)ファイルのオープン */
	/*  ここで、ファイルポインタを取得する */
	if ((fp = fopen("smpl.txt", "r")) == NULL) {
		printf("file open error!!\n");
		exit(EXIT_FAILURE);	/* (3)エラーの場合は通常、異常終了する */
	}
	
	/* (4)ファイルの読み（書き）*/
	while (fgets(s, 256, fp) != NULL) {
	iNum = (int)( s - '0' );
		/* ここではfgets()により１行単位で読み出し */
		printf("%s", s);printf("%d", iNum);
		

	}
	fclose(fp);	/* (5)ファイルのクローズ */
			for (int i = 0; i < 16; i++) {
	printf("%d\n", hantei[i]);
}
	return 0;
}
int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance,LPSTR lpCmdLine, int nCmdShow ){
         //デバッグ用にコンソールを呼び出す
        AllocConsole();
        freopen("CONOUT$", "w", stdout); 
        freopen("CONIN$", "r", stdin);
    int image[16],i,j;
    char Key[256];
    ch_t ch;
 
    if( ChangeWindowMode(TRUE) != DX_CHANGESCREEN_OK || DxLib_Init() == -1 ) return -1; //ウィンドウ化と初期化処理
 
    ch.x    =320;
    ch.y    =160;
    ch.walking_flag=0;
    ch.muki=3;
 
    SetDrawScreen( DX_SCREEN_BACK ) ;                                                 //描画先を裏画面に設定
    LoadDivGraph( "char.png" , 16 , 4 , 4 , 32 , 32 , image ) ;//画像を分割してimage配列に保存
 loadscript();
    while(!ProcessMessage() && !ClearDrawScreen() && !GetHitKeyStateAll( Key ) && !Key[KEY_INPUT_ESCAPE]){
            //↑ﾒｯｾｰｼﾞ処理        ↑画面をｸﾘｱ         ↑ｷｰﾎﾞｰﾄﾞ入力状態取得       ↑ESCが押されると終了
 
                /*白い壁を描画*/
                for(i=0;i<15;i++)
                        for(j=0;j<20;j++)
                                if(hantei[i][j]==1)
                                        DrawBox(j*32,i*32,(j+1)*32,(i+1)*32,GetColor(255,255,255),TRUE);
 
                if(ch.x%32==0 && ch.y%32==0){         //座標が32で割り切れたら入力可能
            ch.walking_flag=1;         //歩くフラグを立てる。
            if     ( Key[ KEY_INPUT_UP   ]  == 1 )  //上ボタンが押されたら
                    ch.muki=0;         //上向きフラグを立てる
            else if( Key[ KEY_INPUT_LEFT ]  == 1 )  //左ボタンが押されたら
                    ch.muki=1;         //左向きフラグを立てる
            else if( Key[ KEY_INPUT_DOWN ]  == 1 )  //下ボタンが押されたら
                    ch.muki=2;         //下向きフラグを立てる
            else if( Key[ KEY_INPUT_RIGHT]  == 1 )  //右ボタンが押されたら
                    ch.muki=3;         //右向きフラグを立てる
            else                                    //何のボタンも押されてなかったら
                    ch.walking_flag=0; //歩かないフラグを立てる
            if(ch.walking_flag==1)    //もし歩くなら
                if(IsAbleToGo(ch.x,ch.y,ch.muki)==1)//行き先が歩けないなら
                    ch.walking_flag=0;                  //歩かないフラグを立てる。
        }
 
        if(ch.walking_flag==1){        //歩くフラグが立っていたら
            if     (ch.muki==0)        //上向きならch.y座標を減らす
                    ch.y--;
            else if(ch.muki==1)        //左向きならch.x座標を減らす
                    ch.x--;
            else if(ch.muki==2)        //下向きならch.y座標を増やす
                    ch.y++;
            else if(ch.muki==3)        //右向きならch.x座標を増やす
                    ch.x++;
        }
 
        ch.img=image[(ch.x%32+ch.y%32)/8 + ch.muki*4];            //画像をセット
 
        DrawGraph( ch.x , ch.y , ch.img , TRUE ) ;//画像を描画
 
        ScreenFlip();
    }
       //コンソール解放
        FreeConsole();
    DxLib_End();
    return 0;
}

[beatle]

まず58行目に注目してみましょう。
(int)(s - '0')
は何をやろうとしているのでしょうか？
何を言われているのか良くわからないならば、下記の質問にお応えください。

• s はどんな値でしょうか（整数でしょうか、文字でしょうか、ポインタでしょうか）
• s - '0' はどんな値でしょうか（文字を整数に変換した値でしょうか、ポインタでしょうか）

[北見里]

すいません。(int)(s - '0')はググって試行錯誤で書いてみたコードです。
消し忘れでした。
s は文字列ですね。テキストファイルは文字列からでないと読み込めない？みたいなことが書いてあったので
s - '0'で文字を整数に変換しようとしましたがうまくできませんでした

[かずま]

これを試してみてください。

コード:

int loadscript(void)
{
    int i, j;
    FILE *fp = fopen("smpl.txt", "r");
    if (fp == NULL) {
        printf("file open error!!\n");
        exit(EXIT_FAILURE);
    }
    for (i = 0; i < 15; i++) {
        for (j = 0; j < 20; j++) {
            if (fscanf(fp, "%d%*c", &hantei[i][j]) != 1) {
                printf("file read error!!\n");
                exit(EXIT_FAILURE);
            }
        }
    }
    fclose(fp);
    return 0;
}

fscanf の書式の中の %*c は、ファイル中の ',' または '\n' を読み捨てるためのものです。

[beatle]

すいません。(int)(s - '0')はググって試行錯誤で書いてみたコードです。
消し忘れでした。
s は文字列ですね。テキストファイルは文字列からでないと読み込めない？みたいなことが書いてあったので
s - '0'で文字を整数に変換しようとしましたがうまくできませんでした

消し忘れってどういう意味でしょうか？
iNum = (int)( s - '0' );
の行は本来なら書く必要がないという意味でしょうか？
でもそれだとiNumに値が入りませんのでダメですよね。

さて、s - '0'についての質問に回答していただいてありがとうございます。
文字列やポインタあたりはハマリポイントですから難しいのは仕方ありません。
北見里さんの回答には矛盾が含まれています。

sは仰るとおり文字列です。正確にはchar型配列ですね。
そしてソースコードに「s」と書くとその配列の先頭位置を表すポインタになります。

ということで、s - '0' は「配列の先頭位置から'0'を引く」ということです。
ASCII文字コードだと仮定すると「配列の先頭位置から48を引く」ということになります。
つまるところ s - '0' は全体で「ポインタ」なんですね。さらに言えば無効なポインタ。

「s - '0'で文字を整数に変換」と書いてますが、sは文字ではなくて配列の先頭位置ですから、ここが間違っているのですね。
でもやろうとしていることは分かります。例えば
char c = '5';
int i = c - '0'; /* i == 5 */
というのは良くやるやり方です。cは文字型（非ポインタ型）であり、文字コードが格納されていますので c - '0' で文字→整数変換ができるんですね。
でも今回はsは文字型ではなくて、ある記憶領域の先頭を表すポインタですから、 s - '0' では無効なポインタを生成してしまっているだけです。

恐らく手っ取り早い方法はかずまさんのソースコードを使うことです。
北見里さんがプログラミング力を向上させたいと願っているなら、いくつかアドバイスがあります。

• ソースコードはきちんとインデントしましょう
• C言語の教科書の、特に「char型」と「配列」と「ポインタ」の章を読みましょう

その上で、いきなり2次元配列を扱おうとするのではなくて、1つの数字を読み取るだけのプログラムを、2次元配列のやり方に拡張できそうなやり方でやってみるのが良いと思います。

[北見里]

かずまさんのコードでうまくいきました。
ポインタに関する知識が曖昧なのでもう一度勉強してきます＾＾
お二方ありがとうございました。