unreachable statement

[Danelle Chamberlain]

I am trying to work on a do while statement but I'm getting an unreachable statement error when running the compiler. where did I go wrong with my code?

 case CALCULATE_PAY: {

// This loop will be repeated based upon the value of again.
do {
       //prompt user for hours and wage        
       System.out.print("\n" + "Enter number of hours worked: ");
       hoursWorked = Keyboard.readDouble();
       System.out.print("\n" + "Enter Wage: ");
       wage = Keyboard.readDouble();
   
        //loop to make sure hours is a valid number
        while (hoursWorked < 0) {
 System.out.println("\n" + "invalid entry");
 System.out.print("\n" + "Enter number of hours worked: ");
         hoursWorked = Keyboard.readDouble();
        }
    
//loop to make sure wage is valid
    
        while (wage < 5.15) {
 System.out.println("\n" + "invalid entry");      
 System.out.print("\n" + "Enter wage: ");
         wage = Keyboard.readDouble();
        }
        //calculate regular hours, overtime, and gross pay        
            
        if (hoursWorked <= 40) {
         regularPay = hoursWorked * wage;
         System.out.println(regularPay);
}
        else {
 regularPay = (double) (40 * wage);
 overTime = (double) ((hoursWorked - 40) * (wage * 1.5));
         grossPay = (double) (overTime + regularPay);
         System.out.println("Regular pay is: " + regularPay);
 System.out.println("Overtime pay is: " + overTime); 
 System.out.println("Gross pay is: " + grossPay); 
        }
          // Skip a line and break out of the switch.
          break;
// Draw a separator, then ask the user if they want to do calculate
      // a new pay and read their reply.
        
        System.out.print("Another Calculation? (Y/N): ");
        repeat = Keyboard.readChar();
        // Repeat the loop if needed.
} while (repeat == 'Y' || repeat == 'y');
      }
 

[Jason Menard]

The way your class is written, these lines following your break statement are unreachable:

// Draw a separator, then ask the user if they want to do calculate
// a new pay and read their reply.
System.out.print("Another Calculation? (Y/N): ");
repeat = Keyboard.readChar();
// Repeat the loop if needed.

Also, what the break statement is doing is breaking you out of your do-while loop, which in effect means that the code inside the loop will only ever execute once. I suspect you want the break statement after your do-while loop.
[ September 26, 2003: Message edited by: Jason Menard ]

[Danelle Chamberlain]

thank you for your help

[AJ Keats]

As a point of interest, unreachable code is always an error in Java. Unreachable code means that there is no path of execution that will cause the code to execute. This can occur a number of different ways, including a misplaced break statement, as is the case with your code. Here is another example:
class Opps {
public static void main(String args[]) {
int i = 99;

if(i < 100) {
System.out.println("Less than 100.");
return;
} else {
System.out.println("Greater than or equal to 100.");
return;
}
System.out.println("This is unreachable!");
}
}
In this program, both paths of execution through the if/else end with a return. Thus, the last call to println() is unreachable. That is, there is no path of execution that can lead to the final println().
Keep in mind that javac can't find all unreachable code. It can only find code that is unreachable because there is no path of execution that can possibly lead to the unreachable code. Code that is unreachable because of logic errors in your program won't be found. For example, assume that your program has a switch statement that contains a case value that will never occur during the execution of the program. This case statement is NOT flagged as unreachable because the compiler has no way of knowing that this case value will never occur at runtime.