I have a program that decides if a given number is prime or not but it isnt working

[wudoug119]

This is my code and it will take any number that I input and say it is a prime number. Please help me...



int Prime(int prime) //declares isPrime as a function using integers
{

int primeCount = 2; //declares primeCount as an int variable and sets it equal to 2
while(primeCount < prime) //checks if primeCount is less than prime
{
if((prime % primeCount) == 0) //checks if prime divided by primeCount has a remainder
{
cout<<"This number is not prime"<<endl; //prints "The number is not prime." on the screen
return 0; //ends program
}
else //if prime divided by primecount does have a remainder
primeCount++; //adds one to primeCount
} //once primeCount is equal to prime...
cout<<"This number is prime."<<endl; //prints "The number is prime." on the screen
return 0; //ends program

}

[horace1]

when I run you code it appears to work OK, are you sure you are calling the function Prime() with the correct data? - put a statement inside Prime to print the value of the parameter to check it is correct, e.g.
cout << prime << endl;

[manontheedge]

i don't quite follow what you're trying to do, but part of the problem is that you're not checking if it's divisible by 3. Here's is some code that finds if ANY number is a prime number. It should help you out...

 
#include <iostream>
using namespace std;
 
int main() 
{
     int number(0); // sets the input number to 0
 
     while ( number != 99 )  // just picked a random condition to keep it looping
     {
          cin >> number;
 
          if ( number == 2 || number == 3 ) // 2 and 3 are always prime numbers
          {
               cout << "this number is prime" << endl;
          }
 
          else if( number % 2 == 0 || number % 3 == 0 ) // if # is divisible by 2 or 3
          {
               cout << "This number is not prime" << endl;
          }
 
          else             
          {
               cout << "This number is prime." << endl;
          }
 
     }        
 
     return 0; 
}