Print n first primes

Hey there guys

Could you please tell me whats wrong with this code? Ive been looking for the error for like 1 hour and still I cant figure out whats missing.
It just prints the number 2 and it doesnt make any sense

Thanks in advance

Cheers

  #include <iostream>
#include <cmath>

using namespace std;

int isprime(int n)
{
	int i,fact=0;
	for(i=1;i<=n;i++)
	{
		if(n%i==0);
		fact++;
	}
	if(fact==2)
		return 1;
	else
		return 0;
} 



void main()
{
	int i = 1;
	int count=0;
	while(count<100)
	{
		
		
		if(isprime(i)==1)
		{
			cout<< i << endl;
			count++;
		}
		i++;
	}

	//system("pause");
}

MiiNiPaa (8886)

1
2

if(n%i==0);
fact++;

Notice ; after if. Here what is means:

1
2
3

if(n%i==0)
    ; //if n is divisible by i then do nothing
fact++; //Always increase fact

Both MSVC and GCC wil warn you about it. Do you have warnings on? If not, turn them on. Either -Wall - Wextra on GCC or /W3 on VS

Also: void main() is invalid C++. main should return int

Last edited on

Topic archived. No new replies allowed.

C++

Forum

Print n first primes