Corrección - Githubissues

Juancete commented 6 months ago

Hola @FariFede

Te paso algunos comentarios del código

Las clases tienen que usar nombre el mayúscula. Es la convención que adoptan la mayoría de los lenguajes orientados a objetos.
La misión tiene detalle. Qué modela detalle? Depende de dónde se utiliza tiene un significado distinto.
El siguiente código está ubicado en Misión
```
(this.detalle === 4 || this.detalle === 5 || this.detalle > 100);
```
Lo cual no tiene sentido porque comparar la cantidad de trolls es cuestión de rescatar princesa mientras que distancia es cuestión de encontrar un objeto. Entonces estas responsabilidades deberían estar en cada sub-clase.
¿listaMisiones qué modela? Si es una lista, la misma ya existe y está dentro de Heroe. De hecho total de puntos lo pregunto al héroe y l resuelve con su lista mientras que misiones difíciles usa la lista que tiene listaMisiones. Ojo ahí que son dos listas diferentes.
Falta mostrar los nombres de los solicitantes de las misiones.

Fíjate de hacer un refactor del código con estos tips y avísame así lo vemos.

Saludos, Juan

FariFede commented 6 months ago

Hola @Juancete

He modificado lo que me comentaste. Me resistí a usar forEach en la sintaxis de la lista de solicitantes, pero con map se me hizo engorroso (me arrojaba un error de referencias, o algo así). El resto de las correcciones pueden leerse en la descripción del commit anterior.

Gracias por la espera!

Juancete commented 6 months ago

Hola @FariFede

Mejoró bastante el código. Te paso las observaciones:

Ojo con esto
```
esDificil() { // hay misiones normales y misiones dificiles, las cuales se dan en condiciones especificas de las subclases
    return false;
}     
```
Es correcto que hay condiciones específicas para cada sub-clase pero hay un comportamiento general que estás repitiendo en cada una de ellas que es this.solicitante.charAt(0) === 'G'. Esto puede subir a la super clase y reutilizarse con super.
detalle sigue modelado más de una cosa. Está en la super clase y fíjate que la misma no la utiliza en ningún método. Por lo tanto podemos eliminarla de ahí y pasarla a cada sub-clase y que tenga expresividad el nombre para identificar qué está modelando.
Para qué está la variable tipo? no se utiliza más.

Saludos! Juan

FariFede commented 6 months ago

Hola Juan,

Ya lo modifiqué, ahí lo subo con estos cambios.

Respecto a la condición del nombre empezado por G: lo dejé en cada subclase suponiendo que a futuro habría misiones difíciles sin necesidad de que empiecen por esa letra. No quise generalizar tanto, aunque como comentas, podría usarlo con la super clase (sin tanto cambio de sintaxis).

Con detalle, pensé que lo necesitaba en el constructor de la super clase porque sino me iban a faltar parámetros, pero como comentas no haría falta ya que en realidad cuando invoco el método Shrek.hacerMision estoy aclarando las misiones específicas, y no a la super clase.

Slds!

Obtener Outlook para iOShttps://aka.ms/o0ukef

De: Juan José Contardo @.> Enviado: Friday, April 26, 2024 10:31:21 AM Para: FariFede/Javascript @.> Cc: FariFede @.>; Mention @.> Asunto: Re: [FariFede/Javascript] Corrección (Issue #1)

Hola @FariFedehttps://github.com/FariFede

Mejoró bastante el código. Te paso las observaciones:

Ojo con esto

esDificil() { // hay misiones normales y misiones dificiles, las cuales se dan en condiciones especificas de las subclases return false; }

Es correcto que hay condiciones específicas para cada sub-clase pero hay un comportamiento general que estás repitiendo en cada una de ellas que es this.solicitante.charAt(0) === 'G'. Esto puede subir a la super clase y reutilizarse con super.

detalle sigue modelado más de una cosa. Está en la super clase y fíjate que la misma no la utiliza en ningún método. Por lo tanto podemos eliminarla de ahí y pasarla a cada sub-clase y que tenga expresividad el nombre para identificar qué está modelando.

Saludos! Juan

— Reply to this email directly, view it on GitHubhttps://github.com/FariFede/Javascript/issues/1#issuecomment-2079403440, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AWDYK3VI27WA45D2RHINFV3Y7JJKTAVCNFSM6AAAAABGRWDPQ6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZZGQYDGNBUGA. You are receiving this because you were mentioned.Message ID: @.***>

Juancete commented 6 months ago

Hola @FariFede

Respecto a la condición del nombre empezado por G: lo dejé en cada subclase suponiendo que a futuro habría misiones difíciles sin necesidad de que empiecen por esa letra. No quise generalizar tanto, aunque como comentas, podría usarlo con la super clase (sin tanto cambio de sintaxis).

Ok, pero en la actualidad es así y hay una lógica duplicada. Por lo tanto está bueno que el modelo se adapte a la realidad de hoy en día. Si aparece el requerimiento de una sub-clase en particular no lo tiene, directamente no usas el super y definís ese comportamiento fuera de lo común. El mañana nunca llega diría un docente que conozco y tiene razón. Cuando llega el momento, es donde refactorizamos y seguimos con el nuevo requerimiento.

Ojo que haces en el constructor
```
this.custodia = custodia;
```
que es custodia? no lo recibe por parámetro el constructor. Entonces tenés que pasarlo en el contructor de cada sub-clase y tener la property local a esa clase.
fíjate que la variable tipo no la utilizás en los métodos. Por lo tanto se puede eliminar también.

Saludos!

FariFede commented 6 months ago

@Juancete

Listo, me había olvidado de agregarlos en el constructor (y de modificarlo en los métodos también). También borré el tipo, ya que como comentas, en este momento no se utiliza. Ante una refactorización, podría evaluarse.

Avisame por favor, saludos!

Juancete commented 6 months ago

Buenísimo Fede!! Quedó muy bien

FariFede commented 6 months ago

Gracias Juan! Voy a usar este ejercicio como punto de partida para seguir aprendiendo.

Programar en React no tenía muchas similitudes en cuanto a la sintaxis, verdad? Si bien cada uno programa como "quiere/puede",

Juancete commented 6 months ago

Hola @FariFede

Voy a usar este ejercicio como punto de partida para seguir aprendiendo.

Me parece perfecto

Programar en React no tenía muchas similitudes en cuanto a la sintaxis, verdad? Si bien cada uno programa como "quiere/puede",

React es JS para el modelado de comportamiento de la app pero para el render utiliza JSX que es una mezcla entre Javascript y HTML. Para lograr comprender los conceptos que hay por detrás de la tecnología te recomiendo que antes veas

Pdep UTN: Acá se ve objetos con Wollok como vimos en el curso. Obviamente hay casi medio año de cursada dedicado a esto y los chicos llegan cono otro background luego de cursar paradigma funcional y lógico. Conceptos que son cross-paradigma como expresividad, declaratividad, etc. ya los tienen más incorporados. Te paso las clases 2023 que están completas y te puede ayudar como repaso.
Algo 2 UNSAM. Te paso el link de la cursada 2023 porque la actual no está completa: https://algo2.uqbar-project.org/cursadas-historicas/2023. Acá vas a ver lógica de negocio pero vas a encontrar patrones de diseño para problemáticas recurrentes. Entonces vas a ver conceptos de diseño que son más complejos que los que vimos hasta ahora.
Algo 3 UNSAM Acá vas a arrancar con maquetado HTML y CSS. Luego vas a pasar a ver un framework que es más orientado a objetos que es Angular y luego pasás a ReactJS que tiene un approach funcional.

Si bien es más largo, creo que transitar ese camino hace que tengas otros conceptos incorporados que te benefician para entender qué es lo que ocurre con estos Frameworks de vista.

Saludos! Juan

FariFede / Javascript

Corrección #1