Loading Hashmap from Config with ArrayList as value

So, I've seen the countless threads asking how to load and save hashmaps into and from configs but I haven't found how to load a hashmap when the value is an arraylist.

My hashmap:

Code:java

public HashMap<Integer, ArrayList<String>> myMap = new HashMap<Integer, ArrayList<String>>();

How I save hashmap to config:

Code:java

for(int x : myMap.keySet()){
  for(ArrayList<String> lloc: myMap.values()){
    for(String loc: lloc){
       Bukkit.broadcastMessage(loc);
       y+= 1;
       getConfig().set(String.valueOf(x) + ".location" + y, loc.toString());
    }
  }
}

^^I have this properly indented in my code but for some reason it keeps deleting the indentations in the editor...

And this is how the config looks:

HTML:

'1':
  location1: -707,14,1062
  location2: -706,15,1053
'2':
  location1: -693,15,1081

So, my question is how do I then load the config back into the hashmap?

 

Do your locations need names? Why are you storing them "key: value" instead of in an actual ArrayList?

Assuming they DO need names for some unknown reason (sometimes you need to get a specific location?..)
I don't understand your saving method. It's very inefficient. You're looping over ALL the values for ALL the keys. Just do this:

Code:JAVA

 
public void saveLocations(HashMap<Integer, ArrayList<String>> map) {
int y = 1;
for(int x : map.keySet()) {
for(String s : map.get(x)) {
getConfig().set(x + ".location" + y, s);
y++;
}
}
}
public HashMap<Integer, ArrayList<String>> loadLocations() {
HashMap<Integer, ArrayList<String>> map = new HashMap<Integer, ArrayList<String>>();
/*
I assume your entire config is JUST these lists of locations
Use this line if your locations are all stored in one specific section of your config instead
If so, replace all getConfig().getKeys(false) with section.getKeys(false)
ConfigurationSection section = getConfig().getConfigurationSection("Locations");
*/
for(String s : getConfig().getKeys(false)) {
ArrayList<String> list = new ArrayList<String>();
int i = Integer.parseInt(s); // This will only work if all the names of your lists of locations are integers, NO EXCEPTIONS
for(String s1 : getConfig().getConfigurationSection(s).getKeys(false)) {
list.add(getConfig().getString(s + "." + s1);
}
map.put(i, list);
}
return map;
}
 

 

Ethan
You should change the way your locations are held in the config... Maybe
Locations:
1: x,y,z
2: x,y,z
Also, change your HashMap to

Code:java

 
public HashMap<Integer, Location> myMap = new HashMap<Integer, Location>();
 

If you do that, then you could do this

Code:java

 
public void createLocations() {
    for (int i = 1; i != -1; i++) {
        String location = getConfig().getString("location." + i);
 
        if (location != null) {
            String[] args = location.split(",");
 
            if (args.length > 2) {
                double x = Double.parseDouble(args[0]);
                double y = Double.parseDouble(args[1]);
                double z = Double.parseDouble(args[2]);
 
                Location loc = new Location();
                loc.setX(x);
                loc.setY(y);
                loc.setZ(z);
 
                myMap.put(i, loc);
            }
        }
    }
}
 

I'm pretty sure that will work, but I didn't create it using an IDE, so there may be some problems... If there are, just let me know (;

 

MrSparkzz
Why would he change his HashMap? He's trying to store lists of strings, not single locations..

 

RealDope
If he were to change his HashMap to an integer and a location, then when retrieving the information would be much easier. You could do something like this

Code:java

 
myMap.get(/*int*/);
 

and that would return a location and inside that location if he wanted to get a specific part he could do

Code:java

 
Location location = myMap.get(1);
 
double x = location.getX();
 

 

He's storing a list of locations as strings... He's not trying to associate one integer with one location, it's one integer with many locations. Hence the use of an ArrayList..