python variables not accepting names

Question

I'm trying to declare a few simple variables as part of a function in a very basic collision detection programme. For some reason it's rejecting my variables (although only some of them even though they're near identical). Here's the code for the function;

def TimeCheck():
    timechecknumber = int(time.time())
    timecheckdiv = backcolourcheck % 5
    if timecheckdiv < 1:
        timecheck = true
    else:
        timecheck = false
    if timecheck == true:
        backgroundr = (int(random.random()*255)+1
        backgroundg = (int(random.random()*255)+1
        backgroundb = (int(random.random()*255)+1

for some reason it accepts backgroundr but not backgroundg, anyone got any ideas why? thanks

What does "accepts" mean? Do you have a specific error message or other problem? Or are you having trouble balancing your ()'s? — S.Lott
– S.Lott, Commented Feb 26, 2009 at 16:55
I meant that it was just highlighting backgroundg in red and saying syntax error. it was just a parenthesis issue though :) — user33061
– user33061, Commented Feb 26, 2009 at 16:59

mipadi · Accepted Answer · 2009-02-26 17:01:29Z

8

You have mismatched parentheses on the line beginning with backgroundr. I think maybe you want this:

backgroundr = int(random.random() * 255) + 1

Note that each of the next two lines also have mismatched parentheses, so you'll have to fix those, too.

edited Feb 26, 2009 at 17:01

answered Feb 26, 2009 at 16:55

mipadi

414k91 gold badges538 silver badges488 bronze badges

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

2 Comments

recursive Over a year ago

You need to multiply by 255 before casting to int.

mipadi Over a year ago

As noted below, if you want the code to do something useful (which you probably do), you most likely actually want the parenthesis in the order below (I've edited the answer to reflect this).

recursive · Accepted Answer · 2009-02-26 17:00:12Z

2

mipadi's answer will always yield a 1. You need to multiply by 255 before you cast to int. Try this.

backgroundr = int(random.random() * 255) + 1

answered Feb 26, 2009 at 17:00

recursive

86.5k36 gold badges155 silver badges246 bronze badges

1 Comment

S.Lott Over a year ago

You're assuming -- of course -- that this is what they want. Since the ()'s were incorrect, perhaps they really did want an answer of 1 each time ;-)

Collectives™ on Stack Overflow

python variables not accepting names

2 Answers 2

2 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related