help counting CHOICES

[nitric]

hey guys, i'm really stuck on this program. It's basically a survey and I have to ask people what drinks they like. 1-4, coffee tea oj and lemonade.

i'm having trouble counting the TOTAL NUMBER OF PEOPLE and counting how many people pick coffee tea oj or lemonade.

i have the 5th option set as a sentinel, where the program ends.

i need help like i said counting the total number of people who participated(person who quits doesn't count) and counting each drink

help?

import java.util.Scanner;
public class Assignment3
{
    public static void main(String[]args)
    {
        int choice = 1;
        int person = 1;
        int person1 = 0;
        int choice1 = 1, choice2 = 1, choice3 = 1, choice4 = 1;
        int total = 0, votes;
        double percent, perc1, perc2, perc3, perc4;
 
        Scanner keyboard = new Scanner(System.in);
 
 
        while (choice != 5)
        {
            System.out.println("Enter choice for person# " + person);
            System.out.println("1. Coffee        2. Tea        3. Orange Juice        4. Lemonade        5. Quit");
            choice = keyboard.nextInt();
            if(choice < 1 || choice  > 5)
                {
                    System.out.print("That was an invalid choice -- enter 1 through 5 only");
                    choice = keyboard.nextInt();
                }
                if(choice == choice1)
                    person++;
                    else 
                        if(choice == choice2)
                            person++;
                                else
                                    if(choice == choice3)
                                        person++;
                                            else
                                                if(choice == choice4)
                                                    person++;
 
        }
        person=choice1+choice2+choice3+choice4;
        perc1 = choice1/person;
        perc2 = choice2/person;
        perc3 = choice3/person;
        perc4 = choice4/person; 
 
 
        System.out.println("The total number of people surveyed: " + person);
        System.out.println("THe results are as follows");
        System.out.println();
 
        System.out.println(" Beverage                Number of Votes            Percentage");
        System.out.println("-------------------------------------------------------------------------------------");
        System.out.println();
        System.out.println("Coffee                    "+choice1+ "                "+perc1);
        System.out.println("Tea                                "+choice2+"                     "+perc2);
        System.out.println("Orange Juice                 "+choice3+"                     "+perc3);
        System.out.println("Lemonade                        "+choice4+"                     "+perc4);
 
    }
}

[Dormilich]

ask one of the moderators to move it to the java forum.

regards

[gits]

done ;)

kind regards
MOD

[JosAH]

@OP: if you want to squeeze all your (convoluted) logic in one single (main) method
you'll end up with a lot of zigzagging code that noone understands, including you.

Unravel your logic in to seperate small simple methods and decompose your
problem. Basically you want something like this:

int choice;
while ((choice=getChoice()) != ENDCHOICE)
   processChoice(choice);
printSummary();
 

The value for ENDCHOICE can be a final static private variable; it's up to you.
The getChoice() method only returns when/if the user has given a legal choice.
The processChoice() method only is passed a valid choice which is not equal
to ENDCHOICE; the logic of the while statement takes care of that,

The printSummary() method prints, *ahem*, a summary. You probably need a
few member variables that hold the data supplied by the user and calculated
by your class. All that data can be private data.

kind regards,

Jos