(win32API)visualstudioでデバグ実行するとGetCurrentDirectoryの値がおかしい

Question

##環境
windows10
visualstudio2017
##やってみたこと
visualstudioでdebugモードで実行するとGetCurrentdiretctoryの値がおかしいです。
debugモードにある実行ファイルをほかの新規作成したフォルダで実行すると正常に
起動します。ソリューションファイル名はfile_searchです。

visualstudio上で実行した場合、またはvisualstudioで作成したディレクトリ上で実行すると、

C:Users\programming\source\repos\file_search\file_search\a.txt
↑ソリューション名が連続してしまう
実際は
C:\Users\programming\source\repos\file_search\x64\Debug
のディレクトリ上にfile_search.exeとa.txtがあります。

自分で用意したディレクトリで実行すると
C:Users\programming\source\repos\file_search\a.txt
実際のディレクトリはC:\Users\programming\Pictures\app
のディレクトリ上にfile_search.exeとa.txtがあります。

デバグするときにGetCurrentDirectoryを使用するとディレクトリを用意し、
そこに実行ファイルと目的のファイルを用意しないといけません。
こうなる原因がわかりません。
改善策はございますでしょうか。ご回答お願いします。

#include <windows.h>
#include <tchar.h>
#define aFILE _T("a.txt")
static TCHAR		msgbuf[257];

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
	PSTR lpCmdLine, int nCmdShow) {
	HANDLE hFile;
	HANDLE findHandle;
	TCHAR Path[260];
	TCHAR Path2[260];
	WIN32_FIND_DATA findData;
	TCHAR			*File;

	GetCurrentDirectory(sizeof Path, (LPTSTR)Path);
	_stprintf_s(Path, lstrlen(Path) + 30, _T("\%s\%s"), Path, aFILE);
	MessageBox(NULL, Path, NULL, MB_OK);
	findHandle = FindFirstFile(Path, &findData);
	if (findHandle == INVALID_HANDLE_VALUE) {
		MessageBox(NULL, Path, NULL, MB_OK);

			if (!SearchPath(NULL, aFILE, NULL, sizeof(Path), Path, &File)) {

				_stprintf_s(msgbuf,lstrlen(msgbuf)+30 ,_T("%s was not found."), aFILE);
				MessageBox(NULL, msgbuf, NULL, MB_OK);
					/*return Abort(hWnd, msgbuf);*/
			}
			else
			{
				MessageBox(NULL, _T("error"), NULL, MB_OK);
				
			}
	
	}
	else
	{
		FindClose(findHandle);
		MessageBox(NULL, _T("success"), NULL, MB_OK);
		
	}
	

	return 0;
}

	

#追記1

#include <windows.h>
#include <tchar.h>
#define aFILE _T("a.txt")
static TCHAR msgbuf[257];

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE hPrevInstance,
PSTR lpCmdLine, int nCmdShow) {
HANDLE hFile;
HANDLE findHandle;
TCHAR Path[260];
TCHAR Path2[260];
WIN32_FIND_DATA findData;
TCHAR *File;

GetCurrentDirectory(sizeof Path, (LPTSTR)Path);
_stprintf_s(Path2, lstrlen(Path2) + 30, _T("\%s\%s"), Path, aFILE);
MessageBox(NULL, Path2, NULL, MB_OK);
findHandle = FindFirstFile(Path2, &findData);
if (findHandle == INVALID_HANDLE_VALUE) {
	MessageBox(NULL, Path2, NULL, MB_OK);

		if (!SearchPath(NULL, aFILE, NULL, sizeof(Path), Path, &File)) {

			_stprintf_s(msgbuf,lstrlen(msgbuf)+30 ,_T("%s was not found."), aFILE);
			MessageBox(NULL, msgbuf, NULL, MB_OK);
				/*return Abort(hWnd, msgbuf);*/
		}
		else
		{
			MessageBox(NULL, _T("error"), NULL, MB_OK);
			
		}

}
else
{
	FindClose(findHandle);
	MessageBox(NULL, _T("success"), NULL, MB_OK);
	
}


return 0;

}

Answer 1

visualstudioでdebugモードで実行するとGetCurrentdiretctoryの値がおかしいです。

おかしくはありません。仕様通りです。

C:Users\programming\source\repos\file_search\file_search\a.txt
↑ソリューション名が連続してしまう

ソリューション名が連続しているのではなく、ソリューション名\プロジェクト名です。Visual Studioで新規にプロジェクトを作る際、「ソリューションのディレクトリを作成」にチェックが付いていると既定でソリューション名はプロジェクト名と同じ名前になるので、結果的に同じ名前のディレクトリーが２階層作られることになります。
そして、Visual Studioからプログラムを実行すると、デフォルトの設定ではプロジェクトファイルが置かれているディレクトリーが作業フォルダー(カレントディレクトリー)となります。

プログラム実行時の作業フォルダーはデバッグ設定で変更できます。プロジェクトのプロパティを開いて左のリストで「デバッグ」を選択するとデバッグに関する設定項目が開きます。その中に「作業ディレクトリ」があり、初期値として$(ProjectDir)が設定されているので、それを好きなように変更すれば良いです。exeファイルと同じフォルダーにしたいのであれば、$(OutDir)です。

Comments

[mercurian-teto]

visualstudio上で動作しました。
ありがとうございました。

Answer 2

_stprintf_s(Path, lstrlen(Path) + 30, _T("%s%s"), Path, aFILE);

これがやばいんでは。
おなじPathに入れずに、別のバッファを確保してそこに展開させてみては？

Comments

[mercurian-teto]

回答ありがとうございます。
ご指摘の通り、ちがうbufferである。
型も配列数も同じの変数Path2を入れました。(追記1をご覧ください）
しかし、同じエラーが発生し、さらにはエラーの後に
例外が発生しました。
Run-Time Check Failure #2 -Stack around the variable 'Path2' was corrupted.

[y_waiwai]

> _stprintf_s(Path2, lstrlen(Path2) + 30, _T("\%s\%s"), Path, aFILE);

第２引数はバッファサイズなので、sizeof(Path2) としましょう

[catsforepaw]

この件に関する問題の本質はそこではないので、あまり深みにはまらない方が良いですよ。
確かにパス操作に関してはいまいちな部分もあるので、改善の余地はありますが。

Windows APIには、パス操作用の便利関数が用意されているので、それを利用するのが簡単だと思います。

#include <shlwapi.h>
#pragma comment(lib, "shlwapi.lib")
:
:
//_stprintf_s(Path, lstrlen(Path) + 30, _T("\%s\%s"), Path, aFILE);
PathAppend(Path, aFILE);

[dodox86]

> _stprintf_s(Path2, lstrlen(Path2) + 30, _T("\%s\%s"), Path, aFILE);
> 第２引数はバッファサイズなので、sizeof(Path2) としましょう
バッファサイズと言うよりか「文字数」なので、sizeofではwchar_t の時に不適切となるので、(sizeof(Path2) / sizeof(Path2[0]))とするか、
_countofマクロが使えればそちらを利用すべきかと。ランタイムエラーが起きるのも、格納可能最大サイズの指定間違いによるものかもしれません。※catsforepawさんの深みにはまらない方が良いと言う点で賛成です。