Moved players and playfields lists into "Global" struct

generic.proto

@@ -15,6 +15,7 @@ message Error {
         AuthenticationRequired = 6;
         IllegalOperation = 7;
         OutOfRange = 8;
+        NotYourTurn = 9;
     }
     Code code = 2;
 }
@@ -33,6 +34,7 @@ message Operation {
         SettingsSync = 5;
         PlayfieldSync = 6;
         RobotUpdate = 7;
+        YourTurn = 8;
     }
     Code code = 1;
 }

server/Connection.cpp

@@ -58,7 +58,7 @@ asio::awaitable<void> Client::handlePacket(const Packet& packet) {
     case Generic::Operation::SimpleAuth: {
         const Generic::SimpleAuth& authData = *static_cast<Generic::SimpleAuth*>(packet.data.get());
         // Check that username is not already taken
-        if (std::find_if(clients.begin(), clients.end(), [&, this] (const auto& o) {return o->getUsername() == authData.username();}) != clients.end()) {
+        if (std::find_if(global.clients.begin(), global.clients.end(), [&, this] (const auto& o) {return o->getUsername() == authData.username();}) != global.clients.end()) {
             throw Error("Username has already been taken", Generic::Error::UsernameAlreadyTaken);
         }
         // Check that username is "good" (no unprintable characters and stuff like that)
@@ -115,10 +115,10 @@ asio::awaitable<void> Server::listen(int port) {
         asio::ip::tcp::acceptor acceptor(ioc, asio::ip::tcp::endpoint(asio::ip::tcp::v4(), port));
         // Handle all incoming connections
         while (true) {
-            auto client = std::make_shared<Client>(ioc, clients);
+            auto client = std::make_shared<Client>(ioc, global);
             co_await acceptor.async_accept(client->getSocket(), asio::use_awaitable);
             asio::co_spawn(ioc, client->connect(), asio::detached);
-            clients.push_back(std::move(client));
+            global.clients.push_back(std::move(client));
         }
     } catch (std::exception& e) {
         std::cerr << "Failed to stay alive: " << e.what() << std::endl;

server/Connection.hpp

@@ -15,10 +15,17 @@
 
 
 
+namespace World {
+class Playfield;
+}
+
 namespace Connection {
 using namespace boost;
 
 
+class Global;
+
+
 struct Packet {
     Generic::Operation op;
     std::unique_ptr<google::protobuf::Message> data = nullptr;
@@ -42,13 +49,13 @@ struct FatalError : public Error {
 
 
 class Client : public std::enable_shared_from_this<Client> {
-    std::vector<std::shared_ptr<Client>>& clients;
+    Global& global;
     asio::ip::tcp::socket socket;
     std::string username;
     bool good = true;
 
 public:
-    Client(asio::io_context& ioc, std::vector<std::shared_ptr<Client>>& clients) : socket(ioc), clients(clients) {}
+    Client(asio::io_context& ioc, Global& global) : socket(ioc), global(global) {}
 
     asio::awaitable<Packet> receivePacket();
     asio::awaitable<void> sendProtobuf(const google::protobuf::Message& buffer);
@@ -98,9 +105,15 @@ public:
 };
 
 
+struct Global {
+    std::vector<std::shared_ptr<Client>> clients;
+    std::vector<std::shared_ptr<World::Playfield>> playfields;
+};
+
+
 class Server {
     asio::io_context& ioc;
-    std::vector<std::shared_ptr<Client>> clients;
+    Global global;
 
 public:
     Server(asio::io_context& ioc) : ioc(ioc) {}

server/World.cpp

@@ -1,8 +1,8 @@
 #include "World.hpp"
+#include "Connection.hpp"
 
 #include <string>
 #include <chrono>
-#include <cmath>
 
 
 

server/World.hpp

@@ -1,14 +1,18 @@
 #ifndef _WORLD_HPP
 #define _WORLD_HPP
 #include "generic.pb.h"
-#include "Connection.hpp"
 
 #include <vector>
 #include <memory>
+#include <cmath>
 #include <boost/asio/awaitable.hpp>
 
 
 
+namespace Connection {
+class Client;
+}
+
 namespace World {
 void fillSettings(Generic::Settings& settings);