Solved Delay between two messages

Hello everybody.

I have spent a massive time just to figure out, how to do this. And it really just bugs me!
The only thing I really want to do, is to implement delay between two messages. (or more messages)
I have tried using Thread.sleep, but that just stops the whole server!

This is what I accomplished so far!

Code:java

@EventHandler
    public void onTalk(PlayerInteractEntityEvent e) {
        Player p = e.getPlayer();
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
            p.sendMessage("Help! I don't have any more cake!");
                        try {
                            Thread.sleep(10000);
                        } catch (InterruptedException e1) {
                            e1.printStackTrace();
                        }
            p.sendMessage("Can you get me four chocolate cakes?");
            }
        }
    }
}

I appreciate any possible help! You learn from your mistakes, right?

 

Do not use thread.sleep.
Use

Code:java

Bukkit.getScheduler().runTaskLater(plugin, new Runnable() {
public void run() {
//send message
}
}, 5*20);//it would be 5 seconds, so 6*20 would be 6 seconds

 

Is this correct? Because i can't seem to get it to work.

Code:java

@EventHandler
    public void onTalk(PlayerInteractEntityEvent e) {
        Player p = e.getPlayer();
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
            Bukkit.getScheduler().runTaskLater(plugin, new Runnable() {
                public void run() {
            p.sendMessage("Help! I don't have any more cake!");
            p.sendMessage("Can you get me four chocolate cakes?");
            }
        }, 5*20);
    }
}

Bump

Okay, I tried to do this, and there is no error in the code. But when I right click a Villager it doesn't work!
Here is the code.

Code:java

public Main plugin;
 
    @EventHandler
    public void onTalk(PlayerInteractEntityEvent e) {
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
                final Player p = e.getPlayer();
                Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
                @Override
                public void run() {
            p.sendMessage("Help! I don't have any more cake!");
            p.sendMessage("Can you get me four chocolate cakes?");
            }
        }, 100L);
    }
}
}
}

 

Put the second message after the time (100L).

 

I did it, but there is something wrong, let me show you the stacktrace and code

Code:java

public Main plugin;
 
    @EventHandler
    public void onTalk(PlayerInteractEntityEvent e) {
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
                final Player p = e.getPlayer();
                Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
                @Override
                public void run() {
            p.sendMessage("Help! I don't have any more cake!");
            }
        }, 100L);
                p.sendMessage("Can you get me four chocolate cakes?");
            }
        }
    }
}

Stacktrace:

Code:

14:09:55 [SEVERE] Could not pass event PlayerInteractEntityEvent to MMORPG v1.0
14:09:55 org.bukkit.event.EventException
14:09:55 at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:427)
14:09:55 at org.bukkit.plugin.RegisteredListener.callEvent(RegisteredListener.java:62)
14:09:55 at org.bukkit.plugin.SimplePluginManager.fireEvent(SimplePluginManager.java:477)
14:09:55 at org.bukkit.plugin.SimplePluginManager.callEvent(SimplePluginManager.java:462)
14:09:55 at net.minecraft.server.v1_6_R3.PlayerConnection.a(PlayerConnection.java:1097)
14:09:55 at net.minecraft.server.v1_6_R3.Packet7UseEntity.handle(SourceFile:36)
14:09:55 at net.minecraft.server.v1_6_R3.NetworkManager.b(NetworkManager.java:296)
14:09:55 at net.minecraft.server.v1_6_R3.PlayerConnection.e(PlayerConnection.java:116)
14:09:55 at net.minecraft.server.v1_6_R3.ServerConnection.b(SourceFile:37)
14:09:55 at net.minecraft.server.v1_6_R3.DedicatedServerConnection.b(SourceFile:30)
14:09:55 at net.minecraft.server.v1_6_R3.MinecraftServer.t(MinecraftServer.java:592)
14:09:55 at net.minecraft.server.v1_6_R3.DedicatedServer.t(DedicatedServer.java:227)
14:09:55 at net.minecraft.server.v1_6_R3.MinecraftServer.s(MinecraftServer.java:488)
14:09:55 at net.minecraft.server.v1_6_R3.MinecraftServer.run(MinecraftServer.java:421)
14:09:55 at net.minecraft.server.v1_6_R3.ThreadServerApplication.run(SourceFile:583)
14:09:55 Caused by: java.lang.IllegalArgumentException: Plugin cannot be null
14:09:55 at org.apache.commons.lang.Validate.notNull(Validate.java:203)
14:09:55 at org.bukkit.craftbukkit.v1_6_R3.scheduler.CraftScheduler.validate(CraftScheduler.java:391)
14:09:55 at org.bukkit.craftbukkit.v1_6_R3.scheduler.CraftScheduler.runTaskTimer(CraftScheduler.java:120)
14:09:55 at org.bukkit.craftbukkit.v1_6_R3.scheduler.CraftScheduler.scheduleSyncRepeatingTask(CraftScheduler.java:116)
14:09:55 at org.bukkit.craftbukkit.v1_6_R3.scheduler.CraftScheduler.scheduleSyncDelayedTask(CraftScheduler.java:100)
14:09:55 at me.MunchMallow.NPC.Bob.onTalk(Bob.java:25)
14:09:55 at sun.reflect.GeneratedMethodAccessor5.invoke(Unknown Source)
14:09:55 at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
14:09:55 at java.lang.reflect.Method.invoke(Unknown Source)
14:09:55 at org.bukkit.plugin.java.JavaPluginLoader$1.execute(JavaPluginLoader.java:425)
14:09:55 ... 14 more

 

On line 4, shouldn't it be:
public void onPlayerInteract (PlayerInteractEvent e) {

 

What he names it doesn't matter.

 

Just tried, if i did that, there will be a red line under the "e.getRightClicked();"

 

Doesn't matter how he names the method but he still needs to name the event correctly

Just simply due to the Reason that "PlayerInteractEvent" doesn't has a method to get the Right Clicked.. Entity it is I think in this Event. Not sure.

So your first Event was the correct one (PlayerInteractEntityEvent)

Your stacktrace clearly says that your variable "plugin" is null which means you did not gave it any value. You would need to assign the plugin variable the Main class. via a constructor or if it is in the main class just simply with "this"

 

As L33m4n123 said, you need to do

Code:java

private final YOUR_MAIN_CLASS plugin;
public YOUR_LISTENER_CLASS(YOUR_MAIN_CLASS instance) {
plugin = instance;
}

 

Okay, I did what you guys said. But it still doesn't work, and there is no error.
Code:

Code:java

public class Bob implements Listener {
 
    public Main plugin;
    public Bob(Main instance) {
    plugin = instance;
    }
 
    @EventHandler
    public void onPlayerInteract (PlayerInteractEntityEvent e) {
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
                final Player p = e.getPlayer();
                Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
                @Override
                public void run() {
            p.sendMessage("Help! I don't have any more cake!");
            p.sendMessage("Can you get me four chocolate cakes?");
                }
        }, 100L);
            }
        }
    }
}

 

The way you have it now will just send both messages after 5 seconds. You want to send one of those messages before sending the other (one before the task, and one within it)

 

Just use a scheduler. Look into bukkit scheduling. For your situation I would recommend using 2 SyncDelayedTasks, with time separating the 2. For example one Can trigger after 0 ticks while the other can be set to after 60. This will put a 3 second delay. Just do some research on scheduling. Good luck MunchMallow

 

Okay i'm totally mindblown away now! I did all that, still doesn't work. I can't get the main class to work.
Class "Bob"

Code:java

public class Bob implements Listener {
 
    public Main plugin;
    public Bob(Main instance) {
    plugin = instance;
    }
 
    @EventHandler
    public void onPlayerInteract (PlayerInteractEntityEvent e) {
        Entity entity = e.getRightClicked();
        if (entity.getType()==EntityType.VILLAGER) {
            if (((LivingEntity)entity).getCustomName().equalsIgnoreCase("§6Bob")) {
            e.setCancelled(true);
                final Player p = e.getPlayer();
                Bukkit.getScheduler().scheduleSyncDelayedTask(plugin, new Runnable(){
                @Override
                public void run() {
            p.sendMessage("Help! I don't have any more cake!");
                }
        }, 100L);
                p.sendMessage("Can you get me four chocolate cakes?");
            }
        }
    }
}

Class "Main"

Code:java

public final class Main extends JavaPlugin {
 
 
 
    @Override
 
    public void onEnable() {
    getServer().getPluginManager().registerEvents(new Bob(), this);
 
    }
 
 
 
    @Override
 
    public void onDisable() {
 
    }
 
}

 

MunchMallow Does this compile?

Show Spoiler Hide Spoiler

What you have shown here should not compile. If it does, then something is else is wrong.

 

Woohoo! I got it fixed! Thank you all! Here is a diamond! MORE DIAMONDS MANY MORE! AND A CAKE