Monster issue

[Spring]

I want the monster not to attack the player, this is my code

@Override
        public void onArmorTick(ItemStack stack, World world, PlayerEntity player) {
            if (player.isAlive() && Screen.hasShiftDown()) {
                if (hasFullArmor(player) && hasCorrectArmorOn(ArmorMaterial.SILK_NIGHT_ARMOR, player)) {
                    List<MobEntity> mobEntities = getNearbyEntities(player);
                    for (MobEntity mobEntity : mobEntities) {
                        Optional<LivingEntity> target = Optional.ofNullable(mobEntity.getTarget());
                        target.ifPresent(res -> {
                            if (target.get().getUUID().equals(player.getUUID())) {
                                mobEntity.setTarget(null);
                            }
                        });
                    }
                }
            }
            super.onArmorTick(stack, world, player);
        }

I don't know how to explain this situation, please watch my video

https://error-1254387167.cos.ap-chengdu.myqcloud.com/bugs-video.mp4

 

Sometimes it’s useful, sometimes it’s not.

All code has been executed

 

Edited December 7, 2021 by Spring

[Spring]

  On 12/7/2021 at 5:17 AM, Spring said:

I want the monster not to attack the player, this is my code

@Override
        public void onArmorTick(ItemStack stack, World world, PlayerEntity player) {
            if (player.isAlive() && Screen.hasShiftDown()) {
                if (hasFullArmor(player) && hasCorrectArmorOn(ArmorMaterial.SILK_NIGHT_ARMOR, player)) {
                    List<MobEntity> mobEntities = getNearbyEntities(player);
                    for (MobEntity mobEntity : mobEntities) {
                        Optional<LivingEntity> target = Optional.ofNullable(mobEntity.getTarget());
                        target.ifPresent(res -> {
                            if (target.get().getUUID().equals(player.getUUID())) {
                                mobEntity.setTarget(null);
                            }
                        });
                    }
                }
            }
            super.onArmorTick(stack, world, player);
        }

I don't know how to explain this situation, please watch my video

https://error-1254387167.cos.ap-chengdu.myqcloud.com/bugs-video.mp4

 

Sometimes it’s useful, sometimes it’s not.

All code has been executed

 

Expand  

 

Found the problem, getNearbyEntities parameter problem. But now the monster will look at me, is there a way to avoid it?

[Luis_ST]

with the way you use, no. If you want to avoid that, you need to replace the Target Goal of the Entity. btw, this is the better way to prevent monsters from attacking the Player instead setting the target to null

 

[Spring]

  On 12/7/2021 at 9:59 AM, Luis_ST said:

with the way you use, no. If you want to avoid that, you need to replace the Target Goal of the Entity. btw, this is the better way to prevent monsters from attacking the Player instead setting the target to null

 

Expand  

What to replace the entity with?

[Luis_ST]

Not the Entity itself, the TargetGoal of the Entity. If you want to do that, create a custom TargetGoal and replace it in EntityJoinWorldEvent

[Spring]

  On 12/7/2021 at 11:24 AM, Luis_ST said:

Not the Entity itself, the TargetGoal of the Entity. If you want to do that, create a custom TargetGoal and replace it in EntityJoinWorldEvent

Expand  

so complicated

[Luis_ST]

No it's not, no less complicated than the code you are currently using, also your code is bad for performance since you run it each tick 

[Spring]

  On 12/7/2021 at 1:31 PM, Luis_ST said:

No it's not, no less complicated than the code you are currently using, also your code is bad for performance since you run it each tick 

Expand  

like this ?

 

@Override
        public void onArmorTick(ItemStack stack, World world, PlayerEntity player) {
            if (invalid(player)) {
                List<MobEntity> mobEntities = getNearbyEntities(player);
                for (MobEntity mob : mobEntities) {
                    if (mob.getTarget() != null && mob.getTarget().getUUID().equals(player.getUUID())) {
                        mob.goalSelector.removeGoal(new NearestAttackableTargetGoal<>(mob, PlayerEntity.class, true));
                    }
                }
            }
        }

 

[Luis_ST]

  On 12/12/2021 at 7:49 AM, Spring said:

like this ?

Expand  

no:

  On 12/7/2021 at 11:24 AM, Luis_ST said:

replace it in EntityJoinWorldEvent

Expand  

also this won't work:

  On 12/12/2021 at 7:49 AM, Spring said:

mob.goalSelector.removeGoal(new NearestAttackableTargetGoal<>(mob, PlayerEntity.class, true));

Expand  

first you need to use the targetSelector,
second use GoalSelector#getAvailableGoals