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

[efwio]

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

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

Спойлер: Скриншот

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

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

2.0

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

1.2

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

1.1

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

1.0

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

 

[Plasticable]

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

 

[Jampski]

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

 

[efwio]

Добавил

 

[Kirich]

не плохо)

 

[FYNTIK1996]

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

 

[efwio]

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

 

[FYNTIK1996]

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

 

[efwio]

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

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

 

[Jampski]

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

 

[efwio]

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

 

[Korvinius]

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

 

[bigclifty]

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

 

[BeLkA]

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

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

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

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

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

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

saveDefaultConfig();
reloadConfig();

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

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

if(cs != null) {

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

}

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

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

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

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

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

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

 

[bigclifty]

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

 

[BeLkA]

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

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

 

[bigclifty]

up

 

[Den_100Channel]

какая версия