[WORKAROUND] getServer().getPlayer() deprecated in Bukkit 1.7.8

So we all must have noticed that in the latest developer build of Bukkit, the getServer().getPlayer(String arg0) method has been deprecated.

This means that the onCommand() methods will not be able to process a player based on args.

I just made a useful workaround for this, which lets you get an online player by his/her name.

Here's the method (just add this somewhere in your code)

Code:java

public Player getPlayer(String name) {
    for(Player p : Bukkit.getOnlinePlayers()) {
        if(p.getName().equalsIgnoreCase(name))
            return p;
    }
    return null;
}

(code shortened by amhokies)


Original Code:

Show Spoiler Hide Spoiler

Code:java

public Player getPlayer(String name)
{
Player[] online = getServer().getOnlinePlayers();
ArrayList<String> names = new ArrayList<String>();
 
Player player = null;
 
for(Player p : online)
{
names.add(p.getName());
}
 
if(names.contains(name))
{
int index = names.indexOf(name);
player = online[index].getPlayer();
}
 
return player;
}

Then to use it in another method (say you want to set him/her as flying) just do

Code:java

getPlayer(playersName).setFlying(true);

Hope this helps!


- EnderLance

 

EnderLance
While this was nice of you to share your idea with everyone, just because it's deprecated doesn't mean you can't use it. In most cases it's a good idea to keep from accessing deprecated methods, but in this case, the only reason it's deprecated is to raise awareness of UUID migration beginning in 1.8.

EDIT: also, your code can be condensed quite a bit to:

Code:java

public Player getPlayer(String name) {
    for(Player p : Bukkit.getOnlinePlayers()) {
        if(p.getName().equalsIgnoreCase(name))
            return p;
    }
    return null;
}

 

Thanks for the quick reply

Ah, I wasn't aware you could use deprecated method... Still a novice here

Anyways, would you mind if I updated my post with the shortened code you provided?

Thanks,

- EnderLance

 

Sure, go ahead.

 

Wrong section. Try Resources next time.

 

1. The method was deprecated to "raise awareness" about the UUID methods, or something like that. You can still use it just fine, in fact the deprecation will be removed with the 1.8 update according to Bukkit. Also, using something that does the same thing as a deprecated method is essentially using the deprecated method, so why not just use the deprecated method?
2. Your method is actually "replicating" getPlayerExact(), not getPlayer(). If you're curious, here is the getPlayer() method.

 

You just posted source code of getPlayer(String) from cbukkit...

if you want to process player without referencing him by name - check my implementation here
https://github.com/RawCode/UBT/blob/master/src/rc/ubt/hnde/ForcedRespawn.java