4

I've been stuck on this issue for over two hours, this is homework so please don't give me a direct answer just point me in the right direction.

So... This program is designed to input a "speed limit", input "Current speed" and give you a print response of either "Speed OK" (if you are below or equal to the speed limit) and "Slow down!" (if you are speeding). When I input the following data, as required on the task: Speed Limit of 50. Current Speed of 50, 45, 55, 52, and 50.

The answer should read -

Speed limit: 50
Current speed: 50
Speed OK
Current speed: 45
Speed OK
Current speed: 55
Slow down!
Current speed: 52
Slow down!
Current speed: 50
Speed OK
Current speed:(White space)

Instead I get -

Current speed: 50
Speed OK
Current speed: 45
*Then the program stops.*

My Program Reads -

limit = int(input("Speed limit: "))
speed = int(input("Current speed: "))
if speed <= limit:
  print("Speed OK")
  speed = int(input("Current speed: "))
false = speed > limit
while false:
  print("Slow down!")
  speed = int(input("Current speed: "))

If somebody could point me in the right direction that would be wonderful ;).

Cheers,

Ron

Improve this question
2
  • 5
    while false: makes me cringe, even though it's technically correct. (didn't downvote, though) Commented Aug 26, 2013 at 10:03
  • 1
    try to debug,you will find where goes wrong Commented Aug 26, 2013 at 10:20

4 Answers 4

Reset to default
4

First of all, keep in mind that the one of the most important rules of programming is to avoid code doubling. This means that it nearly never a good idea to have the same speed = int(input("Current speed: ")) line three times in your program. If there's an error in that line, fixing it will probably leave the error in the two other places untouched.

If you follow that rule, you will probably find out that you only need one loop which should terminate on (White space) (so this check should be in the condition, not false). And inside that loop you should check with if which response your program should give.

Finally, I strongly disapprove of having a variable named false. In case speed is larger than limit, your variable false will hold the value True (which is quite peculiar and surprising). Any other programmer would probably misunderstand this.

Never forget that writing a program also is communicating with the next programmer who has to maintain your code.

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

1 Comment

Thanks for your help, got the answer right an learnt a bit more than i would in a normal lesson. Cheers!
2

Trying to give just a pointer -

Your loop is in the wrong place. Think carefully about what code you want to keep repeating, and what code should execute once for each loop.

Try to see what happens if your first input is OVER the speed limit :)

Improve this answer

Comments

0

Your design is in the wrong way you have to do an infinite loop

ask speed limit 
run the forever loop (based on a true condition) :
-> ask the speed
-> print message

i hope that's will help you

Improve this answer

Comments

0

You could set a counter for how often the loop returns, either with a fixed limit or with a user input limit.

The int(input("...") would sit inside the loop but outside the True/ False check.

Since there are only 2 possible answers ("Speed OK" and "Slow down") look into how to verify for this boolean.

Spoiler.

Improve this answer

Comments

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.