Discussion :

[TCW78]

Je suis a modifier le code d'un ancien programmeur dans une application VB.NET et lorsque je vais dans l'UpdateCommand du TableAdapter, ça me donne quelquechose comme ça :

Code sql :

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

1
2
3
4
5
UPDATE       Table
SET                ALERTNO = ?, Date = ?, NOTES = ?
WHERE        (? = 1 AND ALERTNO IS NULL OR
                         ALERTNO = ?) AND (? = 1 AND Date IS NULL OR
                         Date = ?)

Et je me questionnais sur la validité du code... est-ce moi ou c,est pas normal les "?=1 AND...", qu'est-ce qu'un "1" fait la dedans?

Seulement pour vérifier si je suis fou ou non??!

[Bluedeep]

Ça ressemble à une chaine paramétrée pour un SGBD ne supportant pas les paramètres nommés (exemple ancienne version de PostGreSql).

Quel est le SGBD utilisé et, le cas échéant, le provider utilisé ?

Quel est le code d'exécution de cette requête ?

[TCW78]

Je suis en VB.NET (2005) et les données sont sur un serveur SQL 7.0.

J'ai un DataSet (.xsd) avec un paquet de tables, l'ancien employé a laissé le projet a moitié terminé. J'essai de trouver un sens au garbage que j'ai devant moi. J'ai une base, très basique, en VB.NET.

Dans mon DataSet j'ai des tables et des TableAdapters. Dans mon TableAdapter j'ai la propriété UpdateCommand qui se divise en trois :

• CommandType
• CommandText
• Parameters

Le CommandType contient le statement SQL, et c'est là que je retrouve le "? = 1" et je me demande ce qui devrait être vraiment là dans le CommandType...

[olsimare]

Bonjour.

C'est une ruse pour permettre de requêter soit en passant ALERTNO, date, les 2 ou aucun.
Il manque des parenthéses pour que cela soit bien clair :

Code sql :

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

1
2
3
4
5
6
7
8
WHERE 
(
( ? = 1 AND ALERTNO IS NULL ) OR ALERTNO = ?
)
AND
(
(? = 1 AND Date IS NULL) OR Date = ?
)

Il y a donc 4 façons d'utiliser cette requête, avec des paramétres différents pour récupérer les enregistrements avec :
ALERTNO = null et Date = null
ALERTNO non null et Date = null
ALERTNO null et Date non null
ALERTNO non null et Date non null
A la volée, je suppose que ceux sont 2 champs de filtre facultatif dans une form.

C'est toute la difficulté de gérer les champs Nullable car comme on le sait Null c'est rien, donc à part faire un "IS NULL", pour les récupérer c'est pas évident.

Cdt.

[TCW78]

J'ai continué de lire sur le sujet, ça me semble beaucoup plus facile de gérer les update et insert à partir d'une stored procedure.

Est-ce que je me trompe?

[TCW78]

Dans mon form j'ai le code suivant pour mettre à jour l'enregistrement à l'écran :

Code :

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

1
2
3
4
 
Me.Validate()
Me.TBLKekChoseSBindingSource.EndEdit()
Me.TBLKekChoseTableAdapter.Update(Me.KekChoseDataSet.TBLKekChose)

Et ça me donne l'erreur suivante :
ERROR [07S01] [Microsoft][ODBC SQL Server Driver]Utilisation non valide du paramètre par défaut

Au début, je me disais que c'était peut-être le UpdateCommand qui n'était pas fait correctement...

[olsimare]

Bonjour.

Pour les proc stockées, ça me semble pas forcément plus facile mais par contre plus difficile à maintenir à terme --> tu ajoutes un champs à une table, il faut que tu modifies ta proc (insert, update et delete en fait) et le code du projet.
Donc 3 impacts : table + proc + projet.
avec autant de sources d'erreurs possibles.

La proc stockée, je la vois plus comme un traitement de données que l'on peut déléguer complétement à la base (--> type traitement batch par exemple), ou à laquelle on confie une tâche d'agrégation de donnée (j'entend par là fournir un résultat consolidée de n données sur n table), qui n'aurait pas de plus-value étant faite dans l'appli et permettrait de limiter le volume remonter par la base (exemple : balance des crédits/débits d'un client calculées à partir des mouvements sur le compte --> pas la peine de remonter toutes les lignes de mouvements à l'appli pour faire du cumul conditionné alors qu'un proc peut le faire et ne rendre que le résultat).

Par contre pour l'erreur, je sèche ...

Cdt.

[TCW78]

J'ai créé une stored proc car je ne trouvais aucune autre solution, j'ai monté le code suivant :

Code :

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

1
2
3
4
5
6
7
8
9
Dim myConnection As SqlConnection = New SqlConnection(strConnectionString)
Dim objCommand As SqlCommand = New SqlCommand("usp_UpdateTable", myConnection)
 
        objCommand.CommandType = CommandType.StoredProcedure
        objCommand.Parameters.AddWithValue("@NoID", Me.txtNoID.Text)
        myConnection.Open()
        Dim reader As SqlDataReader = objCommand.ExecuteReader()
        reader.Close()
        myConnection.Close()

Maintenant tout fonctionne. Les mises-à-jour sont plus simples avec une SP.

Merci

[olsimare]

Bonjour.

J'ai vu ton autre post sur l'alim du Datagrid, j'en dis que la proc stockée pour ce que tu veux faire c'est inutile et que finalement ça va te faire trimer comme une bête pour rien !

Replonges-toi dans le dataadapter et le commandbuilder, ça te facilitera la vie.

Cdt.