tostrovsky / wollok

0 stars 0 forks source link

Ojo con el encapsulamiento #1

Open josibuttazzoni opened 4 years ago

josibuttazzoni commented 4 years ago

https://github.com/tostrovsky/wollok/blob/22d3df6ad9652b3a0c3db95c2fb8d11277d10ef8/src/luchadores.wlk#L80 En vez de modificar directamente la energía del contrincante, tendrías que mandarle un mensaje que fuera, por ejemplo, unContricante.perderEnergia(unContrincante.energia()*0,5).

josibuttazzoni commented 4 years ago

https://github.com/tostrovsky/wollok/blob/22d3df6ad9652b3a0c3db95c2fb8d11277d10ef8/src/luchadores.wlk#L112 Lo mismo acá para ver la cantidad de elementos que tiene, tendrías que hacer que entiendan el mensaje cantidadDeElementos() o algo así que devuelva el size de la lista.

josibuttazzoni commented 4 years ago
method robarItem(unContrincante) {
    unItem = unContrincante.inventario().last()
    inventario.add(unItem)
    unContrincante.inventario().remove(unItem)
}

Mismo acá, ya el nombre te tendría que hacer ruido. Fijate siempre de evitar "meterte" en el comportamiento de los otros objetos.