Работа с пакетами

CKATEPTb · 7 Янв 2019

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

Код:

public class MainHandRenderHandler
{

    private static class CachedInventory
    {

        NonNullList<ItemStack> stacks;
        int state;

        CachedInventory(int size)
        {
            stacks = NonNullList.withSize(size, ItemStack.EMPTY);
            state = 0;
        }

    }

    public static boolean HideCosArmor = false;

    private static final boolean isBaublesLoaded = Loader.isModLoaded("baubles");

    private final LoadingCache<EntityPlayer, CachedInventory> cache = CacheBuilder.newBuilder().expireAfterAccess(60, TimeUnit.SECONDS).build(new CacheLoader<EntityPlayer, CachedInventory>()
    {

        @Override
        public CachedInventory load(EntityPlayer owner) throws Exception
        {
            return new CachedInventory(owner.inventory.mainInventory.size());
        }

    });

    @SubscribeEvent(priority = EventPriority.LOWEST, receiveCanceled = true)
    public void handleCanceledEvent(RenderPlayerEvent.Pre event)
    {
        if (!event.isCanceled())
            return;

        CachedInventory cachedInv = cache.getUnchecked(event.getEntityPlayer());
        NonNullList<ItemStack> cachedArmor = cachedInv.stacks;
        NonNullList<ItemStack> armor = event.getEntityPlayer().inventory.mainInventory;

        if (armor.size() > cachedArmor.size())
        {
            cache.invalidate(event.getEntityPlayer());
            return;
        }

        if (cachedInv.state != 0)
        {
            for (int i = 0; i < armor.size(); i++)
                armor.set(i, cachedArmor.get(i));

            if (isBaublesLoaded)
            {
                try
                {
                    IBaublesItemHandler bh = BaublesApi.getBaublesHandler(event.getEntityPlayer());
                    boolean block = bh.isEventBlocked();
                    bh.setEventBlock(true);
                    for (int i = 0; i < bh.getSlots(); i++)
                        bh.setStackInSlot(i, cachedArmor.get(9 + i));
                    bh.setEventBlock(block);
                }
                catch (Throwable ignored)
                {
                }
            }

            cachedInv.state = 0;
        }
    }

    @SubscribeEvent(priority = EventPriority.HIGH)
    public void handleEvent(RenderPlayerEvent.Post event)
    {
        CachedInventory cachedInv = cache.getUnchecked(event.getEntityPlayer());
        NonNullList<ItemStack> cachedArmor = cachedInv.stacks;
        NonNullList<ItemStack> armor = event.getEntityPlayer().inventory.mainInventory;

        if (armor.size() > cachedArmor.size())
        {
            cache.invalidate(event.getEntityPlayer());
            return;
        }

        if (cachedInv.state != 0)
        {
            for (int i = 0; i < armor.size(); i++)
                armor.set(i, cachedArmor.get(i));

            if (isBaublesLoaded)
            {
                try
                {
                    IBaublesItemHandler bh = BaublesApi.getBaublesHandler(event.getEntityPlayer());
                    boolean block = bh.isEventBlocked();
                    bh.setEventBlock(true);
                    for (int i = 0; i < bh.getSlots(); i++)
                        bh.setStackInSlot(i, cachedArmor.get(9 + i));
                    bh.setEventBlock(block);
                }
                catch (Throwable ignored)
                {
                }
            }

            cachedInv.state = 0;
        }
    }

    @SubscribeEvent(priority = EventPriority.HIGH, receiveCanceled = true)
    public void handleEvent(RenderPlayerEvent.Pre event)
    {
        CachedInventory cachedInv = cache.getUnchecked(event.getEntityPlayer());
        NonNullList<ItemStack> cachedArmor = cachedInv.stacks;
        EntityPlayer player = event.getEntityPlayer(); // ВОТ ТУТ ВОТ Я БЕРУ ИГРОКА И ПОЛУЧАЮ ЕГО ИНВЕНТАРЬ
        ICAPCustomInventory cap = player.getCapability(CAPCustomInventoryProvider.INVENTORY_CAP, null);
        CustomInventory cosArmor = cap.getInventory();
        NonNullList<ItemStack> armor = event.getEntityPlayer().inventory.mainInventory;

        if (armor.size() > cachedArmor.size())
        {
            cache.invalidate(event.getEntityPlayer());
            return;
        }

        if (cachedInv.state != 0)
        {
            for (int i = 0; i < armor.size(); i++)
                armor.set(i, cachedArmor.get(i));

            if (isBaublesLoaded)
            {
                try
                {
                    IBaublesItemHandler bh = BaublesApi.getBaublesHandler(event.getEntityPlayer());
                    boolean block = bh.isEventBlocked();
                    bh.setEventBlock(true);
                    for (int i = 0; i < bh.getSlots(); i++)
                        bh.setStackInSlot(i, cachedArmor.get(9 + i));
                    bh.setEventBlock(block);
                }
                catch (Throwable ignored)
                {
                }
            }

            cachedInv.state = 0;
        }

        for (int i = 0; i < armor.size(); i++)
            cachedArmor.set(i, armor.get(i));

        cachedInv.state = 1;

        if (HideCosArmor)
            return;

        for (int i = 0; i < armor.size(); i++)
        {
            if (i < 9)
            {
                if (!cosArmor.getStackInSlot(7).isEmpty()) {
                    armor.set(player.inventory.currentItem, cosArmor.getStackInSlot(7)); //тут указываю предмет который нужно рендерить.
                }
            }
        }

        if (isBaublesLoaded)
        {
            try
            {
                IBaublesItemHandler bh = BaublesApi.getBaublesHandler(event.getEntityPlayer());
                boolean block = bh.isEventBlocked();
                bh.setEventBlock(true);
                for (int i = 0; i < bh.getSlots(); i++)
                {
                    if (!cosArmor.getStackInSlot(7).isEmpty()) {
                        bh.setStackInSlot(player.inventory.currentItem, cosArmor.getStackInSlot(i));
                    }
                    else
                        bh.setStackInSlot(player.inventory.currentItem, ItemStack.EMPTY);
                }
                bh.setEventBlock(block);
            }
            catch (Throwable ignored)
            {
            }
        }
    }

}

Icosider · 7 Янв 2019

Тебе нужно сихронизовать свой инвентарь с инвентарём на клиенте, и затем уже через капу получить свой предмет для рендера. А судя по коду, ты пытаешься достать из клиентской капы(которая пустая обычно) предметы, вот никто и не видит кроме тебя самого(и то вряд ли). Что касается пакета, то тем было много(как регистрировать пакеты учить не буду). Тебе нужно передать в свой пакет класс(к примеру MyCustomInventory), затем в toBytes

Java:

buf.writeInt(myInv.slots.size());
for (ItemStack stack : myInv.slots) {
    ByteBufUtils.writeItemStack(buf, stack);
}

далее в fromBytes:

Java:

final int size = buf.readInt();
stacks = new ArrayList<>(size);

for (int i = 0; i < size; i++) {
    stacks.add(ByteBufUtils.readItemStack(buf));
}

И уже в методе onMessage получить через аргумент message список stacks, получить через ctx.getServerHandler().playerEntity капу и положить все стаки(из листа stacks который ранее получили через аргумент message) в твою капу.
P.s. код чисто для примера, не стоит его себе брать и использовать как есть ибо помимо стаков, в некоторых инвентарях приходится указывать конкретные слоты(в baubles id слотов в инвентаре вроде в enum вынесены)

tox1cozZ · 7 Янв 2019

Добавлю, что отсылать пакет нужно при изменении слота и в PlayerEvent.StartTracking, как это делает майн.

CKATEPTb · 7 Янв 2019

Ivasik написал(а):
Тебе нужно сихронизовать свой инвентарь с инвентарём на клиенте, и затем уже через капу получить свой предмет для рендера. А судя по коду, ты пытаешься достать из клиентской капы(которая пустая обычно) предметы, вот никто и не видит кроме тебя самого(и то вряд ли). Что касается пакета, то тем было много(как регистрировать пакеты учить не буду). Тебе нужно передать в свой пакет класс(к примеру MyCustomInventory), затем в toBytes

Java:

buf.writeInt(myInv.slots.size()); for (ItemStack stack : myInv.slots) { ByteBufUtils.writeItemStack(buf, stack); }

далее в fromBytes:

Java:

final int size = buf.readInt(); stacks = new ArrayList<>(size); for (int i = 0; i < size; i++) { stacks.add(ByteBufUtils.readItemStack(buf)); }

И уже в методе onMessage получить через аргумент message список stacks, получить через ctx.getServerHandler().playerEntity капу и положить все стаки(из листа stacks который ранее получили через аргумент message) в твою капу.
P.s. код чисто для примера, не стоит его себе брать и использовать как есть ибо помимо стаков, в некоторых инвентарях приходится указывать конкретные слоты(в baubles id слотов в инвентаре вроде в enum вынесены)

как мне отправить пакет с сервера на клиент, в тот момент, когда клиент его запросит?

Icosider · 7 Янв 2019

Ну например создать пакет-запрос, пустой, отправляешь его на сервер и в методе onMessage возвращаешь(return) пакет для синхронизации со своими данными.

tox1cozZ · 8 Янв 2019

Я же написал когда нужно отправлять пакет... Тогда у клиентов всегда будет актуальная инфа.

Icosider · 8 Янв 2019

@Agravaine, может у него теперь другая задача, например не капу сихронизовать, а список рангов при открытии гуи или ещё, что-то.

CKATEPTb · 8 Янв 2019

Сделал пакет

Код:

public class PacketPlayerCap implements IMessage {

    public int slot;
    public ItemStack item;
    
    public PacketPlayerCap() {}

    public PacketPlayerCap(ItemStack item, Integer slot) {
        this.slot = slot;
        this.item = item;
    }

    @Override
    public void fromBytes(ByteBuf buf) {
        this.slot = buf.readInt();
        this.item = ByteBufUtils.readItemStack(buf);
        System.out.println("Достаю "+this.item+" в слот " + this.slot);
    }

    @Override
    public void toBytes(ByteBuf buf) {
        buf.writeInt(this.slot);
        ByteBufUtils.writeItemStack(buf, this.item);
        System.out.println("Пакую "+this.item+" в слот " + this.slot);
    }

    public static class Handler implements IMessageHandler<PacketPlayerCap, IMessage> {
        @Override
        public IMessage onMessage(PacketPlayerCap message, MessageContext ctx) {
            /*EntityPlayer player = ctx.getServerHandler().player;
            ICAPCustomInventory cap = player.getCapability(CAPCustomInventoryProvider.INVENTORY_CAP, null);
            CustomInventory inv = cap.getInventory();
            inv.setInventorySlotContents(message.slot, message.item);*/
            System.out.println("ПОЧЕМУ СЮДА НЕ ЗАХОДИТ?");
            return null;
        }
    }

}

При send to all со стороны сервера, клиент не получает public static class Handler implements IMessageHandler<PacketPlayerCap, IMessage>, почему?

Icosider · 8 Янв 2019

Пакет зарегистрирован? Если да, то покажи как регистрируешь?

CKATEPTb · 8 Янв 2019

Java:

public class NetworkHandler {

    public static SimpleNetworkWrapper network;

    public static void init() {

        network = NetworkRegistry.INSTANCE.newSimpleChannel(AvatarCostume.modId);
        network.registerMessage(PacketPlayerCap.Handler.class, PacketPlayerCap.class, 0, Side.SERVER);
    }
}

Код:

        NetworkHandler.init();

CKATEPTb · 8 Янв 2019

изменил

Код:

network.registerMessage(PacketPlayerCap.Handler.class, PacketPlayerCap.class, 0, Side.SERVER);

на

Код:

network.registerMessage(PacketPlayerCap.Handler.class, PacketPlayerCap.class, 0, Side.CLIENT);

И заработало)

ロロロ · 8 Янв 2019

@CKATEPTb Тебе же нужно доставлять клиенту информацию, значит пакет должен быть клиентским.
И ctx.getServerHandler().player; нужно брать из Minecraft.getMinecraft().player. (вроде)

Icosider · 8 Янв 2019

Чего? ctx.getServerHandler().player это для сервера, Minecraft.getMinecraft().player для клиента. И в случаи с первым, его нужно брать из пакета.

ロロロ · 8 Янв 2019

Ivasik написал(а):
И в случаи с первым, его нужно брать из пакета.

Он же шлет пакет клиентам, разве нет?

Icosider · 8 Янв 2019

Если на клиент, то соответственно Minecraft.getMinecraft().player. И поясню к цитате, "в случаи с первым" это относилось к серверному пакету.

CKATEPTb · 8 Янв 2019

А вот на это я уже наткнулся, @Ivasik, как мне отправить игрока с сервера на клиент?

Icosider · 8 Янв 2019

Зачем тебе игрока отправлять?

CKATEPTb · 8 Янв 2019

чтобы получить капу этого игрока.

CKATEPTb · 8 Янв 2019

Код:

public class PacketPlayerCap implements IMessage {

    public int slot;
    public ItemStack item;
    public UUID id;
    
    public PacketPlayerCap() {}

    public PacketPlayerCap(ItemStack item, Integer slot, UUID uuid) {
        this.slot = slot;
        this.item = item;
        this.id = uuid;
    }

    @Override
    public void fromBytes(ByteBuf buf) {
        this.slot = buf.readInt();
        this.item = ByteBufUtils.readItemStack(buf);
        this.id = UUID.fromString(ByteBufUtils.readUTF8String(buf));
        System.out.println("Достаю "+this.item+" в слот " + this.slot);
    }

    @Override
    public void toBytes(ByteBuf buf) {
        buf.writeInt(this.slot);
        ByteBufUtils.writeItemStack(buf, this.item);
        ByteBufUtils.writeUTF8String(buf, this.id.toString());
        System.out.println("Пакую "+this.item+" в слот " + this.slot);
    }

    public static class Handler implements IMessageHandler<PacketPlayerCap, IMessage> {
        @Override
        public IMessage onMessage(PacketPlayerCap message, MessageContext ctx) {
            EntityPlayer player = null; // КАК ЗАСУНУТЬ СЮДА ИГРОКА? Можно ли не используя World?
            ICAPCustomInventory cap = player.getCapability(CAPCustomInventoryProvider.INVENTORY_CAP, null);
            CustomInventory inv = cap.getInventory();
            inv.setInventorySlotContents(message.slot, message.item);
            System.out.println("Ложу в капу "+message.item+" в слот "+message.slot);
            return null;
        }
    }

}

Icosider · 8 Янв 2019

Minecraft.getMinecraft().player, но если это клиент, то укажи для класса Handler что он @SideOnly.CLIENT

Работа с пакетами

CKATEPTb

Icosider

Icosider

Kotliner

tox1cozZ

aka Agravaine

CKATEPTb

Icosider

Kotliner

tox1cozZ

aka Agravaine

Icosider

Kotliner

CKATEPTb

Icosider

Kotliner

CKATEPTb

CKATEPTb

ロロロ

Icosider

Kotliner

ロロロ

Icosider

Kotliner

CKATEPTb

Icosider

Kotliner

CKATEPTb

CKATEPTb

Icosider

Kotliner