Discussions

Not really. It mostly is up to the programmer. Oftentimes, the same name as the field will be used as follows: <Code Omitted>Other times, you might see an underscore being used: <Code Omitted>If you were to add complete documentation and "publish" the class, you may be "forced" to specific naming so that the documentation ma

A lot of code for doing quite little. The following would probably have sufficed: <Code Omitted>

Yes and yes.

It might be easiest if you placed each animation image set in an array; then make an array of animation sets: <Code Omitted>You

Store the two images in a field. In the act method, change the image either every act cycle of every x act cycles with the help of a timer variable.

You remove the actor from the world and then try to get the current position of the actor in its world. But it isn't in a world anymore.

You can probably use setWorld and then create a new instance of the current world you use.

I don't have experience doing this in Greenfoot, but it should not be that different from other solutions, and should work without access to the main method. Could you try to show some of the code you use, that doesn't work?

You have to move the "nextLevel2" method call from the constructor to the "act" method, so the check is performed during each act cycle.

As mentioned by others you should avoid removing and adding objects. However I believe the slowdown happens because you create a new GreenfootImage that is quite large. Two ways you can improve this, if possible make the image smaller than 1000x100, and instead of creating a new image every time, you could store the image in an instance field and update it as needed.

For an easy start, import the Counter class. Try to follow its description and come back with questions if you have any.

I would probably control it from the world and not from the cards. If the cards should do it on their own, you can have one static field which saves the card which was clicked on first. So when you click a second card, you can decide if this card's image equals the static one and after this check, set the static card variable back to null.

Thanks for your answer, I have sent the project files to the given adress. I hope you can help me with my problem!

This line is doing nothing at all: <Code Omitted>