惯性聚合 高效追踪和阅读你感兴趣的博客、新闻、科技资讯
阅读原文 在惯性聚合中打开

推荐订阅源

P
Proofpoint News Feed
U
Unit 42
V
Visual Studio Blog
D
DataBreaches.Net
F
Fortinet All Blogs
让小产品的独立变现更简单 - ezindie.com
让小产品的独立变现更简单 - ezindie.com
The GitHub Blog
The GitHub Blog
Y
Y Combinator Blog
月光博客
月光博客
大猫的无限游戏
大猫的无限游戏
T
The Blog of Author Tim Ferriss
GbyAI
GbyAI
博客园 - 叶小钗
Blog — PlanetScale
Blog — PlanetScale
OSCHINA 社区最新新闻
OSCHINA 社区最新新闻
MongoDB | Blog
MongoDB | Blog
The Cloudflare Blog
云风的 BLOG
云风的 BLOG
D
Docker
G
Google Developers Blog
罗磊的独立博客
博客园 - 三生石上(FineUI控件)
小众软件
小众软件
S
SegmentFault 最新的问题

博客园 - idior

每日代码 - 7/1 减小方法参数的依赖 每日代码 - 6/29 读写分离 每日代码 - 6/26 lambda表达式 每日代码 - 6/27 避免创建非法对象 Resume Covariance and Contravariance How does ElementName Binding work? Weird behavior of DataContext Inheritance How does ElementName Binding work – Part 3 InheritanceContext How does ElementName Binding work – Part 2 BindingExpression Logical Tree & Visual Tree Memory leak caused by EventHandle - weak event Resources on Debugging/Tracing WPF How does ElementName Binding work? - Part 1 Logical Tree & NameScope Inside WCF Runtime 有谁准备参加10月份的MVP聚会? MVP聚会 Practical .NET2 and C#2 翻译样章 Be evil or not?
每日代码 - 6/28 代码逻辑分组
idior · 2013-06-28 · via 博客园 - idior
int officialShopId = shopId;
group.setShopId(officialShopId);
XmpUser xmpSeller = buildXmpUser(seller.getSellerInfo());
group.setSeller(xmpSeller);
List<XmpItem> xmpItems = buildXmpItemList(sellerList,clientName, officialShopId);
group.setItems(xmpItems);

以上代码看上去比较杂乱无章,可以做一个简单分组。

int officialShopId = shopId;
XmpUser xmpSeller = buildXmpUser(seller.getSellerInfo());
List<XmpItem> xmpItems = buildXmpItemList(sellerList,clientName, officialShopId);
                    
group.setShopId(officialShopId);  
group.setSeller(xmpSeller);
group.setItems(xmpItems);

这样就清晰很多,一方面可读性变好了,另一方面会便于进一步的重构,比如抽取小方法。

关于对象设置,C#的语法明显比java简洁多了。

group = new Group() { ShopId = officialShopId, 
                      Seller = xmpSeller,
                      Items = xmpItems
                    };

java下面可能用fluent API是种选择。

group.setShopId(officialShopId)
     .setSeller(xmpSeller)
     .setItems(xmpItems);