ReSharper y var

Question

Duplicar posible:
Why does ReSharper want to use 'var' for everything?

tengo ReSharper 4.5 y han encontrado que es muy valiosa hasta ahora, pero tengo una preocupación;
Parece querer hacer que cada declaración de variable implícita (var).
Como relativamente nuevo desarrollador, ¿cuánto debo confiar en ReSharper cuando se trata de esto?

Tome el siguiente fragmento de código de un método que Pinta encabezados de pestañas.

TabPage currentTab = tabCaseNotes.TabPages[e.Index]; 
Rectangle itemRect = tabCaseNotes.GetTabRect(e.Index); 
SolidBrush fillBrush = new SolidBrush(Color.Linen); 
SolidBrush textBrush = new SolidBrush(Color.Black); 
StringFormat sf = new StringFormat 
{ 
    Alignment = StringAlignment.Center, 
    LineAlignment = StringAlignment.Center 
}; 

ReSharper quiere que cambie todos los 5 de los de var. He leído la siguiente publicación similar, Use of var keyword in C#, pero me gustaría saber desde el punto de vista de ReSharper.

Answer 1

No necesita tener el tipo en la línea para que sea más legible, es una cuestión de preferencia personal. Me gusta la variación var:

var currentTab = tabCaseNotes.TabPages[e.Index]; 
var itemRect = tabCaseNotes.GetTabRect(e.Index); 
var fillBrush = new SolidBrush(Color.Linen); 
var textBrush = new SolidBrush(Color.Black); 
var sf = new StringFormat 
    { 
     Alignment = StringAlignment.Center, 
     LineAlignment = StringAlignment.Center 
    }; 

Actualización: que añadirá un punto de vista polémico a ella. A menos que esté leyendo el código de un libro, generalmente no me importa cuál es el tipo específico para entender algunas líneas de código que estoy leyendo. Considere la .GetTableRectangle (e.Index), para los que no está mostrando el código que opera en él:

var itemRect = tabCaseNotes.GetTableRectangle(e.Index); 
//do some operations on itemRect 

Al leer ese código específico voy a tener más para entenderlo desde las operaciones en itemRect que desde su tipo. Puede ser IRectangle, Rectangle, CustomRectangle, y aún no dice mucho sobre lo que el código está haciendo con él. En cambio, me preocupo más por el itemRect.Height, itemRect.Width o itemRect.GetArea() junto con la lógica involucrada.

Actualización 2: Como han señalado otros, puede desactivarlo. Asegúrese de mantener al equipo con las mismas prácticas, o probablemente termine haciendo cambios de una forma u otra cada vez que una persona diferente toque el código. Ver: http://www.jetbrains.com/resharper/features/codeTemplate.html

Answer 2

Resharper cree que es la mejor práctica, pero algunas personas no están de acuerdo ya que ha leído en la publicación vinculada. Me gusta usar una declaración explícita para una mayor legibilidad, pero para cada uno la suya propia. Si desea usar una declaración explícita, puede desactivar la regla en Resharper.

Answer 3

Esta pregunta es una muy buena manera de comenzar una guerra de llamas. Sin embargo, debes hacer lo que sea que tú y el que estés trabajando piensen que es más legible. Hay buenos argumentos para ambos lados del debate sobre var.

Dicho esto, si cree que es más legible declarar explícitamente el tipo, esa es su tarea. No tiene que hacer todo lo que Resharper le indique. Incluso puede deshabilitar esa regla si lo desea.

Answer 4

Resharper se ocupa principalmente de ayudarlo a refactorizar el código, y la palabra clave var generalmente facilita la refactorización. Por ejemplo, si los valores de retorno de cualquiera de esas funciones alguna vez cambian a tipo compatible, no tiene que cambiar nada de este código. Por lo tanto, ahora es un poco más fácil refactorizar su tipo tabCaseNotes, por ejemplo.

Personalmente, a menudo me inclino a dejar las dos primeras líneas solo, porque me gusta ver el nombre de tipo de una variable explícitamente enumerada en algún lugar en la línea donde se declara la variable. En todo caso, podría buscar una interfaz para usar, de modo que también obtenga el mismo "carácter genérico" que con la palabra clave var sin perder ninguna información importante de tipo legible. Sin embargo, definitivamente usaría var para fillBrush, textBrush y sf.

Answer 5

Resharper no quiere para usar var, le está dando la opción. Si usa var, le dará la opción de usar un tipo explícito, por lo que no puede ganar :-).

EDIT - interesante link discutiendo el tema.

Parece que se puede apagar, vaya a Resharper -> Opciones -> Inspección de código -> Severidad de inspección y desplácese hacia abajo un poco para ver las opciones relacionadas con var.

Answer 6

De hecho, puede apagarlo, y yo tengo. Admito que es bueno en algunos casos en que el nombre del tipo de clase es largo, como:

SuperDisconfibulator sd=new SuperDisconfibulator(); 

sería un buen candidato para ser acortado a var, pero personalmente esa es la única vez que me gustaría que se cambio. No creo que sea una buena idea usarlo cuando se está asignando una variable desde el valor de retorno de un método (como en su segunda línea), porque puede no estar inmediatamente claro exactamente qué tipo de variable devuelve.

Answer 7

En C#, prefiero usar var en todas partes. ¿Por qué? Por la misma razón que uso firstName en lugar de strFirstName o amount en lugar de intAmount. Claro, el modo más detallado es más legible en un pedazo de papel o, como dijiste, un libro, pero ninguno de mis códigos ha aparecido en un libro todavía.

Ahora, le pregunté a un compañero mío hace un tiempo acerca de este negocio intAmount y sacó a relucir un muy buen punto. Dijo que es útil en un editor de texto directo, pero cuando tienes Intellisense, puedes obtener la misma información y más simplemente pasando el cursor sobre la variable.

Por último, aunque aprecio el medio feliz que otros han señalado (es decir, usando var con la declaración new) y el argumento es válido y fuerte, me inclino a alejarme de eso sobre la única base de consistencia y legibilidad de manos cortas. Mi argumento de coherencia es que si la legibilidad es tan importante para usted, ¿por qué no usa también intAmount para poder decir el tipo de datos más adelante en el código?