Kenneth, dungeon game with debug mode and few other upgrades, let me know what do you think.

Awesome,

• Thanks for testing the code, I will fix the bug.
• I knew I had cyclomatic complexity with so many ifs, I should've used elifs, but I loved the switch case scenario.

Thanks alot for reviewing it. :)

Hi there!

haha good job!
Only bug I can see is that in debug mode, if the door or monster are at x == 4, they aren't drawn, but that's a simple fix :) I like it!

Only tip might be to look at those long if/if/if/if/if blocks? For example:

    if move == "LEFT":
        x -= 1
    if move == "RIGHT":
        x += 1
    if move == "UP":
        y -= 1
    if move == "DOWN":
        y += 1
    if move == "XUPRIGHT":
        x +=1
        y -=1
    if move == "XDOWNRIGHT":
        x +=1
        y +=1
    if move == "XUPLEFT":
        x -=1
        y -=1
    if move == "XDOWNLEFT":
        x -=1
        y +=1

This works fine of course, but calling if each time is giving more work for python to do. So say we move left. Python checks if move is left, and adjusts x as we want, but then it continues to check if move is right, then if move is up, then if move is down and so on until the end of the if statements. This is where you'd use an if/elif/elif block. If you do that, python will keep checking until it finds a match, then skip the rest of the block. This is the start of optimising your code's performance :)

Consider this example:

def check(string):
    if string == "yep":
        print("This one")
        return True
    else:
        print("Not this one")
        return False

print("Example 1:")
if check("yep"):
    print("Entered if")
if check("nope"):
    print("Entered if")
if check("nope"):
    print("Entered if")

print("\n\nExample 2:")

if check("yep"):
    print("Entered if")
elif check("nope"):
    print("Entered if")
elif check("nope"):
    print("Entered if")

Also, if you're interested in a bit of an advanced technique, This is basically a what other languages call a switch statement: the parameter Move will only ever be one of some pre-defined value, and you want to do an action based on which it of those it is. Python doesn't have switch statements, but it does have dictionaries, and we can use the key:value property of dictionaries just like switch in other languages. For your example:

moves_dict = {
        "LEFT": (-1, 0),
        "RIGHT": (1, 0),
        "UP": (0, -1),
        "DOWN": (0, 1)
        }
    x += moves_dict[move][0]
    y += moves_dict[move][1]

But I think with that knowledge, you can take it a bit further and refactor some other parts too :)