Try - Catch - Excepciones

[shalom_dy]

Hola foro.. tengo una duda..

Resulta que ejecuto un OleDbSelectCommand.ExecuteReader() con una consulta sql que inserta datos en una tabla de una bd oracle.

Ocupé el try - Catch (lo he probado con varios tipos de exepciones) , pero no me resulta.
pues si inserto un registro que no existe, lo agrega sin problemas
si vuelvo a insertar el mismo registro ... no me envia ningun mensaje de error, ni siquiera se me cae la pagina

Si pongo el mismo sql y lo ejecuto en SQL Navigator me sale el error :[1]: (Error): ORA-00001: unique constraint (DWCO.ESVE_PK) violated

Cómo podre capturar esto ???

gracias desde ya.. ojala y me puedan ayudar.

[RootK]

Cual es el code que estás usando..??

Salu2

[shalom_dy]

El codigo:

Dim myReader As OleDbDataReader
Me.OleConn7.ConnectionString = Strconn
Me.OleDa.SelectCommand = Me.OleDbSelectCommand
Me.OleDbSelectCommand.CommandText = Strparam

Try
Me.OleDbSelectCommand.Connection = Me.OleConn7
myReader = OleDbSelectCommand.ExecuteReader()
Catch Exp As OleDbException
response.write("paso un error")
OleConn7.Close()
Return
End Try
myReader.Close()
Me.OleConn7.Close()

[RootK]

bueno para empezar te quiero recomendar que no cierres una conexion en un catch, podría quedarte así:

Código:

....
...
Try
      Me.OleDbSelectCommand.Connection = Me.OleConn7
      myReader = OleDbSelectCommand.ExecuteReader()
Catch ex As Exception
      Response.write("Error = " + ex.Message)
Finally
If OleConn7.State = ConnectionState.Open Then
        OleConn7.Close()
End Try
myReader.Close()

no quiere decir que asi debe ir, tambien depende de si estas usando aplicacion a capas, por lo que debes separar tu negocio y datos de presentacion y además no debes mostrar el error al usuario, lo debes direccionar a una página amigable donde le inform que ocurrio un error e internamente tu cachas ese error en algun tipo de log mediante el eventviewer, por base de datos, en tu txt, o como tu quieras... pero bueno, creo que me estoy desviando un poco del tema.. jeje y la verdad si estás empezando no quisiera confundirte...

mejor prueba el ejemplo que te mande y nos dices si pudiste cachar el error

cualquier cosa por aqui andaremos..

[shalom_dy]

rootk, gracias por contestar y dale no mas con los consejos que siempre son útiles.

Mira puse el mismo codigo.. incluyendo un end if
Dim myReader As OleDbDataReader
Me.OleConn7.ConnectionString = Strconn
Me.OleDa.SelectCommand = Me.OleDbSelectCommand
Me.OleDbSelectCommand.CommandText = Strparam

Try
Me.OleDbSelectCommand.Connection = Me.OleConn7
myReader = OleDbSelectCommand.ExecuteReader()
Catch ex As Exception
Response.Write("Error = " + ex.Message)
Finally
If OleConn7.State = ConnectionState.Open Then
OleConn7.Close()
End If
End Try
myReader.Close()


y me sale el error:
Referencia a objeto no establecida como instancia de un objeto.

[shalom_dy]

ahhh y marca el error en myReader.Close()

[RootK]

sorry.. se me fue un poco.. primero debes cerrar el objeto datareader antes de cerrar la conexion.. prueba ésto:

...
..
Finally

If Not myReader.IsClosed Then
myReader.Close()

If OleConn7.State = ConnectionState.Open Then
OleConn7.Close()
End If
End Try

[shalom_dy]

gracias .. lo vere y te cuento luego

[shalom_dy]

Hola RootK ahora si, no tengo problemas con myReader.
esta todo Ok...
Gracias !!

[freegirl]

Rootk, ¿entonces no sería muy correcto mostrar lo de:

ex.Message

?

Entonces que muestras un texto general para todos los errores?



saludos!

[peloteitor]

¿Comor?
ex.Message contiene el mensaje de la excepcion generada al producirse un error dentro del try-catch. Las excepciones son distintas dependiendo del error y, por tanto, el mensaje asociado también.

[freegirl]

Comor?
ex.Message contiene el mensaje de la excepcion generada al producirse un error dentro del try-catch. Las excepciones son distintas dependiendo del error y, por tanto, el mensaje asociado también.

Evidentemente, pero no me refiero a eso.

La idea es hacer un registro de errores internos y no mostrar los mensajes de error al usuario que si "system.exception line:900...." , "overflow...." etc etc, porque queda feo. O sea, lo que se pretende es enseñar un mensaje de error "amigable" cara al usuario y el error se capta internamente registrándose en un registro de errores.


saludos

[Legolas_Bilbao]

de todas formas, si estas trabajando con web, despues de comprobar que funciocna correctamente cambia todos los ex.message por un mensaje tipo, por temas de seguridad principalmente

venga un saludo

[RootK]

Iniciado por freegirl

Entonces que muestras un texto general para todos los errores?

Iniciado por RootK

lo debes direccionar a una página amigable donde le inform que ocurrio un error e internamente tu cachas ese error en algun tipo de log mediante el eventviewer, por base de datos, en tu txt, o como tu quieras...

Salu2

[freegirl]

Creo que cada uno habla de algo distinto jajaj Yo hablo de Windows form y creo que vosotros de Web Forms.

Para win forms:

Yo lo que hago es que cuando ocurre un error, es registrarlo en una bd exclusivamente para eso. Se van registrando todos los errores detalladamente.
Lo único que veo es que en Win Forms no puedes hacer lo mismo que en Web Forms, desde mi punto de vista claro. O sea, en Web veo bien redirigirlo a una página de error dónde se diga "ha ocurrido un error interno. COntacte con el administrador" (o el mensaje que sea). Pero en Win forms no vas a sacar todo el rato el mismo mensaje, no? Al menos a mi no me gusta la idea esa para una apliacación Win Forms, cosas de gustos.
Por lo cual, me refería a que si le sacais al usuario un 'ex.message', o bien, escribís un mensaje personalizado distinto cada vez? Alguien me entiende lo que quiero decir? jajaj


saludos!

[Williams M.]

Hola

Yo en lo personal, me inclino por un mensajito con flores, o dulcitos, es decir, un mensaje personalizado menos tecnico que el que puede dar un ex.message.

saludos