崩溃了~关于遍历硬盘下所有文件的问题

[thisisll]

大家应该对下面遍历文件的函数很熟悉

void CSearchThread::LoopFiles(CString sPath)
{
CFileFind oFileFind;
BOOL bFile;
CString sDir = sPath;

if ( sDir.Right(1) != '\\' )
sDir += '\\';
sDir += "*.*";
bFile=oFileFind.FindFile( sDir );
while( bFile )
{
bFile=oFileFind.FindNextFile();
if( oFileFind.IsDirectory()&&!oFileFind.IsDots())//文件夹
{
CString sFilePath,sFileName;
sFilePath=oFileFind.GetFilePath();
LoopFiles( sFilePath );
}
if ( !oFileFind.IsDirectory() && !oFileFind.IsDots())//文件
{

……………………
}
}
}

给了sPath就可以遍历
如果sPath是这样c:\abc.exe
通过这段
if ( sDir.Right(1) != '\\' )
sDir += '\\';
sDir += "*.*";
sPath 应该成了c:\abc.exe\*.*

可是我跟这段的时候
他执行了那3行
可sPath还是c:\abc.exe

这个函数还是可以遍历路径下所有文件的

可是如果sPath是c:
那么他还是不加后面的\*.*
这样就没法遍历c盘下的文件了
奇怪的很

希望有经验的朋友们能给我指点指点

[thisisll]

if ( sDir.Right(1) != '\\' )
sDir += '\\';
sDir += "*.*";
是我试的时候改的
我看别的资料上
应该是
if ( sDir.Right(1) != "\\" )
sDir += "\\";
sDir += "*.*";

反正两种方法都不可以

[luoxiangdong]

void ConvDir(int type,char *fname)
{
char searchstr[2048];
HANDLE handle;
WIN32_FIND_DATA fileinfo;

sprintf(searchstr,"%s\\*.*",fname);
handle=FindFirstFile(searchstr,&fileinfo);
while(INVALID_HANDLE_VALUE!=handle)
{
if((fileinfo.dwFileAttributes & FILE_ATTRIBUTE_DIRECTORY) == FILE_ATTRIBUTE_DIRECTORY)
{
if(strcmp(fileinfo.cFileName,".")!=0 && strcmp(fileinfo.cFileName,"..")!=0)
{
char fullpath[2048];
sprintf(fullpath,"%s\\%s",fname,fileinfo.cFileName);

ConvDir(type,fullpath);
}
}else
{

处理单个文件：
ConvOneFile(type,fname,fileinfo.cFileName);

}

if(!FindNextFile(handle,&fileinfo))
{
FindClose(handle);
handle=INVALID_HANDLE_VALUE;
};

}



}

[luoxiangdong]

我这里的int type是其他用途

[thisisll]

我是想让大家帮我看看我的函数的问题

[martingod]

你的参数sPath是这样写的吗？？

sPath＝"C:\\abc.exe";

可不能写成 sPath＝"C:\abc.exe";

[thisisll]

就是sPath＝"C:\\abc.exe";这样的

[luoxiangdong]

你的代码看起来似乎没有什么问题

[xyw2278]

同志，sPath会变吗？
你变得是sDir!

[thisisll]

是我没说清楚
我就要变sDir

[thisisll]

哈哈!
终于发现了
原来是CString的问题
具体什么问题我还是没搞明白

我解决的方法是用LPCTSTR倒了一下
CString sTmp;
LPCTSTR strTmp;
strTmp=sDir;
sTmp=strTmp;
if(sTmp.Right(1)==":")
{
sTmp+="\\";
sDir=sTmp;
}
希望大家能继续讨论
能让我把这个CString的问题搞明白

[StarsWu]

可以在程序开始用_access判断输入的是文件夹还是文件，然后分别处理。

[vcmute]

测试没问题的,看看调用处代码
void LoopFiles(CString sPath)
{
CFile FindoFileFind;
BOOL bFile;
CString sDir = sPath;

if ( sDir.Right(1) != '\\' )
sDir += '\\';
sDir += "*.*";
}

[koko1998]

没什么问题啊.
如果你的参数传入的是"D:\\a.txt"的话，也不会有问题的啊，D:\\a.txt\\*.* 也是什么也找不到，不过会浪费点时间.
当然也也可以加些if判断

void CAboutDlg::LoopFiles(CString sPath)
{
CFileFind oFileFind;
BOOL bFile;
CString sDir = sPath;

if ( sDir.Right(1) != '\\' ) sDir += '\\';
sDir += "*.*";
bFile=oFileFind.FindFile( sDir );
while( bFile )
{
bFile=oFileFind.FindNextFile();
if( oFileFind.IsDirectory()&&!oFileFind.IsDots())//文件夹
{
CString sFilePath,sFileName;
sFilePath=oFileFind.GetFilePath();
LoopFiles( sFilePath );
}
if ( !oFileFind.IsDirectory() && !oFileFind.IsDots())//文件
{
;
}
}
}