What do I need to fix?

[R3myD]

With the inputs 5, 6, 14, and 4.5, the result should be 374 but i keep getting 4200774. What do i need to fix

#include <stdio.h>

#define PI 3.141592654

int main()
{
int layers, radius_top, radius_bottom, feed, y;
float cake, x, lay_area;

printf("How many layers will your cake have? \n");
scanf("%d", &layers);

printf("What is the radius of the top layer?\n");
scanf("%d", &radius_top);

printf("What is the radius of the bottom layer?\n");
scanf("%d", &radius_bottom);

printf("How much cake in cross-sectional area does each guest need?\n");
scanf("%f", &cake);

x = (radius_bottom - radius_top) / (layers - 1);

for (y = layers; y > 0; y--)
{ lay_area = (radius_top * radius_top) * PI;
feed += lay_area / cake;
radius_top += x; }

printf("Your cake will feed %d guests.", feed);

return 0;

}

[quzah]

If you have one layer, you're going to divide by zero. That's bad. You also never initialize feed.


Quzah.

[R3myD]

Thanks. I didn't realize I had to initialize feed. That was the only problem

[quzah]

That was the only problem

Right, because dividing by zero isn't a problem in math...


Quzah.

[R3myD]

It is, but the program is giving the correct answers when all i did was initialize feed

[CommonTater]

WOW... massively bad form... You should NEVER edit the first message in a thread...

[quzah]

WOW... massively bad form... You should NEVER edit the first message in a thread...

Don't worry, it looked like this:

#include <stdio.h>

#define SMALL 50
#define LARGE 200

int main(void)
{
float small_cost, large_cost, invite_cost = 0;
int invites, largepack, smallpack;

printf("What is the cost of a small package (in dollars)?\n");
scanf("%f", &small_cost);

printf("What is the cost of a large package (in dollars)?\n");
scanf("%f",&large_cost);

printf("How many invitations are you sending out?\n");
scanf("%d", &invites);

while (invites >= 150)
{ invites -= 200;
largepack++; }
while (invites > 0)
{ invites -= 50;
smallpack++; }

printf("You should order %d small package(s).", smallpack);
printf("You should order %d large package(s).", largepack);

invite_cost = (smallpack * small_cost) + (largepack * large_cost);

printf("our cost for invitations will be $%f.", invite_cost);

return 0;

}

Quzah.

[R3myD]

Oh, I didn't know. I just didn't want it to be copied

[quzah]

Oh, I didn't know. I just didn't want it to be copied

Well in its current state, if it's copied, they'll get a bad grade.


Quzah.

[CommonTater]

Quzah beat me to it...

[CommonTater]

Oh, I didn't know. I just didn't want it to be copied

Thing is, WE want it left there so that the ensuing discussion at least appears to make sense....

[R3myD]

That's fine. No problem

[Salem]

I put the original back for context.

> Oh, I didn't know. I just didn't want it to be copied
Your defence is "you published first", so if there is ever any argument, you can always say "look here".