ludica-squamata / mano-gift

Proyecto de engine para el juego, usando pygame.
0 stars 0 forks source link

Nuevos estadisticas para los mobs (issue #42): #125

Closed danielrossyamitrano closed 6 years ago

danielrossyamitrano commented 6 years ago

Ahí están implementados los nuevos atrributos, pero hay cosas que no me gusta como quedaron. En el commit están mis observaciones.

einacio commented 6 years ago

si, a mi tampoco me convence que el ataque_especial sea un consumible. deberia ser otra caracteristica igual que salud, con maximo y actual.

yo consideraria poner las caracteristicas dentro de una propiedad caracteristicas en el objeto, asi me parece que seria mas extensible sin contaminar el espacio global (o sea, despues se pueden agregar caracteristicas sin temer que el nombre de la variable se pise con el de alguna de las otras 5 o 6 subclases)

respecto del nombre del bloque, la clase que lo lee se llama Atribuido, asi que capaz seria mas consistente "atributos" en vez de "caracteristicas"?

por que en mob.py sacaste el tag solido? hostil y pasiva lo entiendo porque eso ahora queda en territorio de AI (aunque este no seria el branch adecuado para limpiar eso), pero solido no tiene un reemplazo que yo recuerde, y puede ser util para cosas estilo fantasmas o god mode

danielrossyamitrano commented 6 years ago

Voy punto por punto.