Discussion :

[Alexandre le Grand]

Bonjour à tous,

J'ai une fonction avec une énumération en paramètre qui me retourne un DataTable après une série d'itération. Le problème est que à chaque tentative, un problème de CAST surgit:

Voici ma fonction:

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
 
        public static DataTable GetLINQToDataTable<t>(IEnumerable<t>  varlist)
        {
            DataTable dtReturn = new DataTable();
 
            // column names 
            PropertyInfo[] oProps = null;
 
            if (varlist == null) return dtReturn;
 
            foreach (IEnumerable<t> rec in varlist)
            {
 
                if (oProps == null)
                {
                    foreach (PropertyInfo pi in oProps)
                    {
                        Type colType = pi.PropertyType;
                        if ((colType.IsGenericType) && (colType.GetGenericTypeDefinition() == typeof(Nullable<>)))
                        {
                            colType = colType.GetGenericArguments()[0];
                        }
 
                        dtReturn.Columns.Add(new DataColumn(pi.Name, colType));
 
                    }
                }
 
                DataRow dr = dtReturn.NewRow();
 
                foreach (PropertyInfo pi in oProps)
                {
                    dr[pi.Name] = pi.GetValue(rec, null) == null ? DBNull.Value : pi.GetValue(rec, null);
                }
 
            }
 
            return dtReturn;
        }

J'appelle cette fonction comme suit :

Code :

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

1
2
3
4
5
6
7
8
 
                    IEnumerable<Person > listePerson  = (from oPerson in ContexteDAO.ContexteDonnees.Persons
                    where oPerson .CodeCompagnie == aCodeCompagnie
                    select oPerson ).ToList();
 
                        DataTable dt = new DataTable();
 
                        dt = LINQToDataTable(listePerson);

Lors du débogage, le programme saute au niveau du :

Code :

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

foreach (IEnumerable<t> rec in varlist)

Merci de me donner un coup de main.

[worm83]

Bon je suis fatigué, ok, mais ton programme compile ????

Le bon code serais plutôt :

Code :

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

1
2
3
4
5
 
foreach t rec in varlist)
{
 
}

[J@ckHerror]

Bonjour,

Remplace

Code :

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

foreach (IEnumerable<t> rec in varlist)

par

Code :

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

foreach (t rec in varlist)

De plus le reste de ta méthode est à revoir je pense, la partie =>

Code :

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

1
2
3
4
5
6
 
if (oProps == null)
{
    foreach (PropertyInfo pi in oProps)
    {...}
}

n'est pas cohérente et provoquera sans doute un nullReferenceException.

Bon code.

[Alexandre le Grand]

Bonjour,



De plus le reste de ta méthode est à revoir je pense, la partie =>

Code :

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

1
2
3
4
5
6
 
if (oProps == null)
{
    foreach (PropertyInfo pi in oProps)
    {...}
}

n'est pas cohérente et provoquera sans doute un nullReferenceException.

Bon code.

Merci pour vos réponses, vous aviez vu juste. Le programme compile correctement, mais l'exécution il y a problème. En modifiant le code comme demandé:

Code :

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

foreach (t rec in varlist)

, certes il n'y a plus de blocage mais une exception de nullité est levée à ce niveau:

Code :

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

1
2
3
4
5
6
 
if (oProps == null)
{
    foreach (PropertyInfo pi in oProps)
    {...}
}

Au fait mon problème est de charger ma liste que voici:

Code :

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

1
2
3
4
 
                   IEnumerable<Person > listePerson  = (from oPerson in ContexteDAO.ContexteDonnees.Persons
                    where oPerson .CodeCompagnie == aCodeCompagnie
                    select oPerson ).ToList();

ou une quelconque dans une datatable pour alimenter un état crystal report, mais ma fonction ne peut faire l'affaire. Auriez-vous une idée?

Merci d'avance.

[J@ckHerror]

Bonjour,

Le but de votre méthode est donc de convertir une IEnumerable<T> en DataTable.
Votre méthode est incomplète, oProps est initialisé à null et n'est jamais affecté ensuite, c'est la raison de NullReferenceException.
Pour obtenir la liste des properties d'un Type =>

Code :

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

1
2
3
 
System.Reflection.PropertyInfo[] oProps = typeof(T).GetProperties(System.Reflection.BindingFlags.Instance | System.Reflection.BindingFlags.NonPublic |
            System.Reflection.BindingFlags.Public);

Doc MSDN

Il ne devrait plus que rester cette petite coquille

Code :

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

1
2
3
if (oProps != null)
{...}

Bonne journée
et bon code.

J@ck

[Alexandre le Grand]

Bonjour,

L'itération saute l'entrée de la seconde boucle foreach, ligne 94 (voir image)


.

Après les modifications sus recommandées un avertissement apparait en bas de page.

Code :

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

Avertissement	1	Possibilité d'instruction vide erronée

[J@ckHerror]

Vous ne faite pas les bonnes choses aux bons endroits .

il vous faut initialiser oProps comme je vous l'ai montré, puis tester si il est null ou non, et si il n'est pas null on le parcours pour créer les columns.
Puis une fois les columns crées il vous faudra reparcourir votre IEnumerable<t> pour ajouter un ligne à la DataTable par élément de l'énumération.

Tout ceci devrait être assez simple, vous avez déjà les principales étapes, mais si vous rencontrez des problèmes faites signe ...

J@ck.

[Alexandre le Grand]

Merci d'avance. J'ai modifié le code comme souhaité, il devient ce ceci:

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
 
        public static DataTable LINQToDataTable<t>(IEnumerable<t> varlist)
        {
            DataTable dtReturn = new DataTable();
 
 
            PropertyInfo[] oProps = typeof(t).GetProperties(System.Reflection.BindingFlags.Instance | System.Reflection.BindingFlags.NonPublic |
            System.Reflection.BindingFlags.Public);
 
            if (varlist == null) return dtReturn;
 
            foreach (t rec in varlist)
            {
 
                if (oProps != null)
                {
                    foreach (PropertyInfo pi in oProps)
                    {
                        Type colType = pi.PropertyType;
                        if ((colType.IsGenericType) && (colType.GetGenericTypeDefinition() == typeof(Nullable<>)))
                        {
                            colType = colType.GetGenericArguments()[0];
                        }
 
                        dtReturn.Columns.Add(new DataColumn(pi.Name, colType));
 
                    }
                }
 
                DataRow dr = dtReturn.NewRow();
 
                foreach (PropertyInfo pi in oProps)
                {
 
                    dr[pi.Name] = pi.GetValue(rec, null) == null ? DBNull.Value : pi.GetValue(rec, null);
                }
 
            }
 
            return dtReturn;
        }

Le seul hic, est que en la dernière boucle foreach saute à la seconde itération avec cette exception:

Code :

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

1
2
 
Une colonne nommée 'MaColonne' appartient déjà à ce DataTable."}

Encore une fois merci pour le coup de main.

[J@ckHerror]

C'est normal, car à chaque tour du premier foreach on va ajouter des Colomns (exécuter en mode debug-pas à pas et vous vous en rendrez compte) et ce n'est pas ce que l'on souhaite...
Ce que l'on veut c'est créer une et une seule fois les entêtes de colonnes, puis autant de ligne qu'il y a d'éléments dans notre énumération.

Pour cela plusieurs possibilités, la plus simple serait de mettre oProps à null juste après la création de toutes les Columns, en gros juste avant le

Code :

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

1
2
 
		DataRow dr = dtReturn.NewRow();

Pour ma part, et pour plus de lisibilité j'aurais séparé l'étape de création des Columns et l'étape de création des Rows, chacune dans un foreach, le premier sur oProps et le second sur varlist ... à vous de voir.

J@ck.

[Alexandre le Grand]

...
Pour cela plusieurs possibilités, la plus simple serait de mettre oProps à null juste après la création de toutes les Columns, en gros juste avant le

Code :

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

1
2
 
		DataRow dr = dtReturn.NewRow();

Pour ma part, et pour plus de lisibilité j'aurais séparé l'étape de création des Columns et l'étape de création des Rows, chacune dans un foreach, le premier sur oProps et le second sur varlist ... à vous de voir.

J@ck.

Vraiment un exemple de code pourrait mieux m'éclaircir et sera le bienvenu . J'éssaie d'optimiser le code comme je peux, mais les exceptions persistent. Avec oProps = null, il y a une exception de nullité. Et en remontant d'une accolade "}" le code suivant:

Code :

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

1
2
3
4
5
6
7
8
9
 
DataRow dr = dtReturn.NewRow();
 
 
                foreach (PropertyInfo pi in oProps)
                {
 
                    dr[pi.Name] = pi.GetValue(rec, null) == null ? DBNull.Value : pi.GetValue(rec, null);
                }

J'ai ce message:

Code :

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

 "La colonne ''MaSecondeColonne" n''existe pas dans la table."

Merci.

[J@ckHerror]

Si c'est ce que vous voulez, la voici, je vous laisse l'analyser et comparer avec la votre.
Pour les indications vous pouvez en trouver ici
Cependant vous ne progresserez qu'en comprenant les exceptions et comment passer outre, du code tout fait n'est qu'une solution à court terme.

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
 
            PropertyInfo[] properties = typeof(T).GetProperties(System.Reflection.BindingFlags.Instance | System.Reflection.BindingFlags.NonPublic |
            System.Reflection.BindingFlags.Public);
 
            DataTable table = new DataTable();
            foreach (System.Reflection.PropertyInfo prop in properties.ToList())
            {
                table.Columns.Add(prop.Name, Nullable.GetUnderlyingType(prop.PropertyType) ?? prop.PropertyType);
            }
 
            foreach (T item in data)
            {
                DataRow row = table.NewRow();
                foreach (System.Reflection.PropertyInfo prop in properties)
                    row[prop.Name] = prop.GetValue(item, null) ?? DBNull.Value;
                table.Rows.Add(row);
            }
            return table;

Bonne journée et bon code.
J@ck