Оцените и посоветуйте как можно улучшить код Крестики-нолики

Текстовый протокол для бота (ASCII):
1.  каждое сообщение завершается символом переводом строки '\n'
2.  первое сообщение каждому содержит через один пробел
    - ширину игрового поля (целое число),
    - высоту игрового поля (целое число),
    - цвет фишки (символ 'O' или 'Х')
3.  каждое последующее сообщение содержит два целых числа
    - ход противника (от сервера)
    - твой ход (от игрока)
4. ???
 
Псевдокод для бота:
 
#include <iostream>
int main()
{
    unsigned short x, y;
    char c;
    // считываем первое сообщение
    if (!(std::cin >> x >> y >> c))
        return -1;
    // инициализация состояния игры
    initialize_state(x, y, c);
 
    for (;;)
    {
        switch (c)
        {
            case 'O': // первый ход за протвником
                c = 'X';
                // считываем ход противника
                if (!(std::cin >> x >> y))
                    return -1;
                    
                break;
            default:
                // отправляем наш ход
                std::cout << x << ' ' << y << '\n';
                // считываем ход противника
                if (!(std::cin >> x >> y))
                    return -1;
        }
        if (do_step(&x, &y) == false)
          break; // если проиграли
    }
    return 0;
}

/* arbiter.hpp */
#include <message.hpp>
 
namespace service::tic_tac_toe
{
  class player;
  class listener;
 
  class arbiter
  {
  public:
    // инициализация - две ссылки на игроков и, возможно,
    // одного слушателя игры
    arbiter(player *first, player *second, listener *subscriber);
 
    // начало игры:
    // утановка таймаутов игрокам (player::set_timeout),
    // установка ссылки на себя для обратной связи (player::set_arbiter)
    // отправка первых сообщений игрокам (тому, что ходит первым, reply == true)
    void start();
 
    // этот метод будет дергать игрок, доставляя сообщение от игрока
    // нужен ли дополнительный аргумент - указатель на игрока?
    // я думаю нет, игра пошаговая, но если нужно для твоей логики, добавь
    void on_player_step(message const &msg);
      
    // ...
  };
} // namespace service::tic_tac_toe

/* listener.hpp */
#include <message.hpp>
 
// ОПЦИОНАЛЬНО!!!
// интерфейс слушателя, это второстепенно пока,
// позже понадобится,
// но можешь реализовать простенкого слушателя,
// который будет печатать ход игры в консоль, например
// (реализация на твое усмотрение, только не завязывайся на winapi)
 
namespace service::tic_tac_toe
{
  class player;
 
  // сообщение от игрока арбитру
  struct player_reply
  {
    message msg;
    player *sender;
  };
 
  // соообщение от арбитра игроку
  struct arbiter_request
  {
    message msg;
    player *receiver;
  };
 
  // добавь что-то еще
 
  using event = ::std::variant<player_reply, arbiter_request>;
 
  class listener
  {
  public:
    virtual void notify(event const &e) = 0;
  };
 
} // namespace service::tic_tac_toe

/* message.hpp */
#include <variant>
 
namespace service::tic_tac_toe
{
  using ushort_t = unsigned short;
 
  enum class color_t : ushort_t
  {
    zero,
    cross,
  };
 
  // начальное сообщение игроку
  struct start_t
  {
    ushort_t field_width;
    ushort_t field_height;
    color_t color;
  };
 
  // ход
  struct step_t
  {
    ushort_t x;
    ushort_t y;
  };
 
  // причина завершения игры
  enum class finish_reason_t : ushort_t
  {
    // вышло время ожидания
    timed_out, 
 
    // ошибка ввода вывода,
    // пока посылается только от игрока арбитру
    // в случае системного сбоя или ошибки парсера,
    // но можно и игроку отправлять, если он не так пошел
    // (боту это не нужно, его мы убиваем в этом случае)
    io_error,
 
    // это тоже опционально, как сообщения игроку,
    // бот обойдется, можешь забить
    winner,
    looser,
    draw,
  };
 
  using message = ::std::variant<start_t, step_t, finish_reason_t>;
 
} // namespace service::tic_tac_toe
 
 
 
/* player.hpp */
#include <message.hpp>
#include <chrono>
 
namespace service::tic_tac_toe
{
  class arbiter;
 
  // минимальный интерфейс, с которым должен взаимодействовать твой арбитр
  class player
  {
  public:
    using timeout_type = ::std::chrono::milliseconds;
 
    // управление таймаутом для обмена сообщением
    // по истечении которого, если игрок не отвечает,
    // арбитру приходит сообщение finish_reason_t::timed_out
    void set_timeout(timeout_type timeout) noexcept;
    timeout_type get_timeout() const noexcept;
 
    // для обратной связи
    void set_arbiter(arbiter *owner) noexcept;
    arbiter *get_arbiter() const noexcept;
 
    // отправка сообщения игроку,
    // если reply == true, то на сообщение требуется ответ
    // (я не хочу завязывать анализ сообщения в логику транспорта,пусть будет флаг)
    void send_message(message const &msg, bool reply);
 
    //...
  };
 
} // namespace service::tic_tac_toe
 
 
/* utility.hpp */
 
#include <message.hpp>
#include <iostream>
 
// возможно тебе пригодится для реализации слушателя(печати лога в консоль)
namespace service::tic_tac_toe
{
  inline char to_char(color_t color) noexcept
  {
    return "OX"[static_cast<ushort_t>(color)];
  }
 
  inline char to_char(finish_reason_t reason) noexcept
  {
    return "TEWLD"[static_cast<ushort_t>(reason)];
  }
 
  inline ::std::ostream &operator<<(::std::ostream &out, start_t const &msg)
  {
    return out << msg.field_width << ' '
               << msg.field_height << ' '
               << to_char(msg.color);
  }
 
  inline ::std::ostream &operator<<(::std::ostream &out, step_t const &msg)
  {
    return out << msg.x << ' ' << msg.y;
  }
 
  inline ::std::ostream &operator<<(::std::ostream &out, finish_reason_t msg)
  {
    return out << to_char(msg);
  }
 
  inline ::std::istream &operator>>(::std::istream &in, step_t &msg)
  {
    return in >> msg.x >> msg.y;
  }
 
  inline ::std::ostream &operator<<(::std::ostream &out, message const &msg)
  {
    return ::std::visit(
        [&out](auto const &value) -> decltype(auto)
        { return out << value; },
        msg);
  }
} // namespace service::tic_tac_toe

/* arbiter.hpp */
#include <message.hpp>
 
namespace service::tic_tac_toe
{
  class player;
  class listener;
 
  class arbiter
  {
  public:
    // инициализация - две ссылки на игроков и, возможно,
    // одного слушателя игры
    arbiter(player *first, player *second, listener *subscriber);
 
    // начало игры:
    // утановка таймаутов игрокам (player::set_timeout),
    // установка ссылки на себя для обратной связи (player::set_arbiter)
    // отправка первых сообщений игрокам (тому, что ходит первым, reply == true)
    void start()
    {
      players[0]->set_timeout(...);
      players[0]->set_arbiter(this);
      players[0]->send_message(start_t{3, 3, cross}, true); // current
 
      players[1]->set_timeout(...);
      players[1]->set_arbiter(this);
      players[1]->send_message(start_t{3, 3, zero}, false);
    }
 
    // этот метод будет дергать игрок, доставляя сообщение от игрока
    // нужен ли дополнительный аргумент - указатель на игрока?
    // я думаю нет, игра пошаговая, но если нужно для твоей логики, добавь
    void on_player_step(message const &msg)
    {
      message next = update_state(msg)
      if (not_finish(next)) // если продолжаем
      {
        current ^= 1; // переключаем игрока
        players[current]->send_message(next, true);
        return;
      }
      subscriber->notify(next); // уведомляем о завершении игры
    }
 
 
    listener *subscriber;
    player *players[2];
    int current; // = 0 текуший игрок
    game_state state; // состояние
  };
} // namespace service::tic_tac_toe