Discussion :

[bluekat]

Bonjour,

Je souhaite ajouter une "Localite" à ma BDD access. Cette dernière génère l'ID, il me faut donc récupérer celui-ci.

Voici mon code :

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
Dim con As New OleDb.OleDbConnection
        Dim cmd As New OleDb.OleDbCommand
        Dim dr As OleDbDataReader
        con.ConnectionString = ModuleGeneral.cheminBDD
        con.Open()
        cmd = New OleDbCommand("insert into Localite(ID_Pays,CP,Denomination) values(" & ID_Pays & "," & CP & ",'" & NomComune & "')", con)
        cmd.ExecuteNonQuery()
        cmd = New OleDbCommand("SELECT ID_Localite FROM Localite WHERE ID_Pays = " & ID_Pays & " AND CP = " & CP & " AND Denomination ='" & NomComune & "'", con)
        dr = cmd.ExecuteReader()
        While dr.Read
            Me.ID_Localite = dr("ID_Localite")
        End While
 
        con.Close()
        listeLocalite.Add(Me)
        MsgBox("Localité correctement ajoutée")

Etant donné qu'il n'y a qu'une localite repondant aux critères de recherches, me suis dit, récupéront la valeur de cette façon :

Code :

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

Me.ID_Localite = dr("ID_Localite")

cependant il faut l'entourer de while.

Code :

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

1
2
3
While dr.Read
            Me.ID_Localite = dr("ID_Localite")
        End While

Pourquoi ? Y a t'il moyen d'éviter les while ? Faut-il utiliser autre chose que OleDbDataReader lorsqu'on on ne recupère pas un enregistrement mais juste une valeur ? Une alternative pour récupérer l'ID ?

Merci beaucoup !

[calagan99]

Tu te pose en effet la bonne question.
Le DataReader n'a pas d'intérêt pour récupérer un seul enregistrement.

Utilise plutôt la fonction ExecuteScalar dont ton objet dbcommand.

D'autre part, dans ta requête, n'utilise pas de concaténation de chaînes. Elles sont non sécurisées et source d'erreurs.
Passe par des requêtes paramétrées.

[bluekat]

Merci pour executeScalar !!

Par contre tu développer "Elles sont non sécurisées et source d'erreurs" ?

Merci !

[calagan99]

De rien

En ce qui concerne les requêtes "non paramétrées".
Elles sont source d'erreur parce que la concaténation à base de "&", de quotes, etc., lorsqu'elles sont courtes, ça passe encore, mais dés que tu commences à avoir de longues requêtes le risque d'erreur est élevé. Sans compter les possibles guillemets et apostrophes dans les string utilisées, qui ne seront pas gérés.

Enfin, concernant la sécurité, les requêtes non paramétrées sont sensibles à l'injection SQL.
En utilisant des requêtes paramétrées, ton SGBD sait se protéger de ce type d'attaque.