Ya que estamos, yo también me sumo a la proclama!!!
Por todos los medios posibles evitar la siguiente estructura de código:
Try
...
...
Catch ex As Exception
Throw(ex)
...
End try
Esto es lo mismo que no capturar el error, en lugar de hacer esto no hagan nada!
Se van a ahorrar tiempo de procesamiento :) Si no, asegurense de hacer los
rollbacks o cancelar la operación antes de la línea "Throw(ex)".
Feliz fin de semana!
Damián Herrera
_____
From: [email protected] [mailto:[EMAIL PROTECTED] On Behalf Of Diego Jancic
Sent: Friday, September 07, 2007 5:44 PM
To: [email protected]
Subject: [puntonet] Proclama para salvar instancias
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
