Player Death Event

Hey!

I need your help with this.

Code:

Set<String> team = new HashSet<String>();
Set<String> spectate = new HashSet<String>();

If a player 'team' dies I want them removed from team and added to 'spectate' and if there is 1 player in 'team' I want it to broadcast that that player is the winner.

Thanks for your time,
Hari

 

Here you go

Code:

@EventHandler
public void onDeath(PlayerDeathEvent e){
Player p = e.getPlayer();
if(team.contains(p.getName()){
team.remove(p.getName();
spectate.add(p.getName());
}
}

 

You're right that you would need to listen to PlayerDeathEvent to do this. Here's a useful tutorial on how to set up a listener: http://wiki.bukkit.org/Event_API_Reference

 

adam753 I know how to do the listener not the rest

 

what do you use to add them in "team" and "spectate"?

 

MyNameIsHariK
What code do you have so far then?

 

adam753

public void onDeath(PlayerDeathEvent e) {
//code here
}

 

If you use the code I gave you... It will work just fine

@MyNameIsHariK Here is a few examples ill give you about array lists. Hope it helps.

Code:java

 public void array(Player p){
//Adds the player to the array lists
green.add(p.getName());
 
//removes the player form the array list
green.remove(p.getName());
 
 
//Checks if the player is in the array list
if(green.contains(p.getName())){
//code here
}
//clears the array list
green.clear();
 
//gets the amount of players in the array list
green.size();
 
 
//Checks if the array list is empty
if(green.isEmpty()){
//code here
}
 
 
//this is almost everything you will ever use array lists for ... btw just to make things clear -
// (green) is the name of my array list ... you will replace (green) with the name of your -
//Array list. I hope this helps!
 
}

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

 

Help please?

 

Well what about this then?

Code:

Player p = e.getPlayer();
 
if(team.contains(p.getName()) {
    team.remove(p.getName());
    spectate.add(p.getName());
}

 

How would I do the > if there is 1 player in 'team' I want it to broadcast that that player is the winner.

 

Code:java

if (team.size() == 1) {
// You have to iterate through the HashSet
}

Also one more thing is I wouldn't use HashSets or HashMaps for this kind of things. You could use a hashmap if you have more than one team, but for this I would use an ArrayList

 

if(team.size == 1 && team.contains(e.getPlayer().getName()){
e.getPlayer() is the winner

 

Tss1410 kevinspl2000 adam753

Tells me to add an extra close bracket:
http://gyazo.com/2a3993e9a9613614ed13414d9cf131d5
http://gyazo.com/ee73f216f6aa2581c12718b2f044fb7c

Getting errors with

http://gyazo.com/7926f21f4bb77ec1f231f7834251eab5
http://gyazo.com/07245f9c56a1fd3d7831f716439771c0

:/

 

MyNameIsHariK
On the last error, add another bracket, see if that solves the problem. As for the other errors, is the variable in the public void (//event e) set to the e or something else.

 

kevinspl2000 It says "Add cast to e"

 

On the first two errors, add extra brackets where it tells you to. I'd have thought it was obvious, but if you count the brackets with your fingers you'll find one missing each time.

The last two errors are my mistake, sorry - the method is called getEntity(), not getPlayer().

 

adam753 kevinspl2000 Just tested it and it didnt work

Code:

    @EventHandler
    public void onDeath(PlayerDeathEvent e){
        Player p = e.getEntity();
     
        if(joinedplayers.contains(p.getName())) {
            joinedplayers.remove(p.getName());
            spectate.add(p.getName());
        }
     
        if(cStarted == true) {
            if(joinedplayers.size() == 1 && joinedplayers.contains(e.getEntity().getName())){
                Bukkit.broadcastMessage(ChatColor.RED + "[" + ChatColor.GRAY + "Tnt" + ChatColor.RED + "] " + ChatColor.YELLOW + e.getEntity() + ChatColor.YELLOW + " won the game!");
            }
        }
     
        e.getDrops().clear();
        p.getInventory().setBoots(null);
        p.getInventory().setChestplate(null);
        p.getInventory().setHelmet(null);
        p.getInventory().setLeggings(null);
    }

 

what is wrong?
Error?

 

MyNameIsHariK
If there's only one player on the team, and when they die you remove them from the team, then what's the size of the team going to be after that? Hint: it's not 1.

 

adam753 What, I am making a ffa game and there are 24 players in 'team'

Tss1410 It dont work. We started the game, and my friend died and it didnt say i won

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

 

Use some common sense. You're removing the player from the set, then checking if the set contains the player. That will always be false.

What I meant with my last post is that after you've removed the last player from the set, its size should be 0, not 1. You're doing things in the wrong order and/or checking things in the wrong way.