Discussion :

[lavande4]

bonjour

j'essaie d'enregistrer sur ma base de données plusieurs informations avec des variables $_session et je ne suis pas sure de ma syntaxe car ca ne marche pas !!!!

je vous montre ce que j'ai ecris :

Code :

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

mysql_query("insert into client values(' '," .$_SESSION['nom']. "," .$_SESSION['prenom']. "," .$_SESSION['adresse']. "," .$_SESSION['code_postal']. "," .$_SESSION['ville']. ",$ref_pays," .$_SESSION['telephone'].  "," .$_SESSION['email']) or die (mysql_error());

le premier champ de ma table est un auto-increment.
j'ai des doutes sur comment mettre ces variables entre guillemet ou non mais j'ai essayé plusieurs choses et rien n'y fais.

merci pour votre aide.

[RideKick]

tu n'as pas de message d'erruers ? les variables ne sont pas vide ?

[Djakisback]

Salut,
a priori il faut que t'énumères tes colonnes :

Code :

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

INSERT INTO table (a,b,c) VALUES (1,2,3)

et pour ton auto-incrément pas d'espace entre les quotes ''.

[RideKick]

Salut,
a priori il faut que t'énumères tes colonnes

Pas forcement

et pour ton auto-incrément pas d'espace entre les quotes ''.

La je pense que tu as raison mais pas sur ...

[lavande4]

le message d'erreur est le suivant :

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rue costeseque,34500,beziers,FR,0468333254,mazezf@sqef.fr' at line 1

les données ne sont pas vides j'ai vérifié.

[lavande4]

g enlever l'espace de debut mais ca ne change rien

[Djakisback]

Pas forcement

Je me suis posé la question en fait ^^ mais dans la doc la syntaxe sans énumération n'est pas proposée.

Sinon il faut aussi que tu mettes des quotes pour tes données qui sont des chaînes de caractères et il te manque une parenthèse fermante :

Code :

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

mysql_query("insert into client values('','" .$_SESSION['nom']. "','" .$_SESSION['prenom']. "','" .$_SESSION['adresse']. "','" .$_SESSION['code_postal']. "','" .$_SESSION['ville']. "','".$ref_pays."','" .$_SESSION['telephone'].  "','" .$_SESSION['email']."')") or die (mysql_error());

Mysql (pas toutes les versions je crois) tolère les quotes pour les champs numériques donc en fait tu peux en mettre partout.

[lavande4]

j'ai fais ce que tu m'a dis mais maintenant ca me met ce message

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

la je sais plus...

[Djakisback]

C'est peut-être un problème d'échappement, d'ailleurs normalement tu dois protéger tes données avec mysql_real_escape_string(). Sinon essaie d'afficher ta requête :

Code :

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

1
2
$sql = "insert into client values('','" .$_SESSION['nom']. "','" .$_SESSION['prenom']. "','" .$_SESSION['adresse']. "','" .$_SESSION['code_postal']. "','" .$_SESSION['ville']. "','".$ref_pays."','" .$_SESSION['telephone'].  "','" .$_SESSION['email']."')";
mysql_query($sql) or die (mysql_error()." - ".$sql);

[Invité]

Salut,

Quels sont les champs de ta table ainsi que leur type?

Code :

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

1
2
VALUES ('',

est-ce le champ correspondant à ta clé primaire? est-il en autoincrément?

Je te conseille de mettre les champs de tes tables dans ton insert pour des raisons de clarté et pour éviter de chercher longtemps.

[lavande4]

voila j'ai protéger mes données et j'ai mis le nom des colonnes

$nom = mysql_real_escape_string($_SESSION['nom']);
$prenom = mysql_real_escape_string($_SESSION['prenom']);
$adresse = mysql_real_escape_string($_SESSION['adresse']);
$CP = mysql_real_escape_string($_SESSION['code_postal']);
$ville = mysql_real_escape_string($_SESSION['ville']);
$tel = mysql_real_escape_string($_SESSION['telephone']);
$mail = mysql_real_escape_string($_SESSION['email']);

mysql_query("insert into client (ref_client, nom_client, prenom_client, adresse_client, code_postal_client, ville_client, ref_pays, telephone_client, email_client) values('', $nom, $prenom, $adresse, ".$CP.", $ville, $ref_pays, ".$tel.", $mail") or die (mysql_error());

ref_client est bigint en autoincrement et cle primaire
nom prenom adresse ville sont en text
codepostal en smallint
ref_pays en varchar
telephone en mediumint
email en varchar


et le message d'erreur est maintenant :
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'rue costeseque, 34500, beziers, FR, 0468333254, maninna@hotmail.com' at line 1

je comprend pas car il commence l'enumération en plein milieu de l'adresse alors que j'ai tester mes variables une par une et tout s'affiche correctement.

[lavande4]

j'ai oublier de preciser que $ref_pays est une variable classique qui ait créer un peu plus haut dans le script et elle s'affiche egalement

merci

[CinePhil]

ref_client est bigint en autoincrement et cle primaire

Comme ref_client est une clé primaire auto-incrémentée, il ne faut pas mettre cette colonne dans l'INSERT.
Il faut donc citer toutes les colonnes qui vont recevoir des données et bien sûr leur faire correspondre les VALUES dans le même ordre.

nom prenom adresse ville sont en text

Des VARCHAR seraient plus appropriés.

codepostal en smallint

Mauvaise conception !
Un code postal n'est pas un nombre mais, comme son nom l'indique, un code.
On ne fait pas d'opération avec un code postal.
Si le code postal ne concerne que des données françaises, mettre cette colonne en CHAR(5).

telephone en mediumint

Idem ci-dessus pour la même raison.

On devrait donc arriver à quelque chose qui ressemble à ça :

Code :

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

1
2
3
4
mysql_query(
"insert into client (nom_client, prenom_client, adresse_client, code_postal_client, ville_client, ref_pays, telephone_client, email_client) 
values('$nom', '$prenom', '$adresse', '$CP', '$ville', '$ref_pays', '$tel', '$mail' ") 
or die (mysql_error());

Pour débugger, il est même préférable de passer ça en deux étapes (en plus c'est plus facile à lire :

Code :

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

1
2
3
4
5
6
$sql = "insert into client (nom_client, prenom_client, adresse_client,   code_postal_client, ville_client, ref_pays, telephone_client, email_client) 
values('$nom', '$prenom', '$adresse', '$CP', '$ville', '$ref_pays', '$tel', '$mail' ";
 
echo "<br />SQL : $sql"; //Ligne à commenter une fois le débuggage fait
 
mysql_query($sql) or die (mysql_error());

[lavande4]

merci beaucoup pour votre réponse, je suis encore en formation de developpeur donc cela m'aide bien.

encore merci