refactor(server): properly use weak pointer

API!(server/client:create): require manually specifying server
API!(server/client:newClient): now returns a SP instead of WP

3 files changed, 39 insertions, 37 deletions

diff --git a/src/server/client.cpp b/src/server/client.cpp
index 6e42b69..e84a654 100644
--- a/src/server/client.cpp
+++ b/src/server/client.cpp
@@ -16,16 +16,16 @@ using namespace LibSock::Server;
 Client::Client(SP<Server> server, SP<Clients> clients, bool track, bool oneShot)
 	: m_track(track)
 	, m_oneShot(oneShot) {
-	pServer	 = std::move(server);
-	pClients = std::move(clients);
+	m_wpServer	= std::move(server);
+	m_wpClients = std::move(clients);
 
-	if (!pClients)
+	if (m_wpClients.expired())
 		throw std::runtime_error("server:client: ClientManager doesn't exist");
-	if (!pServer)
+	if (m_wpServer.expired())
 		throw std::runtime_error("server:client: Server doesn't exist");
 
-	m_sockfd = std::make_shared<LibSock::CFileDescriptor>(accept(pServer->getSocket()->get(), reinterpret_cast<sockaddr *>(&m_addr), &m_addrLen)); // if this is in the init list, it will run before
-																																				   // m_addrLen, so it won't work :/
+	m_sockfd = std::make_shared<LibSock::CFileDescriptor>(accept(m_wpServer.lock()->getSocket()->get(), reinterpret_cast<sockaddr *>(&m_addr), &m_addrLen)); // if this is in the init list, it will run before
+																																							 // m_addrLen, so it won't work :/
 
 	if (!m_sockfd->isValid())
 		throw std::runtime_error("server:client: Failed to create socket");
@@ -91,7 +91,7 @@ UP<SRecvData> Client::read(std::optional<std::function<bool(const SRecvData &)>>
 	}
 
 	if (m_track)
-		pClients->m_vDatas.push_back({recvData->data, self});
+		m_wpClients.lock()->m_vDatas.push_back({recvData->data, m_self});
 
 	return recvData;
 }
@@ -108,11 +108,11 @@ void Client::runLoop(bool resumeHist, std::optional<std::function<bool(const SRe
 
 	recvLoop(cb);
 
-	pClients->kick(self);
+	m_wpClients.lock()->kick(m_self);
 }
 
 void Client::resumeHistory() {
-	for (const auto &chat : pClients->getDatas())
+	for (const auto &chat : m_wpClients.lock()->getDatas())
 		write(chat.msg);
 }
 
diff --git a/src/server/clients.cpp b/src/server/clients.cpp
index e1dd161..a274224 100644
--- a/src/server/clients.cpp
+++ b/src/server/clients.cpp
@@ -2,7 +2,6 @@
 #include "libsock/server/client.hpp"
 #include <algorithm>
 #include <ranges>
-#include <stdexcept>
 #include <unistd.h>
 #include <utility>
 
@@ -29,24 +28,25 @@ void Clients::shutdownClients(std::optional<std::function<bool(const std::vector
 
 Clients::Clients() {
 	if (std::atexit([]() {
-			if (pClients)
-				pClients->shutdownClients();
+			for (auto c : vpClients)
+				if (const auto p = c.lock())
+					p->shutdownClients();
 		}))
 		void(); // failed to register exit handler...
 }
 
-SP<Clients> Clients::create() {
-	if (pClients)
-		throw std::runtime_error("server:clients ClientManager already exist");
-
-	pClients = SP<Clients>(new Clients());
-	return pClients;
+SP<Clients> Clients::create(WP<Server> server) {
+	auto c = SP<Clients>(new Clients());
+	vpClients.emplace_back(c);
+	c->m_self	  = c;
+	c->m_wpServer = server;
+	return c;
 }
 
-SP<Clients> Clients::get() {
-	if (!pClients)
-		pClients = shared_from_this();
-	return pClients;
+WP<Clients> Clients::get() {
+	if (m_self.expired())
+		m_self = shared_from_this();
+	return m_self;
 }
 
 Clients::~Clients() {
@@ -55,13 +55,14 @@ Clients::~Clients() {
 			thread.detach();
 		kick(WP<Client>(client));
 	}
+	vpClients.erase(std::remove_if(vpClients.begin(), vpClients.end(), [this](const WP<Clients> &wptr) { return !wptr.owner_before(m_self) && !m_self.owner_before(wptr); }), vpClients.end());
 }
 
-WP<Client> Clients::newClient(std::function<void(const WP<Client> &)> loopFunc, SP<Server> server) {
-	SP	  client		  = SP<Client>(new Client(server, shared_from_this()));
-	auto &instance		  = m_vClients.emplace_back(std::jthread([&client, &loopFunc]() { loopFunc(client); }), client);
-	instance.second->self = std::weak_ptr<Client>(instance.second);
-	return instance.second;
+SP<Client> Clients::newClient(std::function<void(const WP<Client> &)> loopFunc) {
+	SP	  client			= SP<Client>(new Client(m_wpServer.lock(), shared_from_this()));
+	auto &instance			= m_vClients.emplace_back(std::jthread([&client, &loopFunc]() { loopFunc(client); }), client);
+	instance.second->m_self = std::weak_ptr<Client>(instance.second);
+	return client;
 }
 
 void Clients::broadcast(const std::string &msg, std::optional<std::weak_ptr<Client>> self) {
diff --git a/src/server/server.cpp b/src/server/server.cpp
index c14a39e..aabbb50 100644
--- a/src/server/server.cpp
+++ b/src/server/server.cpp
@@ -1,6 +1,7 @@
 #include "libsock/server/server.hpp"
 #include "libsock/types.hpp"
 #include "misc/FileDescriptor.hpp"
+#include <algorithm>
 #include <cstring>
 #include <mutex>
 #include <stdexcept>
@@ -14,8 +15,6 @@ namespace Server {
 Server::Server(uint16_t port, bool reuseaddr, bool keepalive)
 	: m_sockfd(std::make_shared<LibSock::CFileDescriptor>(socket(AF_INET, SOCK_STREAM, 0)))
 	, m_port(port) {
-	if (pServer)
-		throw std::runtime_error("server:server Server already exist");
 	std::lock_guard<std::mutex> lk(m_mutex);
 	if (!m_sockfd->isValid() || m_sockfd->get() < 0)
 		throw std::runtime_error("server:server Failed to create socket");
@@ -37,20 +36,22 @@ Server::Server(uint16_t port, bool reuseaddr, bool keepalive)
 }
 
 SP<Server> Server::create(uint16_t port, bool reuseaddr, bool keepalive) {
-	if (pServer)
-		throw std::runtime_error("server:server Server already exist");
-	pServer = SP<Server>(new Server(port, reuseaddr, keepalive));
-	return pServer;
+	auto c = SP<Server>(new Server(port, reuseaddr, keepalive));
+	vpServer.emplace_back(c);
+	c->m_self = c;
+	return c;
 }
 
-SP<Server> Server::get() {
-	if (!pServer)
-		pServer = shared_from_this();
-	return pServer;
+WP<Server> Server::get() {
+	std::lock_guard<std::mutex> lk(m_mutex);
+	if (m_self.expired())
+		m_self = shared_from_this();
+	return m_self;
 }
 
 Server::~Server() {
 	m_sockfd.reset();
+	vpServer.erase(std::remove_if(vpServer.begin(), vpServer.end(), [this](const SP<Server> &sptr) { return sptr.get() == this; }), vpServer.end());
 }
 
 SP<LibSock::CFileDescriptor> Server::getSocket() const {