两个if的实现,重构到一个if的实现

1:下面我们先来看看这个代码:
2:private static void OldMethod(BusinessObjectInfo parentBOInfo)
{
IList bosPropertyInfo = parentBOInfo.BOsPropertyInfos;
if ((bosPropertyInfo.Count == 1) && (null !=
parentBOInfo.TreeChildPropertyInfo))
{
//action one
}
else if (((bosPropertyInfo.Count > 1) && (null !=
parentBOInfo.TreeChildPropertyInfo))
|| ((bosPropertyInfo.Count > 0) && (null ==
parentBOInfo.TreeChildPropertyInfo))
)
{
//action two
}

private static void NewMethod(BusinessObjectInfo parentBOInfo)
{
IList childrenProperties =
parentBOInfo.BOsPropertyInfos;
var childrenPropertiesCount = childrenProperties.Count;
if (childrenPropertiesCount <= 0)
{
return;
}
var hasTreeChild = null != parentBOInfo.TreeChildPropertyInfo;
if (hasTreeChild)
{
if (childrenPropertiesCount == 1)
{
//action one
}

else
{ //action two
}

2:这样从业务角上来说是没错,但是写法还是有问题。下面来修改一下
3:private static void NewMethod2(BusinessObjectInfo parentBOInfo)
{
IList childrenProperties = parentBOInfo.BOsPropertyInfos;
parentBOInfo.BOsPropertyInfos;
var childrenPropertiesCount = childrenProperties.Count;
if (childrenPropertiesCount <= 0)
{
return;
}
var hasTreeChild = null != parentBOInfo.TreeChildPropertyInfo;
if (hasTreeChild && childrenPropertiesCount == 1)
{
//action one

else

//action two
}

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论

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

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

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值