lunes, 28 de mayo de 2012

Festival del Humor (VI)

Hay veces que no doy crédito:

public static Boolean esCanalSeguro(ContentInstance ci,String language){
  String isHttps=getPropertyValueFromLocale(ci,ATR_HTTPS,language,"");
  if ("1".trim().equals(isHttps)) return Boolean.TRUE;
  if ("0".trim().equals(isHttps)) return Boolean.FALSE;
  //si no tiene valor suponemos que no es https
  return null;
}

Seguro que muchos me direis que no es para tanto ...

3 comentarios:

  1. Ooooolé. Cuánto genio concentrado en tan pocas líneas.

    ResponderEliminar
  2. Como un código de unha liña que debería devolver un boolean simple ocupa 4 e devolve un obxecto?
    Pero o que non me deixar durmir disto son os Trim() ...

    ResponderEliminar
  3. La función trim() está ahí por programación defensiva. Ten en cuenta que cualquier programador malintencionado podría pasar por ese código y poner espacios dentro de la cadena...

    ResponderEliminar