Variable unique pour plusieurs formulaires

Bonsoir,

Je viens vers vous car je bloque sur des formulaires.
J'ai un tableau généré lors de l'ouverture de la page avec les valeurs de ma BDD, donc avec un nombre de ligne variable. Pour chaque ligne, la dernière colonne contient un bouton "valider", et un bouton "Rejeter".
Admettons qu'il y ai 5 lignes à mon tableau. J'aimerai pouvoir valider/rejeter une des lignes en cliquant sur le bouton correspondant.

Voici le code pour construire le tableau :

Code:

---

1 2 3 4 5 6

while ($uneligne = mysqli_fetch_row($result)) { print("<form action=protection.php method=post>"); print("<TR><TD>".$uneligne[0]."</TD><TD>".$uneligne[1]."</TD><TD>".$uneligne[2]."</TD><TD><input name=ident type=text value=".$uneligne[3]."></TD><TD><input type=submit name=valid".$uneligne[3]." value=Valider>------<input type=submit name=reject".$uneligne[3]." value=Rejeter></TD></TR>"); print("</form"); }

---

Et plus loin une fois que j'ai cliqué sur un des boutons :

Code:

---

1 2 3

//validation ou rejet de l'inscription @$ident = $_POST[ident]; echo $ident

---

J'ai mis les".$uneligne[3]." dans les noms car j’espérai pouvoir récupérer ainsi le nom du bouton sur lequel j'ai cliqué. Mais je doute que ce soit la bonne méthode.
Pouvez-vous m'aider svp?

Tu oublies de mettre des guillemets sur tes attributs.
Tu peux utiliser une notation en tableau :

Code:

---

echo '<input type="submit" name="reject['.$uneligne[3].']" value="Rejeter">';

---

Code:

---

1 2 3

foreach ($_POST['reject'] as $key=>$value) { echo $key; }

---

Merci, mais ça ne fonctionne pas en l'état.
Mettre les guillemets est-il vraiment utile ? Vu que ma page se génère correctement avec mon code, et les bouton prennent bien les noms "valid102" et "reject102" par exemple.

Ensuite voici le message d'erreur obtenu pour le foreach :

Citation:

---

Warning: Invalid argument supplied for foreach() in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\Loto\scripts\protection.php on line 63

---

J'avoue que je n'ai pas bien compris l'utilisation du foreach. Il va parcourir chaque case du tableau ? Dans quel but ?

Citation:

---

Mettre les guillemets est-il vraiment utile ?

---

oui

Si tu fais

Code:

---

var_dump($_POST);

---

tu obtiens quoi ?

Voici ce que j'obtiens comme nom de bouton avec les guillemets :

Code:

---

name="reject['.104.']"

---

Alors qu'avec mon code initial j'obtiens pour la même ligne :

Code:

---

name="reject104"

---

Avec :

Code:

---

var_dump($_POST);

---

J'ai :

Citation:

---

.array(2) { ["ident"]=> string(3) "108" ["reject"]=> array(1) { ["'.104.'"]=> string(7) "Rejeter" } }

---

Tu as mal intégré mon code.
J'ai utilisé des guillemets simples pour le echo :

Code:

---

echo '<input type="submit" name="reject['.$uneligne[3].']" value="Rejeter">';

---

Bonjour à tous.
@TabbHonmin
Il me semble que tu n'utilises pas la bonne méthode. Pour chaque ligne du tableau, tu dois avoir deux formulaires. Un pour le bouton "Valider" et un autre pour le bouton "Rejete"

Code:

---

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

while ($uneligne = mysqli_fetch_row($result)) { print('<tr>'); print('<TD>' . $uneligne[0]. '</TD>'); print('<TD>' . $uneligne[1]. '</TD>'); print('<TD>' . $uneligne[2]. '</TD>');   /* Formulaire pour valider */ print('<TD>'; print('<form action="protection.php" method="post">'); print('<input type="hidden" name="do" value="valider"/>'); print('<input type="hidden" name="key" value="' . $uneligne[3]. '"/>'); print('<input type="submit" name="valider"/>'); print("</form"); print('</TD>');   /* Formulaire pour rejeter */ print('<TD>'; print('<form action="protection.php" method="post">'); print('<input type="hidden" name="do" value="rejeter"/>'); print('<input type="hidden" name="key" value="' . $uneligne[3]. '"/>'); print('<input type="submit" name="valider"/>'); print("</form"); print('</TD>'); print('</tr>'); }

---

Sur la page protection.php, il te suffit de tester la valeur de $_POST['do'] et selon sa valeur tu valideras ou tu rejeteras la demande d'identifiant $_POST['key'];

Non. Il est tout aussi correct d'avoir un seul formulaire avec deux boutons submit dont tu testes la valeur pour connaitre l'action souhaitée.

La solution au problème est celle décrite par sabotage.

@Bovino : Vous avez raison.
Néamoins il me semble que utiliser une methode comme

Code:

---

echo '<input type="submit" name="reject['.$uneligne[3].']" value="Rejeter">';

---

n'est pas très éfficace. Elle implique (si je ne me trompe pas) un traitement supplémentaire pour avoir l'id de la demande à traiter; alors qu'en utilisant un champ de formulaire caché ou encore les cookies (plus sécurisé) est mieux.

Merci pour vos réponse.
J'approche du but puisque le

Code:

---

var_dump($_POST);

---

me donne ceci :

Citation:

---

.array(2) { ["ident"]=> string(3) "108" ["reject"]=> array(1) { ["'.104.'"]=> string(7) "Rejeter" } }

---

Et les valeurs en rouges correspondent bien au bouton cliqué.

J'ai donc une question de débutant : Quel est le nom de la variable qu'il me faut tester pour la suite ?

Citation:

---

alors qu'en utilisant un champ de formulaire caché ou encore les cookies (plus sécurisé) est mieux

---

Je ne vois pas ce que viennent faire les cookies la dedans.
Un cookie n'est pas plus sécurisé qu'un formulaire ou que tout élément qui est géré par le navigateur.

Pour le côté HTML, l'écriture en tableau correspond plus au cas ou on veut transmettre plusieurs éléments.
Dans le cas présent, on pourrait avoir effectivement un champ caché mais un seul formulaire suffit.

Code:

---

echo '<input type="hidden" name="id" value="'. $uneligne[3] . '"><input type="submit" name="action" value="Accepter"><input type="submit" name="action" value="Rejeter">

---

Code:

---

1 2 3 4

echo $_POST['id']; if ($_POST['action'] == "Rejeter") { .... }

---

Merci encore pour votre patience. Je découvre le php depuis quelques jours seulement :P

La ça fonctionne et j'utilise donc le code ci-dessous pour traiter les rejet/validation :

Code:

---

1 2 3 4 5 6 7 8

foreach ($_POST['reject'] as $key=>$value) { echo $key; //la suite de mes requêtes pour rejeter } foreach ($_POST['valid'] as $key=>$value) { echo $key; //la suite de mes requêtes pour valider }

---

Par contre si je rejette, j'obtiens :

Citation:

---

Notice: Undefined index: valid in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\Loto\scripts\protection.php on line 86

Warning: Invalid argument supplied for foreach() in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\Loto\scripts\protection.php on line 86

---

Ceci correspond au foreach de la validation, et inversement si je valide, j'ai le même message d'erreur pour le foreach du rejet. Ca fait pas propre.

J'ai essayé de mettre chacun des foreach dans des if pour n'en exécuter qu'un, mais ça n'a rien changé...


EDIT : C'est bon j'avais juste oublié le isset.

Salut TabbHonmin,

Dit moi ce que tu en penses :

1) tu pourrait n'avoir q'une seule balise form :

Code:

---

1 2 3 4 5 6 7 8 9 10

print("<form action=protection.php method=post>"); while ($uneligne = mysqli_fetch_row($result)) {   print("<TR><TD>".$uneligne[0]."</TD><TD>".$uneligne[1]."</TD><TD>".$uneligne[2]."</TD><TD><input name=ident type=text value=".$uneligne[3]."></TD><TD><input type=submit name=valid".$uneligne[3]." value=Valider>------<input type=submit name=reject".$uneligne[3]." value=Rejeter></TD></TR>");   }   print("</form>");

---

2) Change tes input submit pour :

Code:

---

1 2 3

<input type=submit name=action[valid][".$uneligne[3]."] value=Valider> <input type=submit name='action[reject][".$uneligne[3]."]' value=Rejeter>

---

Comme tu peux voir:
a) J'ai utiliser une notation array, du fait qu'il ny a plus qune seule balise formulaire, il faut classer tout ca
b) Jai changer le name pour "action". Cest une mauvaise pratique de reposer une logique php sur la valeur de value dans un bouton submit, imagine que tu doit mettre ton site anglais un jour, en changeant la valeur de value, ton code qui reposait dessus va planter et tu devras le modifier
c) a la place jai incorporer dans le nom du submit "valid" et "reject"


3) Si une foi ton formulaire envoyer tu print la variable $_POST, tu auras ceci:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

Array ( [ident] => 3 [action] => Array ( [reject] => Array ( [3] => Rejeter )   )   )

---

4) Pour aller chercher les info que tu a besoins :

Code:

---

1 2 3

$action = current(array_keys($_POST['action'])); // ex : reject $ident = current(array_keys($_POST['action'][$action])); // ex : 55

---