Condition sur un textBox qui renvoi une requête vers MYSQL

Version imprimable

Bonjour,

Le but c'est d'envoyer une requête MYSQL pour verifier si la valeur saisie dans le textBox se trouve déjà dans ma colonne (Numéro de pièce) du tableau (pieces)

ça sera donc une condition sur mon textBox
Si je mets RIEN dans mon textBox ça va me permettre d'ajouter autant de valeur (DIFFÉRENTE) int sur ma base de données Mysql
Sinon il vérifie déjà si la nouvelle valeur saisie ne se trouve déjà dans ma colonne

un truc du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 if (textBox.Text = "") { MessageBox.Show("Veuillez entrez le numéro de pièce s'il vous plaît !"); } else if { string insertQuery = "SELECT `Numéro de Pièce` FROM `pieces` WHERE 1"; if (textBox == executeMyQuery(insertQuery)) // ouvrir et fermer une connexion vers ma base de donnée { MessageBox.Show("Pièce déjà faite !"); } else { string qs= "INSERT INTO `pieces`(`Numéro de Pièce`) VALUES('" + this.textBox.Text + "'); executeMyQuery(qs); }
MERCI D'AVANCE

13/06/2018, 09h18
tatayo

Bonjour,
Quelle est la question ?

Tatayo.

Salut tatayo et Merci pour ta réponse

PS : le code en dessous ne marche pas mais je veux réaliser les mêmes requêtes

Me question est la suivante : comment faire une condition sur un textBox et envoie une requete vers ma BDD !

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
if (textBox.Text = "")
{
    MessageBox.Show("Veuillez entrez le numéro de pièce s'il vous plaît !");
}
else if
{              
     string insertQuery = "SELECT `Numéro de Pièce` FROM `pieces` WHERE 1";
     if (textBox == executeMyQuery(insertQuery)                  // ouvrir et fermer une connexion vers ma base de donnée
    {
           MessageBox.Show("Pièce déjà faite !");
    }
    else
    {
           string insertQuery = "INSERT INTO `pieces`(`Numéro de Pièce`) VALUES('" + this.textBox.Text + "');
           executeMyQuery(insertQuery);                 
     }
}

13/06/2018, 09h56
tatayo
Le code ne marche pas, ce n'est pas une erreur…

Voici la requête qui est lancée:
Code:

1 2 SELECT `Numéro de Pièce` FROM `pieces` WHERE 1
A quel moment envoies-tu le numéro de pièce à MySQL ? Jamais. Mais tu sais comment faire (ligne 15 !).

Accessoirement, je pense que la ligne 15 ne compile pas, car tu déclares une variable insertQuery qui existe déjà (déclarée ligne 8).

Tatayo.
13/06/2018, 09h59
popo

Je ne suis pas familier de MySQL mais cette requête me parait bizarre :

Citation:

SELECT `Numéro de Pièce` FROM `pieces` WHERE 1

Where 1 !
Il ne manquerait pas quelque chose ?

D'ailleurs si tu veux réellement tester le numéro de pièce ne serait-il pas plus judicieux de faire ton where dessus.
Genre Where `Numéro de Pièce` = '" + textBox.Text + "'.

Sinon, deux conseils :
1) Évite les nom de champs avec des espaces ou des caractères accentués.
2) Passe par des requêtes paramétrées.
13/06/2018, 10h02
EL.Mardi

Au-delà si le code marche ou pas, ce que je veux c'est réaliser une condition de requete sur un textBox :

Code:

if (textBox == executeMyQuery(insertQuery))

la syntaxe ne marche pas car je l'ai inventer et j'en cherche une qui fait le même boulot

Code:

SELECT `Numéro de Pièce` FROM `pieces` WHERE 1

me permet de sélectionner la colonne entière (Pièce) du tableau (pieces)
et après je veux chercher dans LE RÉSULTA de la requête (SELECT) si ce numéro est déja saisi ou pas

MERCI
13/06/2018, 10h21
tatayo

Pour la condition dans la requête, vois la réponse de popo.
Maintenant tu aurais du trouver tout seul. Tu sais passer un paramètre pour la requête d'insertion, tu sais donc faire de même pour le SELECT.
Concernant le retour de la requête, on ne connais pas la méthode executeMyQuery, on ne peut donc pas deviner ce qu'elle renvoie: Un booléen qui indique si la requête a fonctionné, une collection d'objet qui représente le résultat du select, autre chose ?

Tatayo.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
        public void executeMyQuery(string query)
        {
            try
            {
                openConnection();
                command = new MySqlCommand(query, connection);
 
                if (command.ExecuteNonQuery() == 1)
                    MessageBox.Show("Enrée rajoutée à la base de donnée");
                else
                    MessageBox.Show("Erreur de connection avec la base de donnée !");
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.Message);
            }
            finally
            {
                closeConnection();
            }
        }

Code:

1
2
3
4
5
        public void openConnection()
        {
            if (connection.State == ConnectionState.Closed)
                connection.Open();
        }

Code:

1
2
3
4
5
        public void closeConnection()
        {
            if (connection.State == ConnectionState.Open)
                connection.Close();
        }

13/06/2018, 10h38
tatayo

Donc cette méthode ne renvoie rien (void).
Tu ne peux donc pas comparer rien avec la saisie de l'utilisateur.
Comme en plus elle ferme la connexion après lancement de la requête…

Après l'exécution de la requête, il faut récupérer un reader qui te permet de parcourir le résultat. Et ne pas fermer la connexion.

Tatayo.
13/06/2018, 10h41
popo

Encore une fois, il te manque la condition sur le numéro de la pièce !
C'est la base !!!!

Code:

SELECT `Numéro de Pièce` FROM `pieces` WHERE 1 AND `Numéro de Pièce` = '" + textBox.Text + "'

Ensuite :

Code:

if (textBox == executeMyQuery(insertQuery))

Comment une textbox peut être égale au résultat de l'exécution d'une requête.
D'aileurs, ça ne doit même pas compiler en l'état si ta routine renvoie void.

Donc soit tu renvoie un booléen pour dire que tu as trouvé, soit tu renvoie le nombre d'enregistrement que tu as trouvé et tu vérifié qu'il est à 1 (ce serait le plus logique vu le nom de ta routine)
Mais certainement pas void comme l'a déja dit tatayo.

En passant, tatayo :
Si le but est simplement de vérifier l'existence de l'enregistrement, ça ne sert à rien de parcourir le reader.
Ce qu'il a fait devrait suffire.
Par contre, il est vrai que ExecuteNonQuery n'aurait pas été mon premier choix.
J'aurais appelé ExecuteReader pour obtenir le reader puis HasRow sur le reader.
13/06/2018, 10h59
ZenZiTone

Citation:

Envoyé par EL.Mardi

je veux chercher dans LE RÉSULTA de la requête (SELECT) si ce numéro est déja saisi ou pas

Tu fais une requête qui te retourne les enregistrements dont le numéro de pièce est égal à ta saisie : SELECT * FROM PIECES WHERE NUMERO_PIECE = TextBoxPiece.Text

Si ta requête retourne un résultat c'est que la pièce existe déjà.
13/06/2018, 11h18
tatayo

Citation:

Envoyé par popo

En passant, tatayo :
Si le but est simplement de vérifier l'existence de l'enregistrement, ça ne sert à rien de parcourir le reader.
Ce qu'il a fait devrait suffire.
Par contre, il est vrai que ExecuteNonQuery n'aurait pas été mon premier choix.
J'aurais appelé ExecuteReader pour obtenir le reader puis HasRow sur le reader.

Ca dépend, si cette méthode ne sert qu'à ce cas particulier, ou si elle est utilisée pour tout type de requête. Et comme il semble l'utiliser tant pour lire dans la base qu'y ajouter des lignes, je penche pour le deuxième cas.

Dans le premier cas je suis d'accord (mais il faut renommer la méthode, et changer les messages d'erreur), et pas dans le second (et il faut aussi changer les messages d'erreur).

Tatayo.
15/06/2018, 10h45
popo

Et sinon El.Mardi, il faudrait penser à lire les règles du forum.
Poster la même demande dans deux sections différentes du forum, à la limite ça peux se comprendre si on a pas lu ces fameuses règles et qu'on se rend compte qu'on s'est trompé.
Mais poster trois fois la même demande dont deux fois au même endroit, c'est carrément de l'abus.

https://www.developpez.net/forums/d1...te-vers-mysql/
https://www.developpez.net/forums/d1...te-vers-mysql/
https://www.developpez.net/forums/d1...te-vers-mysql/

Fuseau horaire GMT +2. Il est actuellement 10h31.