为什么这个 StreamReader 只读取一行？

Question

下面的代码应该读取文件的每一行并对其进行操作。但是，它只读取第一行。如果没有 for 循环，它会读取整个文件。老实说，我不知道为什么它没有阅读全文。

StreamReader sr = new StreamReader(gridPath);

string line;
char[] lineCh;
char current;
int x, y;
bool north, east, south, west;

x = y = 0;

while ((line = sr.ReadLine()) != null)
{
    lineCh = line.ToCharArray();
    for (int i = 0; i < lineCh.Length; i++)
    {
        current = lineCh[i];
        north = CheckInput(current);
        current = lineCh[++i];
        east = CheckInput(current);
        current = lineCh[++i];
        south = CheckInput(current);
        current = lineCh[++i];
        west = CheckInput(current);
        i++; // Hop over space
        grid[x, y] = new GridSquare(north, east, south, west);
        x++; // Start next column
    }
    Console.WriteLine(line);
    y++;
}

如果没有 for 循环，以下内容将起作用并打印整个文件：

StreamReader sr = new StreamReader(gridPath);

string line;
char[] lineCh;
char current;
int x, y;
bool north, east, south, west;

x = y = 0;

while ((line = sr.ReadLine()) != null)
{
    lineCh = line.ToCharArray();

    Console.WriteLine(line);
    y++;
}

sr.Close();     

CheckInput 如下：

private bool CheckInput(char c)
{
    switch (c)
    {
        case 'y':
            return true;
        case 'n':
            return false;
        default:
            return true;
    }
}

示例输入文件：

nyyn nyyy nyyy nyyy nyyy nnyy
yyyn yyyy yyyy yyyy yyyy ynny
yyyn yyyy yyyy yyyy ynyy nnnn
yyyn yyyy yyyy yyyy ynyy nnnn
yyyn yyyy yyyy yyyy yyyy nnyy
yynn yyny yyny yyny yyny ynny

Answer 1

你在 for 循环中遇到异常吗？您正在递增 i，也许在某些时候您试图错误地索引 lineCh。

编辑：索引错误的另一个候选者是grid数组。我没有看到初始化代码，x 和 y 的值是在读取文件后确定的。你如何初始化它？

Answer 2

您正在修改循环体内的循环控制变量，这是您应该避免的事情，因为它会导致循环的意外执行。

请显示您正在尝试处理的行的示例，我也许可以建议更好地实现您的 for 循环。

您是否需要立即处理整行，还是需要将其分成 4 个字符的块，处理这 4 个字符，然后移至下一行？

您可以尝试更改处理该行的方式：

        while ((line = sr.ReadLine()) != null)
        {
            string[] segments = line.Split(' ');

            foreach(string segment in segments)
            {
                char[] arr = segment.ToCharArray();
                north = CheckInput(arr[0]);
                east = CheckInput(arr[1]);
                west = CheckInput(arr[2]);
                south = CheckInput(arr[3]);
                grid[x, y] = new GridSquare(north, east, south, west);
            }


            Console.WriteLine(line);
            y++;
        }

这里我根据空格拆分该行，然后我可以通过拆分为字符数组并访问特定字符来对各个段进行操作。

这段代码还假设每个段总是有 4 个字符，情况总是如此吗？您还应该添加验证以确保该行符合您的预期。

Answer 3

您的代码会引发异常，因为您可以在任何这些行处超出数组范围：

current = lineCh[++i];

Answer 4

我认为你的问题可能是......

for (int i = 0; i < lineCh.Length; i++)

结合许多++i语句。

这是带有大量注释的代码...假设每一行都是“1234”。

        StreamReader sr = new StreamReader(gridPath);

        string line;
        char[] lineCh;
        char current;
        int x, y;
        bool north, east, south, west;

        x = y = 0;

        while ((line = sr.ReadLine()) != null)
        // line is "yyyy"
        {
            lineCh = line.ToCharArray();
            // lineCh.Length is 4
            for (int i = 0; i < lineCh.Length; i++)
            {
                current = lineCh[i]; // i is zero
                north = CheckInput(current);
                current = lineCh[++i]; // i is 1
                east = CheckInput(current);
                current = lineCh[++i]; // i is 2
                south = CheckInput(current);
                current = lineCh[++i];  // i is 3
                west = CheckInput(current);
                i++; // Hop over space // i is 4
                grid[x, y] = new GridSquare(north, east, south, west);
                // (true,true,true,true)
                // So essentially the loop ends if there are four,
                // or goes round again for multiples of 4 - of course,
                // it will error if there is ever 3, or 5 or any other non multiple of 4

                x++; // Start next column
            }

Answer 5

在循环本身内增加循环变量是危险的。我建议为您的北、东等变量创建一个自定义类型，然后消耗每一行到最后。或者更好的是返回下一个 GridSquare 对象。

这可以通过返回 GridSquares 迭代器的方法来完成：

StreamReader sr = new StreamReader("input.txt");

string line;
char[] lineCh;
char current;
int x, y;
bool north, east, south, west;

x = y = 0;

while ((line = sr.ReadLine()) != null)
{
    foreach (var gs in GetGridSquares(line))
    {
        // grid[x, y] = gs;
    }

     Console.WriteLine(line);
     y++;
 }

GetGridSquares 是：

 private IEnumerable<GridSquare> GetGridSquares(string line)
    {
        var splittedLine = line.Split(' ');
        foreach (var gsStr in splittedLine)
        {
            if (gsStr.Length != 4)
            {
                continue;
            }

            yield return new GridSquare(gsStr[0], gsStr[1], gsStr[2], gsStr[3]);
        }
    }

Answer 6

StreamReader sr = new StreamReader(gridPath);

var line;       
var y = 0;  

while ((line = sr.ReadLine()) != null)
{
    for(var i =0; i<line.length;i+=2)
    {
        grid[i,y]=new GridSquare(GetBits(line[i],i));
        grid[i+1,y]=new GridSquare(GetBits(line[i],i+1));


    }
    ++y;

}

bool [] GetBits(char bBytes, int n)
{
    var returned = new bool[4];
    bBytes = bBytes << ((n%2)*4);
    for(var i =0; i < 4; ++i)
        returned[i]=(bBytes & (1<<i ) > 0;

}

Answer 7

我在清理代码后发现的实际答案是 x 从未被设置回零；我们永远不会移动到网格的下一行[,]。我意识到从我的例子中很难解决这个问题，对此我深表歉意。