exdevutem/mi-utem

Reformat de clases/widgets complejas/os

RafaelMoralesV opened this issue · 2 comments

Description

Hay un par de archivos que contienen codigo extraño, complejo, demasiado largo, y, como en estos momentos donde no hay muchas mejoras pensadas para la app, puede ser el mejor momento para intentar arreglarlos.

Tengo un draft de PR (#8) donde estoy arreglando warnings. No estoy seguro del todo si debería ser en ese espacio donde hacer este reformat, o si crear una branch nueva; planteo esto por un problema de merging que se pueda generar a futuro. De todas formas, creo que la gran mayoria de estos archivos son antiguos y que no se estan tocando mucho.

Tipo de cambio

  • Documentación.
  • Mejora de codigo existente (performance y/o estilo del codigo y/o otra mejora)

Resources

En primera medida, estos son los archivos que me preocupan:

Archivos complejos o largos

  • screens/asignaturas_nota_tab.dart
  • screens/login_screen.dart
  • widgets/acerca_dialog.dart
  • widgets/acerca_screen.dart

Archivos con codigo comentado

  • screens/asignaturas_resumen_tab.dart *
  • screens/avancemalla_screen.dart *
  • screens/boletin_screen.dart *
  • widgets/bloque_ramo_card.dart *
  • widgets/carrera_list_item.dart *

Cualquier archivo que no se haya mencionado, que no sepan si se esta utilizando, tambien vale la pena mencionarlo en la discusión. Lo ideal es limpiar lo más posible ahora, cosa de no tener que hacerlo dos veces.

Voy a estar aplicando estos arreglos en una branch que hice en mi repo :p

Esto se "avanzó" en #48, pero lo mantendré abierto hasta que tengamos completamente implementado clean architecture.