Fallo en metodo para comprobar antes de hacer un Return

[DCienfuegos]

Os comento, me piden lo siguiente:

Implemente un método recogerCosecha() que devuelva el númeroDeFrutas del árbol y ponga a cero el valor de esa propiedad .
Asegúrese que el estado de un objeto presente siempre valores de númeroDeFlores y NúmeroDeFrutas iguales o superiores a cero.

Y este es el codigo que tengo:

Código:

    
    public int recogerCosecha()
    {
        // comprobamos que numeroDeFlores y numeroDeFrutas es igual o superior a cero
        if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
           return numeroDeFrutas;
           numeroDeFrutas=0;
        }
        

    }

Me da de error: unreacheable stament en numeroDeFrutas=0; y si quito esa linea me da como que espera el return pero por estar en el if no me lo lee, si lo pongo fuera si compilaria pero claro no lo estaria comprobando.

¿como deberia de ser?

gracias de antemano!

[Fuzzylog]

cuando se retorna no se puede ejecutar más código, salvo que estés en un bloque try-catch-finally, donde el finally se ejecutará antes que el return.

Debería ser:

if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
int valorRetornar = numeroDeFrutas;
numeroDeFrutas = 0;
return valorRetornar ;
}

Donde usas una variable local auxiliar para almacenar el valor del número de frutas antes de modificarlo.

[DCienfuegos]

Iniciado por Fuzzylog

cuando se retorna no se puede ejecutar más código, salvo que estés en un bloque try-catch-finally, donde el finally se ejecutará antes que el return.

Debería ser:

if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
int valorRetornar = numeroDeFrutas;
numeroDeFrutas = 0;
return valorRetornar ;
}

Donde usas una variable local auxiliar para almacenar el valor del número de frutas antes de modificarlo.

He puesto lo que me has dicho:

Código:

public int recogerCosecha()
    {
        // comprobamos que numeroDeFlores y numeroDeFrutas es igual o superior a cero
        if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
            int valorRetornar = numeroDeFrutas;
            numeroDeFrutas = 0;
            return valorRetornar ;
        }
        

    }

Pero me sigue dando error al compilar: Missing return statement

[Fuzzylog]

xD, no me fijé.

Si pones un if, debes poner un return en caso de que no entre en el if.

public int recogerCosecha()
{
// comprobamos que numeroDeFlores y numeroDeFrutas es igual o superior a cero
if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
int valorRetornar = numeroDeFrutas;
numeroDeFrutas = 0;
return valorRetornar ;
}
//En tu caso
return 0;
}

[DCienfuegos]

Iniciado por Fuzzylog

xD, no me fijé.

Si pones un if, debes poner un return en caso de que no entre en el if.

public int recogerCosecha()
{
// comprobamos que numeroDeFlores y numeroDeFrutas es igual o superior a cero
if (numeroDeFlores >= 0 && numeroDeFrutas >=0) {
int valorRetornar = numeroDeFrutas;
numeroDeFrutas = 0;
return valorRetornar ;
}
//En tu caso
return 0;
}

Me di cuenta despues tambien, pero puse return -1, ya que cero en el if si que me lo puede devolver. Gracias por la ayuda ;)

[Fuzzylog]

No hace falta que cites continuamente los comentarios previos en el foro. Salvo que te refieras a un comentario concreto muy anterior sobre el que quieras remarcar algo, se sobreentiende a qué comentario estás respondiendo.