Skip to content

SB - Feedback S4 #1

@sbondia

Description

@sbondia

General

  • Separación del código en distintos archivos según funcionalidad y buena estructura de carpetas.
  • El proyecto se entiende perfectamente sin comentarios, código limpio.
  • Buenas prácticas a la hora de validar posibles errores donde todos nos olvidamos, aunque en este caso casi todo son elementos HTML hardcodeados.

Funcionalidad

  • Al abrir la página no carga el primer chiste, ¿qué broma es esta?
  • Renderizado inconsistente, solo refresca la tabla de resultados cuando el último añadido ha sido puntuado.

Código

  • Doble declaración de weatherBox.
  • Uso inconsistente de innerHTML y textContent.
  • La API del tiempo es estática, una solución perezosa que devuelve una string de la temperatura en Alcobendas. Se están pasando headers vacíos.
  • Si no se puntúa el chiste, el valor score queda null, lo que puede provocar problemas.

Estética

  • Está feo.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions