代码之家 › 专栏 › 技术社区 › Moe Sisko

为清楚起见,是否从函数中返回enum而不是bool?

coding-style c#

Moe Sisko · 技术社区 · 16 年前

.NET: bool vs enum as a method parameter

例如

    public bool Poll()
    {
        bool isFinished = false;

        // do something, then determine if finished or not.

        return isFinished;
    }

用法如下:

        while (!Poll())
        {
            // do stuff during wait.
        }

从调用上下文看,Poll()返回的bool的含义并不明显。如果将“Poll”函数重命名为“IsFinished()”,在某些方面可能会更清楚,但是该方法需要做一些工作,并且(IMO)不会真正反映函数的实际功能。像“IsFinished”这样的名称似乎也更适合于属性。另一个选择可能是将其重命名为类似“PollAndReturnIsFinished”的内容,但这感觉也不对。

因此,可以选择返回枚举。例如:

    public enum Status
    {
        Running,
        Finished
    }  

    public Status Poll()
    {
        Status status = Status.Running;

        // do something, then determine if finished or not.

        return status;
    }

这样称呼:

        while (Poll() == Status.Running)
        {
            // do stuff during wait.
        }

但这感觉有点过分了。有什么想法吗?

5 回复 | 直到 9 年前

Robert Paulson 16 年前

方法应该像动词 ,以及 bool Poll() 方法具有误导性,这可能就是为什么使用它会感到尴尬的原因。

// you wrote.
while( !Poll() )
{
    // still waiting .. do something.
}

当我第一次读到你的代码时,我以为它说 当(系统)不轮询时,做些什么 ?

但它真的说。。。 投票,如果没有完成投票做一些事情,而我们等待。

您的enum版本似乎已经改变了调用的语义,但效果更好,这就是人们喜欢它的原因。

可读性最强的代码获胜。

bniwredyc 16 年前

首先,代码是供人们阅读的,在您的示例中,enum版本比bool版本更具可读性。

编辑:

Error 例如。

adrianm 16 年前

我遵循.Net约定,布尔属性的前缀是“Is”,布尔方法的前缀是“Try”(或“Is”(如果合适)。

在你的情况下,我认为问题出在“民意测验”的名字上。命名方法,说明它正在执行的操作或轮询的目的。例如

leppie 16 年前

enum bool .

编辑:

public bool IsRunning  { get {return Poll() == Running; }}
public bool IsFinished { get {return Poll() == Finished; }}

Syd 16 年前

我一遍又一遍地读你想做的事。对我来说,Poll()应该确切地表示您正在轮询某些内容。然后我会检查状态,如果它仍然在运行。

我的结果代码如下所示。

while (myObject.Poll() && myObject.IsRunning)
{
   // myObject successfully poll it .. 
   // and is successful
   // do more things here... 
}

注意:IsRunning是getter/setter。这会更清楚吗?

推荐文章

Hamza Waheed · 关于C#中初始化构造函数中代码的最佳实践?关闭

2 年前

Manelzzz · 以下哪一个是SignIn的BlocListener中if语句的最佳结构?

2 年前

metrallador10 · 哪种代码更好?效率与代码可读性

2 年前

burhan · CSS:我所有的项目都是一个接一个的,我该如何修复它?

2 年前

Justin Xu · 使用return if语句进行重构验证

3 年前

Cino · 如何以体面的方式处理Python异常?

3 年前

xyaw · 编译错误:对“HTTPClient::begin”的调用声明为属性错误:过时的API,使用::begin(WiFiClient,url)

3 年前

SAI BENDE · 如何在多个html文件中使用单个导航栏

3 年前

fstab · 对正常控制流程使用例外情况是一种不鼓励还是不鼓励的做法?

13 年前

SwampYeti · 在CSS中拉伸小背景图像

13 年前