Тема, которую я нахожу, заключается в том, что сетевой код на самом деле не тестируется модулем и должен быть изолирован от кода, который вы хотите протестировать (например, объект Player).Мне кажется, вы тесно связали Player с ClientConnection, что может сделать его менее тестируемым.Я также думаю, что связывание Игрока с его соединением, вероятно, нарушает SRP, поскольку у них очень четкие обязанности.
Я написал систему, которая кажется похожей на ту, которую вы хотите достичь, и я сделал это путемпривязка проигрывателя к соединению только через делегатов и интерфейсы.У меня есть отдельная библиотека классов World и Server и третий проект, Application, который ссылается на них и создает экземпляр Server и World.Мой объект Player находится в Мире, и Соединение на сервере - эти два проекта не ссылаются друг на друга, поэтому вы можете рассматривать Player без каких-либо условий сетевого взаимодействия.
Метод, который я использую, состоит в том, чтобы сделать SocketHandler абстрактными реализовать конкретный обработчик в проекте приложения.Обработчик имеет ссылку на объект мира, который может быть передан ему при создании.(Я делаю это с помощью фабричных классов - например, ClientConnectionFactory (в Приложении), где Сервер знает только об абстрактных Соединениях и Фабриках.)
public class ClientConnectionFactory : IConnectionFactory
{
private readonly World world;
public ClientConnectionFactory(World world) {
this.world = world;
}
Connection IConnectionFactory.Create(Socket socket) {
return new ClientConnection(socket, world);
}
}
Затем ClientConnection может пересылать ссылку World на свой обработчикнаряду с любой информацией, которую требует обработчик, о самом Соединении, например, о методах Send.Здесь я просто передаю сам объект Connection в обработчик.
public ClientConnection(Socket socket, World world)
: base(socket) {
this.handler = new ClientHandler(this, world);
...
}
Большая часть координации между игровой логикой и сетью содержится в ClientHandler и, возможно, в любых других дочерних объектах, на которые он ссылается.
class ClientHandler : SocketHandler
{
private readonly Connection connection;
private readonly World world;
public ClientHandler(Connection connection, World world) {
this.connection = connection;
this.world = world;
...
}
//overriden method from SocketHandler
public override void HandleMessage(Byte[] data) {
...
}
}
Остальное включает в себя ClientHandler, который создает свой объект Player, назначает делегатов или интерфейсы для обновления действий, а затем добавляет игрока в пул игроков в World.Первоначально я сделал это с помощью списка событий в Player, на который я подписываюсь с помощью методов в ClientHandler (который знает о Connection), но оказалось, что в объекте Player были десятки событий, которые стали беспорядочными для обслуживания.
Я выбрал вариант использования абстрактных уведомлений в проекте World для проигрывателя.Например, для перемещения у меня будет объект IMovementNotifier в Player.В приложении я бы создал ClientNotifier, который реализует этот интерфейс и выполняет соответствующую отправку данных клиенту.ClientHandler создаст ClientNotifier и передаст его объекту Player.
class ClientNotifier : IMovementNotifier //, ..., + bunch of other notifiers
{
private readonly Connection connection;
public ClientHandler(Connection connection) {
this.connection = connection;
}
void IMovementNotifier.Move(Player player, Location destination) {
...
connection.Send(new MoveMessage(...));
}
}
Можно изменить ctor ClientHandler для создания экземпляра этого уведомителя.
public ClientHandler(Connection connection, World world) {
this.connection = connection;
this.world = world;
...
var notifier = new ClientNotifier(this);
this.player = new Player(notifier);
this.world.Players.Add(player);
}
Таким образом, в результате получается система, в которойClientHandler отвечает за все входящие сообщения и события, а ClientNotifier обрабатывает все исходящие сообщения.Эти два класса довольно сложно протестировать, поскольку они содержат много другого дерьма.В любом случае, сеть не может быть проверена модулем, но объект Connection в этих двух классах может быть поддельным.Библиотека World полностью модульно тестируется без какого-либо учета сетевых компонентов, что я в любом случае хотел достичь в своем дизайне.
Это большая система, и я не очень подробно рассмотрел ее здесь, но надеюсьэто может дать вам несколько советов.Спросите меня, не хотите ли вы получить какую-то более конкретную информацию по этому поводу.
Если бы я мог дать еще какие-либо советы, я бы хотел избежать каких-либо статических или одиночных ударов - они вернутся, чтобы укусить вас.Подумайте об увеличении сложности, если вам нужно, как альтернатива, но документ, где вам нужно.Моя система может показаться сложной для сопровождающего, но она хорошо документирована.Для пользователя это особенно просто в использовании.Главное по сути
var world = new World();
var connectionFactory = new ClientConnectionFactory(world);
var server = new Server(settings.LocalIP, settings.LocalPort, connectionFactory);