Discussion :

[olibara]

Bonjour

Voici une question que je me pose souvent

Dans une classe j'ai une fonction qui peut etre appellée regulierement

Cette fonction utilise des methodes d'une autre classe
Vaut - il -mieux :

1-Instancier cette autre classe dans la fonction (je crains que ca bouffe chaque fois des ressource : mais peut etre que le FW gere ca tres bien)

2- Rendre cette autre classe membre de ma classe de travail et ne l'instancier donc explicitement qu'une fois.

[Guulh]

Perso, je crée un membre. Statique, même, si possible (i.e. si l'objet que tu crées ne dépend pas de l'objet appelant).

[Kelpan]

A priori, j'opterai pour le deuxième solution, moins de boulot pour le GC...

[olibara]

Merci a vous deux

Oui c'est effectivement ce que j'avais en tete !

[shwin]

J'essaie d'éviter le static

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 MyPerson _person = null;
private  MyPerson Person
{
get 
{
  if (_person == null) _person = new MyPerson ();
 
return _person
}
}
 
public void ExecuteCode()
{
Personn.DoSomething();
}

[Guulh]

J'essaie d'éviter le static

Pourquoi donc ? Dans ton exemple, si ton code est celui d'une classe A, toutes les instances de A ont besoin d'une Personne. Mais si toutes ces personnes sont identiques, ça sert à rien de coller une instance de personne dans chaque A.

[smyley]

Vis à vis de la Person qui s'initialise au premier appel, je préfère dire qu'il faut construire le membre à la création de la classe (même si je ne le fait pas toujours) car on peut parfois avoir des problèmes bizarre si on commence à faire du multithreading (déjà été dans des cas où la propriété s'exécutait deux fois sur deux threads différents et créaient donc deux fois l'objet interne, etc...).
Initialiser un membre statique dans le constructeur statique d'une classe évite, je crois, ce genre de problèmes mais là je ne suis pas sur.

La solution 1 est à éviter. Même si le GC fait bien son travail, ça prend du temps d'appeller le constructeur et ça le sollicite plus souvent pour rien.

Et la solution 2 pourrai très bien fonctionner avec un membre statique, aucune raison de créer un objet par classe quand ils n'ont aucune différence entre les instances de la classe, sauf qu'il est préférable d'initialiser tout ce qu'il faut dans le constructeur (statique ou pas).