代码重构实践

1、怎么让你的代码更优雅

public Info getDetail(Integer id){
    Info info = infoService.getInfo(id);
    if(info != null){
        User  user = userService.getUserById(info.getUserId);
        if(user != null){
           info.setPhone(user.getPhone());
        }
        if(user != null){
           Account account = accountService.getBalance(user.getUserId);
           if(account != null){
               info.setBalance(account.getBalance());
           }
        }
    }
    retrun info;    
}

这段代码很常见,功能没什么问题,但是看起来就是不爽。该怎么重构呢?还有列表,如果一个列表里面的很多字段需要从其他服务里面获取字段,该怎么重构更优雅呢。

我能想到的就是卫句了。

public Info getDetail(Integer id){
    Info info = infoService.getInfo(id);
    if(info == null) return info;
    User user = userService.getUserById(info.getUserId);
    if(user == null) return info;
    info.setPhone(user.getPhone());
    Account account = accountService.getBalance(user.getUserId);
    if(account != null){
       info.setBalance(account.getBalance());
    }
    retrun info;    
}
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 打赏
    打赏
  • 0
    评论

“相关推荐”对你有帮助么?

  • 非常没帮助
  • 没帮助
  • 一般
  • 有帮助
  • 非常有帮助
提交
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包

打赏作者

水木江南

你的鼓励将是我创作的最大动力

¥1 ¥2 ¥4 ¥6 ¥10 ¥20
扫码支付:¥1
获取中
扫码支付

您的余额不足,请更换扫码支付或充值

打赏作者

实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值