重构笔记2-代码的坏味道

方法名为初始化索引,初始化什么索引呢?

另外 方法过长,一眼看不出来具体要几个step 才能做完初始化索引的动作。

如果让我维护这个method,那么我可能要花一些时间看一下怎样进行index 的初始化的。


结果是看了一段时间还是不知其所以然,准备周一问问写这段代码的同事了。。。


private Map<OrderNode, OrdOutData> initializeIndex(OrderNode root) throws TopologyOutputException {

        Map<OrderNode, OrdOutData> dataMap = Maps.newLinkedHashMap();
        OrderNode complex = getComplex(root);
        {
            OrdOutData ordOutData = createIndex(complex);
            dataMap.put(complex, ordOutData);
        }
        {
            /**
             * Just to print log.
             */
            List<OrderNode> systemNodes = complex.getOptionOf();
            Collections.sort(systemNodes, new OrderNodeComparator());
            for (OrderNode systemNode : systemNodes) {
                createIndex(systemNode);
            }
        }
        List<OrderNode> machineNodes = complex.getMachineNodes();
        dataMap.putAll(processMachineNodes(machineNodes));
        List<OrderNode> list = Lists.newArrayList();
        for (OrderNode machineNode : machineNodes) {
            list.addAll(machineNode.flatten());
        }
        list.removeAll(machineNodes);
        Collections.sort(list, new OrderNodeComparator());
        for (OrderNode node : list) {
            OrdOutData ordOutData = createIndex(node);
            dataMap.put(node, ordOutData);
        }
        return dataMap;
    }
  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

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

余额充值