Forum d'entraide PHPFrance

Bonjour à tous,
Je viens vers vous pour solliciter vos compétences suite à un problème rencontré.

Je suis en train de créer un formulaire pour réserver un/des paniers de bouffe avec choix du contenu.

Pour le formulaire de choix :

panier 1 (input checkbox pour sélectionner ce panier)
quantité (input number pour ce panier)
liste des produits dans ce panier (input checkbox pour sélectionner les produits)

panier 2 (input checkbox pour sélectionner ce panier)
quantité (input number pour ce panier)
liste des produits dans ce panier (input checkbox pour sélectionner les produits)

panier 3 (input checkbox pour sélectionner ce panier)
quantité (input number pour ce panier)
liste des produits dans ce panier (input checkbox pour sélectionner les produits)

Code : Tout sélectionner

    $organics_query = query("select c.organic_id, c.sort_order, ci.organic_name, c.status, ci.languages_id
                                            from TABLE_ORGANIC  c, TABLE_ORGANIC_INFO ci
											where c.organic_id = ci.organic_id
											and c.status = '1'
											and ci.languages_id = '4'
                                          ORDER BY c.sort_order, ci.organic_name");
    while ($organic = fetch_array($organics_query)) {

echo '<div style=" border-width:1px; border-style:solid; border-color:#000; vertical-align:top; text-align:center; display:inline-block; width:250px; margin: 10px; padding:10px;" >';

// choix du panier
echo '	<div style=" height:300px; vertical-align:top; text-align:center;">'
.'<strong>'.TEXT_SELECT_ORGANIC.'</strong><input type="checkbox" name="idProduit['.$organic['organic_id'].']" value="'.$organic['organic_id'].'">'.$organic['organic_name']
.'<br>'.
'</div>';

// choix des produits dans le panier
echo '	<div style=" vertical-align:top; text-align:left;"><strong>'.TEXT_SELECT_INSIDE.'</strong><br>';
      $organic_products_query = query("select *
                                            from TABLE_ORGANIC_PRODUCTS  
											where organic_id = '" . $organic['organic_id'] . "'
                                          ORDER BY organic_products_id");
      while($organic_products = fetch_array($organic_products_query)){
		  echo '<input type="checkbox" name="pID['.$organic_products['products_id'].']" value="'.$organic_products['products_id'].'" checked="true">'.$organic_products['products_id'].'<br>';
	  }
echo '</div>';

// choix de la quantité
echo '	<br><div style=" vertical-align:top; text-align:left;"><strong>'.TEXT_SELECT_QTY.'</strong>
	<input type="number" min="1" max="99" style="width: 7em;" autocomplete="off" name="quantite['.$organic['organic_id'].']" id="'.$organic['organic_id'].'" value=""/>
	</div>
 </div>';
	}

et pour envoyer tout ça dans le contenu d'un mail :

Code : Tout sélectionner


if(isset($_POST['idProduit'])){
foreach ($_POST['idProduit'] as $key => $idProduit){

            $sql = query("select c.organic_id, c.sort_order, ci.organic_name, c.status, ci.languages_id
                                            from TABLE_ORGANIC c, TABLE_ORGANIC_INFO ci
											where c.organic_id = ci.organic_id
											and c.status = '1'
											and ci.languages_id = '4'
											and c.organic_id =".$idProduit."
                                          ");
			$recup = fetch_array($sql);

foreach ($_POST['pID'] as $pID){
      $sql2 = query("select *
                                            from TABLE_ORGANIC_PRODUCTS
											where organic_id = '".$idProduit."'
											and products_id = '" . $pID . "'
											");
      $recup2 = fetch_array($sql2);
		$organic_prod .= $recup2['products_id'].', ';
}

		$organic_materials .= $recup['organic_name'].' x '.$_POST['quantite'][$key].' : '.$organic_prod.' </br> ';
}
}
	
$contenu_email = $organic_materials;

Je souhaite obtenir cela :
"nom du panier" x "quantité" : "liste des id de produits"

Le résultat obtenu avec ma formule :
Panier n°2 x 1 : , , , 4363, 4365, 4404, , ,
Panier n°3 x 1 : , , , 4363, 4365, 4404, , , , , , , , , 1239, 1943, 536

Le résultat souhaité :
Panier n°2 x 1 : 4363, 4365, 4404
Panier n°3 x 1 : 1239, 1943, 536

Le problème vient de mon imbrication de foreach.
Pouvez vous me dire ce qui ne va pas dans ma formule ou m'orienter vers une autre solution.

Merci à vous de prendre le temps de regarder.
Bonne fin de journée

Hugo_064

il faut que tu remplisse ta variable $organic_prod que si tu as quelque chose à y rentrer.

$organic_prod .= (empty($recup2['products_id']) ? '' : $recup2['products_id'].', ');

ainsi si $recup['products_id'] est vide, tu ne rempli $organic_prod que d'un caractère vide.

En remarque générale, as tu une idée du nombre de requète que fait ton code ? sachant que c'est bien souvent ce qui prends le plus de temps dans l'éxécution, je te conseillerai de faire juste 2 requète au début pour avoir des tableaux de correspondance id -> nom
je suis pas sur n'ayant pas tes données de base, mais la deuxième requète me semble inutile tu a le Pid venant du formulaire et tu récupère le product_id de la requète qui doit être égale à Pid.

une dernière chose, il est recommandé de ne récupèrer de ta requète que ce que tu vas utiliser ensuite donc d'éviter le * ou la liste des champs exhaustive

Bonjour,
Un grand merci à toi spols d'avoir pris le temps de regarder ma requete.

N'ayant jamais eu de formation au php, je voyais où était le problème mais je m'y perdais.
Je te dits bravo car mon code est un peu illisible .

Avec ton indication :

Code : Tout sélectionner

$organic_prod .= (empty($recup2['products_id']) ? '' : $recup2['products_id'].', ');

J'ai résolu la partie du problème des retours vide.

J'ai rajouté en début de requète :

Code : Tout sélectionner

$organic_prod = "";

qui me supprime les doublons.

Concernant le nombre de requète et la 2eme requète qui semble bien inutile, je te comprends et l'idée des tableaux de correspondance me semble judicieuse.
Je vais retravailer cela.
Ma requète restera courte car il n'y aura que 3 paniers et 5 articles par panier.

Pour l'* dans mon select, je comprend la démarche mais dans mon cas je n'ai que 3 champs dans ma table et je me sert des 3 champs c'est pour ça que j'ai mis l'*.

En tout cas MERCI, ça m'a dépanner après m'être arraché les cheveux pendant 5 heures.
Si un jour je peux te rendre un service, même si mes compétences sont limitées, ce sera avec plaisir.

Hugo_064

Forum d'entraide PHPFrance

[RESOLU] imbrication de deux foreach

imbrication de deux foreach

Re: imbrication de deux foreach

Re: imbrication de deux foreach