radia | DNA Integrated Analysis for Somatic Mutation Detection | Genomics library

Great job so far. You have all the basics of a game/animation set up correctly. You are right- the only real issue is in the collisions function. Look at what's happening in there.

It picks a circle, calling it first. Then for every other circle on the screen

• if they intersect, it changes both colors to the collision color
• if they don't intersect, it changes both colors back to the default

Now look over that again, and consider what happens if when the first circle checks collision for circles that actually collide with it first. Then the LAST thing to happen in that loop will be checking its collision with the other circles (that don't collide with it) and will change all their colors back to the default.

Basically you need to rethink your logic in how you are creating those two loops. I would suggest for instance, adding a boolean flag (say, colliding) that can be checked after the inner loop completes- perhaps even adding it as a property of your Circle instances. So if first is colliding with something in the inner loop, then set first.colliding = true. In your Circle draw() function, you can set the color based on this property.

Luckily, the way these loops were set up, it actually overrode this other bug. You aren't accounting for when the second circle is the same object as the first circle (it will always have zero distance from itself, so should always turn green... but the bug above always turned it back). You can account for this by adding a check like this inside the inner loop:

if(first !== second) or if(i !== p) etc.