MYGGL GOOGLE API CLASS FOR BEGINERZ

Commentaire de phpAnonyme le 23/03/2011 23:33:18

Bon n'ayant rien à faire je me suis replongé dans ton code et j'apporte des modifications parfois non négligeable car il y a avait des erreurs de conception notamment dans la fameuse boucle dont je parle dans mon précèdent poste !

### dans la méthode function initialise($pRequete) ###
function initialise($pRequete)
{
$this->oNombrePagesIndex = 0;
$this->oNombreTotalPages = 0;
$this->oNombreBackLinks = 0;
$this->oNombrePagesSite = 0;
preg_match('{^(.*\//)?(.*)\/?}', trim($pRequete, '/'), $out);
$this->oRequete = $out[2];
$this->oReferer = isset($_SERVER['HTTP_REFERER']) ? htmlspecialchars($_SERVER['HTTP_REFERER'], ENT_COMPAT, 'UTF-8') : FALSE;
}

/// Explication :
- $this->oUrl = ""; ne servait à rien puisque la visibilité de l'objet est privé
- $this->oReferer = 'http://Nom de votre site'; pas très intéressant, encore une fois méthode privé et puis taper dans la classe l'url de provenance pas trop 'génial'
- $this->oRequete = $pRequete; dans la méthode nombrePageIndexReference() tu l'utilise pour vérifier la visibilité de l'url hors celle-ci est uniquement du type www.xxx.com. Si l'utilisateur passe en paramètre une url complète http://... il ne sera donc pas pris en compte.


### Création d'une nouvelle méthode ###
function parseResult($url)
{
$this->oBody = false;
$this->oCh = curl_init();
curl_setopt($this->oCh, CURLOPT_CONNECTTIMEOUT, ini_get('max_execution_time'));
curl_setopt($this->oCh, CURLOPT_URL, $url);
curl_setopt($this->oCh, CURLOPT_SSL_VERIFYPEER, FALSE);
curl_setopt($this->oCh, CURLOPT_SSL_VERIFYHOST, 2);
curl_setopt($this->oCh, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($this->oCh, CURLOPT_REFERER, $this->oReferer);
$this->oBody = curl_exec($this->oCh);
if($this->oBody === false)
throw new Exception('Erreur Curl : ' . curl_error($this->oCh));

return $this->oBody;
}

// Explication
Tu utilise à trois reprises curl pour obtenir tes résultats. Inutile de mettre dans les 3 méthodes 3 fois cette opération.

[ A NOTER ] dans les trois méthodes suivantes :
- nombrePageIndexReference()
- nombreBackLinks()
- nombrePagesSite()
Il faut remplacer les opérations de curl par cette simple ligne :
$this->oBody = $this->parseResult($this->oUrl);   


### dans la méthode nombrePageIndexReference(), DANS la fameuse boucle ###  
for($i = 0; $i < sizeof($this->oJson['responseData']['results']); $i++)
{
if($this->oJson['responseData']['results'][$i]['visibleUrl'] == $this->oRequete)
$this->oNombrePagesIndex++;
}


// Explication
Je n'avais pas remarquer au premier abord(et donc dans mon précèdent poste) que tu bouclais sur le nombre de résultats/pages d'où le trop grand nombre pour 3000000 de résultats/pages. En faite il faut boucler sur le nombre de résultats effectifs/indexés.



Normalement maintenant ça devrait rouler !

Commentaire de phpAnonyme le 24/03/2011 02:31:10

Je t'ai pas dit que tu peux ni que tu dois ! Je t'ai dis que tu faisait la boucle sur le nombre de résulats estimés

## CODE ORIGINALE ##
# $this->oNombreTotalPages = $this->oJson['responseData']['cursor']['estimatedResultCount'];
# for($i = 0; $i < $this->oNombreTotalPages; $i++)
# {
# if($this->oJson['responseData']['results'][$i]['visibleUrl'] == $this->oRequete)
# $this->oNombrePagesIndex++;
# }


Hors s'il y en a 10000 ou 1000000 etc le script plantera !
D'où la correction que j'y est apporté :
for($i = 0; $i < sizeof($this->oJson['responseData']['results']); $i++)
{
if($this->oJson['responseData']['results'][$i]['visibleUrl'] == $this->oRequete)
$this->oNombrePagesIndex++;
}

Commentaire de LeFauve42 le 28/03/2011 12:39:39

Bonjour,

De quelle nouvelle API google parles-tu ?
L'API AJAX qui remplace l'ancienne version SOAP ?

Parce que si c'est le cas, il n'y a aucun interet a utiliser cette API concue pour JavaScript avec PHP (a part lancer des requetes "en batch" ou pour enregistrer les resultats sur ton serveur, deux choses que les conditions d'utilisation interdisent explicitement).

Eric

PS: Si c'est bien l'API Google AJAX, il y a au moins 2 ans qu'elle est nouvelle (il y a 2 ans que Google a discontinue l'API SOAP et conseille d'utiliser AJAX).

Commentaire de LeFauve42 le 28/03/2011 14:48:09

Akhenathon,

Le probleme est justement que la version SOAP de l'API n'existe plus depuis Aout 2009 (ainsi que ses conditions d'utilisation tres permissives).

Je ne dis pas que c'est inutile d'acceder en PHP a une API AJAX en general, mais dans le cas particulier de l'API Google, je pense que ca l'est :
Les CDU de l'API Google AJAX indiquent explicitement que tu ne peux faire une recherche que sur une action directe de l'utilisateur, et que tu ne dois pas alterer le resultat.
Partant de la, pourquoi passer par du PHP et forcer un reload de la page quand on peux simplement afficher les resultats dans une partie de la page ?

Eric