TileEntity для слабаков

tox1cozZ · 23 Янв 2020

Чот он меня вообще разочаровал, логика напрочь отсутствует...

Maxik · 23 Янв 2020

А, там их два, я не тот взял

Ivasik написал(а):
"tick()" с хрена ли? В ITickable метод update, а не tick

Dysphoria написал(а):
логика напрочь отсутствует...

Почему? Все логично - получаем коорды энтити, получаем коорды тайла, затем путем простого вычитания выясняем расстояние между ними и активируем\деактивируем блок.
Правда, я думал можно по дефолту энтити брать, но оказывается только из листа, поэтому через коллижн логичней

Maxik · 23 Янв 2020

Лол, в маяке ведь тоже самое, что я делал в блоке. Нахрена плодить классы, делать лишние тайлы, если все тоже самое делается в блоке?

Java:

    private void addEffectsToPlayers()
    {
        if (this.isComplete && this.levels > 0 && !this.world.isRemote && this.primaryEffect != null)
        {
            double d0 = (double)(this.levels * 10 + 10);
            int i = 0;

            if (this.levels >= 4 && this.primaryEffect == this.secondaryEffect)
            {
                i = 1;
            }

            int j = (9 + this.levels * 2) * 20;
            int k = this.pos.getX();
            int l = this.pos.getY();
            int i1 = this.pos.getZ();
            AxisAlignedBB axisalignedbb = (new AxisAlignedBB((double)k, (double)l, (double)i1, (double)(k + 1), (double)(l + 1), (double)(i1 + 1))).grow(d0).expand(0.0D, (double)this.world.getHeight(), 0.0D);
            List<EntityPlayer> list = this.world.<EntityPlayer>getEntitiesWithinAABB(EntityPlayer.class, axisalignedbb);

            for (EntityPlayer entityplayer : list)
            {
                entityplayer.addPotionEffect(new PotionEffect(this.primaryEffect, j, i, true, true));
            }

            if (this.levels >= 4 && this.primaryEffect != this.secondaryEffect && this.secondaryEffect != null)
            {
                for (EntityPlayer entityplayer1 : list)
                {
                    entityplayer1.addPotionEffect(new PotionEffect(this.secondaryEffect, j, 0, true, true));
                }
            }
        }
    }

Короче ясно, блоком лучше.

Eifel · 23 Янв 2020

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

Маяк чисто для примера, как достать энтитей в радиусе

Maxik · 23 Янв 2020

Ух горит..!
Но тут нету тут метода getAroundEntites, прикинь! Откуда я тебе возьму энтитей вокруг? Только через лист. Только мне не нужно брать всех энтитей вокруг. Мне надо брать конкретно ПЕРЕД блоком, а не ВСЕХ ВОКРУГ.

Eifel написал(а):
о твоих околонулевых знаниях джава и отсутсвия желания улучшаться

А вот это обидно. То что я не изучал и никогда особо не делал тайлы, вообще не имеет отношения к тому, умею я в джаву или нет. Не умел бы - и через блок бы не сделал, лол)
Не надо говорить того, чего не знаешь, совет.

Eifel написал(а):
Маяк чисто для примера, как достать энтитей в радиусе

Держу в курсе, в блоке тоже есть метод, который проверяет всех энтитей, залезших в блокбаунд. ОЙ, *, а оказывается в маяке то - тоже самое! А вы тут все топите за тайл, а тайл то оказыватся, тут и нахрен не нужен! Вот и горит у вас, потому что я прав))

Eifel · 23 Янв 2020

Maxik написал(а):
Только через лист

Так о этом и речь. В чем проблема - взял и проверил что тебе надо.

Maxik написал(а):
Не надо говорить того, чего не знаешь, совет.

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

tox1cozZ · 23 Янв 2020

Maxik написал(а):
Мне надо брать конкретно ПЕРЕД блоком, а не ВСЕХ ВОКРУГ.

Ты не можешь сделать проверку на сторону в которую смотрит блок и относительно этого сместить координаты выборки энтитей?

Maxik написал(а):
Держу в курсе, в блоке тоже есть

Дак ты постоянно добавляешь свой блок в очередь на обновление, добавляя лишние расходы. Зачем это делать, если есть тайл который лять специально и придуман для блоков которые постоянно обновляются без всяких очередей и их обслуживания.
Изобрел свой велосипед и пытается доказать что он лучше... Мда

Maxik · 23 Янв 2020

Могу. Я просто сейчас занялся этим, чтобы понять ,а можно ли тут что то было сделать более удобно, чем в блоке. Пока что, разницы вообще никакой. Даже больше - разница отрицательна, т.к я сейчас создаю кучу всего, чего в принципе и не нужно.

Eifel · 23 Янв 2020

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

Maxik · 23 Янв 2020

Dysphoria написал(а):
Дак ты постоянно добавляешь свой блок в очередь на обновление, добавляя лишние расходы.

Какая еще очередь на обновление? Каждый блок тикает сам по себе, как и тайл.

Liahim · 23 Янв 2020

Могу написать только одно: :m_faceplam:

Liahim · 23 Янв 2020

@Maxik, это называется "архитектура кода".
У себя я тоже очень часто в тайле исполбзую методы блока и наоборот. Бывают случаи, когда приходится несколько раз скакать туда обратно. Главное, чтобы это было логически оправдано и удобно.
Ты, как будто, не видишь или не понимаешь общей картины. Программа не замыкается на одном классе, смотри шире.
И это совсем не связано со знанием языка, как выше и было замечено. Это кое что поглубже.

fukkivdan · 23 Янв 2020

Чел, тайл это очень легко, для себя я просто провел аналогию.
Блок это как итем, бо он тоже синглтон. У итема есть итемстак, что у игрока в инвенте/сундуке/попке лежит (уникальный, да).
А у блока есть тайл(не всегда, кстати), который так же стоит в мире уникальный (Если что можете меня поправить)
Они юзабельны шо капец, но я стараюсь тайлы не юзать особо, пока еще не приспичило

Если ты говоришь что тайлы сложно, то значит ты не практиковался с ними и не хочешь этого делать

Dahaka · 23 Янв 2020

А с чего вы вообще взяли, что через тайл решить эту задачу - лучше? Что насчёт сложности алгоритмов в обоих случаях? Об этом нужно думать изначально, а не после того, как все реализовано.

tox1cozZ · 23 Янв 2020

Потому что замыкать апдейт блока на "срочных блоках" в корне неверно, имхо

Dahaka · 23 Янв 2020

Разве там сделано не на коллизии? (я не видел кода)

tox1cozZ · 23 Янв 2020

Он делает тоже самое что и в тайле советуют, но в апдейтике блока. Так как блок обновляется либо рандомно либо никак, он замыкает апдейт добавляя срочный блок

Maxik · 23 Янв 2020

@Dahaka вот как я сделал на коллизии(тапками не кидать, оно работает!)
Вообще я думаю, что можно внести вызов метода обязательного тика только тогда, когда игрок коллидится этой AABB. Таким образом это будет не "постоянно" тикающий блок, что будет еще лучшим вариантом.

Java:

package ru.lg.SovietMod.Blocks;

import java.util.List;
import java.util.Random;

import javax.annotation.Nullable;

import net.minecraft.block.SoundType;
import net.minecraft.block.material.Material;
import net.minecraft.block.properties.IProperty;
import net.minecraft.block.properties.PropertyBool;
import net.minecraft.block.state.BlockStateContainer;
import net.minecraft.block.state.IBlockState;
import net.minecraft.entity.Entity;
import net.minecraft.entity.EntityLivingBase;
import net.minecraft.item.ItemStack;
import net.minecraft.util.BlockRenderLayer;
import net.minecraft.util.EnumFacing;
import net.minecraft.util.math.AxisAlignedBB;
import net.minecraft.util.math.BlockPos;
import net.minecraft.util.math.RayTraceResult;
import net.minecraft.util.math.Vec3d;
import net.minecraft.world.IBlockAccess;
import net.minecraft.world.World;
import net.minecraftforge.fml.relauncher.Side;
import net.minecraftforge.fml.relauncher.SideOnly;
import ru.lg.SovietMod.Blocks.Basic.BasicBlockSideWithCustomModel;

public class MotionSensor extends BasicBlockSideWithCustomModel {
    public static final PropertyBool IS_ACTIVE = PropertyBool.create("is_active");

    public static final PropertyBool DOWN = PropertyBool.create("down");

    private static AxisAlignedBB[] FANTOM_AABB = new AxisAlignedBB[] {
            new AxisAlignedBB(0D, 0D, 1D, 1D, 1D, -6D),
            new AxisAlignedBB(0D, 0D, 0D, 1D, 1D, 7D),
            new AxisAlignedBB(0D, 0D, 0D, 7D, 1D, 1D),
            new AxisAlignedBB(1D, 0D, 0D, -6D, 1D, 1D),
    };
    private static AxisAlignedBB[] SIDE_AABB = new AxisAlignedBB[] {
            new AxisAlignedBB(0.23D, 0.3D, 0.81D, 0.76D, 0.75D, 1D),
            new AxisAlignedBB(0.23D, 0.3D, 0D, 0.76D, 0.75D, 0.19D),
            new AxisAlignedBB(0D, 0.3D, 0.23D, 0.19D, 0.75D, 0.76D),
            new AxisAlignedBB(0.81D, 0.3D, 0.23D, 1D, 0.75D, 0.76D)
    };

    @SideOnly(Side.CLIENT)
    public BlockRenderLayer getBlockLayer()
    {
        return BlockRenderLayer.CUTOUT_MIPPED;
    }
    public MotionSensor(Material materialIn, String name, float hardness, float resistanse, SoundType soundtype) {
        super(materialIn, name, hardness, resistanse, soundtype);
        this.setDefaultState(this.blockState.getBaseState().withProperty(FACING, EnumFacing.NORTH).withProperty(IS_ACTIVE, false).withProperty(DOWN, false));
        this.setTickRandomly(true);
    }
    @Override
    public IBlockState getActualState(IBlockState state, IBlockAccess world, BlockPos pos)
    {

        return state.withProperty(FACING, state.getValue(FACING)).withProperty(IS_ACTIVE, state.getValue(IS_ACTIVE))
                .withProperty(DOWN, canConnectTo(world, pos));
    }

    private boolean canConnectTo(IBlockAccess world, BlockPos pos)
    {
        {
            //System.out.println("facing " + world.getBlockState(pos.up()).isSideSolid(world, pos.down(), EnumFacing.DOWN));
            if(world.getBlockState(pos.down()).isFullBlock()) {
                return true;
            }

        }
        return false;
    }
    @Override
    public void onBlockPlacedBy(World world, BlockPos pos, IBlockState state, EntityLivingBase placer, ItemStack stack)
    {

        EnumFacing facing = EnumFacing.fromAngle(placer.getRotationYawHead());
        world.scheduleUpdate(pos, state.getBlock(), 40);
        world.setBlockState(pos, state.withProperty(FACING, facing));
    }
    @Override
    public IBlockState getStateFromMeta(int meta)
    {
        return this.getDefaultState().withProperty(FACING, EnumFacing.getHorizontal(meta)).withProperty(IS_ACTIVE, Boolean.valueOf((meta & 8) > 0));
    }

    public int getMetaFromState(IBlockState state)
    {
        int i = 0;
        i = i | ((EnumFacing)state.getValue(FACING)).getIndex();

        if (((Boolean)state.getValue(IS_ACTIVE)).booleanValue())
        {
            i |= 8;
        }

        return i;
    }

    public RayTraceResult collisionRayTrace(IBlockState blockState, World worldIn, BlockPos pos, Vec3d start, Vec3d end)
    {
        return this.rayTrace(pos, start, end, blockState.getCollisionBoundingBox(worldIn, pos));
    }
    public AxisAlignedBB getCollisionBoundingBox(IBlockState state, IBlockAccess worldIn, BlockPos pos)
    {
        switch (state.getValue(FACING))
        {
        case SOUTH:
            return SIDE_AABB[0];
        case NORTH:
        default:
            return SIDE_AABB[1];
        case WEST:
            return SIDE_AABB[2];
        case EAST:
            return SIDE_AABB[3];
        }
    }


    @Override
    public void onEntityCollidedWithBlock(World world, BlockPos pos, IBlockState state, Entity entityIn)
    {
        if (!world.isRemote)
        {    

            if(entityIn instanceof EntityLivingBase ) {
                world.setBlockState(pos, state.withProperty(IS_ACTIVE, true));
                world.scheduleUpdate(pos, state.getBlock(), 20);
            }

        }
    }

    @Override
    public void updateTick(World world, BlockPos pos, IBlockState state, Random rand)
    {
        if (!world.isRemote)
        {    
            world.scheduleUpdate(pos, state.getBlock(), 10);
            List <? extends Entity > list = world.getEntitiesWithinAABBExcludingEntity((Entity)null, state.getBoundingBox(world, pos).offset(pos));
            if (!list.isEmpty())
            {
                if(state.getValue(FACING) == EnumFacing.SOUTH) {
                    world.setBlockState(pos, state.withProperty(IS_ACTIVE, true));
                    list = world.getEntitiesWithinAABBExcludingEntity((Entity)null, FANTOM_AABB[0].offset(pos));
                }
                if(state.getValue(FACING) == EnumFacing.NORTH) {
                    world.setBlockState(pos, state.withProperty(IS_ACTIVE, true));
                    list = world.getEntitiesWithinAABBExcludingEntity((Entity)null, FANTOM_AABB[1].offset(pos));
                }
                if(state.getValue(FACING) == EnumFacing.WEST) {
                    world.setBlockState(pos, state.withProperty(IS_ACTIVE, true));
                    list = world.getEntitiesWithinAABBExcludingEntity((Entity)null, FANTOM_AABB[2].offset(pos));
                }
                if(state.getValue(FACING) == EnumFacing.EAST) {
                    world.setBlockState(pos, state.withProperty(IS_ACTIVE, true));
                    list = world.getEntitiesWithinAABBExcludingEntity((Entity)null, FANTOM_AABB[3].offset(pos));
                }

            } else {
                world.setBlockState(pos, state.withProperty(IS_ACTIVE, false));
            }
        }
    }
    public int getWeakPower(IBlockState blockState, IBlockAccess blockAccess, BlockPos pos, EnumFacing side)
    {
        return ((Boolean)blockState.getValue(IS_ACTIVE)).booleanValue() ? 15 : 0;
    }

    public int getStrongPower(IBlockState blockState, IBlockAccess blockAccess, BlockPos pos, EnumFacing side)
    {
        return getWeakPower(blockState, blockAccess, pos, side);
    }
    @Override
    @SideOnly(Side.CLIENT)
    public AxisAlignedBB getSelectedBoundingBox(IBlockState state, World worldIn, BlockPos pos)
    {
        switch (state.getValue(FACING))
        {
        case SOUTH:
            return SIDE_AABB[0].offset(pos);
        case NORTH:
        default:
            return SIDE_AABB[1].offset(pos);
        case WEST:
            return SIDE_AABB[2].offset(pos);
        case EAST:
            return SIDE_AABB[3].offset(pos);
        }

    }

    @Override
    public AxisAlignedBB getBoundingBox(IBlockState state, IBlockAccess source, BlockPos pos)
    {
        switch (state.getValue(FACING))
        {
        case SOUTH:
            return FANTOM_AABB[0];
        case NORTH:
        default:
            return FANTOM_AABB[1];
        case WEST:
            return FANTOM_AABB[2];
        case EAST:
            return FANTOM_AABB[3];
        }
    }
    @Override
    protected BlockStateContainer createBlockState()
    {
        return new BlockStateContainer(this, new IProperty[] {FACING, IS_ACTIVE,DOWN});
    }
}

tox1cozZ · 27 Янв 2020

Вот, кстати, о чем я говорил. Стоит ограничение на обновление 1000 срочный блоков за тик. Когда ты костылишь "тайл на срочных блоках", ты забиваешь этот список, тем самым не давая нормально работать другим блокам. Например, обновлению редстоуна.

Icosider · 27 Янв 2020

Да на кол его!

TileEntity для слабаков

tox1cozZ

aka Agravaine

Maxik

Maxik

Eifel

Maxik

Eifel

tox1cozZ

aka Agravaine

Maxik

Eifel

Maxik

Liahim

Liahim

fukkivdan

Dahaka

tox1cozZ

aka Agravaine

Dahaka

tox1cozZ

aka Agravaine

Maxik

tox1cozZ

aka Agravaine

Icosider

Kotliner