Вопрос к OOG Developer'ам

[Jenkis]

Добрый вечер!

Собственно пару вопросов к разработчикам
Итак, постепенно забываю c# и перехожу на java/C++ ну...это в общем мало кого волнует. Но время от времени что-то пытаюсь писать

С ООП Я на Вы, да и с алгоритмическим мышлением беда.Хотел спросить , что добавить , что убрать , что переделать.

Собственно сорсы, даже стыдно

Client.cs

public class Client
    {
        private readonly string username;
        private readonly string password;
        private readonly string host;
        private readonly int port;
        private readonly bool strengthen;

        public Client(string username, string password, string host, bool strengthen)
            : this(username, password, host, 29000 , strengthen) { }

        public Client(string username, string password, string host, int port, bool strengthen)
        {
            this.username = username;
            this.password = password;
            this.host = host;
            this.port = port;
            this.strengthen = strengthen;
        }

        public string UserName
        {
            get
            {
                return this.username;
            }
        }

        public string PassWord
        {
            get
            {
                return this.password;
            }
        }

        public string Host
        {
            get
            {
                return this.host;
            }
        }

        public int Port
        {
            get
            {
                return this.port;
            }
        }

        public bool Strengthen
        {
            get
            {
                return strengthen;
            }
        }

Использую для установки данных для входа и подключения.

LoginInWorld.cs

    public class LoginInWorld
    {
        private TCPClient socketClient;
        private QueueSend qsend;
        private QueueReceive qrecv;

        public LoginInWorld(Client client)
        {
            this.socketClient = new TCPClient(client.Host, client.Port);
            if (this.socketClient.Connect())
            {
                ReceivePacket();
            }
        }

        private void ReceivePacket()
        {
            do
            {
                byte[] unknownPacket = this.socketClient.Receive();
                if (unknownPacket != null && unknownPacket.Length > 0)
                {
                    this.qrecv = new QueueReceive(unknownPacket , false);
                }
                else
                {
                    break;
                }
            }
            while (this.socketClient.IsConnected());
        }

Использую для установки сокета, и получение пакетов для дальнейшего разбора.

QueueReceive.cs

public class QueueReceive
    {
        private static Queue<byte[]> recvList = new Queue<byte[]>();
        private PacketContainer pc;

        public QueueReceive(byte[] recvData , bool savePacket)
        {
            if (savePacket)
            {
                recvList.Enqueue(recvData);
            }
            this.pc = new PacketContainer(recvData);
        }
    }

Сделал очередь, так, на случай ведение лога.
По аналогии с QueueSend.cs

PacketContainer.cs

public class PacketContainer : BuildAssist
    {
        public PacketContainer(byte[] recvData)
        {
            switch (recvData[0])
            {
                case 0x01:
                    {
                        _0x01.FirstServerKey = extractionValue(16, recvData, 3);
                        _0x01.serverVersion = extractionValue(3, recvData, 20);
                        break;
                    }
            }
        }

        struct _0x01
        {
          public static byte[] FirstServerKey = new byte[16];
          public static byte[] serverVersion = new byte[3];
        }
    }

Класс контейнер, использую для извлечение нужных данных.
Подумал, что структура для каждого серверного пакета будет не плохо.
Собственно тут у меня ступор, не знаю как дальше, и стоит ли вообще.Мб лучше все переделать

BuildAssist - Класс помощник в сборке пакетов

Ну, как-то так
Внимательно слушаю вас

[krukovis]

Цитата: Сообщение от Jenkis Хотел спросить , что добавить

Добавить нужно комментарии. К каждой строчке.

Добавлено через 3 минуты

Цитата: Сообщение от Jenkis Хотел спросить , что добавить

И нужно еще добавить сформулированный вопрос. Ибо на сформулированный и узкий вопрос - с большей вероятностью можно получить сформулированный ответ.

[vogel]

Цитата: Сообщение от Jenkis Хотел спросить , ... , что переделать.

Для начала - переделайте вопрос, а то мы все тут "телепатию" в школе прогуливали.

[FreePVP)))]

Jenkis, почитайте описание моей старой оог-библиотеки
http://zhyk.ru/forum/showthread.php?...57#post2846157