Discussion :

[joboy84]

Bonjour,

J'ai un soucis avec ma requête, qui me renvoie 2x les 3 résultats de la base de données.

Je suppose que c'est parce que j'ai une boucle dans une boucle...

Mais quel est l'alternative ?

Merci

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
 
$pseudo = 'natsu';
$statut = 'en_cours';
$type = 'amazon';
 
$stmt = $bdd->prepare('SELECT * FROM shop WHERE pseudo = :pseudo AND statut = :statut');
$stmt->bindValue(':pseudo', $pseudo, PDO::PARAM_STR);
$stmt->bindValue(':statut', $statut, PDO::PARAM_STR);
$stmt->execute();
 
$datas = $stmt->fetchAll(PDO::FETCH_ASSOC);
 
if(count($datas) > 0)
{
foreach ($datas as $data) 
  {
 
  if($data['type'] == 'amazon')
  { 
	  $stmt = $bdd->prepare('SELECT * FROM shop WHERE pseudo = :pseudo AND statut = :statut AND type = :type');
          $stmt->bindValue(':pseudo', $pseudo, PDO::PARAM_STR);
          $stmt->bindValue(':statut', $statut, PDO::PARAM_STR);
	  $stmt->bindValue(':type', $type, PDO::PARAM_STR);	
	  $stmt->execute();
 
	  foreach ($datas as $data) 
      {
	  $total = $data['montant'] * $data['quantite'];
 
                echo 'montant : '; echo $data['montant'];
                echo 'quantite : '; echo $data['quantite'];
                echo 'total'; echo $total;  echo'Points</br>';
	 }
  }
 }
}

[Invité]

Bonjour,

1- C'est simple : la 2ème requête (et donc 2ème boucle) est totalement inutile.
Le if suffit.

2- Je dirais même plus, il suffit de :

Code :

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

$stmt = $bdd->prepare('SELECT * FROM shop WHERE pseudo = :pseudo AND statut = :statut AND type = \'amazon\'');

3- De plus :

Code :

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

1
2
3
foreach ($datas as $data)
...
   foreach ($datas as $data)

Le minimum est de mettre des noms de variables différents !

[joboy84]

Merci pour ta solution et réponse et effectivement, j'aurai dû changer le nom des variables.