LucasGaleano
commented
5 years ago El problema es el que decís vos qué falla la lista vacía con max y no queríamos usar un if para ver si estaba vacia
Open sanuzzi opened 5 years ago
LucasGaleano
commented
5 years ago El problema es el que decís vos qué falla la lista vacía con max y no queríamos usar un if para ver si estaba vacia
Veo que en la segunda sección de tests hicieron esto en el fixture: rolando.removeTodosLosArtefactos()
Ahí creo que están un poco a la defensiva de que quizá el día de mañana la lista aparezca con cosas.
Si quieren, siguiendo a la defensiva, podrían inicializar al Personaje usando el constructor de parámetros nombrados para decir cuáles son sus artefactos en la misma creación.
Aunque para eso, tendrían que repensar lo del artefacto "ninguno", y resolver su duda: // como hago para no necesitar el objeto 'ninguno' en artefactos, por el tema del espejoPreferido
No me queda claro el problema. ¿Sería que si la lista es vacía, el max falla? Si es así, no es un problema. Buscar el máximo de una lista vacía no tiene sentido, está bien que falle. ¿Era eso u otra cosa? Si quieren lo vemos en clase.