智能指针+拷贝构造+vector容器+多态引起的bug

今天在调试一段代码的时候，VC编译提示：

error C2280: “T485CommCtrlPara::T485CommCtrlPara(const T485CommCtrlPara &)”: 尝试引用已删除的函数

函数执行部分如下：

看意思是这个pComm485Pro已经消亡了，后续push_back到vec485DevCommPara有问题，但智能指针已经move了，这样new出来资源的所有权应该已经转移了，为啥还会有问题呢？

找了下chatgpt和newbing查了下，他提供了一个馊主意：

这个是把智能指针转成普通指针操作一把，后续还要delete，得不偿失，而且只是编译通过，问题依然存在。

本文就针对这个问题，详细分析下，先把有问题代码精简下，贴出来：

#include <iostream>
#include <string>
#include <vector>
#include <memory>

typedef struct {
    std::string strPortName;
    unsigned int dwBaudRate;
    unsigned char bByteSize;
    unsigned char bStopBits;
    unsigned char bParity;
} TCommPara;

class CComm
{
public:
    config(){;};
} 

class CMeterProto
{
;
}

class cModbusProto:public CMeterProto
{
public:
    CModbusProto(const unsigned char bBAPn, const TCommPara tCommPara)
    : m_bBAPn(bBAPn), m_tCommPara(tCommPara) 
    {
    }
private:
    unsigned char m_bBAPn;
    TCommPara m_tCommPara;
}

typedef struct {
    std::unique_ptr<CMeterProto> pComm485Pro{nullptr};
    CComm cMterCom;
}T485CommCtrlPara;

int LoadBA485CommPara(TCommPara &tCommPara, T485CommCtrlPara &t485CommPara, vector<T485CommCtrlPara>& vec485DevCommPara)
{
    tCommPara.strPortName = "com1";
    tCommPara.dwBaudRate = 9600;
    tCommPara.bByteSize = 8;
    tCommPara.bStopBits = 1;
    
    t485CommPara.cMterCom.config();
    std::unique_ptr<CMeterProto> pComm485Pro(new CModbusProto(1, tCommPara));
    t485CommPara.pComm485Pro = std::move(pComm485Pro);
    vec485DevCommPara.push_back(t485CommPara);
    
    return 1;
}

int main()
{
    TCommPara tCommPara;
    T485CommCtrlPara t485CommPara;
    vector<T485CommCtrlPara> vec485DevCommPara;
    
    int nBaNum = LoadBA485CommPara(tCommPara, t485CommPara, vec485DevCommPara);
    
    std::count<<nBaNum<<std::endl;
}

通过将如上代码交给chatgpt分析，开始他巴拉巴拉说没啥问题，后来我把VC的编译信息交给他

如上代码，在vs 2015上编译，提示：error C2280: “T485CommCtrlPara::T485CommCtrlPara(const T485CommCtrlPara &)”: 尝试引用已删除的函数

这个时候它开始分析代码了，经过几轮的修改，他终于还是发现了是push_back的问题。

主要原因就是push_back本质上是调用拷贝构造函数，将资源拷贝到vector的堆空间上，如果使用默认拷贝函数，就是浅拷贝，不会将pComm485Pro的资源拷贝过去，所以push_back拷贝的只是一个地址，实际资源并没有拷贝。

原因知道后，修改就很简单了，实现拷贝构造函数即可，如上的

typedef struct {
std::unique_ptr<CMeterProto> pComm485Pro{nullptr};
CComm cMterCom;
}T485CommCtrlPara;

要定义下拷贝构造函数，按照chatgpt的实现，

T485CommCtrlPara(const T485CommCtrlPara& other)
       : pComm485Pro(other.pComm485Pro ? std::make_unique<CMeterProto>(*other.pComm485Pro) : nullptr),
       cMterCom(other.cMterCom)
   {
       // 在自定义拷贝构造函数中，正确拷贝智能指针成员
   }

编译不了，因为直接在struct中定义拷贝构造，vc编译还有问题，既然如此，修改成类方式好了：