Discussion :

[BoeufBrocoli]

bjr,

J'aimerai simplifier cette 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
 
private void BuildDate(object date)
{
   switch(date.GetType().ToString())
   {
      case "System.String":
	string _temp = date.ToString();
	SetMembersFrom(_temp);
	break;
 
      case "System.DateTime" :
	DateTime _temp2 = (DateTime)date;
	SetMembersFrom(_temp2);
	break;
   }
}

Mes méthodes SetMembersFrom() sont surchargés (un avec DateTime et un avec string comme argument)
et j'appelle BuildDate comme suit :

Code :

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

1
2
3
 
BuildDate(DateTime.Now);
BuildDate("2005/11/12 00:00:00");

Comment utiliser la reflexion pour caster dans le bon type ?
du style :

Code :

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

1
2
3
4
5
6
 
void FonctionTest(object date)
{
   Type dateType = date.GetType();
    SetMembersFrom((dateType)date);
}

Merci

[neguib]

Excuses mais je comprends pas trop pourquoi tu as cette méthode BuildDate intermediaire qui reçoit un object en paramètre
D'où provient ce date pour qu'il soit un object et non directement un DateTime ou un String :

[BoeufBrocoli]

C'est parce que je ne sais pas si "object date" que l'on fournit à BuildDate() est de type String ou DateTime ou d'un autre type.

En fait c'est pour savoir comment je peux caster un type object en un autre type que je connais pas (sans instancier un nouvel objet).
Puis je passe ce dernier à la fonction appropriée avec le bon prototype.

[neguib]

Et as-tu essayer en effet directement

Code :

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

1
2
3
4
5
 
 private void BuildDate(object date) 
{ 
   SetMembersFrom((date.GetType()) date);  
}

[BoeufBrocoli]

hélas ca ne marche pas.

Il attend une ")".

[neguib]

hélas ca ne marche pas.

Il attend une ")".

oui mais encore

[BoeufBrocoli]

lol non ca ne marche pas.

[Maniak]

C'est parce que je ne sais pas si "object date" que l'on fournit à BuildDate() est de type String ou DateTime ou d'un autre type.

C'est le boulot du code appelant ça.
Au pire, tu fais un BuildDate qui reçoit un string, un autre qui reçoit un DateTime, et l'appelant se débrouille pour appeler le bon.

En fait c'est pour savoir comment je peux caster un type object en un autre type que je connais pas (sans instancier un nouvel objet).

Tu ne peux pas. Ce que tu fais actuellement est le 'mieux'. Enfin t'as juste pas besoin des variables temporaires. Tu castes directement dans l'appel de SetMembersFrom, ça grapille toujours deux lignes comme ça.

Mais bon, deux versions de BuildDate, c'est bien mieux. Et encore, à moins qu'il manque du code dans ce que tu as montré, tu peux virer BuildDate et forcer le code appelant à utiliser directement SetMembersFrom. Là BuildDate sert juste à dispatcher, et c'est pas un genre de dispatching à conserver...

Et as-tu essayer en effet directement

Code :

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

1
2
3
4
 private void BuildDate(object date) 
{ 
   SetMembersFrom((date.GetType()) date);  
}

Euh... Dis, tu as une version de C# croisée vaudou là ? :)

[neguib]

Et as-tu essayer en effet directement

Code :

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

1
2
3
4
 private void BuildDate(object date) 
{ 
   SetMembersFrom((date.GetType()) date);  
}

Euh... Dis, tu as une version de C# croisée vaudou là ?

et non je suis vbciste
En fait tu as très judicieusement exprimé en première partie tout ce que je pensais et ne savais correctement lui formulé

Excuses mais je comprends pas trop pourquoi tu as cette méthode BuildDate intermediaire qui reçoit un object en paramètre
D'où provient ce date pour qu'il soit un object et non directement un DateTime ou un String

[BoeufBrocoli]

Merci pour les reponses que vous m'avez apporté !