Discussions

Your for loops are set to iterate twice the length of the array. Meaning, you are creating more Track objects than the array can hold. That is, your array length is 68880 and you try to place 137760 Track objects in that array. It can't be done. Hence the error once your loop counter reaches 68880, since the last element is indexed at 68879.

Where do you think these would be taken care of and what code do you think wold be appropriate? That is, show some attempt.

Thank you @danpost, its working better now. :)

Thanks man, you carry this site

Thanks so much, I used 1 health for each player and used a switch and it cleaned up my code (and performance) by sooo much

Actually, never mind I managed to fix it, thanks alot for your help @danpost

Thank you

Try to remove the brick from the ball class and remove all code inside the brick class. It’s never good to have intersection checks in the classes of both objects which intersect with each other.

thank you @danpost, that makes sense. Thank you for your help :)

ah alright I will try make some code with this example and then come back to say if it worked just to keep updated

What have you tried? Show attempted codes (which should be in subclass of World)..

alright, i'll check it out. thanks for the help!

I want to clarify that I posted my response above only minutes after the question was asked (some hours before Super_Hippo's response). However, the post got held back due to checking it for spam (seems that a link to the greenfoot website triggers the spam alarm). That's why there is now an additional answer shown despite Super_Hippo's answer already being completely sufficient. Live long and prosper, Spock47

“getIntersectionObject” returns an intersection actor. If there is no Rightplayer actor intersecting the actor this code is in, it will return null. (Since the two paddles will probably never intersect each other, it will always be null.) The error occurs because you call “setLocation” on null.

Got it! thank you so much! :)