Discussion :

[bubuche4815]

Bonjour,

après des heures de vaines recherches je viens me tourner vers vous pour un petit problème :

Je voudrais insérer des données dans une table "perso", mais impossible de faire marcher correctement ma requête SQL,
dont voilà le code :

Code :

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

1
2
$query="insert into perso (email,pseudo,sexe,race,classe,pass,confpass,statut,xp,level,levelup,force,dex,mana,intel,sag,char) values ('$email','$pseudo','$sexe','$race','$classe','$pass','$confpass','$statut','$xp','$level','$levelup',$force,$dex,$mana,$intel,$sag,$char)";
$result=mysql_query ($query) die('Erreur : '.$e->getMessage());

Les champs

Le message d'erreur que j'obtiens est le suivant :

Parse error: syntax error, unexpected T_EXIT in E:\monchemin\nouveau_perso.php

J'ai essayé avec et sans les apostrophes, j'obtiens le même résultat. Et je ne sais pas trop ce que le T_EXIT signifie.

Merci d'avance pour votre aide

[van-bom]

tu as une erreur de syntaxe

Code :

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

$result=mysql_query ($query) die('Erreur : '.$e->getMessage());

essaye de faire ça

Code :

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

1
2
3
4
5
if(mysql_query($query) ){
$result = mysql_query($query);
}else{
die('Erreur : '.$e->getMessage());
}

ou sur une seule ligne

Code :

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

1
2
 // si la requête est bien exécutée tu tu affecte la variabale $result sinon tu arrête le script
( mysql_query ($query) ) ? $result=mysql_query ($query) :  die('Erreur : '.$e->getMessage());

[bubuche4815]

Merci de ta réponse rapide !

J'ai essayé ton code mais j'ai un nouveau message d'erreur :

Fatal error: Call to a member function getMessage() on a non-object

[bubuche4815]

Et quand j'essaye avec

Code :

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

1
2
3
4
5
if(mysql_query($query) ){
$result = mysql_query($query);
}else{
die (mysql_error ());
}

j'obtiens :

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 'force,dex,mana,intel,sag,char) values ('julien.paris3@hotmail.fr','mec','Homme',' at line 1

[van-bom]

pour l'erreur :

Fatal error: Call to a member function getMessage() on a non-object

$e n'est un objet , il faut crée une instance de la classe PDO , en plus pourquoi tu utilise PDO et l'extension mysql ?

pour l'erreur

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 'force,dex,mana,intel,sag,char) values ('julien.paris3@hotmail.fr','mec','Homme',' at line 1

c'est une erreur de syntaxe , vérifier bien les apostrophes

[bubuche4815]

Oui en effet j'ai enlevé le PDO du MySQL :

Code :

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

1
2
$query = "insert into perso (email,pseudo,sexe,race,classe,pass,confpass,statut,xp,level,levelup,force,dex,mana,intel,sag,char) values ('$email','$pseudo','$sexe','$race','$classe','$pass','$confpass','$statut','$xp','$level','$levelup','$force','$dex','$mana','$intel','$sag','$char')";
$result = mysql_query($query) or die (mysql_error ());

Mais j'ai toujours cette erreur :

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 'force,dex,mana,intel,sag,char) values ('julien.paris3@hotmail.fr','a','Homme','H' at line 1

J'ai beau chercher je ne vois pas d'erreur d'apostrophe. Il y en a sûrement un qui traine par là...

Merci de ton aide en tout cas

[van-bom]

essaye ça

Code :

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

1
2
3
4
$query = "insert into perso (email,pseudo,sexe,race,classe,pass,confpass,statut,xp,level,levelup,force,dex,mana,intel,sag,char)"
        . " values "
        . "('".$email."','".$pseudo."','".$sexe."','".$race."','".$classe."','".$pass."','".$confpass."','".$statut."','".$xp."','".$level."','".$levelup."','".$force."',
		'".$dex."','".$mana."','".$intel."','".$sag."','".$char."')";

[bubuche4815]

J'ai toujours la même erreur, je pense qu'on est face à un mystère :p

[rawsrc]

Bonjour,

et comme ç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
$esc = function($p) { return "'".mysql_real_escape_string($p)."'"; };
 
$data = array(
    'email'    => $email,
    'pseudo'   => $pseudo,
    'sexe'     => $sexe,
    'race'     => $race,
    'classe'   => $classe,
    'pass'     => $pass,
    'confpass' => $confpass,
    'statut'   => $statut,
    'xp'       => $xp,
    'level'    => $level,
    'levelup'  => $levelup,
    'force'    => $force,
    'dex'      => $dex,
    'mana'     => $mana,
    'intel'    => $intel,
    'sag'      => $sag,
    'char'     => $char
);
 
$escaped = array_map($esc, $data);
$sql     = 'INSERT INTO perso ('.implode(', ', array_keys($escaped)).') VALUES ('.implode(', ', $escaped).')';

Il faut toujours penser à échapper les valeurs.
Par ailleurs, toute la lib mysql_xxx va être abandonnée par PHP, il faut passer soit à mysqli soit à PDO.

[Celira]

Le problème en l’occurrence, c'est plutôt le nom des colonnes : FORCE est un mot réservé pour MySQL. Si tu l'utilises comme nom de colonne, il faut le mettre entre backquotes. (j'ai pas vérifié les autres colonnes)

Donc en adaptant le code de rawsrc:

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
$esc = function($p) { return "'".mysql_real_escape_string($p)."'"; };
 
$data = array(
    'email'    => $email,
    'pseudo'   => $pseudo,
    'sexe'     => $sexe,
    'race'     => $race,
    'classe'   => $classe,
    'pass'     => $pass,
    'confpass' => $confpass,
    'statut'   => $statut,
    'xp'       => $xp,
    'level'    => $level,
    'levelup'  => $levelup,
    'force'    => $force,
    'dex'      => $dex,
    'mana'     => $mana,
    'intel'    => $intel,
    'sag'      => $sag,
    'char'     => $char
);
 
$escaped = array_map($esc, $data);
$sql     = 'INSERT INTO perso (`'.implode('`, `', array_keys($escaped)).'`) VALUES ('.implode(', ', $escaped).')';

EDIT : Rendons à Césarrawsrc ce qu'il lui appartient

[rawsrc]

Le problème en l’occurrence, c'est plutôt le nom des colonnes : FORCE est un mot réservé pour MySQL. Si tu l'utilises comme nom de colonne, il faut le mettre entre backquotes. (j'ai pas vérifié les autres colonnes)

Donc en adaptant le code de sabotage :

Bien vu pour le mot réservé MySQL FORCE par contre de mon côté j'avais réservé le mot clé rawsrc et aux dernières nouvelles les mots clés sabotage et rawsrc n'avaient pas encore fusionné pour devenir équivalents

P'têt comme ça : `sabotage`, non y a rien à faire c'est toujours différent de `rawsrc` ;-)

[Celira]

Bien vu pour le mot réservé MySQL FORCE par contre de mon côté j'avais réservé le mot clé rawsrc et aux dernières nouvelles les mots clés sabotage et rawsrc n'avaient pas encore fusionné pour devenir équivalents

P'têt comme ça : `sabotage`, non y a rien à faire c'est toujours différent de `rawsrc` ;-)

Oups ! Désolationnée très beaucoup ! (le pire, c'est que je suis sûre d'avoir vérifié... J'ai pas dû regarder le bon thread )

[bubuche4815]

Ahhhhhhh merci beaucoup à vous !!!
Celira a raison, le nom de la colonne posait problème.

J'ai remarqué également que "char" était réservé, j'ai donc modifié le nom de mes deux colonnes et tout marche correctement.

Je vous remercie grandement de votre aide précieuse !

Pour Mysqli j'avais déjà tapé toutes mes requêtes comme ça, mais impossible de les faire fonctionner, j'avais cette erreur :

Fatal error: Call to undefined function mysqli() in

Je suis donc passé à mysql tout court. Mais ceci est un autre problème et je posterai un nouveau sujet au bon endroit