Yet another problem with LOOPS

Prime b

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);

4 Replies

Know the answer? Post it — somebody with the same question will find it here.

Vulpes15y ago
Accepted 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();
}
}
0
Vulpes15y ago
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');
0
15y ago
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;
}
}

}
}
}
0
Prime b15y ago
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 ==
0