my first Java nightmare

[Jesse Crockett]

I'm trying another route. Should this work?

 
 
Double[] pay = processPay(hourlyRate, hoursWorked, taxRate);
 
printEmployeeInfo(employeeName, hourlyRate, hoursWorked, taxRate, pay[0], pay[1]);
_________
 
static Double[] processPay(Double hourlyRate, Double hoursWorked, Double taxRate) {
 
final double OVERTIME_RATE = 1.5;
 
if(hoursWorked > FULL_TIME)
Double grossAmount = new Double((hourlyRate * FULL_TIME) + (hourlyRate * OVERTIME_RATE * (hoursWorked - FULL_TIME)));
else
Double grossAmount = new Double(hourlyRate * hoursWorked);
 
totalGrossAmount = totalGrossAmount + grossAmount.doubleValue();
 
double netAmount = grossAmount * taxRate / 100;
 
totalNetAmount = totalNetAmount + netAmount;
 
return new Double[] {grossAmount, netAmount};
}
 

 
static void printEmployeeInfo(StringBuffer employeeName, Double hourlyRate, Double hoursWorked, Double taxRate, Double pay[0], Double pay[1]) {
 
System.out.printf("%-20s %8.2f %8.2f %8.2f %8.2f %8.2f%n", employeeName, hourlyRate, hoursWorked, taxRate, pay[0], pay[1]);
 
}
 
   

[Garrett Rowe]

That looks fine to me. (By fine I mean it looks like it should work). Does it work like you expected it to?

IMO going through those "Java nightmares" just makes you stronger (to rewrite a tired cliche'). The next time, you will understand the pitfalls involved with this lesson. Especially with things like variable scope and shadowing primitive variables and such.

Garrett
[ February 10, 2006: Message edited by: Garrett Rowe ]

[Jesse Crockett]

It still isnt working. I'm getting some weird ')' and '}' expected, for which I can't figure out where they should go. So I'm moving on to the next exercise, then maybe come back to this fresh (hopefully).

[Ernest Friedman-Hill]

This declaration:

static void printEmployeeInfo(StringBuffer employeeName, Double hourlyRate, Double hoursWorked, Double taxRate, Double pay[0], Double pay[1]) { ...

is causing the errors. The fact that you're passing array elements as arguments to this routine is irrelevant; when you declare the routine, you declare the arguments as plain old Doubles. The "Double pay[0]" syntax is what's wrong; you just want "Double pay0, Double pay1" or something like that (I don't know what those two elements represent or I'd give the parameters better names.)

[Henry Wong]

Originally posted by Jesse Crockett:
It still isnt working. I'm getting some weird ')' and '}' expected, for which I can't figure out where they should go. So I'm moving on to the next exercise, then maybe come back to this fresh (hopefully).

First of all, don't worry about the tons of errors messages from the compiler. Deal with it one at at time. Fix one, then compile again. With experience later, you'll learn to do more than one.

Second, the most important items with the error message is the file and line number. Get a good editor, open the file specified, go to the line specified, and start from there. Errors having to do with braces are generally cause eariler -- so you may have to back up a bit. Having an editor that will help you match braces, would be a good idea too. Again, this will improve with experience.

Henry

[Megs Maquito]

hmmmm, shouldn't an array be initialized as:

int [] myArray = {1,2,4}//use{} instead of ()