Ajout de nombre dans une table ?

Version imprimable

Bonjour tout le monde, aujourd'hui j'ai un petit problème concernant mon programme.
Je possède :
Une base de données
Je suis chez AlwaysData.net
J'utilise Navicat Premium

J'ai fais un système de "Jetons" sur mon programme (comme sur les jeux quoi) , jusque là, tout va bien, j'ai mis un bouton, quand on clique dessus , 50 Jetons s'ajoutent au compte et dans le champ "Jetons" donc pas de problème jusque là. Mais le problème c'est que quand on répète l'action, les jetons ne s'ajoutent pas ! Si je clique une première fois sur le bouton, il va y avoir écrit "50" dans le champ mais si je re clique, le 50 est remplacé par 50 , moi, j'aimerais que cela s'ajoute donc 50, 100, 150 etc etc ... J'ai bien entendu cherché mais bon il n'y avait pas grand chose sur internet
Une partie du code :
Code:

1 2 3 4 5 6 Dim query As String query = "UPDATE lenomdemabdd.lenomdematable SET username='" & Form1.MaterialSingleLineTextField1.Text & "',Jetons='" & +"50" & "' WHERE username='" & Form1.MaterialSingleLineTextField1.Text & "'" command = New MySqlCommand(query, MySqlConnection) reader = command.ExecuteReader MsgBox("Vous avez ajouté les jetons") MySqlConnection.Clone()
Form1.MaterialSingleLineTextField1.Text = On récupère le nom du compte pour pouvoir ajouter les jetons au bon compte (C'est une textbox)

J'ai essayé de mettre +"50" mais ça na pas fonctionné ...
Je sais pas si j'ai été assez clair, si je ne l'ai pas assez été , dites le.

Merci, Cordialement.

20/08/2015, 16h13
jopopmk

Salut,

dans ton update tu écris "Jetons = 50" (une fois concaténé), du coup tu ne fais qu'assigner 50 à chaque fois.
Ce que tu veux c'est ajouter 50 au nombre de jetons que tu as déjà, ce qui s'écrit logiquement :"Jetons = Jetons + 50".

edit PS : à quoi sert l'update du champs username, qui ne change pas vu qu'il sert de critère de sélection ?
20/08/2015, 16h49
mathisdu42

Citation:

Envoyé par jopopmk

Salut,

dans ton update tu écris "Jetons = 50" (une fois concaténé), du coup tu ne fais qu'assigner 50 à chaque fois.
Ce que tu veux c'est ajouter 50 au nombre de jetons que tu as déjà, ce qui s'écrit logiquement :"Jetons = Jetons + 50".

edit PS : à quoi sert l'update du champs username, qui ne change pas vu qu'il sert de critère de sélection ?

Merci d'avoir répondu jopopmk.

Je comprend pas trop ce que tu me dis de faire..

L'update du champs username permet d'attribuer les Jetons au compte qui est connecté, si je ne met pas d'update du champs username , les jetons vont être attribués à tout les compte de la base de données.
20/08/2015, 17h45
jopopmk

Houla ! On part de loin ...

Alors dans la requête que tu nous montres :
- le nom de ta database est : lenomdemabdd
- le nom de ta table est : lenomdematable
- les champs mis à jours (derrière le SET) sont : username et Jetons
- le critère de sélection (derrière le WHERE) est : username

Donc Jetons est un champs, ce n'est pas le nom de ta table. Et tu mets à jours le champs username, indépendamment du critère de sélection, qui lui est derrière le WHERE, et qui est le seul à limiter le nombre d'enregistrements impactés par ton UPDATE.

C'est la première fois que tu fais du SQL ?
21/08/2015, 10h55
mathisdu42

Citation:

Envoyé par jopopmk

Houla ! On part de loin ...

Alors dans la requête que tu nous montres :
- le nom de ta database est : lenomdemabdd
- le nom de ta table est : lenomdematable
- les champs mis à jours (derrière le SET) sont : username et Jetons
- le critère de sélection (derrière le WHERE) est : username

Donc Jetons est un champs, ce n'est pas le nom de ta table. Et tu mets à jours le champs username, indépendamment du critère de sélection, qui lui est derrière le WHERE, et qui est le seul à limiter le nombre d'enregistrements impactés par ton UPDATE.

C'est la première fois que tu fais du SQL ?

Oulaaaaa oui ! J'ai dis une énorme connerie ! Pardonne moi :o Oui tu as raison Jetons et username sont des champs !
Et quand tu me disais de faire : "Jetons = Jetons + 50" je vois pas trop ou y placer..
21/08/2015, 14h06
jopopmk

Ta requête devrait ressembler à un truc du genre :

Code:

query = "UPDATE lenomdemabdd.lenomdematable SET Jetons=Jetons+" & "50" & " WHERE username='" & Form1.MaterialSingleLineTextField1.Text & "'"

Tu peux y voir que :
- j'ai fait sauter l'update du username qui servait à rien,
- j'ai fait un incrément de Jetons, et plus une simple assignation.
- j'ai viré les simples-quotes autours de 50, le champs Jetons devant être un nombre (INT, NUMERIC ...) et non une chaîne (VARCHAR, TEXT ...).

Par contre, pour être plus propre, je te conseille de passer par des requêtes paramétrées.
21/08/2015, 14h56
mathisdu42

1 pièce(s) jointe(s)

Citation:

Envoyé par jopopmk

Ta requête devrait ressembler à un truc du genre :

Code:

query = "UPDATE lenomdemabdd.lenomdematable SET Jetons=Jetons+" & "50" & " WHERE username='" & Form1.MaterialSingleLineTextField1.Text & "'"

Tu peux y voir que :
- j'ai fait sauter l'update du username qui servait à rien,
- j'ai fait un incrément de Jetons, et plus une simple assignation.
- j'ai viré les simples-quotes autours de 50, le champs Jetons devant être un nombre (INT, NUMERIC ...) et non une chaîne (VARCHAR, TEXT ...).

Par contre, pour être plus propre, je te conseille de passer par des requêtes paramétrées.

En effet ça marche, merci ;) Mais comme dis plus haut, le fait de faire sauter l'update du username, ça ajoute les jetons à tout les comptes de la base de données.

Pièce jointe 186024
21/08/2015, 21h14
jopopmk

Alors y'a un méchant bug quelque part, parce que ça n'a pas de sens.
Fais suivre le schéma et les données actuelles qu'on voit ça.
21/08/2015, 21h22
mathisdu42

Citation:

Envoyé par jopopmk

Fais suivre le schéma et les données actuelles qu'on voit ça.

C'est à dire ? :p

Désolé hein mais je fais pas souvent du SQL.
21/08/2015, 22h40
jopopmk

Citation:

Envoyé par mathisdu42

C'est à dire ? :p

Désolé hein mais je fais pas souvent du SQL.

Pas besoin de t'excuser, on peut pas tout connaitre et en info on en apprend tous les jours ;)

J'aimerai connaitre la définition de ta table (champs, types et index).
Les données c'est juste ce qu'il y a dedans (chaque valeur de chaque champ pour chaque ligne).
Tu dois pouvoir obtenir tout ça avec la commande mysqldump.

Une requête SQL peut facilement se traduire en français. Dans ton cas c'est :
Mets à jour (UPDATE) le nombre de jetons (SET Jetons) pour l'utilisateur ayant tel nom (WHERE username = John-Kevin).
Si on met de coté les Jetons ta première requête sonne comme ça :
Mets à jour le nom vers 'John-Kevin' pour l'enregitrement dont le nom est 'John-Kevin'.
21/08/2015, 22h57
mathisdu42

2 pièce(s) jointe(s)

Citation:

Envoyé par jopopmk

Pas besoin de t'excuser, on peut pas tout connaitre et en info on en apprend tous les jours ;)

J'aimerai connaitre la définition de ta table (champs, types et index).
Les données c'est juste ce qu'il y a dedans (chaque valeur de chaque champ pour chaque ligne).
Tu dois pouvoir obtenir tout ça avec la commande mysqldump.

Une requête SQL peut facilement se traduire en français. Dans ton cas c'est :
Mets à jour (UPDATE) le nombre de jetons (SET Jetons) pour l'utilisateur ayant tel nom (WHERE username = John-Kevin).
Si on met de coté les Jetons ta première requête sonne comme ça :
Mets à jour le nom vers 'John-Kevin' pour l'enregitrement dont le nom est 'John-Kevin'.

En effet ;)

Pour les champs et types : (Je sais pas si c'est cela que tu recherche.)

Pièce jointe 186050

Type de table : InnoDB

Dans l'onglet index :

Pièce jointe 186051

J'espère que ça va suffire :p

Oui c'est sûr que ça se traduit facilement mais comme tu dis, il doit y avoir un bug puisque "Mets à jour (UPDATE) le nombre de jetons (SET Jetons) pour l'utilisateur ayant tel nom (WHERE username = John-Kevin)." Semble logique...
22/08/2015, 11h17
mathisdu42

Un miracle ?

Finalement, après plusieurs prises de tête, j'ai voulu réessayer de mettre le code que tu m'avais donner, puis, au passage modifier Jetons en smallint. Et ça a marcher ... :)

En tout cas merci à toi de m'avoir aidé ! ;)

Merci !
22/08/2015, 12h37
Sankasssss

Citation:

Envoyé par jopopmk

Par contre, pour être plus propre, je te conseille de passer par des requêtes paramétrées.

:plusser: très important si vous ne voulez pas que le programme plante quand il y aura un apostrophe dans la chaîne concaténée.

http://webman.developpez.com/article...rameter/vbnet/

Il y a aussi l'utilisation de ExecuteNonQuery au lieu de ExecuteReader qui est plus approprié et l'enlèvement du MySqlConnection.Clone() qui ne sert à rien sans stockage de la connection de retour.