Discussions

This code is so easy, danpost 1v1 me on greenfoot on the 21st September 2029

Along with the addObject line you inserted above, also add the following line: <Code Omitted>(and similiarly with other level changes)

The problem is that when you remove this actor from the world in the vermoordMonster method, then go back to the act method, it then calls the vermoordNachtbaas method, which require the actor still be in the world. Easy fix is to either change line 11 (in act) to: <Code Omitted> OR, insert the following line

What type of actors is the Shield instance protecting the spaceship from?

I did notice that in multiple places you have images being created continuously (every act frame). Image creation/manipulation can be a hog on CPU time and may cause some lagging. Also, I noticed that you have the health bar and the counter doing things they should not. GUI objects, as such, should be prompted to do things, not do them on their own -- and only do those things when required. In other words, they

The current code modifies an already existing zombie. I guess you want to spawn a *new* zombie, right? A new zombie can be created by calling the constructor, presumably "new Zombie();". This zombie then has to be placed into the world (addObject method). Assuming the code you showed is in method spawnZombie of your world class, you can try this:

Thank you very much :)

You're welcome. I am glad to help. I hope you found some fun with programming and keep coding.

Looks like the distance between lanes is 54 (/** lane offset value */).

You are right. Might be better to have an (empty) abstract grabPedestrian method in the Grabber class and have each sub-type deal with their own specific behavior. If it is their specific behavior, then it makes sense to have the code in their specific classes.

Try this:

Seems to me that "getOneObjectAtOffset((int)speed + getImage().getWidth()/2, 0, Pedestrian.class);" is only working correctly if the demon meets the pedestrian's center exactly? Try with "getOneIntersectingObject(Pedestrian.class);":

grabbed can be null there, so replace the quoted lines with this (it moves the relevant line into the part where grabbed is checked to not be null):

Please refer to my Value Display Tutorial scenario.

fr