Skip to main content
4 de 7
Añadida una aclaración

No funciona desde un principio aunque tal y como está montado parezca que sí. Al final está la solución:

Cuando pides la butaca 1 no la encuentra porque Sala[0][0] vale 0 y utiliza el valor por defecto de Posición, que no es mayor ni igual que 1, por lo que te da el error.

El código hace esto:

i = 0
j = 0
Aux = 1
ButacaAsignada = 1
Sala[0][0] == 0
Sala[0][0] != ButacaAsignada

Como no hay ningún elemento en Sala que valga 1 el if no se ejecuta:

if (Sala[i][j] == ButacaAsignada) {
Posicion = Aux; fila = i;  columna = j;
} Aux++;

Entonces las varibles quedan así:

Posicion = 0
fila = 0
columna = 0

Y en este if, como Posicion es 0 ejecuta el else dando el mensaje de Butaca Restringida.

if (Posicion >= 1) {
Sala[fila][columna] = 99;
}
else {
System.out.println("Butaca restringida"); 
}

La segunda vez cuando pides la butaca 5 tampoco la encuentra porque Sala[0][4] también vale 0. El código hace esto cuando llega a la 5ª posición:

i = 0
j = 4
Aux = 5
ButacaAsignada = 5
Sala[0][4] == 0
Sala[0][4] != ButacaAsignada

El resto queda igual.

Cuando pides la butaca 2 la encuentra y le da a Posición el valor de 2 y a Sala[0][1] el valor 99.

i = 0
j = 1
Aux = 2
ButacaAsignada = 2
Sala[0][1] == 2
Sala[0][0] == ButacaAsignada

Entonces el if se ejecuta:

if (Sala[i][j] == ButacaAsignada) {
Posicion = Aux; fila = i;  columna = j;
} Aux++;

Entonces las variables quedan así:

Posicion = 2
fila = 0
columna = 1
Sala[0][1] = 99

Cuando pides dos entradas y vuelves a pedir la butaca 1 no la encuentra de nuevo:

i = 0
j = 0
Aux = 1
ButacaAsignada = 1
Sala[0][0] == 0
Sala[0][0] != ButacaAsignada

Igual que al principio como no hay ningún elemento en Sala que valga 1 el if no se ejecuta:

if (Sala[i][j] == ButacaAsignada) {
Posicion = Aux; fila = i;  columna = j;
} Aux++;

Pero resulta que Posición sigue valiendo 2 desde el ciclo anterior, por lo que vuelve a asignar un 99 a la segunda butaca sin dar error.

Posicion = 2
fila = 0
columna = 1
Sala[0][1] = 99

Te recomiendo que Posición la reinicies a 0 en cada bucle y que en el if compares Aux, que es la que te da la posición en lugar del valor de Sala y puedes ver en los ejemplos de arriba que su valor coincide con el que estás buscando:

do {
System.out.println("Elija número de butaca: ");
ButacaAsignada = Entrada.nextInt();
Aux = 1;

Posicion = 0;

for (int i = 0; i < 4; i++) {
for (int j = 0; j < 11; j++) {

if (Aux == ButacaAsignada) {

Posicion = Aux; fila = i;  columna = j;
} Aux++;
}
}
if (Posicion >= 1) {
Sala[fila][columna] = 99;
}
else {
System.out.println("Butaca restringida"); 
}
Contador++;
} while (Contador <= Compra);
for (int i = 0; i < 4; i++) {
System.out.println();
for (int j = 0; j < 11; j++) {
System.out.print("[" + Sala[i][j] + "]" + "  ");
}
}

Un saludo