5

我知道这肯定是一个 n00b 问题，但是我必须实现一个模型客户端-服务器顺序交互应用程序，并且由于客户端-服务器调用的数量不同，我不能只迭代外部函数中的步骤，总是从客户端，然后将其转发到服务器，反之亦然，所以我需要让我的类Server和Client类相互了解，以便它们可以在它们之间调用它们的公共方法。一种方法是将两者都设计为单例，但我希望以更简单的方式来实现，更准确地说是使用循环引用：客户端存储对服务器的引用，服务器存储对客户端的引用。我知道这可能不是一个好方法，并且当它变得太深时可能会导致调用堆栈爆炸，因此欢迎对我的设计进行任何改进。

为了实现所描述的实现，我认为我可以使用std::shared_ptr, 因为std::unique_ptr如果我还想防止 main 中的两个变量在我调用两个 setter 时被破坏（对吗？）。所以，这就是我所拥有的（简化代码）：

#include <iostream>
#include <memory>

class Server;

class Client
{
public:
    void SetServer (const Server &server);
private:
    std::shared_ptr<const Server> server;
};

void Client::SetServer (const Server &server)
{
    this->server = std::shared_ptr<const Server>(&server);
}

class Server
{
public:
    void SetClient (const Client &client);
private:
    std::shared_ptr<const Client> client;
};

void Server::SetClient (const Client &client)
{
    this->client = std::shared_ptr<const Client>(&client);
}

int main ()
{
    Server server;
    Client client;

    server.SetClient(client);
    client.SetServer(server);

    //Here I ask the client to start interacting with the server.
    //The process will terminate once the client
    //exhausts all the data it needs to send to the server for processing

    return 0;
}

不幸的是，我的代码似乎试图多次调用客户端和服务器（隐式）析构函数，或者一些类似的讨厌的事情，我确信这是由于我对std::shared_ptr应该如何工作的理解不足造成的。请指教。

4

2 回答 2

7

您在堆栈上分配您的服务器和客户端实例，它们将在main()退出时被删除。您也不想std::shared_ptr删除它们。因此，有两种解决方案：

  1. 在客户端和服务器中使用非托管指针，并在外部管理它们的生命周期。

  2. 到处使用托管指针，包括main(). 请注意，这shared_ptr意味着所有权。在当前设计中，服务器拥有客户端，但客户端也拥有服务器。这是一个循环引用：默认情况下，它们永远不会被释放，除非您在仍然可以的时候重置其中一个指针。

    例如，您可以决定客户端保持服务器处于活动状态，因此如果没有其他 shared_ptrs 指向它，最后消失的客户端将关闭服务器。服务器weak_ptr对客户端有一个，但客户端shared_ptr对服务器有一个。

class Client;
class Server;

class Client
{
public:
    void SetServer (const std::shared_ptr<const Server> &server);
private:
    std::shared_ptr<const Server> server;
};

void Client::SetServer (const std::shared_ptr<const Server> &server)
{
    this->server = server;
}

class Server
{
public:
    void SetClient (const std::weak_ptr<const Client> &client);
private:
    std::weak_ptr<const Client> client;
};

void Server::SetClient (const std::weak_ptr<const Client> &client)
{
    this->client = client;
}


int main()
{
  std::shared_ptr<Server> server(new Server);
  std::shared_ptr<Client> client(new Client);

  server->SetClient(client);
  client->SetServer(server);

  // do stuff

  return 0;
}
于 2012-06-20T20:00:03.260 回答
0

您的变量具有自动生命周期，并且在超出范围时将被销毁。

因此，使用任何生命周期管理的智能指针都是错误的，因为它会再次调用 delete。

于 2012-06-20T19:53:30.840 回答