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

Стоит продолжать улучшать?

  • Всего проголосовало
    15
  • Опрос закрыт .
 
DonDays сказал(а):
штоооо


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, если есть то выставляет из конфига. На счет команд, переделываю, сам хочу по человечески сделать и там аргументы буду добавлять еще для новых функций..
 
glavrak сказал(а):
Чем тебе не нравится слушатель эвента? Если нет перма, то меняет дефолтное сообщение на null, если есть то выставляет из конфига. На счет команд, переделываю, сам хочу по человечески сделать и там аргументы буду добавлять еще для новых функций..
Нажмите, чтобы раскрыть...
if(!expression) {
//empty block
} else {
//code
}

доставило
Код: 
                    sender.sendMessage(plugin.loc.getString("HelpLine1"));
                    sender.sendMessage(plugin.loc.getString("HelpLine2"));
 
Нифига не понял, что тебе в слушателе не понравилось, ведь он работает именно так как надо ему работать, на счет строк справки я еще больше тебя не понял. В 2 строки сделал для большего удобства.
 
glavrak сказал(а):
Нифига не понял, что тебе в слушателе не понравилось, ведь он работает именно так как надо ему работать, на счет строк справки я еще больше тебя не понял. В 2 строки сделал для большего удобства.
Нажмите, чтобы раскрыть...
Он намекал на то, что код можно сократить до
Java: 
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"
 
glavrak сказал(а):
Чем тебе не нравится слушатель эвента? Если нет перма, то меняет дефолтное сообщение на null, если есть то выставляет из конфига. На счет команд, переделываю, сам хочу по человечески сделать и там аргументы буду добавлять еще для новых функций..
Нажмите, чтобы раскрыть...
Вместо такого убого реплейса юзай pattern
 
Плагин обновлен до версии 1.2.2
Взяты некоторые наработки @DonDays, скажем спасибо ему за это.
Плагин теперь требует наличия Vault!
Плагин частично перекодирован.
Исправлены сообщения об ошибках в консоли.
Переписаны файлы локализаций.
/cjs setjoinloc - customjoinstream.setloc - Установка стартовой позиции
/cjs tpjoin - customjoinstream.tpjoin - Телепорт на стартовую позицию
/cjs setspawn - customjoinstream.setloc - Установка спавна
/cjs spawn - customjoinstream.spawn - Телепорт на спавн
Добавлена телепортация на стартовую позицию. Включается в конфиге.
 
DonDays сказал(а):
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 и дополнил функциями.
 
glavrak сказал(а):
EventUtils.java перенес в пакет Utils и дополнил функциями.
Нажмите, чтобы раскрыть...

замени этот повторяющийся код, посмотри в
DonDays сказал(а):
https://github.com/DonDays/CustomJo...src/mr_krab/customjoinstream/utils/Utils.java
Нажмите, чтобы раскрыть...

зачем тут ретурны в конце?
Код: 
    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 в обработчике события? ты ничего не должен возвращать
 
DonDays сказал(а):
замени этот повторяющийся код
Нажмите, чтобы раскрыть...
На что ты мне предлагаешь его заменить если там требуется 3 значения Double для воспроизведения звуков и эффектов в радиусе? Скажи, заменю, не проблема.
DonDays сказал(а):
зачем тут ретурны в конце?
Нажмите, чтобы раскрыть...
Хз, смотрел на подсказки эклипса при написании, потребовал return, я вставил. Сейчас убрал, вроде не требует вернуть, иногда чушь кидает, бывает не видит объявленные переменные, приходится перезапускать.
 
glavrak сказал(а):
На что ты мне предлагаешь его заменить если там требуется 3 значения Double для воспроизведения звуков и эффектов в радиусе? Скажи, заменю, не проблема.

Хз, смотрел на подсказки эклипса при написании, потребовал return, я вставил. Сейчас убрал, вроде не требует вернуть, иногда чушь кидает, бывает не видит объявленные переменные, приходится перезапускать.
Нажмите, чтобы раскрыть...
СКорее всего у вас руки из одного места.
Либо комп-мамонт с 1 Гб оперативки.
Если (вдруг) он что-то не видит, достаточно сохранить изменения, спецом для этого там целых две кнопки с дискетами.
 
DonDays сказал(а):
читай внимательно https://github.com/DonDays/CustomJo...src/mr_krab/customjoinstream/utils/Utils.java
Нажмите, чтобы раскрыть...
А вот оно что, в отдельный метод вынес. Ладно применим.
Azim_D3Tm сказал(а):
СКорее всего у вас руки из одного места.
Либо комп-мамонт с 1 Гб оперативки.
Если (вдруг) он что-то не видит, достаточно сохранить изменения, спецом для этого там целых две кнопки с дискетами.
Нажмите, чтобы раскрыть...
Занимаюсь этой писаниной только 2 недели, сохранение не помогает, только перезапуск(иногда F5 достаточно нажать), у компа 8Гб оперативы, а вот сам эклипс я поставил какой-то урезаный, к примеру нельзя создать файл с собственным расширением. Дома на ноуте стоит Java Neon 2. Когда тестирую изменения, то у меня обычно на компе/ноуте запущен серв, 2 клиента, барузер, сам эклипс, скайп(для работы, корпоративный) и что-нибудь еще.
 
Плагин обновлен до версии 1.3.0
Немного оптимизации и фиксов.
Добавлено скрытие всех игроков через опцию в конфиге(отлично подойдет для авторизации)
Добавлены административные фичи
/cjs hide - customjoinstream.hide - Скрыть себя. Это не ваниш, вас просто никто не видит.
/cjs show - customjoinstream.hide - Показать себя игрокам.
При наличии перма customjoinstream.hide вы автоматически становитесь невидимы при входе в игру.
 
Для ответа нужно войти/зарегистрироваться
Назад
Верх Низ