Optimisation de code [Résolu]

[misakilou]

Bonjour j'essaye d'optimiser mon petit bout de code , c'est un script automatique saut qu'il est assez a faire le traitement et je vois pas comment l'optimiser

Code :

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
$cx = Connexion::getConnexion();
	$query = "SELECT id_community FROM community";
	$insert = $cx->prepare($query);
	$insert->execute();
	$result = $insert->fetchAll();
	//$community = new Community();
 
	foreach($result as $key){
		$json = json_decode(file_get_contents('http://graph.facebook.com/'.$key['id_community'])); // Recupere les likes 
		$likes = $json->{'likes'};
 
		$insert2 = $cx->prepare("INSERT INTO fan(community_id_community, fan_nb, fan_date) VALUES (:community_id, :fan_nb, :fan_date)");
		$insert2->bindParam(':community_id', $key['id_community']);
		$insert2->bindParam(':fan_nb', $likes);
		$insert2->bindParam(':fan_date',$date);
		if($insert2->execute()){
		}
	}

[zapbuzz]

Ton programme est difficilement optimisable.
Si il est long à s'exécuter c'est certainement du au temps de récupération de la page (c'est plus un problème réseau qu'applicatif). Si tu veux t'en assurer tu peux placer un compteur de temps avant et après le file_get_contents et regarder le pourcentage de temps passer dans cette fonction par rapport au temps total d'exécution.


---------------

http://www.zap-buzz.com - le programme télé TNT du soir classé par rapport à l'intérêt des internautes.

[stealth35]

fait plutot :

Code :

foreach($insert as $key)

et suprime le fetchAll,
je plus long dans l'histoire des le file_get_contents, y'a peu être moyenne de récupérer plusieurs id

[misakilou]

pas très bien compris stealth35 ^^

[stealth35]

Envoyé par misakilou

pas très bien compris stealth35 ^^

PDOStatement est un Traverable, donc pas besoin de faire un fetchAll si c'est pour la parcourir juste après

[misakilou]

Woowww je suis impressionné la... je pensais pour récupérer des éléments d'un tableau il fallait faire un fetchAll , mais j'ai toujours du mal a comprendre que le fait de parcourir le $insert fait la même chose qu'un fetchAll

[stealth35]

Envoyé par misakilou

Woowww je suis impressionné la... je pensais pour récupérer des éléments d'un tableau il fallait faire un fetchAll , mais j'ai toujours du mal a comprendre que le fait de parcourir le $insert fait la même chose qu'un fetchAll

parce que c'est un Traversable
, donc PDOStatement peux être parcouru via foreach comme un tableau, en interne à chaque tour de boucle un fait appelle au fetch()

[misakilou]

je dormirai moins bête ce soir
Merci !

[stealth35]

je crois que y'a moyen de récupérer plusieurs résultats en même temps avec le FQL