1

He definido el siguiente enum con las posiciones de los jugadores en el campo.

Quiero asociar las posiciones a colores mediante getColor y lo hago de la siguiente forma:

public class Position {

    enum Position{
         GOALKEEPER, DEFENDER, MIDFIELDER, FORWARD;
    }

    public String getColor(Position position) {
        switch (position){
            case GOALKEEPER:
                return "BLACK";
            case DEFENDER:
                return "RED";
            case MIDFIELDER:
                return "GREEN";
            case FORWARD:
                return "BLUE";
        }
    }   // error - missing return statement
}

Los test que debo pasar son los siguientes:

void testGetColor() {

        position = Position.GOALKEEPER;
        assertEquals("BLACK", position.getColor());

        position = Position.DEFENDER;
        assertEquals("RED", position.getColor());

        position = Position.MIDFIELDER;
        assertEquals("GREEN", position.getColor());

        position = Position.FORWARD;
        assertEquals("BLUE", position.getColor());
}

Tengo un error missing return statement (que lo indico dentro del código) pero no entiendo por qué si en cada caso hago un return

Qué es lo que estoy haciendo mal?

Gracias.

1 respuesta 1

0

Debes añadir un caso default: en el switch. Por ejemplo:

    switch (position){
        case GOALKEEPER:
            return "BLACK";
        case DEFENDER:
            return "RED";
        case MIDFIELDER:
            return "GREEN";
        case FORWARD:
            return "BLUE";
        default:
            return "NO-COLOR";// O throw new RuntimeException("Posicion no valida")

    }

Si no, el switch puede tener casos sin retorno, y es lo que provoca el error.

Saludos !

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.