Feda /dev/ - No Javascript allowed

#8394MTX_Anubis:

en ese caso la comprobación debería ir en el setter, no fuera de él

No esoy nada de acuerdo, tu sabes algo de los SOLID PUTO SUBNORMAL?

Ahora enserio, estas haciendo el codigo poco entendible para empezar, para empezar no deberia haber get/set si pretendes hacer oop puro. Segundo, una vez cambiado ese set() por un verbo, te puedo comprar que este la logica de verificacion dentro si la signatura de esa funcion es explicita retornando un Result, es una funcion pura y total para el dominio (no hay dudas del dominio valido).

De hecho si estas haciendo funciones puras y bien definidas, no tiene sentido la logica dentro ya que no es un input valido (del dominio de inputs de ese "setter"). Yo tendria una funcion wrapper:

funcion wrapper(input_raw) -> Result[OK | KO]:
  if is_valid(input_raw) -> method(input_raw)
  else ->  KO

Asi no veria problemas.

Dicho esto que seria la solucion buena, la tuya esta bastante bien y tampoco pasaria nada por un poco de codigo spaghetti... eso si, varias de esa y te tiro la PR

De tener null/undefined en el dominio ... Mejor ni comento, cuando cojas un libro y lo leas lo volvemos a discutir.

#8403 Bueno mi critica mas bien iba dirigida a ser una funcion pura y bien definida para que no haya confusion...el tema de los get/set sabiendo que en 2020 aun hay gente que lo usa y se cree que esta bien.. yo ni entro, los uso si trabajo con monos, sino no.

#8395 O sea que tu usas bucles for en vez de forEach/each o creas un array/lista y usas otro for para ir llenándolo en vez de usar map

Porque es bastante conciso ese código y si me apuras escribes un comentario explicando el flatten
usamos flatten porque podemos recibir un elemento o una lista
Y más humano que eso imposible y te ahorras un código guarro.

Y si no sabes lo que hacen los métodos flatten o compact entonces tu problema no es que el código sea legible o no.

#8401 lo he escrito de memoria y según escribía el post xD. Fue hace unos años y no recuerdo exactamente que se devolvía si es que se devolvía algo al final, lo único los pasos de checkeos de null, array y elemento.

#8402 Yo pienso que no te lees los post y escribes las primeras subnormalidades que te viene a la cabeza. Estás a 1 o 2 posts de ser la primera persona que ignoro en MV

.

#8410 Usar getter/setters rompe uno de los principios básicos de la programación orientada a objetos, encapsulación de datos. Pero como dicen por ahí, que cada uno haga lo que le salga del papo.

Si, mejor debatir sobre cuantos tractores hay en el pueblo de cada uno.

#8414 Había más nivel en ese debate que en este

#8419 Por ahí hay un meme con un listado de libros que no se lee nadie. Una cosa es no exponer la estructura del objeto y otra que no puedas modificarlo.

#8421 estas muy pesado hoy aqui no se viene a aprender

Si eres capaz de entenderlo ya nos diras, ya ves que hay gente que lleva 15 a;os picando y aun no sabe por donde le da. Yo confio en ti, se te ve con ganas de aprender y la constancia es la clave.

edit: cierro por hoy no me citeis.

#8422 Te citaré si me sale de los cojones.

#8421 Simplificandolo mucho:

NO

class Point {
  private int x;
  private int y;

  void setx(int x) {...}
  int getx() {...}
  ... 
}

SI

class Point {
  private int x;
  private int y;

  int coordinate(int x, int y) {...} // 0, 1, 2, 3
  ... 
}