java booléen de l'instruction return

Je suis en train de programmer un jeu, et je suis en train de faire des méthodes pour contrôler les différents côtés d'un acteur de terrain. Je suis à l'aide d'un booléen, mais netbeans me dit que je n'ai pas d'instruction return.

public boolean checkTerrainDown(Level levelToCheck){
    for(Terrain terrainToCheck: levelToCheck.levelTerrain){
        if(y+h<terrainToCheck.getY()){
            return true;
        }else{
            return false;
        }
    }
}
  • Lorsque votre méthode doit retourner quelque chose, vous devez fournir une déclaration pour chaque chemin d'exécution qui peuvent être prises.
  • C'est une bonne pratique de n'avoir qu'une seule instruction de retour! en passant , vous n'avez pas à faire une for pour ce que vous faites.. vous ne demandez pour le premier élément.. en plus de l'utilisation return y+h < terraintoCheck.getY();
  • Non, il n'est vraiment pas. C'est une bonne pratique afin de rendre votre code plus clair possible. Parfois, cela signifie un retour de l'instruction, parfois, cela signifie plusieurs.
  • vous l'avez droit, c'est un decission que je n'aime pas les multiples renvoie la plupart du temps, je pense que je ne l'utilise plusieurs retours pour le "cas faciles", mais lisible par un code propre est ce que nous avons à point :).
InformationsquelleAutor user2817232 | 2013-09-25
  1. 6

    Que faire si il n'y a pas de Terrain vérifier? Ensuite, le corps de la for boucle n'est jamais exécutée. Vous n'avez pas de return déclaration après la for boucle pour tenir compte de ce cas. Que voulez-vous, Java est de retour dans ce cas?

    Place un return déclaration après la for boucle pour gérer le cas où il n'y a pas Terrain dans le Level's levelTerrain. De cette façon, chaque cas possible de l'exécution return quelque chose.

    • Aussi la for boucle est inutile ici..
    InformationsquelleAutor rgettman
  2. 1

    si le for loop n'a pas été exécuté alors il n'y a pas d'instruction return est exécutée.

    • Ce que vous entendez n'est pas clair. Améliorer votre réponse, s'il vous plaît.
    InformationsquelleAutor Yellow Flash
  3. 1
    public boolean checkTerrainDown(Level levelToCheck){
        //add this line
        boolean mark = false;
    for(Terrain terrainToCheck: levelToCheck.levelTerrain){
        if(y+h<terrainToCheck.getY()){
                //add this line,remove this //return true;
            mark = true;
            //add this line
            break;
        }
        //else{
            //return false;
        //}
    }
    //add this line
    return mark;
}
    InformationsquelleAutor sunysen