I cant get the calculations to work. Help please

Question

1.00/5 (1 vote)

See more:

 class Program
    {


        static void Main(string[] args)
        {
            string customerName = "", state = "";
            int quantity = 0;
            double price = 0, sales = 0, tax = 0;

            Console.WriteLine("\n\nWelcome to the Sales Tax Calculator");

            Console.Write("\n\nPlease enter in the customer name:  ");
            customerName = Console.ReadLine();
            Console.Write("\n\nPlease enter in the State (NY / NJ / FL):  ");
            state = Console.ReadLine();
            Console.Write("\n\nPlease enter in the number of items purchased:  ");
            quantity = Convert.ToInt32(Console.ReadLine());
            Console.Write("\n\nPlease enter in the unit price of the item:  ");
            price = Convert.ToDouble(Console.ReadLine());



            sales = ComputeTotal(quantity, price);

            tax = ComputeTax(sales, state);

            Console.WriteLine(customerName + " your total sales are:  ");

            Console.WriteLine("Your total with taxes is:  " + tax);


            Console.ReadLine();
            
        }
        public static double ComputeTotal(int quantity, double price)
        {
            return (quantity * price);
        }

        public static double ComputeTax(double sales, string state)
        {
            double taxAmt = 0;

            if (state == "NY")
                taxAmt = sales * .04;
            else if (state == "NJ")
                taxAmt = sales * .07;
            else if (state == "FL")
                taxAmt = sales * .06;

            return taxAmt;


        }
    }
}

What I have tried:

I honestly don't know what else to try. I was up all night trying to figure this out on my own.

Posted 17-Jun-18 4:21am

Member 13834513

Updated 17-Jun-18 4:32am

Add a Solution

Comments

[no name] 17-Jun-18 10:28am

You should also explain what your problem is. E.g. What you enter, what you expect as result and what result you get from your code.

Member 13834513 17-Jun-18 10:39am

So I need it to compute the sales total, the tax and print them both out. I know that there is something wrong with the calculations but I can't figure out what is going wrong.

OriginalGriff 17-Jun-18 10:40am

So use the debugger and find out!

Member 13834513 17-Jun-18 11:19am

I have used the debugger and I still can't figure it out. I am just a beginner who is trying to learn C#. I found some of my brothers school work and thought I would may like this.

OriginalGriff 17-Jun-18 11:23am

And what did the debugger show you?
You can't just "grab software" and magically understand it, you have to learn the basics properly or the more advanced stuff won't make any sense at all!

[no name] 17-Jun-18 10:42am

Again: Describe what happens and give an example. What are your Inputs, what do you expect as result and what result do you get from your code.

There are so much possibilities where the current code can be wrong. E.g. the comparison state == "XYZ". Think about something like this state.ToUpper() == "XYZ"

Patrice T 17-Jun-18 10:58am

Show sample input, actual output and expected output.

Member 13834513 17-Jun-18 11:08am

So lets say the customerName entered in is Bill, then the state entered in is NJ(.07 tax), 3 items purchased and the unit price is 1.50.
The output should be total sales for Bill are 4.50 and the total with taxes is 4.82.

[no name] 17-Jun-18 11:20am

Console.WriteLine("Your total with taxes is: " + tax);
vs.
Console.WriteLine("Your total with taxes is: " (sales + tax).ToString());

Patrice T 17-Jun-18 11:31am

I said actual output and expected output
Or explain how it is wrong.
Not everyone your language ready to run !

Use Improve question to update your question.
So that everyone can pay attention to this information.

[no name] 17-Jun-18 15:10pm

Why don't you "display" the entered values (i.e. price, qty, etc) to CONFIRM they are what you expect (upper case; lower case; not "empty"; etc.) before doing your calculations?

You're flying blind.

George Swan 19-Jun-18 1:50am

What result are you expecting? If amount is 10, price 100, tax at NY rate= 0.04 then sales=100*10=1000, Tax=1000*0.04= 40,Total Payable=1000+40= 1040

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Answer 1 · 2018-06-17T04:33:00

Compiling does not mean your code is right! :laugh:
Think of the development process as writing an email: compiling successfully means that you wrote the email in the right language - English, rather than German for example - not that the email contained the message you wanted to send.

So now you enter the second stage of development (in reality it's the fourth or fifth, but you'll come to the earlier stages later): Testing and Debugging.

Start by looking at what it does do, and how that differs from what you wanted. This is important, because it give you information as to why it's doing it. For example, if a program is intended to let the user enter a number and it doubles it and prints the answer, then if the input / output was like this:

Input   Expected output    Actual output
  1            2                 1
  2            4                 4
  3            6                 9
  4            8                16

Then it's fairly obvious that the problem is with the bit which doubles it - it's not adding itself to itself, or multiplying it by 2, it's multiplying it by itself and returning the square of the input.
So with that, you can look at the code and it's obvious that it's somewhere here:

C#

private int Double(int value)
   {
   return value * value;
   }

Once you have an idea what might be going wrong, start using the debugger to find out why. Put a breakpoint on the first line of the method, and run your app. When it reaches the breakpoint, the debugger will stop, and hand control over to you. You can now run your code line-by-line (called "single stepping") and look at (or even change) variable contents as necessary (heck, you can even change the code and try again if you need to).
Think about what each line in the code should do before you execute it, and compare that to what it actually did when you use the "Step over" button to execute each line in turn. Did it do what you expect? If so, move on to the next line.
If not, why not? How does it differ?
Hopefully, that should help you locate which part of that code has a problem, and what the problem is.

This is a skill, and it's one which is well worth developing as it helps you in the real world as well as in development. And like all skills, it only improves by use!