Плагин [MISC]PvPStats v0.6 - Статистика игрока на PvP.[Tested: 1.8]

  • Автор темы Автор темы Slavkaa
  • Дата начала Дата начала
Slavkaa сказал(а):
Тупой я, я тупо увидел чей-то код и вставил его. Заработало, вот я и забил. Щас сделал через set, все нормально работает.

Так удовлетворительно?
Java: 
    @EventHandler
    public void join(PlayerJoinEvent e) {
    Player p = e.getPlayer();
        String name = p.getName().toLowerCase();
        FileConfiguration c = plugin.getConfig();
        if(!c.contains("players." + name)) {
        c.set("players." + name + ".kills", 0);
        c.set("players." + name + ".deaths", 0);
        c.set("players." + name + ".rating", 0);
        }
        plugin.saveConfig();
    }
Нажмите, чтобы раскрыть...
Da
 
Slavkaa сказал(а):
Тупой я, я тупо увидел чей-то код и вставил его. Заработало, вот я и забил. Щас сделал через set, все нормально работает.

Так удовлетворительно?
Java: 
    @EventHandler
    public void join(PlayerJoinEvent e) {
    Player p = e.getPlayer();
        String name = p.getName().toLowerCase();
        FileConfiguration c = plugin.getConfig();
        if(!c.contains("players." + name)) {
        c.set("players." + name + ".kills", 0);
        c.set("players." + name + ".deaths", 0);
        c.set("players." + name + ".rating", 0);
        }
        plugin.saveConfig();
    }
Нажмите, чтобы раскрыть...
Нет. Ты должен выставлять значения тогда, когда это понадобится
 
Т.е. проверять наличие каждой строки и только тогда выставлять строчки?
 
CrashInYou сказал(а):
А ты на их вопросы, ответы готовил до создания плагина?

Слушай советы DonDays, по-моему, он тут единственный всезнающий разработчик.
Нажмите, чтобы раскрыть...
Нет, не готовил. Советы слушаю все. Полностью согласен, что DonDays отличный разработчик, но для меня, он говорит загадками и я редко его понимаю из-за скудного запаса знаний.
 
Slavkaa сказал(а):
Нет, не готовил. Советы слушаю все. Полностью согласен, что DonDays отличный разработчик, но для меня, он говорит загадками и я редко его понимаю из-за скудного запаса знаний.
Нажмите, чтобы раскрыть...
не будь тупицей, ставь значения только тогда, когда пацаны умирают/убивают, а не при заходе
 
Zнамя Юности сказал(а):
не будь тупицей, ставь значения только тогда, когда пацаны умирают/убивают, а не при заходе
Нажмите, чтобы раскрыть...
У меня изначально должны стоять нули, иначе как будут просматривать статистику игрока, который только что зашел?
Обновление 0.3:
- Небольшие поправки в коде.
- Команда /stats, для изменения статистики игрока прямо в игре.

Если еще не выпали глаза, декомпилируйте класс Stats.
 
Zнамя Юности сказал(а):
выводить нули, если ее тупо нет, не думал об этом?
Нажмите, чтобы раскрыть...
Смысл писать дополнительные условия, если можно сделать так, как и сейчас?
CrashInYou сказал(а):
Нужна твоя помощь.

Ххахахах
Нажмите, чтобы раскрыть...
Не вижу ничего смешного. Можно подумать, что ты все знал еще с пелёнок.
 
Slavkaa сказал(а):
Не пиши загадками. Обосри, критикуй, но пиши прямо.
Нажмите, чтобы раскрыть...
То, что тебе уже пальцем тыкают в недочеты, и предлагают идеи по улучшению структуры, например, про те же нули, кода и реализации, а ты отмахиваешься, приговаривая
Slavkaa сказал(а):
Зачем писать дополнительный код, когда и так нормально?
Нажмите, чтобы раскрыть...
Если не собираешься улучшать плагин, и собираешься выкладывать его говном, и оставлять таким же, то лучше не выкладывай.
 
DonDays сказал(а):
То, что тебе уже пальцем тыкают в недочеты, и предлагают идеи по улучшению структуры, например, про те же нули, кода и реализации, а ты отмахиваешься, приговаривая

Если не собираешься улучшать плагин, и собираешься выкладывать его говном, и оставлять таким же, то лучше не выкладывай.
Нажмите, чтобы раскрыть...
Коротко и по теме. Сейчас переделаю.
Я смотрю, глаза у вас еще не до конца выпали. Ловите обновление 0.4.
Нововведения:
- Команда /setstats - установить статистику игроку.
- Команда /setstats reset - сбросить статистику игрока.
- Сообщения о смерти/убийстве игрока.
 
Slavkaa сказал(а):
Теперь, при входе игрока на сервер, не выставляются нули по умолчанию.
Нажмите, чтобы раскрыть...
а это зачем тогда?
Код: 
    @EventHandler
    public void join(PlayerJoinEvent e) {
        FileConfiguration c = plugin.getConfig();
        if(!c.contains("players." +  e.getPlayer().getName().toLowerCase() )) {
        c.createSection("players." + e.getPlayer().getName().toLowerCase());
        }
        plugin.saveConfig();
    }
Не проще при не нахождении статы в конфиге выдвать нули, не записывая их в кфг?


В одну строку слабо сделать?
 
Для ответа нужно войти/зарегистрироваться
Назад
Верх