keze87 / TP2

0 stars 1 forks source link

Definicion de modelo #10

Closed ldtg closed 7 years ago

ldtg commented 7 years ago

Gente estaria bueno definir y ver que modelo vamos a usar antes de ponernos a codear asi no hacemos cosas al pedo. Y si quieren del que hagamos puedo hacer y ir subiendo el diagrama de clases asi tenemos algo como guia para ir codeando.

guido-z commented 7 years ago

Estoy de acuerdo, también tenemos que dividirnos qué clases va a coder cada uno, sino nos vamos a volver locos. Yo propongo que nos enfoquemos en lo que tenemos que entregar el lunes, y por el momento dejemos de lado lo demás. Otra cosa, vamos a implementar los tipos de algomon como clases que heredan de AlgoMon o usamos la Interfaz tipo como habían propuesto antes?

Voten(?

El 5 de noviembre de 2016, 21:54, Luis Tejerinanotifications@github.com escribió:

Gente estaria bueno definir y ver que modelo vamos a usar antes de ponernos a codear asi no hacemos cosas al pedo. Y si quieren del que hagamos puedo hacer y ir subiendo el diagrama de clases asi tenemos algo como guia para ir codeando.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG469HztZUK7Bv2i0u60AdXa0PCukCks5q7SVdgaJpZM4Kqbm6 .

ldtg commented 7 years ago

Claro la idea es hacer lo que hay que hacer para el JUEVES jajaja

ldtg commented 7 years ago

Y por eso necesitamos definir si vamos a usar la idea de los ataques como que aplican efectos y eso o de la otra forma. El problema es que hay que pensar si de la otra forma despues no nos va a costar la parte de los estados y elementos.

guido-z commented 7 years ago

Podríamos hacer lo de los efectos ahora, por ejemplo agregando un método aplicarEfecto a la clase Ataque. Y cada ataque que hereda puede reeimplementarlo o no.

2016-11-05 22:09 GMT-03:00 Luis Tejerina notifications@github.com:

Y por eso necesitamos definir si vamos a usar la idea de los ataques como que aplican efectos y eso o de la otra forma. El problema es que hay que pensar si de la otra forma despues no nos va a costar la parte de los estados y elementos.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653389, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG4wUPWnu0fs_93etjzMdNcuqL99dHks5q7SisgaJpZM4Kqbm6 .

guido-z commented 7 years ago

Pero mantengamos sencillo lo que vamos a hacer ahora, porque si tratamos de pensar lo que tenemos que hacer más adelante y tratamos de codear pensando en lo que tenemos que hacer en entregas posteriores nos vamos a volver locos.

El 5 de noviembre de 2016, 22:11, Guido Zaglulguido.zaglul@gmail.com escribió:

Podríamos hacer lo de los efectos ahora, por ejemplo agregando un método aplicarEfecto a la clase Ataque. Y cada ataque que hereda puede reeimplementarlo o no.

2016-11-05 22:09 GMT-03:00 Luis Tejerina notifications@github.com:

Y por eso necesitamos definir si vamos a usar la idea de los ataques como que aplican efectos y eso o de la otra forma. El problema es que hay que pensar si de la otra forma despues no nos va a costar la parte de los estados y elementos.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653389, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG4wUPWnu0fs_93etjzMdNcuqL99dHks5q7SisgaJpZM4Kqbm6 .

ldtg commented 7 years ago

Dale!! Hay que dividirse lo que hay que hacer no? Porque yo quiero empezar a hacer algo pero como no se si alguien ya lo hizo y no lo subio o lo esta haciendo me da cosa

guido-z commented 7 years ago

Estaba viendo tu diagrama, me parece que está bueno, y ahora podríamos empezar a codear la porción de la derecha, que es lo que necesitamos. Yo tengo una buena idea para el asunto de los tipos, es más, ya codeé las clases a la tarde, pero no las subí porque no nos habíamos puesto de acuerdo con el modelo.

El 5 de noviembre de 2016, 22:14, Guido Zaglulguido.zaglul@gmail.com escribió:

Pero mantengamos sencillo lo que vamos a hacer ahora, porque si tratamos de pensar lo que tenemos que hacer más adelante y tratamos de codear pensando en lo que tenemos que hacer en entregas posteriores nos vamos a volver locos.

El 5 de noviembre de 2016, 22:11, Guido Zaglulguido.zaglul@gmail.com escribió:

Podríamos hacer lo de los efectos ahora, por ejemplo agregando un método aplicarEfecto a la clase Ataque. Y cada ataque que hereda puede reeimplementarlo o no.

2016-11-05 22:09 GMT-03:00 Luis Tejerina notifications@github.com:

Y por eso necesitamos definir si vamos a usar la idea de los ataques como que aplican efectos y eso o de la otra forma. El problema es que hay que pensar si de la otra forma despues no nos va a costar la parte de los estados y elementos.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653389, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG4wUPWnu0fs_93etjzMdNcuqL99dHks5q7SisgaJpZM4Kqbm6 .

guido-z commented 7 years ago

Claro. Igual si vamos a seguir tu modelo, con lo cual yo estoy de acuerdo, una parte de lo que tenemos hasta ahora vamos a tener que volarla

2016-11-05 22:18 GMT-03:00 Luis Tejerina notifications@github.com:

Dale!! Hay que dividirse lo que hay que hacer no? Porque yo quiero empezar a hacer algo pero como no se si alguien ya lo hizo y no lo subio o lo esta haciendo me da cosa

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653675, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG43f_y-SoUPUdAnjmRnev7xsqR-xJks5q7SrlgaJpZM4Kqbm6 .

guido-z commented 7 years ago

Fijate qué te parecen, no las agrego al proyecto para evitar errores locos.

El 5 de noviembre de 2016, 22:19, Guido Zaglulguido.zaglul@gmail.com escribió:

Claro. Igual si vamos a seguir tu modelo, con lo cual yo estoy de acuerdo, una parte de lo que tenemos hasta ahora vamos a tener que volarla

2016-11-05 22:18 GMT-03:00 Luis Tejerina notifications@github.com:

Dale!! Hay que dividirse lo que hay que hacer no? Porque yo quiero empezar a hacer algo pero como no se si alguien ya lo hizo y no lo subio o lo esta haciendo me da cosa

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653675, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG43f_y-SoUPUdAnjmRnev7xsqR-xJks5q7SrlgaJpZM4Kqbm6 .

ldtg commented 7 years ago

Si por eso tenemos que estar todos de acuerdo!

ldtg commented 7 years ago

Donde las subiste? no conozco mucho github perdon jaja

guido-z commented 7 years ago

Adjunté un .rar en mi último mensaje

2016-11-05 22:26 GMT-03:00 Luis Tejerina notifications@github.com:

Donde las subiste? no conozco mucho github perdon jaja

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653936, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG48PllKFEgx17iRq3RkhgmBLMYXQ7ks5q7SzGgaJpZM4Kqbm6 .

guido-z commented 7 years ago

Ah, es que lo estás viendo en github, yo estoy acostumbrado a responder por el mail, ahora lo subo de nuevo a github.

2016-11-05 22:27 GMT-03:00 Guido Zaglul guido.zaglul@gmail.com:

Adjunté un .rar en mi último mensaje

2016-11-05 22:26 GMT-03:00 Luis Tejerina notifications@github.com:

Donde las subiste? no conozco mucho github perdon jaja

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258653936, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG48PllKFEgx17iRq3RkhgmBLMYXQ7ks5q7SzGgaJpZM4Kqbm6 .

guido-z commented 7 years ago

Listo, fijate el archivo que se llama Desktop.zip, aparece junto con los demás del proyecto. Después lo borro.

ldtg commented 7 years ago

Ahi lo mire. Esta bueno asi para mi, esperemos a ver que dicen los demas!

keze87 commented 7 years ago

Por mi, todo bien.

Pd: por favor pongan esto en sus eclipses:

http://stackoverflow.com/questions/14178839/is-there-a-way-to-automatically-remove-trailing-spaces-in-eclipse

Pd2: si, se que soy molesto :smiling_imp:

wjma commented 7 years ago

todo bien por mi

guido-z commented 7 years ago

Ahí lo configuré. Bueno, si vamos a usar el diseño de Luis agrego la implementación de los tipos y actualizo el proyecto. ¿Está bien si saco las clases AtaqueFuego, AtaqueNormal, etc?

2016-11-05 23:07 GMT-03:00 keze87 notifications@github.com:

Por mi, todo bien.

Pd: por favor pongan esto en sus eclipses:

http://stackoverflow.com/questions/14178839/is-there-a- way-to-automatically-remove-trailing-spaces-in-eclipse

Pd2: si, se que soy molesto 😈

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/10#issuecomment-258655517, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG49DSj05JeqxN_5r8BRfQ3dq2AYJFks5q7TZfgaJpZM4Kqbm6 .

keze87 commented 7 years ago

Ni siquiera preguntes

guido-z commented 7 years ago

Ahí actualizé el modelo. Al final no borré las clases que dije que iba a borrar, sino que las usé para definir el atributo tipo para los distintos tipos elementales de algomon y ataques. Mírenlo que lo van a entender mejor que leyendo esto. También se me ocurrió hacer un builder para los ataques. ¿Qué les parece?