All you need now is an act() method to check to see if the game is over (100 points?).

Just put that if statement in the act() method of BackGround. And you can remove the 'private boolean gameOver' line and then you must remove the 'gameOver = true' line of gameOver(), because you don't use it. After that: if there is an error, and it uses the gameOver boolean, just remove the line.

@martijn13039, in the code you last posted, your prepare method is RE-creating objects (which is something I do not think you want to do). Change the method like so: Although a lot of this can be done in the variable declaration section as Then all you would have to do is add the objects to the world. The scoreboard was not initialized because you will do that later when you have a score to put in it. Remember: if you do change your declarations to initialize the variables, remove the assignments in the code.

I was mainly comparing lines 6 and 7 with lines 33 and 34. Both lines 6 and 33 each create a seperate Counter object, one called 'Counter' and the other 'counter'. Along the same lines (of thought), lines 7 and 34 both create a seperate concierge object, one called 'user' and the other 'concierge'. Create your object once, and stick with the name you named it (unless you have more than one of that type object to place in your world). One more thing, standard Java naming convention: method and variable names begin with lowercase letters and class names begin with uppercase letters (and static final variables are all uppercase). Java is case sensitive, so 'Counter' is not the same as 'counter'.

I will thank you all so mutch its working great now. thank you martijn

You will still need the boolean gameOver. 1) declare it with 'private boolean gameOver = false;' 2) change the 'if' in your act() method to 'if (counter.getValue() >= 100 && !gameOver)' 3) make sure the statement 'gameOver = true;' is in the gameOver() method @SPower, without the boolean, act() will call gameOver() repeatedly (not wanted)