Помогите Действия с лошадьми [Решено]

Получил сущности. Лошадь - как проверить, сидит ли на ней игрок?
Как принудительно его заставить слезть с лошади?

 

EntityMountEvent[DOUBLEPOST=1484728794,1484728418][/DOUBLEPOST]

Также могут подойти другие вариантьі:

Код:

 @EventHandler
public void onMount(PlayerInteractEntityEvent e){
e.getRightClicked().getPassenger().equals(e.getPlayer());
}

Еще ивент:
VehicleEnterEvent

Думаю хоть один подойдет[DOUBLEPOST=1484729080][/DOUBLEPOST]Просто чтоб слезть, юзай e.setcanceled(true);

 

Идея такова, что мне нужно при вводе команды получить живность вокруг, определить что это лошадь, проверить сидит ли кто на ней и если да, то снять его.[DOUBLEPOST=1484729989,1484729521][/DOUBLEPOST]

Код:

List<Entity> en = senderplayer.getNearbyEntities(x, x, x);
                for (Entity e : en) {
                    if (e.getType().equals(EntityType.HORSE)) {
                        LivingEntity le = (LivingEntity) e;
                        if (!le.isEmpty()) { //Если лошадь не пустая, то:
                            Player p = (Player) le.getPassenger(); //получили игрока
                            p.teleport(le.getLocation()); //телепортировали его на координаты лошади, тем самым сняв его с неё.
                        } //else ничего не делам, т.е. на лошади никого нет.
                    }
                }

[DOUBLEPOST=1484730267][/DOUBLEPOST]Знатоки, норм решение?

 

p.leaveVehicle();

 

p.isInsideVehicle() - проверяй игроков, есть ли они на транспорте

 

Спасибо, поправил.[DOUBLEPOST=1484731903,1484731839][/DOUBLEPOST]

Ты предлагаешь изменить суть проверки? Т.е. получать игроков, а потом смотреть есть ли они на транспорте и снимать их.

 

Это лишнее, получай пасендера и снимай его.

 

Код:

if (!le.isEmpty()&&le.getPassenger() instanceof Player) { //Если лошадь не пустая, и на лошади именно игрок, то:
                    p = (Player) le.getPassenger(); //получили игрока
                    p.leaveVehicle(); //пинаем в кабздец игрока
                } //else ничего не делам, т.е. на лошади никого нет.

Добавляем проверку на игрока.
Зачем? чтобы не выловить каст эксепшон.

Спойлер: откуда каст эксепшон

 

Как второй вариант... Решать тебе, лучше иметь запасной вариант ,при ошибке заменить можно

 

Есть же типы, instanceof тут не уместен.

 

Это как? а то я мало знаю

 

.getPassenger().getType()[DOUBLEPOST=1484732164,1484732125][/DOUBLEPOST]if(h.getPassenger().getType() == EntityType.PLAYER) {

 

а точно.

я тупенький местами.
тогда такой if (не знаю какая разница, но раз @alexandrage так сказал, значит так и есть)
if (!le.isEmpty()&&le.getPassenger().getType().equals(EntityType.PLAYER))

 

Типы проверяются через == Ибо это enum.

 

Версия 1,7,10. Каким образом живность (не Player) может сесть на лошадь?
Какой вариант всё же выбирать, ибо я вижу как минимум 3 работающих. \
Помогите с выбором.

 

Разве еквалс не медленее чем == ?

 

Код:

if (!le.isEmpty()&&le.getPassenger().getType() == EntityType.PLAYER) { //Если лошадь не пустая, и на лошади именно игрок, то:
                    p = (Player) le.getPassenger(); //получили игрока
                    p.leaveVehicle(); //пинаем в кабздец игрока
                } //else ничего не делам, т.е. на лошади никого нет.

Essentials
/spawnmob horse:saddled,pig

 

Просто делаешь проверку игрок там или нет.

if(horse.getPassenger().getType() == EntityType.PLAYER) {

}[DOUBLEPOST=1484732690,1484732605][/DOUBLEPOST]

Он просто и не понимает, что там enum.

 

спасибо.

 

Код:

  Player senderplayer = (Player) sender;
  en = senderplayer.getNearbyEntities(3, 3, 3);

  for (Entity e : en) {
   if (e.getType() == EntityType.PLAYER) {
    Player p = (Player) e;
    if (p.isInsideVehicle()) {
     p.leaveVehicle();
     p.sendMessage(ChatColor.GOLD  + "");
     senderplayer.sendMessage(ChatColor.GOLD  + "");
    }
   }
  }

Я всё же выбрал такой, так как если у живности рядом нет пассажира, то получаю NPI и делать ещё один if не стал. Команда будет применяться только в крайних случаях, по этому не суть важно.