Discussion :

[foxnono06]

Bonjour à tous,

Voilà, ma requête est la suivante :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
string sModifierSQL = "UPDATE [client] SET "
                + "`nom` = "+ "'" + Nom +"',"
                + "`prenom` = "+ "'" + Prenom +"',"
                + "`ad1` = "+ "'" + Ad1 +"',"
                + "`ad2` = "+ "'" + Ad2 +"',"
                + "`cp` = "+ "'" + CP.ToString()+"',"
                + "`ville` = "+ "'" + Ville +"',"
                + "`pays` = "+ "'" + Pays +"',"
                + "`fix` = "+ "'" + Fix +"',"
                + "`port` = "+ "'" + Port +"',"
                + "`email` = "+ "'" + Email +"',"
                + "`nbenf` = "+ "'" + Nbenfant.ToString() +"',"
                + "WHERE id_client = " + m_IndexDataSet.ToString();

Je m'en sers pour mettre a jour une base Access.
Quand j'essaye de l'utiliser, j'obtiens le message d'erreur suivant :

Erreur de syntaxe dans la requête update.

Vous me direz, la réponse est dans la question mais je ne trouve pas l'erreur.

Si quelqu'un à une idée, ca me permettrait de pouvoir avancer car c'est ma première requête UPDATE. (Mes SELECT, INSERT et DELETE fonctionnent :/)

[yonpo]

Salut,

Il n'y a pas d'apostrophe au nom des champs

[Isythiel]

Affiche ta variable sModifierSQL après avoir fait toutes les concaténations, et exécute la requete directement dans access si tu ne trouves pas l'erreur.

[foxnono06]

Salut merci a toi pour ta réponse.

J'ai essayé ta méthode :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
string sModifierSQL = "UPDATE [client] SET "
                + "`nom` = "+ "" + Nom +","
                + "`prenom` = "+ "" + Prenom +","
                + "`ad1` = "+ "" + Ad1 +","
                + "`ad2` = "+ "" + Ad2 +","
                + "`cp` = "+ "" + CP.ToString()+","
                + "`ville` = "+ "" + Ville +","
                + "`pays` = "+ "" + Pays +","
                + "`fix` = "+ "" + Fix +","
                + "`port` = "+ "" + Port +","
                + "`email` = "+ "" + Email +","
                + "`nbenf` = "+ "" + Nbenfant.ToString() +","
                + "WHERE id_client = " + m_IndexDataSet.ToString();]

Et j'obtiens l'erreur suivante :

opérateur absent

et quand j'affiche le contenu de la requete il ne m'affiche que le contenu de ma variable Ad1.

Merci.

[Isythiel]

et quand j'affiche le contenu de la requete il ne m'affiche que le contenu de ma variable Ad1.

C'était le but, pouvoir voir ta requête "brute" et l'exécuter sans passer par C# ...

[foxnono06]

Je suis vraiment débutant et je t'avoue que je ne sais pas comment éxecuter ma requête directement dans Access pour la tester. Lorsque je l'affiche j'obtiens :

UPDATE [client] SET `nom`='Ans',`prenom`='aaa',`ad1`= '112 aaa',`ad2`='ds qsdqs',`cp`='06000',`ville`='Aaa',`pays`='Aaa',`fix`='000000000',`port`='0000000000',`email`='aaa@yahoo.fr',`nbenf`='1',WHERE id_client =2

[Isythiel]

Pardon je ne voulais pas être désagréable.
[client] c'est censé être le nom de ta table ? Il n'y aurait pas des crochets en trop ?

[foxnono06]

yonpo j'ai essayé ta méthode et j'ai tjs une erreur de syntaxe.

[foxnono06]

Isythiel : j'ai enlevé les crochets tjs une erreure de syntaxe

[yonpo]

Il n'y a pas de virgule avant le WHERE

[kheironn]

J'imagine que si tu postes ici, c'est que tu es en C#...
L'utilisation de requêtes concaténées comme celle que tu nous montres est le meilleur moyen de se faire planter sa base de données. Tu connais l'injection SQL ?
Tu devrais te pencher sur les requêtes paramétrées (tu trouveras ça facilement sur le net). Je pense que ça doit marcher avec MS Access, mais je n'ai jamais essayé.

[yonpo]

Je pensais plutôt à :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
string sModifierSQL = "UPDATE [client] SET "
                + "nom = "+ "'" + Nom +"',"
                + "prenom = "+ "'" + Prenom +"',"
                + "ad1 = "+ "'" + Ad1 +"',"
                + "ad2 = "+ "'" + Ad2 +"',"
                + "cp = "+ "'" + CP.ToString()+"',"
                + "ville = "+ "'" + Ville +"',"
                + "pays = "+ "'" + Pays +"',"
                + "fix = "+ "'" + Fix +"',"
                + "port = "+ "'" + Port +"',"
                + "email = "+ "'" + Email +"',"
                + "nbenf = "+ "'" + Nbenfant.ToString() +"',"
                + "WHERE id_client = " + m_IndexDataSet.ToString();

[kheironn]

ou en plus simple, pour répondre à un message beaucoup plus haut :

Code :

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

string sModifierSQL = "UPDATE [client] SET nom = '" + Nom +"', prenom = '" + Prenom +"', ad1 = '" + Ad1 +"', ad2 = '" + Ad2 +"', cp = '" + CP.ToString()+"', ville = '" + Ville +"', pays = '" + Pays +"', fix = '" + Fix +"', port = '" + Port +"', email = '" + Email +"', nbenf = '" + Nbenfant.ToString() +"' WHERE id_client = " + m_IndexDataSet.ToString();

Pourquoi se compliquer la vie aec des concaténations inutiles... ?
et as-tu regardé pour faire une requête paraméétrée ?