Erreur case à cocher en php-mysql Résolu/Fermé

Question

Bonjour, je fais actuellement un formulaire en php-mysql. J'ai un problème au niveau de mes case à cocher c'est-à-dire lorsque je clique sur enregistrer (envoi des données à la base de données) une erreur s'affiche telle que : Notice: Undefined variable: contrat in C:\Documents and Settings\Administrateur\Mes documents\site essai raitement.php on line 93 Cette erreur s'affiche pour mes cinq cases à cocher donc en espèrant que vous puissiez m'aider je vous laisse une partie de mon code. Code du formulaire : Code du traitement.php : Merci d'avance pour votre aide.

lauraM · Accepted Answer

La modification de type et de valeur de champ s'est effectué correctement.
J'ai donc adapté mon code pour les cases à cocher et tout fonctionne à merveille : 

if (isset($_POST['contrat']) == 'contrat') { 
$contrat='oui'; 
} else { 
$contrat=''; 
} 
if (isset($_POST['nc']) == 'non-conformite') { 
$nc='oui'; 
} else { 
$nc=''; 
} 
if (isset($_POST['hm']) == 'hors maintenance') { 
$hm='oui'; 
} else { 
$hm=''; 
} 
if (isset($_POST['comptabilise']) == 'comptabilise') { 
$comptabilise='oui'; 
} else { 
$comptabilise=''; 
} 
if (isset($_POST['facture']) == 'facture') { 
$facture='oui'; 
} else { 
$facture=''; 
}

MERCI A TOUS !!!!!!!

lauraM · Answer

Il n'y a pas une personne qui pourrait m'aider svp.
Ce que je dois faire fait parti du projet pour mon stage.

Sri Lumpa · Answer

Non, ta variable Contrat n'est simplement pas initialisée parce que ton formulaire ne transmet aucune variable dont le nom est "Contrat". Je m'explique. Ton formulaire a quatre boutons : "NC", "HM", "Comptabilisé" et "Facturé". La valeur de chacun de ces boutons sera stockée dans $_POST['nom_de_ta_variable'] lors de la validation du formulaire par l'utilisateur. Le truc, c'est que comme tu n'as aucun champs dans ton formulaire que tu as appelé "Contrat", le champs $_POST['Contrat'] est vide et par conséquent, ta variable $contrat est vide aussi.

Au passage, il vaut mieux éviter en effet de nommer des variables avec des caractères spéciaux (é, è, à etc...) et, en général, on te dira que c'est plus "propre" de commencer le nom de tes variables par une lettre minuscule.

lauraM · Answer

Tout d'abord je tiens à vous remercier de votre aide.
Ensuite j'ai oublié d'indiquer dans mon code formulaire de mon premier Post que j'avais déclaré ma case à coher "Contrat".
Ayant enlevée les majuscules et les accents j'avais toujours le même problème.
Ensuite, en réfléchissant bien, je me suis dit (gràce au post de "Sri Lumpa") que j'avais assigné une valeur à ma variable que si elle était cochée et qu'il fallait donc que je lui assigne Null si ma case n'était pas cochée.
Après avoir fait cette modification mes erreurs obtenues précédemment ont disparus.
Mais malheureusement une autre erreur est apparue à la place et là j'avoue ne pas savoir pourquoi.
La voici :

Exécution de la requête impossible sql5 :Erreur de syntaxe près de '"', '09:51', '10:00', '.Contrat.', '.Non-Conformité.', '.Hors m' à la ligne 2

Pour que vous puissiez m'aider sur ce point je vous donne ma requête sql correspondante :

$sql5= "insert into intervention values ('', STR_TO_DATE('".$DateOuverture."', '%d/%m/%y'), STR_TO_DATE('".$DateCloture."', '%d/%m/%y'),
'".$TypeIntervention."','".$materiel."', '".$Tempsintervention."','".$observation."', '".$HeureOuverture."', '".$HeureCloture."',
'.$contrat.', '.$nc.', '.$hm.', '.$comptabilise.', '.$facture.', '".$operateur2."', '".$operateur3."', '".$operateur4."',
STR_TO_DATE('".$DateOuverture1."', '%d/%m/%y'), STR_TO_DATE('".$DateOuverture2."', '%d/%m/%y'), STR_TO_DATE('".$DateOuverture3."', '%d/%m/%y'),
STR_TO_DATE('".$DateOuverture4."', '%d/%m/%y'), '".$HeureOuverture1."', '".$HeureOuverture2."', '".$HeureOuverture3."','".$HeureOuverture4."',
STR_TO_DATE('".$DateCloture1."', '%d/%m/%y'), STR_TO_DATE('".$DateCloture2."', '%d/%m/%y'), STR_TO_DATE('".$DateCloture3."', '%d/%m/%y'),
STR_TO_DATE('".$DateCloture4."', '%d/%m/%y'), '".$HeureCloture1."', '".$HeureCloture2."', '".$HeureCloture3."','".$HeureCloture4."',
'".$Tpsintervention1."', '".$Tpsintervention2."', '".$Tpsintervention3."', '".$Tpsintervention4."', '".$res3."', '".$res4."')";
$res5=mysql_query($sql5) or die ('Exécution de la requête impossible sql5 :'.mysql_error());

Les autres champs que vous voyez fonctionnent normalement.
Merci d'avance de votre précieuse aide.

lauraM · Answer

Une autre chose à signaler!
En fait la base de données que j'ai pour ce formulaire a été créer à la suite d'une migration sql.
Cette base de donnée était avant cela sous access.
Je viens de remarquer que le type des champs correspondant aux cases à cocher était "bit(1)". Peut-être est-ce cela qui pose problème à mon insertion de données du formulaire dans la base de données?
Qu'en pensez vous?
De plus si c'est ca qui pose problème comment vais-je faire pour ne pas perdre les données avec ce type que j'ai déjà insérées ( en grande quantité).
Merci de votre aide.

Sri Lumpa · Answer

C'est peut-être ça le problème (le fait que le type de champs soit bit(1)). Cela signifie que tu ne peux mettre dans ces champs que 0/false ou 1/true (enfin si je ne me trompe pas).

Maintenant, à la vue du message d'erreur que tu obtiens, ça devrait résoudre ton problème. En effet, la requête essaye (apparemment) d'insérer dans un champs du type bit(1) les valeurs ".Contrat.", ".Non-Conformité.", etc... au lieu de 0 ou de 1... Tu dois toujours avoir un problème d'initialisation quelque part.

lauraM · Answer

Ok j'ai bien essayer avec 0(false) et 1(true).
Le point positif c'est que je n'ai plus d'erreur.
Le point négatif c'est que même si je ne coche pas les cases il me les met toutes à 1(true) dans la base de données.
Aurais-tu une idée du problème?
Merci de ton aide.
Ou si par hasard quelqu'un à déjà eut ce problème j'accepte volontier son aide.

lauraM · Answer

En fait mon problème,je crois, c'est que je ne sais pas utiliser la fonction "isset".
Donc est-ce que quelqu'un pourrait me fournir de l'aide.

Sri Lumpa · Answer

La fonction isset() va vérifier que tes variables sont non-vides, alors je ne sais pas d'où peut venir le problème pour toi. tu peux peut-être affiner ton test comme ça...

if(isset($_POST['variable_1'])){
    if($_POST['variable_1'] == 'valeur_variable_1'){
        $var1 = 1
    } else {
        $var1 = 0;
    }
}

Avec valeur_variable_1 la valeur que tu as mis dans le champs value pour ta checkbox variable_1 (je sais pas si je suis tr_s clair, si c'est pas le cas dis-le moi)

Par exemple, si la personne remplissant ton formulaire coche la case Hors-Maintenance, la variable $_POST['HM'] prendra la valeur "Hors Maintenance" (ce que tu as défini dans le champs value). Si ta variable $_POST['HM'] vaut bien "Hors Maintenance", tu n'as plus qu'à initialiser ta variable $hm à 1.

Je pense que ça devrait régler ton problème, j'espère ne pas me tromper ^^

lauraM · Answer

Merci de ton aide.
J'effectué pour mes cinq champs ce que tu m'as proposés. Malheureusement j'ai les mêmes qu'au tout début c'est-à-dire :

Notice: Undefined variable: contrat in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 104
Notice: Undefined variable: nc in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 104
Notice: Undefined variable: hm in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 104
Notice: Undefined variable: comptabilise in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 104
Notice: Undefined variable: facture in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 104

Et ce qui est bizard c'est que l'enregistrement s'effectue comme même.
Je désespère vraiment la loool.
Merci d'avance à tous ceux qui peuvent m'aider.

Sri Lumpa · Answer

As-tu vérifié que les enregistrements dans ta base de données donnaient bien les bonnes valeurs ? Si on t'indique que tes variables ne sont pas initialisées mais que tes enregistrements passent quand même, honnêtement je comprends plus ^^...

La ligne 104 correspond à quoi dans ton code ?

lauraM · Answer

L'enregistrement passe mais assigne automatiquement vrai aux cases à cocher. C'est comme si il ne prenait pas en compte qu'elles ne soient pas cochées.

lauraM · Answer

Une autre précision c'est que lorsque je remplis mon formulaire et que je coche les cases et bien au moment de les valider je n'ai pas d'erreurs. Mais si je ne les cochent pas alors là j'ai les erreurs.

lauraM · Answer

Je crois que c'est le moment de s'arracher les cheveux lol hihi.
Car ce que je viens de dire précédemment s'annule, je viens de refaire le test et lorsque que je ne coche pas les cases j'ai les erreurs mais les bonnes valeurs dans la base de donnée. Lorsque je les cochent il y a aussi la bonne valeur dans la base de donnée.
A vrai dire je ne sais plus où  donner de la tête.

lauraM · Answer

HELP! Plus personne pour m'aider à résoudre cette énigme qui me coince depuis quelques jours déjà?

Sri Lumpa · Answer

Modifie peut-être ton formulaire comme ça : Comme ça tu stockeras les valeurs de tes cases à cocher dans un seul tableau et se sera plus simple à gérer... Puis pour initialiser tes valeurs, tu fais ainsi : $contrat = 0; $comptabilise = 0; $hm = 0; $nc = 0; $facture = 0; if(isset($_POST['infos'])){ foreach($infos as $info){ switch($infos){ case 'Contrat': $contrat = 1; break; case 'Non-Conformité': $nc = 1; break; case 'Hors maintenance': $hm = 1; break; case 'Comptabilisé': $comptabilise = 1; break; case 'Facturé': $facture = 1 break; } } } Je sais pas si ça réglera entièrement le problème mais ça devrait aider...

lauraM · Answer

Bonjour,
Merci d'avoir pris du temps pour m'aider.
J'ai donc testé ce que vous m'aviez recommandé.
Malheureusement que je coche la case ou non il me l'inscrit coché dans la base de données. De plus quand je coche les cases une erreur s'affiche mais l'enregistrement passe comme même.
Voici l'erreur :

Notice: Undefined variable: infos in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 19
Warning: Invalid argument supplied for foreach() in C:\Documents and Settings\Administrateur\Mes documents\site essai	raitement.php on line 19

La ligne 19 correspond à cette partie du code :

...
foreach($infos as $info){
...

Voilà et je me demandais si le deuxième $info sans s à la fin ne devait pas justement en avoir un.
Merci de votre aide précieuse.

clem285 · Answer

pour tes case à coché je fais sa moi :

if (isset($_POST['NC']) == 'valeur_de_NC')
{
$nc=1;
}
else
{
$nc=0;
}

lauraM · Answer

Merci clem285, j'ai effectué ce que tu ma proposé en adaptant mes variables.
Je n'ai aucunes erreurs. Les enregistrements passent que les cases soient cochées ou non.
Le souci c'est que même si je ne coche pas les cases il me les met cochées dans la base de données.
Je crois que je m'en sortirais jamais lol.
Croyez vous que le problème pourrait venir d'ailleurs?
Auriez-vous une idée ?

lauraM · Answer

Effectivement dans ma base de données mon type est à bit(1) donc je suppose qu'il met 1 par défaut. Ensuite le problème si je change le type c'est que comme j'ai déjà beaucoup de données dans ma base de données et que les cases à cocher ont déjà comme valeur un petit carré qui signifit coché il serait embêtant pour moi d'avoir des valeurs différentes. Enfin une sorte de coupure dans ma base de donnée, j'espère que je suis assez clair sinon dite le moi.
Donc j'aimerai savoir si je change le type en varchar, comme vous me le proposez, si j'aurais toujours les mêmes valeurs actuelles c'est-à-dire les petits carrés ou si j'aurais des 1 et 0.
Merci beaucoup ca m'avance sur le problème. Et oui vive les stages lol.

Sri Lumpa · Answer

Non, il me semble que le fait qu'il soit marqué bit(1) signifie que ton champs est de type bit (un 0 ou un 1) et qu'il est de taille 1. Oui je sais c'est hyper-redondant comme info.
Si tu as une vision de ta table avec PhPMyAdmin ou tout autre gestionnaire de base de données avec une interface graphique, tu dois avoir un moyen de vérifier dans la structure de la base la valeur par défaut.

lauraM · Answer

ba j'ai voulu enlever le 1 qui est dans (taille/valeurs) mais il me pose une erreur comme quoi ce n'est pas un nombre si je ne veux rien mettre. De plus le type est VARCHAR quand je veux modifier des choses d'une case à cocher. Alors que sur la structure c'est bit(1).
Que dois-je faire?

lauraM · Answer

En fait quand je veux modifier les attributs de mon champ j'ai un attribut qui s'appelle défaut. Et dans cet attribut la valeur est : Null.
C'est ca dont vous parliez?

Sri Lumpa · Answer

Normal qu'il te sorte une erreur quand tu veux modifier la taille de ce champs, ce n'est pas permis de le laisser vide...

Le VARCHAR correspond à quel champs tu dis ? 
(PS : je ne sais pas si tu as vu, je n'aurais pas du mettre le message à cet endroit : j'avais fait une petite erreur dans la dernière solution que je t'avais proposée, il faut remplacer dans le foreach $infos par $_POST['infos'].)

Sri Lumpa · Answer

OK, donc par défaut, la valeur est NULL... ça nous arrange pas ça :s. Juste avant de faire la requete, fais un echo de toutes les variables que tu vas insérer histoire de vérifier quelles sont les valeurs que tu transmets.

lauraM · Answer

J'ai modifier le code à l'endroit ou vous m'aviez dit qu'il y avait une erreur.
Donc ensuite j'ai refait un test et j'ai vu que quand je cochais les cases plusieurs erreurs apparaissaient plusieurs fois. En clair le code a un problème.
Si j'essai de mettre 1 au lieu de Null dans ma base de donnée croyez vous que ca pourrait fonctionner?

lauraM · Answer

Je pense que ca serait plus facile si je met les cases à cocher en VARCHAR et que je laisse Null par défaut non?
Ca changerai mon code. Pouvez vous m'aider et me dire ce que vous en pensez?

Sri Lumpa · Answer

Le problème comme j'ai dit plus haut c'est que si tu te mets à changer la structure de la base de données, ça signifie modifier tout ce qui peut y avoir accès, aussi bien en lecture qu'en écriture. Tu pourrais par exemple avoir des scripts d'insertion qui reposent sur le fait que justement la valeur par défaut est NULL et ça signifie devoir les recoder... De toute manière, ça ne changerait rien à ton problème, donc ce n'est pas la solution. Quelles erreurs as-tu lorssque tu coches des cases ? voici un bout de code que j'utilise avec des checkbox, peut-être tu verras mieux que moi où pourrais ce situer ton problème if(is_dir($imagesDir)){ if($dir = opendir($imagesDir)){ echo ""; closedir($dir); } else { echo "Erreur lors de l'ouverture du répertoire $imagesDir"; } } else { echo "$imagesDir n'est pas un répertoir valide
"; } // Si on a valider le formulaire.... if(isset($_POST['go'])){ // Pour chaque image contenu dans $_POST['images'] foreach($_POST['images'] as $image){ // traitement .... } } Ce code marche sans problème pour moi. Je n'ai juste pas régler le cas ou aucune checkbox n'est cochée et il me met alors un warning mais n'entre de toute manière pas dans la boucle.

lauraM · Answer

Quand vous cochez votre case dans votre formulaire qu'est ce qui s'affiche habituellement dans votre base de données si ce n'est pas les valeurs car moi j'ai des petits carrés et c'est pas top!
Quel caractère avez-vous? Moi j'aimerai bien mettre les valeurs de mes checkbox ms quel requête faut-il faire pour changer le caractères par la valeur.

lauraM · Answer

Et après lorsque j'ai modifier les valeurs présentes dans ma base de données et que j'ai attribué le type VARCHAR j'utilise cette structure (qui ne me donne aucunes erreurs et qui devrait fonctionner) :

if (isset($_POST['contrat']) == 'contrat') { 
$contrat=''; 
} else { 
$contrat=; 
} 
if (isset($_POST['nc']) == 'non-conformite') { 
$nc=1; 
} else { 
$nc=0; 
} 
if (isset($_POST['hm']) == 'hors maintenance') { 
$hm=1; 
} else { 
$hm=0; 
} 
if (isset($_POST['comptabilise']) == 'comptabilise') { 
$comptabilise=1; 
} else { 
$comptabilise=0; 
} 
if (isset($_POST['facture']) == 'facture') { 
$facture=1; 
} else { 
$facture=0; 
}

Je remplace à la place du 1 la valeur Null et  à la place du 0 la valeur de ma checkbox.
Qu'en pensez-vous? Le problème c'est ma requête pour modifier toutes les valeurs déjà existantes.

clem285 · Answer

update nom_de_ta_table
set nom_du_champ1 = 'valeur de ta checkbox1'
where nom_du_champ1 = '1';
requete pour modifier les valeur (fais un essai sur une copie de ta table, c'est plus prudent)

Sri Lumpa · Answer

Quand je coche une case de ma checkbox s'affiche la valeur contenue dans l'attribut "valeur" de la balise HTML correspondante à la checkbox.

lauraM · Answer

moi j'ai un petit carré à cause de la migration sql et jsais pas d'où sort ce caractère! En tout cas il me pose problème. Quelqu'un serait comment modifier une migration sql.
Est ce que le type TINYINT est bien le type d'un booleen en mysql???

Sri Lumpa · Answer

Là je suis désolé mais j'avoue que je reste sec, je ne sais plus comment t'aider.... :s

lauraM · Answer

Et bien en fait là je suis en train d'essayer de refaire la mifration SQL et je voudrais juste savoir quel est le type dans mysql pour un booleen. Est ce que c'est bien TINYINT ???
MERCI

lauraM · Answer

J'ai essayé la requête proposé par clem285 mais ca ne marche pas. Mysql me dit qu'il y a une erreur de syntaxe.
Moi en fait ce que j'ai comme vakleur dans ma base de données c'est ca : 
Donc c'est ca que je voudrais changer par un 'oui' par exemple et la je pourrais mieux y arriver avec mes cases à cocher je pense.

lauraM · Answer

C'est bon pour la requête je me suis trompé. Je vous tiens au courant. MERCI encore de toute votre aide.

lauraM · Answer

Comment dois-je marquer le POSt en résolu ? Désolé je suis nouvelle ici

Erreur case à cocher en php-mysql

38 réponses

Discussions similaires