Discussion :

[FRDarkRod]

Bonjour,

J'ai fais un petit script pour lire une API mais le code est lent (disons que quand je fais F5 , il met environ 30 sec à se finir) , si quelqu'un à des astuces , je suis preneur

Voilà le code :

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
21
22
23
24
25
26
27
28
29
30
31
	$test = "https://api.worldoftanks.eu/wgn/clans/info/?application_id=8c12c289eb5203d45131fa79e5ef9f58&clan_id=500143851&language=fr";
	$test2 = "https://api.worldoftanks.eu/wgn/clans/info/?application_id=8c12c289eb5203d45131fa79e5ef9f58&clan_id=500152502&language=fr";
 
 
	$data = file_get_contents($test);
	$data = json_decode($data,TRUE);
 
	$data2 = file_get_contents($test2);
	$data2 = json_decode($data2,TRUE);
 
	for ($i = 0; $i <= $data['data']['500143851']['members_count']; $i++) {
		echo $data['data']['500143851']['members'][$i]['account_name'];
 
		$valid = $data['data']['500143851']['members'][$i]['account_id'];
		$test3 = "https://api.worldoftanks.eu/wot/account/tanks/?application_id=8c12c289eb5203d45131fa79e5ef9f58&account_id=" . $valid . "&language=fr";
 
		$data3 = file_get_contents($test3);
		$data3 = json_decode($data3,TRUE);
		//var_dump($data3);
 
		$length = count($data3['data'][$valid]);
 
		for ($j = 0; $j < $length; $j++)
		{
			if ($data3['data'][$valid][$j]['tank_id'] == "3937") // ID Type 5
			{
				echo " + <b>Type 5 possédé</b>";
			}
		}
		echo "<br/>";
	}

Merci , amicalement

[rawsrc]

Salut, trace d'abord les deux file_get_contents avec microtime

[FRDarkRod]

J'ai pas compris.

Après quelques essais , le premier $data , passe bien , mais avec le $data3 , ça devient lent , mais j'ai pas trouvé d'autres solutions (pour l'instant)

[MaitrePylos]

Bonjour,

Votre boucle fais appel à 100 fois à l'API.
Chaque appel prend entre 0.150 et 0.300 secondes.
Donc votre script tourne entre 15 et 30 secondes et je suis dans un excellent réseau.

Vous devriez essayer Guzzle ou curl pour voir cela améliore.

[FRDarkRod]

Je ne connais pas très bien Guzzle ou Curl , auriez vous quelques pistes pour que je puisse m'y pencher dessus plus facilement ? Merci

[grunk]

Avant de vouloir de changer la méthode de requetage (qui influencera peu voir pas les performance) il faudrait commencer par savoir ce qui est vraiment long.

- Le traitement PHP ?
- L'envoi de la requête ?
- La réponse de l'API ?

Si c'est l'API qui est longue à répondre , tu ne pourras pas y faire grand chose si ce n'est paralléliser les appels. Voir mettre en cache les résultats et ne les exécuter que périodiquement via un cron

[MaitrePylos]

Avant de vouloir de changer la méthode de requetage (qui influencera peu voir pas les performance) il faudrait commencer par savoir ce qui est vraiment long.

- Le traitement PHP ?
- L'envoi de la requête ?
- La réponse de l'API ?

Si c'est l'API qui est longue à répondre , tu ne pourras pas y faire grand chose si ce n'est paralléliser les appels.

Clairement c'est les 100 requetes en boucles qui prennent du temps.

Pour info, comme ceci j'arrive entre 20 et 23 secondes en moyennes

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
21
22
23
24
25
26
27
 
<?php
require_once 'vendor/autoload.php';
 
$client = new GuzzleHttp\Client();
$res = $client->request('GET','https://api.worldoftanks.eu/wgn/clans/info/?application_id=8c12c289eb5203d45131fa79e5ef9f58&clan_id=500143851&language=fr');
 
	$data = json_decode($res->getBody(),true);
 
	for ($i = 0; $i < $data['data']['500143851']['members_count']; $i++) {
		echo $data['data']['500143851']['members'][$i]['account_name'];
 
		$valid = $data['data']['500143851']['members'][$i]['account_id'];
 
		$res = $client->request('GET',"https://api.worldoftanks.eu/wot/account/tanks/?application_id=8c12c289eb5203d45131fa79e5ef9f58&account_id=" . $valid . "&language=fr");
 
		$data3 = json_decode($res->getBody(),true);
 
		$tank_id = $data3['data'][$valid];
 
		foreach($tank_id as $value){
			if($value['tank_id'] == "3937"){
				echo " + <b>Type 5 possédé</b>";
			}
		}
		echo "<br/>";
	}

[grunk]

Guzzle est basé sur curl. Selon les environnements et/ou certains cas spécifiques Curl peut être plus lent. Autant donc s'en passer dans un premier temps et isoler réellement la source de la lenteur.

Si c'est l'API qui est lente changer le client http va pas changer grand chose.
Si c'est la quantité de requête qui fait que c'est lent , il faut paralléliser pour réellement gagner du temps.