Kick player on death...?

Here is the code to my listener...

Code:

package com.Kyle.TheWalls;
 
import org.bukkit.Bukkit;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.entity.PlayerDeathEvent;
import org.bukkit.event.player.PlayerJoinEvent;
 
public class MyListener implements Listener {
 
    @EventHandler
    public void OnPlayerJoin(PlayerJoinEvent event){
        Player player = event.getPlayer();
        player.teleport(player.getWorld().getSpawnLocation());
        Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), "say Player teleported to spawn on connect, sorry if he/she was in a game!");
        player.sendMessage("You have been teleported to spawn. Sorry if you were in a game.");
       
    }
    @EventHandler
    public void OnPlayerDeath(PlayerDeathEvent event){
        Player player = event.getPlayer();
        player.KickPlayer();
       
       
       
    }
}

 

Here's code of an awesome derpy class:

Code:

public class Herp extends Derp {
    public void meow() {
         Derp.foobar("NYAAAAN!");
    }
}

Seriously, though ... what is your problem? Randomly posting code does not allow us to help you. As far as we know, you don't even have a problem - at least you didn't say so. And not to mention what that problem exactly is ...

(Hint: your reply should NOT look like "it doesn't work!")

 

Use the PlayerRespawnEvent and kick them one tick after.

 

player.KickPlayer(); doesn't work...

 

Code:

player.kickPlayer("Kick Message");

 

Probably because it takes a String as a parameter.

If you don't want to kick with a message, try just:

Code:

player.kickPlayer("");

but seeing as you're kicking people on death, maybe something like:

Code:

player.kickPlayer("Kicked for dying");

 

How do I make it 1 tick after?

PS: The reason why I NEED it to be at least 1 tick is because you get into an infinite loop and the player can't get back in...

 

I would assume...

Code:

myPlugin.getServer().getScheduler().scheduleAsyncDelayedTask(myPlugin, new Runnable() {
 
  public void run() {
      // do stuff
  }
}, 1L);

Got this from Scheduler Tutorial.

 

I have this now and Eclipse gives me red lines...

Code:

@EventHandler
    public void OnPlayerRespawn(PlayerRespawnEvent event){
        Player player = event.getPlayer();
       
        Bukkit.getServer().dispatchCommand(Bukkit.getConsoleSender(), "say Player kicked from game for dying.");
       
       
       
        this.getServer().getScheduler().scheduleAsyncDelayedTask(this, new Runnable() {
              public void run() {
                  Player player = event.getPlayer();
                  player.kickPlayer("Kicked for dying. GG!");
              }
            }, 1L);
       
       
       
       
    }
}

 

..and what do the red lines say when you hover over them?

 

Of course it does, because you are calling it from a different class, I would presume.
Replace this with Bukkit, that should work just fine.

EDIT: The first this. The second this should be replaced with your plugin reference.

 

I replaced the first one with Bukkit. What do I do for the second one? The name of my plugin, or what?

 

Tell me where you have the listener. If it is in a separate class, then it should be the plugin object that you should have passed in the constructor. If it's in the main plugin class... not quite sure here, I'll check.

But really, tell us what do the red lines say when you hover over them?

 

Well right now the second one says Bukkit since I have nothing else to say in there lol. It says Bukkit cannot be resolved. I have this Listener in a separate class called MyListener. My main class is called TheWalls.

 

Do you have a constructor so referencing plugin doesn't throw a nullpointerexception?

 

What is a constructor? And no it doesn't throw one.

Anyone?

EDIT by Moderator: merged posts, please use the edit button instead of double posting.

 

Create an ArrayList<String> and when a player dies if the list doesn't contain his name, add his name to it, then on respawn if the list contains his name kick him.

 

If you dont know whats constructor... then i advice you to learn the basics of java first then come back.
(Sorry for being rude)

 

use this event:

Code:

@EventHandler
    public void onEntityDeath(EntityDeathEvent event) {
        if (event instanceof PlayerDeathEvent) {
            Player player = (Player) event.getEntity();
            player.kickPlayer("message");
        }
    }

 

You don't even have to go that far anymore.

Code:

@EventHandler
public void onPlayerDeath(PlayerDeathEvent event){
event.getPlayer().kickPlayer("Message");
}

 

k thanks

 

I tried this just to satisfy my curiosity and I needed to use this.

Code:

@EventHandler(priority=EventPriority.MONITOR)
    public void onPlayerDeath(PlayerDeathEvent event){
    event.getEntity().getPlayer().kickPlayer("Message");
    }

 

Lol... didn't have eclipse open so I didn't check it. Oh well I was close.

 

Of topic but are you actually a US marine? If so I appreciate your service.