典型错误例举 1

/**
* 这个类提供通用的一些方法
*/

public class CommonUntil {
private static final Log LOG = LogFactory.getLog(UserName.class);

// 返回组织的默认办理人列表,从最底层组织开始找,没有的话再找父亲组织,类推
String defaultUserID = “”;
public String getOrgUnitDefaultUserID(String orgUnitID, String userID) {
OrgUnitQuery ouq = new OrgUnitQuery();
OrgUnit orgUnit = null;
try {
orgUnit = ouq.getOrgUnit(orgUnitID, SystemIdentify.LW_OA);
OrgUnitSystem orgUnitSystem = null;
try {
orgUnitSystem = OrgUnitHelper.getOrgUnitSystem(orgUnit.getOrgUnitSystemList());
Assert.assertNotNull(orgUnitSystem);
if(defaultUserID==null||defaultUserID.equals(“”)){
if (orgUnitSystem.getDefaultIds() != null) {
List getDefaultIds = orgUnitSystem.getDefaultIds();
// 开始判断当前用户是否是部门的办理人,只有是部门的办理人才返回部门,否则返回为空
if (getDefaultIds != null) {
defaultUserID = StringBase.listToString(getDefaultIds,” “);
} else {
defaultUserID = “”;
}
} else {
// 如果有父亲,接着找
if (orgUnitSystem.getSuperiorOrgid() != null&& !orgUnitSystem.getSuperiorOrgid().equals(“”)) {
this.getOrgUnitDefaultUserID(orgUnitSystem.getSuperiorOrgid(),userID);
}
}
}
} catch (ResourceException e) {
LOG.error(e.getMessage(), e);
}
} catch (NoResourceException nre) {
System.out.println(“获取资源错误:” + nre);
}
return defaultUserID;
}
}

1.类名CommonUntil,这是什么意思?谁能看到这个单词,就能大概知道这个类是做什么的?
2.类的注释没有写清楚类的职责
3.getOrgUnitDefaultUserID 方法名,个人以为 getDefaultUserIDOfOrgUnit会更合适
4.getOrgUnitDefaultUserID的userID参数是多余的
5.orgUnit,orgUnitSystem 变量的声明没有遵守变量作用域最小化原则
6.List getDefaultIds = orgUnitSystem.getDefaultIds(); 变量是名词,方法才是动词,getDefaultIds变量的命名错误
7.多余的判断
if (orgUnitSystem.getDefaultIds() != null) { ———————————-A
List getDefaultIds = orgUnitSystem.getDefaultIds();
// 开始判断当前用户是否是部门的办理人,只有是部门的办理人才返回部门,否则返回为空
if (getDefaultIds != null) { ———————————-B

A语句已经进行了判断,B语句根本不需要判断

8. 异常需要记录或者被抛出,不是打印到控制台了事

} catch (NoResourceException nre) {
System.out.println(“获取资源错误:” + nre);
}
9.getOrgUnitDefaultUserID方法的注释不但没有写清楚,至少没有按照格式编写(参数、返回值等),
而且所在的行也错误。应该紧挨getOrgUnitDefaultUserID方法上方。

发表评论

电子邮件地址不会被公开。 必填项已用*标注

This site uses Akismet to reduce spam. Learn how your comment data is processed.