CHARからchar*型に変換したことが原因なのかわからないが。全角文字がchar*型に代入されず、英数字？だけになる原因と対処方法

Question

提示コード最下部のfor文内で確認と書かれたprintf();関数で出力されるはずの文章がなぜかsample.txtになってしまいますこれはなぜなのでしょうか？上のコードのファイルを探す**while()文やその上のif()文ないでは普通に表示される(sampl - コピー（２）.txt)**のですがなぜなのでしょうか？

int main()
{
    //ワイルドカード
    const char* extension = "*.txt";
   // const char* extension = "*.lib";

    LPSTR FullPath;//フルパス
    FullPath = (LPSTR)malloc(sizeof(LPSTR) * MAX_PATH);
    DWORD e = GetModuleFileNameA(NULL, FullPath ,MAX_PATH); 

    char drive[MAX_PATH * 2],dir[MAX_PATH * 3],fname[MAX_PATH * 2],ext[MAX_PATH * 2];			    
	_splitpath_s((char*)FullPath,drive,
        size_t(MAX_PATH * 2),dir,size_t(MAX_PATH * 3),
                            fname,size_t(MAX_PATH * 2),ext,size_t(MAX_PATH * 2));//パス名を構成要素に分解します

//	printf("\n完全パス : %s\n",(char*)FullPath);
//	printf("ドライブ : %s\n",drive);
//	printf("ディレクトリ パス : %s\n",dir);
//	printf("ベース ファイル名 (拡張子なし) : %s\n",fname);
//	printf("ファイル名の拡張子 : %s\n",ext);

    //ディレクトリにワイルドカードを付与(パスを作成完了)
    strcat_s(dir,(sizeof(dir)),extension);

  //  printf("%s\n",dir);
    if(e == ERROR_INSUFFICIENT_BUFFER)
    {        
       ErrorConf(e);//エラー表示
    }else
    {
   //    printf("正常です。\n");
    }

    printf("ファイルをリストアップ\n\n");

    FILE *file = NULL;

    if(file == NULL)
    {
        printf("あああ");
    }
    WIN32_FIND_DATAA data;
    LPWIN32_FIND_DATAA Next_data = new WIN32_FIND_DATAA();

    LPCSTR p = dir;
    HANDLE h = FindFirstFileA(p,&data);
    printf("path: %s\n",p);

    int next = 0;
    char *list[100];
//////////////////////////////////////////////////////////////////////////////////////////////
    if(data.cFileName != "list_up.txt")
    {        
        strcat_s(data.cFileName,sizeof(data.cFileName),"\n");
        list[next] = (char*)malloc(sizeof(data.cFileName));
        list[next] = data.cFileName;
        printf("%s",list[ next ]);

        next++;        
    }

    //二つ目以降
    while(FindNextFileA(h,Next_data) != 0)
    {
        strcat_s(Next_data->cFileName,sizeof(Next_data->cFileName),"\n");
        list[next] = (char*)malloc(sizeof(Next_data->cFileName));
        list[next] = Next_data->cFileName;
        printf("%s",list[ next ]);        
        next++;
    }
    --next;
    
    fopen_s(&file,"list_up.txt","w");

    for(int j = 0; j < next; j++)
    {
        printf("確認: %s",list[j]);
    }
///////////////////////////////////////////////////////////////////////////////////////////////
    fclose(file);
    FindClose(h);   
    printf("検索終了\n");
    
    int _c_ = getchar();
	return 0;
}

Comments

[退会済みユーザー]

https://teratail.com/questions/287976 ← ?

[退会済みユーザー]

すいません前回に質問は質問内容を編集してくださいとの指摘がありましたが
もう元の質問内容は覚えていないため修正できないため運営に削除リクエストをしました。そこで新しい質問を作成しました。

[退会済みユーザー]

https://teratail.com/questions/287409
こちらは解決したのでしょうか？
解決してるのであれば、次々と質問する前に、過去に質問した事はクローズしてください。

Answer 1

FullPath = (LPSTR)malloc(sizeof(LPSTR) * MAX_PATH);

LPSTRはchar*ですので無駄に4～8倍ほどのサイズを確保しています。

DWORD e = GetModuleFileNameA(NULL, FullPath ,MAX_PATH); 
if(e == ERROR_INSUFFICIENT_BUFFER)

前回私がソースで伝えたつもりでしたが伝わっていなかったのでハッキリ申すと
まったくもって間違いです。
リファレンス読み直すか、122文字書き込むようにファイル名・ディレクトリ名を調整して挙動を確かめるとよいでしょう。

LPWIN32_FIND_DATAA Next_data = new WIN32_FIND_DATAA();

mallocとnewを安易に混ぜると解放時に混乱することがありますのでおすすめしません。
また、この変数は特に必要ないですね。

if(data.cFileName != "list_up.txt")

文字列へのポインタ同士を比較しても意味はないでしょう。

sizeof(data.cFileName)

strlen等を用いてください

list[next] = (char*)malloc(sizeof(Next_data->cFileName));
list[next] = Next_data->cFileName;

ポインタサイズのメモリを確保して変数に格納後、それを消してNext_data->cFileNameを格納しています。
このせいで、すべてのlist[j]が最後にヒットしたファイル名になっています。

Comments

[退会済みユーザー]

指摘通り編集してみましたがどうやらprintf();あああああ と いいいいいの間でアクセスエラーが発生していようなのですがこれはどうすればいいのでしょうか？char型の確保数が多いのはとりえずです。

//ワイルドカード
const char* extension = "*.txt";
// const char* extension = "*.lib";

LPSTR FullPath;//フルパス
FullPath = NULL;
DWORD e = GetModuleFileNameA(NULL, FullPath ,(DWORD)FullPath);
printf(" ああああああああああああああああああ\n");
char drive[MAX_PATH * 2],dir[MAX_PATH * 2],fname[MAX_PATH * 2],ext[MAX_PATH * 2];
_splitpath_s(FullPath,drive,
size_t(MAX_PATH * 2),dir,size_t(MAX_PATH * 2),
fname,size_t(MAX_PATH * 2),ext,size_t(MAX_PATH * 2));//パス名を構成要素に分解します
printf(" いいいいいいいいいいいいいいいいいいいいいいいいい\n");

質問ですが　書き込んだテキストファイルを画面に表示させる方法はありますか？"エキスプローラーで.txtをクリックして画面を開く"この動作を自動で最後に行いたいです。

[asm]

関連付けされたアプリケーションで開くだけならばShellExecuteにて可能だったはずです。

[asm]

> DWORD e = GetModuleFileNameA(NULL, FullPath ,(DWORD)FullPath);
(DWORD)FullPathではなくMAX_PATH

[退会済みユーザー]

質問ですが終了処理ですがdeleteしてnullptrにしましたがこれは本当に正しいのでしょうか？
一応確認したいので質問しました。

delete(FullPath);
FullPath = nullptr;

delete(list);
*list = nullptr;

fclose(file);
FindClose(h);
printf("検索終了\n");

[asm]

delete[] FullPath; FullPath = nullptr;
for(int i = 0; i < next; i++){ delete[] list[i]; list[i] = nullptr; }
fclose(file);
FindClose(h);
printf("検索終了\n");

FullPath = nullptr;およびlist[i] = nullptr; についてはやらなくても別に問題ありません。

Answer 2

次のコードで、全角文字のファイル名が表示されませんか？

#include <windows.h>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int main()
{
    const char* extension = "*.txt";  //ワイルドカード

    char FullPath[MAX_PATH]; //フルパス
    DWORD e = GetModuleFileNameA(NULL, FullPath ,MAX_PATH); 
    if (e == ERROR_INSUFFICIENT_BUFFER) {
		puts("GetModuleFileNmaeA failed: ERROR_INSUFFICIENT_BUFFER");
		return 1;
    }
    char drive[4], dir[MAX_PATH], fname[MAX_PATH], ext[8];                
    _splitpath_s(FullPath, drive, 4, dir, MAX_PATH, fname, MAX_PATH, ext, 8);
    //ディレクトリにワイルドカードを付与(パスを作成完了)
    strcat_s(dir, sizeof(dir), extension);

    puts("ファイルをリストアップ\n");

    printf("path: %s\n", dir);
    WIN32_FIND_DATAA data;
    HANDLE h = FindFirstFileA(dir, &data);
	if (h == INVALID_HANDLE_VALUE) {
		puts("FineFirstFileA failed: INVALID_HANDLE_VALUE");
		return 1;
	}

    int next = 0;
    char *list[100];
	do {
		if (strcmp(data.cFileName, "list_up.txt")) {        
			if (next >= 100) {
				puts("too many files");
				break;
			}
			size_t len = strlen(data.cFileName) + 1;
			list[next] = (char*)malloc(len);
			strcpy_s(list[next], len, data.cFileName);
        	next++;        
		}
    } while (FindNextFileA(h, &data));
    FindClose(h);   

    FILE *file = NULL;
    if (fopen_s(&file, "list_up.txt", "w")) {
		puts("fopen_s('list_up.txt', 'w') failed");
		return 1;
	}
    for (int j = 0; j < next; j++) fprintf(file, "%s\n", list[j]);
    fclose(file);

    for (int j = 0; j < next; j++) free(list[j]);
    puts("検索終了");

	puts("  list_up.txt 確認");
    if (fopen_s(&file, "list_up.txt", "r")) {
		puts("fopen_s('list_up.txt', 'r') failed");
		return 1;
	}
	while (fgets(fname, sizeof(fname), file)) printf("%s", fname);
    fclose(file);

    getchar();
    return 0;
}

Answer 3

LPWIN32_FIND_DATAA Next_data = new WIN32_FIND_DATAA();

これは一回しか実行していませんが、

//二つ目以降
    while(FindNextFileA(h,Next_data) != 0)
    {
        strcat_s(Next_data->cFileName,sizeof(Next_data->cFileName),"\n");
        list[next] = (char*)malloc(sizeof(Next_data->cFileName));
        list[next] = Next_data->cFileName;
        printf("%s",list[ next ]);        
        next++;
    }

この処理で使いまわしているため、list[next]にはlist[0]にdata.cFileNameの先頭アドレスが入っている以外は全て同じNext_data->cFileNameの先頭アドレスが入っています。

Next_dataはwhile文の中の処理で都度更新されており、ループを抜けた時にはNetx_data->cFileNameには最後に取得した"sample.txt"が入っているので、それしか表示できません。

本来なら文字列をコピーしたかったのだと思いますが、c言語では代入(=)では文字列のコピーはできません。strcpy_sなどを使ってファイル名をコピーすれば意図通りになると思います。

Answer 4

list[next] = (char*)malloc(sizeof(data.cFileName));
list[next] = data.cFileName;

  list[next] = (char*)malloc(strlen(data.cFileName)+1);
  strcpy(list[next], data.cFileName);

じゃなくて?

※ 使い終わったらちゃんと返そう(free)ね。