关于C++设计模式的问题，纯虚方法调用答案

【问题标题】：Question on C++ design pattern, pure virtual method called关于C++设计模式的问题，纯虚方法调用
【发布时间】：2021-04-09 02:50:44
【问题描述】：

我在尝试来自https://en.wikipedia.org/wiki/Bridge_pattern 的代码时遇到了虚拟方法问题。

错误信息是：

调用纯虚方法
在没有活动异常的情况下调用终止
中止（核心转储）

代码是用g++ -o bridge bridge.cpp -std=c++11编译的

为什么drawing_api_.DrawCircle()调用DrawingAPI中的虚函数？

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


class DrawingAPI {
  public:
    virtual ~DrawingAPI() = default;
    virtual std::string DrawCircle(float x, float y, float radius) const = 0;
};

class DrawingAPI01 : public DrawingAPI {
  public:
    std::string DrawCircle(float x, float y, float radius) const override {
      return "API01.circle at " + std::to_string(x) + ":" + std::to_string(y) +
        " - radius: " + std::to_string(radius); 
    }
};

class DrawingAPI02 : public DrawingAPI {
  public:
    std::string DrawCircle(float x, float y, float radius) const override {
      return "API02.circle at " + std::to_string(x) + ":" + std::to_string(y) +
        " - radius: " + std::to_string(radius); 
    }
};

class Shape {
  public:
    Shape(const DrawingAPI& drawing_api) : drawing_api_(drawing_api) {}
    virtual ~Shape() = default;

    virtual std::string Draw() const = 0;
    virtual float ResizeByPercentage(const float percent) = 0;

  protected:
    const DrawingAPI& drawing_api_;
};

class CircleShape: public Shape {
  public:    
    CircleShape(float x, float y, float radius, const DrawingAPI& drawing_api)
      : Shape(drawing_api), x_(x), y_(y), radius_(radius) {}

    std::string Draw() const override {
        return drawing_api_.DrawCircle(x_, y_, radius_);
    }

    float ResizeByPercentage(const float percent) override {
      return radius_ *= (1.0f + percent/100.0f);
    }
  
  private:
    float x_, y_, radius_;
};

int main(int argc, char** argv) {
  std::vector<CircleShape> shapes {
    CircleShape{1.0f, 2.0f, 3.0f, DrawingAPI01{}},
    CircleShape{5.0f, 7.0f, 11.0f, DrawingAPI02{}}
  }; 

  for (auto& shape: shapes) {
    shape.ResizeByPercentage(2.5);
    std::cout << shape.Draw() << std::endl;
  }

  return 0;
}

【问题讨论】：

避免引用持有DrawingAPI。一些 Wikipedia 实现的 C++ 设计模式示例存在错误。
非常感谢，修改指针的引用后问题解决了。

标签： c++ design-patterns

【解决方案1】：

发布的代码有问题——尤其是Shape 类包含对a-DrawingAPI-object 的引用：

class Shape {
[...]

protected:
   const DrawingAPI& drawing_api_;
};

...但是它所引用的对象是一个临时对象，一旦main() 中的CircleShape 构造函数返回，就会被销毁。

避免错误的一种方法是声明 DrawingAPI01 和 DrawingAPI02 对象，使其生命周期比引用它们的 CircleShape 对象的生命周期更长，例如通过将main() 更改为如下所示：

int main(int argc, char** argv) {
   DrawingAPI01 api01;   // declaring these up here keeps them valid
   DrawingAPI01 api02;   // until the end of main()

   std::vector<CircleShape> shapes {
      CircleShape{1.0f, 2.0f, 3.0f, api01},
      CircleShape{5.0f, 7.0f, 11.0f, api02}
   };

   for (auto& shape: shapes) {
      shape.ResizeByPercentage(2.5);
      std::cout << shape.Draw() << std::endl;
   }

   return 0;
}

【讨论】：