Хуки, замена ванильного блока

[Garik]

Доброго времени суток земляне и Чужие!
Сегодня я столкнулся с такой проблемой: В хуках ваших я полный ноль, как самое лёгкое для себя я решил заменить ванильный блок, поискав, подумав я наткнулся на эту ему: Трудный выбор, сделав всё как было написано в той теме у меня получилось что-то такое:

package com.garik.snowplusplus.hook;

import gloomyfolken.hooklib.minecraft.HookLoader;
import gloomyfolken.hooklib.minecraft.PrimaryClassTransformer;

public class HookLoaderPlusPlus extends HookLoader {

     @Override
     public String[] getASMTransformerClass() {
         return new String[]{PrimaryClassTransformer.class.getName()};
     }

        @Override
        public void registerHooks() {
            registerHookContainer("com.garik.snowplusplus.hook.HookContainerPlusPlus");
        }
    }

package com.garik.snowplusplus.hook;

import com.garik.snowplusplus.block.register.BlockVanillaRegister;

import gloomyfolken.hooklib.asm.Hook;
import gloomyfolken.hooklib.asm.ReturnCondition;
import net.minecraft.block.Block;
import net.minecraft.init.Blocks;
import net.minecraft.util.ResourceLocation;

public class HookContainerPlusPlus {

    @Hook(returnCondition = ReturnCondition.ALWAYS, isMandatory = true)
    public static void registerBlock(Block block, int id, ResourceLocation res, Block blockToRegister) {
        if(id == 80){
            Block.REGISTRY.register(id, res, BlockVanillaRegister.snow_plus_plus);
        }
        else
            Block.REGISTRY.register(id, res, blockToRegister);
    }

    @Hook(returnCondition = ReturnCondition.ON_NOT_NULL, isMandatory = true)
    public static Block getRegisteredBlock(Blocks inst, String name) {
        if(name.equals("snow")){
            return BlockVanillaRegister.snow_plus_plus;
        }
        return null;
    }
    
}

package com.garik.snowplusplus.block.register;

import com.garik.snowplusplus.block.SnowPlusPlus;

import net.minecraft.block.Block;

public class BlockVanillaRegister {

    public static Block snow_plus_plus = new SnowPlusPlus().setHardness(0.2F).setUnlocalizedName("snow");

    
}

package com.garik.snowplusplus.block;

import java.util.Random;

import net.minecraft.block.Block;
import net.minecraft.block.BlockSnowBlock;
import net.minecraft.block.material.Material;
import net.minecraft.block.state.IBlockState;
import net.minecraft.client.Minecraft;
import net.minecraft.creativetab.CreativeTabs;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.init.Items;
import net.minecraft.item.Item;
import net.minecraft.item.ItemStack;
import net.minecraft.util.math.BlockPos;
import net.minecraft.world.EnumSkyBlock;
import net.minecraft.world.World;

public class SnowPlusPlus extends BlockSnowBlock {
    
    public SnowPlusPlus() {
        super();
        this.setTickRandomly(true);
        this.setCreativeTab(CreativeTabs.BUILDING_BLOCKS);
    }
    
    public Item getItemDropped(IBlockState state, Random rand, int fortune) {
        return Items.SNOWBALL;
    }

    public int quantityDropped(Random random) {
        return 4;
    }

    public void updateTick(World worldIn, BlockPos pos, IBlockState state, Random rand) {
        if (worldIn.getLightFor(EnumSkyBlock.BLOCK, pos) > 11) {
            worldIn.setBlockToAir(pos);
        }
    }
    
    @Override
    public void onBlockPlacedBy(World world, BlockPos pos, IBlockState state, EntityLivingBase entity, ItemStack stack) {
        super.onBlockPlacedBy(world, pos, state, entity, stack);
        Minecraft.getMinecraft().player.sendChatMessage("ASM и Хуки - говно (с)Doc");
    }

}

Как видите это всё что я сделал, но что-то BlockSnowBlock не хочет выполнять мой код в классе SnowPlusPlus, хотя по идей я сделал всё правильно и я добавил код к блоку BlockSnowBlock.

На всякий случай скидываю лог:

 

[hohserg]

самое лёгкое

заменить ванильный блок

Не та дверь.

Хуками это не надо делать. Максимум - изменить поведение отдельных методов блока

Injecting location in coremod gloomyfolken.hooklib.example.ExampleHookLoader
[01:20:20] [main/INFO] [STDOUT]: [gloomyfolken.hooklib.asm.HookLogger$SystemOutLogger:debug:20]: [DEBUG] Injecting hooks into class net.minecraft.entity.Entity
[01:20:20] [main/INFO] [STDOUT]: [gloomyfolken.hooklib.asm.HookLogger$SystemOutLogger:debug:20]: [DEBUG] Patching method net.minecraft.entity.Entity#getBrightnessForRender()
[01:20:20] [main/INFO] [STDOUT]: [gloomyfolken.hooklib.asm.HookLogger$SystemOutLogger:debug:20]: [DEBUG] Injecting hooks into class net.minecraft.entity.player.EntityPlayer
[01:20:20] [main/INFO] [STDOUT]: [gloomyfolken.hooklib.asm.HookLogger$SystemOutLogger:debug:20]: [DEBUG] Patching method net.minecraft.entity.player.EntityPlayer#getPortalCooldown()

В логе чет другие хуки видны.


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

 

[Garik]

В логе чет другие хуки видны.

Это я забыл примеры использования хуков удалить.

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

А что, заменить блок намного сложнее чем вот это всё? 0_о

 

[Могучий горгон]

Что-то незаметно что у тебя вообще хуки инжектятся. Если бы это произошло у тебя бы все работало или крашнулось. И в логах бы написалось.

 

[Garik]

Ааааааа... Я забыл указать в VM аргументах путь к моему загрузчику хуков! Спасибо @Override

 

[hohserg]

заменить блок намного сложнее чем вот это всё?

Вообще то да. Хуки в глубины forge могут иметь стремные последствия, вроде несовместимости с модами. Например, какой-то мод(или сам майн) захотел получать снег этим методом(getRegisteredBlock), и у него произошла ошибка приведения типов

if(id == 80){

Не стоит полагаться на численные представления предметов, они могут меняться(хотя ванильные не должны по идее)

 

[Могучий горгон]

Ванильные не меняются.

им методом(getRegisteredBlock), и у него произошла ошибка приведения типов

Да ну? он же наследует класс обычного снега.

 

[Garik]

(хотя ванильные не должны по идее)

Верно!

Например, какой-то мод(или сам майн) захотел получать снег этим методом(getRegisteredBlock)

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

 

[hohserg]

Тогда этот отдельный пример выглядит безопасным