Mar-06-2021, 05:49 PM
Indexing is not fine. Whenever you can make code easier to read without adverse side effects you should do so. a[0] is meaningless to the casual reader, or you when you come back and look at this code a year from now. aleft from ship.bounds is self documenting. Ideally it should be ship_left_bounds, but that makes the collision detection expression really long and I don't think it adds much to understanding.