c++ - undefined object 引用错误

Question

#include <iostream>
#include <string>
using namespace std;

class Part{
public:
    std::string spec;
    Part(std::string str){
        this->spec = str;
    }
    std::string getSpec(){
        return spec;
    }
};

class Car{
public:
    Part getEngine();
    Part getWheels();
    Part getBody();
};


class Benz:public Car{
public:
    Part getEngine(){
        return Part("Benz Engine");
    }   
    Part getWheels(){
        return Part("Benz Wheels");
    }
    Part getBody(){
        return Part("Benz Body");
    }
};

class Audi:public Car{
public:
    Part getEngine(){
        return Part("Audi Engine");
    }   
    Part getWheels(){
        return Part("Audi Wheels");
    }
    Part getBody(){
        return Part("Audi Body");
    }
};

class CarFactory{
public:
    Car *pcar;
    Car *getCar(std::string carType){
        if (carType.compare("Benz") == 0){
            pcar = new Benz();
        }else if (carType.compare("Audi") == 0){
            pcar = new Audi();
        }
        return pcar;
    }
};

int main(){
    CarFactory *cf = new CarFactory();
    Car *mycar = cf->getCar("Benz");
    Part myCarBody = mycar->getBody();
    cout <<myCarBody.getSpec() <<endl;
    //cout << mycar->getBody().getSpec() <<endl;
}

在上面的代码中，main函数的Part myCarBody = mycar->getBody();行获取对象 undefined reference 错误。你能帮我解决这个问题吗？请解释一下 Car *mycar 和 Car mycar 之间的区别。什么时候选择哪一个？

感谢您的帮助。

Answer 1

Car 的成员函数应该是纯虚函数，而不是你的非虚函数声明:

class Car
{
public:
    virtual ~Car() = default;     // Virtual destructor
    virtual Part getEngine() = 0;
    virtual Part getWheels() = 0;
    virtual Part getBody() = 0;
};

这就是链接器错误的原因。 编译器需要 Car 的函数定义，因为这些函数不是纯虚函数，但它找不到它。
使用纯虚拟成员函数，Car 成为一个抽象类 并提供一个接口(interface)(纯虚拟)，该接口(interface)必须由从它派生的类实现。否则，派生类也将是抽象的并且不可实例化。

通过调用基类的虚函数，您最终会调用重写它的派生类的函数。 Destructor也是如此，需要做成virtual，这样才能调用派生类的析构函数，避免未定义的行为。如果您在其中一个派生类中有动态分配的数据，就会发生内存泄漏。

此外，您必须在处理多态性时使用指针或引用。其背后的逻辑是指针具有相同的大小，但是像 Benz 这样的派生类可能有额外的数据并且不适合 Car 占用的空间并且会被切片。因此，如果没有指针或引用，赋值/复制构造将无法正常工作。

在评论中建议:

• 访问函数(“getters”和“setters”)通常与私有(private)或 protected 成员。你的是公开的，访问者是无用且可以规避。保留访问器(accessor)，制作数据私有(private)的。当您退回成员(member)时，您可以通过引用到常量。但不要为 getEngine、getWheels 和getBody，因为它们返回的是本地对象，而不是成员。

  std::string const& getSpec() { return spec; }
  

  当我们这样做时，您也可以通过对常量的引用来传递 std::string。

  Part(std::string const& str) : spec(str) {}
  

• 工厂类

  class CarFactory
  {
  public:
      Car *getCar(std::string const& carType)
      {
          if(carType == "Benz") // std::string::compare is for comparing by alphabetical order
              return new Benz();
  
          if(carType == "Audi")
              return new Audi();
  
          return nullptr; // invalid carType, return nullptr and check for that in main()
          // You were returning previously constructed Car, was that desired?
      }
  };
  

您还忘记删除 myCar，请在其他答案中查看更多有用的信息。