public List<ItemDetail> queryItemDetails(List<Cart> carts){ ItemRepository itemRepository=...; List<ItemDetail> result = List.newArrayList(); List<SkuId> ids = buildItemSkuIds(carts); result = itemReposioty.query(itemSkuIds);//Do some additional things to the result...
return result; }
无意义地的扩大方法的依赖。在本例中, queryItemDetail中其实仅仅用到了carts的中的id属性，却需要调用者传入整个carts列表，无谓地使得该方法所依赖的参数扩大了。同样List<T>这个参数类型也没有太大意义，Iterable<T>很可能就足够了。
此内容由惯性聚合(RSS阅读器)自动聚合整理，仅供阅读参考。 原文来自 — 版权归原作者所有。