[1.7.10] Pickup on key / Подбор на кнопку 0.0.1

[Fr0Le]

Fr0Le добавил(а) новый ресурс:

[1.7.10] Pickup on key / Подбор на кнопку - Создание кнопки для подбора предметов

В этом ресурсе Вы узнаете:

• Как создать кнопку, которая при нажатии подберает предмет с земли;

Скачать исходники
Скачать мод

Немного кода:
Регистрируем кнопку

public static final KeyBinding
    PICKUP_KEY = new KeyBinding(Type.getTranslate("key.pickup"), Keyboard.KEY_F...

Узнать больше об этом ресурсе...

 

[Eban]

годно!
когда-нибудь дождемся мод на обмен....

 

[Dazay]

Годнота

 

[hohserg]

когда-нибудь дождемся мод на обмен....

Такой уже есть: [1.12.2] Oxygen - расширения для сервера (100% GUI)

 

[Eban]

Это под новые версии. А на 1.7.10 нету

 

[NullPointerException]

А на 1.7.10 нету

Написать под 1.7.10 тоже не составит большого труда, особенно все ключевые темы для создания трейда уже давно описаны на форуме.

 

[Plasticable]

Ты специально сюда не приложил содержимое класса PickupPacket, чтобы народ не шокировать?
Я полез в репозиторий и офигел:

public class PickupPacket implements IMessage {

    public int entityID;
    public int playerID;

    // ...

    public static class Handler    implements IMessageHandler<PickupPacket, IMessage> {
        public IMessage onMessage(PickupPacket message, MessageContext ctx) {
            MinecraftServer server = MinecraftServer.getServer();
            if (server.getEntityWorld().getEntityByID(message.playerID) != null && server.getEntityWorld().getEntityByID(message.entityID) != null && server.getEntityWorld().getEntityByID(message.playerID) instanceof EntityPlayer) {
                EntityPlayer thePlayer = (EntityPlayer)server.getEntityWorld().getEntityByID(message.playerID);
                EntityItem theItem = (EntityItem)server.getEntityWorld().getEntityByID(message.entityID);
                theItem.setDead();
                thePlayer.worldObj.playSoundEffect(thePlayer.posX, thePlayer.posY, thePlayer.posZ, "random.pop", 1.0f, 1.0f);
                thePlayer.inventory.addItemStackToInventory(theItem.getEntityItem());
                thePlayer.inventoryContainer.detectAndSendChanges();
            }
            return null;
        }
    }

}

1. Нет проверки на расстояние до предмета. С каким-нибудь фрикамом можно потроллить игроков, у которых "исчезает" дроп с противника после их убийства в пвп
2. Читер может отправить пакет с любым playerID и заставить любого игрока подобрать любой выброшенный предмет, я правильно понял? Это даже ещё веселее предыдущего пункта.
Игрока, от которого пришёл пакет, можно получить с помощью ctx.getServerHandler().playerEntity
3. Зачем вызывать getEntityByID 5 раз, если можно всего один раз для предмета?
4. Проверка на null перед instanceof не нужна
5. Этот код сломает событие (оно просто не вызывается) поднятия предмета с земли, которое могут использовать твои или сторонние моды.
6. Ты убиваешь выброшенный предмет без проверки на невозможность его подобрать игроком (из-за заполненного инвентаря, например)
Результат метода addItemStackToInventory оберни в if-блок, в котором неплохо бы вызвать
FMLCommonHandler.instance().firePlayerItemPickupEvent(thePlayer, theItem), inventoryContainer.detectAndSendChanges() и уже потом theItem.setDead()


Ещё забавный момент в ItemRenderOnFloor#drawItems: у тебя уже есть переменная с инстансом Minecraft, но ты всё равно ещё 5 раз вызываешь Minecraft.getMinecraft(). Очень похоже на код, собранный из скопитыренных кусков.

Остальное решил не читать...

 

[Fr0Le]

@Plasticable, в любом моде где есть пакеты, читер может отослать пакет с подменой или вызвать нужный пакет, когда ему удобно, здесь речь не об этом, имхо. Обмажся античитами и будет тебе счастье.
Я не гуру, никого не учу и учить не собираюсь и не говорю что нужно делать именно так, это чисто пример одной из реализаций, которая может быть. Кому нужно тот переделает под себя, кому нет, вообще юзать не будет. Вот по этому это и слив, а не гайд или туториал. Все остальное учту.

 

[Plasticable]

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

Верно. Но задача хорошего кодера - минимизировать вред таких пакетов с помощью всяческих серверных проверок.

Обмажся античитами

Клиентскими? В данном случае, это не панацея, в отличии от грамотной обработки пакетов на сервере.

не говорю что нужно делать именно так, это чисто пример одной из реализаций

Плохой пример, без обид.

Надеюсь, ты учтёшь мои рекомендации и исправишь это хотя бы на своём проекте.

 

[tox1cozZ]

worldObj.rayTraceBlocks как бы не находит энтитей, угу

 

[Icosider]

@tox1cozZ, а ну да, но на его основе можно сделать поиск сущностей