PSET6 Mario Python

Question

I have been experiementing with this code for a couple hours and have come across an issue with each implementation. I can no longer put in any time at the moment so I figured I would go ahead and ask. As of now it keeps telling me that "else" is invalid syntax and when I try to implement it without the else my returns have issues and are outside the function. Here is my code:

from cs50 import get_int

condition = True
# originally n < 1 or n >8
while condition:
    n = get_int("Height:") in range(8)
    if n != range:
        print("Pick a number between 1 and 8 please!")
    return 
    else:
        return False
for i in range(n):
    for j in range(n):
        if i + j < n - 1:
            print(" ")
        else:
            print("#")

Updated code:

from cs50 import get_int

condition = True
# originally n < 1 or n >8
while condition:
    n = get_int("Height:") in range(8)
    if n > 0 and n < 8:
        break
for i in range(n):
    for j in range(n):
        if i + j < n - 1:
            print(" ")
        print("#")

Answer 1

Just like in C there were couple of bugs you have noted when you try to pair if and else and certain syntax error

I think it makes the computer to extra think as it should not because obviously computer is smart enough to know and understand when if statement is false don't execute it rather move to next line which is actually an else statement but you don't need to tell the computer that

hey, computer this is else statement when you are not happy with if statement

So you shouldn't do that, remove else from the code and try some different approach

Note: I am not gonna tell you the key because it is a practice based PSET for python which makes your python basics strong.

Answer 2

I think its because after your if argument, your return's indentation matches if's indentation. if and else should be consecutive and should not have anything other than argument between them. Try changing indentation of your return after if.

Answer 3

i have a feeling that range(8) is messing things up. when you say range(8) it means from 0 to 7.. Try removing range while getting the input from user and only check it once like you are doing with if n < 0 or n > 8 statement.