这是太多的重构吗?

时间:2015-07-26 07:40:42

标签: java refactoring

我尝试重构代码,以便它使用单独的方法进行一些计算。只是说清楚。

我想知道的是,编写一个单独的方法来找出一个像数字是奇数甚至 这样的简单的东西是一个好习惯还是坏的?

原始代码是,

    int n = 11;
    if (n % 2 == 0) {
        System.out.println("Not selected");
    } else {
        boolean isPrime = true;
        if (n == 0 || n == 1) {
            isPrime = false;
        } else {
            int i = 2;
            double a = Math.sqrt(Math.abs(n));
            while (i <= a) {
                if (n % i == 0) {
                    isPrime = false;
                }
                ++i;
            }
        } 
        if(isPrime){
            System.out.println("Prime it is");
        }
    }

重构的代码是,

    int n = 11;
    if (isEven(n)) {
        System.out.println("Not selected");
    } else { 
        if (isPrime(n)) {
            System.out.println("Prime it is");
        }
    }

public static boolean isEven(int n) {
    return n % 2 == 0 ? true : false;
}

public static boolean isPrime(int n){
    if(n==0 || n==1)return false;        
    int i=2;
    double a = Math.sqrt(Math.abs(n));
    while(i<=a){
        if(n%i==0){
            return false;
        }            
        ++i;
    }
    return true;
}

2 个答案:

答案 0 :(得分:7)

对于可读性,长度或圈复杂度等问题,将代码分解为单独的方法通常被认为是一种很好的做法,特别是如果您不改变代码的工作方式。

布尔表达式,就像你提取的一样,通常是快速提取函数重构的不错选择。它允许代码库的读者通过能够读取描述性函数名称与复杂域相关的布尔数学来了解布尔表达式为何重要或者它做什么,他们可能不在乎知道错综复杂的细节。

一本关于代码组织中通常被认为是Java最佳实践的好书是一本名为Clean Code的书。这是一个非常简单和愉快的阅读,我会建议。

答案 1 :(得分:3)

这被称为功能分解,(在我看来)总是很好的做法。

不仅更容易阅读,您还可以更轻松地找到该代码中的问题,因为您现在可以轻松地单独测试每个部分。

例如,您现在可以确保isEven实际返回偶数。如果以后出现问题,您知道该方法不是问题。这称为unit testing

我建议切换

if(isEven(n)) {

} else {
    if(isPrime(n)) {

    }
}

if(isEven(n)) {

} else if(isPrime(n)) {

}

由于它提供了相同的功能,但为您节省了一条线。

但是,如果您将0视为偶数,则甚至不需要isPrime;如果它不均匀,它必须是素数。您无需在isPrime中执行所有处理:

if(isEven(n)) {
    System.out.println("Even");
} else {
    System.out.println("Odd");
}

为了让它更清洁:

String result = isEven(n) ? "Even" : "Odd";
System.out.println(result);