par
Cyrano » 22 mai 2007, 14:11
D'abord, on ne met pas d'accent sur une variable (
$erreurnontrouve et non
$erreurnontrouvé).
Ensuite, Où sont donc initialisées les variables
$erreurpasconnecte et
$erreurnontrouve 
Enfin, écrit un code lisible. Faire compact, c'est bien, mais trop, c'est comme pas assez :
<?php
function getdonnee($a)
{
$curl = curl_init("www.unsite.com/donne.php?donne=". $a);
curl_setopt($curl,CURLOPT_FOLLOWLOCATION,true);
curl_setopt($curl,CURLOPT_COOKIEFILE,'cookie.txt');
curl_setopt($curl,CURLOPT_RETURNTRANSFER,TRUE);
$page = curl_exec($curl);
if($page == $erreurpasconnecte)
{
connexion();
getdonnee($a);
}
elseif($page == $erreurnontrouve)
{
return(false);
}
else
{
return($page);
}
}
?>
D'abord, on ne met pas d'accent sur une variable ([i]$erreurnontrouve[/i] et non [i]$erreurnontrouv[color=red]é[/color][/i]).
Ensuite, Où sont donc initialisées les variables [i]$erreurpasconnecte[/i] et [i]$erreurnontrouve[/i] :?:
Enfin, écrit un code lisible. Faire compact, c'est bien, mais trop, c'est comme pas assez :
[php]<?php
function getdonnee($a)
{
$curl = curl_init("www.unsite.com/donne.php?donne=". $a);
curl_setopt($curl,CURLOPT_FOLLOWLOCATION,true);
curl_setopt($curl,CURLOPT_COOKIEFILE,'cookie.txt');
curl_setopt($curl,CURLOPT_RETURNTRANSFER,TRUE);
$page = curl_exec($curl);
if($page == $erreurpasconnecte)
{
connexion();
getdonnee($a);
}
elseif($page == $erreurnontrouve)
{
return(false);
}
else
{
return($page);
}
}
?>[/php]