Discussion :

[Freedolphin]

Bonjour à tous,

J'ai un code qui permet de faire une requête en fonction du nombre de mots présents dans une variable, séparés par des tirets.
J'isole donc d'abord chaque mot :

Code :

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

$mot=explode("-",$variable);

Puis je lance ma requête de cette façon :

for ($i=0;$i<=1;$i++)
{
$sql=mysql_query("select colonne from table where colonne='$mot[$i]'");
}

Ça fonctionne très bien quand il y a 2 mots mais pas pour 1 seul : il affiche toujours 2 résultats.
Je patauge un peu avec $i, quelqu'un pourrait-il m'aiguiller ?
Merci d'avance.

[stealth35]

utilise foreach pour parcourir ton tableau

[ABCIWEB]

Faits suivant le conseil de stealth35

Si tu devais utiliser "for" (mais encore une fois "foreach" est plus indiqué), tu pourrais faire quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
$mot = explode("-",$variable);
 
$nb_mots = count($mot);
 
for ($i=0; $i<$nb_mots; $i++)
{
echo $mot[$i].'<br />';
}

Remarque qu'avec cette méthode il n'est possible d'accéder aux éléments du tableaux qu'avec des indices numériques. Il faut par ailleurs que les indices soient parfaitement incrémentés et sans trou dans les index du tableau. Autant de contraintes et d'arguments qui font qu'il est bien plus simple et universel d'utiliser foreach.

[Freedolphin]

Merci à tous les deux.
Je vais essayer de réécrire mon code avec foreach.
Mais je retiens l'astuce de ABCIWEB, surtout que j'avais prévu le comptage du nombre de mots, du coup mettre $nb_mots dans le For est très malin.
Merci encore de votre aide !

[Bebel]

Pour le "for" ton erreur était sur la condition de fin, comme le montre le code de ABCIWEB

Mais il y a quand même des contres indications à l'utilisation d'un foreach.

Par exemple quand ton tableau contient des objets, niveau optimisation en mémoire ce n'est pas le top. ( mais on peut très bien passer par un array_keys + un foreach pour pallier ce problème )

Après l'avantage du for également c'est pouvoir jouer aussi sur les index.

Après un explode, il y a peu de chance d'avoir un problème sur les indices d'un tableau.

[Freedolphin]

Ça fonctionne en effet très bien avec Foreach, c'est plus simple, plus clair.
Le seul hic, c'est que je n'arrive pas à limiter les requêtes, je ne voulais que 3 requêtes, même si le tableau contient 5 mots-clés.
Savez-vous comment indiquer une limite ?

EDIT
J'ai trouvé la réponse à ma question en faisant :

Code :

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

1
2
3
4
5
6
$i = 0;
foreach ($tableau as $motcle)
{
if(++$i > 3) break;
**ma requete**
}

C'est assez propre ?

[Bebel]

Dans ce cas la, le plus simple, à mon avis, est de passer par un for

Code :

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

1
2
3
4
5
6
7
8
9
 
$mot = explode("-",$variable);
 
$nb_mots = count($mot);
 
for ($i=0; $i<$nb_mots && $i < 3; $i++)
{
echo $mot[$i].'<br />';
}

Sinon avec un foreach, tu peux utiliser break pour provoquer la sortie du foreach.

A toi de voir.

[Freedolphin]

Merci Bebel, du coup je vais rester sur Foreach avec un break comme indiqué plus haut.

[ABCIWEB]

J'ai trouvé la réponse à ma question en faisant :

Code :

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

1
2
3
4
5
6
$i = 0;
foreach ($tableau as $motcle)
{
if(++$i > 3) break;
**ma requete**
}

C'est assez propre ?

C'est une solution, mais si les index de ton tableau sont numériques et incrémentés proprement (sans trous) - ce qui sera le cas avec un explode - tu n'as pas besoin de $i : récupères simplement l'index dans le foreach :

Code :

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

1
2
3
4
5
foreach ($tableau as $key => $motcle)
{
if($key > 3) break;
**ma requete**
}

[stealth35]

on peut toujours limiter le explode et supprimer la dernière entrée

[ABCIWEB]

Oui ou utiliser array_slice sur le explode ce qui donnerait un code plus générique

Code :

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

1
2
3
4
5
6
7
8
9
10
$mot = explode("-",$variable);
 
$select = 3;
 
$tab_mots = array_slice($mot,0,$select);
 
foreach ($tab_mots as $mot_cle)
{
**ma requete**
}

[Freedolphin]

Merci à tous pour vos suggestions.
Ce forum est plus qu'une bouée de sauvetage, c'est un vrai rendez-vous de passionnés pour progresser.