Discussion :

[drake56]

Bonjour,

Je développe actuellement un webservice, je souhaiterai récupérer les informations de plusieurs tables que je regrouperai dans une seule.
Beaucoup d'insertion se passent très bien (135 sur 150 environ) et pour le reste j'ai cette erreur :

Un objet qui autorise la valeur Null doit posséder une valeur.

Comment dois-je procéder pour la résoudre ?

Voici différentes méthodes qui pourraient être utilisées pour résoudre mon problème :

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
        public void updateQuery(string query)
        {
            try
            {
                if (maConnection == null || maConnection.State != ConnectionState.Open)
                {
                    this.Connect();
                }
                // Objet Command
                SqlCommand command = new SqlCommand(query, maConnection);
                // Execution
                command.ExecuteNonQuery();
            }
            catch (Exception e)
            {
                Console.WriteLine(e.Message);
            }
            //fermeture connexion
            this.Disconnect();
        }

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
        public string insertAgd(string rqtexcde, DateTime rqtsollimdte, bool wrktyppnt, int origcpyaddrincde, string origcpylocdsc, int ctcincde, int agdincde, int agdelapsedel, DateTime agdfromdte, DateTime agdtodte, DateTime agddonedte, string wrkdsc, int wrkstsincde, bool pdasendpnt, string typexcde)
        {
 
            return "INSERT INTO [SOMTEST].[dbo].[w_agdpec] " +
            "([RqtExCde] "+
            ",[RqtSolLimDte] "+
           ",[WrkTypPnt] "+
           ",[OrigCpyAddrIncde] "+
           ",[OrigCpyLocDsc] "+
           ",[CtcInCde] "+
           ",[AgdInCde] "+
           ",[AgdElapseDel] "+
           ",[AgdFromDte] "+
           ",[AgdToDte] "+
           ",[AgdDoneDte] "+
           ",[WrkDsc] "+
           ",[WrkStsInCde] "+
           ",[PDASendPnt] "+
           ",[TypExCde] "+
           ",[RqtPecInCde] "+
           ",[RqtPecFromDte] "+
           ",[RqtPecToDte] "+
           ",[WrkExCde] "+
           ",[RqtNatExCde]) "+
     "VALUES " +
           "('" + rqtexcde + "' " +
           ",'" + rqtsollimdte + "' " +
           "," + convertToBool(wrktyppnt) + " " +
           "," + origcpyaddrincde + " " +
           ",'" + origcpylocdsc + "' " +
           "," + ctcincde + " " +
           "," + agdincde + " " +
           "," + agdelapsedel + " " +
           ",'" + agdfromdte + "' " +
           ",'" + agdtodte + "' " +
           ",'" + agddonedte + "' "+
           ",'" + wrkdsc + "' "+
           "," + wrkstsincde + " "+
           "," + convertToBool(pdasendpnt) + " " +
           ",'"+typexcde+"' " +
           ",NULL" +
           ",NULL " +
           ",NULL " +
           ",NULL " +
           ",NULL)";
        }

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
public List<AGD> ListAGD(string d, int ctcincde)
        {
            Console.WriteLine("Contact : " + ctcincde.ToString());
            DataSet monDataSet = som.getQuery(r.getListeAGD(d, ctcincde));
            List<AGD> maliste = new List<AGD>();
 
            try
            {
                foreach (DataRow dr in monDataSet.Tables[0].Rows)
                {
                    AGD row = new AGD();
 
                    row.WrkTypPnt = (bool)dr["WrkTypPnt"];
                    row.OrigCpyAddrIncde = (int)dr["OrigCpyAddrIncde"];
                    row.WrkDsc = dr["WrkDsc"].ToString();
                    row.OrigCpyLocDsc = dr["OrigCpyLocDsc"].ToString();
                    row.AgdElapseDel = (int)dr["AgdElapseDel"];
                    row.AgdFromDte = (DateTime)dr["AgdFromDte"];
                    row.AgdToDte = (DateTime)dr["AgdToDte"];
                    row.PDASendPnt1 = (bool)dr["PDASendPnt"];
                    row.AgdInCde = (int)dr["AgdInCde"];
                    row.WrkStsInCde = Convert.ToInt32(dr["WrkStsInCde"]);
                    row.RqtExCde = dr["RqtExCde"].ToString();
                    row.CtcInCde = (int)dr["CtcInCde"];
 
                    try
                    {
                        row.RqtSolLimDte = (DateTime)dr["RqtSolLimDte"];
                        row.AgdDoneDte = (DateTime)dr["AgdDoneDte"];
                    }
                    catch
                    { }
 
                    maliste.Add(row);
                    Console.WriteLine(row.ToString());
                    somtest.updateQuery(r.insertAgd(row.RqtExCde, (DateTime)row.RqtSolLimDte, (bool)row.WrkTypPnt, row.OrigCpyAddrIncde, row.OrigCpyLocDsc, row.CtcInCde, row.AgdInCde, row.AgdElapseDel, (DateTime)row.AgdFromDte, (DateTime)row.AgdToDte, (DateTime)row.AgdDoneDte, row.WrkDsc, row.WrkStsInCde, (bool)row.PDASendPnt1, row.TypExCde));
                }
 
            }
            catch (Exception e)
            {
                Console.WriteLine(e.Message);
            }
 
            return maliste;
        }

Merci d'avance pour votre aide

[Nathanael Marchand]

Manifestement y'a un truc qui n'est pas cohérent entre ton code et ta base.
Il faut déjà appliquer quelques bonnes pratiques avant d'y voir plus clair:
Première bonne pratique: Les requêtes parametrées!, ca t'évitera les concaténations de valeurs ce qui déjà n'est pas génial en performance mais en plus dangereux (injections SQL) et source d'un nombre considérable de problèmes!
http://webman.developpez.com/article.../sqlparameter/
http://johannblais.developpez.com/tu...s-donnees/#LIV
Seconde bonne pratique: pourquoi un try avec un catch vide?

[drake56]

Merci pour l'astuce je vais regarder ça.
Sinon pour le catch vide c'est ou oubli de ma part, je vais rajouter un message sur la console.

[drake56]

J'ai effectué les modifications conseillées mais toujours la même erreur et en enlevant le try/catch j'ai une autre qui est

Le cast spécifié n'est pas valide

.

[Nathanael Marchand]

Ben voila! Tu as ta réponse! Au lieu de caster les dates, regarde quelles têtes elles ont, elles sont peut être au format texte. Auquel cas, il faudrait faire un DateTime.Parse()

[drake56]

Merci en les affichant, j'ai toujours l'erreur mais j'en ai certainess qui sont nulles ou qui sont au 1er janvier 1900, le format me semble correct pourtant

Dès que j'ai une valeur null à insérer, l'insertion de la ligne échoue. Comment ça se fait car j'ai bien vérifié dans la base, le champs peut être null.