Why do I get this error?

[EscSequenceAlpha]

I'm revising polymorphic arrays and I get an error. I dont know why this is. Anyway my array just processes different employee types (they each inherit from the abstract Employee class and have their own implementation of earnings() - I also make changes to a commission employee

public static void main(String[] args)
		{
			SalariedEmployee salariedEmployee = new SalariedEmployee("John", 35);
			CommissionEmployee commissionEmployee = new CommissionEmployee("Haruka", 43, 0.5);
			
			Employee[] employees = new Employee[2];

			employees[0] = salariedEmployee;
			employees[1] = commissionEmployee;
			
			for( Employee currentEmployee : employees );
			{
				if( currentEmployee instanceof CommissionEmployee )
				{
					CommissionEmployee employee = (CommissionEmployee) currentEmployee;
					
					employee.setCommissionRate(0.8);
				}
				
				System.out.println("Employee type: " + employees[currentEmployee].getClass().
						getName() + " earns " + currentEmployee.earnings();
			}
		}

Eclipse says "currentEmployee cannot be resolved to a variable" in the if statement, but havent I already declared it in the enhanced for loop? the stack trace is

Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
	currentEmployee cannot be resolved to a variable
	currentEmployee cannot be resolved to a variable
	currentEmployee cannot be resolved to a variable
	currentEmployee cannot be resolved
	Syntax error, insert ")" to complete Expression

	at Workshop.main(Workshop.java:184)

what ")"? I've looked over this and dont know what I'm missing

[EscSequenceAlpha]

I had an extra semi-colon... why does this happen? I looked it over beforehand and then realise only after I've posted. typical. ugh if I hadnt stopped java this syntax wouldnt have happened... I also see how I cant use the enhanced loop for printing the class name and stuff at the end I need a normal loop

[Fubarable]

This:

            for( Employee currentEmployee : employees );
            {
                if( currentEmployee instanceof CommissionEmployee )
                {
                    CommissionEmployee employee = (CommissionEmployee) currentEmployee;
                     
                    employee.setCommissionRate(0.8);
                }
                 
                System.out.println("Employee type: " + employees[currentEmployee].getClass().
                        getName() + " earns " + currentEmployee.earnings();
            }

Is equivalent to this:

            for( Employee currentEmployee : employees )
            {
                ; // this empty statement gets called in the loop.
            }

            // this block runs no matter what
            {
                if( currentEmployee instanceof CommissionEmployee )
                {
                    CommissionEmployee employee = (CommissionEmployee) currentEmployee;
                     
                    employee.setCommissionRate(0.8);
                }
                 
                System.out.println("Employee type: " + employees[currentEmployee].getClass().
                        getName() + " earns " + currentEmployee.earnings();
            }

[EscSequenceAlpha]

Yes I see thank you, to make this thread somewhat more productive I do have a question I would like to ask of you.

I'm implementing interface payable on Employee class.
payable contains as abstact(obviously) method called getpayment

now, because Employee class is abstract it doesnt provide an implementation (the method is only visable in payable and the subclasses of Employee.

Is this how it should be? can I write

public abstract double getpayment()

in Employee class even know I dont need to (to make it clearer that this is being passed down) or do I just leave it out? what is best practice.

[Tolls]

Leave it out.
Java doc would pick up the existence of the inherited method, and the compiler (or IDE) would pick up the lack of an implementation in any of the Employee subclasses. No need to fill up Employee with irrelevant stuff.

Though I would call it Payable, and getPayment()...naming standards and all that.