slight code peoblem

[Ciaran789]

i am attempting to enter a max and min value for a factorial table but the application will only let me enter the first number and i cant see where i have gone wrong. any help would be apreciated

#include <stdio.h>

void main()
{
	int maxfactorial=0, minfactorial=0;

	printf("enter max factorial\n");
	scanf("%d" maxfactorial);
	getchar();
	printf("enter min factorial\n");
	scanf("%d" minfactorial);	
	getchar();

}

[JOZZY& Wakko]

I am just a beginner and might not have the "right" answer to your question but a few other points to mention.

1. The following line(s) need to be changed.

scanf("%d" maxfactorial);

To.

scanf("%d", &maxfactorial);

scanf("%d" minfactorial);

To.

scanf("%d", &minfactorial);

2. Try to include "return 0;" at the end of every function (it is advised in most books on C).

3. You are assigning a value to your variables in the beginning while you later are asking for user input. I do not see the point in this.

4. Not sure on this one but if I am right you should change.

void main()

To.

int main(void)

Hope this helped you it is all the advise I can give but I am sure there are more people on here willing to help.

[quzah]

2. Try to include "return 0;" at the end of every function (it is advised in most books on C).

Not necessarily every function, but you should on every function that is expected to return an integer--not specifically zero, but something meaningful to you. We use return 0; at the end of main because the OS expects to receive a return value (typically 0 for success, or something else like EXIT_FAILURE) when your program exits.

Functions generally only need to return something if you need them to. If you don't, you generally make them void functions:

void printx( int x )
{
    printf( "x is %d\n", x );
}

Here our function doesn't really need to return anything, because when we wrote it, we decided we didn't care if it did anything more than just print x. But yes, if your function is expected to return something, then you need to make sure it does so:

int thisiswrong( int x )
{
    if( x == 5 )
        printf( "hello world!\n" );

    /* not returning something here should generate compiler warnings/errors */
}

Here we're expected to return an int, not doing so is an error.

3. You are assigning a value to your variables in the beginning while you later are asking for user input. I do not see the point in this.

It's generally advised that you initialize your variables to something before you use them. This helps you avoid pulling your hair out because for some reason you're getting an odd number where you least expect it, scouring your code to figure out why...

int x;
... way later ...
printf( "x * 5 is: %d\n", x * 5 );

Generally if you compile with warnings on you'll get something like 'possible use of uninitialized variable, first used at <line number>', or something like that.


Quzah.

[Ciaran789]

thanks for the help. i have uploaded the assignment i have been given. i dont want anyone to do it for me. but any suggestion of where to go next would be a great help as i am completely lost. thanks again,
Ciaran.

[Epy]

you're going to need a for loop to go from your min to max, a function to calculate the factorial (you can do this recursively or with a for loop), a for loop within the min to max for loop to print out the expressions like 5*4*3*2*1

1. print your header
2. loop from max to min (or min to max)
3. foreach line: print n!, a couple tabs, for loop to do 5*4*3*2*1 etc, couple tabs, calculated factorial
4.end

[Ciaran789]

thank you. that is very helpful.

[Ciaran789]

#include <stdio.h>

void main()
{
	int maxfactorial=0, minfactorial=0;

	printf("enter max factorial\n");
	scanf("%d", &maxfactorial);
	printf("enter min factorial\n");
	scanf("%d", &minfactorial);	

	if (maxfactorial>10){
	printf("enter a max factorial of ten or less");
	}

}

if i want that to restart the application if the if statement prints that line, how would i set that up?

[quzah]

You would start by learning about loops: for, while, and do-while.


Quzah.

[JOZZY& Wakko]

Not necessarily every function, but you should on every function that is expected to return an integer--not specifically zero, but something meaningful to you. We use return 0; at the end of main because the OS expects to receive a return value (typically 0 for success, or something else like EXIT_FAILURE) when your program exits.

Functions generally only need to return something if you need them to. If you don't, you generally make them void functions:

void printx( int x )
{
    printf( "x is %d\n", x );
}

Here our function doesn't really need to return anything, because when we wrote it, we decided we didn't care if it did anything more than just print x. But yes, if your function is expected to return something, then you need to make sure it does so:

int thisiswrong( int x )
{
    if( x == 5 )
        printf( "hello world!\n" );

    /* not returning something here should generate compiler warnings/errors */
}

Here we're expected to return an int, not doing so is an error.

Thanks for the explanation. But does that mean that in this case I was right because the program returns 2 values?

It's generally advised that you initialize your variables to something before you use them. This helps you avoid pulling your hair out because for some reason you're getting an odd number where you least expect it, scouring your code to figure out why...

int x;
... way later ...
printf( "x * 5 is: %d\n", x * 5 );

Generally if you compile with warnings on you'll get something like 'possible use of uninitialized variable, first used at <line number>', or something like that.

Ok, but would this be an issue when you are expecting input from an user to set a value for the integer?