Mysql + Select Wher ComboBox1.Text.Split(" | ")

Version imprimable

Salut....

voila j'ai un combobox avec 3 valeurs :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
Try
            MysqlConn.Close()
            MysqlConn.Open()
            Dim Query As String
            Query = "select id, name,id_maker, id_types from product ORDER BY name ASC"
            COMMAND = New MySqlCommand(Query, MysqlConn)
            READER = COMMAND.ExecuteReader
            While READER.Read
                Dim sName = READER.GetString("name")
                Dim sMaker = READER.GetString("id_maker")
                Dim sTypes = READER.GetString("id_types")
                Dim sId = READER.GetString("id")
                '
                'ComboBox1.Items.Add(sName)
                ComboBox1.Items.Add(String.Format("{0} | {1} | {2}", sName, sMaker, sTypes))
 
            End While
        Catch ex As Exception
            MessageBox.Show(ex.Message)
        Finally
            MysqlConn.Dispose()
        End Try

Comment je fais pour faire un select mysql pour retrouver la valeur du combobox ? J'ai essayer ceci mais sans succes.

Code:

1
2
Dim Str() As String = ComboBox1.Text.Split(" | ")
        COMMAND.CommandText = "select id,name, foto,id_maker,id_types from product where name = '" & Str(0) & "' AND id_maker = '" & Str(1) & "' AND id_types = '" & Str(2) & "'"

24/04/2017, 17h06
jopopmk

Salut,

on repère facilement tes sujets, ils commencent toujours par MySql + xxx :mrgreen:

Bon -sans considérer tes requêtes qui me semblent suspectes :lol:- c'est quoi le souci exactement ?
Ton appel à Split est souligné en rouge ?
Son retour n'est pas bon ?
Ta deuxième requête ne renvoie pas les données escomptées ?
Il y a un plantage (quelle est l'exception et où est-elle levée) ?

Question subsidiaire : tu fais une requête paramétrée pour ton premier SELECT, pourquoi pas pour le second ?

Bonjour,

A priori j'opterais pour :

Citation:

Envoyé par jopopmk

Ton appel à Split est souligné en rouge ?

Code:

Dim Str() As String = ComboBox1.Text.Split('|')

devrait mieux fonctionner mais il conviendra faire un Trim des chaines retournées.

Maintenant pour ma part, je ferai une classe du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 public class product { public string id { get; set; } public string name { get; set; } public string id_maker { get; set; } public string id_types { get; set; } public override string ToString() { return name; } }
Puis pour chaque product lu dans la base, j'ajouterais un objet Product dans la ComboBox. Ca serait plus élégant à l'affichage et éviterait un bricolage de Split : A la sélection dans la ComboBox, on récupère directement le product sélectionné.
Cela pourrait même éviter de refaire une requête SQL pour récupérer les infos de product à condition bien sûr que l'objet product puisse être entièrement monté en mémoire (mais je ne sais pas ce que contient le champ foto).

ps : Pardon, c'est une classe C# mais facilement adaptable en VB.net

24/04/2017, 17h42
superthx

Citation:

Envoyé par jopopmk

Salut,

on repère facilement tes sujets, ils commencent toujours par MySql + xxx :mrgreen:

Bon -sans considérer tes requêtes qui me semblent suspectes :lol:- c'est quoi le souci exactement ?
Ton appel à Split est souligné en rouge ?
Son retour n'est pas bon ?
Ta deuxième requête ne renvoie pas les données escomptées ?
Il y a un plantage (quelle est l'exception et où est-elle levée) ?

Question subsidiaire : tu fais une requête paramétrée pour ton premier SELECT, pourquoi pas pour le second ?

Haha 😂 c'est clair au moiComboBox1.Text.Split(" | ")ns.

En fait j'ai découvert depuis que si je met "{0}|{1}|{2}" au lieu de "{0} | {1} | {2}" et
ComboBox1.Text.Split("|") Au lieu de ComboBox1.Text.Split(" | ")
Donc apparement un probleme dans mon cas c'est un probleme des espaces entre les barres....

Il renvoi pas la veleur, va savoir ce qu'il quand il y a des espaces. Mais je voudrais savoir pourquoi diable cela marche sans espace et pas avec espace...

Depuis que je scrute le forum je progresse a toute vitesse et sa fais plaisir d'avoir de l'aide quant on echoue.

Rien n'est souligner en rouge...
24/04/2017, 17h52
jopopmk

Citation:

Envoyé par superthx

[..]Mais je voudrais savoir pourquoi diable cela marche sans espace et pas avec espace...[..]

ebastien l'explique bien juste au-dessus, il faut penser à "trimer" (enlever les espaces en rab en début et en fin de string) chaque string du tableau renvoyé par Split. Il parle aussi d'une méthode qui consiste à mapper ta ligne dans un objet, qui a deux avantages intéressants :
- la manipulation de l'objet sera plus simple et évolutif,
- ça évite des soucis sur les données folklo (là, si un de tes champs contient un pipe ('|', ou 0x7C) ton programme plante gentiment).

Citation:
Envoyé par ebastien
Bonjour,

Code:

Dim Str() As String = ComboBox1.Text.Split('|')

Pas en rouge.

Maintenant pour ma part, je ferai une classe du genre :
Code:

1 2 3 4 5 6 7 8 9 10 11 public class product { public string id { get; set; } public string name { get; set; } public string id_maker { get; set; } public string id_types { get; set; } public override string ToString() { return name; } }
Puis pour chaque product lu dans la base, j'ajouterais un objet Product dans la ComboBox. Ca serait plus élégant à l'affichage et éviterait un bricolage de Split : A la sélection dans la ComboBox, on récupère directement le product sélectionné.
Cela pourrait même éviter de refaire une requête SQL pour récupérer les infos de product à condition bien sûr que l'objet product puisse être entièrement monté en mémoire (mais je ne sais pas ce que contient le champ foto).

ps : Pardon, c'est une classe C# mais facilement adaptable en VB.net
Je m'explique, avant ma requête faisait appel au nom direct de product. Mais si j'avais 2 produits Voiture, on savait pas lequel prendre directement. D'ou le bricolage de String, de cet façon j'ai dans mon combobox "Voiture | Renault | Megane" pour donner un exemple.

24/04/2017, 17h58
superthx

Citation:

Envoyé par jopopmk

ebastien l'explique bien juste au-dessus, il faut penser à "trimer" (enlever les espaces en rab en début et en fin de string) chaque string du tableau renvoyé par Split. Il parle aussi d'une méthode qui consiste à mapper ta ligne dans un objet, qui a deux avantages intéressants :
- la manipulation de l'objet sera plus simple et évolutif,
- ça évite des soucis sur les données folklo (là, si un de tes champs contient un pipe ('|', ou 0x7C) ton programme plante gentiment).

Ok si je pige bien le truc d'avant me donne Product mais il intègre tous déjà ?
Je connais pas trop C et VB. Donc comment je converti ce code en VB ?
24/04/2017, 18h21
ebastien

Citation:

Envoyé par superthx

Ok si je pige bien le truc d'avant me donne Product mais il intègre tous déjà ?
Je connais pas trop C et VB. Donc comment je converti ce code en VB ?

Un convertisseur de code C# vers VB.Net : Code Converter

Le "truc d'avant" te permet en effet de retrouver facilement un Product. L'objet Product intégrera ce que tu lui auras défini comme propriété. Donc éventuellement tous les champs de ta table (il faudra bien-sûr renseigner ces propriétés avec une requête SQL).
Maintenant attention quand même à la volumétrie. Si tu as un million de produits, ça peut poser problème de les monter en mémoire (ainsi que de les afficher dans une combo d'ailleurs ;))

Je comprends ta problèmatique d'affichage lié aux catégories de produit. Tu peux gérer cela facilement avec une classe puisque. lorsque tu insères un objet dans une ComboBox, la Combo appellera la méthode ToString() de l'objet pour afficher son libellé. Il suffit donc d'implémenter correctement cette méthode pour retourner ce que tu veux.

Soit dit en passant, si tu as une liste de produits avec des catégories de produit puis des modèles (Voiture/Renault/Mégane), tu peux les présenter dans un Control Treeview.

Finalement j'ai opter pour ceci.
Je sais pas si c'est ce que tu voulais me dire ebastien.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
        Dim connetionString As String = Nothing
        Dim connection As MySqlConnection
        Dim command As MySqlCommand
        Dim adapter As New MySqlDataAdapter()
        Dim ds As New DataSet()
        Dim i As Integer = 0
        Dim sql As String = Nothing
        sql = "SELECT id, CONCAT(name,' | ',id_maker,' | ',id_types) AS value FROM product ORDER BY name ASC"
        connection = New MySqlConnection(ConfigurationManager.ConnectionStrings("test.My.MySettings.testConnectionString").ToString)
        Try
            connection.Open()
            command = New MySqlCommand(sql, connection)
            adapter.SelectCommand = command
            adapter.Fill(ds)
            adapter.Dispose()
            command.Dispose()
            connection.Close()
            ComboBox1.DataSource = ds.Tables(0)
            ComboBox1.ValueMember = "id"
            ComboBox1.DisplayMember = "value"
        Catch ex As Exception
            MessageBox.Show("Can not open connection ! ")
        End Try

26/04/2017, 00h25
ebastien

Non, je ne pensais pas à ça et c'est vrai que je n'ai pas le reflèxe du Binding à un DataSource en Winform.
Ca doit aussi fonctionner comme ça, je ne sais pas, je n'ai jamais utilisé. Maintenant encore une fois, ça dépend des données et de l'utilisation que tu feras de ta ComboBox suite à la sélection.