Prices will not display

[suzukijohn Jj]

import java.util.Scanner;

public class SwitchISP 
{

	public static void main(String[] args) 
	{
		String inputPackage;					// Choose Package A, B, C
		char packages;
		
		final float	
					Pack_A 		= 9.95f,		// Package A Price
					Pack_B 		= 13.95f,		// Package B Price
					Pack_C 		= 19.95f;		// Package C Price
		
			float		
					XFee_A 		= 2.00f,		// Package A extra Fee
					XFee_B 		= 1.00f;		// Package B extra Fee
	
			float
					totCharge 	= 0f,			// Total Charge
					xCharge 	= 0f;			// Extra Charge 
					
			
			int
					PackHR_A 	= 10,			// Package A standard Hours
					PackHR_B 	= 20,			// Package B Standard Hours
				//	PackHR_C 	= Unlimited 	   Package C Standard Hours
					hrsOver 	= 0,
					inputHours	= 0;
					
					
			
			
///////////////////////////////////////////						
			Scanner ISP = new Scanner( System.in );
			
			System.out.print( "What is the package you are signed up for?" );
			inputPackage = ISP.nextLine();
			
			System.out.print( "How many hours were used?" );
				inputHours = ISP.nextInt();				
				packages = inputPackage.charAt(0);
////////////////////////////////////////////////			
			
			
			
			switch(packages) 					// Switch expression
			{
			case 'A':							// $9.95 per month -- 10hr -- $2.00 per hour over
					if(inputPackage.equals('A'))
					totCharge = Pack_A;
					if( inputHours > PackHR_A );
					totCharge = Pack_A + (inputHours * XFee_A);
					
							break;
			
			case 'B':							// $13.95 per month -- 20hr -- $1.00 per hour over
					if(inputPackage.equals('B'))	
						totCharge = Pack_B;
					if( inputHours > PackHR_B );
					totCharge = Pack_B + (inputHours * XFee_B);
					
							break;
			case 'C':							// $19.95 per month -- unlimited -- $0.00 per hour over
					if(inputPackage.equals('C'))
						totCharge = Pack_C;
				
							break;
			default:
					
				System.out.printf("Your Total Charge is $%,.2f",totCharge);
			
			
					
				
		
		}					
	}
}

[Junilu Lacar]

You have your System.out.println() statement under the default case of the switch statement. That means it will only be executed if the package is not one of any of the cases above it. That is, if the package is A, B, or C, then the System.out.println() statement will NOT be performed. Only when it is NOT any of A, B, or C will it display something.

[Junilu Lacar]

Also, the if-statements under each case are redundant. For example, for case 'A', you check if (inputPackage.equals('A'). Why? The case 'A' label already establishes that packages is in fact 'A' so why do you need to check it again?

[Junilu Lacar]

Another thing: look carefully at your if-statements on line 53 and 61. You have a very subtle bug on those two if-statements.  If you test your program thoroughly, you'll find that it's always charging extra fees for A and B.

[Knute Snortum]

On line 48 the OP is switching on packages, which is an int, not a character.  This will be very confusing.  You probably want to switch on inputPackage.

[Campbell Ritchie]

Junilu Lacar wrote:. . . line 53 and 61. You have a very subtle bug . . . .

The error is so similar on those two lines that I suspect there has been copying'n'pasting going on. That shows why copying'n'pasting code is a bad idea.