为什么我不能将扩展方法的返回值与布尔值进行比较？

Question

我尝试创建一个扩展方法来测试流读取器是否可以读取数据并返回一个布尔值。

我的扩展方法代码：

 using System.IO;

    namespace ExtensionMethods
    {
        public static class StreamReaderExtension
        {
            public static object isReady(this StreamReader sr)
            {
                if ((sr.ReadLine()) != null)
                {
                    return true;
                } else
                {
                    return false;
                }
            }
        }

    }

我想使用如下图：

StreamReader sr = new StreamReader(tcpClient.GetStream(), Encoding.ASCII);
if (sr.isReady() == true)
{
//Do something
}

但 Visual Studio 告诉我此错误消息：运算符“==”不能应用于“对象”和“布尔”类型的操作数

我错过了什么？

抱歉英语不好。

Answer 1

你的方法很危险——它有副作用，你丢弃了完整的第一行数据。

另外，你的方法绝对是多余的，不需要，就这么简单：

var sr = new StreamReader(tcpClient.GetStream(), Encoding.ASCII);
if (sr.Peek() != -1)
{
    //Do something
}

阅读此Peek()

Answer 2

这里有一些问题。首先，您的方法返回类型object，您无法直接将其与类型bool 进行比较。就编译器“知道”而言，绝对没有理由相信比较是明智的 - 据它所知，您可能正在做类似 "bird" == true 的事情，这是没有意义的。

其次，即使编译后，您的方法也不会像您认为的那样做：

        public static object isReady(this StreamReader sr)
        {
            if ((sr.ReadLine()) != null)
            {
                return true;
            } else
            {
                return false;
            }
        }

首先，在风格上，如果你有一个布尔表达式，为什么不直接“直接”返回呢？这恰好在 (sr.ReadLine()) != null 为 true 时返回 true，而在表达式为 false 时返回 false。

其次，正如 cmets 中所指出的那样，这会“消耗”一行而不实际将其存储在任何地方，因此您实际上会“丢失”数据。 （您只需丢弃该行）。

第三，这个方法的意义何在？您实际上所做的只是单行比较，因此根本没有理由使用这种方法。

最后，用这一行：

if (sr.isReady() == true)

没有理由明确比较true 和false。你可以写

if (sr.isReady())

（尽管正如已经指出的那样，该方法首先是毫无意义的）。

Answer 3

您将isReady() 函数定义为返回一个对象：

public static object isReady(this StreamReader sr)

让它返回一个布尔值：

public static bool isReady(this StreamReader sr)