FedericoFuentesWeber / TPs-PdeP-2018

0 stars 0 forks source link

5) Para este TP, al final, no hacen falta fixtures #37

Closed sanuzzi closed 6 years ago

sanuzzi commented 6 years ago

Me parece bien que hayan creado a sus objetos afuera de los fixtures. La documentación de wollok ahora dice de forma más clara que esto está correcto porque en cada corrida los vuelve a crear.

Y ya que hicieron esto, podrían inicializar todo. Por ejemplo, tienen esto en el de lucha más avanzada:

    const espadaDelDestino = new ArmaDeFilo()
    const mascaraOscura = new Mascara(indiceDeOscuridad = 1)
    const armadura = new Armadura()
    var rolando = new Personaje()

    fixture{
        rolando.hechizoPreferido(new Hechizo(nombre = "espectro malefico"))
        rolando.agregaUnosArtefactos([espadaDelDestino, collarDivino, mascaraOscura, armadura, espejoFantastico])
    }

Y podrían tener esto:

    const espadaDelDestino = new ArmaDeFilo()
    const mascaraOscura = new Mascara(indiceDeOscuridad = 1)
    const armadura = new Armadura()
    var rolando = new Personaje(
        hechizoPreferido = new Hechizo(nombre = "espectro malefico"),
        artefactos = [espadaDelDestino, collarDivino, mascaraOscura, armadura, espejoFantastico]
    )