Плагин [TP] TpPoint v1.0 - Варп-зоны! (Физический телепорт) [1.6.*-1.7.*]

А, сейчас скажу, подожди немного.
Во-первых, в цикле создаётся огромное количество String из-за загрузки точек. Я бы хранил все точки в списке, и заполнял бы его при загрузке плагина.

Дальше - какое-то странное двойное условие цикла. Я его не понял совсем. Я бы сделал for (Warp warp : warps), и в цикле if, проверяющий совпадение локаций игрока и варпа и наличие target.

Ну и, если переписать всё на список, то телепорт игрока сводится просто к player.teleport(warp.getTarget()).

Короче, можно было лучше и более ООП.
Представь, когда у тебя варпов 30 и 100 игроков на сервере, ОМГ

 

Шедулер

 

@ql_Nik_lp
А как его прикрутить в этот плугин?

 

Я изначально хотел как раз сделать ООП в плане варпа. Но появились трудности, которые не смог решить. На тот момент.

И каждый тик проветять?.. Почти тоже самое. Ну да, минус поворот головой, но всё равно похоже.

 

Что мешает проверять каждые 10 тиков?

 

Не знаю в чем может быть дело. Мне вот тоже жаловались на MW. Я себе поставил, проверил -- всё работает нормально.

Неее! Это некрасиво. И излишне, на мой вкус. Во-первых, там ничего страшного нет. Цикл и сравнение - его туда много влезет
Если уж добавлять "оптимизации", то я бы проверял переходит ли игрок при движении на новый блок и если нет - ничего не делал. Т.е. что-то вроде:

Код:

    @EventHandler(priority=EventPriority.NORMAL, ignoreCancelled = true)
    public void onPlayerMove (PlayerMoveEvent event){
        if (event.getFrom().getBlockX()==event.getTo().getBlockX()&&
                event.getFrom().getBlockY()==event.getTo().getBlockY()&&
                        event.getFrom().getBlockZ()==event.getTo().getBlockZ()) return;
         // здесь остальной код
    }

 

Командные блоки типа не в моде?)

 

Это непотребство сойдет разве что для адвенчур карт, у нас же есть инструменты мощнее.

 

Окей, оптимизирую. Спасибо за советы.