newbie help

Just started C programming, and I'm trying to make this little code. I'm guessing you guys know what its point is since ur all expererts:

Code:

#include <stdio.h>

int main(void)
{
	int input, x, count;
	printf("Enter a positive integer: ");
	scanf("%d", &input);
	printf("The factors of %d are: \n", input);
	for (x = 1, count = 0; x <= input; x++)  	
		{if (input % x == 0)		
		printf("%d is a factor of %d\n", x, input);
		count++;
		}
	printf("# of factors is %d\n", count);	
}

The problem is when I run this the "count" vcariable always equals "input". Why is it so, I mean I did the if statement so that the count increases only if input % x == 0. A bit confusing . Thanks in advance

> I'm guessing you guys know what its point is since ur all expererts:[code]
Ohh, so close with the code tags - but no cigar today
Press edit and try again

>The problem is when I run this the "count" vcariable always equals "input".
Of course it does, the loop iterates input times, and because count starts out at 0, it will be equal to input - 1 on the last iteration and input after the loop ends. Walk through the code, it makes sense.

>I mean I did the if statement so that the count increases only if input % x == 0.
Ah, but you didn't. If an if statement body is not surrounded by braces then only the next statement will be part of the body. Properly indented, your loop looks like this:

Code:

for (x = 1, count = 0; x <= input; x++) 
{
  if (input % x == 0) 
    printf("%d is a factor of %d\n", x, input);
  count++;
}

If you only want count to be incremented if input is evenly divisible by x then you need braces:

Code:

for (x = 1, count = 0; x <= input; x++) 
{
  if (input % x == 0) {
    printf("%d is a factor of %d\n", x, input);
    count++;
  }
}

And don't forget return 0 at the end of main, that's important.

>I'm guessing you guys know what its point is since ur all expererts
What I wouldn't give to be able to understand code at a glance. Knowing what each piece does isn't the same as understanding how they work together as a whole. The former is easy, the latter takes a bit of staring even for simple programs.

Originally Posted by Prelude

>
And don't forget return 0 at the end of main, that's important..

I've read smewhere something like this:
"
Finally, the ANSI/ISO C++ Standard states that a

return 0;

statement implicitly is understood to come at the end of the main() function (but of no other function) if you don't explicitly provide it. "

Does that means that C standard require return 0; and why is that important ?

>statement implicitly is understood to come at the end of the main() function
This is only true of the C99 standard, which is not widely implemented enough to assume at this point. Therefore, we assume C89, where omitting a return value is undefined.

Thanks guys for the help.