Как запускать код на стороне сервера?

ATRIK · 10 Июн 2023

У меня есть код, который выдает предмет игроку, но это происходит только на стороне клиента. Этот код находится в классе, который расширяет ContainerScreen, if (!level.isClientSide) не помогает. В коде команды give не нашел ничего полезного. Как решить эту проблему?
Вот код:

Java:

protected void giveButtonPressed(){
    if (!inventory.player.level.isClientSide) {
        inventory.add(new ItemStack(getItemByStrID(menu.nbtData.getString("itemID")), quantity));
    }
}

Заранее спасибо

_mudblood · 10 Июн 2023

ATRIK написал(а):
ContainerScreen

Потому что это клиентский класс, гуишный.

ATRIK · 10 Июн 2023

_mudblood написал(а):
Потому что это клиентский класс, гуишный.

То есть если я перенесу этот метод в класс, который расширяет LockableTileEntity, оно начнет работать?

_mudblood · 10 Июн 2023

Ну, ты ещё можешь просто через пакет выполнить то, что тебе нужно. Если через гуи, то так обычно и делается, например жмёшь кнопку - отправляешь пакет с клиента на сервер, где уже выполняется.

ATRIK · 10 Июн 2023

_mudblood написал(а):
Ну, ты ещё можешь просто через пакет выполнить то, что тебе нужно. Если через гуи, то так обычно и делается, например жмёшь кнопку - отправляешь пакет с клиента на сервер, где уже выполняется.

Тогда появляется вопрос: как записать ItemStack в ByteBuf? Нашел решение только для 1.7.10, там использовался ByteBufUtils, но на 1.16.5 у него есть только один метод: getContentDump()

TheAndrey · 10 Июн 2023

Хорошее решение. Заинтересованный круг лиц не прочь будет себе выдать нужных вещичкк, прям как в креативе.

ATRIK · 10 Июн 2023

_mudblood написал(а):
Только этот код для 1.12.2, на 1.16.5 тоже поди работает или есть похожее.

К сожалению, на 1.16.5 в ByteBufUtils почти ничего нет, пойду искать решение

_mudblood · 10 Июн 2023

TheAndrey написал(а):
Хорошее решение. Заинтересованный круг лиц не прочь будет себе выдать нужных вещичкк, прям как в креативе.

Я не имел ввиду, писать в самом пакете выдачу предметов.

_SupCM_ · 10 Июн 2023

Выдаёшь в тайле предмет на сервере и отсылаешь пакет об обновлении гуи на клиент. Иначе возможен пакетхак.

ATRIK · 10 Июн 2023

Я запутался

Что надо обрабатывать в пакете и как, объясните пожалуйста

Amiri163 · 10 Июн 2023

код чтобы выдать предмет игроку...

Java:

 player.addItem(Items.DIAMOND.getDefaultInstance());

ATRIK · 10 Июн 2023

NoTrue (●'◡'●) написал(а):
код чтобы выдать предмет игроку...

Вы, вероятнее всего, не полностью прочитали первое сообщение

ATRIK · 10 Июн 2023

_mudblood написал(а):
Ну, ты ещё можешь просто через пакет выполнить то, что тебе нужно. Если через гуи, то так обычно и делается, например жмёшь кнопку - отправляешь пакет с клиента на сервер, где уже выполняется.

Что именно нужно обработать через пакет?

ATRIK · 10 Июн 2023

Перенес метод в TileEntity, но все равно не выполняется на сервере, и опять же !level.isClientSide не помогает.

_SupCM_ · 10 Июн 2023

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

ATRIK · 10 Июн 2023

_SupCM_ написал(а):
По нажатию кнопки отсылаешь пакет на сервер. Этот пакет приходит в тайл/контейнер. После получения пакета отправляешь пакет с сервера на клиент, содержащий в себе предмет. При получении пакета на клиенте вставляешь в клиентский контейнер предмет и обновляешь гуи.

Звучит страшно

, сейчас буду пробовать. То есть я отправляю пакет на сервер, в котором содержится предмет/тип предмета и в TileEntity добавляю его в инвентарь? Если да, то как через ByteBuf передать предмет/тип предмета? Пробовал через ByteBufUtils, но на 1.16.5 не работает.

ATRIK · 10 Июн 2023

Похоже я нашел альтернативу ByteBufUtils! На 1.16.5 вместо него PacketBuffer!

ATRIK · 10 Июн 2023

Я всё сделал!!! Отправляется пакет после нажатия на кнопку, через него вызывается метод, в котором добавляется предмет!
Вот код:

Java:

public class BuyBtnMessagePacket {

    public final ItemStack sellItemStack;
    public final String playerUUID;
    public final int quantity;

    public BuyBtnMessagePacket(ItemStack itemStack, String playerUUID, int quantity) {
        this.sellItemStack = itemStack;
        this.playerUUID = playerUUID;
        this.quantity = quantity;
    }

    public BuyBtnMessagePacket(PacketBuffer byteBuf){
        this(byteBuf.readItem(), byteBuf.readUtf(), byteBuf.readInt());
    }
    public void encode(PacketBuffer byteBuf){
        byteBuf.writeItem(this.sellItemStack);
        byteBuf.writeUtf(this.playerUUID);
        byteBuf.writeInt(this.quantity);
    }

    public boolean handle(Supplier<NetworkEvent.Context> ctx) {
        final AtomicBoolean success = new AtomicBoolean(false);
        ctx.get().enqueueWork(() -> {
            PlayerEntity playerEntity = ctx.get().getSender().getLevel().getPlayerByUUID(UUID.fromString(this.playerUUID));
            PlayerInventory playerInventory = playerEntity.inventory;
            buyButtonPressed(playerInventory, sellItemStack, quantity);
            success.set(true);
        });

        ctx.get().setPacketHandled(true);
        return success.get();
    }

}

Пожайлуста, дайте знать, если может быть пакетхак.
Всем огромное спасибо за помощь :0 Особенно mudblood и _SupCM!

TheAndrey · 10 Июн 2023

Неплохо для новичка, однако есть проблемы:

Вылетит NPE если игрока с указанным UUID нет на сервере.
UUID записывается в пакет в виде двух long (most&least), строка весит больше.
Использовать Int для передачи количества тоже слишком жирно. short хватит за глаза или byte, если кол-во не более 64 ед.

ATRIK написал(а):
Пожайлуста, дайте знать, если может быть пакетхак.

Зачем передавать предмет с клиента на сервер вообще? Для справки: все предметы находятся на сервере, с ними взаимодействуют путём указания номера слота в инвентаре.

ATRIK · 11 Июн 2023

TheAndrey написал(а):
Неплохо для новичка

Спасибо) По UUID вообще проблем нет. Только игрок с таким же UUID сможет выдать себе предмет.

TheAndrey написал(а):
Зачем передавать предмет с клиента на сервер вообще? Для справки: все предметы находятся на сервере, с ними взаимодействуют путём указания номера слота в инвентаре.

ItemStack я беру из класса, который расширяет ContainerScreen. Мне сказали то что он клиентский, и как я понял, оттуда его только через пакет получить. В принципе у меня ID предмета записан в NBT TileEntity могу его оттуда брать, но мне тогда в PacketBuf надо еще BlockPos передавать. В любом случае, думаю так будет нормально.