Thread: Fibonacci Questoin

Hey guys first post here. I just switched my major to computer science last semester. So first my c++ class and this would be my second assignment. I need to write a function(in the from of a program for testing purposes) that prints the all the Fibonacci numbers less than 30,000. So my below code does all this except it prints one number after 30,000. Can anyone give me some idea on how to fix it? Or maybe a better a better way to implement it with a for loop? Thanks a ton in advance.

Code:

#include<iostream>

using namespace std;

int main()
{
	//Declaration
	unsigned int fib1=0, fib2=1, fib3=0;
	
        //fibonacci loop
	while(fib3<30000)
	{
		fib3=fib1+fib2;
		cout<<fib3<<endl;
		fib1=fib2;
		fib2=fib3;
	}
	return(0);
}

Originally Posted by drumerboy3841

Code:

#include<iostream>

using namespace std;

int main()
{
	//Declaration
	unsigned int fib1=0, fib2=1, fib3=0;
	
        //fibonacci loop
	while(fib<30000)
	{
		fib3=fib1+fib2;
		cout<<fib3<<endl;
		fib1=fib2;
		fib2=fib3;
	}
	return(0);
}

My guess is that it has something to do with your while loop. However I can't tell since fib doesn't exist. Assuming you intended fib3 note when during your loop fib3 exceeds 30,000. It is after you print it. Just move your summation to the last line in the loop.

It's not reassigned at the end of the loop. That line is the only one that assigns to fib3.

@anos's right. Why would you think that?