Need HELP with Program

Ok..I got my second assignment in my programming class last week and I am getting frustrated. Ive gotten as far as displaying the menu but i am stuck and have no clue what to do next. I am still very new at this and im not very good. Can anyone help? If you need to see my code i can post what i have. Thanks


The assignment is

1. create a program that displays a menu with 6 options as seen in the example below.

2. Accept numbers from the user and calculate the selected total, product, average, minimum, or maximum.

Sample Screen Interaction:
Black is the program's output.
Blue is the user's input.

1 - Calculate the Total
2 - Calculate the Product
3 - Calculate the Average
4 - Calculate the Minimum
5 - Calculate the Maximum
9 - Exit
Enter selection: 1

Enter numbers (Enter 0 to stop):
2
3
4
5
0
You entered 4 numbers with a Total of 14.

1 - Calculate the Total
2 - Calculate the Product
3 - Calculate the Average
4 - Calculate the Minimum
5 - Calculate the Maximum
9 - Exit

Enter selection: 9

Thank you come again.


The program should include the following functions:

void findTotal( )
This function will accept numbers from the user until 0 has been entered.
When 0 has been entered, the number of values entered will be displayed followed by the sum of all the numbers entered, as seen in the above example.

void findProduct( )
This function will accept numbers from the user until 0 has been entered.
When 0 has been entered, the number of values entered will be displayed followed by the product of all the numbers entered, as seen in the above example.

void findAverage( )
This function will accept numbers from the user until 0 has been entered.
When 0 has been entered, the number of values entered will be displayed followed by the average of all the numbers entered, as seen in the above example.

void findMinimum( )
This function will accept numbers from the user until 0 has been entered.
When 0 has been entered, the number of values entered will be displayed followed by the minimum of all the numbers entered, as seen in the above example.

void findMaximum( )
This function will accept numbers from the user until 0 has been entered.
When 0 has been entered, the number of values entered will be displayed followed by the maximum of all the numbers entered, as seen in the above example.

Get a response from the user and then do something like this:

Code:

---

switch(choice)
{
  case '1':
    findTotal();
    break;
  case '2':
    findProduct();
    break;
  .
  .
  .
  default:
    puts("Invalid selection");

}

---

That's the way I'd do it anyway. That's the beauty of programming...there's usually a bunch of different ways to accomplish the same task. That's just one idea that I gave you, but I can think of at least 3 other ways to do it off the top of my head.

Ok so i got this. Whats wrong with the way i have it. I dont really understand how to use the switch staments. Its obviously not correct cause it wont compile. What else do i need to include?

Code:

---

#include <stdio.h>

int main()

{
printf(" 1-Calculate the Total\n 2-Calculate the Product\n 3-Calculate the Average\n 4-Calculate the Minimum\n 5-Calculate the Maximum\n 9-Exit");

printf("\n\nEnter Selection: ");
}



switch(choice)

 { case '1':
    findTotal();
    break;
 
  case '2':
    findProduct();
    break;
 
  case '3':
          findAverage();
    break;
 
  case '4':
    findMinimum();
        break;
 
  case '5':
  findMaximum();
  break;
 
 
  default:
    puts("Invalid selection");

}

---

Well, you need to actually get the input from the user first. Check out getchar() or scanf() or fgets() or something.

Ok i got the user imput. Now how do i actually calculate all this?

Code:

---

#include <stdio.h>
void main()

{

int choice;
printf(" 1-Calculate the Total\n 2-Calculate the Product\n 3-Calculate the Average\n 4-Calculate the Minimum\n 5-Calculate the Maximum\n 9-Exit");

printf("\n\nEnter Selection: ");
scanf("%d",&choice);
}


switch(choice)

 { case '1':
    findTotal();
    break;
 
  case '2':
    findProduct();
    break;
 
  case '3':
          findAverage();
    break;
 
  case '4':
    findMinimum();
        break;
 
  case '5':
  findMaximum();
  break;
 
 
  default:
    puts("Invalid selection");

---

To calculate all that I believe(I'm new here so don't quote me on it, just what I have learned till now) you would need something in the order of:

Code:

---

printf (" The sum is %d\n", a+b+c+d);
printf (" The Average is %d\n", (a+b+c+d)/4);
printf (" The Product is %d\n", a*b*c*d);

---

You would of course need to declare the variables.

For the minimum and largest I know a way of doing it but it is not very efficient and I actually have a post on it just now, but anywayz here it is:

Code:

---

smallest = a;
    if (b < smallest)
        smallest = b;
    if (c < smallest)
        smallest = c;
    if (d < smallest)
        smallest = d;
     
      printf ( results, "smallest is %d\n", smallest);
     
    largest = a;
    if (b > largest)
        largest = b;
    if (c >largest)
        largest = c;
    if (d >largest)
        largest = d;
 
        printf ( results, "largest is %d\n", largest);

---

There you go, hope this helps you out in some way :D

The problem with your code is that your user input is in the form of an int, but your case statements are checking for a char. '1' is not the same as 1. Remove the single quotes and the case statements should get executed.

I obviously dont understand what i am doing, can someone tell me where i am going wrong and tell me what i need to do i would greatly apreciate it. Thanks

Code:

---


#include <stdio.h>
void main()

{
char choice;
int a,b,c,d,e,total,product,average;

printf(" 1-Calculate the Total\n 2-Calculate the Product\n 3-Calculate the Average\n 4-Calculate the Minimum\n 5-Calculate the Maximum\n 9-Exit");
printf("\n\nEnter Selection: ");
scanf("%d",&choice);

Total = a + b + c + d + e;
Product = a * b * c * d * e;
Average = ( a + b + c + d + e) / 4;
}


switch(choice)

 { case 1:
    findTotal();
    break;
 
  case 2:
    findProduct();
    break;
 
  case '3':
          findAverage();
    break;
 
  case '4':
    findMinimum();
        break;
 
  case '5':
  findMaximum();
  break;
     
 
  default:
    puts("Invalid selection");

}

---

Well, it being your second assignment and all, I'm assuming the instructor has taught you all you need to know to complete the assignment. So, take the stuff you've learned and apply this basic theory to some workable code.

1. display menu
2. get choice from user
3. if choice is equal to 1 call function that prints total.
4. otherwise if choice is equal to 2 call function that prints product.
5. otherwise if choice is equal to 3 call function that prints average.
6. otherwise if choice is equal to 4 call function that prints minimum.
7. otherwise if choice is equal to 5 call function that prints maximum.
8. otherwise if choice is equal to 6 exit program.
9. return to step 1.

As I mentioned earlier, there's several ways to accomplish this. Take what you've learned and apply it. For instance, if you've learned if/else, but you haven't learned switch/case then forget using switch/case because it's only going to confuse you. It can be written just as well using if/else and that's probably what you've learned by your second assignment.

The actual computations and printing will happen in the individual functions (findTotal(), etc.) since the instructor told you those functions will return type void. Don't let us confuse you by giving you code that you haven't learned yet. Just take what you've learned from your instructor and run with it :)

Your case statements are still comparing an int against a char. Remove the single quotes from '3', '4' and '5'. Ok, take this example. A switch block can be converted into if/else:

Code:

---

if (1 == '1')
    printf("true");
else
    printf("false");

---

If you run this, you'll notice that it will print "false". Why? Because the char '1' is a character. Each character maps to a numerical value on the ASCII chart. If you did:

Code:

---

printf("%d\n", '1');

---

it prints out 49. So when your input from the user is an integer value 1 and you compare it against character '1', then the result is false, because it's comparing if 1 == 49.

Ok this is what i have so far. Now that i can accept input from the user , what do i do to make it figure what it is suppose to figure. Where do i put the void findTotal() for example.

Code:

---

#include <stdio.h>
main()

{
int selection,total,product,average,minimum,maximum;

printf(" 1-Calculate the Total\n 2-Calculate the Product\n 3-Calculate the Average\n 4-Calculate the Minimum\n 5-Calculate the Maximum\n 9-Exit");
printf("\n\nEnter Selection: ");
scanf("%d",&selection);

switch(selection)
{
case 1: printf("\n\nEnter Numbers (Enter 0 to stop):");
break;


case 2: printf("\n\nEnter Numbers (Enter 0 to stop):");
break;


case 3: printf("\n\nEnter Numbers (Enter 0 to stop):");
break;

case 4: printf("\n\nEnter Numbers (Enter 0 to stop):");
break;


case 5: printf("\n\nEnter Numbers (Enter 0 to stop):");
break;

case 9: printf("\n\nThank You, Come Again!");
break;

default: printf("\n\nInvalid Entry!");

}

---

Can someone help me please

Ok, Ill show you an example for the first case statement.

Code:

---

/* declare function prototypes for each function you create */
void findTotal();

int main(void)
{
    .
    .
    .
    case 1: findTotal(); break
    .
    .
    .
}


void findTotal()
{
    int total = 0;
    int input = 0;

    printf("Enter numbers (0 to stop)\n");
    do
    {
        scanf("%d", &input);
        total += input;
    } while (input != 0);

    printf("Total: %d\n", total);
}

---

You should be able to figure the rest from here.

Quote:

---

Originally Posted by Jeff

Can someone help me please

---

Don't bump your threads. Go read the Announcements over again, if you even read them the first time. Here, I'll make it clear for you:

Quote:

---

5. Don't bump your threads, (Bumping: posting messages on your own threads to move them up the list).

---

We are not your personal servents. We don't get paid for this. We do it because we enjoy it. We do not enjoy people breaking the rules, even though it happens all the time. Just because it happens, doesn't mean it's right.

While it is nice you actually used code tags, assuming someone didn't edit your post for you and add them, you should indent your code so it's actually readable.

The "void findtotal" you talk about, as listed in your homework's description, is a rough prototype of what the function should be called, and that it doesn't need to return anything.

You do know how to make functions, right? Well how about making one? In case you don't know how, just look at main, because it too is a function. So you do the same sort of thing.

You put your functions either:
a) Before main, so main can see them.
b) Or you put them after it, and prototype them.

Give it a try, pust your latest attempt, including properly indented and readable code, after you go and read the Announcements again.

Quzah.

ok i figured all of it out but the max and min. this is what i have for the min. i dont think it is correct? does anyone know

Code:

---

void findMinimum()
{
        int minimum = 0;
        int input = 0;
        int count = 0;
        printf("\nEnter Numbers (Enter 0 to Stop): ");
       
do
{
        scanf("%d",&input);
        if (input <= minimum)
        count ++;
}       
        while (input != 0);
        printf("\nYou entered %d numbers with a minimum of: %d",count, minimum);
       
}

---

Are you trying to actually display what the minimum number was? If so, shouldn't you put that into a variable some place when it is less than the current minimum? If so, you don't want to start the value of "minimum" low, you want it high, so that, when you start entering numbers, it will find one lower than your default value, and put it there instead.

Quzah.

yes i want the computer to spit out the lowest number that the user entered. So i need to make minimum = a large number when declaring min or do something with this? or neither?

Code:

---

  if (input <= maximum)

---

Do you actually know how to think things out? If so, do it. Walk though it on paper:

Code:

---

number = small number
get input
if input less than small number
    number = input

---

Ok, so what happens if input is greater than small number, and they don't input anything more? Have you saved their smallest input? Yes? No?

Code:

---

number = big number
get input
if input less than big number
    number = input

---

What about now?

We're not here to do everything for you. If you really don't want to learn, give it up. If you do, then take a moment and actually think about what you're trying to do. It's not a race. Just take your time and figure out what you need to do, and then how to do it.

Quzah.

look quzah..if you dont want to help me then dont reply to my post. I can do without the smart comments about my learning ability. I am new at this and dont know what im doing. I dont want you to do everything for me and you have not even come close to doing that. thanks anyway for your help

You don't seem to understand. You need to take the time and think through your program. It's not a hard concept. If you want to be a programmer, even as a hobbyist, you have to take the time to think your project through.

You aren't doing that. You are simply saying "this didn't work, now this didn't work". Think about why it doesn't work. Think about what exactly is supposed to happen, and why it is or isn't.

If you don't take the time to learn that process, there is no point in even trying to continue.

Quzah.

I think what quzah is trying to point out is that dividing your challenge into logic and specifics portions helps get you to where you're going when you're stuck. For instance, say you're trying to find the end of a string. You know the end of strings always have a special character at the end to mark the end of the string. So you write some logic down.

Code:

---

Iterate through string
  If current character equals EndOfString marker
    Break out of loop

---

Then you can turn it into specifics:

Code:

---

for(i = 0;;++i)
  if(string[i] == '\0')
    break;

---

The same logic applies to pretty much every programming language and pretty nearly corresponds to how you'd look for the end of a string yourself. So even if you're really new to a programming language you can usually get the logic written down. Then it's just a matter of finding out how the programming language you're using implements that logic.

can someone tell me why the minimum is still figuring zero in as a number? what do i need to do to fix this?

Code:

---

void findMinimum()
{
  int minimum = 0;
  int number = 0;
  int count = 0;
  printf("\nEnter Numbers (Enter 0 to Stop): ");
for(;;)
 {
  scanf("%d",&number);
if (number != 0)
    {
if (count == 0)
  minimum = number; 
  count ++;
if (number < minimum )
  {minimum = number; }
    }
else
    {
  printf("\nYou entered %d numbers with a minimum of: %d",count,number);
break;
    }
 }
}

---

Quote:

---

Originally Posted by Jeff

can someone tell me why the minimum is still figuring zero in as a number? what do i need to do to fix this?

Code:

---

void findMinimum()
{
  int minimum = 0;

---

---

I don't have the desire or time to keep going over this. I gave you a perfect illustration of this exact series of events, and obviously you, rather than opting to learn from it, ignored it. This has been answered before. All it requires is that you think about what's going on in your code.

Quzah.

Quote:

---

Originally Posted by quzah

I don't have the desire or time to keep going over this.Quzah.

---

Then dont!!

Code:

---

void findMinimum()
{
  int minimum = 0;
...
if (number < minimum )
  {minimum = number; }

---

Unless number is negative it will never be less than minimum.

Like quzah said in an earlier post:

Quote:

---

Originally Posted by quzah

you don't want to start the value of "minimum" low, you want it high, so that, when you start entering numbers, it will find one lower than your default value, and put it there instead.

---

He won't, but by that comment, quzah is actually trying to help you, because quzah is not alone in thinking this. If you follow the advice quzah gives to a lot of people on this board, then you are more likely to get more help from lots of other people.