Плагин На удаление CustomJoinStream

штоооо


wtf?
https://github.com/Mr-Krab/CustomJo...mr_krab/customjoinstream/events/Join.java#L30
https://github.com/Mr-Krab/CustomJo...mr_krab/customjoinstream/events/Quit.java#L25
https://github.com/Mr-Krab/CustomJo..._krab/customjoinstream/utils/CmdExecutor.java

 

Чем тебе не нравится слушатель эвента? Если нет перма, то меняет дефолтное сообщение на null, если есть то выставляет из конфига. На счет команд, переделываю, сам хочу по человечески сделать и там аргументы буду добавлять еще для новых функций..

 

if(!expression) {
//empty block
} else {
//code
}

доставило

Код:

                    sender.sendMessage(plugin.loc.getString("HelpLine1"));
                    sender.sendMessage(plugin.loc.getString("HelpLine2"));

 

Нифига не понял, что тебе в слушателе не понравилось, ведь он работает именно так как надо ему работать, на счет строк справки я еще больше тебя не понял. В 2 строки сделал для большего удобства.

 

Он намекал на то, что код можно сократить до

Код:

if(p.hasPermission("customjoinstream.stream")) {
    je.setJoinMessage(instance.getConfig().getString("Messages.JoinMessage").replace("%prefix", prefix).replace("%nickname", je.getPlayer().getName()).replace("%suffix", suffix).replace("&", "§"));
    return true;
}

без этого бубна с '!' и "else"

 

никогда не думал, что бубен - туалетная бумага

таким бубном только жопу подтереть

 

принимай
https://github.com/Mr-Krab/CustomJoinStream/pull/1

 

Вместо такого убого реплейса юзай pattern

 

@glavrak

 

Занят доработкой плагина, кое какие твои правки использовал, но не все.

 

Плагин обновлен до версии 1.2.2
Взяты некоторые наработки @DonDays, скажем спасибо ему за это.
Плагин теперь требует наличия Vault!
Плагин частично перекодирован.
Исправлены сообщения об ошибках в консоли.
Переписаны файлы локализаций.

Спойлер: Добавлены команды

/cjs setjoinloc - customjoinstream.setloc - Установка стартовой позиции
/cjs tpjoin - customjoinstream.tpjoin - Телепорт на стартовую позицию
/cjs setspawn - customjoinstream.setloc - Установка спавна
/cjs spawn - customjoinstream.spawn - Телепорт на спавн

Добавлена телепортация на стартовую позицию. Включается в конфиге.

 

https://github.com/Mr-Krab/CustomJo..._krab/customjoinstream/events/EventUtils.java
https://github.com/DonDays/CustomJo...src/mr_krab/customjoinstream/utils/Utils.java
getNearby

зачем регистрировать ОДНУ команду таким сложным способом?
https://github.com/Mr-Krab/CustomJoinStream/blob/master/src/mr_krab/customjoinstream/Plugin.java

 

Работает же все проблем не вызывает. По поводу команды, тут больше для себя как-то, для обучения. И через Plugin.yml не стал делать еще и потому, что буду пробовать на губку перенести плагин с привязкой к конкретному плагину пермов, скорее всего к LuckPerms.
Сейчас еще обнову подготавливаю. Добавлю скрытие всех игроков(для авторизации самое то), скрытие при входе для админов/модеров(перм выдал и готово). Со скрытием для админов и модеров помимо активации при входе есть еще включение и выключение командой с 2мя аргументами, хотел сделать одним как переключатель, но блин зараза простая проверка на статус видимости идет через задницу. Надо еще сделать проверку на наличие дополнительного конфига в папке плагина и доработать проверку на сендера команд, а то в консоли заметил все-таки не блочит аргументы. EventUtils.java перенес в пакет Utils и дополнил функциями.

 

замени этот повторяющийся код, посмотри в

зачем тут ретурны в конце?

Код:

    public static void playerJoinTp (Player player) {
        if(!Plugin.getInstance().getConfig().getBoolean("TpToStartLoc.Join")) {
            return;
        }
        Location join = Plugin.getInstance().locm.TpJoinLoc(player.getName());
        player.teleport(join);
    }

https://github.com/Mr-Krab/CustomJo...rc/mr_krab/customjoinstream/events/Death.java
методы с большой буквы не называют
зачем вообще boolean в обработчике события? ты ничего не должен возвращать

 

На что ты мне предлагаешь его заменить если там требуется 3 значения Double для воспроизведения звуков и эффектов в радиусе? Скажи, заменю, не проблема.

Хз, смотрел на подсказки эклипса при написании, потребовал return, я вставил. Сейчас убрал, вроде не требует вернуть, иногда чушь кидает, бывает не видит объявленные переменные, приходится перезапускать.

 

СКорее всего у вас руки из одного места.
Либо комп-мамонт с 1 Гб оперативки.
Если (вдруг) он что-то не видит, достаточно сохранить изменения, спецом для этого там целых две кнопки с дискетами.

 

читай внимательно https://github.com/DonDays/CustomJo...src/mr_krab/customjoinstream/utils/Utils.java

 

А вот оно что, в отдельный метод вынес. Ладно применим.

Занимаюсь этой писаниной только 2 недели, сохранение не помогает, только перезапуск(иногда F5 достаточно нажать), у компа 8Гб оперативы, а вот сам эклипс я поставил какой-то урезаный, к примеру нельзя создать файл с собственным расширением. Дома на ноуте стоит Java Neon 2. Когда тестирую изменения, то у меня обычно на компе/ноуте запущен серв, 2 клиента, барузер, сам эклипс, скайп(для работы, корпоративный) и что-нибудь еще.

 

Плагин обновлен до версии 1.3.0
Немного оптимизации и фиксов.
Добавлено скрытие всех игроков через опцию в конфиге(отлично подойдет для авторизации)
Добавлены административные фичи
/cjs hide - customjoinstream.hide - Скрыть себя. Это не ваниш, вас просто никто не видит.
/cjs show - customjoinstream.hide - Показать себя игрокам.
При наличии перма customjoinstream.hide вы автоматически становитесь невидимы при входе в игру.