2Karl

Yes, getObjectsInRange would be more efficient. the reason I used getOneIntersecting Object is because the person I was helping out is not only new to greenfoot, he's new to Java as well. rather than confuse him with optimisation from the off, I thought I'd show him the logic behind collision detection so that he can investigate more efficient methods as he develops. Thanks for the input though guys ;)

I should mention that the code is fully commented for those of you who are curious.