why this piece of code is not working as it suppose to do ?!
#FizzBuzz
for i in range (1,101):
if i%3 ==0:
print 'Fizz'
if i%5 ==0:
print 'Buzz'
if i%3==0 or i%5==0:
print 'FizzBuzz'

Question

why this piece of code is not working as it suppose to do ?!
#FizzBuzz
for i in range (1,101):
    if i%3 ==0:
        print 'Fizz'
    if i%5 ==0:
        print 'Buzz'
        if i%3==0 or i%5==0:
            print 'FizzBuzz'

osanseviero · Answer

The first thing I see is the identation for the third block (when the number is divisible by 3 and 5). You want to check if it is div by 3 (print fizz), by 5 (print buzz), or by 3 and 5 (fizzbuzz), so the 3 if should be at the same lvl.  Then you use i%3==0 OR i%5==0, you should be using and, not or. You want to make sure that both are true

curry · Answer

And to add on, you should check for (3 && 5) first and then 3 individually and 5 individually.

curry · Answer

or else you'll print somtehing like fizz, buzz, fizzbuzz all on the same line.

anonymous · Answer

or you could just remove the check for both entirely, since as it stands now it will print "Fizz" followed by "Buzz" to give "FizzBuzz" for numbers divisible by 15 (although you might want to print a newline afterwards)