C++编程新手容易犯的10种编程错误

Posted 2021-11-11 IT老张

        IT公司每年都会有一定的人员流动，相应地也会招一些应届生补充进来，指导应届生已经成为老员工的必修课了。平日里会我们会经常帮新人排查代码中的问题，在此过程中发现了C++新手容易犯的一些编程错误，在此简单的总结一下，给新人们提供一个参考。

1、有些关键字在cpp文件中多写了

       对于C++类，一些关键字只要写在.h中就好，cpp中就不用再加上了，比如virtual、static等关键字，如果再cpp中多写，编译器会报错。比如如下的虚接口与静态成员变量的定义，只要在头文件中声明就可以了。

class shape
{
    virtual Draw();
    //...
    static int nLevel;
}

2、函数参数的默认值写到函数实现中了

       带有参数默认值的函数，默认值是加在函数声明处的，函数实现处的参数是不需要带上的。为了方便查看代码，在函数实现处的参数中，将默认值注释起来。正确的做法是，头文件中有默认值：

BOOL CreateConf( const CString& strConfName, const BOOL bAudio = FALSE );

在函数实现处的参数中不用添加默认值：

BOOL CreateConf( const CString& strConfName, const BOOL bAudio/* = FALSE*/ );
{
    // ......
}

3、在编写类的时候，在类的结尾处忘记添加";"分号了

       在类的结尾处忘记添加分号，编译会报错，新人们有可能找了半天也没找出引起编译错误的原因。其实很简单，在类的结尾处忘记添加分号了。

class Shape
{
    // ...
};

4、只添加了函数声明，没有函数实现

       在添加类的函数时，只在类的头文件中添加了函数声明，但在cpp中却没有添加函数的实现。如果其他地方调用到该函数，在编译链接的时候会报 unresolved external symbol错误。因为没有实现，所有没有供链接使用的obj文件。

5、cpp文件忘记添加到工程中，导致没有生成供链接使用的obj文件

       在添加C++类时，我们一般会添加.h头文件和一个.cpp源文件。结果忘记把.cpp文件添加到工程中了，即没有参与编译，没有生成供链接使用的obj文件。如果有代码调用到该C++类的接口，则在编译链接的时候会报 unresolved external symbol错误，即链接不到该C++类对应的接口。

6、函数中返回了一个局部变量的地址或者引用

       在函数中返回了一个局部变量的地址或者引用，而这个局部变量在函数结束时其生命周期就结束了，内存就被释放了。当外部访问到该变量的内存，会触发内存访问违例的异常，因为该变量的内存已经释放了。比如如下的错误代码：

char* GetResult()
{
    char chResult[100] = { 0 };

    // ......

    return chResult;
}

7、忘记将父类中的接口声明virtual函数，导致多态没有生效

       代码中本来要借助于C++多态的虚函数调用，调用子类实现的接口，结果忘记在父类中将对应的接口声明为virtual，导致没有调用到子类实现的函数。一定要记住，要实现多态下的函数调用，父类的相关接口必须声明为virtual。

class Shape()
{
    // ...

    virtual void Draw();

    // ...
} 

8、该使用双指针的地方，却使用了单指针

       有时我们需要调用一个接口去获取某些数据，接口中将数据拷贝到传入的参数对应的内存中，此时设计参数时会传入指针或引用。我们在调用GetData之前定义了结构体指针p，并new出了对应的结构体对象内存，应该在定义GetData接口时应该使用双指针（指针的指针）的，结果错写成了单指针。

      有问题的代码如下：

struct CodecInfo     // 编码信息
{
    int nFrameRate；

    // ...
}


CodecInfo* pInfo = new CodecInfo;

GetAudioCodecPtr()->GetCodecInfo(pInfo);   // 调用AudioCodec::GetCodecInfo获取编码信息


AudioCodec::GetCodecInfo( CodecInfo* pInfo)  // 此处的参数不应该使用单指针
{
    memcpy(pInfo, m_codecInfo, sizeof(CodecInfo));
}

       上图中的AudioCodec::GetCodecInfo接口的参数不应该为单指针，应该用双指针，修改后的代码应该如下：

AudioCodec::GetCodecInfo( CodecInfo** pInfo)  // 此处的参数类型使用双指针
{
    memcpy(*pInfo, m_codecInfo, sizeof(CodecInfo));
}

9、发布exe程序时，忘记将exe依赖的C运行时库和MFC库带上

        比如新人用VS-MFC库编写一个测试用的工具软件，结果在发布release版本程序时，没有将程序依赖的C运行时库带上，导致该工具软件在某些电脑中启动报错，提示找不到C运行时库：

因为程序中依赖了动态版本的运行时库和MFC库，在发布程序时要将这些库带上。有些系统中没有这些库，程序启动时就会报找不到库，就会启动失败。

10、应该使用深拷贝，却使用了浅拷贝

       本来应该要进行深拷贝的，却使用了浅拷贝（直接赋值），导致另个不同生命周期的C++对象指向了同一块内存，一个对象将内存释放后，另一个对象再用到这块内存，就造成了内存访问违例，产生异常。

       有个经典的C++笔试题，让我们实现String类的相关函数，其主要目的就是用来考察对深拷贝与浅拷贝的理解的。题目中给出String类的声明：

class String{
public:
    String();
    String(const String & str);
    String(const char* str);
    String& operator=(String str);
    char* c_str() const;
    ~String();
    int size() const;
private:
    char* data;
};

让写出上述几个函数的内部实现。这些函数的实现代码如下：

//普通构造函数  
String::String(const char *str)
{
	if (str == NULL)
	{
		m_data = new char[1];// 得分点：对空字符串自动申请存放结束标志'\\0'的，加分点：对m_data加NULL判断  
		*m_data = '\\0';
	}
	else
	{
		int length = strlen(str);
		m_data = new char[length + 1];// 若能加 NULL 判断则更好
		strcpy(m_data, str);
	}
}
 
 
// String的析构函数  
String::~String(void)
{
	delete[] m_data; // 或delete m_data;  
}
 
 
//拷贝构造函数  
String::String(const String &other)// 得分点：输入参数为const型  
{		 
	int length = strlen(other.m_data);
	m_data = new char[length + 1];// 若能加 NULL 判断则更好  
	strcpy(m_data, other.m_data);
}
 
 
//赋值函数  
String & String::operator = (const String &other) // 得分点：输入参数为const型  
{
	if (this == &other)//得分点：检查自赋值  
		return *this; 
	if (m_data)
	    delete[] m_data;//得分点：释放原有的内存资源  
	int length = strlen(other.m_data);
	m_data = new char[length + 1];//加分点：对m_data加NULL判断  
	strcpy(m_data, other.m_data);
	return *this;//得分点：返回本对象的引用    
}