Помогите Не работают ивенты .Помогите

[TorwaldOswarld]

Добрый день , дорогие форумчане . Я столкнулся с такой проблемой , я написал плагин (мне не много помоги с кодом) . Дело в том , что не работают ивенты . То есть , при съедании золотого яблока игрок должен получать сообщение , и далее он вноситься в список на время (3 минуты) , теперь игрок сможет съесть яблоко , только через 3 минуты . Но ничего не пишет и игрок может есть яблоко . Помогите пожалуйста найти ошибку . Вот весь код http://pastebin.com/dxJYy6S7 . Лог инфо работает , в консоль пишет цифру 1 . Заранее спасибо !

 

[Plasticable]

Какое ядро?

 

[TorwaldOswarld]

Какое ядро?

spigot 1.7.9

 

[LaoTheLizard]

Зачем ты проверяешь, что у него в руке?

if(event.getItem().getType() != Material.GOLDEN_APPLE){return;}

Может, так?

 

[TorwaldOswarld]

Зачем ты проверяешь, что у него в руке?

if(event.getItem().getType() != Material.GOLDEN_APPLE){return;}

Может, так?

неа , не помогло

 

[LaoTheLizard]

неа , не помогло

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

 List<Player> players

Это раз.
А главное (это и есть твоя ошибка) - ты не добавил строку, в которой ты добавляешь игроков в этот список.
Что-то вроде:

players.add(player);

 

[TorwaldOswarld]

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

 List<Player> players

Это раз.
А главное (это и есть твоя ошибка) - ты не добавил строку, в которой ты добавляешь игроков в этот список.
Что-то вроде:

players.add(player);

Спасибо , сейчас по пробую

 

[LaoTheLizard]

И фраза у тебя некорректная. Она всегда говорит, что через три минуты можно будет. А на самом деле это не так. И фраза будет логичнее, если писать так: "Золотое яблоко можно использовать не чаще раза в три минуты." Впрочем, это тоже не идеальное решение.

 

[TorwaldOswarld]

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

 List<Player> players

Это раз.
А главное (это и есть твоя ошибка) - ты не добавил строку, в которой ты добавляешь игроков в этот список.
Что-то вроде:

players.add(player);

а это не то ? if (players.contains(player.getName())) , и я не понял куда этот ( players.add(player); ) код нужно вставить ?

 

[LomNar]

а это не то ? if (players.contains(player.getName())) , и я не понял куда этот ( players.add(player); ) код нужно вставить ?

Ты свой код видел? Ты проверяешь есть ли игрок в списке, но список пуст т.к. ты ни где не добавляешь в него игрока! Вот и решение. Думаю сдесь больше нечего обсуждать...

 

[serega6531]

А по вопросу, сделай лист не имен игроков, а игроков.

 List<Player> players

БОЖЕ, ЗАЧЕМ????

 

[TorwaldOswarld]

БОЖЕ, ЗАЧЕМ????

Подскажи как надо сделать , чтобы всё работало . Пожалуйста

Ты свой код видел? Ты проверяешь если игрок в списке, но список пуст т.к. ты ни где не добавляешь в него игрока! Вот и решение. Думаю сдесь больше нечего обсуждать...

я только начинаю в этом разбираться , и поэтому по просил помощи у более опытных людей

 

[serega6531]

Подскажи как надо сделать , чтобы всё работало . Пожалуйста

я только начинаю в этом разбираться , и поэтому по просил помощи у более опытных людей

Никогда нигде не храните объект игрока.

 

[LomNar]

Никогда нигде не храните объект игрока.

Лучше UUID ну или ник

 

[TorwaldOswarld]

Никогда нигде не храните объект игрока.

я не знаю как сделать таймер по другому

Никогда нигде не храните объект игрока.

Подскажите пожалуйста как правильно сделать .

 

[LomNar]

я не знаю как сделать таймер по другому

Подскажите пожалуйста как правильно сделать .

Пиши в скайп: radius981. Через 2 часа отвечу

 

[ensirius]

Регистрация события правильное, сообщение что плагин запущен вылазит? Если нет, то возможно проблема в plugin.yml

 

[TorwaldOswarld]

Код нормальный, сообщение что плагин запущен вылазит? Если нет, то возможно проблема в plugin.yml

да , сообщение появляется

 

[LaoTheLizard]

В любом случае, так должно работать:

Спойлер

package NoGoldenApple;

import java.util.ArrayList;
import java.util.List;
import java.util.logging.Logger;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerItemConsumeEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class Main extends JavaPlugin implements Listener {

Logger log = getLogger();
public void onEnable()
{
getLogger().info("plugin has been Enabled");
getServer().getPluginManager().registerEvents(this, this);
}

List<Player> players = new ArrayList<Player>();

@EventHandler
public void onItemConsume(PlayerItemConsumeEvent event){
   
    final Player player = event.getPlayer();
    if (event.getItem().getType() != Material.GOLDEN_APPLE) return;
    if (players.contains(player)){
    player.sendMessage(ChatColor.GOLD + "Золотое яблоко можно использовать не чаще раза в три минуты.");
    event.setCancelled(true);
    return;
    }
    players.add(player);
    getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable()
    {
        public void run()
        {
        players.remove(player);
        }
    }, 180L);
}
}

 

[serega6531]

В любом случае, так должно работать:

Спойлер

package NoGoldenApple;

import java.util.ArrayList;
import java.util.List;
import java.util.logging.Logger;

import org.bukkit.ChatColor;
import org.bukkit.Material;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.PlayerItemConsumeEvent;
import org.bukkit.plugin.java.JavaPlugin;

public class Main extends JavaPlugin implements Listener {

Logger log = getLogger();
public void onEnable()
{
getLogger().info("plugin has been Enabled");
getServer().getPluginManager().registerEvents(this, this);
}

List<Player> players = new ArrayList<Player>();

@EventHandler
public void onItemConsume(PlayerItemConsumeEvent event){
  
    final Player player = event.getPlayer();
    if (event.getItem().getType() != Material.GOLDEN_APPLE) return;
    if (players.contains(player)){
    player.sendMessage(ChatColor.GOLD + "Золотое яблоко можно использовать не чаще раза в три минуты.");
    event.setCancelled(true);
    return;
    }
    players.add(player);
    getServer().getScheduler().scheduleSyncDelayedTask(this, new Runnable()
    {
        public void run()
        {
        players.remove(player);
        }
    }, 180L);
}
}

Пожалуйста, не надо так делать.