¿Cómo desactivo el "esto" implícito en C#?

Question

Esto me molesta mucho y me parece que escribo errores estúpidos cuando se combina con Intellisense (VS 2008 Pro):

class Foo 
{ 
    public Foo(bool isAction) 
    { 
     this.IsAction = IsAction; 
    } 

    public bool IsAction { get; private set; } 
} 

¿Viste eso? Ciertamente no lo hice hasta que IsAction nunca cambió, causando errores.

Intellisense alguna manera convertidos "isA<tab>" a "IsAction" para mí lo que significa la propiedad Foo.IsAction es siempre falsa independientemente de la entrada del constructor. Sólo brillante.

Tengo que decir que odio particularmente el "implícito this" (no sé si tiene un nombre formal) y me gustaría desactivarlo para que nunca lo asuma. ¿Hay alguna manera de hacer esto? Esto también se aplica al llamar a métodos estáticos de la misma clase.

Alternativamente, ¿qué convenciones de nomenclatura evitan este pequeño problema? La propiedad debe seguir siendo "IsAction", por lo que tiene que ser una convención sobre el nombre del parámetro del constructor. Por extraño que parezca, si lo nombro con la ortografía coincidente exacta, entonces this.IsAction = IsAction; funciona correctamente.

El problema no son los idiomas sensibles a las mayúsculas y minúsculas sino la implícita de this. Ahora que lo pienso, esto también es más una pregunta de VS 2008 Pro que una C#. Puedo vivir con código ya escrito sin la this pero no quiero escribir nuevo código sin ella, que significa decirle En

---

respuesta de noldorin me hizo pensar.

Ahora que lo pienso, esto también es más una pregunta de VS 2008 que una C#. Puedo vivir con el código ya escrito sin el this (aunque sí lo cambio si estoy allí rechinando) pero no quiero escribir un código nuevo sin él, lo que significa decirle a Intellisense que deje de hacerlo. ¿Puedo decirle a Intellisense que lo apague?

Answer 1

Este es un problema común. Microsoft tiene algunos recommendations for parameter names pero no son terriblemente útiles en su caso.

Como han mencionado otros respondedores, no se puede "desactivar" el comportamiento de resolución del alcance del lenguaje C#: su mejor enfoque es una convención de nomenclatura. Otros han mencionado la notación "húngara": algunas personas tienen una reacción instintiva ante esto debido al confusion over the original intent de la notación.

Mi enfoque personal, ha sido utilizar el carácter 'p' como prefijo de los nombres de los parámetros de las funciones públicas.Es discreto, simple, fácilmente identificable y fácil de aplicar con herramientas como Resharper.

La convención de nomenclatura particular que elija es una cuestión de preferencia y estilo; sin embargo, hay algún beneficio de ser consistente en la práctica que seleccione.

Usando mi convención de nomenclatura sugerida, que iba a escribir su constructor para:

class Foo 
{ 
    public Foo(bool pIsAction) 
    { 
     this.IsAction = pIsAction; 
    } 

    public bool IsAction { get; private set; } 
} 

Answer 2

Siempre se puede volver a la notación húngara [Me estoy preparando para recibir llamas cuando escribo esto]. Si puedes lidiar con la fealdad, eso resolvería tu problema. Esta es una sugerencia, no una recomendación.

Como alternativa, estoy bastante seguro de que el análisis de código estático lo detectará y le advertirá al respecto. Prueba FxCop.

EDITAR

He estado usando ReSharper durante más de un año, y sé que es muy inteligente acerca de que le asiste en caso de una manera sensible. Entre otros beneficios, su problema intellisense se resolverá instalando Resharper.

EDITAR 2

simplemente he comprobado. Ni FxCop ni Resharper detectan este error directamente. Lo que sí atrapan es el hecho de que el parámetro isAction no se utiliza en el método Foo. En este caso, la advertencia lo guiará al error. En los casos en que el parámetro se utiliza de otra manera dentro del método, puede pasar por el análisis de código estático.

Answer 3

Esto me pone todo el tiempo.Me he tomado a variables prepending que se pasa al constructor con un '_', como:

class Foo 
{  
    public Foo(bool _isAction) 
    { 
     this.IsAction = _isAction; 
    } 
    public bool IsAction { get; private set; }} 

Answer 4

Me temo que no hay manera de desactivar la función "implícita this". Es parte de la especificación del lenguaje y del compilador, y no hay forma de desactivarlo.

Personalmente, no considero que sea un gran problema. Es cierto que es importante tener cuidado con la capitalización de los nombres de los miembros y los parámetros, pero esta es siempre la situación en un lenguaje que distingue entre mayúsculas y minúsculas, como C#.

Mi "solución" recomendada (que ya parece que está haciendo) es usar siempre la palabra clave this para hacer referencia a propiedades/campos, de modo que se destaque inmediatamente cuando deba usar un parámetro. No va a resolver el problema para usted, pero si lo tiene en cuenta, sin dudas lo ayudará. Solo adquirir el hábito de esto (así como recordar todos los nombres de parámetros/variables locales deben comenzar con minúsculas) le ayudará a evitar este problema en el futuro.

Answer 5

Es un problema molesto en intellisense de Visual Studio. Resharper lo hace bien la mayor parte del tiempo sin embargo.

Answer 6

Creo que es más probable que el desarrollador haya elegido "IsAction" en lugar de "isAction" de Intellisense. No creo que Intellisense cambie "isA" a "this.IsAction".

Si los nombres difieren solo por caso, entonces creo que la única forma de evitar errores como este es ser consciente de ellos y cuidadoso, y al usar las pruebas unitarias de manera efectiva.

Answer 7

Podría ser molesto en otras capacidades, pero podría desactivar la opción para que Intellisense preseleccione el miembro utilizado más recientemente. Me doy cuenta de que realmente no solucionará el problema directamente, pero podría ayudar a prevenir algunas pestañas accidentales cuando realmente no tiene seleccionado el elemento correcto.

Answer 8

FxCop se quejará de esto porque el parámetro isAction nunca se utiliza. Específicamente, extraerá la regla CA1801: ReviewUnusedParameters.

Personalmente, siempre he pensado que el compilador C# debería dar una advertencia sobre los parámetros no utilizados.

Answer 9

Nota:

Si está utilizando ReSharper, tiene algunos atajos que escriben un montón de este código para usted y evitar el error.

Puede crear la propiedad primero, luego presionar Alt-Ins y elegir "generar constructor", o puede agregar un parámetro "isAction" al constructor, presionar Alt-Enter con el cursor en el parameterName, y elegir la acción "Crear e inicializar propiedad de la propiedad IsAction" del menú que aparece.

Answer 10

He intentado simplemente su código en Visual Studio 2008. Al encender el análisis de estática que produce el siguiente error:

Warning 3 CA1801 : Microsoft.Usage : Parameter 'isAction' of 'Foo.Foo(bool)' is never used. Remove the parameter or use it in the method body.

Mi sugerencia es que al activar esto en adelante encontrará errores como este al principio. Para habilitar esto, seleccione propiedades del menú contextual en el proyecto, luego seleccione la pestaña Análisis de código y seleccione "Habilitar análisis de código en compilación"