Discussion :

[Kropernic]

Hello à tous,

J'ai récemment découvert une série de 3 articles via ce message.

N'ayant jamais eu de cours d'archi, je me forme sur le tas et cette série d'article tombe à pic !

Notemment, dans la partie où il traite des DTOParser et notamment des ordinaux afin d'accéder aux éléments des datareaders via leur index plutôt que leur nom (car c'est plus performant).

Pour un DTO simple, pas de souci. Mais comment fait-on pour un DTO qui contient d'autres DTO's ?

Prenons par exemple une classe Tel (pour téléphone) :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Public Class Tel
    Public Property Id As Integer
    Public Property Type As TypeTel
    Public Property Pays As Pays
    Public Property Num As String
 
    Public Sub New(ByVal id As Integer, ByVal type As TypeTel, ByVal pays As Pays, ByVal num As String)
        Me.Id = id
        Me.Type = type
        Me.Pays = pays
        Me.Num = num
    End Sub
End Class

Où TypeTel indique de quel type de téléphone il s'agit (fixe, portable, fax, etc.) et Pays indique le pays histoire d'avoir l'indicatif téléphonique du-dit pays.

Donc en suivant l'article, je commence vaillamment à créer une classe ParserTel comme suit :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Public Class Parser_Tel
 
    Private ord_id As Integer
    Private ord_num As Integer
 
    Public Function PopulateDTO(reader As System.Data.SqlClient.SqlDataReader) As Tel
 
    End Function
 
    Public Sub PopulateOrd(reader As System.Data.SqlClient.SqlDataReader)
 
    End Sub
End Class

Mais je bloque pour Pays et TypeTel. Toutes les données nécessaires à la création de ces deux instances se trouvent dans la datareader (pour ne pas devoir refaire un accès db pou récupérer le pays et le type sur base de leur Id).
Faut-il faire appel au parser du DTO Pays dans la fonction PopulateDTO ??

J'espère avoir été clair

[DotNetMatt]

Mais comment fait-on pour un DTO qui contient d'autres DTO's ? Faut-il faire appel au parser du DTO Pays dans la fonction PopulateDTO ??

Oui il le faut, sinon ça t'oblige à implémenter deux fois le code nécessaire à la création d'un objet PaysDTO, ce qui est une perte de temps, et qui va te compliquer la vie lors des maintenances

[Kropernic]

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
Public Class Parser_Tel
 
    Private ord_id As Integer
    Private ord_num As Integer
 
    Public Function PopulateDTO(reader As System.Data.SqlClient.SqlDataReader) As Tel
        Dim tel As New Tel
        tel.Id = reader.GetInt32(ord_id)
        tel.Num = reader.GetString(ord_num)
 
        Dim parser As New Parser_Pays
        parser.PopulateOrd(reader)
        tel.Pays = parser.PopulateDTO(reader)
 
        Return tel
    End Function
 
    Public Sub PopulateOrd(reader As System.Data.SqlClient.SqlDataReader)
        ord_id = reader.GetOrdinal("TEL_ID")
        ord_num = reader.GetOrdinal("TEL_NUM")
    End Sub
End Class

Je dois encore faire pour le type mais pour le pays, ça donne quelque chose dans ce goût là non ?

[DotNetMatt]

Oui tout à fait, comme ça le jour où tu modifies la structure de ton objet Pays, tu as juste à modifier la classe Parser_Pays et le tour est joué, tu n'auras pas besoin de répercuter ces modifications ailleurs.

Petit conseil au passage, n'oublie pas de "blinder" tes parser, en vérifiant que les valeurs sont bien dans le SqlDataReader. Si elles n'y sont pas, prévois de mettre une valeur par défaut, ou de lancer un traitement particulier, ou une erreur.

[Kropernic]

Oki doki !

Merci du conseil

[Kropernic]

Pas si résolu que ça finalement.

Mais plus rien à voir avec l'architecture.

Tout l'intérêt d'utiliser les ordinaux est d'accéder aux items via l'index en utilisant la méthode GetXXXX du datareader où XXXX est le type approprié (ça évite le castint également).

Seulement pour les objets de type Integer, il existe juste GetInt32 mais ça colle pas.... Je trouve ça un peu moche d'aller foutre un cast là alors que tout le reste fonctionne sans cast.

Une solution alternative et élégante ? Suis-je passé à côté de quelque chose ?


EDIT : J'étais bien passé à côté de quelque chose... La colonne que je récupère avec GetInt32 est de type Tinyint dans SQL SERVER