Quelle est la bonne façon d'écrire la méthode 'Get'?

Question

Avec un de mes collègues de travail se disputent souvent sur "la bonne façon" d'écrire des méthodes "Get". Mon opinion est object GetSomeObject(). Mon collègue pense qu'il est préférable d'être void GetSomeObject(object obj). Je sais que le résultat est le même dans les deux cas. Je veux entendre et d'autres opinions. Ohhh j'ai oublié de dire pour quelle plate-forme nous parlons - .NET Framework la langue est C#.

Answer 1

Il lit plus facile et est une bonne pratique de codage pour renvoyer une valeur, sur la modification d'un paramètre de référence (qui pour moi au moins, est un moyen de la vieille école d'obtenir des valeurs de retour)

Si vous recherchez une signification plus profonde:

Les paramètres de fonction en C# sont créés en tant que Value parameters by default (en fonction des paramètres de référence).

Pour modifier la valeur du paramètre et conserver cette modification pour le code appelant, le paramètre doit être déclaré référence. Vous savez probablement tout cela.

est ici la différence: la valeur et les paramètres ref sont stockés sur la pile (ce qui est très efficace), mais données du paramètre de référence est stored on the heap. Il y a donc une fraction de surcoût en utilisant des valeurs de référence.

Dans la plupart des cas, ce n'est pas un problème du tout, mais certains problèmes pourraient surgir, comme:

• fonctions récursives qui utilisent trop de pile (et vous obtenez un débordement de pile)
• fonctions qui nécessite la vitesse, comme le calcul des nombres premiers ou des fractales

Il y a probablement plus, et de meilleurs exemples, que ce que j'ai donné, vous avez cependant l'idée.

Answer 2

objet getSomeObject(); est mieux.

Answer 3

Bien sûr object GetSomeObject() est mieux. L'autre est plus un setter qu'un getter ...

Answer 4

Si elle est simple obtenir alors il devrait être une propriété

public object SomeObject 
{ 
    get { return _someObj; } 
} 

si elle est calcul alors

object GetSomeObject() { ... } 

est loin plus communément attendu. En outre, l'autre devrait avoir un ref ou un out qui est déconseillé si le premier peut être atteint

Answer 5

void GetSomeObject(object obj) n'obtiendra rien. Si vous avez activé dans un paramètre out vous pouvez attribuer une valeur à lui, et il serait travailler techniquement, acheter pourquoi, lorsque vous pouvez utiliser les types de retour exactement comme ils étaient destinés:

public void GetSomeObject(out returnObject) 
{ 
    returnObject = ... 
} 

ou

public object GetSomeObject() 
{ 
    return ... 
} 

Answer 6

D'une manière générale la première façon est plus typique et meilleure car object GetSomeObject() vous permet de faire ce qui suit: GetSomeObject().Foo(). Et c'est un peu plus intuitif. Cependant, bool GetSomeObject(out object obj) peut être utile comme dans le cas de TryGetValue() dans la classe Dictionary.

Answer 7

Je peux assez vois bien une seule raison d'utiliser void GetSomeObject(out object obj) au lieu de object GetSomeObject() et qui est si vous vous débarrasser du vide et au lieu de faire quelque chose comme ErrorResult GetSomeObject(out object obj) (et la GetSomeObject-opération est velue et sujette à erreur) puisque vous peut ensuite signaler le statut via la valeur de retour.

Cependant, cela serait encore mieux géré via de simples anciennes exceptions IMHO. Bien que je sache que certaines normes de codage disent que les exceptions ne devraient pas être utilisées du tout et dans ces cas, vous pourriez vouloir faire quelque chose comme ça.

Encore, je dirais juste aller avec une propriété ou le object GetSomeObject() sauf si vous avez un vraiment bonne raison de ne pas le faire.