Thread by @Manz

Siempre había escuchado quejas de la web de Renfe, y ayer, tras un tweet de @MarkVillacampa, me dio curiosidad por examinar por encima el código Javascript de la web. Lo que verás a continuación te sorprenderá (o no). Abro hilo.

Lo primero, el código JS está sin minificar/ofuscar. Un código JS no tiene porque estarlo, pero al menos sería interesante hacer "strip" de comentarios y eliminarlos. Hay muchos comentarios que pueden ser útiles para el desarrollador, pero no deberían estar en producción.

A lo largo del código Javascript, vemos comentarios de historial de cambios. Se están usando los comentarios como control de versiones. Git llora en una esquina.

Y algunos con bastante información...

Se hace bastante «Monkey Patching» sobre tipos primitivos, considerado muchas veces un antipatrón y mala práctica: stackoverflow.com/questions/5741…

Dead code. El código muerto hay que retirarlo, no mantenerlo.

... si no se puede convertir en "código zombie"

Sí, lo sabemos. La programación puede ser frustrante en algunos casos cuando no nos sale lo que queremos hacer...

Y a veces hay que cambiar cosas sobre la marcha...

Refactorizar siempre es una buena práctica...

const quitaesp = cadena => cadena.trimStart(); // Equivalente en ES10

Cuando el código se repite muchas veces, ya nos debería oler raro...

Nos podemos encontrar funciones que con el tiempo terminan por no hacer nada...

Algunas, incluso, van de frente...

Código spaguetti...

O incluso saber de que lenguaje proviene el programador a través de los comentarios...