public void saveUserInfo(User user) { log.info("保存的用户信息:"+ JSONObject.toJSONString(user)); int ins = 0; try { user.setUpdateTime(new Date()); user.setCreateTime(new Date()); ins = userInfoMapper.insert(user); if (ins > 0) { log.info("保存用户信息成功"); } } catch (Exception e) { log.error("保存用户信息失败" + ",报错信息："+ e.getMessage()); } }简化代码

可以简化为：

public void saveUserInfo(User user) {
    user.setUpdateTime(new Date());
    user.setCreateTime(new Date());
    if (userInfoMapper.insert(user) > 0) {
        log.info("保存用户信息成功");
    } else {
        log.error("保存用户信息失败");
    }
}

这里去掉了无用的日志输出和异常处理，同时将插入操作的判断改为直接判断返回值即可。

相关问题

@Override public void saveUserInfo(User user, UserAccount account) { log.info("保存的用户信息:"+ JSONObject.toJSONString(user)); if (userInfoMapper.insert(user) > 0) { userAccountMapper.insert(account);//保存用户密码 log.info("保存用户信息成功"); } else { log.error("保存用户信息失败"); } }

这段代码是用 Java 语言编写的，它的功能是保存用户信息和账户信息到数据库中。首先，这个方法接受两个参数，一个是用户信息对象，一个是用户账户信息对象。然后，它使用日志记录保存的用户信息。接着，它通过 userInfoMapper.insert() 方法将用户信息插入到数据库中。如果插入成功，那么就使用 userAccountMapper.insert() 方法将用户密码插入到数据库中，并使用日志记录保存用户信息成功。如果插入失败，那么就使用日志记录保存用户信息失败。

public MiniappDTO getMiniappToken(String code) { MiniappDTO dto = null; try { //调用微信小程序接口，得到openid和sessionKey WxMaJscode2SessionResult result = getWxSessionInfo(code,"userMini"); String openid = result.getOpenid(); String sessionKey = result.getSessionKey(); log.info("openid："+openid + "，sessionKey："+sessionKey); //保存用户信息 User user = User.builder().userId(openid).sessionKey(sessionKey).build(); if(userInfoService.selectCountById(openid)) { log.info("用户登录并修改用户信息："+JSONObject.toJSONString(user)); userInfoService.updateUserInfo(user); } else { user.setRole(1); log.info("用户登录并保存用户信息："+JSONObject.toJSONString(user)); userInfoService.saveUserInfo(user); } // 将 openid 和 sessionKey 存入自定义的登录态 token 中并返回 String token = generateToken(openid, sessionKey); //查询用户信息 User userRole = userInfoService.selectUserByKey(openid); //返回token和用户角色 dto = MiniappDTO.builder() .roleType(2 == userRole.getRole() ? true : false) .token(token).msg("登录成功").build(); } catch (Exception e) { log.error("登录失败：" + e.getMessage(), e); dto = MiniappDTO.builder().msg("登录失败").build(); } return dto; }优化代码

以下是对代码的一些优化建议：

1. 避免过多的 try-catch 块，可以将方法的异常抛出，让调用者处理异常。
2. 避免代码重复，比如多次使用 log.info() 方法记录日志，可以将其提取成一个独立的方法，减少冗余代码。
3. 使用 Java 8 的 Optional 类型，避免返回值为 null 的情况，提高代码的可读性和健壮性。
4. 如果使用了 lombok，可以使用 @AllArgsConstructor 注解替换掉构造方法中的代码，简化代码。
5. 尽量使用接口而不是具体的实现类，提高代码的可扩展性。

根据上述建议，修改后的代码如下所示：

public MiniappDTO getMiniappToken(String code) throws Exception {
    WxMaJscode2SessionResult result = getWxSessionInfo(code,"userMini");
    String openid = result.getOpenid();
    String sessionKey = result.getSessionKey();
    logInfo(openid, sessionKey);

    User user = User.builder().userId(openid).sessionKey(sessionKey).build();
    if(userInfoService.selectCountById(openid)) {
        logInfo("用户登录并修改用户信息：" + JSONObject.toJSONString(user));
        userInfoService.updateUserInfo(user);
    } else {
        user.setRole(1);
        logInfo("用户登录并保存用户信息：" + JSONObject.toJSONString(user));
        userInfoService.saveUserInfo(user);
    }

    String token = generateToken(openid, sessionKey);
    User userRole = userInfoService.selectUserByKey(openid);
    boolean isAdmin = userRole.getRole() == 2;
    return Optional.ofNullable(MiniappDTO.builder()
        .roleType(isAdmin)
        .token(token)
        .msg("登录成功")
        .build())
        .orElseThrow(() -> new Exception("Failed to create MiniappDTO"));
}

private void logInfo(String openid, String sessionKey) {
    log.info("openid：" + openid + "，sessionKey：" + sessionKey);
}

这样修改的代码，可读性更好，也更易于维护和扩展。