Can't find all the files (GETTING FUSTERATED!!!)

I have been going over this code for a day now with no prevail...

Run this code, it says that there is less files in C:\WINNT\ than there really is. WHY!?!?

Please help!
August.

Code:

#define _WIN32_WINNT 0x0501
#include <windows.h>
#include <string.h>
#include <stdio.h>
#include <tchar.h>
int FindFilesNow(HWND,char[],bool);
int FindFilesNow_(bool,char[]);
 
int end(int i)
{
   return i;
}
 
int count_ion = 0;
 
int FindFilesNow_(bool isFile, char fileName[])
{
  count_ion++;
  return 0;
}
 
char title[] = "dhhg";
int main(int argc, char *argv[])
{
  FindFilesNow(0,"C:\\WINNT\\",false);
  return 0;
}
int FindFilesNow(HWND hwnd, char path[], bool DisplayErrors)
{
   WIN32_FIND_DATA FindFileData;
   HANDLE hFind = INVALID_HANDLE_VALUE;
   DWORD dwError;
   char DirSpec[MAX_PATH + 1];  // directory specification
   strcpy (DirSpec, path);
   strcat (DirSpec, "\\*");
   
   hFind = FindFirstFile(DirSpec, &FindFileData);
   if (hFind == INVALID_HANDLE_VALUE) 
   {
	  if(DisplayErrors == true){
	  char cab[MAX_PATH + 1];
	  wsprintf(cab,"Invalid file handle. Error is %u\n",GetLastError());
	  MessageBox(hwnd,cab,title,MB_OK|MB_ICONERROR);  }
	  return end(-1);
   } 
   else 
   {
	  int i;
	  for(i = 0; DirSpec[i] != '\0'; i++){
	  if(DirSpec[i] == '*'){
	  DirSpec[i] = '\\' + 0; } }
	  printf(DirSpec);
		 
	  char source[1000];
	  lstrcpy(source,DirSpec);
	  char tempVar1[MAX_PATH];
	  DWORD dwAttrs = 0;
	  
	  lstrcpy(tempVar1,source);
	  lstrcat(tempVar1,FindFileData.cFileName);
	  lstrcat(tempVar1,"\\");
	  if(rename(tempVar1,tempVar1) == 0)
	  { 
		 FindFilesNow_(false,tempVar1);
		 FindFilesNow(hwnd,tempVar1,DisplayErrors);
	  }
	  else
	  {
		 FindFilesNow_(true,tempVar1);
	  }
	  
	  while (FindNextFile(hFind, &FindFileData) != 0) 
	  {	  
		 lstrcpy(tempVar1,source);
		 lstrcat(tempVar1,FindFileData.cFileName);
		 lstrcat(tempVar1,"\\");
		 if(rename(tempVar1,tempVar1) == 0)
		 { 
			 FindFilesNow_(false,tempVar1);
			 FindFilesNow(hwnd,tempVar1,DisplayErrors);
		 }
		 else
		 {
			 FindFilesNow_(true,tempVar1);
		 }
	  }
	
	  dwError = GetLastError();
	  FindClose(hFind);
	  if (dwError != ERROR_NO_MORE_FILES) 
	  {
		 if(DisplayErrors == true){
		 char cab[MAX_PATH + 1];
		 wsprintf(cab,"FindNextFile error. Error is %u\n",dwError);
		 MessageBox(hwnd,cab,title,MB_OK|MB_ICONERROR);  }
		 return end(-1);
	  }
   }
   printf("\n\n\nNumber of files in C:\\WINNT\\: %d",count_ion); 
   return end(0);
}

EDIT: Opps....read the code


Is it a big difference?

Why are you not testing that is a folder/file before recursively calling the FindFilesNow() function?

ie FindFileData.dwFileAttributes==FILE_ATTRIBUTE_DIRE CTORY


What is this supposed to do?

DirSpec[i] = '\\' + 0;

or this?

if(rename(tempVar1,tempVar1) == 0)

> if(rename(tempVar1,tempVar1) == 0)
What the hell is the point of this?

>> > if(rename(tempVar1,tempVar1) == 0)
>> What the hell is the point of this?

If you try to rename a folder with a slash after it to it's own name with a slash after it, it will return 0. But if you try that with a file, is will return -1. Try it, you'll see what I mean.

>> ie FindFileData.dwFileAttributes==FILE_ATTRIBUTE_DIRE CTORY

Now when I replace: "if(rename(tempVar1,tempVar1) == 0)" with "if(FindFileData.dwFileAttributes == FILE_ATTRIBUTE_DIRECTORY)", it locks the program up, and Maxes the CPU out.

More like

Code:

if ( ( FindFileData.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY ) == FILE_ATTRIBUTE_DIRECTORY )

Okay, I try that, it will do somthing now, (now that iv'e added, so that the escape key will quit the looking and looping, but still save the resaults.) but look at one of it's resualts:

E:\\\Documents\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\. \\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\ .\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\ \.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\. \\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\.\\ .\\..\\boot.ini

"boot.ini" is under "E:\" yes, but not under "E:\Documents\". Also why is it making a lot of slashes and dots?

You need to make special cases of the two directory entries "." (aka yourself) and ".." (aka your parent).
Otherwise you keep going up and down on the same spot, just like you're doing here.

It's not like any of this is explained in the FAQ entry or anything...
http://faq.cprogramming.com/cgi-bin/...&id=1044780608

You can lead a horse to water, but a pencil must be lead.

Okay, the examples gave me what I needed to know, thanks for your help, Salem.