Question

Problem set 1 - problem 1 (code). All the feedback will be highly appreciated. Thanks

http://myprogrammingstuff.wordpress.com/2011/03/13/problem-set-1-problem-1/

Answer 1

Seems like a decent solution. Couple of suggestions. First, use the builtin boolean values rather than comparing against 0 and 1. is_prime = True rather than is_prime = 1.

Also in your last conditional, you used 1000, when you actually had set up a variable for it (called limit), and you should maintain consistency so that if you wanted to change the code you'd only have to change the value of that one variable.

Answer 2

you are completely right :) I forgot about the limit variable. Thanks for your suggestions.