This should work: Your main problem was that you were telling the computer to set the image to a string. What you needed waas something that took that string, figured out what image it was refering to, and then create a copy of that image that is usable by setImage().

Try: if (getImage().equals(new GreenfootImage("square1.png")) If that does not work, then you will probably need either a boolean instance variable or an int instance variable to keep track of which image is currently being used (as an alternative, an instance String variable to hold the current image file name).

For MathrMagician, you're comparing the pointers of the 2 GreenfootImages, but they are different since they ar e 2 different objects. The equals method compares the objects themselves, so danposts solution should work.

You should create a separate boolean to flag what image you current have set as the main image, use that to decide when to switch instead of the images themselves. And Spower, the equals() command is not defined for GreenfootImage, so it uses the default Object implementation, which is the same thing as using ==

That is what i am saying, the equals() function will not work :P The reason it works for cases like String is because String class overrides the equals() function to give it better meaning

I actually prefer to use an int which controls which element in an array to use. The array can be of String type or of GreenfootImage type. I guess the reason I prefer this way is that each step is absolute processing (no decision-making with 'if's or other control flow commands); the control code can be used multiple times to switch the image back and forth; and, the control code can be put inside a method to be called upon when neccessary (call the method something like 'swapImage'). 'imageNum' could be initially set to -1 and the method called to initially set the image.

Just came up with this: The ImageVisitor is part of the greenfoot package btw, so you don't need additional work to get access to it.