Discussion :

[diablotine]

Bonjour,
je me permets d'ouvrir un post car j'ai un peu de mal à régler un petit soucis sur mon developpement de site.

Je ne suis pas codeuse en PHP mais j'essaie au maximum de ma débrouiller seule.

MON PROBLEME:
Je souhaite faire tourner au hasard une série de mots clés sur des alt d'images

Voilà mon script:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$nb_mc=5;
$kw_tmcs[1]="mot1";
$kw_tmcs[2]="mot2";
$kw_tmcs[3]="mot3";
$kw_tmcs[4]="mot4";
$kw_tmcs[5]="mot5";
$kw_m2=mt_rand(1,$nb_mc); 
$kw_mc=$kw_tmcs[$kw_m2];
 
$html .= $this->image_tag($db->f("product_thumb_image"), "alt=\"".$kw_mc." ".$db->f("product_name")."\"", 0);

pourriez vous s'il vous plaît me dire si mon script est correct ?

J'ai vu dans mes recherches qu'il serait possible de faire en sorte que pour un "product_name" partiiculier et défini, on pourrait faire en sorte d'assigner un mot clé particulier.
Apparemment la fonction utilisée serait un IF mais je n'arrive pas à la mettre en pratique.

Je vous remercie d'avance de votre aide,
Nathalie

[julien.63]

salut,
A priori ton script doit fonctionner dans le sens où il va bien prendre aléatoirement une valeur dans ton tableau.
tu peux utiliser mt_rand qui est plus rapide que rand (cf http://no2.php.net/manual/fr/function.mt-rand.php).

aussi, un petit mot pour te dire qu'en principe, on met le code entre des balises. Il te suffit de cliquer sur le bouton # qui se trouve dans le menu de l'éditeur quand tu rédiges ton post. ça facilite la lecture

[diablotine]

Merci beaucoup, y compris pour la fonction mt_rand.

J'ai également mis le code dans des balises dans le 1er message.

N'aurais tu pas également une idée pour la deuxième partie de ma question, à savoir utiliser un IF. Là j'ai beau chercher j'arrive pas du tout à voir comment mettre en syntaxe.

Je vois bien potentiellement qu'il faut faire un truc comme ça mais j'arrive pas du tout à le mettre en forme et à l'appeler proprement pour qu'ensuite ce soit pris en compte:

Code :

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

if($db->f("product_name")=="SPECIAL") { $kw_mc="MOT-SPECIAL"; $kw1_i=$kw1_i-1; }

Ou quelque chose du genre, mais là je sèche complètement.
Merci,
Nathalie

[julien.63]

ton code parait également correct.
Est ce que tu as pu le tester? As-tu un message d'erreur?
Es-tu sûr que $db->f("product_name") te renvoie une chaine de caractère?
Pour tester, tu peux faire un simple

Code :

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

echo $db->f("product_name");

tu peux simplifier ça

Code :

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

$kw1_i=$kw1_i-1

en ceci

Code :

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

$kw1_i--;

[diablotine]

Bonsoir Julien,

je te remercie beaucoup, ca marche effectivement parfaitement.

Ca s'affiche correctement si je paramètre un product_name pour y afficher un texte spécial, du type MOT-SPECIAL, le alt="" va alors afficher:

MOT-SPECIAL + product_name : En 1 mot => PARFAIT.

Voici mon code final, est il propre ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$nb_mc=5;
 
$kw_tmcs[1]="MOT1";
$kw_tmcs[2]="MOT2";
$kw_tmcs[3]="MOT3";
$kw_tmcs[4]="MOT4";
$kw_tmcs[5]="MOT5";
$kw_m2=mt_rand(1,$nb_mc); 
$kw_mc=$kw_tmcs[$kw_m2];
 
if($db->f("product_name")=="PRODUIT-1") { $kw_mc="MOT-SPECIAL-1"; $kw1_i--; }
if($db->f("product_name")=="PRODUIT-2") { $kw_mc="MOT-SPECIAL-2"; $kw1_i--; }
 
$html .= $this->image_tag($db->f("product_thumb_image"), "alt=\"".$kw_mc." ".$db->f("product_name")."\"", 0);

Merci encore de ton aide précieuse,
Nathalie

[julien.63]

Merci encore de ton aide précieuse,

de rien, ton code fonctionnait très bien sans mon aide

Juste une petite remarque pour te dire qu'en principe les tableaux commencent à l'indice 0. Il serait plus correct d'écrire :

Code :

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

1
2
3
4
5
$kw_tmcs[0]="MOT1";
$kw_tmcs[1]="MOT2";
$kw_tmcs[2]="MOT3";
$kw_tmcs[3]="MOT4";
$kw_tmcs[4]="MOT5";

on pourrait également écrire ceci

Code :

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

'alt="'.$kw_mc.' '.$db->f("product_name").'"'

qui doit être un peu plus rapide et à mon sens plus clair (on évite les \).

Mais c'est un peu du chipotage...

[diablotine]

C'est absolument parfait, j'ai remis la syntaxe de ton alt qui effectivement est plus rapide.

Te serait il possible en qq mots m'expliquer pourquoi on gagne en rapidité entre les 2 syntaxes de code ?
notamment pourquoi l'utilisation des [\] ralentit le truc ?

Merci
Nathalie

[julien.63]

salut,
en fait, ce n'est pas les \ qui ralentissent.
l'interpréteur PHP ne traite pas de la même façon les " et les '.
* une chaine entre ' est prise telle quelle. PHP ne s'occupe pas de son contenu.
* une chaine entre " est parsée. C'est à dire que l'interpréteur examine le contenu et s'il trouve quelque chose qui commencent par $ alors il cherche la variable correspondante et affiche son contenu. Ce qui prend plus de temps.

voici un lien qui rentre plus dans les détails http://www.vulgarisation-informatiqu...imiser-php.php

si tu veux tu peux chercher sur google "php optimisation", tu trouveras pleins d'infos. Encore une fois, c'est des petits détails qui font une différence notable sur des gros scripts.

J'ai un petit site où je mets en ligne des trucs dont je veux me rappeler.
ici j'ai fait un résumé de quelques optimisations de code. Tu es libre de t'en servir si tu veux.
http://kz.objectis.net/php/divers/op...n-du-code-php/.