Solved How would you scan through a config to find a match?

Hello guys, it's me again, still trying to make that regioning plugin. I've managed to save a players name, and 6 different values to define a region. Here's what it looks like in a config.yml:

Code:

Regions:
  players:
    SupersonicSauce:
      bounds:
        minX: 335.0
        miny: 71.0
        minZ: -289.0
        maxX: 342.0
        maxY: 77.0
        maxZ: -280.0
    Bob:
      bounds:
        minX: 999
        miny: 0
        minZ: -12
        maxX: 1000
        maxY: 256
        maxZ: -3
 
 
 

Here is the code that saves it:

Code:java

    public void saveRegion(Player player, double minX, double minY, double minZ, double maxX, double maxY, double maxZ){
        config.addDefault("Regions.players."+player.getName()+".bounds.minX", minX);
        config.addDefault("Regions.players."+player.getName()+".bounds.miny", minY);
        config.addDefault("Regions.players."+player.getName()+".bounds.minZ", minZ);
        config.addDefault("Regions.players."+player.getName()+".bounds.maxX", maxX);
        config.addDefault("Regions.players."+player.getName()+".bounds.maxY", maxY);
        config.addDefault("Regions.players."+player.getName()+".bounds.maxZ", maxZ);
        this.saveConfig();
 
 
    }
 

Now, here's the part that I'm totally clueless on. On a server with potentially hundreds of players, how would I be able to iterate quickly through the entire config to check if a player is building inside of an existing region?

 

supersonicsauce I can help you, but I really need to know what you're trying to do.

• Can players only edit in their regions
• Can only certain players edit in certain regions
• Can players edit in any region

 

unrealdesign
Well, players will not be able to edit other players regions, but they can edit their own regions and any unclaimed land. But the important thing I need to know how to do is finding if a player is inside of a region. I think once I know how to do that, it should be relatively straightforward to restrict players from building in certain areas.

 

Have a player region class. Have an set of all instances of player regions class. In the class, have the start location, end location, and an set of all inbetween. Also have a method to calculate all the locations and put then into an array separate from getting the array, because this will save calculations. Just update it every time you change the start and/or end point. Then every time a player builds, check if he is in his region, and if he isnt, then check if he is in someone elses, then if he isnt, he isn't in a region. To iterate between one location to the next and get all blocks, use this code (not tested):

Code:java

for(it x=xMin; x<=xMax; x++)
    for(y=yMin; y <= yMax; y++)
        for(z=zMin; z <= zMax; z++)
            set.add (new Location(world, x, y, z))

 

unrealdesign

But...how? What do you mean by this?

 

unrealdesign *cringe* instead of checking if player location is in the region set, why not check if the player coords are between the region low and high points? 6 math checks would be cheaper than hundreds of equals.

 

Oh my bad I'm at work and replied fast haha

 

Necrodoom unrealdesign
Ok, well, how would I check? That's the main problem here. I'm at some point going to have a config with dozens of sets of 6-point region data, and have no idea how to scan through them to find if a player is inside them. How would I do this?

Edit: I should say that I can probably figure out how to check if a player is inside of a region. My problem is mainly with getting the 6 location points from each individual player's region, and repeating that for as many players are in the list. I have no idea how to accomplish this.

 

supersonicsauce Why would you want to loop through all players in the list? At most you'd want to loop through all online players which can be done with a for-each loop, and looping through getServer().getOnlinePlayers().

Then just use the getInt(path + p.getName() + restOfPath) method of the config object.

And add a check to see if it returns null, if it does that player does not have a region and you can return

 

Flamedek If I looped only through online players, then players could break the blocks in the regions of other players who have logged off. All someone would have to do to grief is to wait until their target logged off.

But just out of curiosity and for the sake of getting closer to finishing this, how exactly would I do a for loop with getOnlinePlayers()? Could you give me an example of the syntax?

 

supersonicsauce

1. Create a instance variable Map<String, Location[]>. The string will be for the players name, and the array of locations will be for the the 2 corners of the region.
2. On plugin enable, read from the config and write to your new Map with the correct information.
3. On creation/deletion of a region, add/remove it to your config like normal, but also add/remove it to your new Map.
4. Create a method canBuild(Player p) method. In this method loop through the new Map and see if the player is in one of the regions between the two locations, if he isn't then he is outside of that region. If he is, check if he is the owner, or check the config and see if he is a member of the region. If then cancel it
5. Check your canBuild(Player p) method for every event you want

OR

1. Create a Region class *PREFERRED*
2. Create instance variables: Player owner, Set<String> members, Location corner1, Location corner2.
3. Create getters, setters, and other methods: boolean hasPermission(Player p), boolean inRegion(Location loc).
4. In your main class or manager class, create a Set<Region>.
5. Create a new Region instance for every part in your config and add it to the Set<Region> on plugin enable.
6. When a region is removed/added also update your Set<Region> appropriately.
7. In your events, loop through your Set<Region> instance variable and see if the player is inRegion(Location loc). If he isn't any any then you're fine. If he is in one, check if he is the hasPermission(). If he does, you're fine, if not, cancel the event.

 

unrealdesign

How do I get a Map from a config?

Ok. I fixed it.
For anyone who is having the same issue, here's what I did:

Make an ArrayList that you will keep your players in.

Code:java

 ArrayList<String> playerlist;
public void onEnable(){
playerlist=newArrayList<String>();
//then, after you've declared it, make it read the list from the config.
playerlist=(ArrayList<String>)getConfig().getStringList("PlayerList");
// don't forget to save your config.
saveConfig();
}
 

Save your region with this method.

Code:java

public void saveRegion(Player player, double minX, double minY, double minZ, double maxX, double maxY, double maxZ){
config.addDefault("Regions.players."+player.getName()+".bounds.minX", minX);
config.addDefault("Regions.players."+player.getName()+".bounds.miny", minY);
config.addDefault("Regions.players."+player.getName()+".bounds.minZ", minZ);
config.addDefault("Regions.players."+player.getName()+".bounds.maxX", maxX);
config.addDefault("Regions.players."+player.getName()+".bounds.maxY", maxY);
config.addDefault("Regions.players."+player.getName()+".bounds.maxZ", maxZ);
this.saveConfig();
 
 
}
 
 

When a player joins, you must add them to the PlayerList that we made earlier. Since there is no way a player can be on the server without joining, it's impossible that any player will not be added to the list.

Code:java

@EventHandler
public void onJoin(PlayerJoinEvent event){
Player player = event.getPlayer();
//if our array DOES NOT contain the player who is joining
if(!playerlist.contains(player)){
playerlist.add(player.getName());
getConfig().set("PlayerList", playerlist);
 
}
this.saveConfig();
}

We have everything we need now. So, as an example, here's how we would detect if a player is breaking blocks inside of a region:

Code:java

 
@EventHandler
public void onMove(BlockBreakEvent event){
//get the list from the config
config.getList("Playerlist");
Player player = event.getPlayer();
Block block = event.getBlock();
//run through our playerlist
for(String p : playerlist){
//basically we are getting the 6 location points from our config, but substituting in the player name for each repetition.
double minX = config.getDouble("Regions.players."+p+".bounds.minX");
double minY = config.getDouble("Regions.players."+p+".bounds.minY");
double minZ = config.getDouble("Regions.players."+p+".bounds.minZ");
double maxX = config.getDouble("Regions.players."+p+".bounds.maxX");
double maxY = config.getDouble("Regions.players."+p+".bounds.maxY");
double maxZ = config.getDouble("Regions.players."+p+".bounds.maxZ");
 
double X = block.getLocation().getX();
double Y = block.getLocation().getY();
double Z = block.getLocation().getZ();
 
if(X <= maxX && X >= minX && Y <= maxY && Y >= minY && Z <= maxZ && Z >= minZ){
System.out.println(player.getName()+" IS BREAKING BLOCKS INSIDE OF "+p+"'s region!");
 
}
}
 
}

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

 

supersonicsauce Could just use ConfigurationSection#getKeys(), where the ConfigurationSection is "Regions.players"

 

xTigerRebornx
Could you give me an example of the syntax?

 

supersonicsauce Something like

Code:

config.getConfigurationSection("path").getKeys(false);

Would return a List<String>, which contains the keys.
If you have a config like this:

Code:

path:
  node:
  value1: 1
  node2:
  value1: 2
  node3:
  value1: 2124142

Calling getKeys(false) on the section 'path' would return a List<String> containing node, node2, and node3

 

xTigerRebornx
So something along the lines of this?

Code:java

 playerlist = (ArrayList<String>) config.getConfigurationSection("PlayerList").getKeys(false);
 

Also, what if getKeys(true)? Would it retrieve value1 as 1, then 2, then 2124142?

Edit: Also, when I don't cast, I get the error Type mismatch: cannot convert from Set<String> to ArrayList<String>

 

supersonicsauce No, it would just return the paths. You would then have to do config.get(path)

What do you mean? You create the map, then you iterate through your code and would getKeys(false) for the path to the names, then inside that, you would getKeys(false) to get your locations, then you save to the map with your name, and then create/add the locations from the second getKeys(false) loop. But anyways you found a workingish way, so it doesn't matter until you maybe run into problems. Goodluck

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

 

supersonicsauce No, you don't need the PlayerList. The Path "Regions.players"'s keys are the Player Names. You call getKeys(), passing in false to only get the keys and not its children, and it will return a Set (I believe)with all of those names. tldr, getKeys() replaces the PlayerList.
Also, you should be storing with UUIDs to prepare for name changes.

 

xTigerRebornx unrealdesign
Ok, I did what both of you suggested and got rid of the playerlist after realizing how pointless it was when I could just get keys. So thank you both very much for your help!
Now, how would I store a UUID in a config? And how can I get the last known username?

 

Save UUID: uuid.toString() and save it to the config
Last name: save all uuids along with the players name to the config. Once a player joins with a uuid the same as one in your config, replace the name with their current name in the config with the corresponding uuid.

 

unrealdesign
How do I replace their name with the one in my config?
Edit: Ah I see, you mean to replace what I have in my config with the more recent one.