Discussion :

[lutecefalco]

Hello tout le monde

Voilà le problème

J'ai une classe qu'on va appeler Common
J'ai 2 classes A et B qui héritent de Common

Code :

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

1
2
3
4
5
6
 
public class A : Common
{}
 
public class B : Common
{}

J'ai une méthode avec une surcharge pour chacune des classes précédentes

Code :

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

1
2
3
4
5
6
 
public void Fill(IDataReader dr, A value)
{}
 
public void Fill(IDataReader dr, B value)
{}

J'ai une méthode Generic de la forme:

Code :

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

1
2
3
4
5
6
7
8
9
 
public T GetById<T>(long id) where T : Common, new()
{
     // requete en base qui fournit un datareader
     ....
 
     // On fill l'objet avec les valeurs du dr
     Fill(dr, T);
}

Et biensûr ça compile pas et je comprends.
Mais comment indiquer la bonne surcharge de Fill à utiliser?

Merci

[ctxnop]

Salut,
C'est normal que ca ne compile pas. tu as passé le type et non pas la valeur.
Je ne sais pas comment est agencé ta méthode générique, mais si tu regarde tes méthodes Fill elle reçoivent un paramètre valorisé et non pas un type.

Code C# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
public T GetById<T>(long id) where T : Common, new()
{
    // requete en base qui fournit un datareader
    ....
 
    T val = ???; // C'est à toi de déterminer comment tu valorise le paramètre
 
    // On fill l'objet avec les valeurs du dr
    Fill(dr, val);
 
    // Tu peux aussi, si jamais tes méthodes le supportent, passer null
    Fill(dr, (T)null);
}

Edit: Ah oui, je précise au cas où qu'en plus méthode doit retourner un objet typé T.

[lutecefalco]

Yep, j'ai pas tout écrit, j'ai ça:

Code :

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

1
2
3
var result = new T();
Fill(dr, result);

[ctxnop]

Et le message d'erreur du compilateur il dit quoi ?

[lutecefalco]

Invalid argument dans l'appel de la surcharge

[davcha]

Common n'est pas de type A, ni de type B.

Donc Fill(Common), ça marche pas normal. Les types correspondent pas.

[Guulh]

Dans une optique pure POO (c'est à dire en évitant un switch case moche sur le type), il faudrait que GetById<T> soit dans une classe générique<T>, et appelle une méthode abstract Fill(Datareader dr, T item) ; après, ce sont des classes filles non génériques qui overrident Fill pour un type T donné.

Mais ça t'oblige d'avoir une instance par type d'objet Common, ce qui peut t'obliger à revoir pas mal de choses dans ton code.

[davcha]

Code c# :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
Func<IDataRecord, T> GetFiller<T>()
{
	var p = Expression.Parameter(typeof(IDataRecord));
	return Expression.Lambda<Func<IDataRecord, T>>(
		Expression.MemberInit(Expression.New(typeof(T)),
			typeof(T).GetProperties().Select(pi =>
				Expression.Bind(pi, Expression.Convert(
					Expression.Property(p, "Item", Expression.Constant(pi.Name)),pi.PropertyType
				))
			)
		)
	, p).Compile();
}