Legacy Trash (I)

How'd y'all!
Me cayó un proyecto legacy en PHP para que estime el tiempo que me llevaría agregar un par de funcionalidades nuevas a la codebase actualmente en producción. Tl;dr: demasiado tiempo. El código tiene tanta deuda técnica que adecentarlo no será negocio para nadie. Creo que lo único productivo que se puede hacer con el código es analizarlo para encontrar malas prácticas y pensar en porque uno debe evitarlas. Esta es la primera entrada de una serie de entradas que tiene ese fin: desenmascarar malas prácticas tratando de mantenernos lo más calmos posibles para ver que podemos aprender. Ojalá ese aprendizaje no solo sea respecto a lo que hace enojar a un dev que toma un proyecto legacy sino que nos haga reflexionar un poquito.

Acción oculta


if ($accion=='oculta')
{
   shell_exec("rm -r /var/www/html/A/B/");
   mkdir("../B/");
}

¿Vieron eso? Un operador de comparación no-estricta (==) para comparar strings. Si $accion valiera 0 la condición sería true de cualquier manera. Para comparar strings uno debiera siempre usar strcmp() o un operador de comparación estricta (===). Para continuar supongamos que "oculta" es un buen valor para una acción. Entonces ¿qué más tenemos? que no solo es confuso usar paths absolutos y relativos de manera arbitraria sino que es tonto borrar una carpeta para volver a crearla inmediatamente. Está bien, entendemos que quería borrar el contenido de la carpeta, pero hubiera bastado un 


 shell_exec("rm /var/www/html/A/B/*");


Not invented here


function entero($num)
{
  $num=(int)($num)
  return $num;
}

Este fragmento de código no sólo se encuentra en un archivo "funciones.php" que posee todas las funciones usadas por todos los scripts de la aplicación, si no que hace exactamente lo mismo que la función floor() que es parte de la API estándar de PHP desde su versión 4.

Culto cargo
Cerca del 15% del código son comentarios cuyo único sentido de existencia es el miedo a la amnesia del desarrollador.

You give devolverDato() a bad name
Intentaré no perder el tiempo hablando de los nombres inauditos que tienen algunas variables. Pero para las funciones, ya no puedo resistirme. Hasta el momento los nombres de funciones candidatos son devolverDato() y devolverNum().

Complemento
Existen 2 scripts con nombres casi idénticos, la diferencia es que lo que está descomentado en uno está comentado en otro y viceversa. No aventuraré hipótesis porque ni siquiera puedo estar seguro de cuál es el que está funcionando actualmente.

Acoplamiento, después de tantos años
Tener código CSS en los atributos de los elementos HTML ya es un poco doloroso. Tenerlos este código CSS saliendo por un echo desde el script PHP es homicida.

Comentarios

Publicar un comentario

Entradas populares de este blog

Introduccion a x86 ASM - Registros

Imagen
Assembler no es un lenguaje. Es una familia de lenguajes dependientes de la arquitectura de los procesadores. Esta familia de lenguajes es fundamental en ingeniería inversa. Me fascina el concepto de ingeniería inversa aunque aún soy un lego casi total en la materia: esta guía también me sirve a mi. Por otra parte, demasiados desarrolladores no saben lo que es la ingeniería inversa y demasiados desarrolladores piensan en Assembler como si fuera un lenguaje de programación obsoleto. Esta serie de entradas, inspirada en esta otra Guía de la Universidad de Virginia pretende ser una guía introductoria al lenguaje assembler usado en los procesadores Intel x86. Como guía introductoria cubre solamente una pequeña porción (pero útil) de las directivas e instrucciones disponibles. Así como existen un lenguaje ensamblador para cada arquitectura, existen varios lenguajes para generar código máquina para procesadores x86. Esta guía utiliza el assembler de Microsoft Macro Assembler (MASM) , q...
Leer más

JS: Triángulo de Pascal

Hoy, un pequeño problema de código Javascript. El siguiente snippet pretende crear un triángulo de Pascal . Al constructor Triangle se le pasa el número de filas que se quiere generar para la solución. La función fillRows() llama a fillRow(numRow) una vez por cada fila que se quiere generar. La función fillRow(numRow) se encarga de poner los números que van en cada posición de cada fila. Para ello usa un Array llamado row donde pone auxiliarmente los valores antes de agregarlos a this.rows . Estarán de acuerdo conmigo si les digo que el console.log() comentado como log "B" debería devolver lo mismo que el de la línea 5 -log "A"-. Pero no. Los resultados de cada console.log para new Triangle(2).rows son: Log "B": [[1], [1, 1]]. Log "A": [[1], undefined]. Es decir, this.rows tiene los valores correctos dentro de la función fillRow(numRow) pero no fuera de ella. ¿Cuál es el problema?¿Es una cuestión de scopes?¿Debería cambiar la...
Leer más

Legacy Trash (II)

Segunda entrega de este paseo por una aplicación PHP sangrienta. Después de pasar los ojos sobre un montón de líneas de código cuya complejidad solo es justificable en caso de que el programador estuviera realmente preocupado por su estabilidad laboral y después de encontrar: -Una simpática variable llamada $cúmulo. -Un script cuyo título era el nombre y apellido del autor. -Un comentario /***SEGUIMOS***/ , que me tranquilizó porque pensé que ya no estábamos siguiendo... Encontré un par de cosas más interesantes. Dialecto simplificado function quitarAcentos($cad) { //40 lineas de código que no quiero discutir $arr['z']='s'; $arr['ce']='se'; $arr['ci']='si'; $arr['ge']='je'; $arr['gi']='ji'; $arr['v']='b'; $arr['ll']='y'; $arr[' h']=' '; $arr['x']=''; foreach ($arr as $i => $v) { $cad=str_replace($i,$v,$cad); } } ¿Vieron eso? a...
Leer más