unify commands

src/commands/Command.cpp

@@ -9,11 +9,11 @@ Command *Command::create(int clientIndex, yojimbo::Message *message)
     switch (message->GetType())
     {
     case (int)GameMessageType::TEST:
-        return new TestCommand(clientIndex, message);
+        return new TestCommand(clientIndex, ((TestMessage *)message)->m_data);
     case (int)GameMessageType::SETNAME:
-        return new SetNameCommand(clientIndex, message);
+        return new SetNameCommand(clientIndex, std::string(((SetNameMessage *)message)->buffer));
     default:
-        std::cout << "Unknown message type received!" << std::endl;
+        std::cout << "Unknown message type received over network!" << std::endl;
         return nullptr;
     }
 }
@@ -22,16 +22,16 @@ Command *Command::create(std::string command)
 {
     std::stringstream ss(command);
     std::string token;
-            ss >>
+    ss >>
         token;
     if (token == "stop")
     {
-        return new StopCommandInternal();
+        return new StopCommand();
     }
     else if (token == "test")
     {
-        return new TestCommandInternal();
+        return new TestCommand(-1, 100);
     }
-    std::cout << "Internal commando don't exists: " << command << std::endl;
+    std::cout << "Local command don't exists: " << command << std::endl;
     return nullptr;
 }

src/commands/Command.h

@@ -24,9 +24,9 @@ public:
 class TestCommand : public Command
 {
 public:
-    TestCommand(int clientIndex, yojimbo::Message *message)
+    TestCommand(int clientIndex, int data)
         : m_clientIndex(clientIndex), 
-        m_data(((TestMessage *)message)->m_data) 
+        m_data(data) 
         {}
 
     virtual void execute(GameServer *server) override;
@@ -39,9 +39,9 @@ private:
 class SetNameCommand : public Command
 {
 public:
-    SetNameCommand(int clientIndex, yojimbo::Message *message)
+    SetNameCommand(int clientIndex, std::string name)
         : m_clientIndex(clientIndex), 
-        m_name(std::string(((SetNameMessage *)message)->buffer)) 
+        m_name(name) 
         {}
 
     virtual void execute(GameServer *server) override;
@@ -51,15 +51,9 @@ private:
     std::string m_name;
 };
 
-// --- Classes Internal:
+// --- Classes local:
 
-class TestCommandInternal : public Command
-{
-public:
-    virtual void execute(GameServer *server) override;
-};
-
-class StopCommandInternal : public Command
+class StopCommand : public Command
 {
 public:
     virtual void execute(GameServer *server) override;

src/commands/StopCommand.cpp

@@ -1,6 +1,6 @@
 #include "Command.h"
 
-void StopCommandInternal::execute(GameServer *server) 
+void StopCommand::execute(GameServer *server) 
 {
     server->m_running = false;
 }

src/commands/TestCommand.cpp

@@ -4,9 +4,14 @@
 
 void TestCommand::execute(GameServer *server)
 {
-    // ... process test message ...
-    entityx::Entity entity = server->serverToEntityX.at(m_clientIndex);
-    entityx::ComponentHandle<NetworkPlayerProperties> playerproperties = entity.component<NetworkPlayerProperties>();
-
-    std::cout << "Test message received from client " << playerproperties->name << ":" << m_clientIndex << " with data: " << m_data << std::endl;
+    if (m_clientIndex == -1)
+    {
+        std::cout << "Test message received from server cmd :" << m_clientIndex << " with data: " << m_data << std::endl;
+    }
+    else
+    {
+        entityx::Entity entity = server->serverToEntityX.at(m_clientIndex);
+        entityx::ComponentHandle<NetworkPlayerProperties> playerproperties = entity.component<NetworkPlayerProperties>();
+        std::cout << "Test message received from client " << playerproperties->name << ":" << m_clientIndex << " with data: " << m_data << std::endl;
+    }
 }

src/commands/TestCommandInternal.cpp

@@ -1,8 +0,0 @@
-#include "Command.h"
-
-#include <iostream>
-
-void TestCommandInternal::execute(GameServer *server) 
-{
-    std::cout << "Internal testcommand launched!" << std::endl;
-}