Discussion :

[Valadone]

Bonjour,

Je souhaite faire une checkbox suivi d'une validation par un bouton et l'envoie et l'insertion en BDD ou l'envoie et la modification en BDD.

Le problème c'est qu'avec mon code, il y a une insertion de ligne en continue malgré que je veuille simplement modifie la valeur en BDD du checkbox en fonction qu'il soit coché ou non.
Et j'ai également un problème sur l'affichage de la checkbox déjà coché lorsqu'en BDD la valeur est true.

Je vous montre mon code. Le soucis est dans la partie Newsletter.

Code PHP :

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
 
// ------------------------------ Abonnement newsletter ---------------
 
if(isset($_POST['abonner']) && $_SESSION['utilisateur']['id_membre'] != 1)
	{
		$valeur_checkbox = $_POST['checkbox_news'];
 
		$recup_abo = $mysqli->query("SELECT checkbox FROM newsletter WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
		$recherche_abo = $recup_abo->fetch_assoc();
		$bdd_abo = $recherche_abo['checkbox'];
 
			if($valeur_checkbox == 'true'){
				$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='$_POST[checkbox_news]' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
				executeRequete("REPLACE INTO newsletter (id_newsletter,id_membre,checkbox) VALUES ('',' ". $_SESSION['utilisateur']['id_membre'] ." ','$_POST[checkbox_news]')");
			}
			if($valeur_checkbox == 'false'){
				$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='$_POST[checkbox_news]' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
			}
	}

La valeur de l'id_newsletter est vide car je veux qu'il s'auto incrémente. Mais ça doit poser problème pour la modification ( désabonnement ).

Et la validation de celui-ci :

Code HTML :

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

1
2
3
4
5
6
7
8
9
10
11
 
	<div id="newsletter">
		<h2>S'abonner à la newsletter</h2>
			<form method="POST" action="">
 
				<label for="resaisir_mdp">Souhaitez-vous recevoir des informations sur les promotions et les nouveautés du site par mail ? </label>
				<input type="checkbox" onchange="if(this.checked) this.value='true';else this.value='false';" value="false" name="checkbox_news">
 
				<input type="submit" name="abonner" value="Valider">
			</form> 		
	</div>

Je souhaite faire varier l'affichage de ce bouton. Si valeur = false => non coché, sinon la case sera déjà coché et pouvoir décoché cette case pour qu'en BDD la valeur change en false est que l'envoie de la newsletter ne se fasse pas.

[Bovino]

Le problème c'est qu'avec mon code, il y a une insertion de ligne en continue malgré que je veuille simplement modifie la valeur en BDD du checkbox en fonction qu'il soit coché ou non.

Je pense qu'il va falloir que tu expliques un peu mieux, parce que là, c'est assez incompréhensible.

A part ça, ton fonctionnement n'est pas terrible.

Code html :

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

<input type="checkbox" onchange="if(this.checked) this.value='true';else this.value='false';" value="false" name="checkbox_news">

Tu sembles oublier (ou ne pas savoir) que si la case n'est pas cochée, elle n'est pas transmise avec le formulaire, donc tes manips JavaScript, c'est joli, mais particulièrement inutile car tu ne peux jamais recevoir la valeur 'false' côté serveur.
Le corollaire de ça, c'est qu'ensuite

Code :

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

$valeur_checkbox = $_POST['checkbox_news'];

provoquera une erreur (qu'à priori tu n'affiches pas ) si la case n'a pas été cochée.

[Valadone]

Y'a un tout qui ne fonctionne clairement pas ensemble, je le comprends.

Mais étant vraiment novice dans le PHP et le JavaScript, je ne vois pas comment résoudre le problème.

Je souhaite simplement que lorsque l'utilisateur coche la case, en BDD la valeur true s'affiche ce qui permettra par la suite d'envoyer à tous les utilisateur où cette valeur est true d'envoyer la newsletter.
Et si le membre veut se désinscrire, la case sera déjà coché, il aura juste à la décoché et à valider pour qu'en BDD, cette valeur soit false et l'envoie de la newsletter à celui-ci impossible.

Au départ j'étais parti sur un select avec des options oui et non mais c'est pas beau visuellement.

[Celira]

Pour expliciter ce qu'a dit Bovino, une checkbox n'est envoyée que si elle est cochée. Donc plutot que de modifier la valeur avec un onchange et ensuite tester la valeur, il vaudrait tester directement si une valeur arrive ou pas :

Code html :

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

<input type="checkbox" value="true" name="checkbox_news">

Code :

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

1
2
3
4
5
6
if(!empty($_POST['checkbox_news'])){
	$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='$_POST[checkbox_news]' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
	executeRequete("REPLACE INTO newsletter (id_newsletter,id_membre,checkbox) VALUES ('',' ". $_SESSION['utilisateur']['id_membre'] ." ','true')");
} else {
	$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='false' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
}

[Valadone]

Donc avec ce code là, si la checkbox n'est pas coché, il enverra ' ' est donc passera dans le else ce qui lui donnera la valeur false est me permettra de désabonner le membre ?

[Bovino]

Perso, j'aurais plutôt vu un truc du style

Code :

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

1
2
3
4
5
6
7
if(isset($_POST['checkbox_news'])){
	$check = 'true';
} else {
	$check = 'false';
}
$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='$check' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
executeRequete("REPLACE INTO newsletter (id_newsletter,id_membre,checkbox) VALUES ('',' ". $_SESSION['utilisateur']['id_membre'] ." ','$check')");

Parce que :
• si la seule présence d'une valeur suffit à dire que la case a été cochée, inutile de présager que la valeur n'a pas été modifiée ;
• inutile d'écrire 2 fois la même requête, ça facilitera d'éventuelles maintenances ;
• si l'on peut se désinscrire de la newsletter, il doit aussi être possible de se réinscrire.

EDIT :
et en simplifiant encore un peu :

Code :

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

1
2
3
$check = isset($_POST['checkbox_news']) ? 'true' :  'false';  // ou $check = empty($_POST['checkbox_news']) ? 'false' :  'true';
$modif_abo_bdd = executeRequete("UPDATE newsletter SET checkbox='$check' WHERE id_membre=' " . $_SESSION['utilisateur']['id_membre'] . " ' ");
executeRequete("REPLACE INTO newsletter (id_newsletter,id_membre,checkbox) VALUES ('',' ". $_SESSION['utilisateur']['id_membre'] ." ','$check')");

[Valadone]

Merci, je vais tester tout ça.

EDIT:

Ca fonctionne mais j'ai toujours ce problème d'ajout en BDD à chaque fois que l'utilisateur valide son choix.



La condition de je dois mettre c'est un < 1 ou != 1 pour qu'il regarde si dans la BDD la ligne de l'id_membre est déjà présente et qu'il modifie cette ligne et n'en rajoute pas ?

[Celira]

Je ne vois pas d'INSERT dans le code que tu as donné. C'est donc d'un autre morceau de ton script qu'il est question ?

[Valadone]

Le REPLACE INTO jouant la fonction d'INSERT ou UPDATE je me suis dit qu'il suffirait

[Bovino]

Ton champ id_membre n'est pas une clé primaire ?
Cela devrait et tu devras faire INSERT INTO ... ON DUPLICATE KEY UPDATE ...

[Valadone]

Non mon id_membre est une clef étrangère.
La clef primaire c'est l'id_newsletter dans cette table.

[Bovino]

Ca correspond à quoi id_newsletter ?
Tu as plusieurs newsletters mais chaque newsletter ne peut être envoyée qu'à un seul membre ?

[Celira]

Ca correspond à quoi id_newsletter ?

Vu la tête de la clé, je dirais un id auto-incrémenté...
Ce ne serait pas plus simple d'avoir une colonne "newsletter" dans ta table des utilisateurs ?

[Bovino]

Oui, c'est bien ce que je pensais aussi... je vois pas trop l'utilité de cette table...

[Valadone]

C'est exact. A vrai dire c'est dans le cahier des charges qu'on nous a donné. Table newsletter avec clef primaire id_newsletter et clef étrangère id_membre.
Je pense que ça ne dérangera pas si je modifie la table membre pour que ce soit plus simple à faire.

[Bovino]

A moins de rater une information importante, ça n'a à priori aucun sens...
Par exemple, dans ta table membre, tu as une clé primaire (enfin, j'espère ) id_membre parce que tu as plusieurs membres distincts, mais là, tu n'as (enfin, je suppose) qu'une seule newsletter et plusieurs membres peuvent s'y abonner, tu n'as pas plusieurs newsletters avec pour chacune un seul membre pouvant s'y abonner, donc la structure de la table ne correspond à rien...

[Celira]

Dans certains cas, avoir une clé primaire auto-incrémentée qui sert de clé technique peut être utile lorsque ta table a une palanquée de colonnes et que la vraie clé fonctionnelle est une combinaison de plusieurs colonnes. Mais c'est à utiliser dans des cas limites (en combinaison avec un index d'unicité pour la clé fonctionnelle)

Là, pour 2 colonnes...

[Valadone]

C'est peut être moi qui n'est pas compris l'utilité de cette table.

Mais ça serait 100 fois moins pénible à faire l'abonnement à la newsletter en mettant une colonne en plus dans la table membre.

Je vais tenter cette technique là. Je vais demander à quoi sert cette table exactement.

Merci =)

[Bovino]

Je ne conteste pas, mais là, la contrainte d'unicité est clairement sur le champ id_membre.
Du coup, la seule info pertinente est le champ checkbox et la sortir de la table membre et lui donner une valeur autre que booléenne me semble relever plus de l'erreur de conception qu'autre chose...

Mais bon, il doit certainement y avoir des infos que l'on a pas qui justifient ce choix.

[Valadone]

J'ai une petite question, pas en rapport avec le sujet de la salle mais je ne veux pas flood le forum de mes sujets.

Je souhaite alterner les couleurs des lignes pour que mon tableau soit lisible bien plus facilement.

Les tutos que j'ai trouvé sont tous fait avec la boucle for et j'utilise que des foreach et des while...

Donc est-il possible d'alterner la couleur des lignes sur ce code là ?

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
 
	$resultat = executeRequete("SELECT * FROM produit");
	echo "Nombre de produit(s) : " . $resultat->num_rows;
 
	$nbcol = $resultat->field_count;
	echo "<table style='border-color:red' border=10> <tr>";
	for ($i=0; $i < $nbcol; $i++)
	{    
		$colonne = $resultat->fetch_field(); 
		echo '<th>' . $colonne->name . '</th>';
	}
	echo "<th>Modification</th>";
	echo "<th>Supression</th>";
	echo "</tr>";
 
	while ($ligne = $resultat->fetch_assoc())
  {  
		echo '<tr>';
		foreach ($ligne as $indice => $information)
		{
				echo "<td>" . $information . "</td>";
		}
		echo '<td><a href="?action=modification&id_produit=' . $ligne['id_produit'] .'"><img src="'. RACINE_SITE . 'photo/modif.png" alt="" title="Modifier la fiche du produit"></a></td>';
		echo '<td><a href="?action=suppression&id_produit=' . $ligne['id_produit'] .'" OnClick="return(confirm(\'En êtes vous certain ?\'));"><img src="'. RACINE_SITE . 'photo/suppr.png" alt="" title="Supprimer ce produit"></a></td>';
		echo '</tr>';
	}
	echo '</table>';