Question

我意识到在C ++中有很多关于朋友类的问题。不过，我的问题与具体情况有关。鉴于以下代码，以这种方式使用朋友是否合适？

class Software
{
    friend class SoftwareProducer;

    SoftwareProducer* m_producer;
    int m_key;
    // Only producers can produce software
    Software(SoftwareProducer* producer) : m_producer(producer) { }

public:
    void buy()
    {
        m_key = m_producer->next_key();
    }
};

class SoftwareProducer
{
    friend class Software;

public:
    Software* produce()
    {
        return new Software(this);
    }

private:
    // Only software from this producer can get a valid key for registration
    int next_key()
    {
        return ...;
    }
};

谢谢，

致以最诚挚的问候，

Answer 1

当然，这是完全合理的。基本上你所做的与工厂模式非常相似。我认为没有问题，因为你的代码似乎暗示每个Software对象都应该有一个指向其创建者的指针。

虽然，通常你可以避免使用像SoftwareProducer这样的“Manager”类，并且只需要在Software中使用静态方法。因为似乎只有一个SoftwareProducer。也许是这样的事情：

class Software {
private:
    Software() : m_key(0) { /* whatever */ }
public:
    void buy() { m_key = new_key(); }
public:
    static Software *create() { return new Software; }
private:
    static int new_key() { static int example_id = 1; return example_id++; }
private:
    int m_key;
};

然后你可以这样做：

Software *soft = Software::create();
soft->buy();

当然，如果您计划拥有多个SoftwareProducer对象，那么您所做的事情似乎是合适的。

Answer 2

这对我来说似乎完全合情合理。

查看Bruce Eckel Factories: encapsulating object creation的Thinking in C++，了解有关此朋友使用的更多信息。

您可能还需要查看C++ FAQ on friends以获取更多经验法则。

Answer 3

我认为将SoftwareProducer作为软件的朋友是可以接受的，但我认为没有理由为什么Software必须是SoftwareProducer类的朋友。这是他们之间不必要的依赖。您可以将key作为Software类的构造函数参数。此外，您可能希望将Software类的析构函数设置为private，以便除SoftwareProducer之外的任何人都可以销毁它们。

Answer 4

我不会讨论友谊问题，但我倾向于不喜欢循环依赖。软件依赖于依赖于软件的SoftwareProducer ......我会尝试重构。

另请注意，友谊关系会打开其他类的所有实例的内部。这就像说 next_key 中的评论为假：

使 next_key 私有禁止任何类调用它，但是一旦打开 Software 类，任何软件都可以调用 next_key SoftwareProducers

C ++朋友类

4 个答案: