FedeMacias / Tp-objetos2018

0 stars 0 forks source link

Constructores innecesarios en la armadura, y valor inicial del refuerzo #7

Open sanuzzi opened 6 years ago

sanuzzi commented 6 years ago

Por empezar, para la entrega 1 podía ser un WKO la armadura. Pero asumiendo que se hizo para entregas siguientes, comento que los constructores están de más.

Si quieren que por defecto el refuerzo sea 0, hagan "var property refuerzo = 0" y listo.

Igual, es raro que el refuerzo sea un número. ¿La idea del refuerzo no es una entidad más compleja? Debería poder ser el hechizo directamente o el refuerzo que sea.

Y ahí nos preguntamos: ¿Con qué refuerzo arranca? Vean lo que les decía de los issues anteriores, de tener un objeto que simbolice cuando estamos "sinRefuerzo". Podemos modelar "la nada" como nos sea más útil.

Por último, en vez de los constructores convencionales, recuerden que pueden usar los constructores con parámetros nombrados. Revisen apuntes si hay dudas, en general ya no vamos a codificar un constructor nunca (o casi nunca) más.