csv(txt)ファイルから読み込んだ文字の描画

[mi_l]

コード:

あいうえお,
かきくけこさしすせそ,
たちつてと,
/ここから下は・・・・・・・・・・・・・・・・・・・・・
なにぬねの,
はひふへほ,
まみむめも,
やゆよ,
らりるれろ,

龍神録からコピペして所々変えただけです（load_storyなど関数名はあまり気にしないでください）
全て同じファイルに書いているわけではありません（点線内は同じファイル）

コード:

//-----------------------------------------
typedef struct
{
	char *st;
}String;
//------------------------------------
String str;
//-----------------------------------
#define MAX_STR 8//今のところは8行なので・・(コメント文は除く)
//----------------------------------------------------------------------------
memset(str , 0 , sizeof(String)*MAX_STR);//初期化
//--------------------------------------------------------------------------
void load_story(){
        int n,num,i,fp;
        char fname[256]={"上のテキストファイル"};
        int input[256];
        char inputc[256];

        fp = FileRead_open(fname);//ファイル読み込み
        if(fp == NULL){
                printfDx("read error\n");
                return;
        }
     /*   for(i=0;i<2;i++)//最初の2行読み飛ばす
                while(FileRead_getc(fp)!='\n');*/

        n=0 ;
        while(1){
                for(i=0;i<256;i++){
                        inputc[i]=input[i]=FileRead_getc(fp);//1文字取得する
                        if(inputc[i]=='/'){//スラッシュがあれば
                                while(FileRead_getc(fp)!='\n');//改行までループ
                                i=-1;//カウンタを最初に戻して
                                continue;
                        }
                        if(input[i]==',' || input[i]=='\n'){//カンマか改行なら
                                inputc[i]='\0';//そこまでを文字列とし
                                break;
                        }
                        if(input[i]==EOF){//ファイルの終わりなら
                                goto EXFILE;//終了
                        }
                }
					srt[n].st = inputc;//文章1のポインタに「,」まで読み込んだ文章を格納
					n++;//コメントを除いて８行
                }
        }
EXFILE:
        FileRead_close(fp);
}
//-----------------------------------------------------------------------------------------------------------------------------
//グラフ
void str_graph()
{
	int i ;
	int col = GetColor(255 , 255 , 255);//色

	for(i=0 ; i < MAX_STR ; i++)
	{
		DrawString(100 , 30+(50*i) , str[i].st , col);
	}

}

大雑把に書きました。

上のコードで、訊きたいことは、ファイル（文字）読み込み＆格納はできているか・・・・（）

もし、textファイルで１００行あったとすると、このやり方はスマートであるか？

グラフ（描画）はあまり気にしないでください

思いつき（少し前のトピックの続きなのですが・・）でＤＸライブラリもインクルードされてないcppファイルにまとめて書いたので実行はしていません・・・

実行してないのに質問とは変ですが、ご教授お願いします。

[みけCAT]

DXライブラリには、ファイルから1行読み出す関数があります。
必ず1行に1個のテキストだけを入れるのなら、こちらのほうが遥かに簡単です。
コメントの処理は、読み込んだテキストの最初の文字が'/'だったらnを増やさない、という処理でできます。

[mi_l]

返信ありがとうございます

while文の中を自分なりに変えてみました

コード:

while(1)
		{
                for(int i=0 ; i < MAX_STR ; i++)
				{
                    FileRead_gets(str[i].st , 256 , fp);
                    if(str[i].st==EOF || i == (MAX_STR-1))
					{//ファイルの終わりなら
                            goto EXFILE;//終了
                    }
                }
		}

もしくは、

コード:

while(1)
		{
                for(int i=0 ; i < MAX_STR ; i++)
				{
                    FileRead_gets(inputc , 256 , fp);
                    if(str[i].st==EOF || i == (MAX_STR-1))
					{//ファイルの終わりなら
                            goto EXFILE;//終了
                    }
                }
				str[n].st = inputc;
				n++;

		}

上記二つとも
#define MAX_STR 9 //これは行数＋１の意味（そうしないと最後の行が読み込まれる前にgotoに行ってしまう）


このようなやり方でいいのでしょうか？

ご教授お願いします


コメント文はなくなったという設定で・・

[softya（ソフト屋)]

細かいことを言うと、FileRead_openの戻り値はファイルハンドルであって、ＦＩＬＥ構造体のポインタではないのでfpと言う名前は間違いです。
ちなみにファイル終端はFileRead_eof（）関数を使ってください。if(str.st==EOF などと言うサンプルは何処にもないはずです。
それとi == (MAX_STR-1)は余分です。

【追記】while(1)いらないのでは？ FileRead_gets（）は一行読むのであって、一文字読むのではありません。
【付記】MAX_STRと言う名前自体が変えたほうが良いです。

【追記の追記】
あと、こちらもお忘れなく。
「龍神録のタイトル画面の表示と遷移 • C言語交流フォーラム　～ mixC++ ～」
http://dixq.net/forum/viewtopic.php?f=3&t=13346#p106977
ちゃんと終わらせましょう。そうしないと、基本的な処で何度もミスを繰り返すことになります。

[mi_l]

ソフト屋さん、返信ありがとうございます。

なんか次に訊きたかったことまで答えてくださりありがとうございます。

while文は必要なのか？ということなのですけど・・・やっぱり要らないですよね！


そして、一応コードを載せます

コード:

//-----------------------------------------
typedef struct
{
    char *st;
}String;
//------------------------------------
String str;
//-----------------------------------
#define MAX_SENTENCE 8//今のところは8行なので・・(コメント文は除く)
//----------------------------------------------------------------------------
memset(str , 0 , sizeof(String)*MAX_STR);//初期化
//--------------------------------------------------------------------------
void load_story()
{
    int fh;//ファイルハンドル
    char fname[256]={"上のテキストファイル"};
 
    fh = FileRead_open(fname);//ファイル読み込み

    if(fh == NULL)
	{//空だったらエラー
            printfDx("read error\n");
            return;
    }
	 for(int i=0 ; i < MAX_SENTENCE ; i++)
    {
        FileRead_gets(str[i].st , 256 , fh);//i番目のstrのポインタに（i行目の）一文を格納

        if(FileRead_eof( fh ) != 0)//ファイルの終わりなら
        {
                goto EXFILE;//終了
        }
    }
    
EXFILE:
        FileRead_close(fp);
}

コメント文「/ここから・・・・・・・・・・・・・・・・・」は消したという設定で

そして、上記コードで私が、出来てほしいこと。
１、ファイルの読み込み
２、１番目のStringのポインタstに読み込んだファイルの１行目の文章を格納、（以下同様に２番目ー＞２行目・・・・ＭＡＸ＿ＳＥＮＴＥＮＣＥ－１番目まで）
３、ファイルの最後にいったら閉じてくれる。


ご教授、ご指摘お願いします。

[mi_l]

訂正

コード２７行目のコメント「//i番目のstrのポインタに（i行目の）一文を格納」

これは「//i番目のstrのポインタに（(i+1)行目の）一文を格納」


の間違いです。

[non]

ファイルハンドラはエラーのとき0を返します。0とNULLは仮に同じ0だとしても、意味が違います。

エラーの時、呼び出し側でなんらかの処理が必要になると思うので
void load_story()
でなく、何か返した方が良いと思います。

[mi_l]

実行しましたら思った通りに描画しました。


皆様ありがとうございます。

[softya（ソフト屋)]

最終形を投稿してほしのですが。
nonさんの提案は吸収されましたか？
ちなみに、gotoレスに出来ます。

[mi_l]

コード:

typedef struct
{
    char st[256];
}string_t;

void load_txt()
{
	int fh;//ファイルハンドル
    char fname[256]={"../dat/csv/sample001.txt"};
 
    fh = FileRead_open(fname);//ファイル読み込み
 
    if(fh == NULL)
    {//空だったらエラー
            printfDx("read errortxt\n");
            return;
    }
     for(int i=0 ; i < MAX_SENTENCE ; i++)
    {
        FileRead_gets(str[i].st , 256 , fh);//i番目のstrのポインタに（i行目の）一文を格納
 
        if(FileRead_eof( fh ) != 0)//ファイルの終わりなら
        {
                goto EXFILE;//終了
        }
    }
    
EXFILE:
        FileRead_close(fh);
}

です。

龍神録のやり方を参考にやっており、特に問題はないので、

EXFILEレスのやり方など、テクニックはとりあえずやりたいことができた後に自分で模索し、またここにトピックを出したいと思っております。

[softya（ソフト屋)]

nonさんの物が反映されてないですね。後だと忘れるのでやるなら今だと思います。