2010-07-29 5 views
0

J'ai un problème avec ce code:question Noob (Java): problème avec le jeu de cuirassés simple,

import java.util.Random; 

public class DotComObjects { 

public int[][] setBarcos(int tablero[][]) { 

    boolean repetido; //Variable booleana para comprobar si una casilla ya esta ocupada por otro barco 

    do { 

    repetido = false; //Suponesmos que no esta ocupada ninguna de las casillas 
    Random aRandom = new Random(); 
    boolean horizontal = aRandom.nextBoolean(); //Booleana al azar para la colocacion del barco. True = horizontal, false = vertical 

    if (horizontal == true) { //Si el barco va en horizontal 

    int ancho = aRandom.nextInt(tablero.length - 2); //Calculamos al azar la casilla 
    int alto = aRandom.nextInt(tablero[0].length); //central del barco (tendra 3) 

    if ((tablero[ancho - 1][alto] == 1) || (tablero[ancho][alto] == 1) || (tablero[ancho + 1][alto] == 1)) { //Si una de las casillas ya esta ocupada 

    repetido = true; //Variable booleana repetida en true 

    } 

    } else { //Si el barco va en vertical 

    int ancho = aRandom.nextInt(tablero.length); //Calculamos al azar la casilla 
    int alto = aRandom.nextInt(tablero[0].length - 2); //central del barco (tendra 3) 

    if ((tablero[ancho][alto - 1] == 1) || (tablero[ancho][alto] == 1) || (tablero[ancho][alto + 1] == 1)) { //Si una de las casillas ya esta ocupada 

    repetido = true; //Variable booleana repetida en true 
    } 

    } 

    } while (repetido == true); //Repetimos hasta que no haya una casilla ocupada (variable repetido = false) 

*** if (horizontal == true) { //Si el barco va en horizontal 

*** tablero[ancho - 1][alto] = 1; 
*** tablero[ancho][alto] = 1; //Colocamos el barco en el tablero 
*** tablero[ancho + 1][alto] = 1; 

    } else { //Si el barco va en vertical 

*** tablero[ancho][alto - 1] = 1; 
*** tablero[ancho][alto] = 1; //Colocamos el barco en el tablero 
*** tablero[ancho][alto + 1] = 1; 

    } 

    return tablero; //Devolvemos el tablero con el barco colocado 

} 

} 

Son sur *** s. Le problème est le même pour les 3 variables: horizontal/ancho/alto ne peut pas être résolu en variable.

Merci.

+0

Pourriez-vous essayer de reformater votre message précédent et utiliser des blocs de code autour de votre code entier? Cela rendrait la lecture plus facile. – Chrisc

+0

Si votre problème a été résolu, il est recommandé de sélectionner l'une des réponses affichées en tant que réponse en cliquant sur la coche qui se trouve à côté. – Joel

Répondre

3

Il s'agit d'un problème de portée classique.

Ces trois variables sont déclarées dans la boucle do {] while et n'existent donc pas en dehors de celle-ci.

Pour résoudre le problème, déplacez les déclarations à avant les do

boolean repetido; //Variable ..... 
boolean horizontal = false; 
int ancho = 0; 
int alto = 0; 
do { 

    repetido = false; //Suponesmos ... 
    Random aRandom = new Random(); 
    horizontal = aRandom.nextBoolean(); 
    ancho = aRandom.nextInt(tablero.length - 2); //Calculamos ... 
    alto = aRandom.nextInt(tablero[0].length); //central ... 
+0

Merci beaucoup. Je ne savais pas à ce sujet. – Javi

0

Vos variables sont déclarées à l'intérieur de la boucle do ... while, donc ils ne peuvent pas être accessibles en dehors de cette boucle.

+0

Thx vous aussi. Comme je l'ai dit, je ne le savais pas. – Javi

Questions connexes