Mala práctica 4

La sección de malas prácticas de programación, la había dejado abandonada desde hace algún tiempo.

Hoy la retomo con un fragmento de código muy simple, que habla de consistencia.

Como podéis ver, es un PHP que se encarga de ir concatenando un string, con el contenido del mensaje que se mandará por email.

La mala práctica está en la falta de coherencia del código. En ese caso, a veces el operador de concatenación (.), va precedidos de espacios, y en otros casos no.

Puede parecer algo tonto, pero ello dificulta la lectura, y lo que es más importante, puede introducir problemas al efectuar búsquedas y reemplazos masivos.

Para el caso que nos ocupa, no hay excusas que valgan en cuanto a prisas, o presión en el desarrollo. El tiempo necesario para hacerlo en todos los sitios igual, es exactamente el mismo (¡o incluso menor!), que hacerlo de formas diferentes. Es puramente una cuestión de hábito, y buenas maneras.



5 comentarios en “Mala práctica 4”

  1. mmm, yo no programo en PHP, no sé si habrá un IDE que te ayude en ese sentido, es cierto q los estilos de programar los decidimos nosotros, pero hay IDE's que te ayudan mucho a la hora de escribir código.

  2. Javier Gutiérrez Chamorro (Guti)

    Jorge, llevas razón en que un buen IDE ayuda en estas tareas. Pero en mi opinión, son labores puramente correctivas.

    Es decir: si el programador no es capaz de escribir el código de forma aparentemente coherente, ¿qué garantías hay de que la lógica lo sea?

    Sobre IDE PHP, el que más me gusta es el phpEd de Nusphere, de pago, pero muy potente. No tiene opciones de reformatear código en ese sentido, pero a cambio incluye un depurador, un perfilador, y autocompletar. Tienes un review algo antigua que hice de él aquí.

    Otra opción que tienes es el Delphi for PHP 2007 que acaba de salir, aunque a ese estoy esperando tener algo de tiempo libre para echarle un vistazo.

Deja un comentario