[Piercing Blow] PROTOCOL_LOBBY_CHATTING_ACK

[Boris2105]

При попытке отправить сообщение в чат, ничего не происходит.
Хотя сервер все отправляет.

 
            public void WriteUnicode(string text, int count)
            {
                if (text != null)
                {
                    WriteB(Encoding.GetEncoding(1200).GetBytes(text));
                    WriteB(new byte[count - (text.Length)]);
                }
            }
 
Code: C#

 
public class PROTOCOL_LOBBY_CHATTING_ACK : ServerPacket
    {
        Player _player;
        ChatType _type;
        string _message;
        public PROTOCOL_LOBBY_CHATTING_ACK(Player player, ChatType type, string message)
        {
            _player = player;
            _type = type;
            _message = message;
        }
 
        public override void WriteImpl()
        {
            Logger.Instance.Info("Message:{0}, Name:{1}", _message, _player.Name);
            WriteD(1);
            WriteC(_player.Name.Length + 1);
            WriteUnicode(_player.Name, _player.Name.Length + 1);
            WriteC(0);
            WriteC(0);
            WriteH(_message.Length);
            WriteUnicode(_message, _message.Length);
 
Code: C#

[Awiion]

Boris2105,
Не верная структура.

[Boris2105]

Сообщение от Awiion

Boris2105,
Не верная структура.

Да вроде верная.

[Awiion]

Boris2105, Смотри лучше.
Во первых, в Unicode один символ, равен двум.

Во вторых, включи рассудок,

   WriteH(_message.Length);
   WriteUnicode(_message, _message.Length);

Write<ushort>(Text.Leneth)
WriteUnicode(Text);

У тебя первый указывает массив, второй столбец записывает.
Записываешь ты не правильно, WriteUnicode по этому используешь +1

  [OpCode(3087)]
    class SM_LOBBY_CHATTING : WriterPacket
    {
        private Player player;
        private ChatType type;
        private string text;

        public SM_LOBBY_CHATTING(Player player, ChatType type, string text)
        {
            this.player = player;
            this.type = type;
            this.text = text;
        }

        public override WriteImpl()
        {
            Write<int>(player.Id);
            Write<byte>(player.Name.Length);
            WriteUnicode(player.Name);
            Write<byte>(player.Color);
            Write<byte>(type);
            Write<ushort>(text.Length);
            WriteUnicode(text);
        }
    }

[Boris2105]

Сообщение от Awiion

Boris2105, Смотри лучше.
Во первых, в Unicode один символ, равен двум.

Во вторых, включи рассудок,

   WriteH(_message.Length);
   WriteUnicode(_message, _message.Length);

Write<ushort>(Text.Leneth)
WriteUnicode(Text);

У тебя первый указывает массив, второй столбец записывает.
Записываешь ты не правильно, WriteUnicode по этому используешь +1

  [OpCode(3087)]
    class SM_LOBBY_CHATTING : WriterPacket
    {
        private Player player;
        private ChatType type;
        private string text;

        public SM_LOBBY_CHATTING(Player player, ChatType type, string text)
        {
            this.player = player;
            this.type = type;
            this.text = text;
        }

        public override WriteImpl()
        {
            Write<int>(player.Id);
            Write<byte>(player.Name.Length);
            WriteUnicode(player.Name);
            Write<byte>(player.Color);
            Write<byte>(type);
            Write<ushort>(text.Length);
            WriteUnicode(text);
        }
    }

так там же используется ник и + 1 символ

[Sojang]

Сообщение от Awiion

Boris2105, Смотри лучше.
Во первых, в Unicode один символ, равен двум.

Во вторых, включи рассудок,

   WriteH(_message.Length);
   WriteUnicode(_message, _message.Length);

Write<ushort>(Text.Leneth)
WriteUnicode(Text);

У тебя первый указывает массив, второй столбец записывает.
Записываешь ты не правильно, WriteUnicode по этому используешь +1

  [OpCode(3087)]
    class SM_LOBBY_CHATTING : WriterPacket
    {
        private Player player;
        private ChatType type;
        private string text;

        public SM_LOBBY_CHATTING(Player player, ChatType type, string text)
        {
            this.player = player;
            this.type = type;
            this.text = text;
        }

        public override WriteImpl()
        {
            Write<int>(player.Id);
            Write<byte>(player.Name.Length);
            WriteUnicode(player.Name);
            Write<byte>(player.Color);
            Write<byte>(type);
            Write<ushort>(text.Length);
            WriteUnicode(text);
        }
    }

оффтоп:

Свернуть ↑
У вас один сервер на весь форум что ли?
В какую тему по пб не смотрю везде код у друг друга похож
Что у PROGRAMMATOR что у Awiion

Свернуть ↑Развернуть ↓

[PROGRAMMATOR]

partyvan, Write<T> юзаем, этим и похожи.
Я не юзаю модификаторы private и атрибутов [OpCode] у меня нет.

[Sojang]

Сообщение от PROGRAMMATOR

partyvan, Write<T> юзаем, этим и похожи.
Я не юзаю модификаторы private и атрибутов [OpCode] у меня нет.

Если память не подводить раньше был у вас атрибут [Packet()] или что то вроде того

[PROGRAMMATOR]

Да, но уже давным давно всё переехало в отдельный enum:

namespace PiercingBlow.Common.Network
{
    public enum PacketOpcode : ushort
    {
        PROTOCOL_BASE_LOGIN_REQ = 257,
        PROTOCOL_BASE_LOGIN_ACK = 259,
        PROTOCOL_BASE_CONNECT_ACK = 514,
        PROTOCOL_BASE_LOGOUT_REQ = 515,
        PROTOCOL_BASE_LOGOUT_ACK = 516,
 
        // ...
    }
}
 
Code: C#

И теперь на лету подхватывает нужны опкод.

[Sojang]

Сообщение от PROGRAMMATOR

Да, но уже давным давно всё переехало в отдельный enum:

namespace PiercingBlow.Common.Network
{
    public enum PacketOpcode : ushort
    {
        PROTOCOL_BASE_LOGIN_REQ = 257,
        PROTOCOL_BASE_LOGIN_ACK = 259,
        PROTOCOL_BASE_CONNECT_ACK = 514,
        PROTOCOL_BASE_LOGOUT_REQ = 515,
        PROTOCOL_BASE_LOGOUT_ACK = 516,
 
        // ...
    }
}
 
Code: C#

И теперь на лету подхватывает нужны опкод.

Ох, так же не удобно править в случае чего.ну по крайней мере я так считаю)