Пакетики да сумочки

[GoogleTan]

Есть у меня пакет:

public class MovingPacket implements IMessage, IMessageHandler<MovingPacket, MovingPacket>
{
    private ByteBuf buf;
    private int key;

    public MovingPacket()
    {
       key = -1;
    }

    public MovingPacket(int keyIn)
    {
        key = keyIn;
    }

    @Override
    public MovingPacket onMessage(MovingPacket message, MessageContext ctx)
    {
        if (ctx.side.isServer())
            message.server(ctx.getServerHandler().playerEntity, key);
        else
            message.client(clientPlayer());
        return null;
    }

    public void client(EntityPlayer player)
    {
    }

    public void server(EntityPlayerMP player, int keyIn)
    {
        if(player.ridingEntity instanceof MinitrixEntity)
        {
            MinitrixEntity entity = (MinitrixEntity) player.ridingEntity;
            switch (keyIn) {
                case 3:
                    entity.motionX -= 1;
                    break;
                case 4:
                    entity.motionZ -= 1;
                    break;
                default:
                    entity.motionY += 1;
            }
        }
    }

    @Override
    public final void fromBytes(ByteBuf buf)
    {
        try{
            key = buf.readInt();
        }catch (Exception e)
        {
            e.printStackTrace();
            key = -1;
        }
    }

    @Override
    public final void toBytes(ByteBuf buf)
    {
        if(buf == null)
            buf = Unpooled.buffer();
        try {
            buf.writeByte(key);
        }catch (Exception e){
            e.printStackTrace();
        }
    }

    @SideOnly(Side.CLIENT)
    private EntityPlayer clientPlayer()
    {
        return Minecraft.getMinecraft().thePlayer;
    }
}

Хендлер:

public final class NetworkHandler {
    private static short id;

    /**
     *  ChannelName - название канала. Канал должен быть один на весь мод, создавать дополнительные каналы не нужно!
     */
    public static SimpleNetworkWrapper NETWORK;

    public static void initNetwork()
    {
        NETWORK = NetworkRegistry.INSTANCE.newSimpleChannel("HYDRA");
        register(MovingPacket.class, Side.SERVER);
    }
    /**
     * Получение дистанции от определённой точки в мире.
     * @param world - мир
     * @param updateDistance - радиус в котором будет действовать пакет
     * @return позиция
     */
    public static void sendKeysToServer(MovingPacket packet) {
        NETWORK.sendToServer(packet);
    }

    /**
     * Регистрация пакета через один метод.
     * @param packet - класс пакета
     * @param side - сторона (клиент/сервер)
     */
    private static void register(Class<? extends MovingPacket> packet, Side side) {
        try {
            NETWORK.registerMessage(packet.newInstance(), packet, id++, side);
        } catch (Exception e) {
            e.printStackTrace();
        }
    }
}

И эррор. Что делаю я не так?

 

[tox1cozZ]

Зачем у тебя поле private ByteBuf buf? Убери его, он у тебя есть в аргументах методов.

 

[GoogleTan]

@tox1cozZ Так он нигде не используется. В аргументах методов свой буфф из которого читаю и в который пишу. Просто названия совпали

 

[ロロロ]

writeByte

readInt

Запишу байт, прочитаю инт, посмотрим, кто кого перечитает!

P.S. Мне кажется, проверка на null в врайте и try/catch лишние.
Никогда не юзал, все хорошо работает.

@ロロロ

 

[RareScrap]

1) Зачем в register() ты делаешь newInstance()? NETWORK.registerMessage принимает класс обработчика и класс пакета, а не инстансы этих классов.
2) Твой Message реализует и IMessage, и IMessageHandler. Это логически неверно. Как посылка может быть получателем самой себя? Имхо, это вносит чрезмерную путаницу.
3) в дженерике IMessageHandler<MovingPacket, MovingPacket>первый тип является классом пакета, которые принимается хандлером, а второй - типом который отсылается назад. Т.к. ты шлешь назад null то сделай второй тип просто IMessage, чтобы путаницу не усиливать.
4) @ロ[B]ロ[B]ロ [/B][/B]верно говорит: ты пишешь байт, а читаешь инт. втф? (И да, чувак, смени уже ник с этих ущербных квадратиков. Вообще не понятно что ты за зверь. Ты это сделал чтобы пипл тебя не упоминал в постах?)
5) Зачем ты создаешь новый буфер когда входной null? Он ведь не может быть равен null. Ты где-то дергаешь toBytes() самостоятельно? Ну так и сделай null-проверку до вызова toBytes().

И у тебя почти каждый метод в try'е. Твой код выглядит так, как будто его склеили изолентой.

 

[ロロロ]

Ты это сделал чтобы пипл тебя не упоминал в постах?)

Спокойно можно корректно сделать упоминание. @ロロロ

 

[GoogleTan]

1) Зачем в register() ты делаешь newInstance()? NETWORK.registerMessage принимает класс обработчика и класс пакета, а не инстансы этих классов.

registerMessage(IMessageHandler<? super REQ, ? extends REPLY> messageHandler, Class<REQ> requestMessageType, int discriminator, Side side)

2) Твой Message реализует и IMessage, и IMessageHandler. Это логически неверно. Как посылка может быть получателем самой себя? Имхо, это вносит чрезмерную путаницу.

Наоборот. Весь пакет целиком в 1 классе. Везде где я видела ванильную систему так было. И в гайде на форуме тоже)))

 

[ロロロ]

message.server(ctx.getServerHandler().playerEntity, key);

И мне кажется, key нужно брать из message. Т.Е. message.key.
Потому чтоо... Хотя, хз, может и он отдает этот же объект.

Но, вызов переменной server берется из message, а не из this. )

 

[RareScrap]

Везде где я видела ванильную систему так было

В ванильных пакетах не содержатся методы, реализующие логику пакета. Там только есть processPacket() который дергает нужный метод в обработчике, которыми является NetHandlerPlayClient или NetHandlerPlayServer. При этом сам пакет не является своим обработчиком как таковым.

Наоборот. Весь пакет целиком в 1 классе

Ты можешь достичь такого же результата, создав Nested-класс, реализующий IMessageHadler. Тогда весь код пакета будет находится в одном файле, но при этом сам пакет и его обработчик будут разными сущностями, как и должно быть.

И в гайде на форуме тоже)))

Это не делает реализацию и месседжа, и хандлера в одном классе логически верным.

registerMessage(IMessageHandler<? super REQ, ? extends REPLY> messageHandler, Class<REQ> requestMessageType, int discriminator, Side side)

Там есть версия метода, которому достаточно передать классы. Это удобнее, чем создавать инстанс самому
void registerMessage(Class<? extends IMessageHandler<REQ, REPLY>> messageHandler, Class<REQ> requestMessageType, int discriminator, Side side)