Format My Source Code for Blogging

Está perfecto... me volvieron las ganas de seguir escribiendo código desastroso.

Format My Source Code for Blogging

Comentario super específico

Un comentario revelador...

//Si pone cualquier numero devuelve null
private static MethodBase GetMethod(Int32 number)
{
....
}

obviamente, antes de usar este método tenemos que leer el código fuente!!!

Qué quisieron decir? Con cualquier número de los infinitos posibles retorna un null? Si pone un número que es cualquiera retorna null? Get qué method???

Casteate algo...

Código de un iterador...

public class MyIterador:Enumerator
....
   private object myArray;
.....

   public virtual bool MoveNext()
   {
       return (position < length);
   }

   public virtual object Current
   {
       get
       {
           object result = ((System.Array) myArray).GetValue(position);
           position++;
           return result;
       }
   }
}

y castéa para cada vuelta... para cada vuelta... para cada vuelta....

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!!!