//Below is a simple code, but it,s not working fine, guess there is a logical error but i can,t figure it out
#include <stdio.h>
#include <stdlib.h>

/* run this program using the console pauser or add your own getch, system("pause" or input loop */

int main(int argc, char *argv[]) {
int i;
int n;
int sum =0;
printf("Enter the number of students\n";
scanf("%d\n", &n);
int a[n];
for (i=0;i<n;i++){
printf("Enter the score for student %d\n", i);
scanf("%d\n", &a[i]);
sum=sum+a[i];

}

printf("The sum is %d\n", sum);
float avg;
avg=sum/n;
printf("The average is %f\n", avg);

return 0;
}
// Thanks

What is wrong with your code:

1. Check how you are using the function

scanf(...)

remove the newline

"\n"

you are using. scanf is not the same as printf, though they might be similar.

2. Why is your average though declared to be float has integer division? Integer division doesn't given you float "results".
With integer division the "decimal" part is truncated. So you might have to cast one of the operand of the integer division as float
to get what you want. Of course, you will get your answer as "float" with decimal places and ZEROs without casting. But that would
not be the right answer.

What you don't NEED.
Throw out the header

 #include <stdlib.h>

because you don't need it here and it will make your file unnecessarily large.
Why is that? I will allow you to search that out.
My point, use only what you need.

Lastly, format your code properly using spacing. It makes your code easy to read.

Of course, I could have just corrected your code and repost. But I would not have helped you and but now that you have
to make the changes yourself. When you get it I would not have robbed you of the joy of accomplishment.

Hope this helps.

Remove '\n' from your scanf. It shouldn't be there

timtoday:
What is wrong with your code:

1. Check how you are using the function

scanf(...)

remove the newline

"\n"

you are using. scanf is not the same as printf, though they might be similar.

2. Why is your average though declared to be float has integer division? Integer division doesn't given you float "results".
With integer division the "decimal" part is truncated. So you might have to cast one of the operand of the integer division as float
to get what you want. Of course, you will get your answer as "float" with decimal places and ZEROs without casting. But that would
not be the right answer.

What you don't NEED.
Throw out the header

 #include <stdlib.h>

because you don't need it here and it will make your file unnecessarily large.
Why is that? I will allow you to search that out.
My point, use only what you need.

Lastly, format your code properly using spacing. It makes your code easy to read.

Of course, I could have just corrected your code and repost. But I would not have helped you and but now that you have
to make the changes yourself. When you get it I would not have robbed you of the joy of accomplishment.

Hope this helps.

Thanks alot, the program is now ok.

remove the \n from the scanf ,it is not an output statement

op try to understand what every character you type in ur code is doing. this will enable u to know what your program is doing. it will also enable to code faster with less error.