Плагин [MISC] FireLoto v 2.1 - Лотерея!?[UPDATE]

FireLoto - Лотерея!?:
Версия: v2.1
Этот плагин добавляет ежеминутную лотерею, в которой участвуют все игроки на сервере.
P.S. При старте сервера лотерея включается автоматически!

Команды:
/loto-help - справка
/loto-give - выдать приз
/loto-start - запустить лотерею
/loto-stop - отключить лотерею
/loto-reload - перезагрузить конфиг

Permissions:
/loto-give - fireloto.give
/loto-start - fireloto.start
/loto-stop - fireloto.stop
/loto-reload - fireloto.reload

Скачать плагин

Spoiler: Скриншот

Лог изменений:
2.1

• Добавлена команда /loto-reload
• Добавлена возможность менять количество игроков для старта лотереи

2.0

• Переписана система выдачи приза
• Добавлен конфиг
• Добавлена возможность редактировать призы

1.2

• Добавлена команда /loto-start
• Добавлена команда /loto-stop

1.1

• Мелкие багфиксы

1.0

• Релиз моего крутого плагина =)

 

А не проще сделать одну коману - /loto и слушать аргументы?

 

Хоть бы 1 скрин закинул.

 

Добавил

 

не плохо)

 

Сори за мат плагин ******* =)[DOUBLEPOST=1427559506,1427559418][/DOUBLEPOST]Есть один вопрос а игрокам какой pex надо или он не нужна для выдачу приза????

 

Нет, все игроки участвуют в лотерее. Без всяких Пермов.
Перм fireloto.give нужен администратору, чтобы он запустил лотерею досрочно.

 

оки сегодня хочу поставить плагин на сервер посмотрю игрокам понравиться или нет мне очень такое понравилось =)

 

Обновление!
2.1

• Добавлена команда /loto-reload
• Добавлена возможность менять количество игроков для старта лотереи

 

Cделай так что бы лотерейные билеты нужно было покупать.

 

Возможно в будующем реализую.

 

Это можно реализовать с помощью ReActions и в более разнообразных вариациях.

 

Добавь поддержку префиксов в сообщениях о лотерее

 

@efwio
И так, разберем все по полкам:

1. Создание конфига выглядит как говно не очень хорошо.

Code:

    this.config = getConfig();
    getConfig().options().copyDefaults(true);
    saveDefaultConfig();

Из этого всего нужен только последний метод, все остальное стирай.

2. В onDisable() ты тоже сохраняешь конфиг. Зачем?

3. Переменная cfile тут абсолютно не нужна. Просто используй метод getConfig(). Если тебе нужна эта переменная для перезагрузки конфига, то перезагрузку лучше в твоем случае сделать так:

Code:

saveDefaultConfig();
reloadConfig();

4. int prizeNum = prizeRnd.nextInt(5);
Всего 5 предметов? Маловато :c
Можно сделать так:

Code:

ConfigurationSection cs = getConfig().getConfigurationSection("items");

if(cs != null) {

int prizeNum = new Random().nextInt(cs.getValues(false).keySet().size());

}

5. Говнокод, сорри :c

Code:

    else
    {
      sender.sendMessage(ChatColor.RED + "Only players can use this!");
      return true;
    }

Вот мой совет:

Code:

onCommand() {
if(!(sender instanceof Player)) {
sender.sendMessage("Console is not allowed!");
return false;
}

Player player = (Player) sender;
...
}

Удачи тебе в дальнейшей разработке плагинов! Надеюсь, учтешь мои замечания.

 

Добавь поддержку префиксов и увеличь количество призов. 5 максимум

 

Два рза одно и тоже писать нехорошо :c

 

up

 

какая версия