Cita de: Lekim en 12 Noviembre 2015, 11:43 AMTambién vale?
No, no me vale. Estás haciendo uso de málas prácticas, desaprovechando las ventajas del lenguaje utilizando métodos deprecados para llevar a cabo las cosas.
1. Usa la directiva Usings, esto aporta la ventaja de evitar bloques Try/Catch innecesarios, y mayor seguridad en general.
2. Usa IsNot Nothing en lugar de Not IsNothing()
3. Usa DirectCast para asumir de forma implícita el casting sin evaluaciones innecesarias, en lugar de CType, es algo innecesario en tu código y consume más tiempo, ya que ambos trabajan de manera distinta.
4. En tu bloque Try/Catch, estás lanzando una nueva excepción, no la excepción capturada en la expresión Catch.
Incorrecto:
Código (vbnet) [Seleccionar]
Catch ex As Exception
Throw New ExceptionNo del todo correcto:
Código (vbnet) [Seleccionar]
Catch
ThrowTampoco del todo correcto:
Código (vbnet) [Seleccionar]
Catch ex As Exception
Throw exCorrecto:
Código (vbnet) [Seleccionar]
Catch ex As Exception
ThrowTambién correcto:
Código (vbnet) [Seleccionar]
Catch ex As Exception
Throw New Exception("Mensaje personalizado", innerException:=ex)Saludos

.