Question

This is similar to :

.NET: bool vs enum as a method parameter

but concerns returning a bool from a function in some situations.

e.g.
Function which returns bool :

    public bool Poll()
    {
        bool isFinished = false;

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

        return isFinished;
    }

Used like this :

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

Its not obvious from the calling context what the bool returned from Poll() means.
It might be clearer in some ways if the “Poll” function was renamed “IsFinished()”, but the method does a bit of work, and (IMO) would not really reflect what the function actually does. Names like “IsFinished” also seem more appropriate for properties. Another option might be to rename it to something like : “PollAndReturnIsFinished” but this doesn’t feel right either.

So an option might be to return an enum. e.g :

    public enum Status
    {
        Running,
        Finished
    }  

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

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

        return status;
    }

Called like this :

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

But this feels like overkill.
Any ideas ?

Answer 1

A method should be read like a verb, and the result of the bool Poll() method is misleading, and this is probably why it feels awkward to use.

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

When I first read your code, I thought it said While (the system is) not polling, do something?

But it really says … Poll, and if not finished polling do something while we wait.

Your enum version appears to have changed the semantics of the call, but for the better, which is why people like it. While Poll() is still Running, do something while we wait.

The most readable code wins.