Discussion :

[Immobilis]

Salut,

J'ai une fonction "LoadFromReader" qui comme son nom l'indique charge un objet à partir d'un datareader.
Vaut-il mieux faire:

Code :

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

1
2
 
obj = LoadFromReader(dr, obj);

ou bien

Code :

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

1
2
 
LoadFromReader(dr, ref obj);

Merci

A+

[Jatukam]

Je pense qu'il faut mieux utiliser :

Code :

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

1
2
 
LoadFromReader(dr, ref obj);

car bien qu'il n'y ai pas recopie de l'objet dans les 2 cas, avec un passage par reference, tu auras une affectation de moins.
Je pense que par rapport à un passage par reference d'un type valeur, l'avantage d'utiliser la seconde methode est moins évident.

[Guulh]

As-tu besoin de passer l'objet en paramètre ? obj = LoadFromReader(dr) ne suffit pas ?

Mais sinon, je suis d'accord avec Jatukam ; si on veut avec un appel de méthode pouvoir changer la référence de l'objet, l'utilisation de paramètre "ref" me parait toute indiquée.

[Immobilis]

Ben en fait, je me suis rendu compte que ref n'est pas nécessaire:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
public MonType Load()
{
      MonType obj = new MonType();
      // creation du datareader ...
      LoadFromReader(dr, obj);
      return obj;
}
 
private void LoadFromReader(IDataReader dr, MonType obj)
{
      obj.Id = Convert.ToInt32(dr["ID"]);
}

suffit.

A+

[SirJulio]

Salut,

oui c'est normal, tu ne fais qu'acceder à un champ, la reference de l'objet obj est toujours la meme. Il aurait fallu faire un ref, si tu avais une methode de la forme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
public MonType Load()
{
      MonType obj = new MonType();
      // creation du datareader ...
      LoadFromReader(dr, obj);
      return obj;
}
 
private void LoadFromReader(IDataReader dr, MonType obj)
{
      obj = new MonType(); // <=
      obj.Id = Convert.ToInt32(dr["ID"]);
}

Bon courage.