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

А, там их два, я не тот взял
Ivasik написал(а):
"tick()" с хрена ли? В ITickable метод update, а не tick
Нажмите для раскрытия...
Dysphoria написал(а):
логика напрочь отсутствует...
Нажмите для раскрытия...
Почему? Все логично - получаем коорды энтити, получаем коорды тайла, затем путем простого вычитания выясняем расстояние между ними и активируем\деактивируем блок.
Правда, я думал можно по дефолту энтити брать, но оказывается только из листа, поэтому через коллижн логичней
 
Последнее редактирование:
Лол, в маяке ведь тоже самое, что я делал в блоке. Нахрена плодить классы, делать лишние тайлы, если все тоже самое делается в блоке?
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));
                }
            }
        }
    }

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

Маяк чисто для примера, как достать энтитей в радиусе
 
Ух горит..!
Но тут нету тут метода getAroundEntites, прикинь! Откуда я тебе возьму энтитей вокруг? Только через лист. Только мне не нужно брать всех энтитей вокруг. Мне надо брать конкретно ПЕРЕД блоком, а не ВСЕХ ВОКРУГ.
Eifel написал(а):
о твоих околонулевых знаниях джава и отсутсвия желания улучшаться
Нажмите для раскрытия...
А вот это обидно. То что я не изучал и никогда особо не делал тайлы, вообще не имеет отношения к тому, умею я в джаву или нет. Не умел бы - и через блок бы не сделал, лол)
Не надо говорить того, чего не знаешь, совет.
Eifel написал(а):
Маяк чисто для примера, как достать энтитей в радиусе
Нажмите для раскрытия...
Держу в курсе, в блоке тоже есть метод, который проверяет всех энтитей, залезших в блокбаунд. ОЙ, *, а оказывается в маяке то - тоже самое! А вы тут все топите за тайл, а тайл то оказыватся, тут и нахрен не нужен! Вот и горит у вас, потому что я прав))
 
Последнее редактирование:
Maxik написал(а):
Только через лист
Нажмите для раскрытия...
Так о этом и речь. В чем проблема - взял и проверил что тебе надо.
Maxik написал(а):
Не надо говорить того, чего не знаешь, совет.
Нажмите для раскрытия...
Ну про нулевые да наверное, немного загнул, извини, но смотря на то, что получаеться, складывается весьма не отдаленное мнение...
 
Maxik написал(а):
Мне надо брать конкретно ПЕРЕД блоком, а не ВСЕХ ВОКРУГ.
Нажмите для раскрытия...
Ты не можешь сделать проверку на сторону в которую смотрит блок и относительно этого сместить координаты выборки энтитей?

Maxik написал(а):
Держу в курсе, в блоке тоже есть
Нажмите для раскрытия...
Дак ты постоянно добавляешь свой блок в очередь на обновление, добавляя лишние расходы. Зачем это делать, если есть тайл который лять специально и придуман для блоков которые постоянно обновляются без всяких очередей и их обслуживания.
Изобрел свой велосипед и пытается доказать что он лучше... Мда
 
Могу. Я просто сейчас занялся этим, чтобы понять ,а можно ли тут что то было сделать более удобно, чем в блоке. Пока что, разницы вообще никакой. Даже больше - разница отрицательна, т.к я сейчас создаю кучу всего, чего в принципе и не нужно.
 
Потому что ты еще ничего не сделал и даже не понятно, что ты делаешь. Создай хотя бы рабочий вариант с тайлом, а там уже поговорим о оптимизации твоего кода...
 
Dysphoria написал(а):
Дак ты постоянно добавляешь свой блок в очередь на обновление, добавляя лишние расходы.
Нажмите для раскрытия...
Какая еще очередь на обновление? Каждый блок тикает сам по себе, как и тайл.
 
@Maxik, это называется "архитектура кода".
У себя я тоже очень часто в тайле исполбзую методы блока и наоборот. Бывают случаи, когда приходится несколько раз скакать туда обратно. Главное, чтобы это было логически оправдано и удобно.
Ты, как будто, не видишь или не понимаешь общей картины. Программа не замыкается на одном классе, смотри шире.
И это совсем не связано со знанием языка, как выше и было замечено. Это кое что поглубже.
 
Чел, тайл это очень легко, для себя я просто провел аналогию.
Блок это как итем, бо он тоже синглтон. У итема есть итемстак, что у игрока в инвенте/сундуке/попке лежит (уникальный, да).
А у блока есть тайл(не всегда, кстати), который так же стоит в мире уникальный (Если что можете меня поправить)
Они юзабельны шо капец, но я стараюсь тайлы не юзать особо, пока еще не приспичило :D
Если ты говоришь что тайлы сложно, то значит ты не практиковался с ними и не хочешь этого делать
 
А с чего вы вообще взяли, что через тайл решить эту задачу - лучше? Что насчёт сложности алгоритмов в обоих случаях? Об этом нужно думать изначально, а не после того, как все реализовано.
 
Разве там сделано не на коллизии? (я не видел кода)
 
@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});
    }
}
 
Последнее редактирование:
1580125152037.png
Вот, кстати, о чем я говорил. Стоит ограничение на обновление 1000 срочный блоков за тик. Когда ты костылишь "тайл на срочных блоках", ты забиваешь этот список, тем самым не давая нормально работать другим блокам. Например, обновлению редстоуна.
 
Войдите или зарегистрируйтесь для ответа.
Назад
Сверху