Pourquoi ReSharper veut-il utiliser 'var' pour tout ?

Question

Je viens de commencer à utiliser ReSharper avec Visual Studio (après les nombreuses recommandations sur SO). Pour l'essayer, j'ai ouvert un projet ASP.NET MVC récent. L'une des premières et plus fréquentes choses que j'ai remarquées, c'est qu'il suggère de changer la plupart/toutes mes déclarations explicites en var au lieu de cela. Par exemple :

//From This:
MyObject foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);
//To This:
var foo = DB.MyObjects.SingleOrDefault(w => w.Id == 1);

et ainsi de suite, même avec des types simples tels que int , bool , etc.

Pourquoi cette recommandation ? Je n'ai pas de formation en informatique ou en .NET, je suis "tombé" dans le développement .NET récemment, donc j'aimerais vraiment comprendre ce qui se passe et savoir si c'est utile ou non.

Answer 1

Ce que ReSharper suggère est clairement une utilisation excessive du mot-clé var. Vous pouvez l'utiliser lorsque le type est évident :

var obj = new SomeObject();

Si le type n'est pas évident, il est préférable de l'écrire :

SomeObject obj = DB.SomeClass.GetObject(42);

Answer 2

L'une des raisons est l'amélioration de la lisibilité. Qu'est-ce qui est mieux ?

Dictionary<int, MyLongNamedObject> dictionary = new Dictionary<int, MyLongNamedObject>();

ou

var dictionary = new Dictionary<int, MyLongNamedObject>();

Answer 3

Personnellement, je préfère désactiver cette suggestion. L'utilisation de var peut souvent améliorer la lisibilité ; mais comme vous l'avez mentionné, elle la réduit parfois (avec des types simples, ou lorsque le type résultant est obscure ).

Je préfère choisir quand j'utilise var et quand je ne le fais pas. Mais encore une fois, cela ne concerne que moi.

Answer 4

var peut augmenter la lisibilité du code tout en diminuant la compréhension immédiate du code. De même, elle peut diminuer la lisibilité du code dans d'autres situations. Parfois, son utilisation est neutre. La mesure de la lisibilité par rapport à la compréhension n'est pas proportionnelle mais dépend de la situation. Parfois, les deux sont augmentés ou diminués en même temps.

Le facteur est ce qui var et dans quelle mesure la cible supporte l'obscurcissement immédiat de son type de données pour le lecteur, ou si l'information sur son type est nécessaire pour comprendre la partie du programme en question.

Par exemple, une mauvaise dénomination peut conduire à var entraînant une diminution de la compréhension du code. Ce n'est pas var Mais c'est la faute de l'homme :

var value1 = GetNotObviousValue(); //What's the data type? 
//vs. 
var value2 = Math.Abs(-3); // Obviously a numeric data type. 

Parfois, il n'est pas judicieux d'utiliser var pour les types de données simples lorsque le code est plus lisible en son absence :

var num = GetNumber(); // But what type of number?
// vs. 
double num = GetNumber(); // I see, it's a double type. 

Parfois var peut être utile pour cacher des informations sur les types de données dont on ne veut pas nécessairement voir la complexité :

    IEnumerable<KeyValuePair<string,List<Dictionary<int,bool>>>> q = from t in d where t.Key == null select t; // OMG! 
    //vs. 
    var q = from t in d where t.Key == null select t;

    // I simply want the first string, so the last version seems fine.  
    q.First().Key; 

Vous doit utiliser var lorsqu'un type anonyme est présent car il n'y a pas de nom de type pour l'appeler :

var o = new { Num=3, Name="" };

Lorsque l'Intellisense de Visual Studio fournit des informations sur les types en dépit de la présence de var Vous devez alors moins vous fier à votre compréhension en lisant le code sans aide. Il est probablement sage de supposer que tout le monde n'a pas ou n'utilise pas Intellisense.

En résumé, sur la base des exemples ci-dessus, Je suggérerais l'application carte blanche de var n'est pas une bonne idée, car la plupart des choses doivent être faites avec modération et en fonction des circonstances, comme le montre l'exemple suivant.

Pourquoi Resharper l'utilise-t-il toujours par défaut ? Je dirais que c'est par facilité, parce qu'il ne peut pas analyser les nuances des situations pour décider quand il vaut mieux ne pas l'utiliser.

Answer 5

Je n'ai pas aimé cela non plus.

Je ne veux pas que cela se transforme en un débat sur l'utilisation des var Il a son utilité, mais ne devrait pas être utilisé partout.

Ce qu'il faut retenir, c'est que ReSharper est configuré selon les normes de codage que vous souhaitez.

Edita: ReSharper et var