1条回答 默认 最新
- _GX_ 2022-02-09 00:21关注
operator+()函数里调用strcat()函数把p1对象的字符串拷贝到p对象字符串结尾,导致p对象的字符串越界修改破坏堆数据。
你应该调用时strcpy函数,把p和p1的字符串都拷贝到t_str里,不过这里t_str有内存泄漏风险(如果调用函数没有释放返回的指针的话,比如在你的main函数里直接赋给str3了,所以t_str的内存没有被释放),你可以改为如下形式。这样实现比较低效,因为字符串拷贝了很多次。如果要提高效率,可以考虑如下一些优化,比如添加move构造函数/赋值运算符重载,实现copy-on-write语义等等friend MyString operator + (const MyString &p1, const MyString &p2) { int num = strlen(p1.m_str) + strlen(p2.m_str); char *t_str = new char[num+1]; strcpy(t_str, p1.m_str); strcat(t_str, p2.m_str); MyString str(t_str); delete []t_str; return str; }
本回答被题主选为最佳回答 , 对您是否有帮助呢?解决 1无用
悬赏问题
- ¥15 unity第一人称射击小游戏,有demo,在原脚本的基础上进行修改以达到要求
- ¥15 买了个传感器,根据商家发的代码和步骤使用但是代码报错了不会改,有没有人可以看看
- ¥15 关于#Java#的问题,如何解决?
- ¥15 加热介质是液体,换热器壳侧导热系数和总的导热系数怎么算
- ¥100 嵌入式系统基于PIC16F882和热敏电阻的数字温度计
- ¥15 cmd cl 0x000007b
- ¥20 BAPI_PR_CHANGE how to add account assignment information for service line
- ¥500 火焰左右视图、视差(基于双目相机)
- ¥100 set_link_state
- ¥15 虚幻5 UE美术毛发渲染