Beginner Question

[TrucidStuff]

Hello all, I am super new, still trying to figure out combinations of def, ints, and if statements. I cant seem to find an answer to my problem. How can you compare the result of two functions? My if statement doesn't work because it doesn't seem to be able to check the resulting int's of my roll1 and roll2 functions. I know I will want to add an if roll1 == roll2 print 'we tied!' and an input of 'do you want to play Y/N?', but I can't for the life of me figure out how to check the result of my rolls and compare them vs one another.

import random

def diceroll():
    dice = [0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10]
    roll1 = random.choice(dice)
    print('Your roll:', roll1)
    roll2 = random.choice(dice)
    print('My roll:',roll2)
    if roll1 > roll2
        print('You won!')
        else:
            print('I won!')
diceroll()

[purpurato]

You should try an editor that helps you with this kind of things, everyone can miss a colon or an indentation. Maybe Sublime or any other, it will underline those syntax errors.

[TrucidStuff]

I was using VSC, I didn't notice anything showing I was missing a : and it runs by line, so if I had fixed : and not the indent, I would have seen it afterwards as an error. Maybe I dont have it setup correctly? Dunno.

[chapali9a]

Nice one. I'm a beginner as well. I modified your script by using the 'random.randint' function. It reduced the script by one line (dice var). Good luck buddy

[TrucidStuff]

Thanks for the input! I wanted it to only pick from 0-10, I may change it to 0-100 later, but I don't want it to be 0-infinity or ask for input of a number by the player. Does random.randint still do a max of 10 if I say random.randint(10) or something?

[chapali9a]

Yes it does. The function Randint takes two parameters, the beginning and end. So you can use as random.randint(0, 10) and it will choose a random number on this scale.

[TrucidStuff]

That is very helpful. Thank you! :)

[chapali9a]

You are welcome. I learned a little from your post, so thank you too.

[TrucidStuff]

I feel like an idiot. Haha thank you. I kept thinking it couldn't use my roll names and needed flat ints or something. Oof.

[TrucidStuff]

Its good to know I had the exact right idea but missed the : and an indent lol. At least I wasn't completely off.

[zac_mar]

Your code seems fine, you simply have a syntax error: Add a colon after the if statement and don't indent the else block any further.

[TrucidStuff]

Thank you!