Problemes de requêtes

**acheo** · 04/10/2011, 05h54

Bonjour,

J'ai une table avec laquelle je remplis un dictionnaire:

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
foreach (DataRow row in myLocalDataTable.Rows)
                {
                    Dictionary<String, Object> parameterList = new Dictionary<String, Object>(); 
                    parameterList["idsResultSessionLap"] = row["idsResultSessionLap"];
                    parameterList["idsResultSession"] = row["tblResultSessionLap.idsResultSession"];
                    parameterList["lngCycle"] = row["lngCycle"];
                    parameterList["lngLapNumber"] = row["lngLapNumber"];
                    parameterList["dtmTestEnd"] = row["dtmTestEnd"];
                    parameterList["blnECEnable"] = row["blnECEnable"];
                    parameterList["strECMode"] = row["strECMode"];
                    parameterList["dblECTemperature"] = row["dblECTemperature"];
                    parameterList["intECSegment"] = row["intECSegment"];
                    parameterList["blnAllTest"] = row["blnAllTest"];
                    parameterList["blnSuccess"] = row["blnSuccess"];
                    parameterList["blnNewLapTranfered"] = row["blnNewLapTranfered"];
                    parameterList["blnReadyToTransfer"] = row["blnReadyToTransfer"];

puis, j'insere certains enregistrements:

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
String queryResultSessionLap = "INSERT INTO ResultSessionLap ";
                queryResultSessionLap += " (idsResultSession, lngCycle, lngLapNumber, dtmTestStart, dtmTestEnd, blnECEnable, strECMode, dblECTemperature, intECSegment, blnAllTest, ";
                queryResultSessionLap += " blnSuccess) ";
                queryResultSessionLap += " VALUES (" + iParameterList["idsResultSession"] + "," + iParameterList["lngCycle"] + ",";
                queryResultSessionLap += iParameterList["lngLapNumber"] + ", CONVERT(DATETIME, " + iParameterList["dtmTestStart"] + ")," + ", CONVERT(DATETIME, " + iParameterList["dtmTestEnd"] + "),";
                queryResultSessionLap += iParameterList["blnECEnable"] + ", CONVERT(NVARCHAR(255), " + iParameterList["strECMode"] + ")," + iParameterList["dblECTemperature"] + ",";
                queryResultSessionLap += iParameterList["intECSegment"] + ", CONVERT(BIT, " + iParameterList["blnAllTest"] + ")," + " CONVERT(BIT, " + iParameterList["blnSuccess"] + "))";
 
  SqlConnection myConnection = null;
            try
            {
                String connectionString = @"Data Source=SRVSQL2\DEV;Initial Catalog=ATE_Results;User ID=sa;Password=sadevpass";
                myConnection = new SqlConnection(connectionString);
                SqlCommand myCommand = new SqlCommand(iQuery, myConnection);
                myConnection.Open();
                myCommand.ExecuteNonQuery();
 
 
            }
            catch (Exception e)
            {
                Console.WriteLine("L'erreur suivante a été rencontrée :" + e.Message);
            }
 
            finally
            {
                myConnection.Close();
            }

Mon probleme est le suivant: bien souvent, des valeures du dictionnaire sont vides et ma requete lancée au serveur ne passe pas. La chaine de caractere qui est passée est:

Code :

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

INSERT INTO ResultSessionLap  (idsResultSession, lngCycle, lngLapNumber, dtmTestStart, dtmTestEnd, blnECEnable, strECMode, dblECTemperature, intECSegment, blnAllTest,  blnSuccess)  VALUES (30,1,20, CONVERT(DATETIME, 2011-07-06 09:57:49),, CONVERT(DATETIME, ),False, CONVERT(NVARCHAR(255),),,, CONVERT(BIT, True), CONVERT(BIT, False))

On voit bien qu'il manque certaines valeures. De quelle façon puis-je modifier l'initialisation du dictionnaire pour que la requête fonctionne?

J'obtiens l'exception suivante : {"Incorrect syntax near '09'."}

merci

**SaumonAgile** · 04/10/2011, 09h27

Ce n'est pas le dictionnaire qui pose problème, c'est ton INSERT. C'est une catastrophe du point de vue de la sécurité/maintenance/efficacité.
Utilise des paramètres SQL pour passer proprement les valeurs dans ta requête.
Plus d'infos ici: http://webman.developpez.com/article...ameter/csharp/

**tomlev** · 04/10/2011, 09h37

Le problème n'est pas seulement les valeurs vides... par exemple je suis pas convaincu que ce bout là fonctionne :

Code :

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

CONVERT(DATETIME, 2011-07-06 09:57:49)

De toutes façons il faut utiliser des requêtes paramétrées pour éviter ce genre de problèmes (cf. ce tuto)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
string queryResultSessionLap = "INSERT INTO ResultSessionLap ";
queryResultSessionLap += " (idsResultSession, lngCycle, lngLapNumber, dtmTestStart, dtmTestEnd, blnECEnable, strECMode, dblECTemperature, intECSegment, blnAllTest, blnSuccess) ";
queryResultSessionLap += " VALUES (@idsResultSession, @lngCycle, @lngLapNumber, @dtmTestStart, @dtmTestEnd, @blnECEnable, @strECMode, @dblECTemperature, @intECSegment, @blnAllTest, @blnSuccess)";
 
...
 
SqlCommand myCommand = new SqlCommand(iQuery, myConnection);
myCommand.Parameters.AddWithValue("@idsResultSession", iParameterList["idsResultSession"]);
myCommand.Parameters.AddWithValue("@lngCycle", iParameterList["lngCycle"]);
...
 
myCommand.ExecuteNonQuery();

C'est plus lisible, il n'y a pas de problèmes de format, pas de risque d'injection SQL, et c'est plus performant si tu l'exécutes souvent. Bref, tu gagnes sur tous les tableaux...

EDIT: oops, grillé

**acheo** · 04/10/2011, 12h51

Bon, j'ai implémenté la procédure mais je un probleme de DateTime que je ne peux résoudre. J'ai essayé plusieurs combinaisons (SqlDbType.DateTime, SqlDbType.DateTime2, etc..) et je ne trouve pas.

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
SqlConnection maConnexionSQLsrv; 
            SqlCommand myCommand; 
            String ConnexionSQL; 
 
            //Récupération de la chaine de connexion au serveur de base de données 
            ConnexionSQL = @"Data Source=SRVSQL2\DEV;Initial Catalog=ATE_Results;User ID=sa;Password=sadevpass";
            maConnexionSQLsrv = new SqlConnection(ConnexionSQL); 
 
            String queryResultSessionLap = "INSERT INTO ResultSessionLap ";
            queryResultSessionLap += " (idsResultSession, lngCycle, lngLapNumber, dtmTestStart, dtmTestEnd, blnECEnable, strECMode, dblECTemperature, intECSegment, blnAllTest, ";
            queryResultSessionLap += " blnSuccess) ";
            queryResultSessionLap += " VALUES (@idsResultSession,@lngCycle, @lngLapNumber, @dtmTestStart, ";
            queryResultSessionLap += " @dtmTestEnd, @blnECEnable, @strECMode, @dblECTemperature, @intECSegment, @blnAllTest, @blnSuccess)";
 
 
            myCommand = new SqlCommand(queryResultSessionLap, maConnexionSQLsrv); 
 
            //Parameters
            myCommand.Parameters.Add(new SqlParameter("@idsResultSession", SqlDbType.Int, 4));
            myCommand.Parameters.Add(new SqlParameter("@lngCycle", SqlDbType.Int, 4));
            myCommand.Parameters.Add(new SqlParameter("@lngLapNumber", SqlDbType.Int, 4));
 
            myCommand.Parameters.Add(new SqlParameter("@dtmTestStart", SqlDbType.DateTime, 32));
            myCommand.Parameters.Add(new SqlParameter("@dtmTestEnd", SqlDbType.DateTime, 32));
            myCommand.Parameters.Add(new SqlParameter("@blnECEnable", SqlDbType.Bit, 4));
 
            myCommand.Parameters.Add(new SqlParameter("@strECMode", SqlDbType.NVarChar, 255));
            myCommand.Parameters.Add(new SqlParameter("@dblECTemperature", SqlDbType.Float, 25));
            myCommand.Parameters.Add(new SqlParameter("@intECSegment", SqlDbType.SmallInt, 4));
 
            myCommand.Parameters.Add(new SqlParameter("@blnAllTest", SqlDbType.Bit, 8));
            myCommand.Parameters.Add(new SqlParameter("@blnSuccess", SqlDbType.Bit, 25)); 
 
 
 
            //Parameters initialization
            myCommand.Parameters["@idsResultSession"].Value = Convert.ToInt32(iParameter["idsResultSession"]);
            myCommand.Parameters["@lngCycle"].Value = Convert.ToInt32(iParameter["lngCycle"]);
            myCommand.Parameters["@lngLapNumber"].Value = Convert.ToInt32(iParameter["lngLapNumber"]);
 
            myCommand.Parameters["@dtmTestStart"].Value = Convert.ToDateTime(iParameter["dtmTestStart"]);
            myCommand.Parameters["@dtmTestEnd"].Value = Convert.ToDateTime(iParameter["dtmTestEnd"]);
            myCommand.Parameters["@blnECEnable"].Value = Convert.ToByte(iParameter["blnECEnable"]);
 
            myCommand.Parameters["@strECMode"].Value = Convert.ToString(iParameter["strECMode"]);
            myCommand.Parameters["@dblECTemperature"].Value = Convert.ToDouble(iParameter["dblECTemperature"]);
            myCommand.Parameters["@intECSegment"].Value = Convert.ToInt32(iParameter["intECSegment"]);
 
            myCommand.Parameters["@blnAllTest"].Value = Convert.ToByte(iParameter["blnAllTest"]);
            myCommand.Parameters["@blnSuccess"].Value = Convert.ToByte(iParameter["blnSuccess"]); 
 
            try 
            { 
 
              myCommand.Connection.Open(); 
              myCommand.ExecuteNonQuery(); 
              myCommand.Connection.Close(); 
            } 
            catch (SqlException ex) 
            {
                Console.WriteLine("Ex: " + ex);  // shows exception message in console if any errors occured             
            }

**BenoitM** · 04/10/2011, 13h46

La colonne dans la table est bien défini en datetime?

quelle est l'erreur?

**acheo** · 04/10/2011, 14h10

Voila! Merci!

{System.Data.SqlTypes.SqlTypeException: SqlDateTime overflow. Must be between 1/1/1753 12:00:00 AM and 12/31/9999 11:59:59 PM.
at System.Data.SqlTypes.SqlDateTime.FromTimeSpan(TimeSpan value)
at System.Data.SqlTypes.SqlDateTime.FromDateTime(DateTime value)
at System.Data.SqlClient.MetaType.FromDateTime(DateTime dateTime, Byte cb)
at System.Data.SqlClient.TdsParser.WriteValue(Object value, MetaType type, Byte scale, Int32 actualLength, Int32 encodingByteSize, Int32 offset, TdsParserStateObject stateObj)
at System.Data.SqlClient.TdsParser.TdsExecuteRPC(_SqlRPC[] rpcArray, Int32 timeout, Boolean inSchema, SqlNotificationRequest notificationRequest, TdsParserStateObject stateObj, Boolean isCommandProc)
at System.Data.SqlClient.SqlCommand.RunExecuteReaderTds(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, Boolean async)
at System.Data.SqlClient.SqlCommand.RunExecuteReader(CommandBehavior cmdBehavior, RunBehavior runBehavior, Boolean returnStream, String method, DbAsyncResult result)
at System.Data.SqlClient.SqlCommand.InternalExecuteNonQuery(DbAsyncResult result, String methodName, Boolean sendToPipe)
at System.Data.SqlClient.SqlCommand.ExecuteNonQuery()
at DataNotUploadedOnSqlServer.DataBaseGateway.insertResultSessionLap(Dictionary`2 iParameter) in C:\Documents and Settings\mbazinet\My Documents\Visual Studio 2010\Projects\DataNotUploadedOnSqlServer\DataNotUploadedOnSqlServer\DataBaseGateway.cs:line 212
at DataNotUploadedOnSqlServer.Program.uploadRecords(Dictionary`2 iParameterList) in C:\Documents and Settings\mbazinet\My Documents\Visual Studio 2010\Projects\DataNotUploadedOnSqlServer\DataNotUploadedOnSqlServer\Program.cs:line 217
at DataNotUploadedOnSqlServer.Program.Main(String[] args) in C:\Documents and Settings\mbazinet\My Documents\Visual Studio 2010\Projects\DataNotUploadedOnSqlServer\DataNotUploadedOnSqlServer\Program.cs:line 130}

**BenoitM** · 04/10/2011, 14h34

Beuh mets un break point et regarde ce que vaut

Code :

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

1
2
Convert.ToDateTime(iParameter["dtmTestStart"]);
Convert.ToDateTime(iParameter["dtmTestEnd"]);

et si la table est bien definie en datetime

**Bluedeep** · 04/10/2011, 15h05

Si tes colonnes Sql Server et tes variables .net sont typées proprement, il n'est pas utile de préciser le DbType pour créer le paramètre.

En général, c'est même complétement inutile pour des paramétres en entrée.

**acheo** · 04/10/2011, 15h38

J'ai enlevé le Convert et tout se passe bien.

Un gros merci!

**BenoitM** · 04/10/2011, 15h48

Moi j'aurai regarder pourquoi le ConvertTo pose problème plutot que de le supprimer

**Bluedeep** · 04/10/2011, 16h01

Envoyé par BenoitM

Moi j'aurai regarder pourquoi le ConvertTo pose problème plutot que de le supprimer

Dans la mesure où il ne sert strictement à rien ici je ne vois pas trop l'interêt.

**BenoitM** · 04/10/2011, 16h42

1) Comprendre pourquoi une opération plante est toujours utile, même si celle-ci ne sert à rien et qu'on la supprime après

2) Oui c'est vrai qu'ici iParameter[] sont correctement typer,je pensais que c'était des strings et donc laisser la conversion en mode automatique n'est pas une bonne idée ("05/11/2011" faudra-t-il le 05 novembre ou le 11 mai...)

ps : donc en analysant le problème, il aurait enlever les ConvertTo parce qu'ils sont déjà typer et donc inutile

Problemes de requêtes

C#

Discussions similaires

Partager

Partager