public WIFIInfo selectCode(String id) { log.info("查询wifi信息，查询id：" + id); WIFIInfo wifiInfo = WIFIInfo.builder().id(id).build(); try{ wifiInfo = wifiInfoMapper.selectOne(wifiInfo); log.info("查询wifi信息:" + JSONObject.toJSONString(wifiInfo)); } catch (Exception e){ log.error("查询wifi信息出错",e.getMessage()); } return wifiInfo; }简化代码

可以简化为：

public WIFIInfo selectCode(String id) {
    log.info("查询wifi信息，查询id：" + id);
    WIFIInfo wifiInfo = wifiInfoMapper.selectByPrimaryKey(id);
    log.info("查询wifi信息:" + JSONObject.toJSONString(wifiInfo));
    return wifiInfo;
}

这里将查询方法改为了直接使用 `selectByPrimaryKey` 方法，去掉了无用的异常处理。

相关问题

public boolean selectCountById(String id) { int count = 0; WIFIInfo wifiInfo = WIFIInfo.builder().id(id).build(); try{ count = wifiInfoMapper.selectCount(wifiInfo); log.info("通过id："+id+"查询是否存在对应wifi，结果数="+count); } catch (Exception e){ log.error("通过id查询是否存在对应wifi信息出错",e.getMessage()); } return count > 0 ? true : false; }简化代码

可以简化为：

public boolean selectCountById(String id) {
    WIFIInfo wifiInfo = WIFIInfo.builder().id(id).build();
    int count = wifiInfoMapper.selectCount(wifiInfo);
    log.info("通过id："+id+"查询是否存在对应wifi，结果数="+count);
    return count > 0;
}

这里同样去掉了无用的三目运算符和异常处理。

public void insertOrUpdate(WIFIInfo wifiInfo) { log.info("创建wifi账号密码,账号密码信息:"+ JSONObject.toJSONString(wifiInfo)); //查询是否已存在对应的wifi信息 boolean isHave = selectCountById(wifiInfo.getId()); wifiInfo.setUpdateTime(new Date()); //如果已存在wifi数据则修改,不存在则添加 int ins = 0; try { if(isHave){ ins = wifiInfoMapper.updateByPrimaryKeySelective(wifiInfo); if (ins > 0) { log.info("修改wifi账号成功"); } } else { wifiInfo.setCreateTime(new Date()); ins = wifiInfoMapper.insert(wifiInfo); if (ins > 0) { log.info("创建wifi账号成功"); } } } catch (Exception e) { log.error(isHave ? "wifi账号修改失败" : "wifi账号创建失败" + ",报错信息："+ e.getMessage()); } }简化代码

可以简化为：

public void insertOrUpdate(WIFIInfo wifiInfo) {
    log.info("创建wifi账号密码,账号密码信息:"+ JSONObject.toJSONString(wifiInfo));
    wifiInfo.setUpdateTime(new Date());
    boolean isHave = selectCountById(wifiInfo.getId());
    int result = 0;
    try {
        if (isHave) {
            result = wifiInfoMapper.updateByPrimaryKeySelective(wifiInfo);
            log.info(result > 0 ? "修改wifi账号成功" : "wifi账号修改失败");
        } else {
            wifiInfo.setCreateTime(new Date());
            result = wifiInfoMapper.insert(wifiInfo);
            log.info(result > 0 ? "创建wifi账号成功" : "wifi账号创建失败");
        }
    } catch (Exception e) {
        log.error((isHave ? "wifi账号修改失败" : "wifi账号创建失败") + ",报错信息：" + e.getMessage());
    }
}

去掉了无用的局部变量，将修改和创建的判断改为使用返回值来判断。