Shorter location getMethod

Code:java

public void getLocation(String xpath, String ypath, String zpath){
double x = config.getDouble(xpath);
double y = config.getDouble(ypath);
double z = config.getDouble(zpath);
org.bukkit.Location name =new org.bukkit.Location(null, x,y,z);
return name;

I am stuck now, i want that this method will return the location in one variable called name, how can i do this?

 

public Location getLocation(String xpath, String ypath, String zpath) {
double x = config.getDouble(xpath),
y = config.getDouble(ypath),
z = config.getDouble(zpath);
return new Location(null, x, y, z);
}

Changing null to Location designates that the method should return a Location.

 

TomTheDeveloper
'void' is the return type, you change that and then you return the object you want, something like Scizzr posted but you should not use 'null' in the new Location(), that's where you need to supply a world, it throw an exception instantly or it will bite you back when you're trying to use it on something that's world related.

 

Good point; oversight.

TomTheDeveloper
You should store the world's name in the config, since you've got everything else there.

Code:

public Location getLocation(String worldpath, String xpath, String ypath, String zpath) {
    String w = config.getString(worldpath);
    double x = config.getDouble(xpath),
           y = config.getDouble(ypath),
           z = config.getDouble(zpath);
    return new Location(w, x, y, z);
}

 

Scizzr
The world is not a string, it needs a World object
Either parse the World object directly as argument or use Bukkit.getWorld() to get the World object from a world name.

 

Sigh. #SleepDeprivation

Code:

public Location getLocation(String worldpath, String xpath, String ypath, String zpath) {
    String w = config.getString(worldpath);
    double x = config.getDouble(xpath),
           y = config.getDouble(ypath),
           z = config.getDouble(zpath);
    return new Location(Bukkit.getWorld(w), x, y, z);
}

 

Okey thx

Thx man,

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