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

Feedback entrega final #53

Open
fmachadopiriz opened this issue Jul 15, 2019 · 0 comments
Open

Feedback entrega final #53

fmachadopiriz opened this issue Jul 15, 2019 · 0 comments

Comments

@fmachadopiriz
Copy link

Falta el README en la raíz del repo pero está en una subcarpeta; parece que en algún momento bajaron todo el repo y lo volvieron a subir en otro lado.

El proyecto compila. La funcionalidad está muy completa. La presentación fue muy buena.

Bien el uso de milestones en el repo. Bien el uso de pull request en el repo. Bien el uso de issues en el repo. Valeria tiene significativamente menos actividad que el resto en el repo.

Los test son pocos pero están bien.

Tienen excepciones que no procesan: IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs

No están siempre bien elegidos los nombres de variables: la variable "e" debería ser por ejemplo "result" en IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs

No controlan posibles resultados null: currenUser puede ser null en void OnGet() en Controller\TechnicianLayout\MyProjects.cs

  Experto En desarrollo Incipiente Insuficiente
Aplicas patrones de diseño Bien      
Respetas principios de diseño Bien      
Conoces un catálogo de patrones y lo usas Bien      
Programación en C# Bien      
Excepciones   No siempre controlan bien    
Programa fácil de mantener, modificar, reusar   A veces mal nombres y formato    
Repo Bien      
DBC y tests   Pocos pero bien    

Nota: 90

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