(Forge 1.16.5) Не понимаю, почему крашает

[PainWizzard]

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

package com.painwizzard.magical.entities;

import net.minecraft.entity.EntityType;
import net.minecraft.entity.LivingEntity;
import net.minecraft.entity.player.PlayerEntity;
import net.minecraft.entity.projectile.DamagingProjectileEntity;
import net.minecraft.network.IPacket;
import net.minecraft.particles.IParticleData;
import net.minecraft.particles.ParticleTypes;
import net.minecraft.potion.EffectInstance;
import net.minecraft.potion.Effects;
import net.minecraft.util.DamageSource;
import net.minecraft.util.SoundCategory;
import net.minecraft.util.SoundEvents;
import net.minecraft.util.math.BlockRayTraceResult;
import net.minecraft.util.math.EntityRayTraceResult;
import net.minecraft.world.World;
import net.minecraftforge.fml.network.NetworkHooks;

public class VampireBall extends DamagingProjectileEntity {
    private LivingEntity caster = null;
    public IPacket<?> getAddEntityPacket() {
        return NetworkHooks.getEntitySpawningPacket(this);
    }
    protected VampireBall(EntityType<? extends DamagingProjectileEntity> p_i50173_1_, World p_i50173_2_) {
        super(p_i50173_1_, p_i50173_2_);
    }

    public VampireBall(EntityType<? extends DamagingProjectileEntity> pEntityType, double pX, double pY, double pZ, double pOffsetX, double pOffsetY, double pOffsetZ, World pLevel) {
        super(pEntityType, pX, pY, pZ, pOffsetX, pOffsetY, pOffsetZ, pLevel);
    }

    public VampireBall(EntityType<? extends DamagingProjectileEntity> pEntityType, LivingEntity pShooter, double pOffsetX, double pOffsetY, double pOffsetZ, World pLevel) {
        super(pEntityType, pShooter, pOffsetX, pOffsetY, pOffsetZ, pLevel);
        caster = pShooter;
    }

    @Override
    protected void onHitEntity(EntityRayTraceResult pResult) {
        this.remove();
        if (pResult.getEntity() instanceof LivingEntity) {
            World world = this.level;
            LivingEntity entity = (LivingEntity) pResult.getEntity();
            entity.hurt(DamageSource.MAGIC, 4);
            caster.heal(2.0f);
        }
        super.onHitEntity(pResult);
    }

    @Override
    protected boolean shouldBurn() {
        return false;
    }

    @Override
    protected float getInertia() {
        return 1.0f;
    }

    @Override
    protected IParticleData getTrailParticle() {
        return ParticleTypes.ENCHANT;
    }

    @Override
    protected void onHitBlock(BlockRayTraceResult pResult) {
        this.remove();
        super.onHitBlock(pResult);
    }
}

 

[_SupCM_]

private LivingEntity caster = null;

Тебе нужно внутри шарика хранить UUID кастера и хилить уже по UUID, а не по внутриклассовому полю. На всякий: UUID хранится в NBT.
Так сделано в ванилле (стрелы, файерболлы и т.д.), так следует сделать и тебе.

 

[PainWizzard]

я в этом не очень силен, можешь дать пример кода, пожалуйста? я с uuid ни разу не работал

 

[Ingresso]

Скинь код, где у тебя происходит создание projectile

 

[_SupCM_]

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

Нет IDE под рукой на эту версию.

я с uuid ни разу не работал

UUID можно получить с ЛЮБОГО entity. Этот UUID можно легко записать в NBT. Entity (который кастует) можно указывать в конструкторе, в конструкторе же и вносить все необходимые изменения.
Пример только для 1.20 есть:

    public SpellEntity(EntityType<? extends SpellEntity> type, Level level) {
        super(type, level);
    }

    public SpellEntity(EntityType<? extends SpellBallEntity> type, LivingEntity entity, Spell spell) {
        super(type, entity, getSpellVec(entity).x, getSpellVec(entity).y, getSpellVec(entity).z, entity.level());
        this.spell = spell;
        Vec3 pos = new Vec3(entity.getPosition(0.5f).x,
                entity.getPosition(0.5f).y + 1, entity.getPosition(0.5f).z);
        moveTo(pos);
    }

    //.....

    @Override
    public void addAdditionalSaveData(CompoundTag tag) {
        super.addAdditionalSaveData(tag);
        CompoundTag spellTag = spell.writeToNbt();
        if(spellTag != null)
            tag.put("Spell", spellTag);

    }

    @Override
    public void readAdditionalSaveData(CompoundTag tag) {
        super.readAdditionalSaveData(tag);
        CompoundTag spellTag = (CompoundTag) tag.get("Spell");
        if(spellTag != null)
            this.spell = Spell.readFromNbt((ServerLevel) level(), spellTag);
    }

 

[PainWizzard]

Скинь код, где у тебя происходит создание projectile

@Override
    public ActionResult<ItemStack> use(World pLevel, PlayerEntity pPlayer, Hand pHand) {
        if (!pLevel.isClientSide()) {
            pLevel.playSound(null, pPlayer.getX(), pPlayer.getEyeY(), pPlayer.getZ(),
                    ModSoundEvents.WAND.get(), SoundCategory.BLOCKS, 0.8f, 1);
            VampireBall vampireBall = new VampireBall(ModEntities.VAMPIRE_BALL.get(), pPlayer.getX(), pPlayer.getEyeY(), pPlayer.getZ(), 0, 0, 0, pLevel);
            vampireBall.caster = pPlayer;
            Vector3d look = pPlayer.getViewVector(1.0f);
            double d0 = 4.0d;
            vampireBall.setDeltaMovement(look.x() * d0, look.y() * d0, look.z() * d0);
            pLevel.addFreshEntity(vampireBall);
            pPlayer.getCooldowns().addCooldown(this, 60);
            ItemStack stack = pPlayer.getItemInHand(pHand);
            stack.setDamageValue(stack.getDamageValue() + 1);
            if (stack.getDamageValue() >= stack.getMaxDamage()) stack.setCount(0);
            return ActionResult.success(stack);
        }
        return super.use(pLevel, pPlayer, pHand);
    }

 

[PainWizzard]

UUID можно получить с ЛЮБОГО entity. Этот UUID можно легко записать в NBT. Entity (который кастует) можно указывать в конструкторе, в конструкторе же и вносить все необходимые изменения.

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

 

[_SupCM_]

В CompoundTag есть отдельный метод putUUID(), UUID ты берёшь из Entity (в твоём случае это caster). Также происходит и считывание UUID.
Ничего сложного.

 

[_SupCM_]

@Override
    public ActionResult<ItemStack> use(World pLevel, PlayerEntity pPlayer, Hand pHand) {
        if (!pLevel.isClientSide()) {
            pLevel.playSound(null, pPlayer.getX(), pPlayer.getEyeY(), pPlayer.getZ(),
                    ModSoundEvents.WAND.get(), SoundCategory.BLOCKS, 0.8f, 1);
            VampireBall vampireBall = new VampireBall(ModEntities.VAMPIRE_BALL.get(), pPlayer.getX(), pPlayer.getEyeY(), pPlayer.getZ(), 0, 0, 0, pLevel);
            vampireBall.caster = pPlayer;
            Vector3d look = pPlayer.getViewVector(1.0f);
            double d0 = 4.0d;
            vampireBall.setDeltaMovement(look.x() * d0, look.y() * d0, look.z() * d0);
            pLevel.addFreshEntity(vampireBall);
            pPlayer.getCooldowns().addCooldown(this, 60);
            ItemStack stack = pPlayer.getItemInHand(pHand);
            stack.setDamageValue(stack.getDamageValue() + 1);
            if (stack.getDamageValue() >= stack.getMaxDamage()) stack.setCount(0);
            return ActionResult.success(stack);
        }
        return super.use(pLevel, pPlayer, pHand);
    }

Entity создаётся 2 раза: для левой и для правой руки.

 

[PainWizzard]

В CompoundTag есть отдельный метод putUUID(), UUID ты берёшь из Entity (в твоём случае это caster). Также происходит и считывание UUID.
Ничего сложного.

в этом то и суть я вообще ни один из вышеперечисленных терминов не понимаю и как с этим работать, соответственно, тоже

 

[_SupCM_]

UUID - это отдельный идентификатор для каждой сущности. Он уникален.
Мир о каждом таком идентификаторе знает, и может по нему достать соотв. сущность.
UUID представляет собой некоторую стр. данных вида "{1D2E4F5-H6K8....}" (если будем это дело переводить в String).

И для осуществления передачи сущности из п. А в п. Б (через сущность, тайл и т.д.) нам необходим UUID. Мы записываем его в NBT (см. 5 сообщение в этом треде). Этот NBT мы сможем по итогу прочитать в необх. нам момент времени и получить необходимую нам сущность через мир.

Напрямую в конструктор UUID пихать нежелательно, поэтому мы туда пихаем Entity, который вызывает нашу сущность. И уже с Entity мы берём UUID, который запишем в NBT.

 

[PainWizzard]

угу, и как же достать entity?

 

[_SupCM_]

PlayerEntity pPlayer

Вот твой Entity. Там буквально это и указано.

 

[PainWizzard]

я или слишком тупой или я не знаю. вот я впихнул в конструктор себя(плеера), после чего я достану у него uuid, запишу его в nbt, а дальше я даже на словах не понимаю, не говоря уже о коде

 

[_SupCM_]

Значит у тебя нет понимания программирования в принципе. Программируй больше, и однажды осознаешь всё здесь написанное.
Из примера ниже можешь увидеть пример реализации (но для кастомного класса Spell). На этих костях можешь построить свой прожектайл.

public abstract class SpellEntity extends AbstractHurtingProjectile implements ISpellEntity {

    private Spell spell;
    public SpellEntity(EntityType<? extends SpellEntity> type, Level level) {
        super(type, level);
    }

    public SpellEntity(EntityType<? extends SpellBallEntity> type, LivingEntity entity, Spell spell) {
        super(type, entity, getSpellVec(entity).x, getSpellVec(entity).y, getSpellVec(entity).z, entity.level());
        this.spell = spell;
        Vec3 pos = new Vec3(entity.getPosition(0.5f).x,
                entity.getPosition(0.5f).y + 1, entity.getPosition(0.5f).z);

        moveTo(pos);
    }

    @Override
    protected void onHitEntity(EntityHitResult result) {
        if(!level().isClientSide && spell != null && result.getEntity() instanceof LivingEntity le) {
            List<ISpellPart> parts = spell.getPartsWithoutTrigger();
            parts.remove(0);
            damageBySpell(le);
            spell.getElement().onHitEntity(level(), le, spell.getPlayer());
            for (ISpellPart part : parts) {
                part.effect(result.getEntity(), spell);
            }
            if (spell.getParts().stream().anyMatch(part -> part instanceof RangeModifier)) {
                List<LivingEntity> entities = le.level().getEntitiesOfClass(LivingEntity.class,
                        new AABB(result.getEntity().blockPosition()
                                .below((int) spell.getArea())
                                .north((int) spell.getArea())
                                .west((int) spell.getArea()),
                                result.getEntity().blockPosition()
                                        .above((int) spell.getArea())
                                        .south((int) spell.getArea())
                                        .east((int) spell.getArea()))
                );
                for (LivingEntity e : entities) {
                    if(e == spell.getPlayer())
                        continue;
                    damageBySpell(e);
                    spell.getElement().onHitEntity(level(), e, spell.getPlayer());
                    for (ISpellPart part : parts) {
                        part.effect(e, spell);
                    }
                }
            }
        }
        super.onHitEntity(result);
    }

    abstract void damageBySpell(LivingEntity e);

    public Spell getSpell() {
        return spell;
    }

    @Override
    protected void onHitBlock(BlockHitResult result) {
        super.onHitBlock(result);

        if(!level().isClientSide && spell != null)
            spell.getElement().onHitBlock(level(), result.getBlockPos(), spell.getPlayer());
    }

    @Override
    protected void onHit(HitResult result) {
        super.onHit(result);
        discard();
    }

    static Vec3 getSpellVec(LivingEntity entity) {
        return new Vec3(entity.getLookAngle().x, entity.getLookAngle().y, entity.getLookAngle().z);
    }

    @Override
    public void addAdditionalSaveData(CompoundTag tag) {
        super.addAdditionalSaveData(tag);
        CompoundTag spellTag = spell.writeToNbt();
        if(spellTag != null)
            tag.put("Spell", spellTag);
    }

    @Override
    public void readAdditionalSaveData(CompoundTag tag) {
        super.readAdditionalSaveData(tag);
        CompoundTag spellTag = (CompoundTag) tag.get("Spell");
        if(spellTag != null)
            this.spell = Spell.readFromNbt((ServerLevel) level(), spellTag);
    }

    @Override
    protected boolean shouldBurn() {
        return spell != null && spell.getElement().getId().equals("fire");
    }
}

 

[PainWizzard]

Значит у тебя нет понимания программирования в принципе

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

 

[Ingresso]

я или слишком тупой или я не знаю. вот я впихнул в конструктор себя(плеера), после чего я достану у него uuid, запишу его в nbt, а дальше я даже на словах не понимаю, не говоря уже о коде

Если вкратце, то это нужно для того, чтобы при перезаходе в мир не обнулялся атакующий игрок в Projectile. Т.е. если бы ты перезашел, и этот projectile до кого-то долетел, ты бы также словил NullPointerException, но только по другой причине. А так при столкновении с другим entity, всегда будет известен атакующий entity. Наверное это хотел донести @_SupCM_

 

[_SupCM_]

Т.е. если бы ты перезашел, и этот projectile до кого-то долетел, ты бы также словил NullPointerException, но только по другой причине.

Даже без перезахода такая ситуация возникает. Здесь уже не подскажу почему.

А так при столкновении с другим entity, всегда будет известен атакующий entity.

Именно так.

 

[Ingresso]

Даже без перезахода такая ситуация возникает. Здесь уже не подскажу почему.

Такое может произойти при выгрузке чанков. Было у меня такое