stuck on purse program.

hey guys-
yes, this is a homework assignment. i'm stuck and if anyone can point me in the right direction it would be much appreciated. i need a program to remove and insert coins into a purse. thanks.

Code:

---

#include <stdio.h>
#include <math.h>
int main(void)
  { 
  int pennies=0, nickels=0, dimes=0, quarters=0, total=0, p,n,d,q,x;
  int choice;


printf("Would You Like to Insert or Remove Coins?\n");
printf("Press 1 for Insert.\n");
printf("Press 2 for Remove.\n");
scanf("%d", &choice);


if (choice == 1)
{
printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, total);
scanf("%d%d%d%d", &p, &n, &d, &q);
pennies += p;
nickels += n;
dimes += d;
quarters += q;
float x = pennies + 5*nickels + 10*dimes + 25*quarters;
x =  (float)x/100;
printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, total);
}
else (choice == 2)
{
printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, total);
scanf("%d%d%d%d", &p, &n, &d, &q);
pennies -= p;
nickels -= n;
dimes -= d;
quarters -= q;
float x = pennies + 5*nickels + 10*dimes + 25*quarters;
x =  (float)x/100;
printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, total);
}


return 0;


}

---

My first suggestion would be to find an indentation style you like and use it consistently. This will make following your program logic much easier.

The next suggestion would be to ask specific questions based on the information you provide. Do you have some kind of question or problem with the code you provided?



Jim

First thing, I am not getting a correct total. It will show the correct number of coins, but the total reads $0.

so what is the real problem with your code at least give some basic errors so i can help...

I think your code does not compile.


Anyway, it is wrong

Code:

---

if (choice == 1)
{
    /* stuff */
}
else (choice == 2)
{
    /* stuff */
}

---

You should write that as

Code:

---

if (choice == 1)
{
    /* stuff */
}
else /* choice == 2 (unless the user didn't follow instructions) */
{
    /* stuff */
}

---

That is probably being caused because you have defined x as an int, and an int has no fractions. So this line is probably the problem.

Code:

---

x =  (float)x/100;

---

What did you input into the program? Are you inserting into the purse or extracting? How many of each coin did you enter?



Jim

Quote:

---

Originally Posted by jimblumberg

So this line is probably the problem.

Code:

---

x =  (float)x/100;

---

---

Yes, but I do not know how to adjust this.

Quote:

---

What did you input into the program? Are you inserting into the purse or extracting? How many of each coin did you enter?Jim

---

I tried inserting. I entered

Code:

---

1+2+3+4

---

It displayed

Code:

---

1 pennies+2 nickels+3 dimes+4 quarters=$0

---

K. I moved float to the top, indented, and changed to just else.

Code:

---

#include <stdio.h>
#include <math.h>
int main(void)
  { 
  int pennies=0, nickels=0, dimes=0, quarters=0, p,n,d,q;
  int choice=0;


float(x) = 25*quarters + 10*dimes + 5*nickels + pennies;
x =  x/100;


        printf("Would You Like to Insert or Remove Coins?\n");
        printf("Press 1 for Insert.\n");
        printf("Press 2 for Remove.\n");
        scanf("%d", &choice);


if (choice == 1)
{
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, x);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies += p;
                nickels += n;
                dimes += d;
                quarters += q;
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, x);
}
else
{
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, x);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies -= p;
                nickels -= n;
                dimes -= d;
                quarters -= q;
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%d \n", pennies, nickels, dimes, quarters, x);
}


return 0;


}

---

Quote:

---

Originally Posted by rosemary

First thing, I am not getting a correct total. It will show the correct number of coins, but the total reads $0.

---

That's because you never change the value of "total" (it is initialized with 0).

Bye, Andreas

ugh. the indentation didnt copy properly :(

Quote:

---

K. I moved float to the top, indented, and changed to just else.

---

What good does moving the calculation to the beginning, before you enter any values, do you?

Does the following even compile?

Code:

---

float(x) = 25*quarters + 10*dimes + 5*nickels + pennies;
x =  x/100;

---

Where is x defined? What type of variable is x?

Jim

[QUOTE=jimblumberg;1125437

Where is x defined? What type of variable is x?

Jim[/QUOTE]

Code:

---

#include <stdio.h>
#include <math.h>
int main(void)
  { 
  int pennies=0, nickels=0, dimes=0, quarters=0, total=0, p,n,d,q,t;
  int choice=0;
       
        printf("Would You Like to Insert or Remove Coins?\n");
        printf("Press 1 for Insert.\n");
        printf("Press 2 for Remove.\n");
        scanf("%d", &choice);


if (choice == 1)
{
        printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2d total\n", pennies, nickels, dimes, quarters);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies += p;
                nickels += n;
                dimes += d;
                quarters += q;
                total=total+(25*quarters + 10*dimes + 5*nickels + pennies/100);
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2d total\n", pennies, nickels, dimes, quarters, total);
}
else
{
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2d total\n", pennies, nickels, dimes, quarters, total);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies -= p;
                nickels -= n;
                dimes -= d;
                quarters -= q;
                total=total-(25*quarters + 10*dimes + 5*nickels + pennies/100);
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2d total\n", pennies, nickels, dimes, quarters, total);
}


return 0;


}

---

So does your modified code solve your problems? If not maybe you need to ask another question.

Jim

one more point, keep it in a loop. Your program would exit just after single iteration.

Quote:

---

Originally Posted by varshneyabhi

one more point, keep it in a loop. Your program would exit just after single iteration.

---

This is a homework exercise, so a single iteration is sufficient - unless the assignment calls for the program to loop, which does not seem to be the case.

I would like to make it a loop eventually, but first I was just trying to get the "float" part figured out.

It does not loop. It does not add/subtract correctly. I am stuck.

Code:

---

  float total=0,t;
  int choice=0;       
        printf("Would You Like to Insert or Remove Coins?\n");
        printf("Press 1 for Insert.\n");
        printf("Press 2 for Remove.\n");
        scanf("%d", &choice);


while(!choice)
{
}
if(choice == 1)
{
        printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                    pennies, nickels, dimes, quarters);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies += p;
                nickels += n;
                dimes += d;
                quarters += q;
                total=total+(25*quarters + 10*dimes + 5*nickels + pennies/100);
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                    pennies, nickels, dimes, quarters, total);
}
else
{
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                      pennies, nickels, dimes, quarters, total);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies -= p;
                nickels -= n;
                dimes -= d;
                quarters -= q;
                total=total-(25*quarters + 10*dimes + 5*nickels + pennies/100);
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f\n",
                pennies, nickels, dimes, quarters, total);
}
}

---

Let's just look at one section of your code:

Code:

---

}
if(choice == 1)
{
    printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters);
    scanf("%d%d%d%d", &p, &n, &d, &q);
        pennies += p;
        nickels += n;
        dimes += d;
        quarters += q;
        total=total+(25*quarters + 10*dimes + 5*nickels + pennies/100);
    printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters, total);
}

---

In the above code your first printf():

Code:

---

    printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters);

---

You have 5 format specifiers, yet you only have 4 parameters. Always pass the proper number of parameters to printf(). At this point there is no use in printing total because you haven't computed the total yet.

Next why do you have all the "temporary" variables, p,n,q and d? Just use your pennies, nickles, quarters and dimes in your scanf().

Next when you do math with integers there are no fractions. Therefore in your total calculation you will have problems. The variable pennies is an int, the constant 100 is an int therefore this will yield an int value, no fractions. So if pennies is less than 100 you will get zero. Your total calculation should look like:

Code:

---

total=total+(25*quarters) + (10*dimes) + (5*nickels) + pennies);

---

Once you have all the values added together and placed in your floating point number you can then do the division:

Code:

---

total = total / 100.0

---

Jim

We posted at same time!
When I enter $1.41 in change inserted, I get back the total $141.00 (should be $1.41).
When I enter $1.41 in change removed, I get back the total $141.00 (should be $0.00).

Why is this? Thanks.

Code:

---

#include <stdio.h>
#include <math.h>
int main(void)
  { 
  int pennies=0, nickels=0, dimes=0, quarters=0, p,n,d,q;
  float total=0,t;
  int choice=0;   
    printf("Would You Like to Insert or Remove Coins?\n");
    printf("Press 1 for Insert.\n");
    printf("Press 2 for Remove.\n");
    scanf("%d", &choice);


if(choice == 1)
{
    printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters, total);
    scanf("%d%d%d%d", &p, &n, &d, &q);
        pennies += p;
        nickels += n;
        dimes += d;
        quarters += q;
        total=total+(25*quarters + 10*dimes + 5*nickels + pennies);
    printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters, total);
}
else
{
    printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
              pennies, nickels, dimes, quarters, total);
    scanf("%d%d%d%d", &p, &n, &d, &q);
        pennies -= p;
        nickels -= n;
        dimes -= d;
        quarters -= q;
        total=total-(25*quarters + 10*dimes + 5*nickels + pennies);
    printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
            pennies, nickels, dimes, quarters, total);


}
}

---

Quote:

---

Originally Posted by jimblumberg

At this point there is no use in printing total because you haven't computed the total yet.

---

I put that because I thought I would want to show the beginning balance of my purse (if I ever get this into a loop).

Quote:

---

When I enter $1.41 in change inserted, I get back the total $141.00 (should be $1.41).

---

You never do the division. Look closely at my last post.

Quote:

---

When I enter $1.41 in change removed, I get back the total $141.00 (should be $0.00).

---

Actually you should get a value of $-1.41 not zero. Remember total is starting at zero. But look at this calculation:

Code:

---

total=total-(25*quarters + 10*dimes + 5*nickels + pennies);

---

Also remember that quarters, dimes, nickels, and pennies will be negative. What happens if you subtract a negative value from zero?

Quote:

---

I put that because I thought I would want to show the beginning balance of my purse (if I ever get this into a loop).

---

Then you need to add total to the parameters. Remember printf() can really cause problems if you don't pass the correct number of parameters for the number of specifiers.

Jim

Yes, we posted at the same time so I hadn't seen your post on total/100 after the point. So, using that, I get the correct total when inserting money.
For removing money, I can get the correct answer if I use total/-100, but is that an incorrect way to do it?

Code:

---

#include <stdio.h>
#include <math.h>
int main(void)
  { 
  int pennies=0, nickels=0, dimes=0, quarters=0, p,n,d,q;
  float total=0,t;
  int choice=0;       
        printf("Would You Like to Insert or Remove Coins?\n");
        printf("Press 1 for Insert.\n");
        printf("Press 2 for Remove.\n");
        scanf("%d", &choice);


if(choice == 1)
{
        printf( "%d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                    pennies, nickels, dimes, quarters, total);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies += p;
                nickels += n;
                dimes += d;
                quarters += q;
                total=total+(25*quarters) + (10*dimes) + (5*nickels) + pennies;
                total=total/100;
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                    pennies, nickels, dimes, quarters, total);
}
else
{
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                      pennies, nickels, dimes, quarters, total);
        scanf("%d%d%d%d", &p, &n, &d, &q);
                pennies -= p;
                nickels -= n;
                dimes -= d;
                quarters -= q;
                total=total-(25*quarters) - (10*dimes) - (5*nickels) - pennies;
                total=total/-100;
        printf( " %d pennies + %d nickels + %d dimes + %d quarters = $%.2f total\n",
                pennies, nickels, dimes, quarters, total);


}
}

---

Yes, it really is incorrect. In my opinion you should not use negative numbers with your individual coins, how can you have negative coins? By dealing with only positive coins your calculation would be correct:

Code:

---

total = total - ((25 * quarters) + (10 * dimes) + (5 * nickels) + pennies));

---

I also suggest that you use parentheses when using division/multiplication and addition/subtraction. They will make your code much easier to follow. Also a little white space also helps readability.

Jim

I figured out I needed to change it to +- above and then divide by 100 for the removal. It gives me negative numbers now.
I have negative coins and an awkward layout because I am only in week 4 of of programming :)
I

Quote:

---

I have negative coins and an awkward layout because I am only in week 4 of of programming

---

The amount of time programming really has noting to do with negative coins, it's really just common sense.

Jim

i understand that. i just dont know how to do it.

if i'm not doing a loop, then there would be no point to the "removed" function if i didn't allow it to go negative. it would always be zero. i'm going to try to figure out a loop. thanks for all the help.