How would i check if the player is on this certain team

Code:

package me.blueninjn.minigame;
 
import java.awt.List;
 
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.plugin.java.JavaPlugin;
 
public class minigame extends JavaPlugin{
   String version = "0.1";
   List TeamBlue = new List();
   List TeamRed = new List();
public void onEnable() {
System.out.print("MiniGamePlugin" + version + "Enabled");
}
public void onDisable() {
System.out.print("MiniGamePlugin" + version + "Disabled");
}
@Override
public boolean onCommand(CommandSender sender, Command command, String label,
String[] args) {
Player player = (Player)sender;
if(label.equalsIgnoreCase("blue")){
if(!TeamRed.equals(player.getName())){
TeamBlue.add(player.getName());
player.sendMessage("you have been added to the blue team");
}else{
player.sendMessage("you are already in the red team !");
}
}
 
if(label.equalsIgnoreCase("red")){
if(!TeamBlue.equals(player.getName())){
TeamRed.add(player.getName());
player.sendMessage("you have been added to the red team");
}else{
player.sendMessage("you are already in team blue!");
}
}
 
if(label.equalsIgnoreCase("whoisblue")){
player.sendMessage(TeamBlue.getItems());
}
if(label.equalsIgnoreCase("whoisred")){
player.sendMessage(TeamRed.getItems());
 
}
if(label.equalsIgnoreCase("MyTeam")){
if(TeamBlue.toString().contains(player.getName())){
player.sendMessage("you are in the blue team");
}
else if(TeamRed.toString().contains(player.getName())){
player.sendMessage("you are in the red team");
}else{
player.sendMessage("you are not in a team");
}
}
return false;
}
 
 
   
}

I'm wanting to find out the player team so i want it to check whether it is red, blue or none and i cant seem to get this to work , as how i am doing it says i am not in a team even though i am

 

Use

Code:java

List<String> TeamBlue = new List<String>();
 
if(TeamBlue.contains(player.getName())){
  doStuff();
} else {
  doOtherStuff();
}

instead; this should help you find out what you need to do.

 

i get a error saying remove type arguments and will not allow me to do that , any fixes for this

 

ahuby09

Code:java

 
ArrayList<Player> teamRed = new ArrayList<Player>();
ArrayList<Player> teamBlue = new ArrayList<Player>();
 
if(label.equalsIgnoreCase("Blue"){
Player player = (Player) sender;
if(teamRed.contains(player)){
return false;
}else{
teamBlue.add(player);
}
}
 
 
else if(label.equalsIgnoreCase("Red"){
Player player = (Player) sender;
if(teamBlue.contains(player)){
return false;
}else{
teamRed.add(player);
}
}

Should work.

 

Thanks gonna try it out now

can you help me on this i put

Code:

player.sendMessage(TeamBlue.toString().split(", "));

so it shows the blueteam but it gives a really long line like craftbukkit[playerName = blueninjn] how would i make it so it just said blueninjn

Dubehh

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

 

ahuby09

Code:java

if(label.equalsIgnoreCase("CheckTeam")){ //
Player player = (Player) sender;
if(redTeam.contains(player)){
player.sendMessage(ChatColor.RED + "You are in the Red team!")
}
else if(blueTeam.contains(player)){
player.sendMessage(ChatColor.BLUE + "You are in the Blue team!");
}else{
player.sendMessage(ChatColor.YELLOW + "You don't have a team :c");
}

If you mean that

 

Next time you need help, please format your code better so it is easier for us to read. Your code looks revolting at the moment due to the lack of indentations.

 

It is indented on my eclipse how would i do it on here ?

No i mean it shows a long string when i just want it to show blueninjn and i want it for multiple players not me if you understand it very hard for me to explain , but thanks for trying to help

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

 

Replace TABS with Spaces.
I prefer using [ code ] and not [ syntax ] anyway.

 

KingFaris11 IMHO,

Code:java

 is much better for beginners.

 

FlareLine
We all have our own opinions. =) I mean, yes, syntax is better for line numbering but I just quote code and it helps people recognise code easily if I were to quote it (as they'd have to find it), hence proving your point about being better for beginners. I just like how

Code:

This is so clean.

 

KingFaris11 Heh.