Problème de requètes parametrés avec DAC

Version imprimable

salut à tous,

j'ai réalisé une application c#/SQLServer qui fonctionne bien, mais depuis que je suis passé à l'indépendance vis à vis du type de base de données, je ne sais pas quel stratégie adopté pour mes requètes paramètrés.

En gros, j'ai un objet commande qui dans un premier temps se prépare (définition du nom et du type des paramètres) puis plus loin dans le code je lui renseigne les valeurs des paramètres et je l'exécute. ces 2 parties se font dans 2 méthodes différentes.

Voila mon code actuel, qui me parait complètement absurde (je pense que la je part tout l'intérèt de "préparer" la commande)

Méthode de préparation de l'objet commande "cmdNonQuery"

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
IDbDataParameter parameter;
string[] paramsName = { "@id", "@ref", "@name", "@prtref", "@prtid", "@publish" };
DbType[] paramsType = { DbType.Int32, DbType.String, DbType.String, DbType.String, DbType.Int32, DbType.Boolean };
            for (int i = 0; i < 6; i++)
            {
                parameter = cmdNonQuery.CreateParameter();
                parameter.ParameterName = paramsName[i];
                parameter.DbType = paramsType[i];
                cmdNonQuery.Parameters.Add(parameter);
            }
cmdNonQuery.Prepare();

Méthode d'exécution de la commande

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
IDbDataParameter parameter;
            for (int i = 0; i < 6; i++)
            {
                parameter = (IDbDataParameter)cmdNonQuery.Parameters[i];
                switch (i)
                {
                    case 0:
                        parameter.Value = int.Parse(id);
                        break;
 
                    case 1:
                        parameter.Value = reader.GetValue(0).ToString();
                        break;
 
                    case 2:
                        parameter.Value = reader.GetValue(1).ToString();
                        break;
 
                    case 3:
                        parameter.Value = reader.GetValue(2).ToString();
                        break;
 
                    case 4:
                        parameter.Value = parentId;
                        break;
 
                    case 5:
                        parameter.Value = true;
                        break;
                }
                cmdNonQuery.Parameters[i] = parameter;
            }
cmdNonQuery.ExecuteNonQuery();

Je pense bien qu'il y'ait une meilleur solution mais je n'arrive pas à la trouver :(

Merci de votre aide!

09/07/2008, 17h39
tomlev

Citation:

Envoyé par thelpi

Voila mon code actuel, qui me parait complètement absurde (je pense que la je part tout l'intérèt de "préparer" la commande)

Ben non, en faisant comme ça la commande n'est parsée qu'une fois (lors du Prepare), et après tu changes seulement les valeur des paramètres. Donc si tu exécutes souvent cette requête, tu devrais gagner en performance grâce à cette méthode.
Pour ce qui est de l'"absurdité" de ton code, moi ça ne me choque pas... par contre je ne vois pas trop l'intérêt des boucles for dans ce contexte, ça rend juste le code un peu moins lisible...

Ce ne serait pas plus simple de faire :

Code:

1
2
3
4
5
6
7
 
cmd.Parameters.Add("@id", SqlDbType.Int32);
cmd.Parameters.Add("@ref", SqlDbType.String);
....
cmd.Parameters("@id").Value = id;
cmd.Parameters("@ref").Value = reader.GetValue(0).ToString();
...

:mouarf:

09/07/2008, 17h52
thelpi

OK merci les gars, je te fait confiance tomlev si tu me dit que mon Prepare est toujours utile ;)

pvialatte, en fait je crois qu'avec des command issue IDbCommand c'est pas possible, la méthode cmd.parameters.add() ne peut prendre qu'un objet en argument :cry: mais je peux me tromper :D

c'est pour cette raison que je fait ce truc un peu bizarre...

merci en tout cas de vous ètre pencher la dessus, bonne journée à vous

D'où ma méthode de la mort :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
public static IDbDataParameter AddParameter(IDbCommand cmd, string paramName, DbType dbType, ParameterDirection paramDirection)
{
	IDbDataParameter result = cmd.CreateParameter();
	cmd.Parameters.Add(result);
	result.Direction = paramDirection;
	result.DbType = dbType;
	result.ParameterName = paramName;
	return result;
}
// ...
IDbCommand cmd = conn.CreateCommand();
AddParameter(cmd, "hoho", DbType.Int32, ParameterDirection.Input);

J'en aurais même fait une extension method si j'utilisais C# 3 :)

09/07/2008, 18h24
thelpi

Hum ça me plait ça :D je te le pique, même si la "direction" je connait pas...

merci
09/07/2008, 18h32
tomlev

ben c'est juste pour indiquer qu'un paramètre est en entrée ou en sortie...

j'ai remplacé par ça moi :

Code:

1
2
3
4
5
6
7
8
9
 
public void AddParam(ref IDbCommand cmd, string nameParam, DbType typeParam, int sizeParam)
        {
            IDbDataParameter result = cmd.CreateParameter();
            result.DbType = typeParam;
            result.Size = sizeParam;
            result.ParameterName = nameParam;
            cmd.Parameters.Add(result);
        }

ça peut marcher ?

On peut simplifier encore :
Code:

1 2 3 4 5 6 7 8 9 public static IDbDataParameter AddParameter(IDbCommand cmd, string paramName, object value) { IDbDataParameter result = cmd.CreateParameter(); cmd.Parameters.Add(result); result.Value = value; result.ParameterName = paramName; return result; } AddParameter(cmd, "hoho", 15);
Par contre, j'ai pas idée de la fiabilité du truc (marche avec ODP en tous cas), parce que j'ignore si Ado.net impose à ceux qui l'implémentent de "deviner" le DbType en fonction de la Value.

09/07/2008, 18h39
Guulh

Citation:

Envoyé par thelpi

ça peut marcher ?

Le mot clé "ref" sert à rien, mais à part ça oui.