Return to Question

Commonmark migration

edited Jun 10, 2020 at 13:24

I'm a fairly new programmer (just started yesterday!). I decided to tackle Project Euler #2 today, as I did #1 yesterday without many problems. I came up with what seems to me to be a working solution, but I feel like I did it in an exceedingly ugly way. Could anyone suggest improvements to my code and/or logic?

Each new term in the Fibonacci sequence is generated by adding the previous two terms. By starting with 1 and 2, the first 10 terms will be:

1, 2, 3, 5, 8, 13, 21, 34, 55, 89, ...

By considering the terms in the Fibonacci sequence whose values do not exceed four million, find the sum of the even-valued terms.

fib = 1
fib2 = 2
temp = 0
total = 0
while temp <=4000000:
 temp = fib2
 if temp % 2 == 0:
 total += temp
 temp = fib + fib2
 fib = fib2
 fib2 = temp
print total

I just sort of used my temp variable as a holding ground for the results, and then juggled some variables around to make it do what I wanted... Seems like quite a mess. But it does result in 4613732, which seems to be correct!

Each new term in the Fibonacci sequence is generated by adding the previous two terms. By starting with 1 and 2, the first 10 terms will be:

1, 2, 3, 5, 8, 13, 21, 34, 55, 89, ...

By considering the terms in the Fibonacci sequence whose values do not exceed four million, find the sum of the even-valued terms.

fib = 1
fib2 = 2
temp = 0
total = 0
while temp <=4000000:
 temp = fib2
 if temp % 2 == 0:
 total += temp
 temp = fib + fib2
 fib = fib2
 fib2 = temp
print total

Each new term in the Fibonacci sequence is generated by adding the previous two terms. By starting with 1 and 2, the first 10 terms will be:

1, 2, 3, 5, 8, 13, 21, 34, 55, 89, ...

By considering the terms in the Fibonacci sequence whose values do not exceed four million, find the sum of the even-valued terms.

fib = 1
fib2 = 2
temp = 0
total = 0
while temp <=4000000:
 temp = fib2
 if temp % 2 == 0:
 total += temp
 temp = fib + fib2
 fib = fib2
 fib2 = temp
print total

Question Protected by 200_success

occurred Jul 11, 2015 at 6:30

edited tags

Link

edited Dec 22, 2013 at 23:09

Jamal

edited Dec 22, 2013 at 23:09

Jamal

35.2k
13
134
238

Tweeted twitter.com/#!/StackCodeReview/status/178037354391547904

occurred Mar 9, 2012 at 8:39

Source Link

asked Mar 8, 2012 at 19:40

bckwth

asked Mar 8, 2012 at 19:40

bckwth

Python Solution for Project Euler #2 (Fibonacci Sums)

Each new term in the Fibonacci sequence is generated by adding the previous two terms. By starting with 1 and 2, the first 10 terms will be:

1, 2, 3, 5, 8, 13, 21, 34, 55, 89, ...

By considering the terms in the Fibonacci sequence whose values do not exceed four million, find the sum of the even-valued terms.

fib = 1
fib2 = 2
temp = 0
total = 0
while temp <=4000000:
 temp = fib2
 if temp % 2 == 0:
 total += temp
 temp = fib + fib2
 fib = fib2
 fib2 = temp
print total

python project-euler

lang-py