You haven't actually asked a question.
To build a String out of multiple arguments, you can do something like this: StringBuilder sb = new StringBuilder(); for(int i =...
Try adding some debug messages to see if that code is even being executed.
KmanCrazy break and continue are two very useful keywords to know when dealing with loops. The keyword break will immediately terminate the loop...
I personally don't see anything wrong with using if(booleanExpression == false) over using if(!booleanExpression) In some cases it may be easier...
You would only need a player variable if you need to use methods that are specific to players (such as teleporting). Otherwise, you would be fine...
You will need to get a String representation of x. You can do this using Integer.toString(x) I believe.
Post your whole code then.
Could you be more specific about what you mean when you say it "doesn't work"?
Separate names with a comma.