@Override

public Response checkUserToken(long uid, String accessToken, String deviceToken){

Response response = new Response();

//TODO 先到session中找

try {

UserLogin userLogin = userLoginDao.getUserLoginByUid(uid);

if(userLogin != null) {

if(userLogin.getStatus() !=-1){

if(accessToken == userLogin.getAccessToken() && deviceToken == userLogin.getDeviceToken()){

//验证正确, 生成新的accessToken

String newAccessToken = regenerateAccessToken(uid);

//保存到数据库

String sql = "UPDATE " + UserLoginDao.DEFAULT_TABLE_NAME + " SET " +

"accessToken=?, online=?";

Object[] args = new Object[]{newAccessToken, 1};

int[] argTypes = new int[]{Types.VARCHAR, Types.TINYINT};

userLoginDao.executeUpdate(sql, args, argTypes);

response.setRc(Rc.RC_SUCCESS);

response.setData(new LoginDTO(uid, accessToken, deviceToken));

}else{

response.setRc(Rc.RC_USER_ACCESS_ERROR);

response.setErrMsg("验证失败,请重新登陆");

}

}else{

response.setRc(Rc.RC_USER_STATUS);

response.setErrMsg("账号存在风险,已暂时锁定");

}

}else{

response.setRc(Rc.RC_USER_INVALID);

response.setErrMsg("不合法用户请求");

}

}catch (Exception e){

Yin.logError(e, getClass());

response.setRc(Rc.RC_DB_ERROR);

response.setErrMsg("数据库异常");

}

return response;

}

这是一个业务层方法, 里面我直接try catch捕获了dao层的可能的异常. 并作为一个对象返回.

我的考虑是:这么做的在action层就无需try, catch了,因为统一通过Response返回结果

我看有的人是封装了一个业务层的异常, 返回给action

这两种方法哪个好些?为什么?

另外, 如果是封装业务层的异常,这个按照什么原则分的呢? 比如 前台传个id, 如果这个id没找到,难道我要构造个UserNotFoundException, 而不是在Response对象里加一个status?

小弟学程序至今, 一直觉得自己写的代码很糟糕, 最近开始看代码大全这本书, 觉得受益匪浅, 并重新修改了代码.麻烦各位前辈解惑的同时, 能指点一下如上这段代码. 请指教