Teleporting players in hashmaps to individual locations.

Here is part of my code.. this works, but doesnt work how i want it to
I'm trying to add different spawn location for players in 2 different hasmaps..

Code:

      if (args[0].equalsIgnoreCase("bluespawn")) {
                if(p.hasPermission("mccod.setspawn")){
                        Location loc = p.getLocation();
                        if(!(warpCounter > 100)){
                            warpLocations[warpCounter] = loc;
                            warpName[warpCounter] = args[0];
                            warpCounter++;
                            p.sendMessage(ChatColor.GREEN+ "Blue team spawn set!");
                        } else {
                            p.sendMessage(ChatColor.RED+ "Limit exceeded!");
                        }
                }
            }
       
        if (args[0].equalsIgnoreCase("start")) {
            for(int i = 0; i < warpName.length; i++){
                for(Player player : Bukkit.getOnlinePlayers()){
                    Location warpLocation = warpLocations[i];
                    if(blue.contains(player.getName())){
                        player.teleport(warpLocation);
                }
                   
                    if(red.contains(player.getName())){
                        player.teleport(warpLocation);
                   
            }
        }
    }
}

Obviously that teleports both of the "teams" to the same place.

 

String name = ...;
HashMap<String, ArrayList<Location>> spawn = ...;
Random rand = ...;

// Adding a spawn location for a given team:
ArratList<Location> teamSpawns = spawn.get(teamName);
if(teamSpawn = null){
teamSpawn = new ArrayList<Location>();
spawn.put(teamName);
}
teamSpawn.add(newSpawnLocation);

// Getting a random spawn location for a given team:
teamSpawns = spawn.get(teamName);
if(teamSpawns = null) teamSpawns = new ArrayList<Location>();
int index = rand.nextInt(teamSpawns.size());
Location spawnLocation = teamSpawns.get(index);

 

I dont want random spawn locations, if you have a look at my code, you see i'm defining the spawn location with a command.
So wherever i'm standing, it will set a warp there, which i am going to use for the team spawns.

 

Your code is very messy, it is hard to understand what you want.

Use "HashMap<teamName, spawnLocation> spawnLocations" then.

Location loc;
if(blue.contains(player.getName())) loc = spawnLocations.get("blue");

 

My code is always messy
Also, thats still not what i'm asking.

Lets say i have 2 teams, red and blue.
I want to set the red team, and blue team spawn location with a command.



And when i use this start command, it would teleport the red and blue team to their spawns.

I have this "start" command set up, but as i said, it teleports both of the teams to same location.

Code:

 if (args[0].equalsIgnoreCase("start")) {
 
            for(int i = 0; i < warpName.length; i++){
 
                for(Player player : Bukkit.getOnlinePlayers()){
 
                    Location warpLocation = warpLocations[i];
 
                    if(blue.contains(player.getName())){
 
                        player.teleport(warpLocation); //over here, i would change the "warpLocation" to whatever i defined the blue team spawn.
 
                }
 
                    if(red.contains(player.getName())){
 
                        player.teleport(warpLocation); //same thing over here.

I could probably just use a config, and set the x, y, z locations, but its easier to get the player location with command and save it as warp.

 

Well off the top of my head you could reference your warping plugin you are using and save the location to there as a warp.

You could also make it yourself, it is actually really easy. You would probably make a HashMap that looks kind of like this:

Code:

Map<Integer, Player> teamPlayer = new HashMap<Integer, Player>();

Then you could do something like this to assign players to teams:

Code:

            public boolean onCommand(CommandSender sender, Command command, String label, String[] args) {
                int blueTeam = 0;
                if(label.equalsIgnoreCase("blue")){
                    teamPlayer.put(blueTeam, (Player) sender);
                    sender.sendMessage("You joined the blue team!");
                    return true;
                }else{
                    return false;
                }
            }

Doing that you can tp all players in the blue team to a location from a For statement like this:

Code:

for(Player player : teamPlayer.get(blueTeam)){
    player.teleport(warpLocation);
}

Hope that helped at least a little bit

 

That's what i have atm...

This

Code:

for(Player player : teamPlayer.get(blueTeam)){
    player.teleport(warpLocation);
}

is the actual problem.. i am trying to teleport the blue team, and red team to different spawnpoints, but i dont know how to set the locations of the spawnpoints for each team.
I want them to be different

Okay people, i made a picture for you, i hope you understand what i want!
View attachment 11342
How hard can it be to understand such a simple thing?

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

 

Let me see more of your source, That way I can review it better. I understand what you are trying to do..

 

Code:

Location redLocation = null; // or have set locations: new Location(world, X, Y, Z);
Location blueLocation = null;  // or have set locations: new Location(world, 100, 60, 100);
 
// Setting the location would be like this, obviously:
public void onCommand(parameters and shit)
{
    if (args[0].equalsIgnoreCase("setblue"))
        blueLocation = ((Player)sender).getLocation();
    if (args[0].equalsIgnoreCase("setred"))
        redLocation = ((Player)sender).getLocation();
 
    if (args[0].equalsIgnoreCase("start"))
    {
        for (Player p: Bukkit.getOnlinePlayers())
        {
             if (blueTeam.contains(p))
                 p.teleport(blueLocation);
             else if (redTeam.contains(p))
                 p.teleport(redLocation);
        }
    }
}

You mean like that?
Because to me it seems the answer has been given multiple times already.

 

That looks promising, ill try that tomorrow.
Thanks!

 

Do you want to have multiple spawn locations or just one the members of one team will go to?

Because I don't really get what you mean with:

Code:

                        if(!(warpCounter > 100)){
                            warpLocations[warpCounter] = loc;
                            warpName[warpCounter] = args[0];
                            warpCounter++;
                            p.sendMessage(ChatColor.GREEN+ "Blue team spawn set!");
                        } else {
                            p.sendMessage(ChatColor.RED+ "Limit exceeded!");
                        }

 

That looks very identical to most posts. OP needs to slow down.

In your code blueTeam and redTeam are HashMaps right?

 

Neither do i
It just allows you to set warp where you are spawning, so for example: /setwarp something
And then you could do /warp something

Arraylists actually, but i dont understand the difference between them. Both works just the same.
Also, show me the "identical posts"

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

 

I don't believe he used HashMaps in his code, that's why he used some weird array stuff. That you can also not understand because you can't see the whole source and he didn't tell you what it is

 

And how are you going about saving the locations so you dont have to keep setting them each reload/restart.

Ide suggest using YamlConfiguration http://wiki.bukkit.org/Introduction_to_the_New_Configuration

Oh, yes I saw the square brackets. It confused me because the title of the post is HashMap.

Well they have some differences.

With the HashMap you could save a Player object to the team's ID with relative ease. In my opinion anyways.

Well most the code I've seen does the same thing..

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

 

I was using hashmaps earlier, but got some better code from a friend, and he used Arraylists, so i decided to keep it

 

Ok Ill adjust to your situation, I thought you wanted to use HashMaps due to the title. If you could better help us to help you some updated code would be nice.

 

Cant get on my computer anymore, but i'll post the code here tomorrow.
WARNING - The code is very messy!

 

Alright I'll follow this post, and its ok. I'm sure we can help you clean it up a bit.

 

Okay well, here is the main class.
The purple text is the part where i am trying to set the spawns.

Code:JAVA

 
package me.woobie.mccod;
 
import java.util.ArrayList;
 
import org.bukkit.Bukkit;
import org.bukkit.ChatColor;
import org.bukkit.Location;
import org.bukkit.command.Command;
import org.bukkit.command.CommandSender;
import org.bukkit.entity.Player;
import org.bukkit.event.EventHandler;
import org.bukkit.event.Listener;
import org.bukkit.event.player.AsyncPlayerChatEvent;
import org.bukkit.plugin.java.JavaPlugin;
 
public class Main extends JavaPlugin implements Listener {
 
public final Location[] warpLocations = new Location[100];
public final String[] warpName = new String[100];
public int warpCounter = 0;
 
public void onEnable() {
getServer().getPluginManager().registerEvents(this, this);
}
 
public ArrayList <String> red = new ArrayList <String> ();
public ArrayList <String> blue = new ArrayList <String> ();
 
@EventHandler
public void onPlayerChat (AsyncPlayerChatEvent e) {
Player p = e.getPlayer();
if (red.contains(p.getName())) {
p.setDisplayName(ChatColor.RED + "[Red] "+ ChatColor.GRAY + p.getName());
p.setPlayerListName(ChatColor.RED+ p.getName());
}
if (blue.contains(p.getName())) {
p.setDisplayName(ChatColor.BLUE + "[Blue] "+ ChatColor.GRAY + p.getName());
p.setPlayerListName(ChatColor.BLUE+ p.getName());
}
}
 
public boolean onCommand (CommandSender sender, Command cmd, String commandLabel, String[] args) {
 
if (!(sender instanceof Player)) {
sender.sendMessage("The console can't use MCCoD!");
return true;
}
 
Player p = (Player) sender;
 
if (cmd.getName().equalsIgnoreCase("mccod")) {
if(args.length == 0){
if(p.hasPermission("mccod.main")){
p.sendMessage("The main command");
}
 
else {
p.sendMessage(ChatColor.GREEN+ "You do not have the correct permissions to use this command");
}
}
}
 
 
// Location blueLocation = null;
// Location redLocation = null;
//
// if (args[0].equalsIgnoreCase("setblue")){
// blueLocation = ((Player)sender).getLocation();
// p.sendMessage(ChatColor.GREEN+ "Blue team spawn set!");
// }
//
// if (args[0].equalsIgnoreCase("setred")){
// redLocation = ((Player)sender).getLocation();
//p.sendMessage(ChatColor.GREEN+ "Red team spawn set!");
// }
//
// if (args[0].equalsIgnoreCase("start")){
// for (Player plr: Bukkit.getOnlinePlayers())
// if (blue.contains(p)){
// plr.teleport(blueLocation);
// }
//
// else if (red.contains(p)){
// plr.teleport(redLocation);
// }
// Bukkit.getServer().broadcastMessage(ChatColor.GREEN+ "The MCCoD game has started!");
// Bukkit.getServer().broadcastMessage(ChatColor.GREEN+ "15 minutes until the next game starts.");
 
 
}
if (args[0].equalsIgnoreCase("info")) {
p.sendMessage(ChatColor.GREEN+ "==========[MCCoD]==========");
p.sendMessage(ChatColor.GREEN+ "Name: " +ChatColor.BLUE+ "MCCoD");
p.sendMessage(ChatColor.GREEN+ "Version: " +ChatColor.BLUE+ "v0.1");
p.sendMessage(ChatColor.GREEN+ "Description: " +ChatColor.BLUE+ "Call of Duty in Minecraft!");
p.sendMessage(ChatColor.GREEN+ "/mccod info2" +ChatColor.BLUE+ "For the next page");
p.sendMessage(ChatColor.GREEN+ "==========[Page 1/2]==========");
 
}
if (args[0].equalsIgnoreCase("info2")) {
p.sendMessage(ChatColor.GREEN+ "==========[MCCoD]==========");
p.sendMessage(ChatColor.GREEN+ "Author: " +ChatColor.BLUE+ "Woobie");
p.sendMessage(ChatColor.GREEN+ "Dev page: " +ChatColor.BLUE+ "Coming soon");
p.sendMessage(ChatColor.GREEN+ "Source code: " +ChatColor.BLUE+ "Coming soon");
p.sendMessage(ChatColor.GREEN+ "==========[Page 2/2]==========");
 
}
 
if (args[0].equalsIgnoreCase("red")) {
if (!(blue.contains(p.getName()))) {
red.add(p.getName());
p.sendMessage(ChatColor.GREEN+ "You have joined the " + ChatColor.RED + "red" + ChatColor.GREEN + " team.");
return true;
}
else p.sendMessage(ChatColor.GREEN+ "You are already on the" + ChatColor.BLUE + " blue " + ChatColor.GREEN+ "team!");
return true;
}
 
if (args[0].equalsIgnoreCase("blue")) {
if (!(red.contains(p.getName()))) {
blue.add(p.getName());
p.sendMessage(ChatColor.GREEN+ "You have joined the " + ChatColor.BLUE + "blue" + ChatColor.GREEN+ " team.");
return true;
}
else p.sendMessage(ChatColor.GREEN+ "You are already on the" + ChatColor.RED + " red " + ChatColor.GREEN + "team!");
}
if (args[0].equalsIgnoreCase("leave")) {
if (red.contains(p.getName())) red.remove(p.getName());
if (blue.contains(p.getName())) blue.remove(p.getName());
else {
p.sendMessage(ChatColor.GREEN+ "You never joined a team!");
return true;
}
p.sendMessage(ChatColor.GREEN + "You have left.");
p.setDisplayName(ChatColor.RESET + p.getName());
p.setPlayerListName(ChatColor.RESET+ p.getName());
}
return false;
}
{
}
{
{
}
}
}