Feb-26-2019, 04:39 PM
You've got an indentation problem in the code shown. The three lines at the end need to be unindented one level.
Once that is fixed, that loop should be over range(n2 - 1). A for loop over range(x) will repeat x times. If you want the n2th Fibonacci number after n1, the while loop gives you the first, so that is why you subtract one.
The for loop should do exactly what the while loop is doing. It should keep generating Fibonacci numbers that same way.
Note the way you are calculating the next Fibonacci number on lines 10-12. There's nothing wrong with it. But that changing one number and then swapping is such a common thing to do that Python has a shortcut for it called tuple assignment. That is, if you have a tuple (comma separated values) on both sides of an assignment (=), the items will be assigned in order.
Once that is fixed, that loop should be over range(n2 - 1). A for loop over range(x) will repeat x times. If you want the n2th Fibonacci number after n1, the while loop gives you the first, so that is why you subtract one.
The for loop should do exactly what the while loop is doing. It should keep generating Fibonacci numbers that same way.
Note the way you are calculating the next Fibonacci number on lines 10-12. There's nothing wrong with it. But that changing one number and then swapping is such a common thing to do that Python has a shortcut for it called tuple assignment. That is, if you have a tuple (comma separated values) on both sides of an assignment (=), the items will be assigned in order.
a, b = 1, 2 assigns 1 to a and 2 to b. So you can do those three lines in one line:G1, G2 = G2, G1 + G2
Craig "Ichabod" O'Brien - xenomind.com
I wish you happiness.
Recommended Tutorials: BBCode, functions, classes, text adventures
I wish you happiness.
Recommended Tutorials: BBCode, functions, classes, text adventures