Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Excelente Trabajo! #3

Open
Edward-TL opened this issue Jun 9, 2021 · 0 comments
Open

Excelente Trabajo! #3

Edward-TL opened this issue Jun 9, 2021 · 0 comments

Comments

@Edward-TL
Copy link

Hemos realizado la revisión de tu repositorio y hemos encontrado los siguientes puntos de mejora :)

Si tienes alguna inquietud o deseas solicitar una nueva revisión puedes comunicarte con Ana Belisa por Slack y enviar por el formulario nuevamente tu repositorio.

El proyecto tiene un gran rendimiento, así como una interfaz amigable, al punto que (confieso), sin leer la documentación, pude entender de qué iba el proyecto, lo cual es excelente. En cuanto al código, se puede observar ciertas áreas de mejora:

  • La función cancelOrder (así como otras), podría reducirse usando funciones auxiliares, en la que se pasa la lista de objetos a las que se les aplicará el remove o el add. En pseudocódigo:
Por objeto en lista:
    objeto.remove(“hidden”)
  • De igual forma, el string que tiene el prepPanel (líneas 36 – 41) es casi idéntico como en la función startPrep (líneas 105 – 110). Podría volverse un string formateado, en el que solo se pasa el texto del

    . Ya sea “Preparing” o “Serving”.

  • Para el caso de los setTimeout (líneas 93-100), se ve que quieres que ocurra algo después de otra cosa, considero que podrías hacer uso de async await para el mejor manejo de encadenamientos, y así garantizar que se ejecutan en orden, sin tener que calcular los tiempos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant