C / C++ / MFC

void CTimer::OnTimer(UINT_PTR nIDEvent)
{
	// TODO: Add your message handler code here and/or call default
OnBnClickeTimer();
	CDialog::OnTimer(nIDEvent);
}

void CTimer::OnStartTimer()
{
	SetTimer(2,10000, 0);
}

void CTimer::OnStopTimer()
{
	  KillTimer (0);
}

BOOL CTimer::OnInitDialog()
{
OnBnClickeTimer();

}

void CTimer::OnBnClickedTimer()
{
	
	
	
		
		check:
		for(;aq<10)
		{
		
			
		finalString=a1[aq];
				
		char * szMyString = (char *)(LPCTSTR)finalString;	
		
		
		CString string = _T(szMyString);
		CString DirPath;
		

		CString string1=string;
		string1 +='\0';
		
		
		int nLen = string1.GetLength();

		LPCSTR lpszBuf = string1.GetBuffer(nLen);

		string1.ReleaseBuffer();
	
	
		LPCSTR pF=lpszBuf;
		
		LPCSTR pT="c:\\Backup";
		
		
		if (!fo.Copy(pF,pT));
		aq++;
		}
		OnStopTimer();
}

• Please check the posted code (for instance use < to escape the symbol < when required).
• Why are you calling directly the timer function inside OnInitDialog (instead of starting the timer? You may actually do it, but usually you haven't...)?
• Since you're calling directly (form OnInitDialog) your timer function (OnBnClickedTimer) it doesn't make sense the OnStopTimer call in the latter.

try
{
   fstream file;
   vector<lpwin32_find_dataw> files;
   LPCTSTR namepointer;
   HANDLE handle;
   string line = "";
   int i = 0;
   int size;
   char directory [] = "C:\\Folder";
   char *stringpointer = directory ;
	
   files.assign(1, LPWIN32_FIND_DATAW());
   handle = FindFirstFile(namepointer, files[i]);
		
   while (handle != INVALID_HANDLE_VALUE)
   {
      FindNextFile(handle, files[i]);
      files.assign(1, LPWIN32_FIND_DATAW());
      i++;
   }	
   size = files.size();

   cout << files.size() << endl;
   for (int k = 0; k < size; k++)
   {
      cout << "Reading File:" << files[k]->cFileName << endl;
   }
   getchar();
   return 0;
}
catch (exception ex)
{
   cout << "An error has occurred." << endl;
   cout << ex.what() << endl;
}</lpwin32_find_dataw>

Megidolaon wrote:

...but still throws a System.AccessViolationException...

Megidolaon wrote:

handle = FindFirstFile(namepointer, files[i]);

DavidCrow wrote:

Why do you have a vector of WIN32_FIND_DATA structures? One is sufficient.

namepointer = stringpointer;

Megidolaon wrote:

FindFirstFile and FindNextFile seemed to return only as single file, so I assumed LPWIN32_FIND_DATAW represented a single file.

vector<string> files;
WIN32_FIND_DATA findData;
BOOL bFound = TRUE;
 
HANDLE handle = FindFirstFile("C:\\*.*", &findData);		   
if (handle != INVALID_HANDLE_VALUE)
{
    while (bFound)
    {
        files.push_back(findData.cFileName);
        bFound = FindNextFile(handle, &findData);      
    }	   
}
 
for (int k = 0; k < files.size(); k++)   
    cout << "Reading File:" << files[k].c_str() << endl;   

Megidolaon wrote:

Thanks, though this code stil throws 2 errors.

Megidolaon wrote:

It doesn't display anything...

vector<wstring> files;
	WIN32_FIND_DATA findData;
	BOOL bFound = TRUE;
	 
	HANDLE handle = FindFirstFile(L"C:\Dokumente und Einstellungen\Student\Eigene Dateien", &findData);		   
	if (handle != INVALID_HANDLE_VALUE)
	{
		while (bFound)
		{
			files.push_back(findData.cFileName);
			bFound = FindNextFile(handle, &findData);      
		}	   
	}
	 
	for (int k = 0; k < files.size(); k++)   
	{
		cout << "Reading File:" << files[k].c_str() << endl;   
	}

getchar();</wstring>

Megidolaon wrote:

HANDLE handle = FindFirstFile(L"C:\Dokumente und Einstellungen\Student\Eigene Dateien", &findData);

Megidolaon wrote:

I have double backslashes, codeproject didn't display the code properly.

Megidolaon wrote:

I get that error in both cases.

Megidolaon wrote:

I cannot check the return value of FindFirstFile...

vector<string> files;	
WIN32_FIND_DATA findData;	
BOOL bFound = TRUE;	 	
 
HANDLE handle = FindFirstFile(L"C:\\Dokumente und Einstellungen\\Student\\Eigene Dateien\\*.*", &findData);		   	
if (handle != INVALID_HANDLE_VALUE)	
{		
    while (bFound)		
    {			
        files.push_back(findData.cFileName);			
        bFound = FindNextFile(handle, &findData);      		
    }	   	
 
    for (int k = 0; k < files.size(); k++)   	
        cout << "Reading File:" << files[k].c_str() << endl;   	
}
else
    cout << GetLastError() << endl;;
 
getchar();

vector<wstring> files;
	WIN32_FIND_DATA findData;
	BOOL bFound = TRUE;
	HANDLE handle;
	string name ="";
	wofstream myfile;
	wstring line;

	handle = FindFirstFile(L"C:\\Dokumente und Einstellungen\\Student\\Eigene Dateien\\*.mp", &findData);		   
	
	if (handle != INVALID_HANDLE_VALUE)
	{
		while (bFound)
		{
			files.push_back(findData.cFileName);
			bFound = FindNextFile(handle, &findData);      
		}	   
	}
	else
	{
		cout << GetLastError() << endl;
	}

	for (int k = 0; k < files.size(); k++)   
	{
		
         myfile.open(files.at(k));
		
	    while (myfile.eof() == false)
	    {
		getline(myfile, line);
		wcout << line << endl;
	    }
	    myfile.close();
	}</wstring>