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

可以简化为：

public void updateUserInfo(User user) {
    user.setUpdateTime(new Date());
    if (userInfoMapper.updateByPrimaryKeySelective(user) > 0) {
        log.info("修改用户信息成功,用户id：" + user.getUserId());
    } else {
        log.error("修改用户信息失败,用户id：" + user.getUserId());
    }
}

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

相关问题

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);
}

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

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 = null; if(userInfoService.selectCountById(openid)) { user = User.builder().userId(openid).sessionKey(sessionKey).build(); log.info("用户登录并修改用户信息："+JSONObject.toJSONString(user)); userInfoService.updateUserInfo(user); } else { user = User.builder().userId(openid).sessionKey(sessionKey).role(1).build(); 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; }简化代码

下面是简化后的代码：

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);
        }
        
        //生成token并返回
        String token = generateToken(openid, sessionKey);
        //查询用户信息
        User userRole = userInfoService.selectUserByKey(openid);
        //返回token和用户角色
        dto = MiniappDTO.builder().token(token).roleType(userRole.getRole() == 2).msg("登录成功").build();
    } catch (Exception e) {
        log.error("登录失败：" + e.getMessage(), e);
        dto = MiniappDTO.builder().msg("登录失败").build();
    }
    return dto;
}