Java敏捷开发技巧之消除代码异味 - 编程入门网
asswordMaxAgeInDays, boolean allowedToPrintReport) {
this.passwordMaxAgeInDays = passwordMaxAgeInDays;
this.allowedToPrintReport = allowedToPrintReport;
}
int getPasswordMaxAgeInDays() {
return passwordMaxAgeInDays;
}
boolean canPrintReport() {
return allowedToPrintReport;
}
static UserType normalUserType = new UserType(90, true);
static UserType adminUserType = new UserType(30, true);
static UserType guestUserType = new UserType(Integer.MAX_VALUE, false);
}
class InventoryApp {
void login(UserAccount userLoggingIn, String password) {
if (userLoggingIn.checkPassword(password)) {
GregorianCalendar today = new GregorianCalendar();
GregorianCalendar expiryDate = getAccountExpiryDate(userLoggingIn);
if (today.after(expiryDate)) {
//提示用户修改密码
...
}
}
}
GregorianCalendar getAccountExpiryDate(UserAccount account) {
int passwordMaxAgeInDays = getPasswordMaxAgeInDays(account);
GregorianCalendar expiryDate = new GregorianCalendar();
expiryDate.setTime(account.dateOfLastPasswdChange);
expiryDate.add(Calendar.DAY_OF_MONTH, passwordMaxAgeInDays);
return expiryDate;
}
int getPasswordMaxAgeInDays(UserAccount account) {
return account.getType().getPasswordMaxAgeInDays();
}
void printReport(UserAccount currentUser) {
boolean canPrint;
canPrint = currentUser.getType().canPrintReport();
if (!canPrint) {
throw new SecurityException("You have no right");
}
//打印报表.
}
}
Java敏捷开发技巧之消除代码异味(8)时间:2011-04-09注意到了吧!用一个对象代替类别,同样可以移除switch或者if-then-else-if。 总结一下类别代码的移除 要移动一些类别代码和switch表达式,有两种方法: 1、用基于同一父类的不同子类来代替不同的类别。 2、用一个类的不同对象来代替不同的类别。 当不同的类别具有比较多不同的行为时,用第一种方法。当这些类别的行为非常相似,或者只是差别在一些值上面的时候,用第二个方法。 普遍的代码异味 类别代码和switch表达式是比较普遍的代码异味。此外,还有其他的代码异味也很普遍。 下面是大概的异味列表: ◆代码重复 ◆太多的注释 ◆类别代码(type code) ◆switch或者一大串if-then-else-if ◆想给一个变量,方法或者类名取个好名字时,也怎么也取不好 ◆用类似XXXUtil, XXXManager, XXXController 和其他的一些命名 ◆在变量,方法或类名中使用这些单词“And”,“Or”等等 ◆一些实例中的变量有时有用,有时没用 ◆一个方法的代码太多,或者说方法太长 ◆一个类的代码太多,或者说类太长 ◆一个方法有太多参数 ◆两个类都引用了彼此(依赖于彼此) |
凌众科技专业提供服务器租用、服务器托管、企业邮局、虚拟主机等服务,公司网站:http://www.lingzhong.cn 为了给广大客户了解更多的技术信息,本技术文章收集来源于网络,凌众科技尊重文章作者的版权,如果有涉及你的版权有必要删除你的文章,请和我们联系。以上信息与文章正文是不可分割的一部分,如果您要转载本文章,请保留以上信息,谢谢! |