Revisar algunos nombres tanto de funciones como de archivos y carpetas, ya que dan un indicio erróneo de lo que representan o de su contenido.
Por ejemplo, una función como MenuLoop hace pensar en que en su interior tiene un loop (como un while o un for), cuando en realidad se llama así porque es llamada dentro del contexto de un loop. No debería llamarse así por el contexto desde donde se llama sino por lo que ejerce o ejecuta. Siguiendo este caso en particular, la función actualiza el estado del menú o hace el chequeo de input en el menú, por lo que debería llamarse UpdateMenu o CheckInputMenu o similar.
Notar también que hay una falta de consistencia en las siguientes funciones:
- InitMenu
- MenuLoop
- DrawMenu
La del medio tiene la palabra Menu al principio y el resto al final.
El nombre del archivo, por otra parte, tal vez sería mejor que se llame MenuScreen o simplemente Menu, en vez de MenuLoop, para sugerir justamente que tiene funciones y variables relacionadas con esa pantalla.
Revisar algunos nombres tanto de funciones como de archivos y carpetas, ya que dan un indicio erróneo de lo que representan o de su contenido.
Por ejemplo, una función como MenuLoop hace pensar en que en su interior tiene un loop (como un while o un for), cuando en realidad se llama así porque es llamada dentro del contexto de un loop. No debería llamarse así por el contexto desde donde se llama sino por lo que ejerce o ejecuta. Siguiendo este caso en particular, la función actualiza el estado del menú o hace el chequeo de input en el menú, por lo que debería llamarse UpdateMenu o CheckInputMenu o similar.
Notar también que hay una falta de consistencia en las siguientes funciones:
La del medio tiene la palabra Menu al principio y el resto al final.
El nombre del archivo, por otra parte, tal vez sería mejor que se llame MenuScreen o simplemente Menu, en vez de MenuLoop, para sugerir justamente que tiene funciones y variables relacionadas con esa pantalla.