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 porque la única validación que haces es si el valor de Sala es mayor o igual a 1.
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]>=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