我检讨我们的承包商之一的一些代码:我对这段代码的理解是否正确?
if (userLists != null)
{
Int32 numberOfItems = userLists.Count;
if ((numberOfItems & 1) == 1)
{
var emptyList = new tblList();
userLists.Add(emptyList);
}
}
现在,我想明白这一点,所以,可以有人请我确认如果我有这个正确与否?
- 我们有一个userList的实例吗?
- 是的。获取userLists对象中的项目数量。
- if(项数等于1(是/ true)AND 1等于1(yes/True))并且结果等于1(是/ True),然后向列表中添加一个空列表对象。
- 其他:没什么。
如果是这样(WTFFFFFFFFFFFFFFFFFFFF !!!!!!!!!!),可重构为
if (numberOfItems == 1)
{
..
}
但即使这是废话,因为我不想用“空列表'项目。
所以我读了这段代码是否正确?
哦,在使用Int32
VS Int
一个更感叹:((但我离题)。
这是一个例子,其中一个扩展或辅助方法或解释变种会帮忙与代码可读性。例如,代替(numberOfItems&1)== 1:numberOfItems.IsOdd()或IsOdd(numberOfItems)或isOdd =(numberOfItems&1)== 1; – 2009-05-06 05:50:05
完全同意三宝! JP回复了一个很好的扩展方法:) – 2009-05-06 05:53:45