Jump to content

(1.16.5 MCP) getShooter in DamagingProjectileEntity throws nullPointerException


ElTotisPro50

Recommended Posts

so i have an entity that extends DamagingProjectileEntity and it has tick method and i want to play a sound(inside of my projectile class) when a little time passes but im trying to do getShooter.playsound or getShooter.getEntity().playsound but it doesnt work(as i said it throws nullPointerException) by the way the crash log ONLY says nullPointerException in that line

 

@Override
    public void tick() {
        super.tick();
        ++this.time;
        System.out.println(time);
        if(!world.isRemote) {
            if(time == 70) {
                getShooter().playSound(ModSounds.MYSOUND.get(),1,1); //or getShooter().getEntity().playSound(ModSounds.MYSOUND.get(),1,1); doesnt work either
                 this.remove();
            }
        }
    }

 

Edited by ElTotisPro50
Link to comment
Share on other sites

58 minutes ago, diesieben07 said:

getShooter must be explicitly set by you and even then it can return null (e.g. if the player logged out). So you can't just use it directly, you must check first.

Are you sure you want to play the sound at the shooter's position? Why do you not just use this.playSound?

check if shooter is playerentity?

if(getShooter() instanceof PlayerEntity)

 

this.playsound will play the sound in the throwable entity's position right?if i throw my entity with acceleration or motion the player could'nt hear it because the entity is far enough

Link to comment
Share on other sites

5 hours ago, diesieben07 said:

Yes, this will also ensure the shooter is not null.

Just making sure. If you do want to play it at the player's position then what you did is correct. You just have to actually set the shooter when creating the projectile.

is not throwing nullPointerException or crashing or something it just doesnt do anything

@Override
    public void tick() {
        super.tick();
        ++this.time;
        System.out.println(time);
        if(!world.isRemote) {
            if(time == 70) {
                if(getShooter() instanceof PlayerEntity) { //if(getShooter().getEntity() instanceof PlayerEntity) { this just will throw null again
                    getShooter().playSound(ModSounds.MYSOUND.get(),1,1); //getShooter().getEntity().playSound(ModSounds.MYSOUND.get(),1,1); doesnt work either
                }
                this.remove();
            }
        }
    }

 

Link to comment
Share on other sites

10 hours ago, diesieben07 said:

Show more of your code, specifically the whole projectile class and where you spawn the projectile.

My item class

public class MyItem extends Item {

    public MyItem(Properties properties) {
        super(properties);
    }

	@Override
    public ActionResult<ItemStack> onItemRightClick(World world, PlayerEntity player, Hand handIn) {
        ItemStack stack = player.getHeldItemMainhand();
        if(player.isSneaking()) {
            world.playSound(player,player.getPosition(),ModSounds.SOUND.get(),SoundCategory.MASTER, 1, 1);
        }
        if(!world.isRemote) {
            if (player.isSneaking()) {
                if (stack.getItem() instanceof MyItem) {
                    ThrownMyEntity entity = new ThrownMyEntity(ModEntityTypes.THROWNMYENTITY.get(), world);
                    Vector3d aim = player.getLookVec();
                    entity.setPosition(player.getPosX() + aim.x * 1.7,
                            player.getPosY() + (aim.y + 0.5) * 1.7,
                            player.getPosZ() + aim.z * 1.7);
                    entity.rotationYaw = player.rotationYaw;
                    entity.rotationPitch = player.rotationPitch;
                    double throwSpeed = 0.7;
                    entity.setMotion(aim.x * throwSpeed,aim.y * throwSpeed,aim.z * throwSpeed);
                    world.addEntity(entity);
                    player.setItemStackToSlot(EquipmentSlotType.MAINHAND, ItemStack.EMPTY);
                }
            }
        }
        return super.onItemRightClick(world, player, handIn);
    }
}

 

ThrownMyEntity Class

public class ThrownMyEntity extends DamagingProjectileEntity implements ITickableTileEntity {

    private int time;

    public ThrownMyEntity(EntityType<? extends DamagingProjectileEntity> entityType, World world) {
        super(entityType,world);
    }

    //This makes that the entity cant be pushed
    @Override
    public boolean canBePushed() {
        return false;
    }

    //This makes the entity can be hurt(like a ghost)
    @Override
    public boolean canBeCollidedWith() {
        return false;
    }

	@Override
    public void tick() {
        super.tick();
        ++this.time;
        System.out.println(time);
        if(!world.isRemote) {
            if(time == 70) {
                if(getShooter() instanceof PlayerEntity) { //if(getShooter().getEntity() instanceof PlayerEntity) { this just will throw null again
					getShooter().getEntity.DOSOMETHINGTOTHESHOOTER
                }
                this.remove();
            }
        }
    }

    @Override
    protected void registerData() {

    }

    @Override
    public boolean isInWater() {
        return false;
    }

    @Override
    public boolean isInLava() {
        return false;
    }

    @Override
    public boolean isBurning() {
        return false;
    }

    @Override
    public boolean isGlowing() {
        return false;
    }

    @Override
    public IPacket<?> createSpawnPacket() {
        return NetworkHooks.getEntitySpawningPacket(this);
    }

    @Override
    public boolean isWet() {
        return false;
    }

}

 

Link to comment
Share on other sites

23 minutes ago, ElTotisPro50 said:

right?

no not to getShooter, call setShooter from your Item in onItemRightClick then use the Player which you get as a parameter

4 minutes ago, ElTotisPro50 said:

what i just told you didnt work, throws null again

yeah since:

Entity entity = getShooter(); // shooter is at this point null
setShooter(entity); // the new shooter is null since the old shooter was null

 

Link to comment
Share on other sites

1 hour ago, Luis_ST said:

no not to getShooter, call setShooter from your Item in onItemRightClick then use the Player which you get as a parameter

yeah since:

Entity entity = getShooter(); // shooter is at this point null
setShooter(entity); // the new shooter is null since the old shooter was null

 

it worked :)

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.