Войти или зарегистрироваться
  • Этот сайт использует файлы cookie. Продолжая пользоваться данным сайтом, Вы соглашаетесь на использование нами Ваших файлов cookie. Узнать больше.
  • Вы находитесь в сообществе Rubukkit. Мы - администраторы серверов Minecraft, разрабатываем собственные плагины и переводим на различные языки плагины наших коллег из других стран.

Плагин Nospace

Статус
Закрыто для дальнейших ответов.
DimaRUS сказал(а):
Глобаьно глобально, я могу вступить, как только найду мною потерянный учебник по яве:D
Нажмите, чтобы раскрыть...
Эх..плохо что с админом нескоро свяжусь :(
А так у Unofficial Rubukkit Team практически два проекта)
 
Добрый день. Просмотрел код плагина и не сильно понял, зачем вы сделали так:
Код: 
    Player p = event.getPlayer(); //получаем игрока
    String name = p.getDisplayName();//получаем его имя
    String name_lower = name.toLowerCase();
name.toLowerCase(); - уменьшаем регистр букв. Это лишнее. Вы всего навсего проверяете наличие пробела в строке. Далее проверку
Код: 
    if (name_lower.contains(" ") | name_lower.isEmpty() | name_lower.startsWith(" ") | name_lower.endsWith(" "))
можно сократить так
Код: 
if (name.contains(" ")) {...}
. Кстати еще одно, во многих плагинах (в том числе и забугорного производства) видел вот такой код
Код: 
    public NoSpacePlayerListener (NoSpace instance)
    {
        plugin = instance;
    }
. И объект plugin нигде не используется. Всё видимо пошло от одного плагина, после чего все начали копипастить этот код.

P.S. Данный пост просто совет.
 
Bizon сказал(а):
Добрый день. Просмотрел код плагина и не сильно понял, зачем вы сделали так:
Код: 
    Player p = event.getPlayer(); //получаем игрока
    String name = p.getDisplayName();//получаем его имя
    String name_lower = name.toLowerCase();
name.toLowerCase(); - уменьшаем регистр букв. Это лишнее. Вы всего навсего проверяете наличие пробела в строке. Далее проверку
Код: 
    if (name_lower.contains(" ") | name_lower.isEmpty() | name_lower.startsWith(" ") | name_lower.endsWith(" "))
можно сократить так
Код: 
if (name.contains(" ")) {...}
. Кстати еще одно, во многих плагинах (в том числе и забугорного производства) видел вот такой код
Код: 
    public NoSpacePlayerListener (NoSpace instance)
    {
        plugin = instance;
    }
. И объект plugin нигде не используется. Всё видимо пошло от одного плагина, после чего все начали копипастить этот код.

P.S. Данный пост просто совет.
Нажмите, чтобы раскрыть...

Прикол, в том что этот плагин сначала кикал игрока с определённым ником , а потом после модернизации не подумал удалить переменую.:) Это первое.

Насчет if (name_lower.contains(" ") | name_lower.isEmpty() | name_lower.startsWith(" ") | name_lower.endsWith(" ")) я просто тестил эти типы условий.
 
Будет ли обновление плагина ?
 
D_ART сказал(а):
Будет ли обновление плагина ?
Нажмите, чтобы раскрыть...

А что там обновлять? Я планирую добавить лог файл в который будут записываться игроки, которые пробовали присоединиться.
 
Так же, если плагин D_ART, LogManager фиксирует все события сервера, то в лог файле вы можете найти сообщения об игроках, которые были кикнуты из-за пробела.:)
С никами конечно же.
 
Gradi сказал(а):
Так же, если плагин D_ART, LogManager фиксирует все события сервера, то в лог файле вы можете найти сообщения об игроках, которые были кикнуты из-за пробела.:)
С никами конечно же.
Нажмите, чтобы раскрыть...
Мой плагин нечего не фиксирует. Он только работает с самим лог файлом bukkit
 
D_ART сказал(а):
Мой плагин нечего не фиксирует. Он только работает с самим лог файлом bukkit
Нажмите, чтобы раскрыть...
Хотя можно добавить. Всё таки это лог менеджер!
З.Ы афтар вступай в Unofficial Rubukkit Team . Я уже там)
 
Пока в скайп.(слево под аватаркой)
Unofficial Rubukkit Team это совместное создание плагинов их обновление ;)
 
Статус
Закрыто для дальнейших ответов.

Поделиться этой страницей

Назад
Верх