代码review发现的难以阅读的代码-蒲公英云

代码review发现的难以阅读的代码

一、前言

现在还有人写运行有空指针的代码吗？今天review了一个组员写的程序，一看就是头大。现在复盘分析一下。

代码片段如下：

List<MSMallDTO> msMallDTOS = BeanUtil.copyToList(mSubMallDTOList, MSMallDTO.class);
        List<MSMallDTO> mallDTOS = msMallDTOS.stream().map(msMallDTO -> {
            List<MSubMall> mSubMallList = getSMallByGroupName(msMallDTO.getGroupName());
            if (mSubMallList != null && mSubMallList.size() > 0){
                List<MsubMallVo> mallVoList = mSubMallList.stream().map(mSubMall -> {
                            MsubMallVo mallVo = new MsubMallVo();
                            BeanUtil.copyProperties(mSubMall, mallVo);
                            if (relIds != null && relIds.size() > 0){
                                if (relIds.contains(mSubMall.getId())) {
                                    mallVo.setCollect(true);
                                }
                            }
                            return mallVo;
                        }).sorted(Comparator.comparing(MsubMallVo::getAppSort))
                        .collect(Collectors.toList());
                msMallDTO.setAppList(mallVoList);
            }
            return msMallDTO;
        }).sorted(Comparator.comparing(MSMallDTO::getSort)).collect(Collectors.toList());
        redisUtils.set(RedisKeys.getSubMallByUserId(userId,EnumType.CollectionType.SUB_MALL.getValue()),
                JSON.toJSONString(mallDTOS),
                RedisConstants.EXPIRE_TIME);
        return mallDTOS;

二、代码分析

第一步： List msMallDTOS = BeanUtil.copyToList(mSubMallDTOList, MSMallDTO.class);

将 mSubMallDTOList复制到MSMallDTO中返回对象msMallDTOS

第二步：中间那一堆匿名函数实在难以读懂，先一行行看、

List mSubMallList = getSMallByGroupName(msMallDTO.getGroupName());

根据groupName查询mSubMallList列表

然后是判断 mSubMallList 是否为空 if (mSubMallList != null && mSubMallList.size() > 0)

感觉这里有点画蛇添足 mSubMallList != null && mSubMallList.size() > 0

直接用 null != mSubMallList不就行了吗？

第三步，接下来又是一个匿名函数，看着头大

List<MsubMallVo> mallVoList = mSubMallList.stream().map(mSubMall -> {
                                MsubMallVo mallVo = new MsubMallVo();
                                BeanUtil.copyProperties(mSubMall, mallVo);
                                mallVo.setCollect(false);
                                return mallVo;
                            }).sorted(Comparator.comparing(MsubMallVo::getAppSort))
                            .collect(Collectors.toList());
                    msMallDTO.setAppList(mallVoList);