Problem with Function

[bitong]

please help me...I can't get the correct output...My professor suck's because she's didn't discuss thoroughly the in's and out's of C programming..pl ease help...Here's the problem: I have to write a program using a function that when you input a number, it will add the number from 1 to the number that was input, also you will show how the numbers where squared.

Desired output:
Enter a number: 5
1+4+9+16+25
The sum is: 55

My ouput:
Enter a number: 5
1+4+9+16+25
The sum is: 5869

Thank you very much in advance..here's my code, there's definitely wrong with this

#include <stdio.h>
int funct(int x);
int functi(int y);
int main()
{
   int c, d, num;
   clrscr();
   printf("\nEnter a number: ");
   scanf("%d",&num);
   for(c=1;c<=num;c++){
      printf("%d+",funct(c));
   }
   printf("\nThe sum is: %d",functi(d));
   getch();
}

int funct(int x)
{
   int a;
   a=x*x;
   return a;
}
int functi(int y)
{
   int b;
   b=y+b;
   return b;
}

[sicarie]

please help me...I can't get the correct output...My professor suck's because she's didn't discuss thoroughly the in's and out's of C programming..pl ease help...Here's the problem: I have to write a program using a function that when you input a number, it will add the number from 1 to the number that was input, also you will show how the numbers where squared.

Desired output:
Enter a number: 5
1+4+9+16+25
The sum is: 55

My ouput:
Enter a number: 5
1+4+9+16+25
The sum is: 5869

Thank you very much in advance..here's my code, there's definitely wrong with this

#include <stdio.h>
int funct(int x);
int functi(int y);
int main()
{
int c, d, num;
clrscr();
printf("\nEnter a number: ");
scanf("%d",&num );
for(c=1;c<=num; c++){
printf("%d+",fu nct(c));
}
printf("\nThe sum is: %d",functi(d) );
getch();
}
int funct(int x)
{
int a;
a=x*x;
return a;
}
int functi(int y)
{
int b;
b=y+b;
return b;
}

You never initialize d. When you declare it, the compiler gives it memory space - and d becomes x04a3d (some arbitrary slot of memory). However, it does not initialize this with a value. Some program you were using before used that memory spot x04a3d, and there is a random "garbage" value in it that d now has. That is the value you are passing to functi() (by the way, it is considered bad practice to name two functions and only have them differ by one character... it will also get very confusing for you when you get into longer programs - I would recommend mult(int x) and addr(int y) for them... somewhat indicative of what they do... But you need to set values for 'd', the numbers you want to add.

[sicarie]

And since you got it to compile, I'm guessing that's not your code copied directly from your program - the compiler would have yelled about a few things in there - you're using getch() (the function that annoys me the most out of any language and should be banned forever), but don't declare conio, and you declare main as int, but don't return anything at the end. Just a heads up if you try to copy that code...

[DeMan]

Perhaps, in the OP's opinion, a compiler error is not the right output? ;)

[bitong]

You never initialize d. When you declare it, the compiler gives it memory space - and d becomes x04a3d (some arbitrary slot of memory). However, it does not initialize this with a value. Some program you were using before used that memory spot x04a3d, and there is a random "garbage" value in it that d now has. That is the value you are passing to functi() (by the way, it is considered bad practice to name two functions and only have them differ by one character... it will also get very confusing for you when you get into longer programs - I would recommend mult(int x) and addr(int y) for them... somewhat indicative of what they do... But you need to set values for 'd', the numbers you want to add.

so, i have to add another for loop to set a value for d? or am i going to edit function with regard to 'd'? am i going to delete the last function?

[bitong]

And since you got it to compile, I'm guessing that's not your code copied directly from your program - the compiler would have yelled about a few things in there - you're using getch() (the function that annoys me the most out of any language and should be banned forever), but don't declare conio, and you declare main as int, but don't return anything at the end. Just a heads up if you try to copy that code...

what do you mean by that? Will i disregard the getch() function? what function am i going to use in lieu of the getch()? is it obsolete or a bad practice to use getch?

[DeMan]

The point that was being made is that getch() comes from a library that you haven't included......( abnd some people have strong feelings about that library, as you will soon learn....)

[bitong]

The point that was being made is that getch() comes from a library that you haven't included......( abnd some people have strong feelings about that library, as you will soon learn....)

but if you don't include getch() there is no display in the screen?

[Ganon11]

There are other, library independent methods you can use to keep the output on the screen. Not everyone has conio.h, but getch() is only defined in conio - a much more universal solution would be to use a scanf call.

You shouldn't need two functions to do this - you should have a variable that will keep the sum of the numbers as you call the function in the loop.

[bitong]

There are other, library independent methods you can use to keep the output on the screen. Not everyone has conio.h, but getch() is only defined in conio - a much more universal solution would be to use a scanf call.

You shouldn't need two functions to do this - you should have a variable that will keep the sum of the numbers as you call the function in the loop.

int funct(int x)
{
int a;
a=x*x;
return a;
}
int functi(int y)
{
int b;
b=y+b;
return b;
}

so i'll write the code like this...
int funct(int x)
{
int a, b=0;
a=x*x;
b=b+a;
return b;
}

is that what you mean? sorry for the question...i just want to understand the program...thanx =)

[sicarie]

int funct(int x)
{
int a;
a=x*x;
return a;
}
int functi(int y)
{
int b;
b=y+b;
return b;
}

so i'll write the code like this...
int funct(int x)
{
int a, b=0;
a=x*x;
b=b+a;
return b;
}

is that what you mean? sorry for the question...i just want to understand the program...thanx =)

DeMan - he said he was getting a value printed out, so I was kind of assuming he was able to compile and run. Then I added that second part just in case.

I personally consider getch() bad practice because of what Ganon11 said - many people like using it, I was just going on my usual tirade against it (and I can be ignored if you feel a certain fondness for it - it's not "wrong", but you will have to do more checking if you write larger distributed programs with it).

In your updated code, I believe you have your logic error fixed - but did you need to print out the squared values to add together - if so, you will need one more printf call - but that function looks right (at first glance).

[sicarie]

The point that was being made is that getch() comes from a library that you haven't included......( abnd some people have strong feelings about that library, as you will soon learn....)

By some people, he means me. As this is just another thread I am hijacking to speak out against non-standard libraries!

::ties my tie around my head, twirls mouse over head by its cord::

[Banfa]

int functi(int y)
{
   int b;
   b=y+b;
   return b;
}

This function will not work as intended because b is on the stack. Since it is on the stack it is not implicitly initialised to anything and you do not explicitly initialise it to anything so it will have a random value (from whatever last used that piece of memory).

Since b starts with a random value and all you do is increment it by y it will end with a random value.

[bitong]

int functi(int y)
{
   int b;
   b=y+b;
   return b;
}

This function will not work as intended because b is on the stack. Since it is on the stack it is not implicitly initialised to anything and you do not explicitly initialise it to anything so it will have a random value (from whatever last used that piece of memory).

Since b starts with a random value and all you do is increment it by y it will end with a random value.

hello banfa,
what do you mean by stack?
But on the other hand what's the difference between this two code?

int sum=0, x;
for(x=1;x<=5;x+ +){
sum+=x;
}
printf("%d",sum );

compare to this-->

int functi(int y)
{
int b;
b=y+b;
return b;
}

or did I forgot to initialize b to zero(0)?

[dwayne]

help me, i have a program that should prompt a user for 3 sides of a triangle and i should use functions to determine if the values form a right-angled triangle, isosceles, scalene, or an equilateral triangle....im having some difficulties as in what i should do in the functions.