PHP

[resolu]imbrication de foreach

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [resolu]imbrication de foreach

hugo_064

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 :

$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 :

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

Message édité par hugo_064 le 16-11-2017 à 12:36:35

Publicité

pluj

Ce n'est franchement pas le code de l'année :

pas d'indentation donc c'est vraiment illisible
il y a des injections SQL
on peut réduire le nombre de requêtes avec IN(...) voire des UNION (ou autre suivant le SGBD)

Bref, déjà, je pense qu'il faudrait (ré)initialiser $organic_prod à chaîne vide en début du premier foreach. ($organic_prod = ''; après la ligne foreach ($_POST['idProduit'] as $key => $idProduit){)

Et pour la série de virgules où elles sont seules, c'est parce qu'une des deux requêtes ne renvoie rien ?

EDIT : il y a un truc qui ne va pas. Ce n'est pas plutôt name="pID['.$organic['organic_id'].']" au lieu de name="pID['.$organic_products['products_id'].']" (mais c'est probablement la même chose ?) et foreach ($_POST['pID'][$key] as $pID){ (ou foreach ($_POST['pID'][$idProduit] as $pID){ étant donné que $idProduit == $key - du moins en théorie) au lieu de foreach ($_POST['pID'] as $pID){ ? Parce que sinon tes paniers n'ont pas vraiment de corrélation avec les produits d'où les virgules qui traînent (tu parcours tous les produits indépendamment de leur panier donc quand tu tapes sur les produits d'un autre panier, ils n'existent pas et génèrent ces virgules d'autant qu'il n'y a pas non plus de test pour vérifier que tes fetch_array renvoient bien une ligne)

Code :

if(isset($_POST['idProduit']) && is_array($_POST['idProduit'])) {
    $sql = query('
        SELECT c.organic_id, c.sort_order, ci.organic_name, c.status, ci.languages_id
            FROM TABLE_ORGANIC c
            JOIN TABLE_ORGANIC_INFO ci USING(organic_id)
        WHERE c.status = 1
            AND ci.languages_id = 4
            AND c.organic_id IN(' . implode(', ', array_map('intval', $_POST['idProduit'])) . ')
    ');
    while ($recup = fetch_array($sql)) {
        if (isset($_POST['pID'][$recup['organic_id']]) && is_array($_POST['pID'][$recup['organic_id']])) {
            $sql2 = query('
                SELECT *
                    FROM TABLE_ORGANIC_PRODUCTS
                    WHERE organic_id = ' . $recup['organic_id'] . '
                        AND products_id = IN(' . implode(', ', array_map('intval', $_POST['pID'][$recup['organic_id']])) . ')
            ');
            $organic_prod = '';
            while ($recup2 = fetch_array($sql2)) {
                $organic_prod .= $recup2['products_id'] . ', ';
            }
            $organic_materials .= /*htmlspecialchars*/($recup['organic_name']) . ' x ' . intval($_POST['quantite'][$key]) . ' : ' . $organic_prod . ' </br> ';
        }
    }
}

Message édité par pluj le 15-11-2017 à 23:05:35

hugo_064

Bonjour,
Un grand merci à toi 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 voir complètement illisible et incompréhensible.

En rapide mais pas éfficace,
j'ai rajouté en début de requète :

Code :

$organic_prod = "";

qui me supprime les doublons.

ainsi que :

Code :

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

pour régler le problème des retours vide.

Merci de m'avoir ouvert les yeux.
Je vais tenter d'appliquer tes conseils et reprendre mon développement en me basant sur ton codage
et en faisant attention aux incohérences et aux injections sql.

En tout cas MERCI, ça m'a dépanner après m'être arraché les cheveux pendant 5 heures.

Hugo_064

FORUM HardWare.fr

Programmation

PHP

[resolu]imbrication de foreach

Sujets relatifs
probleme foreach php	Formulaire créé en javascript limité à 90 "lignes"
Problème avec boucle foreach powershell	problème avec une boucle foreach
[SQL/ACCESS] Requête Jointure + imbrication?	Problème d'imbrication de références et de tableaux de références
$_GET, foreach et array	impossible de rentrer dans mon foreach
Récuperer plusieurs variables $_POST dans un foreach	Problème imbrication d'objet Jquery IE
Plus de sujets relatifs à : [resolu]imbrication de foreach

Page générée en 0.104 secondes