Solved How to code anti teamkill?

Whats Name of the event for Anti Team Kill (PVP)?

So if Player 1 team blue and attacks Player 2 (team blue) the 2. Player shouldn't get Damage from Player 1, so that they can't take damage, if they in the same teams?

 

Team team = <Scoreboard>.registerNewTeam(name);
team.setAllowFriendlyFire(false);

 

Thank you, will try it tommorrow!

What is "<Scoreboard>" mean? (6 months ago, that i have the plugin updated)

 

You could do what calebbfmv said or depending on how you are doing teams you could just do a simple

Code:java

public void onDamage(EntityDamageByEntityEvent e)
    {
 
        Entity damager = e.getDamager();
        Entity damaged = e.getEntity();
        if(!(damager instanceof Player)) return;
 
        if(!(damaged instanceof Player)) return;
}
 
 

And then just check what team the damager and damaged are on and do e.setCancelled(true) if they are.

 

The name of your scoreboard.

 

Also, couldn't you get the player's team and check if the damager is the other team, and of it isn't from the enemy cancel the event? Also this only applies to certain ways ie: array lists.

 

*Removed*

//EDIT: Ok should work now, ty guys.

 

jordanzilla02 xDjBomber
Why so complicated guys? Scoreboards are amazing! Let them do their jobs!

 

What if you don't want to work with scoreboards? I like making two or more arraylist for the teams then check in the damage listener if the damager is from the same team, the damage will be cancelled because you can make a player called damager and use the teamname.contains(damager) or something similar to that.

 

jordanzilla02
But,think in a larger scale:
Your getting the list, reading it all, then comparing the strings to another. if you have 1000+ people for it, thing of the CPU having to deal with that, every time someone hits another. Unnecessary waste of time.

 

You could do what jordanzilla02 said.

Make an arraylist for each team. When the player is assigned to a team, put them in the proper arraylist, and make an event that prevents EntityDamageByEntity for that specific arraylist.

 

Booshayy
That is NOT a good idea.....

 

calebbfmv
Why not? Elaborate.

 

Booshayy
Memory:
Every single time a Player is hit by another player, you are getting 2 lists, getting contents, then comparing. Think of it on a larger scale:
1000 people battle:
500 on 1 team, 500 on another.
Each team hits 1 person, you do 500+ checks, all in 1 tick. Tell me that won't cause lag at some time?

 

What is the likeliness of this?

 

jordanzilla02
For me, 100%
For others, idk. Good practice though.

 

I am talking about 500 players on one team, I am assuming this is for a minigame and that is sort of ridiculous to have 1k players on one server, you will have some lag... (a lot of lag)

 

jordanzilla02
Unless you run the better version.
Regardless of having 500 on 1 team, you still do checks for each player, checking lists and things. It is just overall better, trust me.

 

I just use, EntityDamagedByEntityEvent.
Then I have two ArrayLists, blue and red.
Check if the player and attacker is on the same ArrayList, if so then cancel the event.

 

calebbfmv
### OFFTOPIC ###
since you are so against the comparing two lists. I started to wonder, what do the scoreboards do?

 

IkBenHarm
Metadata

 

Already solved, ty guys.

 

calebbfmv
ah okay