关于CString类中format方法的奇怪问题.

[fish_kun]

今天帮同学调试代码,发现了一个很奇怪的问题.

他在资源中定义了一个字符串 IDS_USER_COUNT "Users: %d"

在其函数中,他这样使用:

int count = 0;
......
//count被赋值
......

CString sRes;
sRes.LoadString(IDS_USER_COUNT);
sRes.Format(sRes, count); //当count >= 100时,此处会导至程序crash.

如何将代码改为
sRes.Format(IDS_USER_COUNT, count)或者sRes.Format(_T("Users: %d"), count),
可以正常运行.

这里,
sRes.LoadString(IDS_USER_COUNT);
sRes.Format(sRes, count); //当count >= 100时,此处会导至程序crash.
这种写法肯定不好,但它在count>=100让我有些整不明白,各位能帮我解释一下吗,
先谢过了,我是知其然,不知其所以然,知道修改,但不知道为什么,呵呵.

[zyyoung]

LoadString的函数原型，看一下

[shelliu]

这种用法还是第一次看到，有趣！

[caofusheng]

看看这个吧。也许有用。
http://wflyfox.bokee.com/670276.html

[fish_kun]

关键是为什么count>=100才导致错误呢？
随便说一下，这个程序用的是Unicode编码。

[真相重于对错]

100会替换为把%d 3个字符，所以出错
改为
CString sRes,sFormat;
sFormat.LoadString(IDS_USER_COUNT);
int count;
count = 1000;
sRes.Format(sFormat, count);

[fish_kun]

To hdt:
再说详细一些嘛,我没有完全明白.

[junheng]

原因不在于被格式化的数据，也不在于LoadString有什么问题。而是CString::Format函数的问题。
一般情况下，Format函数不允许将其本身格式化。比如：
CString str="这样是不对的";
str.Format("%s%d",str,100);
这样就会导致错误。
所以，如果要对一个字符串格式化，就应该写成这种格式：
CString strFmt="这样就对了";
CString str;
str.Format("%s%d",strFmt,100);

[lsyxp17]

楼上说的好有道理哦

[iyranly]

严重同意一下7楼的意见！

是的，不能只用一个变量，这中间需要一个变量来进行格式化的中转

CString strFmt="这样就对了";
CString str;
str.Format("%s%d",strFmt,100);

这个代码看着就舒服多了！

[fish_kun]

是啊，他的代码不该那样写，可是，为什么在count<100的时候没有出问题呢？

[cospolo]

CString::Format这个函数用自己来作参数是很危险的，根据语义，它会对"%d"这类的做替换。

sRes.Format(sRes, count);语义变为将内容为“Users: %d”的字符串中的%d用对应整数的字符串替换，而此字符串本身又作为格式参数。会有%d替换的问题出现，当count>=100后，%d会被“100”替换，而此时替换的字符串是“100”，是三个字符组成，而此前的“%d”是两个字符组成，而这种用法本身迭代“%d“替换两次，所以当将"100"或类似超过两个字符组成的字符串拿来替换“%d”时，肯定会有越界访问发生。