Discussion :

[almoha]

Bonjour,

J'utilise la classe "PHP Simple HTML DOM Parser" pour récupérer les informations d'un site distant (après avoir tapé un n° ISBN dans un formulaire) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
<?php
 
if(!empty($_POST) && !empty($_POST['isbn_search']))
 
{
 
$isbn_search = mysql_real_escape_string($_POST['isbn_search']);
 
require_once 'scripts/simplehtmldom_1_5/simple_html_dom.php';
 
$html = new simple_html_dom();
 
$html->load_file('http://www.site.com/livres/'.$isbn_search.'.html');
 
$titre= $html->find('h1',0)->plaintext;
 
echo'<input type="text" value="'.$titre.'"size="100"/>';
 
?>

Ce code fonctionne dès lors que la page chargée du site distant existe. Mais si la page n'existe pas (en l'occurrence si le site ne référence pas le livre recherché), j'ai les erreurs suivantes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
Warning: file_get_contents(http://www.site.com/livres/97822620.html) [function.file-get-contents]: failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in C:\wamp\www\scripts\simplehtmldom_1_5\simple_html_dom.php on line 850
 
Fatal error: Call to a member function find() on a non-object in C:\wamp\www\scripts\simplehtmldom_1_5\simple_html_dom.php on line 879

Pour éviter ces erreurs, je souhaiterais ne pas exécuter la récupération de la page (et afficher un message "livre non référencé") si la page appelé ($html->load_file('http://www.site.com/livres/'.$isbn_search.'.html') n'existe pas. Voyez-vous comment procéder ? Merci.

[rawsrc]

Bonsoir,

Pour faire simple, ce code devrait rouler :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
<?php
 
$page = @file_get_contents('http://www.dghfkrti.com/');
if (false === $page) {
   echo 'Livre non référencé';
}
?>

[transgohan]

Utilises plutôt file_exists() pour tester l'existence du fichier.
Cela ne sert à rien de surcharger la mémoire pour rien.

Le caractère @ devrait être à bannir de tout code... C'est l'art de jouer à la patate chaude, on se fout des erreurs et donc on peut répercuter une erreur sur tout le reste du script. Mais le plus vicieux c'est que vu qu'on affiche pas l'erreur et bien on ne le sait pas...
Gérer les erreurs n'est pas si compliqué.

[rawsrc]

Le caractère @ devrait être à bannir de tout code... C'est l'art de jouer à la patate chaude, on se fout des erreurs et donc on peut répercuter une erreur sur tout le reste du script. Mais le plus vicieux c'est que vu qu'on affiche pas l'erreur et bien on ne le sait pas...
Gérer les erreurs n'est pas si compliqué.

Je suis plutôt du même avis, mais vu que dans son cas il semble utiliser une lib qui ne gère pas les erreurs (en particulier celle en ligne 850), la rustine la plus simple était ce bout de code. Et puis tu ne te refiles pas toujours la patate chaude, si à chaque requête tu obtiens une livre non référencé, tu vas bien finir par te poser des questions, non ? Enfin, j'ai déjà vu le contraire.

[almoha]

Merci à tous les deux pour vos réponses. La méthode de rawsrc fonctionne. Je n'ai pas pu encore tester la méthode file_exists(). Mais je croyais que cette fonction ne marchait pas si le fichier testé était un fichier distant... Qu'en pensez-vous ? Merci.

[stealth35]

Merci à tous les deux pour vos réponses. La méthode de rawsrc fonctionne. Je n'ai pas pu encore tester la méthode file_exists(). Mais je croyais que cette fonction ne marchait pas si le fichier testé était un fichier distant... Qu'en pensez-vous ? Merci.

en effet file_exists ne marche pas,
pour éviter de prendre du temps et de la mémoire, de te soit de faire juste un fopen, soit de faire un get_headers