[JAVA]请JAVA专家帮忙看看，下面代码哪种编写方式更好。

[JiekWang]

方法一、使用运算符

//话单计费方式配置价格为0免费
final boolean isBillWayFree = null != mainChargeWay
&& mainChargeWay.isBillChargeWay())
&& mainChargeWay.isFreeChargeWay();

//免费情况处理
if (isBillWayFree)
{
//获取日志并打印
final String message = LogResUtils.getChargeRes("Charge.free.zeroprice");
LOG.info(message, MODULE);

//设置免费返回结果
return CHARGESUCCESS;
}


另一种方式：使用If - else 同时提取方法。
//判断当配置为话单计费时0元是否免费
if (isBillWayFree(mainChargeWay))
{
//获取日志并打印
final String message = LogResUtils.getChargeRes("Charge.free.zeroprice");
LOG.info(message, MODULE);

//设置免费返回结果
return CHARGESUCCESS;
}

/**
* 判断当配置为话单计费时0元是否免费
* @param mainChargeWay 具体的扣费方式
* @return boolean true：免费，false：不免费
* @see [类、类#方法、类#成员]
*/
private static boolean isBillWayFree(final IChargeWay mainChargeWay)
{
// 如果扣费方式为空，则返回不免费
if (null == mainChargeWay)
{
return false;
}

// 如果是话单扣费，则判断0元的价格是否写话单 1:写话单 0：不写话单
if (mainChargeWay.isBillChargeWay())
{
// 当且仅当配置为0元的价格不写话单时，返回true：表示免费
return mainChargeWay.isFreeChargeWay();
}

// 其他返回不免费
return false;
}


请大家发表一下意见，哪种方式更好或者其他什么方式更好。

[stu202060510]

看这段代码好吃力。。。。。。

[hjh811]

非专家。。。不发表意见。。

[whlcy]

mark！！！

-_-

[yijiulove]

标志一下 学习开心

[SambaGao]

我看你还是先把代码重构一下吧。。。。

[goosman]

如果是不需要后期扩充, 直接写
if(null != mainChargeWay
&& mainChargeWay.isBillChargeWay()
&& mainChargeWay.isFreeChargeWay()) {
//处理代码
}
这样就好了.

如果后期可能需要扩充, 我觉得第二种好.
if (isBillWayFree(mainChargeWay)) , 将免费的判断封装起来, 任何需要判断免费情况的地方都可以直接调用, 另外一方面, 如果对免费的情况定义改变了, 修改也就只有这一个地方.


以上纯属个人意见, 不是专家, 理解错误请指正

[JavaAlpha]

另一种方式： 这种方式我喜欢 官方的也是这种，比较正规。

[ivfangwang_long]

学习了~~~~~~~~~~~~~~~~~~~~~

[reve123456]

第二种方式不伦不类，IChargeWay 里已经有isBillChargeWay和isFreeChargeWay了，再去弄一个static的isBillWayFree(IChargeWay)。

要么IChargeWay只管自己是哪种付费方式，但不管是不是免费还是什么（比方说对某种付费方式免费不免费的规定常变，所以有必要单独用static的函数），那么isBillWayFree就不应该出现在IChargeWay里。要么IChargeWay懂自己是不是该免费，那么isBillWayFree就在IChargeWay里，由它全权负责（就是第一种方式）。