jueves, 19 de enero de 2012

RWJ. Keep it simple!!

"La belleza de lo simple"

boolean orderByAsc = false;
if (orden.equals("asc")) {
    orderByAsc = true;
}

El proyecto que estoy sufriendo está lleno de cosas así. ¿El programador no podría haber hecho algo tan fácil como... ?

boolean orderByAsc = "asc".equals(orden);

Y además, no petaría si orden fuese null.

4 comentarios:

  1. Parece mentira Pedro que no estés acostumbrado a eso.
    Hay gente que si no escribe true o false en la asignación de una variable no se queda tranquilo.

    código como este

    if (condicion)
    return true;
    else
    return false;

    es el pan nuestro de todos los días. Y en java lo del equals al revés, ya es un estandar de facto

    ¿no ves que se lee todo mucho mejor?

    Pareces nuevo por estos lares :P

    ResponderEliminar
  2. Me encantan los comentarios "constructivos" xD Menos mal que la esencia del post es otra :P

    ResponderEliminar
  3. Lo siento, Pedro, pero a mi ya me tiene pasado con una línea de código parecida a la que propones, que luego me vengan a preguntar que 'cojones' hace esa línea.
    Mucha gente no es capaz de asimilar más de un concepto por línea. Hacer una declaración y una comparación en una sola línea puede saturarles.

    ResponderEliminar
  4. Creo que si un programador (Java) no domina un equals(), mucho menos va a ser capaz de entender lo que es la reflexión o un proxy dinámico().

    A mi me cuesta mucho más entender lo que hacen cuatro líneas de código (y que, además petan si el objeto es null) que poner mi simple equals().

    Eso si, vosotros en .NET poneis orden == "true"

    ResponderEliminar