Why does my print statement produce a "SyntaxError: invalid syntax"?

Question

How do I make the following work (in Python)?

import random
def roll():
    input1 = input("Player1, type 'ROLL' to roll.")
    if (input1 == "ROLL"):
        dice1 = random.randint(1,6)
        dice2 = random.randint(1,6)
        print("You rolled a " + dice1() + " and a " + dice2() ".")

    else:
        pass

roll()

I get:

  File "main.py", line 9
    print("You rolled a " + dice1() + " and a " + dice2() ".")
                                                            ^
SyntaxError: invalid syntax

I also want it to repeat the "Player1, type 'ROLL' to roll." if the input doesn’t equal ROLL.

What have you tried? What you wanna do...please specify in the question — manshu, Mar 20 '19 at 09:52
Why do you apply parentheses on dice1 and dice2? They have each been assigned some int. Just use str(dice1) and str(dice2), or a format string with %d. — Inon Peled, Mar 20 '19 at 09:55
Dupe [https://stackoverflow.com/questions/12018992/print-combining-strings-and-numbers](https://stackoverflow.com/questions/12018992/print-combining-strings-and-numbers) — Patrick Artner, Mar 20 '19 at 09:56
If you want to repeat something - research `for` and `while` loops. — Patrick Artner, Mar 20 '19 at 10:01
[asking-the-user-for-input-until-they-give-a-valid-response](https://stackoverflow.com/questions/23294658/asking-the-user-for-input-until-they-give-a-valid-response) — Patrick Artner, Mar 20 '19 at 10:01
@Blue, you may consider accepting an answer that helped you: https://meta.stackexchange.com/questions/5234/how-does-accepting-an-answer-work cheers — DirtyBit, Mar 21 '19 at 06:02

score 3 · Answer 1 · edited Nov 04 '19 at 06:58

You have an invalid syntax near:

print("You rolled a " + dice1() + " and a " + dice2() ".")

Using string format:

Replace this:

print("You rolled a " + dice1() + " and a " + dice2() ".")

With this:

print("You rolled a {} and {}".format(dice1,dice2))

Note: You can use a while loop to keep taking the user-input unless the input matches with ROLL:

Hence:

import random

def roll():
    while True:
        input1 = input("Player1, type 'ROLL' to roll.")
        if input1 == "ROLL":
            dice1 = random.randint(1,6)
            dice2 = random.randint(1,6)
            print("You rolled a {} and {}".format(dice1, dice2))
            break  
roll()

Output:

Player1, type 'ROLL' to roll.whaaaat?
Player1, type 'ROLL' to roll.okay
Player1, type 'ROLL' to roll.roll
Player1, type 'ROLL' to roll.ROLL
You rolled a 2 and 2

score 1 · Answer 2 · 2019-03-20T10:00:50.537

1

There's a syntax error in the print.

Both int values need to be cast to string using str().

dice1 and dice2 are not functions so we can't write () next to them.

print("You rolled a " + dice1() + " and a " + dice2() ".")

should be

print("You rolled a " + str(dice1) + " and a " + str(dice2) + ".")

Preferrably, you can use the format function that comes with any string.

print("You rolled a {dice1} and a {dice2}.\n".format(
        dice1=dice1,
        dice2=dice2
    )
)

edited Mar 20 '19 at 10:00

answered Mar 20 '19 at 09:53

2

How can you add `int` and a `str`? – Alif Jahan Mar 20 '19 at 09:54

score 0 · Answer 3 · answered Mar 20 '19 at 09:54

The whole function should be:

import random
def roll():
    input1 = input("Player1, type 'ROLL' to roll.")
    if (input1 == "ROLL"):
        dice1 = random.randint(1,6)
        dice2 = random.randint(1,6)
        print("You rolled a %s and a %s." % (dice1,dice2))

    else:
        pass

roll()

If you mean to show the two variables, dice1 and dice2.

Use:

print("You rolled a %s and a %s." % (dice1,dice2))

Instead of:

print("You rolled a " + dice1() + " and a " + dice2() ".")

score 0 · Answer 4 · answered Mar 20 '19 at 09:59

You forgot a comma or + in the print statment. You also call variables dice1 dice2 as functions.

import random

def roll():
    input1 = input("Player1, type 'ROLL' to roll. ")
    print(input1)
    if input1 == 'ROLL':
        dice1 = random.randint(1,6)
        dice2 = random.randint(1,6)
        print("You rolled a " + str(dice1) + " and a " + str(dice2) + ".")
    else:
        pass

roll()

score -2 · Answer 5 · edited Nov 04 '19 at 06:57

-2

Python's random.randint returns an integer, so you don't need to call your dice1 and dice2 variables. Replace this line:

print("You rolled a " + dice1() + " and a " + dice2() ".")

with this:

print("You rolled a " + dice1 + " and a " + dice2 ".")

Reference for Python's random module.

edited Nov 04 '19 at 06:57

Peter Mortensen

30,030
21
100
124

answered Mar 20 '19 at 09:55

Hrabal

2,158
2
19
28

Why does my print statement produce a "SyntaxError: invalid syntax"?

5 Answers5