Mostrando una advertencia cuando el resultado de la función no está asignado a la variable

Question

Tengo una función que devuelve una copia modificada del objeto que llamó a esta función. A menudo hago algo como esto:

obj = obj.Foo(param); 

No preguntes por qué, simplemente tengo que hacerlo. Pero a veces (y otros) se olvidó de asignar resultado de la función, hacer esto:

obj.Foo(param); 

que conduce repetidamente para debuging tiempo.

¿Hay alguna manera de mostrar una advertencia o un error cada vez que el resultado de la función no se asigna a una variable? ¿O alguna otra sugerencia sobre cómo resolver este problema?

Answer 1

Se podría utilizar un parámetro de salida, por lo que la llamada se vería así:

obj.Foo(param, out obj); 

Answer 2

que es totalmente gratis y con frecuencia deseable no asignar el parámetro de retorno por lo que sería malo tener una advertencia para él. Henrik's respuesta para utilizar un parámetro out es lo que recomendaría también para garantizar que el resultado se asigna cada vez.

Answer 3

puede habilitar warnings visual studio.

incluso se puede personalizar la regla que desea aplicar.

debería ver las advertencias en el caso de que no se asigna la función de una variable

también puede decidir tratar el Waring como errores

Ejemplo:

public static class MyClass 
    { 

     public static string GetStr() 
     { 
      return ""; 
     } 
     public static void Main() 
     { 
      GetStr(); 
     } 
    } 

Answer 4

No puedo comentar las respuestas, ya que no tengo créditos de stackoverflow. Pero estoy de acuerdo con Chris en que es totalmente legal y a menudo deseable no asignar valores devueltos por un método. También ocasionalmente no es deseable. p.ej.

public static int Square(this int myValue) 
{ 
    return myValue * myValue; 
} 

Está claro que llamar a este método sin asignarlo es probablemente un error. Creo que crear una regla de análisis de código que advirtiera cada vez que no asignaba un valor como Massimiliano sugirió sería peor que no tener la regla en absoluto. En tales casos, sería bueno poder aplicar un atributo con el método ...

[MustAssign] 
public static int Square... 

Se puede crear una regla como se sugiere Massimiliano pero sólo invocar la advertencia cuando el método está adornada con el atributo y el el valor devuelto por el método no está asignado. No es un ejercicio trivial sin embargo.

Answer 5

Puede usar Resharper para ayudar con este problema; lo necesario para decorar su método con el atributo [Pure]:

[Pure] 
public static IList<T> RemoveItem<T>(this IEnumerable<T> thisList, T item) 
{ 
    var list = thisList.ToList(); 
    list.Remove(item); 
    return list; 
} 

a continuación, cuando usted lo llama sin asignar el valor de retorno se verá:

El atributo [Pure] se define en Resharpers datos Anotaciones : Necesita copiar las clases en su proyecto para que pueda referenciarlas (muchas otras anotaciones muy útiles también)