Yet another problem with LOOPS

Question

Problem:

Th ree salespeople work at Sunshine Hot Tubs—Andrea,
Brittany, and Eric. Write a console-based application that
prompts the user for a salesperson's initial ('A', 'B', or 'E').
While the user does not type 'Z', continue by prompting for
the amount of a sale the salesperson made. Calculate the
salesperson's commission as 10% of the sale amount, and add
the commission to a running total for that salesperson. After
the user types 'Z' for an initial, display each salesperson's total
commission earned

Question: it doesn't work, what should I do

What I have done so far:

            char A, B, E, Z;
            string userInputString, userInputAmountString;
            char userInput;
            double userInputAmount, tip =.10, comission, total;

            do
            {
                Console.WriteLine("Please enter initial of the salesperson");
                userInputString = Console.ReadLine();
                userInput = Convert.ToChar(userInputString);
                Console.WriteLine("Please enter the amount of a sale the saleperson made");
                userInputAmountString = Console.ReadLine();
                userInputAmount = Convert.ToDouble(userInputAmountString);

                comission = userInputAmount * tip;
                total = comission + userInputAmount;



            }
            while ( userInput == A || userInput == B || userInput == E );
            Console.WriteLine("The total saleperson made is {0}", total);

Vulpes · Answer

In general you need a separate loop for each input which might be invalid. Also you're going to need an 'outer' infinite loop here to keep getting input until 'Z' is pressed when you can break out of the infinite loop and print the results. As it's a bit trickier than the previous examples I thought I'd complete it for you before it's time for me to turn in: using System; class Program { static void Main() { string inputString userInputAmountString; char userInput; double userInputAmount commission; double andreaTotal = 0 brittanyTotal = 0 ericTotal = 0; while(true) { do { Console.WriteLine(Please enter initial of the salesperson); inputString = Console.ReadLine().ToUpper(); userInput = Convert.ToChar(inputString); } while(userInput != 'A' && userInput != 'B' && userInput != 'E' && userInput != 'Z'); if (userInput == 'Z') break; Console.WriteLine(Please enter amount of sale made); userInputAmountString = Console.ReadLine(); userInputAmount = Convert.ToDouble(userInputAmountString); commission = userInputAmount * 0.1; if (userInput == 'A') { andreaTotal += commission; } else if (userInput == 'B') { brittanyTotal += commission; } else { ericTotal += commission; } } Console.WriteLine(); Console.WriteLine(Commission earned by Andrea = {0:C} andreaTotal); Console.WriteLine(Commission earned by Brittany = {0:C} brittanyTotal); Console.WriteLine(Commission earned by Eric = {0:C} ericTotal); Console.ReadKey(); } }

Vulpes · Answer

do/while carries on looping as long as the while condition is true. It is similar to a while loop except that unlike the latter the code in the loop is guaranteed to execute at least once. Now if the initial entered is not A B E or Z then we want the loop to continue so we can get a valid initial. The following condition achieves that: while(userInput != 'A' && userInput != 'B' && userInput != 'E' && userInput != 'Z');

Maha · Answer

Another way of doing this program I am not sure whether it is correct or not. But if you have any problem with this program let me know. using System; namespace ConsoleApplication1 { class Program { static void Main(string[] args) { char initial; double sAmountA=0 sAmountB=0 sAmountE=0 tip = .10 Acomission=0 Bcomission=0 Ecomission=0 Atotal=0 Btotal=0 Etotal=0; Console.Write(Initial ); initial = Convert.ToChar(Console.ReadLine()); while (true) { if (initial == 'A') { Console.Write(Sale $ sAmountA); sAmountA = Convert.ToDouble(Console.ReadLine()); Acomission = sAmountA * tip; Atotal = Acomission + Atotal; Console.WriteLine(Comission {0} Acomission.ToString(C)); } if (initial == 'B') { Console.Write(Sales $ sAmountB); sAmountB = Convert.ToDouble(Console.ReadLine()); Bcomission = sAmountB * tip; Btotal = Bcomission + Btotal; Console.WriteLine(Comission {0} Bcomission.ToString(C)); } if (initial == 'E') { Console.Write(Sale $ sAmountE); sAmountE = Convert.ToDouble(Console.ReadLine()); Ecomission = sAmountE * tip; Etotal = Ecomission + Etotal; Console.WriteLine(Comission {0} Ecomission.ToString(C)); } total: if (initial == 'Z') { Console.WriteLine(A's total Comission {0} Atotal.ToString(C)); Console.WriteLine(B's total Comission {0} Btotal.ToString(C)); Console.WriteLine(E's total Comission {0} Etotal.ToString(C)); } Console.WriteLine(); Console.Write(Initial ); initial = Convert.ToChar(Console.ReadLine()); if (initial == 'Z') { Console.WriteLine(Total commission of each sales persons); Console.WriteLine(); goto total; } } } } }

Prime b · Answer

That's pretty complicated... But Thank you again! See I don't understand the while (userInput != 'A' && userInput != 'B' && userInput != 'E' && userInput != 'Z'); To me it seems like if userInput NOT equals A or B or E or Z then proceed  but I thought it should be ==