Feda /dev/ - No Javascript allowed

#36961 soy abstemio, pero si que me preocupa que el precio de comer fuera de vez en cuando sea mucho más caro que aquí

#36967 te va dejar marca el reloj, parguela

#36967 te queda poca batería desu

#36975 roma no se hizo en 1 dia

vuelvo de la piscina y me encuentro con una review en una PR de mierda...

porque la gente discute mis decisiones?

antes: si un parametro era NULL o VACIO = IllegalArgument
mi pr de mierda: si un parametro NULL = NPE, si VACIO = IllegalArgument

me dicen que no... que cambia el comportamiento...

mi respuesta:

Then we should enrich the message at least or we are going to lose information both in comp and runtime... you won't be able to distinguish when was a null or "invalid" parameter if you always return an IllegalArgumentException...

It's better to have the two types so we don't lose that typing but whatever... I can go with the bad code it's not that critical.

although it's always better not to erase type information so you can switch on types...

la gente de java que usa nulls por dentro el codigo en lugar de optionals y luego te devuelve runtimeexceptions para todo en lugar de algo tipado en que mundo viven? en el 1970?

Lo de tener comportamientos iguales para null y vacío, sea el lenguaje que sea, es muy de junior. El que te ha hecho esa PR es junior, @desu?

#36983 Y tanto. Parsea el cuerpo de una llamada HTTP y mira a ver si un campo vacío da el mismo resultado que un campo que no está XD

#36985 Mi objetivo es probar sabores y texturas. Voy a seguir comiendo vacas pero también soja y tofu.

Os puedo poner el codigo que me la pela:

como estaba:

  public String generate(String path) {
    if (path == null || !path.startsWith("/")) {
      throw new IllegalArgumentException("Path should start with /");
    }

return  algo;

como deberia estar para no perder runtime info:

  public String generate(String path) {
    if (path == null) {
      throw new IllegalArgumentException("Path is null");
    }

if (!path.startsWith("/")) {
  throw new IllegalArgumentException("Path should start with /");
}

return algo;
  }

mi solucion (LA BUENA):

  public String generate(String path) {
    Objects.requireNonNull(path);

if (!path.startsWith("/")) {
  throw new IllegalArgumentException("Path should start with /");
}

return algo;
}

Tienes un NPE cuando un null y te dice donde, y un IAE para el caso del path no empieza con /

Alguien en el hilo no lo entiende de verdad? Algo tan basico? Abro stream?

#36989 no lo entiendes? xq hasta un ni;o de 12 a;os ya sabe estas cosas basicas.