Hola gente!,

 

Estoy modificando un programa que hicieron muchos programadores (incluidos
semi-seniors y seniors), y me canse de ver algunas cosas sin sentido.

Por ese motivo estoy realizando esta proclama general, para que si alguien
lo hace, lo evite la proxima vez. ;-)

 

Cuando tenemos un codigo de este estilo:

 

MiClase instancia = new MiClase();

 

instancia = UnServicio.ObtenerDesdeBD( … );

 

En ese caso, se estan creando 2 objetos cuando se deberia crear uno… Fijense
que la 2da linea sobrescribe completamente la primera!!

Y para agravar el problema, el metodo UnServicio.ObtenerDesdeBD  hace algo
con el parecido pero llamando a la base de datos o a un WS o a donde sea.

Entonces se termina creando una instancia por capa, cuando en realidad se
necesita una sola (o ninguna si el metodo devuelve null)

 

Espero haberles ahorrado aunque sea 1 segundo de procesamiento en todo el
tiempo de vida de su aplicación.

 

Bueno, esto fue una pequeña contribución a todos ellos que por costumbre
repiten estas cosas desde hace mucho tiempo (ojo, hace bastante yo tambien
lo hacia!).

 

Saludos y Happy Programming!,

Diego

Responder a