other_entities_in_vicinity() flawed

[dgets]

Should return the entity that is in the vicinity, or None. Code outside of the method can handle determining an intercept angle or whatever. It's far too specific to have the method handling that; destroys the point of having things modular.

[dgets]

I was just thinking that a better purpose for this method is as follows:

• check for enemies in a given radius
• determine whether or not they are docked
  • target docked enemies first, as they cannot fight back

Later improvements, or additions via another method, may be to determine the course of enemies within a given radius. If they're on an intercept course, match course with them to prolong time damaging them and avoid being destroyed in a collision.