C++ 类型转换错误设置和从数组中获取派生类成员 (**)

Question

您好，我在使用 Computer.cpp 中的 set 和 get 方法时遇到问题。我不断收到类型转换错误，例如

从表达式中引用类型“const Case&”的初始化无效 'Case' 或 'Part'、'Part*' 等类型的。

我想要做的是将不同的计算机部件（“部件”是各种具体部件的超类，如机箱、显示器、cpu 等）存储到数组中

Part** m_requiredParts; 

这是代码摘录，我在 qt creator 3.6.0 上运行它。

头文件，

Computer.h 

#ifndef COMPUTER_H
#define COMPUTER_H

class Computer : public ComputerPlan
{
public:

    /**
     * @brief Computer default constructor
     */
    Computer();

    /**
     * @brief ~Computer destructor
     */
    virtual ~Computer();

    // Getters
    /**
     * @brief getCase retrieves the computer's case information
     * @return the computer's Case
     */
    virtual const Case& getCase() const;

    // Setters
    /**
     * @brief setCase setter method for Case
     * @param computerCase computer's Case
     */
    virtual void setCase(const Case &computerCase);

protected:

    Part** m_requiredParts;

};

#endif // COMPUTER_H

.CPP 文件

Computer.cpp

#include "Computer.h"

Computer::Computer()
{
    m_requiredParts = new Part* [8];
}

Computer::~Computer()
{

}

const Case& Computer::getCase() const
{
    const Case& c = (Case&)m_requiredParts[0];
    return c;
}

void Computer::setCase(const Case &computerCase)
{
    m_requiredParts[0] = (Part*) &computerCase;
    //Part**
}

添加一些附加信息，

我在 main.cpp 中通过以下方式设置和获取它们

Computer* computer = new Computer();
computer->setCase(Case("NZXT", Tower));
std::string expected = "Case: NZXT, Case Type: Tower";
std::string actual = std::string(computer->getCase().getPartInformation());
delete computer;
return expected == actual;

Answer 1

这本身不是一个答案，但它太大而不能成为评论，需要指出

computer->setCase(Case("NZXT", Tower));

创建一个临时的Case，您可以在setCase 中使用const & 捕获该Case。 const & 将临时的生命周期延长到函数中，所以我们在那里没问题。不幸的是，您在setCase 中存储了指向该临时文件的指针。当函数结束时，const & 超出范围，临时对象现在没有任何绑定，因此它也被销毁。现在你有了一个指向不再存在的对象的指针。

您需要确保您粘贴到 setCase 的对象比您放入的 Computer 的寿命更长。

Answer 2

您必须从指针数组中取消引用指针：

const Case& c = (Case&)(*m_requiredParts[0]);

此外，使用 c 样式类型转换也不是好习惯，请改用 static_cast 或 dynamic_cast：

const Case& c = static_cast<Case&>(*m_requiredParts[0]);

在你的类实例中存储指向外部对象的指针是个坏主意。

关于这个computer->setCase(Case("NZXT", Tower)); 用例，在类实例中存储指向临时的指针绝对是个坏主意，使用new 并更改原型以接受指针。并且不要忘记在析构函数中进行清理。