ファイルから読込んだ文字列が文字化け

Question

C言語で、タスク情報を記録できるシステムを作成しています。due dateおよびTask nameはtxtファイルで読み込み、)表示することができたのですが、その他の情報が書かれたtxtファイルを読込んで表示させようとすると文字化けしてしまいます。文字化けしないようにするにはどのようにしたらよいでしょうか？以下に、文字化けの結果、ソースコードをupしますので、ご教授よろしくお願いいたします。

#include<stdio.h>

#include<stdlib.h>

#include<conio.h>

#include<string.h>


void addtask();

void viewtask();

void edittask();

void deletetask();

struct task

{

    char time[8];

    char name[50];

    char status[100];

    char category[100];

    char note[2000];

} ;

int main()

{

    int ch;

    printf("\n\n\t---------------------------------\n");



    printf("\t---------------------------------");

    while(1)

    {

        printf("\n\n\t\tMain Menu:");

        printf("\n\n\tAdd Task\t[1]");

        printf("\n\tView Task\t[2]");

        printf("\n\tEdit Task\t[3]");

        printf("\n\tDelete Task\t[4]");

        printf("\n\tExit\t\t[5]");

        printf("\n\n\tEnter your choice:");

        scanf("%d",&ch);

        switch(ch)

        {

        case 1:

            addtask();

            break;

        case 2:

            viewtask();

            break;

        case 3:

            edittask();

            break;

        case 4:

            deletetask();

            break;

        case 5:

            printf("\n\n\t\tThank you for using Task manager!!^^");

            getch();

            exit(0);

        default:

            printf("\nYou entered wrong choice...");

            printf("\nPress any key to try again");

            getch();

            break;

        }

        system("cls");

    }

    return 0;

}

void addtask( )

{

    system("cls");

    FILE *fp ;

    char another = 'Y' ,time[10];

    struct task e ;

    char filename[15];

    int choice;

    printf("\n\n\t\t---------------------------------\n");

    printf("\t\t* Welcome to Add Task Menu! *");

    printf("\n\t\t---------------------------------\n\n");

    printf("Enter Task due date:[yyyy-mm-dd]:");

    fflush(stdin);

    gets(filename);

    fp = fopen (filename, "ab+" ) ;

    if ( fp == NULL )

    {

        fp=fopen(filename,"wb+");

        if(fp==NULL)

        {

            printf("\nSystem Error...");

            printf("\nPress any key to exit");

            getch();

            return ;

        }

    }

    while ( another == 'Y'|| another=='y' )

    {

        choice=0;

        fflush(stdin);

        printf ( "\n\tEnter Task due date again:[yyyy-mm-dd]:");

        scanf("%s",time);

        rewind(fp);

        while(fread(&e,sizeof(e),1,fp)==1)

        {

            if(strcmp(e.time,time)==0)

            {

                printf("\n\tThe Task already exists.\n");

                choice=1;

            }

        }

        if(choice==0)

        {

            strcpy(e.time,time);

            printf("\tEnter Task name:");

            fflush(stdin);

            gets(e.name);

            fflush(stdin);

            printf("\tEnter status:");

            gets(e.status);

            fflush(stdin);

            printf("\tEnter category:");

            gets(e.category);

            fflush(stdin);

            printf("\tNote:");

            gets(e.note);

            fwrite ( &e, sizeof ( e ), 1, fp ) ;

            printf("\nYour Task is just added!\n");

        }

        printf ( "\n\tWant to add another Task?(Y/N) " ) ;

        fflush ( stdin ) ;

        another = getchar( ) ;

    }

    fclose ( fp ) ;

    printf("\n\n\tPress any key to exit...");

    getch();

}

Comments

[tatsu99]

その文字化けするテキストの文字コードは何でしょうか。
OSは何でしょうか？

[episteme]

> fwrite ( &e, sizeof (e), 1, fp ) ;

テキストファイルにバイナリを書き込んでいませんか?

[yuto_jake]

tatsu99さん、コメントありがとうございます。文字コードを判別したとこと、
Unicode (UTF-8) - utf-8
判定詳細
-候補1-
繁体字中国語 (hkscs) - Big5-HKSCS
-候補2-
日本語 (シフト JIS) - shift_jis
　　又は下記の類似する文字エンコード
　　・日本語 (シフト JIS) - CP932
　　・日本語 (シフト JIS) - shift_jis2000
　　・日本語 (シフト JIS) - shift_jis2004
　　・日本語 (Mac) - x-mac-japanese

-候補3-
繁体字中国語 (Eten) - x-Chinese-Eten
　　又は下記の類似する文字エンコード
　　・繁体字中国語 (Mac) - x-mac-chinesetrad
となりました。

またOSはWindows11になります。何卒宜しくお願い致します。

[yuto_jake]

epistemeさん、失礼いたしました。
テキストファイルにはバイナリで書き込んでおりません。
何卒宜しくお願い致します。

[tatsu99]

そのテキストファイルは、最初は空の状態から開始するのでしょうか？
そして、このプログラムで、ファイルに書き込み、書き込んだ結果を読み込むのですか？
そのテキストファイルを読み込んで表示している個所が見当たらないのですが、
どこでおこなっていますか？
プログラムの一部だけですが、全部を提示していただけませんでしょうか。
ソースのアップ方法です。
以下のようにすると、きちんとソースが表示されます。

</>をクリックします。
```ここに言語を入力
コード
```

と表示されるので
```①
②
```

①の箇所にCと入力し
②の箇所にソースをコピペして貼り付けます。

そうすると、きちんと表示されます。

[yuto_jake]

tatsu99さん、ご教授ありがとうございます。
おっしゃる通り、テキストファイルは空の状態から始まり、このプログラムからファイルに書き込み、結果が、以上のように文字化けしてしまいました。

また、ソースの書き方についてご教授ありがとうございます。tatsu99さんのアドバイスをもとにコード編集させていただきます。

[tatsu99]

ソース拝見しました。
入力データに漢字を使用しない前提での回答です。（半角英数字、半角記号のみ使用）
task内のtime,name,status,....
ですが、C言語の文字として扱うなら、終端NULLにする必要があります。
但し、そうすると、その文字をstruct毎まるまる書き込むとNULL文字が書き込まれます。
そうなると、メモ帳などのエディタで開いたとき、その個所が正しく表示されなくなります。
それを避けるためには、終端NULLではなく、残りに全てスペースを挿入する必要があります。
viewtask,edittask,deletetaskでどのように扱っているのかが不明ですが、
終端NULLではなく、残りに全てスペースを挿入する方法でaddtaskを書き直してみます。

[tatsu99]

終端NULLではなく、残りに全てスペースを挿入する方法だと後々、無理が発生しそうなので、
残りに全てNULLを挿入するようにします。
そうすると、エディタで表示したとき、（完全ではありませんが）、それなりに読めるようになります。

[yuto_jake]

Tatsuさん、コードの確認および、ご説明ありがとうございます。
NULLやstruct等、各コードについて十分な理解ができておりませんでした。

また、viewtask,edittask,deletetaskに関しましてもNULLを使ってコードを書いておりす。。現在投稿させていただいている、質問では文字数が足らなかったため、省略させて頂きました。

>終端NULLではなく、残りに全てスペースを挿入する方法でaddtaskを書き直してみます。
→ありがとうございます。残り全てにスペースを挿入する方法も存じ上げていなかったため、大変助かります。
何卒よろしくお願い致します。

[tatsu99]

すみません。残りに全てスペースを挿入する方法は、いろいろ、影響がありそうなので、
残りに全てNULLを挿入する方法に変えました。その内容で回答しています。
この方が、あなたの目的に沿っているとおもいます。
本来の目的は、エディタで正しく表示することではなくて、プログラム上で正しく表示し、
追加、表示、削除などが正しく行われることのはずですから。

[yuto_jake]

Tatsuさん、ご連絡及びコードの訂正ありがとうございます。

了解致しました。コード、ご説明のほどありがとうございました。<(_ _)>💦

Answer 1

他にももんだいになりそうなところはありますが、
まず、
memset(&e,0x00,sizeof(e));
この1行を追加してください。
eの中にゴミがあるので、ゴミのまま書き込むと、意味不明な文字が表示されます。
まず、eのごみをきれいにNULLクリアしてから書き込むようにします。

void addtask( )
{
    system("cls");
    FILE *fp ;
    char another = 'Y' ,time[10];
    struct task e ;
    char filename[15];
    int choice;
    printf("\n\n\t\t---------------------------------\n");
    printf("\t\t* Welcome to Add Task Menu! *");
    printf("\n\t\t---------------------------------\n\n");
    printf("Enter Task due date:[yyyy-mm-dd]:");
    fflush(stdin);
    gets(filename);
    fp = fopen (filename, "ab+" ) ;
    if ( fp == NULL )
    {
        fp=fopen(filename,"wb+");
        if(fp==NULL)
        {
            printf("\nSystem Error...");
            printf("\nPress any key to exit");
            getch();
            return ;
        }
    }
    while ( another == 'Y'|| another=='y' )
    {
        choice=0;
        fflush(stdin);
        printf ( "\n\tEnter Task due date again:[yyyy-mm-dd]:");
        scanf("%s",time);
        rewind(fp);
        while(fread(&e,sizeof(e),1,fp)==1)
        {
            if(strcmp(e.time,time)==0)
            {
                printf("\n\tThe Task already exists.\n");
                choice=1;
            }
        }
        if(choice==0)
        {
            memset(&e,0x00,sizeof(e));       //この行を追加
            strcpy(e.time,time);
            printf("\tEnter Task name:");
            fflush(stdin);
            gets(e.name);
            fflush(stdin);
            printf("\tEnter status:");
            gets(e.status);
            fflush(stdin);
            printf("\tEnter category:");
            gets(e.category);
            fflush(stdin);
            printf("\tNote:");
            gets(e.note);
            fwrite ( &e, sizeof ( e ), 1, fp ) ;
            printf("\nYour Task is just added!\n");
        }
        printf ( "\n\tWant to add another Task?(Y/N) " ) ;
        fflush ( stdin ) ;
        another = getchar( ) ;
    }
    fclose ( fp ) ;
    printf("\n\n\tPress any key to exit...");
    getch();
}

Comments

[yuto_jake]

Tatsu99さん、ご回答、ご説明本当にありがとうございます。どこを改善すれば良いか分からなかったので、とても助かりました。
tatsu99さんにご教授頂いたコードで1度実行させていただきます。