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

2008-01-14 10:29:46 233 阅读 AI 写同款 · GEO 优化AI 选题数据监测一键分发 ›

MFC自定义弹出式菜单实战：从CMenu类到TrackPopupMenu高级应用弹出式菜单作为经典的桌面软件交互元素，其核心原理是通过资源定义与动态加载实现按需触发的快捷操作。在Windows编程中，菜单资源管理与消息映射机制构成了其技术基础，CMenu类作为菜单控制器，通过LoadMenu加载资源、GetSubMenu获取子菜单、TrackPopupMenu控制显示位置与行为。这种设计模式在嵌入式上位机、工控软件等对交互效率要求高的场景中价值显著，能够提升操作的专业性与用户体验。本文以MFC框架为例，深入解析如何为CListCtrl等控件实现智能右键菜单，涵盖坐标转换、动态菜单项更新阅读详情

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

他在资源中定义了一个字符串 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让我有些整不明白,各位能帮我解释一下吗,
先谢过了,我是知其然,不知其所以然,知道修改,但不知道为什么,呵呵.

...全文

不止是加粗变色：用MFC RichEdit控件打造一个简易Markdown预览器（附源码）但对于Windows平台开发者而言，如何在传统MFC应用中集成Markdown支持却鲜有系统性的解决方案。本文将展示如何利用MFC的RichEdit控件构建一个轻量级Markdown渲染引擎，实现从Markdown语法到富文本的实时转换。通过这个项目，开发者可以深入理解Windows富文本处理机制，同时获得一个可直接集成到现有MFC项目中的Markdown解决方案。最终的实现效果证明，即使是传统的MFC框架，经过精心设计也能完美支持现代标记语言的渲染需求。阅读详情

fish_kun

VC/MFC 16,550 成员 421,606 内容

+ 关注 + 加入社区

切换为时间正序

cospolo 2008-01-17

打赏
举报

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

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

fish_kun 2008-01-15

打赏
举报

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

iyranly 2008-01-15

打赏
举报

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

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

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

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

lsyxp17 2008-01-15

打赏
举报

楼上说的好有道理哦

junheng 2008-01-15

打赏
举报

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

fish_kun 2008-01-14