幾乎在那裏，只需要在這個工資計算中向正確的方向推一點點？

需要幫助來計算兩種不同工資級別的工資！錯誤是淨總數不正確。謝謝所有他們是非常有益的意見和現在的作品:)我希望我的這部分代碼可以幫助其他初學者學習使用循環和if語句：幾乎在那裏，只需要在這個工資計算中向正確的方向推一點點？

public static void main(String[]args) 
{ 
    Scanner input = new Scanner (System.in); 
    DecimalFormat Currency = new DecimalFormat("###,###,##0.00"); 

    double normHrs = 0, overHrs = 0, bonusHrs = 0, actualHrs = 0, normPay = 0, overPay =0, bonusPay = 0, grossWage = 0,vat23 = 0, netWage =0; 
    String empName, nextEMP = "y"; 



    System.out.println ("************************"); 
    System.out.println ("HUMAN RESOURCES: PAYROLL"); // 
    System.out.println ("************************"); 


    while (nextEMP.equalsIgnoreCase("y")) 
    { 

     System.out.print("Please enter an employee's first name: "); 
     empName = input.next(); 
     System.out.print("Please enter their total hours worked: "); 
     actualHrs = input.nextDouble(); 


     if (actualHrs <=36) 
     { 
      normHrs=actualHrs; 

     } 

     else 
     { 

      normHrs = 36; 
      bonusHrs = actualHrs - 50; 
      overHrs = actualHrs -36; 
     } 

     if ("bob".equalsIgnoreCase(empName)) 
     { 
      bonusPay = bonusHrs > 0 ? bonusHrs : 0; 
      normPay = normHrs * 8; // £8 x 36 hours 
      overPay = overHrs * 12; // £12 x 36+ hours 

     } 

     else 
     { 
      bonusPay = bonusHrs >0 ? 20 : 0; 
      normPay = normHrs * 5.75; 
      overPay = overHrs * 9.25; 

     } 


     }

來源

2014-02-24 user3058983

我會穿越後這https://codereview.stackexchange.com/以獲得更好的佈局驗證碼出來，使之更具可讀性過的一些技巧。 –

看起來該熟悉使用調試器了。 –

我只是一個初學者，並遵循導師設置的指示。我們必須在提高代碼效率之前保持基礎知識，但是要感謝您的提示:) – user3058983

我認爲錯誤是在部分評估鮑勃是否應該獲得獎金。

既然你已經計算bonusHrs爲actualHrs - 50，這樣的：

bonusPay = bonusHrs > 50 ? bonusHrs : 0;

只會給鮑勃獎金工資只有當actualHrs是100+。這也許應該是：

bonusPay = bonusHrs > 0 ? bonusHrs : 0;

而且，這樣的：

normPay = actualHrs * 8; // £8 x 36 hours

可能是錯誤的，也許應該是：

normPay = normHrs * 8; // £8 x 36 hours

來源

2014-02-24 09:56:06 jpw

在if塊你有一個錯誤的分配normPay時：

if ("bob".equalsIgnoreCase(empName)) { 
      bonusPay = bonusHrs > 50 ? bonusHrs : 0; 
      normPay = actualHrs * 8; // £8 x 36 hours 
      overPay = overHrs * 12; // £12 x 36+ hours 
}

它必須是

normPay = normHrs * 8;

來源

2014-02-24 10:15:15 Boris

37小時bob的當前GROSS結果是308英鎊。這是錯誤的，它應該是36x8 + 1x12 = 300英鎊。在上面的錯誤修復之後，結果是300英鎊。 – Boris

幾乎在那裏，只需要在這個工資計算中向正確的方向推一點點？

回答

相關問題