abell95 / galaga_invaders

A retro remix of Galaga and Space Invaders
1 stars 0 forks source link

Rename player.rb to ship.rb #16

Closed bmchristensen closed 5 years ago

bmchristensen commented 5 years ago

Specificity in class name is potentially restrictive. If this class was extended or inherited by another, the name would be misleading since the player does not control any other actors in the system. Specificity is important, but probably not at the cost of flexibility and tolerance to change. I propose that the player.rb aka player_ship.rb be renamed to ship.rb.

_Originally posted by @bmchristensen in https://github.com/abell95/galaga_invaders/issues/9#issuecomment-462470130_