Actually, there are a couple of problems with the given code. (a) you are getting a random number; but not doing anything with it; you are asking for a random value between zero and the current value stored in the 'grn' field; this does not assign the returned random value to the 'grn' field; the line has the same effect as a 'no-op' command; (b) using 'setColor' does not produce any immediate change to the image; it only sets the drawing color for future drawing method calls; You should use the constructor of the class to set the value of 'image', set the drawing color an set the rotation of the actor, although the 'image' field is not needed (the Actor class supplies a private field for the image and it is returned by calling the getImage method); this would make your class look something like this: I change the name of your 'COME_DOWN' field to 'actCounter' since you are changing its value by one each cycle; however, I do not, with the code given, see the use for the field. Field names by convention, unless defined as 'final static' should be in lowercase.

Thanks so much! Everything is better except for one thing... The colour isn't changing... Thanks anyway!

When do you want the color to change? what conditions are required for that action to take place?

RIGHT AWAY! As soon as the game starts. I'm a noob.

It's not working....

Show your revised constructor (use 'code tags').

public Block() { setRotation(90); getImage().setColor(Color.BLACK); }

Where is the method call to draw or fill the image with black? The 'setColor' method is similar to picking a crayon out of the box. It does not take the crayon to the paper. Or, like putting paint of the brush from the pallet; not placing the brush on the easel.

public Block() { setRotation(90); getImage().setColor(Color.BLACK); } public void act() { moveDown(); fillBlock(); } public void fillBlock() { GreenfootImage.fill(this); }

Several problems with the code that was given: (1) if you want the image to be black from the start, you want to fill the image in the constructor of the class -- not in the act method; (2) the 'fill' method does requires zero parameters -- and 'this' refers to the Block object, not its image; (3) the 'fill' method is an instance method (not a 'static', or class, method) -- meaning you cannot call the method on the class,'GreenfootImage' -- you need to call it on the image you want to fill: (you can get the reference with the 'getImage' method)