Python script crashes when I input an integer

Question

I was trying to make a simple dice in py, and I tried to make so you can change the amount of sides the dice has and if it's left empty, to default to 6. But when I input something, it crashes.

import keyboard #Import keyboard stuff like enter (pip install keyboard)
import random #Import random stuff
import time

sides = 1
sidesSelect = input("Amount of sides the dice has. If empty, 6: ")
is_non_empty= bool(sidesSelect)
if is_non_empty is False:
    sides = 6
else:
    sides = sidesSelect

time.sleep(0.5)

while True: 
    nmb = random.randint(1,sides) #Get random integer
    print("The dice rolled ", nmb) 
    input('Press enter to roll the dice again') #Ask if you want to throw again
    time.sleep(random.uniform(0.2,0.8))

I already tried changing == is, and nothing happened

if is_non_empty is false:

Thanks, it fixed my problem. bool(sideSelect) was there to check if they input something in the input, although i'll try to remove it — rOxANDtOl
– rOxANDtOl, Commented Nov 13, 2021 at 23:54
What does "it crashes" mean exactly? Please always include detailed information on the problem you are encountering — juanpa.arrivillaga
– juanpa.arrivillaga, Commented Nov 14, 2021 at 0:40

Sam Daniel · Accepted Answer · 2021-11-14 00:12:01Z

1

One issue your code has is it is trying to use the input directly without taking care of the type.

input() returns a string, so it has to be converted to proper type before using it in randint

Try something like this.


#python3

sidesSelect = int(input("Amount of sides the dice has. If empty, 6: ") or "6")

nmb = random.randint(1, sidesSelect) #Get random integer

input()

answered Nov 14, 2021 at 0:12

Sam Daniel

1,92213 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

2pichar · Accepted Answer · 2021-11-14 00:27:16Z

0

You should instead convert it to an float, with float(input(...)), and then check to make sure that it is not NaN with math.isnan(), and then if it is a valid number, convert it to an int and continue on.

import math
sidesSelect = float(input('Enter sides') or 'nan')
if math.isnan(sidesSelect):
    sides = 6
else:
    sides = int(sidesSelect)
#rest of code

Because float(invalid) will return nan, you can see if the number entered is valid or not, and then if it is, convert to an int. Otherwise, you can use your default and then continue on.

edited Nov 14, 2021 at 0:27

answered Nov 13, 2021 at 23:56

2pichar

1,3788 silver badges20 bronze badges

3 Comments

rOxANDtOl Over a year ago

I tried putting that piece of code into mine, but when I test it without any numbers, it brings me this error pastebin.com/ctT8PS83

2pichar Over a year ago

What is the error?

2pichar Over a year ago

So if you change the sidesSelect line to sidesSelect = float(input('Enter sides:') or 'nan'), then it will be correct. But if you are doing that, then you might as well go with the other answer.

Collectives™ on Stack Overflow

Python script crashes when I input an integer

2 Answers 2

Comments

3 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Related