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

 最近在写一个任意数据类型调用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 ,编译器会在编译时为你提供一条错误消息,而不会让你的程序在运作中才出现问题。

同样,如果函数声明和定义分开,最好把参数名字写上,可以避免上述疏漏之余,还起到注释的作用。

 

 

 

posted @ 2013-03-02 15:12  KelvinDesus  阅读(459)  评论(0)    收藏  举报