CString.Right的问题，真要把我给气疯了，大家帮忙看看。

[cmoth]

程序如下，就是把IDC_FILENAME所指的文件中结尾为;号的行删掉，

void CGetValClsDlg::OnOK()
{
CString sLineContent;
TCHAR szPathOrig[_MAX_PATH], szPathClone[_MAX_PATH];

GetDlgItemText(IDC_FILENAME,szPathOrig,_MAX_PATH);
ofstream ofile(szPathOrig,ios::app);
GetTempPath(_MAX_PATH, szPathClone);
GetTempFileName(szPathClone, __TEXT("gzz"), 0, szPathClone);
if (0==CopyFile(szPathOrig, szPathClone, FALSE))
return;
ifstream tempfile(szPathClone);
while (0==tempfile.eof())
{
tempfile.getline ((LPSTR)(LPCTSTR)sLineContent,_MAX_PATH);
sLineContent.TrimRight(" ");
if (sLineContent.Right(1)) //<-----------------------
{
ofile <<sLineContent<<"\n";
}
}
MessageBox("fffffff");

}

现在的问题是打标注一行sLineContent.Right(1)返回值就是等于sLineContent，例如如果sLineContent是"Window;",那么sLineContent.Right(1)就是"Window;",这到底是怎么回事？哎，我现在都快被这个玩艺给气爆了！各位帮忙看看是什么地方的问题

[xunknown]

同意XLB2000()的说法！
cmoth (容量够大) 的错误在于哪里是一目了然了的！

[sclzmbie]

你要判断字符串是否为空，可以用CString::IsEmpty(),或者CString::GetAt(length)。在DOS时代常常自己找结尾的0来判断字符串，现在有很多可以用的类了。另外你可以继承CString，然后写一些你自己能用得上的成员函数。

关于CString::Right(),你仔细看看MSDN吧！

[cmoth]

XLB2000() ( )

#include <afx.h>
#include <iostream.h>

void main()
{
CString s("jgkghkhk ");
s.TrimRight (" ");
cout <<s<<endl;
cout <<s.Right(1)<<endl;
if (s.Right (1)!="k")
cout <<"gkhkhk"<<endl;
}

以上s.Right(1)返回的指就是对的，另外我是要删除行结尾是；号的行，你改的好像并不是我要求的，请帮忙再看看。主要是什么原因会导致Right出错？

[cmoth]

对不住，才发现标注行错了，应该是这样：
if (sLineContent.Right(1)!=";")
请大家再看看

[ydogg]

to 楼上兄弟
你刚才贴的例字当然是正确的
s.Right (1)!="k" 这是判断的返回值而非s本身

Right操作并不修改本身，只是返回一个字符串而已

[cmoth]

ydogg(灰毛兔频频) ( )
对啊！但为什么在最上的那个程序里
if (sLineContent.Right(1)!=";")
语句sLineContent.Right(1)返回值就不对呢？哎.........yum

[cmoth]

#include <afx.h>
#include <iostream.h>
#include <fstream.h>

void main(int argc,char *argv[])
{
CString sLineContent;
CString sss;
TCHAR szPathOrig[_MAX_PATH], szPathClone[_MAX_PATH];
if (argc==2)
{
strcpy(szPathOrig,argv[1]);
ofstream ofile(szPathOrig,ios::app);
GetTempPath(_MAX_PATH, szPathClone);
GetTempFileName(szPathClone, __TEXT("gzz"), 0, szPathClone);
if (0==CopyFile(szPathOrig, szPathClone, FALSE))
return;
ifstream tempfile(szPathClone);
while (0==tempfile.eof())
{
tempfile.getline ((LPSTR)(LPCTSTR)sLineContent,_MAX_PATH);
sLineContent.TrimRight(" ");

cout <<"sLineContent:\"" <<sLineContent<<"\""<<endl;
cout <<"sLineContent.Right(1):\"" <<sLineContent.Right(1)<<"\""<<endl;

if (sLineContent.Right(1)!=";")
{
ofile <<sLineContent<<"\n";
}
}
}


}
可以试试这段程序，结果就是不对啊！

[shenleav]

这样试一下:
if(sLineContentRight(1)!=_T(";")
{
....
....
}

[shenleav]

是这样:
if(sLineContentRight(1)!=_T(";"))
{
...
}

[ahao]

// tempfile.getline ((LPSTR)(LPCTSTR)sLineContent,_MAX_PATH);
改成
char buf[1024];
tempfile.getline (buf,1024);
sLineContent = buf;
即可

象tempfile.getline ((LPSTR)(LPCTSTR)sLineContent,_MAX_PATH);
这样的用法根本就是错误的,你跟进源码看看就明白了

[cmoth]

ahao(天·狼·星星)

确实，谢谢你了，hoho

[xlb2000]

sLineContent.TrimRight();
sLineContent=sLineContent.Right(1);
if (sLineContent) //<-----------------------
{
ofile <<sLineContent<<"\n";
}