Question

struct PacketBase
{
    virtual ~PacketBase() {}

    template<class T> 
    const T& get() const
    {
        return static_cast<const PacketVal<T>&>(*this).val;
    }

    template<class T> 
    void set(const T& rhs)
    {
        return static_cast<PacketVal<T>>(*this).val = rhs;
    }
};


template <typename T>
struct PacketVal : public PacketBase
{
    T val;

    PacketVal(const T& rhs) : val(rhs) {}
    ~PacketVal() {}
};


class CResponsePacket
{
private:
    std::vector<std::pair<int, const PacketBase*>> m_vPacketData;

public:
    void addValue(int n, const PacketBase& packetData)
    {
        m_vPacketData.emplace_back(std::make_pair(n, &packetData));
    }

    void print()
    {
         for ( auto& data : m_vPacketData )
         {
              std::printf("%d - ", data.first);
              std::printf("%s\n", data.second->get<std::string>().c_str());
         }
    }
};

CResponsePacket具有成员变量模板类。

int main()
{
    CResponsePacket packet;
    std::string strAAA("AAA");

    PacketVal<std::string> pVal(strAAA);
    packet.addValue(1, pVal);

    for ( int nIdx = 0; nIdx < 5; ++nIdx )
    {
        PacketVal<std::string> pp(strAAA);
        packet.addValue(nIdx + 1, pp);
    }

    packet.print();

    return 0;
}

结果是

1-AAA
1-
2-
3-
4-
5-

由于实例处于循环状态，因此内存被破坏。但这只是一个例子，实际上我必须使用循环。我该如何解决...？

................. ..................

Answer 1

您可以使用std::shared_pointer或std::unique_ptr（取决于您的使用），并让向量看起来像这样：

std::vector<std::pair<int, std::shared_pointer<PacketBase>>> m_vPacketData;

因此addValue方法应如下所示：

void addValue(int n, const PacketBase& packetData)
{
    m_vPacketData.emplace_back(
        std::make_pair(
             n, 
             std::make_shared<PacketBase>(packetData)
        )
    );
}

这将在堆上复制packetData，并将引用存储在共享指针中。如果您不打算制作此副本，则必须在PacketBase

中定义一个新的构造函数

class PacketBase{
public: 
     PacketBase(PacketBase&& el): attr(std::move(el.attr))... {...}
}

，然后重新定义接受右值引用的addValue

void addValue(int n, PacketBase&& packetData)
{
    m_vPacketData.emplace_back(
        std::make_pair(
             n, 
             std::make_shared<PacketBase>(std::move(packetData))
        )
    );
}

然后从addValue实际调用正确的main：

for ( int nIdx = 0; nIdx < 5; ++nIdx )
{
    PacketVal<std::string> pp(strAAA);
    packet.addValue(nIdx + 1, std::move(pp));
}

或者实际上只是：

for ( int nIdx = 0; nIdx < 5; ++nIdx )
    packet.addValue(nIdx + 1, PacketVal<std::string>(strAAA););

显然，这将迫使您重写整个类，因为您正在更改属性类型...实现此目的的另一种方法是从

切换

m_vPacketData.emplace_back(std::make_pair(n, &packetData));

到

m_vPacketData.emplace_back(std::make_pair(n, new PacketVal(packetData)));

但是您要记住要删除堆上的对象，否则会发生内存泄漏

具有模板成员变量的C ++类。和参数存储

1 个答案: