Discussion :

[michel71]

Bonjour,

Je cherche des avis pour améliorer la rapidité d'affichage du bloc central de ma page index

https://www.timbres-de-france.com/index.php

Toutes les images (sauf le timbre au hasard) sont incluses a partir du code ci-dessous (un "include" pour chaque série d'images)
Les index des images sont créés chaque nuit à minuit et chargés dans une table utilisée en suite pour la boucle (for($b=0;$b<$lg;$b++))

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
<?php
// vérification des données
if(isset($base_gene) && $base_gene<>'' && $base_gene<>'x')
{
//-------------- Affichge des derniers timbres émis -----------------------------------------------
// on teste si $compteur de vaut pas 0
if (count($lig_id_dernier)>1 )
{
echo'<FONT color="00CCFF"><b>Les dernières émissions de la Poste Française</b></font><p>&nbsp;</p>';
$chemin='x';
$largeur=0;
$lg=count($lig_id_dernier);
for($b=0;$b<$lg;$b++)
{
if($lig_id_dernier[$b]<>'' && $lig_id_dernier[$b]<>'*')
{
	$query = 'SELECT * FROM collec_0 Where id='.$lig_id_dernier[$b];
	$result = mysqli_query($base_gene, $query);
	while ($data = mysqli_fetch_assoc($result))
	{ 	
			// recherche du chemin de la photo
			if($data['chemin_photo']<>'x')
			{
			$tab=explode('/',$data['chemin_photo']);
			if(isset ($tab[1])) $chemin='image/'.$tab[1].'/'.trim($data['photo']);
			}
			// recherche de l'image
			if(file_exists($chemin))
			{
				$img_miniature=miniature_image($chemin);
				$dim=explode('width=',$img_miniature);
 
				$largeur=$largeur+$width+3; 
				//echo $largeur;
echo '<td align="center" ><a href="collection/pop.php?'.$data["desi"].'&ligne='.trim($data['id']),'"><img src='.$img_miniature.'/></a></td>';
echo '   ';
 
if($largeur>600) { echo '<tr>';	$largeur=0; }							
			}
 
	} // fin de  while ($data = mysql_fetch_array($req))
}// fin de if($lig_id_dernier[$b]<>'')	
} // fin de for($b=0;$b<$_SESSION['cp'];$b++)
 
} // fin de if (count($lig_id_dernier)>0)
 
//----------------------------------Fin d'affichage des données ---------------------------------------------						
} // fin de if(isset($base_gene) && $base_gene<>'' && $base_gene<>'x')
?>

[grunk]

Je suis pas certains que ton code soit fautif ici.

Quand on regarde les graphique de chargement , on voit bien que ce qui prend du temps c'est le téléchargement des images depuis le serveur :



Ce qui veux dire que :
- le serveur ne semble pas très rapide (qui est ton hébergeur ?)
- les images pourrait sans doute être optimisées (dommage de télécharger une image en 400*500 pour afficher une 40*50 par exemple)

[Invité]

Bonjour,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   $img_miniature=miniature_image($chemin);

Que fait cette fonction ?

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

$largeur=$largeur+$width+3;

Quel intérêt de "calculer" la largeur ??
Alors qu'on peut définir des dimensions en CSS pour les images.
Par ex. :

Code css :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
#colonne_gauche img { 
  max-width:50px;
  max-height:50px;
}

Il serait en effet judicieux d'avoir, pour chaque image, une "miniature" de 50 x 50 px maxi.
Voir : Fonctions de redimensionnement d'images

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

if($largeur>600) { echo '<tr>';	$largeur=0; }

Ce n'est pas très "responsive friendly"...
Tu devrais supprimer tout bonnement toute la structure en <table>, qui n'a pas lieu d'être ici.
Tu as des <td> et <tr> qui se promène un peu n'importe où (sans balise <table>)

=> CORRIGE ton code.


Je t'invite aussi à passer la page au validateur : https://validator.w3.org
Tu as notamment une balise <style> APRES </head>, alors qu'elle doit être placée AVANT.
Et aussi :

• align="center" est obsolète. Il faut le remplacer par du CSS : style="text-align:center;" ou margin:0 auto; (selon le contexte).
• <FONT color="00CCFF"> est aussi obsolète.
• ...

[michel71]

Bonjour,

J'ai bien pensé à stocker les images miniatures (elles sont créées par une fonction) mais je n'ai pas trouver
la solution pour les stocker temporairement (1 journée)
je peux les créer en même temps que je récupères tous les index, mais après je n'ai pas trouver comment les installer
dans un dossier "tempo" sur le serveur.
j'ai aussi pensé utiliser des objets qui seraient mes images mais là je n'ai encore regardé suffisamment la doc

Je vais regarder ce que tu me dis pour les balises et tous tes conseils en général

Bonne journée

[Invité]

Bonjour,

Pourquoi de façon "temporaire" ??

Il suffit de créer chaque miniature en même temps que tu télécharges l'image, non ?

A moins que tes images soient mises "manuellement" dans le dossier ?

[michel71]

Bonjour,
comme préconisé je suis en train de vérifier la page index avec Validator et j'ai cette erreur que je ne comprend pas

for attribute href on element a: Illegal character in query: space is not allowed.

Code HTML :

Sélectionner tout - Visualiser dans une fenêtre à part

 <a href="collection/pop.php? Carnet «LE NU dans l'art - sculptures - femmes et déesses» &ligne=9973">

La ligne complète est

Code HTML :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
<a href="collection/pop.php? Carnet «LE NU dans l'art - sculptures - femmes et déesses» &ligne=9973">
					<img src=image/tempo/2019-61.jpg></a>

Merci de m'éclairer

Bonne journée

[Invité]

Bonjour,

" Carnet «LE NU dans l'art - sculptures - femmes et déesses» " n'a rien à faire dans une URL !
D'autant que tu ne l'affectes même pas à un paramètre.


Néanmoins, si tu veux vraiment que ce titre soit dans l'URL, il faut d'abord le convertir avec urlencode().

[michel71]

Ok
c'est supprimé

Bonne soirée
Je ferme le post vu le boulot que j'ai pour mettre tout ça d'aplomb