清理java代码、多个if语句_Java_Code Cleanup

清理java代码、多个if语句

java

清理java代码、多个if语句,java,code-cleanup,Java,Code Cleanup,我应该清理一个java代码，去掉很多东西，但是是否应该清理其他东西，也许以某种方式去掉多个if语句，而不完全重写此代码？似乎无法理解，因为它们是如此不同，以至于可以将它们叠加在一个“如果”中。有什么想法吗 public class Calc { // employee types public static final int SELLER; public static final int COOK; pu

我应该清理一个java代码，去掉很多东西，但是是否应该清理其他东西，也许以某种方式去掉多个if语句，而不完全重写此代码？似乎无法理解，因为它们是如此不同，以至于可以将它们叠加在一个“如果”中。有什么想法吗

 public class Calc {
           // employee types
           public static final int SELLER;
           public static final int COOK;
           public static final int CHIEF;



       public static void main(final String[] args) {
              Calc c = new Calc();
              System.err.println(c.pay(CHIEF) + " should be 66");
       }

       private int pay(final int type, final int h) {
              int Sum = 0;
              if (type == SELLER) {
                     if (h > 8) { 
                           Sum = 20 * (h - 8);  
                           Sum += 80;
                     } else {
                           Sum += 10 * h;
                     }
              }
              if (type == COOK) {
                     if (h > 8) { 
                           Sum = 30 * (h - 8); 
                           Sum += 15 * 8;
                     } else {
                           Sum += 15 * h;
                     }
              }
              if (type == CHIEF) {
                     if (h > 8) { 
                           Sum = 66 * (h - 8); 
                           Sum += 22 * 8;
                     } else {
                           Sum += 22 * h;
                     }
              }
              if (h > 20) {
                     if (type == SELLER) {
                           Sum += 10;
                     }
                     if (type == COOK) {
                           Sum += 20;
                     }
                     if (type == CHIEF) {
                           Sum += 30;
                     }
              }
              return Sum;
       }
}

您编写的代码纯粹是过程性的，在大多数情况下，在使用面向对象语言（如Java）编写代码时被认为是一种不好的做法。您应该了解多态性的威力，不应该手动执行类型检查：

if (type == COOK) { //Avoid doing this in OO languages!

您应该将域实体（员工）视为对象，每个特定类型的员工都可以定义自己的薪酬计算规则

让我们使用单个抽象方法创建一个抽象类Employee

intCalculatePay（inth）

：

单词abstract表示该方法没有实际实现，但计算工资的所有逻辑将放在子类Seller、Cook和Chief中：

public class Cook extends Employee {

    public Cook() {}

    int calculatePay(int h) {
        int sum = (h > 20) ? 20 : 0;
        if (h > 8) {
            sum = 30 * (h - 8);
            sum += 15 * 8;
        } else {
            sum += 15 * h;
        }
        return sum;
    }
}

请注意这一行：

    int sum = (h > 20) ? 20 : 0;

这是三元运算符。有时，它适用于表达式中的条件赋值。因此，当

大于20时，我们用20初始化

sum

变量，否则用0初始化。现在我们在方法的末尾不使用额外的

if

语句

现在，每个员工都负责自己计算工资，无需在PayCalculator类中执行类型检查-它在运行时根据参数类型动态解析要执行的代码：

public class PayCalculator {

    int pay(Employee e, int hours) {
        return e.calculatePay(hours);
    }

    public static void main(String[] args) {
        Seller seller = new Seller();
        Cook cook = new Cook();
        Chief chief = new Chief();

        PayCalculator calc = new PayCalculator();

        System.out.println("Seller is payed " + calc.pay(seller, 15));
        System.out.println("Cook is payed " + calc.pay(cook, 10));
        System.out.println("Chief is payed " + calc.pay(chief, 22));
    }
}

这被称为多病态。如果这个术语对您来说是新术语，您可以阅读OOP基础中的Oracle教程：

Bruce Eckel的《Java中的思考》一书也很好地解释了OOP的基本概念。

您编写的代码纯粹是过程性的，在大多数情况下，在使用Java等面向对象语言编写时被认为是一种不好的做法。您应该了解多态性的威力，不应该手动执行类型检查：

if (type == COOK) { //Avoid doing this in OO languages!