Problema con try catch

[quike88]

Hola a todos,

Les comento mi problema, el asunto es que tengo un array con una gran cantidad de datos que quiero ingresar a una tabla en mi BD, el problema es que algunas filas del array me dan error, ya sea por tener un tipo de dato diferente al definido en la columna o porque no tiene datos y esa columna no acepta NULL.

Entonces se me ocurrio usar try/catch (que no había usado antes) para atrapar esos errores y evitar que el proceso se interrumpa, lo que hice es esto para probar si funcionaba:

Código PHP:

  foreach ($array() as $row) {
    try {
        if (!$this->db1->insert("tabla", $row)) {
            throw new Exception('error en fila '.$row['NUMROW']);
        } else {
            echo 'Fila '.$row['NUMROW'].' insertada correctamente <br />';    
        }
    } catch (Exception $e) {
        echo 'Excepción capturada: ',  $e->getMessage(), " <br />";    
    }
} 
   

Pero cuando llega a la fila que da problemas, me sale un error "Ocurrió un Error con la Base de Datos..." y se detiene el proceso.

Estoy usando mal try/catch? o es un problema con la manipulación del errores del CI? Espero que puedan darme un mano con este problema.

Gracias.

[maycolalvarez]

según tengo entendido CI es compatible con PHP POO versión 4 la cual no incluye excepciones, consulta la API de CI para ver que devuelve $this->db1->insert() en caso de error

[NUCKLEAR]

Podrias usar profiles http://codeigniter.com/user_guide/ge...profiling.html
Pero no entiendo la consulta, si te da error por que no coincide el tipo de campo con lo que le envias... O si no permite null y no le envias nada... Ya sabes donde esta el error, CI te deberia imprimir esos errores...
O lo mas rapido es activar los logs http://codeigniter.com/user_guide/general/errors.html Fijate al final..

Saludos.

[quike88]

Gracias por las respuestas. NUCKLEAR, efectivamente se donde esta el error y CI me lo imprime, pero lo que busco es que por mas que aparezca un error igual se siga ejecutando el bucle y complete los demás insert.

maycolalvarez, $this->db->insert() te devuelve TRUE en caso de que se ejecute correctamente la inserción y FALSE caso contrario.

Al parecer el problema esta en la clase de base de datos del CI, que te lanza el error antes de devolver FALSE. Si desactivo db_debug en config/database.php funciona de la forma que quiero pero no se si es lo adecuado o hay otra forma de hacer esto.

Saludos!

[maycolalvarez]

me imagino que dependerá de si CI implementa un entorno (environment) para desarrollo o producción, si estás desde producción el error no debe aparecer, en cambio debe producir un error genérico tipo HTTP 5xx, porque mostrar errores en producción es un grave fallo de seguridad.

como te indica @nucklear, para eso tenemos los log's, para verificar el error y corregirlo, como te indique anteriormente si el fallo es grave el FW ha de producir en producción un error fatal

[quike88]

Gracias por tu respuesta, como sugirió NUCKLEAR activé los log's para poder revisar ahí se hubo algún error en la carga de los datos.

Gracias por su ayuda.

Saludos!

[Smolky]

Hola. Un aporte:

Dices que tienes gran cantidad de datos y todo lo estás insertando en la misma tabla. ¿Cierto? Entiendo que de la forma que lo estás haciendo estás realizando una consulta por cada dato que deseas insertar lo cual puede llegar a hacer estragos en la base de datos.

La idea sería crear un nuevo array de datos con los datos preparados para la inserción (eliminados los NULL y los que no tienen ese valor)

Mira el ejemplo de $this->db->insert_batch(); en el siguiente enlace
http://codeigniter.com/user_guide/da...rd.html#insert

Por otra parte, y esto si que puede sonar un poco quisquilloso, estás usando excepciones para "controlar" el flujo del programa. lo cual no es una práctica muy recomendable.

¿No es más lógico hacer esto?

Código PHP:

  foreach ($array() as $row) 
{ 
  try 
  { 
    if (!$this->db1->insert("tabla", $row)) 
    { 
      echo 'Error en la fila...<br />' ;
    }
    else 
    { 
      echo 'Fila insertada correctamente...<br />' ; 
    } 
  }
} 
   

Un cordial saludo

[GatorV]

@Smolky,

¿Cuando usar excepciones para controlar el flujo del programa es no recomendable? De hecho es una de las mejores herramientas que tienes cuando estas programando en POO, porque puedes definir exactamente el flujo que requieres de tu programa, y controlar los errores.

La persona que te dijo que no es lo más recomendable, seguramente no tiene un background fuerte en POO, ya que todos los lenguajes que son 100% POO, usar precisamente excepciones para controlar el flujo del programa.

Es una de las "desventajas" de PHP, que las funciones nativas no generan excepciones que podamos controlar a nivel código.

Saludos.

[masterpuppet]

Me extraña la pregunta GatorV, como comenta Smolky utilizar excepciones para controlar el flujo es generalmente considerado una mala practica, pero esto no es algo nuevo, yo tengo esta idea desde que trabajaba con Java(hace muuuuucho tiempo).

No es lo mismo lanzar una excepción porque la conexión a la bbdd falla, a lanzar una excepción porque el login no es correcto, esto último es lo que se considera dañino y es como no se debería utilizar una excepción.

A ver que esto se puede poner interesante

[GatorV]

Yo lo veo de otra manera, y es que depende también como hagas la excepción, pero por ejemplo, el login, es algo sencillo:

Código PHP:

try {
      $myLoginService->login($user, $pass);
      $this->createLocalSession($user);
      $this->redirect('/panel');
} catch (LoginException $ex) {
      Logger::log($ex);
      switch ($ex->getCode()) {
      case 1: // not found
             $sErrorMessage = 'No se encontro el usuario';
             break;
      case 2: // bad password
             $sErrorMessage = 'Password incorrecto';
             break;
      }
} catch (SessionException ($ex) {
      if ($ex->getCode() == 5) { // Por ejemplo que el disco esta lleno y no hay espacio
           Alerts::dispatchAlert($ex, 'Error en disco');
           $sErrorMessage = 'Error en disco';
           $this->redirect('/');
      }
 
      Logger::log($ex);
      $sErrorMessage = 'Error al crear la sesión';
}
 

De esta forma tienes un mejor control sobre lo que el código puede generar, aunque claro puede esto llegar a muy malos habitos, la idea es controlar exactamente que excepcion esperas encontrar, ya que como dices muchos programadores de Java, tienden a solamente throw new Exception() y se acabo y es casi lo mismo que el Goto.

Es por eso que por ejemplo Zend define un tipo de excepción por cada paquete, para que puedas controlar específicamente que error esperas encontrar, y tengas una mejor forma de mostrar errores al usuario, logear, enviar alertas, etc. en lugar de presentar una pantalla en blanco cuando ocurre un error.

[masterpuppet]

Para mi son cosas diferentes, una cosa es el control granular de excepciones(algo con lo que estoy totalmente de acuerdo) y otra muy diferente el control del flujo del programa a traves de las exepciones, en el ejemplo que muestras, para mi LoginException no deberia ser una excepcion, que falle el login por que el usuario ha puesto mal los datos es algo normal y esperable, sin embargo que el disco este lleno y no se puedan guardar mas sesiones es un caso excepcional y como tal deberia lanzar la excepción(valga la redundancia) correspondiente(SessionException).

Saludos.

[GatorV]

Cierto, ahora que lo pienso más tienes razón en lo que dices, la excepción por la misma palabra debe de ser por casos excepcionales, y entrando más a detalle así debe de ser

[Smolky]

Iniciado por GatorV

Cierto, ahora que lo pienso más tienes razón en lo que dices, la excepción por la misma palabra debe de ser por casos excepcionales, y entrando más a detalle así debe de ser

Exacto, a eso mismo me refería. Sólo usarlas para esos casos excepcionales.