您的位置:首页 > 其它

Bug Found:避免成员函数参数名字与类成员名字相同

2013-03-02 15:12 253 查看
最近在写一个任意数据类型调用lua的模块,在我有点眼困的时候被BUG乘虚而入,差点破坏了我整个工程,幸好我有一种习惯,会用记事本把稳定版本的主要代码保存下来,才让我的代码在修改BUG过程中的突发情况下得以生存。

这次BUG的原因很简单:

#include <iostream>
using namespace std;

inline void Assignment(char* &left_str,const char* right_str){
if( right_str){
left_str = new char[strlen(right_str)+1];
strcpy(left_str,right_str);
}
else
cerr<<"right_str is nil .\n";
};
class Test{
public:
char *id;
Test(){
setID("IDTesting");
}
 
void setID(const char * /*id*/ ){            //参数列表应为(const char *id) ,参数与成员的名字相同会为bug提供藏身之处
Assignment(this->id,id);
        }        //上面函数没有参数名字,编译时没有提示信息,但使用时会使内存出错

void show(){
cout<<id<<'\n';
};
};

int main(){
Test test;
test.show();
getchar();
return 0;
}


 __: 

  函数setID(const char * id )的参数列表命名不应该和类成员id一致,如果百密一疏漏写了参数名字,会使得id赋值失败,id指向一堆无用内存,在以后使用成员id的过程中发生内存错误。

  如果参数的名字和成员的名字不一样,e.g. __id ,编译器会在编译时为你提供一条错误消息,而不会让你的程序在运作中才出现问题。

同样,如果函数声明和定义分开,最好把参数名字写上,可以避免上述疏漏之余,还起到注释的作用。
内容来自用户分享和网络整理,不保证内容的准确性,如有侵权内容,可联系管理员处理 点击这里给我发消息
标签: 
相关文章推荐