我的java while循環沒有返回每週薪酬

Question

好吧，所以第二部分就這一點。我重構了代碼，但現在我無法找到一種方法來返回empWeeklyPay，而無需將其初始化爲零。如果我將它初始化爲零，那麼我的工資總是零？任何人都可以想到我能做的任何事情嗎？

public double getWeeklyPay()//Call the getWeeklyHours method of the Emp's timecard to  get the total 
//hours worked and then multiply the returned value by the emp's hourly rate and return the value. 
{ 
TimeCard empTimeCard = timeCard; 
double empWeeklyPay; 
double totalOtHours; 
double totalRegHours; 
int i = 0; 

while(i <= empTimeCard.NUMDAYS && empTimeCard.getHoursByDay(i) > 8) 
{ 
double empHourlyRate = getHourlyRate(); 
double otHours = 0; 
double regHours = 0; 
double sumOtHours = 0; 
int sumRegHours = 0; 

//grabbing the overtime and reghours and storing them 
otHours = empTimeCard.getHoursByDay(i) % 8; 
regHours = empTimeCard.getHoursByDay(i) - otHours; 
sumOtHours += otHours; 
sumRegHours += regHours; 
double tmpBasePay = (sumRegHours * empHourlyRate) + (sumOtHours * empHourlyRate * 1.5); 

if(Integer.toString(getEmployeeId()).charAt(0) == '0' || Integer.toString(getEmployeeId()).charAt(0) == '2' 
|| Integer.toString(getEmployeeId()).charAt(0) == '9') 
//Java reads in 1010 so need to convert to a string to do a count on the value. 
{ 
    tmpBasePay += (tmpBasePay * .10);   
    i++; 
} 
else if(Integer.toString(getEmployeeId()).charAt(0) == '3') 
{ 
    tmpBasePay -= (tmpBasePay *.10); 
    i++; 
} 
else if(Integer.toString(getEmployeeId()).charAt(0) == '8') 
{ 
    tmpBasePay += (tmpBasePay * .20); 
    i++; 
} 
totalRegHours = regHours; 
totalOtHours = sumOtHours; 
if(totalRegHours > 34) 
{ 
tmpBasePay -= (tmpBasePay * .06); 
//empWeeklyPay = tmpBasePay; 
empWeeklyPay = tmpBasePay; 
} 
else 
{ 
empWeeklyPay = tmpBasePay; 
} 

}

return empWeeklyPay;// <---need to return back this value. 

Answer 1

以下是我的問題的答案。我能弄明白。請參閱下面的代碼以防其他人在近期或遠期將面臨同樣的問題：

public double getWeeklyPay() { 
// Call the getWeeklyHours method of the Emp's timecard to get the total hours worked 
// and then multiply the returned value by the emp's hourly rate and return the value. 

double empWeeklyPay; 
double sumOtHours = 0; 
double sumRegHours = 0; 
int i = 0; 


while(i < getTimeCard().NUMDAYS) { 
// grabbing the overtime and regHours and storing them 

    double otHours = 0; 
    double regHours = 0; 
    otHours += getTimeCard().getHoursByDay(i); 
    regHours += getTimeCard().getHoursByDay(i) - otHours; 
    sumOtHours += otHours; 
    sumRegHours += regHours; 
    i++; 
}  

empWeeklyPay = (sumRegHours * getHourlyRate()) + (sumOtHours * getHourlyRate() * 1.5); 

if(Integer.toString(getEmployeeId()).charAt(0) == '0' || Integer.toString(getEmployeeId()).charAt(0) == '2' 
|| Integer.toString(getEmployeeId()).charAt(0) == '9') { 
// Java reads in 1010 so need to convert to a string to do a count on the value. 

    double tmpBasePay = (sumRegHours * getHourlyRate()) + (sumOtHours * getHourlyRate() * 1.5); 
    tmpBasePay += (tmpBasePay * .10); 
    empWeeklyPay = tmpBasePay; 
} 

else if(Integer.toString(getEmployeeId()).charAt(0) == '3') { 
    double tmpBasePay = (sumRegHours * getHourlyRate()) + (sumOtHours * getHourlyRate() * 1.5); 
    tmpBasePay -= (tmpBasePay * .10); 
    empWeeklyPay = tmpBasePay; 
} 

else if(Integer.toString(getEmployeeId()).charAt(0) == '8') { 
    double tmpBasePay = (sumRegHours * getHourlyRate()) + (sumOtHours * getHourlyRate() * 1.5); 
    tmpBasePay += (tmpBasePay * .20); 
    empWeeklyPay = tmpBasePay; 
} 

if(sumRegHours > 34) { 
    empWeeklyPay -= (empWeeklyPay * .06); 
    return empWeeklyPay; 
} 

else 
    return empWeeklyPay; 
} 

Answer 2

你的第一個while循環，而只能做一個迭代，因爲它總是遇到return關鍵字。 return將使您的while迴路和您的getWeekPay()方法。

你只需要在你的方法結束時返回你的empWeeklyPay變量，所以在這裏只有最後一個返回值是有用的。你的方法的到底應該是這樣的：

if(RegHours > 34) { 
    empBasePay -= (empBasePay * .06); 
    empWeeklyPay = empBasePay; 
} else { 
    empWeeklyPay = empBasePay; 
} 
return empWeeklyPay; 

編輯

爲什麼你總是給我0：

double empBasePay = (RegHours * empHourlyRate) + (otHours * empHourlyRate * 1.5); 

由於RegHours和otHours等於0，你還不如寫：

double empBasePay = 0; 

A因爲該方法的其餘部分基於empBasePay的基數值，即0，所以您的方法將始終返回0.