what is wrong?here...help

heres the code primeseries but it doesnt work

#include<iostream>
#include<conio>
void primeseries()
{ for(int m=2;m<10;m++)

{

for(int i=2;i<=m;i++)
{

for(int j=1;j<i;j++)

{

if (( m % i == 0 ) && ( i * j == m ))
{cout<<m;}
}
}


}


getch();}



void main(){
primeseries();
}
help

What's the error? And next time you paste codes.. use code tags...

this code compiles:

Code:

#include<iostream.h> 
#include<conio.h> 

void primeseries() 
{ 
	for(int m=2;m<10;m++) 
	{ 
		for(int i=2;i<=m;i++) 
		{ 
			for(int j=1;j<i;j++) 
			{ 
				if (( m % i == 0 ) && ( i * j == m )) 
					cout<<m; 
			} 
		} 

	} 
} 

int main()
{ 
	primeseries(); 
	return 0;
}

Note that you should rather use int main() than void main().

If you wanted to use <iostream> then include the following line in your code:

using namespace std;

then it should work.

newbie,

First, aside from Ruski's admonition about the code tags , when using <iostream>, you must either write 'std::cout', or you must place 'using namespace std;' below your 'includes'.

You can get away with 'cout' if you write your 'include' thusly: #include <iostream.h>. (Note the .h extension.)

Second thing, you must add the .h extension to <conio> in order to use "getch();". <conio.h> is not part of the standard namespace, so you have to rely on the "old" header file designated by the .h extension.

Third, place 'm % i' and 'i * j' in parentheses inside of your IF statement. Yes, someone will tell you that it's not necessary because of the "precedence" of the math operators over the "equal to" relational operator (==), but it's better form and easier to read.

Last, getting back to Ruski's point, let us know what the error is when you post questions of this type. Your program is small enough to see the obvious problems, but in a larger program, they're not nearly as apparent.

-Skipper

And the best way is to get a book. There are free ones online