Capturar excepcion no funciona

[neodani]

Buenas,

Estoy intentando captura esta excepción de si no encuentra la pagina web, pero no lo consigo, no sé que estoy haciendo mal.

Código PHP:

function ParserWebSimple($url,$expresion,$debug=false,$notifica=false){
    try {
        $web = file_get_contents($url);
    }catch (Exception $e) {
        echo 'ERROR: ',  $e->getMessage(), "\n";
    }
    preg_match_all($expresion,$web, $matches);
    $resultado=trim(strip_tags($matches[1][0]));
    
    return $resultado;
}
 
// MAIN
$url='http://www.alternate.es/html/product/Placas_base_Socket_1366/Asus/SABERTOOTH_X58/475921/?tn=HARDWARE&l1=Placas+base&l2=Intel&l3=Socket+1366';
$expresion='|<div id="price" style="position:relative;">(.*?)</div>|is';
echo ParserWebSimple($url,$expresion,$debug=false,$notifica=false);
 

Si lo ejecuto con una URL correcta funciona, muestra el resultado
177,90 €

En cambio si pongo una URL no correcta

Código PHP:

$url='http://www.alternate33.es/html/product/Placas_base_Socket_1366/Asus/SABERTOOTH_X58/475921/?tn=HARDWARE&l1=Placas+base&l2=Intel&l3=Socket+1366';
 

Me muestra el siguiente error por pantalla

Warning: file_get_contents() [function.file-get-contents]: php_network_getaddresses: getaddrinfo failed: Host desconocido. in C:\AppServ\www\Parseando_Webs\componentes_PC\compo nentes_PC.php on line 13

Warning: file_get_contents(http://www.alternate33.es/html/produ...l3=Socket+1366) [function.file-get-contents]: failed to open stream: php_network_getaddresses: getaddrinfo failed: Host desconocido. in C:\AppServ\www\Parseando_Webs\componentes_PC\compo nentes_PC.php on line 13

La linea 13 es precisamente
$web = file_get_contents($url);

El script no está capturando el error, sabéis qué hago mal?

Muchas gracias de antemano!

[Hidek1]

bueno pues pq file_get_contents no crea una exception en caso de fallar

www.php.net/file_get_contents

te recomiendo leer un poco mas sobre excepciones

www.php.net/exception

Código PHP:

  function foo()
{
    throw new Exception('capturame');
}

try {
    foo();
} catch (Exception $e) {
    echo $e->getMessage();
}  
   

saludos.

[neodani]

Sino crea un exepción como puedo saber si se ha ejecutado bien y que no produzca el warning que produce? hay alguna forma?

He probado tu ejemplo y eso sirve para crear excepciones, me falta el paso previo que es como detecto que la función no funciona bien (ejemplo en el caso que la URL que estoy capturando no es correcta por algún motivo)

Muchas gracias de antemano!

[Hidek1]

pues en el manual dice

Valores devueltos

Esta función devuelve la información leída o FALSE en caso de error.

If you want to check if the function returned error, in case of a HTTP request an, it's not sufficient to test it against false. It may happen the return for that HTTP request was empty. In this case it's better to check if the return value is a bool.

Código PHP:

  <?php 
$result=file_get_contents("http://www.example.com"); 
if ($result === false) 
{ 
    // treat error 
} else { 
    // handle good case 
} 
?>   

saludos.

[GatorV]

Si lees el Manual:

This function is similar to file(), except that file_get_contents() returns the file in a string, starting at the specified offset up to maxlen bytes. On failure, file_get_contents() will return FALSE.

Es decir si falla regresa false.

Para hacer una versión que soporte excepciones podrías hacer algo así:

Código PHP:

function getFileContents($sFile)
{
        $sContents = @file_get_contents($sFile);
        if ($sContents === false) {
               throw new Exception("Unable to read $sFile");
        }
 
        return $sContents;
}
 

Saludos.

[neodani]

Gracias a ambos, cuando puse la respuesta volví a releer el manual y vi lo que comentabais un poco después. Al final me ha quedado de la siguiente forma

Código PHP:

function getFileContents($sFile)
{
    $sContents = @file_get_contents($sFile);
    if ($sContents === false) {
           throw new Exception("Unable to read $sFile");
    }
    return $sContents;
}
 
function ParserWebSimple($url,$expresion,$debug=false,$notificaErrores=false){
 
    $error_msg=''; // string que recopila los errores que va encontrando
    
    try {
        getFileContents($url);
    }
    catch(Exception $e){
        print $e->getMessage();
    }
    preg_match_all($expresion,$web, $matches);
    $resultado=trim(strip_tags($matches[1][0]));
    if (empty($resultado)){
        $error_msg.='La captura está vacia';
    }
    // Si debug está activado imprime los echo dentro del programa
    // Si notificaErrores esta activado te imprime el string $error_msg
    
    return array($resultado,$error_msg);
}
 
// MAIN
$url='http://www.alternate33.es/html/product/Placas_base_Socket_1366/Asus/SABERTOOTH_X58/475921/?tn=HARDWARE&l1=Placas+base&l2=Intel&l3=Socket+1366';
$expresion='|<div id="price" style="position:relative;">(.*?)</div>|is';
list ($resultado,$error_msg)=ParserWebSimple($url,$expresion,$debug=false,$notificaErrores=false);
 
echo "$resultado<br/>";
echo $error_msg;
 

Qué diferencia hay entre poner el try/catch y luego hacer el
throw new Exception("Unable to read $sFile");

Que simplemente poner...

Código PHP:

function getFileContents($sFile)
{
    $sContents = @file_get_contents($sFile);
    if ($sContents === false) {
           echo "Unable to read $sFile";
    }
    return $sContents;
}
 

Que ventajas tiene...? porque ha efectos visuales no soy capaz de verlo.

Y porqué el IF tiene 3 === en lugar de solo dos? ($sContents === false)

Muchas gracias de antemano por vuestros consejos!

[GatorV]

Si lees lo que te dejo Hidek1, es para comparar el valor, y el tipo de valor del resultado, en el Manual puedes ver más información.

Respecto a tu segunda duda, si lo pones así no tiene ninguna ventaja, pero es para el control de errores, tal como lo tienes ahorita, no sirve de nada que uses o no uses excepciones, la idea de las excepciones es que floten hacía arriba en las llamadas para que puedas mostrar los errores a los usuarios de una mejor forma:

Código PHP:

try {
      $web = ParserWebSimple($url, $exp);
} catch (Exception $e) {
      log($e); // Escribe el error en un log para debug
      echo "Lo sentimos se produjo un error al procesar el request. El error es: " . $e->getMessage();
}
 

Saludos.

[neodani]

Entonces Gatorv

¿El código completo quedaría asi?

Código PHP:

function getFileContents($sFile)
{
    $sContents = @file_get_contents($sFile);
    if ($sContents === false) {
           throw new Exception("Unable to read $sFile");
           // En este momento todo el codigo que venga despues no le ejecuta.
    }
    return $sContents;
}
 
function ParserWebSimple($url,$expresion,$debug=false,$notificaErrores=false){
 
    $error_msg=''; // string que recopila los errores que va encontrando
    getFileContents($url);
    preg_match_all($expresion,$web, $matches);
    $resultado=trim(strip_tags($matches[1][0]));
    if (empty($resultado)){
        $error_msg.='La captura está vacia';
    }
    // Si debug está activado imprime los echo dentro del programa
    // Si notificaErrores esta activado te imprime el string $error_msg
    
    return array($resultado,$error_msg);
}
 
// ------------ MAIN ---------------------------------
$url='http://www.alternate33.es/html/product/Placas_base_Socket_1366/Asus/SABERTOOTH_X58/475921/?tn=HARDWARE&l1=Placas+base&l2=Intel&l3=Socket+1366';
$expresion='|<div id="price" style="position:relative;">(.*?)</div>|is';
 
try {
    $web = ParserWebSimple($url,$expresion,$debug=false,$notificaErrores=false);
    echo "$resultado<br/>";
    echo $error_msg;
} catch (Exception $e) {
      log($e); // Escribe el error en un log para debug
      echo "Lo sentimos se produjo un error al procesar el request. <br/>El error es: " . $e->getMessage();
}
 

Una pregunta más, supuestamente nunca podrán haber/existir dos excepciones juntas/seguidas no? Ya que cuando hay una excepción automaticamente el programa finaliza y salta al apartado del catch, correcto?

Muchas gracias

[GatorV]

Así es, y también dentro de tu función ParserWebSimple podrías lanzar otra excepción en caso de que este vacio, etc.

[neodani]

Iniciado por GatorV

Así es, y también dentro de tu función ParserWebSimple podrías lanzar otra excepción en caso de que este vacio, etc.

Y las excepciones no se escriben en log de error de PHP, tienes que forzar su escritura si quieres a través de la funcion error_log() como mencionas o crearte tu propio fichero de errores donde quieres que escriba cada excepcion, verdad?

Y una pregunta más

Se puede saber qué excepción fue la que saltó, para actuar de una manera u otra?
Si consigo que escriba en log de errores lo podré ver allí, pero si quiero saberlo en la propia ejecución del script, se les puede dar algún nombre o identificarlas de alguna manera?

Para que en el código principal del programa pueda saber si ha saltado porque no ha sido capaz de leer el fichero o porque la expresion regular ha devuelto vacio.

Código PHP:

try {
    $web = ParserWebSimple($url,$expresion,$debug=false,$notificaErrores=false);
    echo "$resultado<br/>";
    echo $error_msg;
} catch (Exception $e) {
     error_log($e); // Escribe el error en un log para debug
      echo "Lo sentimos se produjo un error al procesar el request. <br/>El error es: " . $e->getMessage();
}
 

Muchas gracias de antemano!

[GatorV]

Claro es lo padre de POO y de las excepciones puedes crear cuantas necesites y usarlas, por ejemplo:

Código PHP:

class File_Exception extends Exception {}
class Blank_Content_Exception extends Exception {}
 
try {
        // Codigo que peude lanzar File_Exception o Blank_Content_Exception
} catch (File_Exception $fe) {
       echo "No se encontro ese archivo";
} catch (Blank_Content_Exception $bce) {
       echo "No tenia contenido";
} catch (Exception $e) {
       echo "Cualquier otro error no contemplado";
}
 

La función log() no existe es solo un ejemplo de como podrías implementar tu el manejo de errores y escribir en un log la excepcion y todo lo que contiene, en la pagina de PHP puedes ver todos los métodos e información que tiene la clase Exception.

Aparte de que puedes hacer cosas como en tu función getFileContents lanzar una exception del tipo File_Exception y en ParserWebSimple solo cachar ese tipo de excepciones, y dejar que las demás suban.