加入收藏 | 设为首页 | 会员中心 | 我要投稿 李大同 (https://www.lidatong.com.cn/)- 科技、建站、经验、云计算、5G、大数据,站长网!
当前位置: 首页 > 百科 > 正文

c# – 是否有更可读的方法来制作一长串的\u0026\u0026陈

发布时间:2020-12-16 01:52:20 所属栏目:百科 来源:网络整理
导读:假设我有一个冗长,复杂的条件列表,为了运行if语句,必须为true. if(this == that foo != bar foo != that pins != needles apples != oranges){ DoSomethingInteresting();} 通常情况下,如果我被迫做这样的事情,我会将每个语句放在自己的行上,如下所示: if(
假设我有一个冗长,复杂的条件列表,为了运行if语句,必须为true.

if(this == that && foo != bar && foo != that && pins != needles && apples != oranges)
{
    DoSomethingInteresting();
}

通常情况下,如果我被迫做这样的事情,我会将每个语句放在自己的行上,如下所示:

if
(
         this == that 
    &&    foo != bar 
    &&    foo != that 
    &&   pins != needles 
    && apples != oranges
)
{
    DoSomethingInteresting();
}

但我仍觉得这有点乱.我很想将if语句的内容重构为它自己的属性

if(canDoSomethingInteresting)
{
    DoSomethingInteresting();
}

但是那只是将所有的混乱移动到canDoSomethingInteresting()并没有真正解决问题.

正如我所说,我的goto解决方案是中间解决方案,因为它不会像最后一个那样模糊逻辑,并且比第一个更具可读性.但必须有更好的方法!

响应Sylon评论的示例

bool canDoSomethingInteresting
{
    get{
        //If these were real values,we could be more descriptive ;)
        bool thisIsThat = this == that;
        bool fooIsntBar = foo != bar;
        bool fooIsntThat = foo != that;
        return 
        (
               thisIsThat
            && fooIsntBar
            && fooIsntThat
        );
    }
}
if(canDoSomethingInteresting)
{
    DoSomethingInteresting();
}

解决方法

在我看来,把混乱变成一个属性或一个方法并不是一个坏主意.这样它就是自包含的,你执行if(..)检查的主要逻辑变得更具可读性.特别是如果要检查的条件列表很大,最好是在属性中,这样如果您需要重新使用,那么您不会复制该检查.

if(IsAllowed)
{
   DoSomethingInteresting();
}

(编辑:李大同)

【声明】本站内容均来自网络,其相关言论仅代表作者个人观点,不代表本站立场。若无意侵犯到您的权利,请及时与联系站长删除相关内容!

    推荐文章
      热点阅读