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

防止参数命名冲突导致的内存错误：C++代码实践

最新推荐文章于 2018-10-03 02:22:48 发布

转载最新推荐文章于 2018-10-03 02:22:48 发布 · 161 阅读

1 ·

CC 4.0 BY-SA版权

原文链接：http://www.cnblogs.com/Kelvinshere/archive/2013/03/02/2940117.html

文章标签：

#lua

本文详细阐述了在C++编程中，函数参数命名与类成员同名可能导致的内存错误问题，并通过实例展示了如何正确命名参数以避免此类问题的发生，确保代码稳定性和安全性。

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

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

转载于:https://www.cnblogs.com/Kelvinshere/archive/2013/03/02/2940117.html