C# 表达式和循环之美

Question

这可能相当主观，但是当控制变量在循环中更新时，人们通常如何在 C# 中布置循环控制？我这个学究气的人不喜欢单独的声明和重复。例如。

string line = reader.ReadLine();
while (line != null)
{
    //do something with line

    line = reader.ReadLine();
}

我的 C 编码员想将其更改为

while (string line = reader.ReadLine() != null)
{
    //do something with line
}

，但 C# 的表达式似乎不能那样工作:(

Answer 1

选项：

1) 将变量声明为循环，但在条件中对其进行赋值：

string line;
while ((line = reader.ReadLine()) != null)
{
}

2) 使用 for 循环：

for (string line = reader.ReadLine(); line != null; line = reader.ReadLine())
{
}

3) 创建一个扩展方法，将读取器转换为 IEnumerable，然后使用：

foreach (string line in reader.ReadLines())
{
}

就我个人而言，我喜欢尽可能使用最后一种形式 - 否则我会使用第一种形式。

Answer 2

您不能在表达式内声明变量。

您可以写

string line;
while (line = reader.ReadLine() != null)

为了更清楚，我更喜欢写

string line;
while (null != (line = reader.ReadLine()))

然而，最好的选择是

foreach(string line in File.ReadLines(path) {

}

这将执行等效的操作。
如果您正在读取其他流，则可以创建一个扩展方法，该方法使用以前的语法来启用 foreach 循环。

Answer 3

就我个人而言，我更喜欢：

string line;
while (line = reader.ReadLine() != null)
{
  ...
}

尽管总是有 for 结构作为替代：

for (string line = reader.ReadLine(); line != null; line = reader.ReadLine())
{
  ...
}

Answer 4

我通常会写这样的内容：

string line;
while((line = reader.ReadLine()) != null)
{
}

Answer 5

我同意重复 reader.ReadLine() 表达式不好。一种方法是使用 while(true) 和 break：

while(true)
{
  string line = reader.ReadLine();
  if(line == null)
    break;

  //do something with line
}

Answer 6

怎么样：

public static IEnumerable<R> ToSequence<T, R>(this T self, 
    Func<T, R> yielder, Func<R, bool> condition)
{
    R result = yielder(self, R);
    while (condition(result))
    {
        yield return result;
    }
}

然后像这样使用它：

foreach (var line in reader.ToSequence(
    (r) => r.ReadLine(), 
    (line) => line != null))
{
    // do some stuff to line
}

或者你更喜欢：

foreach (var line in reader.NotNull(r => r.ReadLine()))
{
    // do some stuff to line...
}

可以定义为：

public static IEnumerable<R> NotNull<T, R>(this T self, Func<T, R> yielder)
{
    return self.ToSequence(yielder, (r) => r != null);
}