Preguntar por si acaso...

private void Cerrar()
{

if (ObjetoNoCargado(obj))

return;

/*...muchas lineas de código

haciendo cosas locas... y finalmente:*/

this.Close();

}


Se supone que estoy cerrando alguna la página.... ObjetoNoCargado ??
Casi como que resulta innecesario el contexto para que se den cuenta que el if está descolgadísimo.
Segundo, ya podríamos evitar los return y los breaks y los halts y empezar usar un identador, no?

Retornar por las dudas

public bool ValidarLista(IList lista, HttpContext context)
{

if (... == false)
{

context.Response.Redirect("~/Invalid.aspx");
return false;

}
else

return true;

}


y se supone que tengo que saber que si falla me mandan allá.... el return para que servía??

Calentando los motores

Comencemos con un caso que lo tiene todo:

public class CalendarItem
{

...
public virtual bool IsMileston()
{

return this.Item is ProjectMilestone;

}

}

....todo: un método por propiedad, un 'is' cortaextensibilidad, una llamada a propiedad por si acaso, le falta una 'e' al método y, para sacarnos de toda duda, un categórico 'virtual'... como si alguien fuera a sobreescribir esto!!!