java – 这是太多的重构吗?
发布时间:2020-12-15 00:40:58 所属栏目:Java 来源:网络整理
导读:我尝试重构代码,以便它将使用单独的方法来进行一些计算.只是为了说清楚. 我想知道的是,编写一个单独的方法来找出像数字奇数或偶数这样的简单事物,这是一个好习惯还是坏事? 原始代码是, int n = 11; if (n % 2 == 0) { System.out.println("Not selected");
|
我尝试重构代码,以便它将使用单独的方法来进行一些计算.只是为了说清楚.
我想知道的是,编写一个单独的方法来找出像数字奇数或偶数这样的简单事物,这是一个好习惯还是坏事? 原始代码是, 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;
}
解决方法
对于可读性,长度或圈复杂度等问题,将代码分解为单独的方法通常被认为是一种很好的做法,特别是如果您不更改代码的工作方式.
布尔表达式,就像你提取的一样,通常是快速提取函数重构的好选择.它允许代码库的读者通过能够读取描述性函数名称与复杂域相关的布尔数学来了解布尔表达式为何重要或者它做什么,他们可能无需知道错综复杂的细节. 一本关于代码组织中通常被认为是Java最佳实践的好书是一本名为Clean Code的书.这是一个非常简单和愉快的阅读,我建议它. (编辑:李大同) 【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容! |
