Thread: Need help with simple code

What is wrong with your code right now?

I can see one potential logistic mistake and 2 errors that your compiler should catch:
Logistic Mistakes
In the following calculations in your main:

double planA = (((hours - 10) *2) + 9.95);
         double planB = (((hours - 20) *1) + 13.95);
         double planC = 19.95;

You are assuming that the number of hours inputted will be at least 20. What if the user enters just 9 hours? That would cause a reimbursement of hours from the calculations since the hours, multiplied by the cost, will result in a negative number. To have a "complete" calculation, you should check if the number of hours is less than 10 before calculating Plan A and you should check if the number of hours is less than 20 before calculating Plan B.

Compiler Error(1)
Compare the following two lines:

char plan = bestPrice(planA, planB, planC);
public static double bestPrice(double planA, double planB, double planC)

You are declaring the bestPrice method to return a double, but when you retrieve that value in your main you are setting the value to a char. There should (theoretically) be a compiler error. What you should do is change either the call in your main or your method declaration to be consistent with what you are trying to do.

Compiler Error(2)
Look at the following method:

public static double bestPrice(double planA, double planB, double planC)
      {
 
 
         if (planA < planB && planC)
 
            System.out.println("Plan A is best for you.");
 
         else 
            if (planB < planC)
               System.out.println("Plan B is best for you.");
 
            else 
               System.out.println("Plan C is best for you.");
 
      }

You have declared this method to return a double but no where in the method do you have a return statement that returns a double. This will cause a compiler error as a double is expected to be returned.



Since you have said you are new to programming, I'll help you as much as you can without actually doing it for you. Try to fix the above (compiler errors specifically, the logistical error can be dealt with later). If you still are having trouble with it after you attempt to fix it with what you know, post the new code and we will go from there. If you are unsure how to do something, feel free to ask specifically what you are unsure of.