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

[transgohan]

Au temps pour moi, ce n'est pas ce qu'indiquait la doc.

Depuis PHP 5.0.0, cette fonction peut aussi être utilisée avec quelques protocoles url. Lisez Liste des protocoles et des gestionnaires supportés pour connaître les protocoles supportant la famille de fonctionnalités de stat().

http://fr2.php.net/manual/fr/wrappers.php (liste dont il est question et comportant le protocole http)

La dernière phrase semble indiquer qu'il y a en effet des distinctions, mais rien sur la page est les sous-pages liées n'indique ces familles dont il est question.

[stealth35]

Au temps pour moi, ce n'est pas ce qu'indiquait la doc.


http://fr2.php.net/manual/fr/wrappers.php (liste dont il est question et comportant le protocole http)

La dernière phrase semble indiquer qu'il y a en effet des distinctions, mais rien sur la page est les sous-pages liées n'indique ces familles dont il est question.

la liste des protocole est commune, ils disent juste que ca marche avec les protocoles qui accepte stats, et http ne l'accepte pas

[transgohan]

A titre d'information, tu pourrais nous diriger vers la partie de la doc listant cette famille ?
Car en allant sur la page de la fonction stat on a le même charabia...

[stealth35]

A titre d'information, tu pourrais nous diriger vers la partie de la doc listant cette famille ?
Car en allant sur la page de la fonction stat on a le même charabia...

c’était ton lien
http://fr2.php.net/manual/fr/wrappers.php

ensuite tu regardes ce que propose http
http://fr2.php.net/manual/fr/wrappers.http.php

Support de la fonction stat() Non

[transgohan]

Ah oui d'accord ! Et moi qui en voyant le terme "famille" cherchait une liste de fonction...

[almoha]

Merci à tous pour ces remarques pertinentes qui m'ont permis de trouver la solution à ma problématique