En una parte del libro que estoy leyendo necesito reescribir un juego de "Laberinto" que usa sentencias goto
a otro que no las use.
El laberinto es muy sencillo, solo tiene 4 habitaciones y se comienza desde la habitación #1:
---------
| 1 | 2 |
---------
| 3 | 4 |
---------
Esta es la versión original del libro usando goto
:
goto room1 -- initial room
::room1:: do
local move = io.read()
if move == "south" then goto room3
elseif move == "east" then goto room2
else
print("invalid move")
goto room1 -- stay in the same room
end
end
::room2:: do
local move = io.read()
if move == "south" then goto room4
elseif move == "west" then goto room1
else
print("invalid move")
goto room2
end
end
::room3:: do
local move = io.read()
if move == "north" then goto room1
elseif move == "east" then goto room4
else
print("invalid move")
goto room3
end
end
::room4:: do
print("Congratulations, you won!")
end
Y este es mi intento de reescribirlo:
local rooms = {}
rooms[1] = {
south=3,
east=2
}
rooms[2] = {
south=4,
west=1
}
rooms[3] = {
north=1,
east=4
}
currentRoom = 1
repeat
local move = io.read()
local room = rooms[currentRoom][move]
if room == nil then
room = currentRoom
print("Invalid move")
elseif room == 4 then
print("Congratulations, you won!")
end
currentRoom = room or currentRoom
until currentRoom == 4
En este caso me pareció más adecuado usar repeat
en vez de un while
aunque en este caso específico creo que ambos funcionan igual.
¿Mi solución es la más óptima o podría cambiar algo en mi código para mejorarlo?
currentRoom = room
porqueroom
siempre tendrá un valor válido, ¿no?room = currentRoom
y dejar que la lógica booleana de la última línea haga su trabajocurrentRoom = room or currentRoom
se ejecuta más veces:room = currentRoom
sólo se ejecuta si el movimiento es incorrecto, mientras que la comparación lógica se realizaría en cada iteración (aunque no sé si realmente hay una ganancia tan grande, para un programa tan pequeño realmente no importa), por lo que sería mejor simplificar esa línea