将指向类对象的智能指针与类对象混合

Question

我有一个代码，其中一些类通过设计使用工厂函数来生成实际类，而另一些则不使用。许多类都实现了同名的函数，并且这些函数被顺序调用（见下文）。这种设计导致混合了指向对象和对象本身的智能指针。下面的代码是不是设计不好，我应该到处使用智能指针吗？

#include <iostream>
#include <memory>

class A
{
    public:
        void print_name() { std::cout << "A\n"; }
};

class B
{
    public:
        virtual void print_name() = 0;
        static std::unique_ptr<B> factory(const int n);
};

class B1 : public B
{
    public:
        void print_name() { std::cout << "B1\n"; }
};

class B2 : public B
{
    public:
        void print_name() { std::cout << "B2\n"; }
};

std::unique_ptr<B> B::factory(const int n)
{
    if (n == 1)
        return std::make_unique<B1>();
    else if (n == 2)
        return std::make_unique<B2>();
    else
        throw std::runtime_error("Illegal option");
}

int main()
{
    A a;
    std::unique_ptr<B> b1 = B::factory(1);
    std::unique_ptr<B> b2 = B::factory(2);

    // The block below disturbs me because of mixed . and ->
    a.print_name();
    b1->print_name();
    b2->print_name();

    return 0;
}

编辑

我在下面的 cmets 后面的示例中添加了智能指针。

Answer 1

这在我看来是一个合理的设计。在客户端代码中，您将通过基类接口工作。

class Base {};
class A: public Base {};
class B: public Base {};
class B1: public B {};
class B2: public B {};

class Factory {
  std::unique_ptr<Base> create(const int n) {
    // Instantiate a concrete class based on n
    return std::unique_ptr<Base>(new A());
  }
}

Answer 2

通常，如果不需要，您应该避免使用指针。在大多数情况下，诸如返回值优化之类的主题使它们成为可选的。另外，不要使用 C 样式指针。 C++11 引入了 memory-header，其中包含许多有用的智能指针。

是的，我觉得在这样的程序中使用这些指针是一个糟糕的设计决定。

Answer 3

除了混合之外，为什么代码会打扰您？你认为什么实际上会因为它而变得糟糕？您认为这种差异真的会损害可读性吗？

是的，调用 print 的行看起来不同，并且这种差异告诉您正在以不同的方式管理对象。

如果开发人员不能同时理解两者，那么将它们设为相同并没有明显的好处。然后 -> 那么你有更大的问题。

我曾在一个代码库中工作，其中几乎所有内容都由共享指针处理以保持一致性，即使有些东西不应该是指针，而且大多数东西并不是真正共享的。应该避免不必要的不​​一致，但在隐藏合法差异的情况下保持一致确实没有帮助。

您的代码对所有指针的一个好处是，由于 a 不是指针，您可以确定 a 不为空。