Помогите Ошибка в BukkitRunnable

[MrSharky]

Объясните, почему вылетает ошибка:

    public void startRegeneration(Player p){
        final Player pl = p;
        BukkitRunnable runnable = new BukkitRunnable(){
            @Override
            public void run(){
                while(pl.isOnline()){
                    @SuppressWarnings("deprecation")
                    Score score = pl.getScoreboard().getObjective("Stats").getScore(Bukkit.getOfflinePlayer("Mana:"));
                    int mana = score.getScore();
                    int maxMana = EnableClass.config.getInt(pl.getName() + "_MAX_MANA");
                    if(mana < maxMana){
                        mana += EnableClass.config.getInt(pl.getName() + "_MANA_REGEN");
                        score.setScore(mana);
                    }
                }
                cancel();
            }
        };
        runnable.runTaskTimer(plugin, 1L, 20L);
    }

Ошибка указывает только на одну строчку а именно:

        runnable.runTaskTimer(plugin, 1L, 20L);

 

[serega6531]

Эмм, зачем вы делаете cancel в конце выполнения, если запустили таймер? Кидайте ошибку.

 

[Den_Abr]

Объясните, почему вылетает ошибка:

    public void startRegeneration(Player p){
        final Player pl = p;
        BukkitRunnable runnable = new BukkitRunnable(){
            @Override
            public void run(){
                while(pl.isOnline()){
                    @SuppressWarnings("deprecation")
                    Score score = pl.getScoreboard().getObjective("Stats").getScore(Bukkit.getOfflinePlayer("Mana:"));
                    int mana = score.getScore();
                    int maxMana = EnableClass.config.getInt(pl.getName() + "_MAX_MANA");
                    if(mana < maxMana){
                        mana += EnableClass.config.getInt(pl.getName() + "_MANA_REGEN");
                        score.setScore(mana);
                    }
                }
                cancel();
            }
        };
        runnable.runTaskTimer(plugin, 1L, 20L);
    }

Ошибка указывает только на одну строчку а именно:

        runnable.runTaskTimer(plugin, 1L, 20L);

Нельзя завершать таск внутри его самого.

 

[MrSharky]

Эмм, зачем вы делаете cancel в конце выполнения, если запустили таймер? Кидайте ошибку.

И убрав cancel выдает тоже самое

 

[serega6531]

Зачем мне кусок стактрейса, на котором не видно ошибку?

 

[MrSharky]

Тогда вот:

 

[serega6531]

Тогда вот:

Ну вот вам и ответ. Plugin cannot be null.

 

[MrSharky]

Тогда не ясно как получить Plugin. Я доставал его через главный класс командой getPlugin().
Точнее Bukkit.getPluginManager().getPlugin("название"), просто getPlugin() требует аргумент получить который я тоже не знаю как.

 

[ql_Nik_lp]

Нельзя завершать таск внутри его самого.

Можно, но не так)

 

[Reality_SC]

Указатель на твой плагин в теле главного класса плагина — это this.

 

[Den_Abr]

Можно, но не так)

И как же? Через id?

 

[CraftCoder]

И как же? Через id?

Наверное

 

[Reality_SC]

Зачем его получать, вы чо =\ this

 

[Den_Abr]

Зачем его получать, вы чо =\ this

Мы про завершение таска

 

[MrSharky]

Зачем его получать, вы чо =\ this

мой Runnable в другом классе, а через static нельзя с this. Создать объект главного класса тож нельзя - выдает ошибку.

 

[Reality_SC]

Ну, например — не используй статик.
Предположим, у тебя другой класс такой:

public class OtherClass
{
   private final JavaPlugin plugin;
   public OtherClass(JavaPlugin plugin)
   {
      this.plugin = plugin;
   }
   ...
   // Где-то тут ты задаёшь свой Runnable
}

В теле главного класса:

   private final OtherClass other = new OtherClass(this);
   ...
      other.doThatShitWithYourRunnable();

 

[MrSharky]

Как то о конструкторе, я и забыл... Спасибо, пойду пробывать.

   private final OtherClass other = new OtherClass(this);
      other.doThatShitWithYourRunnable();

Хорошее название метода

 

[ql_Nik_lp]

Я обычно для этой цели использую Singlet-on
Выглядит это так: (Это меин класс плагина)

    private static Plugin plugin;
    @Override
    public void onLoad() {
        plugin = this;
    }

    public static Plugin getInstance() {
        return plugin;
    }

В последствии откуда угодно можно получить плагин посредством Main.getInstance() и не извращаться с созданием лишних объектов для послед. передачи и / или еще хуже разными getPlugin("MyPlug");

 

[Reality_SC]

Не люблю синглтоны, хотя иногда без них никуда. В Bukkit API они не очень, как мне кажется, соответствуют модели загрузки плагинов — один и тот же плагин может быть загружен несколько раз (в т.ч. разных версий) и может возникнуть неопределённое поведение в недовыгруженном участке.

 

[ql_Nik_lp]

Не люблю синглтоны

Зря-зря-зря-зря-зря..

В Bukkit API они не очень, как мне кажется, соответствуют модели загрузки плагинов

Тут соглашусь, правильным синглтоном это не назовешь, но это работает и работает отлично.

один и тот же плагин может быть загружен несколько раз

Не правда. Только если специально пошаманить.
А так plugin.yml не даст (Одинаковое имя)

(в т.ч. разных версий)

Какие извращения о.о

может возникнуть неопределённое поведение в недовыгруженном участке.

Поле - private
При особых боязнях в onDisable можно запихнуть plugin = null;