Discussion :

[Equinoxe5]

Bonjour,
Je bute sur un petit problème tout con, mais je ne vois pas comment faire.
J'ai un formulaire dans lequel je peux cocher de multiples checkbox (Nombre variable).
Je récupère les valeurs des checkbox pour écrire ma requête et c'est là que ça coince.
Voici mon code

Formulaire

Code :

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

1
2
3
4
5
6
7
8
9
10
<table width='100%'>
        <?php
	$query="SELECT DISTINCT(Section) FROM `adherents` GROUP BY Section ORDER BY Section";
	$result = mysql_query($query) or die('Erreur SQL !'.$sql.''.mysql_error());
	while($row = mysql_fetch_array($result))
  		{
        	echo "<tr><td class='lister'>".$row['0']."</td><td width='25' align='center' class='lister'><input type='checkbox' name='section[]' id='section[]' value='".$row['0']."'></td></tr>\n";
        	}
        ?>
</table>

Recuperation des données et création de la requête

Code :

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

1
2
3
4
foreach($_POST['section'] as $section)
{
$req .= " Section='$section' OR";
}

et la requête avec une erreur

Code :

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

SELECT ID, Section, Nom, Prenom, Sexe, Date_naissance, Adresse, Code_postal, Ville, Tel_perso, Tel_famille, Montant, Mode_paiement, Paye, Caution, C_medical, Photo, Licence, Adresse_mail FROM adherents WHERE Section='JUNIOR F' OR Section='LOISIR COMPET' OR Section='LOISIR JEUDI' OR ORDER BY Nom ASC

Je la vois bien entendu mon erreur mais je ne vois pas comment enlever ce dernier OR
A moins que ma solution d'utiliser foreach soit mauvaise ?

Merci d'avance de votre aide

[Madfrix]

bonjour,

pourquoi ne pas faire un substring sorti de ton foreach ?

Code php :

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

1
2
 
$req = substr($req, 0, -2);

[hiul dragonfel]

essay de mettre ça après ton foreach:

Code :

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

1
2
 
$requete = substr($requete, 0, -2);

ça devrait enlever ton OR en trop


PS: je suis trop lent comme d'hab

[Equinoxe5]

C'est ce que j'avais pensé faire, je l'avais essayé et ça fonctionnait.
En fait je voulais surtout savoir s'il n'y avait pas une manière disons plus "jolie" de récuperer ces checkbox pour écrire la requête.

[Madfrix]

Si tu y tiens vraiment, tu peux faire quelque chose comme ca :

Code php :

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

1
2
3
4
5
6
7
8
9
10
 
while(isset($_POST['section'])){
	$req .= $_POST['section'][key($_POST['section'])];
 
	if(next($_POST['section']) !== false)
		$req .=  ',';
	else
		break;
 
}

[hiul dragonfel]

c'est vrai que la requète est moche.

tu peux la changer en ça :

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
 
$req = "
SELECT 
ID, 
Section, 
Nom, 
Prenom, 
Sexe, 
Date_naissance, 
Adresse, 
Code_postal, 
Ville, 
Tel_perso, 
Tel_famille, 
Montant,
Mode_paiement, 
Paye, 
Caution, 
C_medical, 
Photo, 
Licence, 
Adresse_mail 
FROM adherents 
WHERE Section in (";
 
foreach($_POST['section'] as $section)
{
   $req .= "'$section', ";
}
 
$req = substr($req, 0, -2).")";

ça simplifie la requête pour sql et c'est plus esthétique

l'avantage de substr() c'est que tu n'a pas à faire le traitement du if dans chaque bouclage.
en même temps c'est pas comme si ils allaient coche 100 000 cases

c'est comme tu veux tu choise