Alright, my eyes hurt from staring at this monitor trying to figure this out.

[Greg Cook]

This is my first program and it is giving me so many errors. It seems like as soon as I get rid of one error, two or more pop up in their place!
I really need some help with this, where is my big error(s)?
I keep getting a " 'case'outside of switch" error after the first case.

/**
* This program will allow a user to input the number of hours worked and the pay rate
* the worker receives per hour. It will then calculate the regular pay, overtime pay
* (If received), and the gross pay.
* It will also allow the user to tally the total number of successful transactions
* and the totals for regular, overtime, and gross pays.
*
* Written by: Greg Cook
*/
public class App {
  public static void main(String[] args) {
    // Variables needed for individual calculations
    double hoursWorked;
    double payRate;
    double regularPay;
    double overtimePay;
    double grossPay;
    // Variable to keep tally of total calculations
    int counter;
    // Variables used for keeping totals of all calculations
    double totalRegPay;
    double totalOTPay;
    double totalGrossPay;
    // Variables for user interaction
    int choice;
    int answer;
    // Prompt for and read data
    System.out.println("Your possible choices are:");
    System.out.println("\t" + "1. - Calculate Workers Pay");
    System.out.println("\t" + "2. - Show Statistics for Calculations");
    System.out.println("\t" + "3. - Exit");
    System.out.print("Which function would you like to perform: ");
    choice = Keyboard.readInt();
    // Process based upon user information entered
    switch (choice) {
      case '1':
      // User prompted for data entry
      System.out.print("Please enter the Number of Hours worked: ");
      hoursWorked = Keyboard.readDouble();
      System.out.print("Please enter the Hourly Rate of the Worker: ");
      payRate = Keyboard.readDouble();
      // Beginning of the loop to calculate the pay rate of a worker
        do {
          totalOTPay = '0';
          totalRegPay = '0';
          totalGrossPay = '0';
          counter = '0';
          answer = 'N';
          if (hoursWorked > 0 & payRate >= 5.15) {
            if (hoursWorked > 40) {
              overtimePay = (hoursWorked - 40) * (1.5 * payRate);
              regularPay = ((hoursWorked - (hoursWorked - 40)) * payRate);
              grossPay = (overtimePay + regularPay);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(grossPay));
              totalOTPay = totalOTPay + overtimePay;
              totalRegPay = totalRegPay + regularPay;
              totalGrossPay = totalGrossPay + (totalOTPay + totalRegPay);
              counter = counter++;
            }
            else {
              regularPay = (hoursWorked * payRate);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(regularPay));
              totalRegPay = totalRegPay + regularPay;
              totalGrossPay = totalGrossPay + totalRegPay;
              counter = counter++;
            }
            System.out.println("You entered an invalid choice for either the Hours Worked or the Pay Rate.");
            System.out.print("Would you like to calculate another employee? (Y/N) ");
            answer = Keyboard.readInt();
        }while (answer == 'Y' || answer == 'y');
      break;
      case '2':
        totalOTPay = '0';
        totalRegPay = '0';
        totalGrossPay = '0';
        counter = '0';
        System.out.println("The total number of calculations performed is :" + counter);
        System.out.println("The Total Regular pay of all the employees is :" + totalRegPay);
        System.out.println("The Total Overtime Pay of all the employees is :" + totalOTPay);
        System.out.println("The Total Gross Pay of all the employees is :" + totalGrossPay);
        break;
      case '3':
        System.out.println("Have a good day!");
        break;
      default:
        System.out.println("Invalid transaction code");
        break;
      }
    }
  }

[ October 03, 2002: Message edited by: Thomas Paul ]

[Thomas Paul]

In the do loop, the first if doesn't have a matching }

[Greg Cook]

Thank you. That did fix my problem. But, just like I said before. Once that problem was out of the way, 11 more popped to take its place.
Now all my variables are saying:
"App.java": Error #: 553 : variable totalOTPay might not have been initialized at line 66, column 28
Or something very similar with the variable name and locations being different. This is getting frustrating b/c I have no one experienced to ask directly but you guys out here.
I am going to keep staring and playing with this, hopefully someone can tell me what is causing this new set of errors now.

[Greg Cook]

I may have figured my last problem already, but I think I went about it in the wrong way. Here's the new code:

/**
* This program will allow a user to input the number of hours worked and the pay rate
* the worker receives per hour. It will then calculate the regular pay, overtime pay
* (If received), and the gross pay.
* It will also allow the user to tally the total number of successful transactions
* and the totals for regular, overtime, and gross pays.
*
* Written by: Greg Cook
*/
public class App {
  public static void main(String[] args) {
    // Variables needed for individual calculations
    double hoursWorked;
    double payRate;
    double regularPay;
    double overtimePay;
    double grossPay;
    // Variable to keep tally of total calculations
    int counter;
    // Variables used for keeping totals of all calculations
    double totalRegPay;
    double totalOTPay;
    double totalGrossPay;
    // Variables for user interaction
    int choice;
    char answer;
    // Prompt for and read data
    System.out.println("Your possible choices are:");
    System.out.println("\t" + "1. - Calculate Workers Pay");
    System.out.println("\t" + "2. - Show Statistics for Calculations");
    System.out.println("\t" + "3. - Exit");
    System.out.print("Which function would you like to perform: ");
    choice = Keyboard.readInt();
    // Process based upon user information entered
    do {
    switch (choice) {
      case '1':
      // User prompted for data entry
      System.out.print("Please enter the Number of Hours worked: ");
      hoursWorked = Keyboard.readDouble();
      System.out.print("Please enter the Hourly Rate of the Worker: ");
      payRate = Keyboard.readDouble();
      // Beginning of the loop to calculate the pay rate of a worker
          totalOTPay = '0';
          totalRegPay = '0';
          totalGrossPay = '0';
          counter = '0';
          if (hoursWorked > 0 & payRate >= 5.15) {
            if (hoursWorked > 40) {
              overtimePay = (hoursWorked - 40) * (1.5 * payRate);
              regularPay = ((hoursWorked - (hoursWorked - 40)) * payRate);
              grossPay = (overtimePay + regularPay);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(grossPay));
              totalOTPay = totalOTPay + overtimePay;
              totalRegPay = totalRegPay + regularPay;
              totalGrossPay = totalGrossPay + (totalOTPay + totalRegPay);
              counter = counter++;
            }
            else {
              regularPay = (hoursWorked * payRate);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(regularPay));
              totalRegPay = totalRegPay + regularPay;
              totalGrossPay = totalGrossPay + totalRegPay;
              counter = counter++;
            }
            System.out.println("You entered an invalid choice for either the Hours Worked or the Pay Rate.");
          break;
        }
      case '2':
        totalOTPay = '0';
        totalRegPay = '0';
        totalGrossPay = '0';
        counter = '0';
        System.out.println("The total number of calculations performed is :" + counter);
        System.out.println("The Total Regular pay of all the employees is :" + totalRegPay);
        System.out.println("The Total Overtime Pay of all the employees is :" + totalOTPay);
        System.out.println("The Total Gross Pay of all the employees is :" + totalGrossPay);
        break;
      case '3':
        System.out.println("Have a good day!");
        break;
        }
      System.out.print("Would you like to calculate another employee? (Y/N) ");
      answer = Keyboard.readChar();
        if (answer == 'N' || answer == 'n'); {
          break;
        }
      }
    while (answer == 'Y' || answer == 'y');
  }
}

[ October 03, 2002: Message edited by: Greg Cook ]
[ October 03, 2002: Message edited by: Greg Cook ]

[Thomas Paul]

You can't use a method variable until it has been initialized.
This will give an error:

int myVar;
System.out.println(myVar);

This will not:

int myVar = 0;
System.out.println(myVar);

[Greg Cook]

Thomas, you have been a big help so far.
I have everything working so far in case 1. I figured it out this morning, had to sleep on it I guess. I woke up and it was just there in my head.
Here's my new code:

/**
* This program will allow a user to input the number of hours worked and the pay rate
* the worker receives per hour. It will then calculate the regular pay, overtime pay
* (If received), and the gross pay.
* It will also allow the user to tally the total number of successful transactions
* and the totals for regular, overtime, and gross pays.
*
* Written by: Greg Cook
*/
public class App {
  public static void main(String[] args) {
    // Variables needed for individual calculations
    double hoursWorked;
    double payRate;
    double regularPay;
    double overtimePay;
    double grossPay;
    // Variable to keep tally of total calculations
    int counter;
    // Variables used for keeping totals of all calculations
    double totalRegPay;
    double totalOTPay;
    double totalGrossPay;
    // Variables for user interaction
    int choice;
    char answer;
    // Initialize all the variables to zero
    totalOTPay = '0';
    totalRegPay = '0';
    totalGrossPay = '0';
    counter = '0';
    // Beginning of the loop to calculate the pay rate of a worker
    do {
    // Prompt for and read data
    System.out.println("Your possible choices are:");
    System.out.println("\t" + "1. - Calculate Workers Pay");
    System.out.println("\t" + "2. - Show Statistics for Calculations");
    System.out.println("\t" + "3. - Exit");
    System.out.print("Which function would you like to perform: ");
    choice = Keyboard.readInt();
    switch (choice) {
      case 1:
      // Add one to the counter each time "case 1" is selected
      counter = counter++;
      // User prompted for data entry
      System.out.print("Please enter the Number of Hours worked: ");
      hoursWorked = Keyboard.readDouble();
      System.out.print("Please enter the Hourly Rate of the Worker: ");
      payRate = Keyboard.readDouble();
      // Process based upon user information entered
          if (hoursWorked <= 0 | payRate < 5.15) {
              System.out.println("You have either entered too few hours or too little pay.");
              break;
          }
          if (hoursWorked > 0 & payRate >= 5.15) {
            if (hoursWorked > 40) {
              overtimePay = (hoursWorked - 40) * (1.5 * payRate);
              regularPay = ((hoursWorked - (hoursWorked - 40)) * payRate);
              grossPay = (overtimePay + regularPay);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(grossPay));
              totalOTPay = (totalOTPay + overtimePay);
              totalRegPay = (totalRegPay + regularPay);
              totalGrossPay = (totalGrossPay + (totalOTPay + totalRegPay));
              break;
            }
            else {
              regularPay = (hoursWorked * payRate);
              System.out.println("The Amount owed to the worker is: " + Utility.moneyFormat(regularPay));
              totalRegPay = (totalRegPay + regularPay);
              totalGrossPay = (totalGrossPay + totalRegPay);
              break;
            }
        }
      case 2:
        System.out.println("The total number of calculations performed is :" + counter);
        System.out.println("The Total Regular pay of all the employees is :" + totalRegPay);
        System.out.println("The Total Overtime Pay of all the employees is :" + totalOTPay);
        System.out.println("The Total Gross Pay of all the employees is :" + totalGrossPay);
        break;
      case 3:
        System.out.println("Have a good day!");
        break;
        }
      System.out.print("Would you like to calculate another employee? (Y/N) ");
      answer = Keyboard.readChar();
        if (answer == 'N' || answer == 'n') {
          break;
        }
    }while (answer == 'Y' || answer == 'y');
  }
}

[Greg Cook]

Now, I just have a problem with Case 2
It will give me a counter value = 48 no matter how many times I run it. I have moved the counter around to different areas of the program to see if it works better, but nothing seems to affect it. So, I have come to the conclusion that something external to the actual line of code is affecting it.
The loop works, the case 1 works flawless too. Just something to sort out in Case 2. And I thought that would be the easy one.

[Barry Gaunt]

Is there any special reason why you initialize
your variables (including counter) with character literals?

[Thomas Paul]

You can't do:
counter = counter++
Try doing this instead:
counter++

[Barry Gaunt]

Oh yes you can, he did it!
That explains why once he stuffed '0' into counter,
it always stayed the same. I wonder what the decimal value of '0' is?
-Barry

[Chris Stewart]

Why pick something like this for a first application? Start low and learn the basics.

[Layne Lund]

As for fixing one error and 11 more popping up, that's just the nature of the beast. My strategy when I start compiling is to just look at the first error and fix it. I don't usually worry about the other errors because, as you noticed, they can change drastically after one error is fixed. After a while, you tend to get a feeling for which errors are related and which are not.
Keep coding!
Layne

[Greg Cook]

To answer some of the questions:
I have already fixed the counter++
I came to that conclusion earlier today.
Why did I pick this for my first assignment? I didn't, it was assigned this way.
I initialized the variables at '0' because it was telling me that "variable xxx may not have been initialized at row xx column xx"
So, I initialized them all to '0' to take care of the errors.
The whole program now runs fine, except that case 2 spits out nothing but '48' or '48.0' for an answer for all 4 variables. If I ran case 1, it will add 48 to the correct answer. Case 1 works flawlessly.

[Thomas Paul]

You want to initialize your numeric values to 0 not '0'!
Try that.
int totamount=0;

[Greg Cook]

Thank Thomas. I was given that solution earlier. It solved all my problems.
With some cosmetic seperator lines and spaces, it looked and ran good.

Is there any special reason why you initialize
your variables (including counter) with character literals?

Oh yes you can, he did it!
That explains why once he stuffed '0' into counter,
it always stayed the same. I wonder what the decimal value of '0' is?
-Barry

These two comments helped, after I thought about them.
I know Case 3 didn't quite work right, but I wasn't going to try to re-write half the program to have its break statement drop me out of the program. It asks you if you want to run the program again, but I didn't care. I was just happy to see the rest of the program run.
Thanks for all the help guys! This place is great.

[Barry Gaunt]

That's what those two weird comments were supposed
to do. Man am I weird...
If I just tell you how to fix it, you do it and
it works. If you think, then see it, then say to
youself "Oh Sh*T, why didn't I see that", a few
more neurons are fired and the connections burned in.
Suggestion: Go get these out the library and work through them
On to Java
Objects First
[ October 05, 2002: Message edited by: Barry Gaunt ]