Dlipko by dnlipko · Pull Request #1 · lostmap/CardGame

dnlipko · 2018-05-16T17:10:50Z

Обмен данными между сервером и клиентом происходит через xml файлы

Клиент подключается к серверу и отправляет xml с логином и паролем
(abcd12345)
Клиент отправляет на сервер команду о желании начать партию
(1)

-Клиент получает данные: победа, количество побед в раунде, его ли очередь хода, карты в руке, карты на поле, количество карт в руке соперника

Об игре

Игра длиться 2 или 3 раунда в зависимости от того кто победит в 1 и 2 раунде. В игре побеждает тот кто быстрее наберет 2 победы в раундах. 3ий раунд играется, если через 2 раунда ничья.
В раунде побеждает тот кто наберет больше всего очков.
Очки складываются из силы карт лежащих на поле
Раунд завершается когда пасанут оба игрока.
-После того как пасанет один игрок, другой может продолжать кидать карты пока он сам не пасанет.
-За ход можно сыграть одну карту из руки (не считая карт со способностями, которые могут вытягивать на поле другие карты)
-В каждом раунде поле обновляется(не считая карт со специальных карт со способностями)
-Колода одна на всю игру
-Карты в руке и колода при перехода от раунда к раунду не изменяются

Карты(будет дополняться, тестовая версия)
Есть три типа карт:
-Сущность (обладает силой)
Скорее всего от сущности будет зависеть её сила
-Сущность со способностью(обладает силой и способностью)
-Способность (Какие-нибудь карты погоды или что-нибудь подобное)
3.1. buff Удваивает свою силу и силу карты на поле с соответствующей сущностью (Для сущности со способностью)
3.2. spy карта кладется на стол противника, но в руку получаете +2 карты (Для сущности со способностью)
3.3 rollcall вытягивает подобную сущность из колоды вместе с собой на поле (Для сущности со способностью)

Воспользовался паттерном Factory Method. Но генерация карт пока не продуманная. Планирую вынести генерацию карт в отдельный класс. Пока что все в юзере.
Перенес генерацию xml(qdomelement) в сами классы.

Есть возможность подключится к серверу. Он объединяет в парочки каждых двух подключившихся. Потом если один из них пишет на сревер, он отправляет это обратно отправителю и его противнику по игре.

user may login or signep by sending xml <user><signup><login>dlipko</login><password>12345</password></signup></user> or <user><login><login>dlipko</login><password>12345</password></login></user>

Добавил коментарии. Генерации карт нет. Из карт пока только студет. Свойств у карт пока тоже нет.

Добавил карты сущности со свойствами, просто сущности и карты со свойствами. Воспользовался паттерном Factory Method. Но генерация карт пока не продуманная. Планирую вынести генерацию карт в отдельный класс. Пока что все в юзере. Перенес генерацию xml(qdomelement) в сами классы.

Накидал пару классов на клиенте, разделил логигу сервера промежуточного слогя и игры. Планирую парсить данные и создавать объекты // сравнивать их с существующими. И уже слой игровой логики прикрепить к GUI

leshiy1295

сейчас самая большая проблема - большая привязка к Qt

leshiy1295 · 2018-05-17T08:12:00Z

client/abstractcard.cpp

@@ -0,0 +1,17 @@
+#include "abstractcard.h"
+
+AbstractCard::AbstractCard(int id, QString info):


отделите код от Qt, где в этом нет необходимости

leshiy1295 · 2018-05-17T08:12:34Z

client/abstractcard.h

+#ifndef ABSTRACTCARD_H
+#define ABSTRACTCARD_H
+
+#include <QString>


Qt - это реализация, поэтому постарайтесь не включать без необходимости его библиотеки в .h файлах

leshiy1295 · 2018-05-17T08:13:30Z

client/client.cpp

+{
+    _socket = new QTcpSocket(this);
+
+    _socket->connectToHost("127.0.0.1", 1234);


это необходимо пробрасывать через конструктор и по-хорошему брать либо из argv, либо из конфигурационных файлов

leshiy1295 · 2018-05-17T08:14:04Z

client/client.cpp

+
+void Client::disconnected()
+{
+    qDebug() << "disconnected...";


лучше создать отдельную сущность логгера и пользоваться ей через единый интерфейс везде, в main же создавать реализацию на Qt

leshiy1295 · 2018-05-17T08:14:15Z

client/client.cpp

+
+    QByteArray data;
+    data = _socket->readAll();
+    while (!data.contains("</server>") && _socket->waitForReadyRead()) {


кажется, что это хардкод

leshiy1295 · 2018-05-17T08:44:22Z

server/player.cpp

+
+    for (int i= 4; i < HEAND_SIZE; ++i)
+    {
+        newDeck->addCard(new EntitySpy(i, "Woodcutter Spy", Woodcutter, qrand() % MAX_STRENGTH + 1));


не нужно использовать Qt без необходимости

leshiy1295 · 2018-05-17T08:44:29Z

server/player.cpp

+
+void Player::setWin()
+{
+    _win = 1;


хардкод

leshiy1295 · 2018-05-17T08:45:03Z

server/player.cpp

+
+QDomElement Player::toQDomElement(QString playerName)
+{
+    QDomDocument document;


методы сериализации стоит вынести в отдельный интерфейс, от которого наследовать Player

leshiy1295 · 2018-05-17T08:45:18Z

server/player.h

+    Deck *_deckGenerator();
+    Deck *_heandGenerator();
+
+    QDomElement _domElement(QString elementName, int value);


вынести Qt из реализации

leshiy1295 · 2018-05-17T08:45:31Z

server/user.h

@@ -0,0 +1,25 @@
+#ifndef USER_H


Полностью избавился от привязки к qt там где в этом нет необходимости. Добавил конфигурационный файл и логгер. Постарался воспользоваться умными указателями. По возможности исправил все указанные недочеты.

leshiy1295

Намного лучше
Нужно немного доправить архитектуру

leshiy1295 · 2018-06-16T11:59:58Z

server/configparser.h

+    std::map<PROPERTY_TYPE, std::string> getPropertiesInfo() const;
+
+private:
+  pugi::xml_document doc;


зависимость осталась в header-файле - нехорошо
в качестве решения можно предложить использовать идиому PIMPL

leshiy1295 · 2018-06-16T12:01:30Z

server/game.cpp

+        if (!_waitingForCouple.empty()){
+
+            // создает игроков
+            auto player1 = std::shared_ptr<Player>(new Player(_getUser(_waitingForCouple)));


есть make_shared

leshiy1295 · 2018-06-16T12:03:22Z

server/interlayer.cpp

+InterLayer::~InterLayer() {
+
+    for (auto socket: _socket)
+        delete socket.second;


удобнее использовать умные указатели

Уже несколько раз пытался использовать с сокетами shared_ptr. Каждый раз появляется не понятная проблема, с которой мне пока что так и не удалось разобраться.

leshiy1295 · 2018-06-16T12:03:36Z

server/interlayer.h

+    std::string _parseGetStr3(pugi::xml_document &message,std::string ancestor1,
+                              std::string ancestor2, std::string parent) const;
+
+    bool _signIn(pugi::xml_document &message, Socket* client);


leshiy1295 · 2018-06-16T12:07:53Z

server/socket.h

+
+#include <string>
+
+class Socket: public QTcpSocket


нужно сделать Socket абстрактным
и QSocket уже Ваш, который будет наследоваться от Socket и от QTcpSocket

leshiy1295 · 2018-06-16T12:09:32Z

server/game.h

+#include <memory>
+
+
+class Game: QDomSerializer


немного не так - Вы сделали Game QT-классом
нужно сделать абстрактный интерфейс Serializer, добавить поле Parser внутрь Game, передавать ему QtParser, который уже будет сериализовывать

Использовал идиому PIMPL для pugi. Сделал Socket абстрактным классом. Создал класс QSocket, который унаследовал от Socket и от QTcpSocket. Сделать абстрактный интерфейс Serializer, добавил поле Parser внутрь Game, в который передаю QDomSerializer.

leshiy1295 · 2018-06-17T09:45:03Z

server/configparser.h


 private:
-  pugi::xml_document doc;
+    std::shared_ptr<pugi::xml_document> doc;


это не PIMPL..
PIMPL - это когда тут Вы оставляете указатель на класс реализации...
А в нём уже подключаете этот файл.
Из этого же класса Вы проксируете все запросы в класс реализации

leshiy1295 · 2018-06-17T09:45:41Z

server/game.cpp

            // создает игроков
-            auto player1 = std::shared_ptr<Player>(new Player(_getUser(_waitingForCouple)));
-            auto player2 = std::shared_ptr<Player>(new Player(_getUser(login)));
+            auto player1 = std::make_shared<Player>(_getUser(_waitingForCouple));


когда тип однозначно определён, то make_shared можно явно не инстанцировать - в этом вся соль)

leshiy1295 · 2018-06-17T09:47:22Z

server/server.h

+#include <QHostAddress>

-class Server: public QTcpServer
+class Server


раз это сервер, который не привязан к Server, то у него не должно быть методов с QByteArray - нужно просто с байтами. В Вашем же классе Вы просто преобразуете QByte к обычным (ну или наоборот)

dnlipko added 5 commits April 20, 2018 00:19

user ligin

6039d19

user may login or signep by sending xml <user><signup><login>dlipko</login><password>12345</password></signup></user> or <user><login><login>dlipko</login><password>12345</password></login></user>

рабочий вариант

e427a7a

Добавил коментарии. Генерации карт нет. Из карт пока только студет. Свойств у карт пока тоже нет.

delete this sh**

67e1add

dnlipko added the enhancement New feature or request label May 16, 2018

dnlipko requested a review from leshiy1295 May 16, 2018 17:10

dnlipko added 2 commits May 17, 2018 03:42

Merge branch 'dlipko' of https://github.com/lostmap/CardGame into dlipko

d38bc53

leshiy1295 requested changes May 17, 2018

View reviewed changes

dnlipko added 2 commits June 16, 2018 16:31

delete lod file

b5186f8

leshiy1295 requested changes Jun 16, 2018

View reviewed changes

dnlipko added 2 commits June 17, 2018 14:14

Merge branch 'dlipko' of https://github.com/lostmap/CardGame into dlipko

99439f5

leshiy1295 requested changes Jun 17, 2018

View reviewed changes

		@@ -0,0 +1,17 @@
		#include "abstractcard.h"

		AbstractCard::AbstractCard(int id, QString info):

Conversation

dnlipko commented May 16, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

leshiy1295 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

leshiy1295 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

dnlipko commented May 16, 2018 •

edited

Loading