Discussion :

[StringBuilder]

Bonjour,

J'ai l'impression d'être tombé sur un trou dans la raquette, et j'aimerais votre avis sur la méthode la plus propre pour contourner le problème.

Dans ma base de données, j'ai une table contenant une colonne du type datetime2.
Cette colonne accepte les NULL.

Dans mon programme, je souhaite charger cette donnée dans une variable de type DateTime.

Avec mon SqlDataReader, si je fais dr.GetDatetime(x) alors tout fonctionne bien jusqu'à ce que j'aie une valeur nulle.
A ce moment la méthode GetDatetime plante sauvagement.

Avec le type SQL datetime tout court (sans le 2) je pouvais faire :
dr.GetSqlDatetime(x) et tester si le résultat avait la proproété Value égale = DBNull.Value

Seulement, dr.GetSqlDatetime(x) plante lorsque je tente de charger une colonne du type datetime2

Il n'y a évidement pas de méthode GetSqlDatetime2, ça aurait été trop beau…

Du coup, j'ai fait cette méthode d'extension, mais je trouve ça relativement crade...

Code csharp :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public static DateTime? GetNullableDateTime(this SqlDataReader dr, int column)
    {
        if (dr.IsDBNull(column))
        {
            return null;
        }
        else
        {
            return dr.GetDateTime(column);
        }
    }

Notamment, ceci ne fonctionnera pas si mon SqlCommand a pour optimisation "forwardonly".

Habituellement je gère les NULL de cette manière, que je trouve plus élégante :

Code csharp :

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

1
2
3
4
5
 
    public static int? OrNull(this SqlInt32 value)
    {
        return value.IsNull ? (int?)null : value.Value;
    }

[François DORIN]

Bonjour,
Du coup, j'ai fait cette méthode d'extension, mais je trouve ça relativement crade...

Code csharp :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
public static DateTime? GetNullableDateTime(this SqlDataReader dr, int column)
    {
        if (dr.IsDBNull(column))
        {
            return null;
        }
        else
        {
            return dr.GetDateTime(column);
        }
    }

Ce n'est pas crade. C'est même ainsi qu'il faut faire.

De mon côté, je n'ai pas de méthode d'extension, je fais à la mano en utilisant l'opérateur ternaire : value = dr.IsDBNull(column) ? (DateTime?)null : dr.GetDateTime(column)

Notamment, ceci ne fonctionnera pas si mon SqlCommand a pour optimisation "forwardonly".

C'est-à-dire ? J'avoue ne pas comprendre. Un SqlDataReader obtenu via un SqlCommand est par nature forwardonly. Et ce code est tout à fait fonctionnel dans le cas présent. Le forward only ne signifie pas qu'on ne peut pas appeler plusieurs méthodes sur la même colonne, il indique seulement qu'on ne peut pas revenir à une ligne précédente.

[StringBuilder]

Si c'est pas forward only, en tout cas j'ai déjà eu le souci par le passé avec une option de lecture qui interdisait :
- De lire les colonnes dans le désordre (forcément de manière incrémentielle)
- De lire plusieurs fois la même colonne

Du coup j'avais été obligé de faire des GetValues() à chaque ligne et me démerder avec un object[], ce qui ne me plaisait pas du tout.

Après, les méthodes d'extension je trouve ça beaucoup plus facile à écrire et à relire :

Code csharp :

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

1
2
 
value = dr.GetNullableDateTime(column);

Seul truc, c'est que ça fait tâche par rapport aux autres types que je gère de cette manière :

Code csharp :

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

1
2
 
value = dr.GetSqlInt32(column).OrNull();

L'avantage c'est que si la colonne n'est pas nullable dans la base, j'ai juste à supprimer le "OrNull()", et si je veux que ma variable prenne une valeur par défaut, je peux faire un "OrDefault(default)" sur le même modèle

[François DORIN]

Si c'est pas forward only, en tout cas j'ai déjà eu le souci par le passé avec une option de lecture qui interdisait :
- De lire les colonnes dans le désordre (forcément de manière incrémentielle)
- De lire plusieurs fois la même colonne

Du coup j'avais été obligé de faire des GetValues() à chaque ligne et me démerder avec un object[], ce qui ne me plaisait pas du tout.

Je peux tout à fait comprendre la gêne, mais je n'ai jamais rencontré ce soucis avec SqlDataReader. Peut être était-ce avec un autre fournisseur ?

Après, les méthodes d'extension je trouve ça beaucoup plus facile à écrire et à relire :

Code csharp :

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

1
2
 
value = dr.GetNullableDateTime(column);

Seul truc, c'est que ça fait tâche par rapport aux autres types que je gère de cette manière :

Code csharp :

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

1
2
 
value = dr.GetSqlInt32(column).OrNull();

L'avantage c'est que si la colonne n'est pas nullable dans la base, j'ai juste à supprimer le "OrNull()", et si je veux que ma variable prenne une valeur par défaut, je peux faire un "OrDefault(default)" sur le même modèle

Rien ne t'empêche de faire une méthode d'extension qui permet de retourner un SqlDateTime à partir d'une colonne DATETIME2. Par contre, il faut avoir conscience qu'il peut y avoir de légère perte. SqlDateTime est fait pour stocker un DATETIME et non un DATETIME2, il doit certainement souffrir des mêmes limitations. Mais ainsi, tu pourrais avoir quelque chose d'homogène, avoir une méthode nommée GetSqlDateTime2.

Après, si les limitations sont un soucis, tu peux aussi aller plus loin et créer une classe SqlDateTime2.

Avec tout ça, tu pourrais tout gérer de la même manière.

J'en profite pour donner un lien utile, regroupant l'ensemble des mappings entre les types SQL Server et les types .NET : https://docs.microsoft.com/fr-fr/dot...-type-mappings

[StringBuilder]

A l'inverse, le datetime2 de SQL Server a été créé afin de répondre convenablement à la norme SQL, elle-même basée sur ISO 8601... sur laquelle est basée elle-même le C#.

Ainsi, le DateTime de C# et le datetime2 de SQL Server ont la même précision et le même time range.
J'imagine qu'ils ont aussi la même représentation interne, ce qui maximise potentiellement les performances, là où le datetime de SQL Server oblige à effectuer une conversion.

DateTime C# :
https://msdn.microsoft.com/fr-fr/lib...v=vs.110).aspx

Time values are measured in 100-nanosecond units called ticks, and a particular date is the number of ticks since 12:00 midnight, January 1, 0001 A.D. (C.E.) in the GregorianCalendar calendar (excluding ticks that would be added by leap seconds).

DateTime SQL Server :
https://docs.microsoft.com/en-us/sql...ql-server-2017

datetime is not ANSI or ISO 8601 compliant.

YYYY is four digits from 1753 through 9999 that represent a year.

MM is two digits, ranging from 01 to 12, that represent a month in the specified year.

DD is two digits, ranging from 01 to 31 depending on the month, that represent a day of the specified month.

hh is two digits, ranging from 00 to 23, that represent the hour.

mm is two digits, ranging from 00 to 59, that represent the minute.

ss is two digits, ranging from 00 to 59, that represent the second.

n* is zero to three digits, ranging from 0 to 999, that represent the fractional seconds.

Accuracy
Rounded to increments of .000, .003, or .007 seconds

datetime2 SQL Server :

The ANSI and ISO 8601 compliance of date and time apply to datetime2.

YYYY is a four-digit number, ranging from 0001 through 9999, that represents a year.

MM is a two-digit number, ranging from 01 to 12, that represents a month in the specified year.

DD is a two-digit number, ranging from 01 to 31 depending on the month, that represents a day of the specified month.

hh is a two-digit number, ranging from 00 to 23, that represents the hour.

mm is a two-digit number, ranging from 00 to 59, that represents the minute.

ss is a two-digit number, ranging from 00 to 59, that represents the second.

n* is a zero- to seven-digit number from 0 to 9999999 that represents the fractional seconds. In Informatica, the fractional seconds will be truncated when n > 3.

Accuracy
100 nanoseconds

[François DORIN]

A l'inverse, le datetime2 de SQL Server a été créé afin de répondre convenablement à la norme SQL, elle-même basée sur ISO 8601... sur laquelle est basée elle-même le C#.

Tout à fait. Le plus gros problème était surtout la date minimale (1753) pour DATETIME au lieu de 0001 comme le prévoit la norme.

J'imagine qu'ils ont aussi la même représentation interne, ce qui maximise potentiellement les performances, là où le datetime de SQL Server oblige à effectuer une conversion.

Ce n'est pas sûr. Le type DATETIME2 a une longueur variable alors que la structure DateTime en C# est fixe. Je doute donc que la représentation interne soit la même.