Résolu Multiblock plus compacte

[DailyCraft]

Bonjour,

je viens de faire un multiblock mais je me demander si je pouvais le faire plus compacte ou plus optimiser.

Voici ma class :

public class BlockPersonalBlock extends Block
{
    public BlockPersonalBlock()
    {
        super(Block.Properties.create(Material.IRON));
        setRegistryName("personal_block");
    }
 
    @Override
    public void onBlockPlacedBy(World world, BlockPos pos, BlockState state, @Nullable LivingEntity placer, ItemStack stack)
    {
        if (!world.isRemote)
        {
            if (placer instanceof PlayerEntity)
            {
                Block diamond = Blocks.DIAMOND_BLOCK;
                Block gold = Blocks.GOLD_BLOCK;
                Block iron = Blocks.IRON_BLOCK;
                Block bars = Blocks.IRON_BARS;
                Block water = Blocks.WATER;
 
                if (
                        getBlock(world, pos.down()) == diamond &&
                                getBlock(world, pos.down().north().west()) == diamond &&
                                getBlock(world, pos.down().north()).getBlock() == diamond &&
                                getBlock(world, pos.down().north().east()) == diamond &&
                                getBlock(world, pos.down().west()) == diamond &&
                                getBlock(world, pos.down().east()) == diamond &&
                                getBlock(world, pos.down().south().west()) == diamond &&
                                getBlock(world, pos.down().south()) == diamond &&
                                getBlock(world, pos.down().south().east()) == diamond &&
 
                                getBlock(world, pos.north()) == water &&
                                getBlock(world, pos.north().east()) == water &&
                                getBlock(world, pos.north().west()) == water &&
                                getBlock(world, pos.west()) == water &&
                                getBlock(world, pos.south()) == water &&
                                getBlock(world, pos.south().west()) == water &&
                                getBlock(world, pos.south().east()) == water &&
                                getBlock(world, pos.east()) == water &&
 
                                getBlock(world, pos.north(2).west()) == gold &&
                                getBlock(world, pos.north(2).west(2)) == gold &&
                                getBlock(world, pos.north(2)) == gold &&
                                getBlock(world, pos.north(2).east()) == gold &&
                                getBlock(world, pos.north(2).east(2)) == gold &&
                                getBlock(world, pos.east(2).north()) == gold &&
                                getBlock(world, pos.east(2)) == gold &&
                                getBlock(world, pos.east(2).south()) == gold &&
                                getBlock(world, pos.south(2).east()) == gold &&
                                getBlock(world, pos.south(2).east(2)) == gold &&
                                getBlock(world, pos.south(2)) == gold &&
                                getBlock(world, pos.south(2).west()) == gold &&
                                getBlock(world, pos.south(2).west(2)) == gold &&
                                getBlock(world, pos.west(2).north()) == gold &&
                                getBlock(world, pos.west(2)) == gold &&
                                getBlock(world, pos.west(2).south()) == gold &&
 
                                getBlock(world, pos.up().north(2).west(2)) == iron &&
                                getBlock(world, pos.up().north(2).east(2)) == iron &&
                                getBlock(world, pos.up(2).north(2).west(2)) == iron &&
                                getBlock(world, pos.up(2).north(2).east(2)) == iron &&
                                getBlock(world, pos.up(3).north(2).west(2)) == iron &&
                                getBlock(world, pos.up(3).north(2).east(2)) == iron &&
                                getBlock(world, pos.up().south(2).west(2)) == iron &&
                                getBlock(world, pos.up().south(2).east(2)) == iron &&
                                getBlock(world, pos.up(2).south(2).west(2)) == iron &&
                                getBlock(world, pos.up(2).south(2).east(2)) == iron &&
                                getBlock(world, pos.up(3).south(2).west(2)) == iron &&
                                getBlock(world, pos.up(3).south(2).east(2)) == iron &&
                                getBlock(world, pos.up(4).north(2).west()) == iron &&
                                getBlock(world, pos.up(4).north(2).east()) == iron &&
                                getBlock(world, pos.up(5).north(2)) == iron &&
                                getBlock(world, pos.up(4).east(2).north()) == iron &&
                                getBlock(world, pos.up(4).east(2).south()) == iron &&
                                getBlock(world, pos.up(5).east(2)) == iron &&
                                getBlock(world, pos.up(4).south(2).west()) == iron &&
                                getBlock(world, pos.up(4).south(2).east()) == iron &&
                                getBlock(world, pos.up(5).south(2)) == iron &&
                                getBlock(world, pos.up(4).west(2).south()) == iron &&
                                getBlock(world, pos.up(4).west(2).north()) == iron &&
                                getBlock(world, pos.up(5).west(2)) == iron &&
 
                                getBlock(world, pos.up(6).north(2)) == bars &&
                                getBlock(world, pos.up(7).north()) == bars &&
                                getBlock(world, pos.up(7).north(2)) == bars &&
                                getBlock(world, pos.up(6).west(2)) == bars &&
                                getBlock(world, pos.up(7).west()) == bars &&
                                getBlock(world, pos.up(7).west(2)) == bars &&
                                getBlock(world, pos.up(6).south(2)) == bars &&
                                getBlock(world, pos.up(7).south()) == bars &&
                                getBlock(world, pos.up(7).south(2)) == bars &&
                                getBlock(world, pos.up(6).east(2)) == bars &&
                                getBlock(world, pos.up(7).east()) == bars &&
                                getBlock(world, pos.up(7).east(2)) == bars &&
                                getBlock(world, pos.up(8)) == bars &&
                                getBlock(world, pos.up(8).north()) == bars &&
                                getBlock(world, pos.up(8).west()) == bars &&
                                getBlock(world, pos.up(8).south()) == bars &&
                                getBlock(world, pos.up(8).east()) == bars &&
                                getBlock(world, pos.up(9)) == bars
                ) {
                    Main.loggger.info("Structure good");
                }
            }
        }
    }
 
    private Block getBlock(World world, BlockPos pos)
    {
        return world.getBlockState(pos).getBlock();
    }
}

[En0ri4n]

Je ne connaît pas la structure mais tu peux essayer de mettre des boucles for et de ne garder juste qu’un boolean donc tu le met à true puis si un Block n’est pas bon tu le met sur false (après peut être que ca ne marchera pas, mais tu pourrais t’inspirer du beacon :winking_face: )
EDIT : tu pourrais carrément break si un Block n’est pas bon comme ça ça continue pas

[DailyCraft]

Oui mais moi et les boucles (même si je sais que c’est hyper simple) ça va pas trop bien ensemble, si j’arrive à en comprendre une, le temps que j’essaye d’en comprendre une autre j’ai déjà l’ai déjà oublié.

J’ai quand même déjà essayer voir si j’avait compris if (i = 0; i < 10; i++) et il n’y à aucun problème à part que c’est for mais c’est la seul que je comprends bien sinon les autres behhhh

[En0ri4n]

Pourtant c’est très simple :upside-down_face:

for (int x = 0; x < 3; x++)
         for (int y = 0; y < 3; y++)
                  for (int z = 0; z < 3; z++)
                          world.setBlockState (new BlockPos (posX + x, posY + y, posZ + z), Blocks.DIRT);

La ca va te faire un cube de 3x3 de dirt

[DailyCraft]

Ouai mais pour ce que je veux faire je ne vois pas comment je peux faire.

Et ton exemple est assez pourrie car il y a juste à faire world.setBlockState(new BlockPos(posX + 3, posY + 3, posZ + 3), Blocks.DIRT);

et la boucles que tu as faites et celle que je comprends le mieux, mais sinon il y à le while qui se répète jusqu’à que ce qui à en les parenthèses soit vrai (comme un if mais voila) et le for (Block block : blocks) je ne sais pas ce qui fais.
Il y en à d’autre ou pas ?

Et pour le while je sens que je me suis mal exprimer dans mon explication voir si j’ai compris :

int i = 1
 
while (i == 2)
    i++;

donc la il s’execute que une, et pour faire une boucle infinie, il faut utiliser while (true)

[Twiguinou]

Si tu es en 1.15 tu peux essayer de mettre en relation une boucle for avec la classe Direction pour aller plus vite. Sinon tu peux faire une HashMap avec les BlockPos et les BlockState.

[DailyCraft]

Mon block n’a pas de direction et le multiblock à les même block au nord sud est et ouest donc j’ai pas besoin de faire de vérification dans la quel il est fait.

[Twiguinou]

J’ai testé d’utiliser des boucles for pour ta structure mais le haut est trop complexe à réaliser avec, la base elle est plutôt simple.

[DailyCraft]

Mais sinon c’est pas très grave mais merci quand même