Discussion :

[thelpi]

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 :

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

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 :

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
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!

[tomlev]

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...

[Philippe Vialatte]

Ce ne serait pas plus simple de faire :

Code :

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

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();
...

[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 mais je peux me tromper

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

[Guulh]

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

Code :

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

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

[thelpi]

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

merci

[tomlev]

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