Question

我正在尝试使用c＃读取.txt文件并显示其内容，但我收到的错误为IndexOutOfRangeException，错误代码为0xc000013a。

这是我的代码：

static void Main(string[] args)
    {
        StreamReader sStreamReader = new StreamReader("d:\\TEST.txt");
        while (!sStreamReader.EndOfStream)
        {
            string sLine = "";
            if (sLine != null)
            {
                sLine = sStreamReader.ReadLine();
                if (sLine != null)
                {
                    string[] rows = sLine.Split(",".ToCharArray());
                    double a = Convert.ToDouble(rows[1]);
                    Console.Write(a);
                    int b = Convert.ToInt32(rows[3]);
                    Console.WriteLine(b);
                    Console.WriteLine();
                }
            }
        }
    }

我的文本文件如下：

1,2,3,4,5,6,7

1,2,3,4,5,6,7

5,6,2,7,3,8,4

3,4,3,4,3

5,3,23,12

12,30000,12,99

Answer 1

我会将其更改为以下内容：

static void Main(string[] args)
    {
        // StreamReader is IDisposable which should be wrapped in a using statement
        using (StreamReader reader = new StreamReader(@"d:\TEST.txt")) 
        {
            while (!reader.EndOfStream)
            {
                string line = reader.ReadLine();
                // make sure we have something to work with
                if (String.IsNullOrEmpty(line)) continue;

                string[] cols = line.Split(',');
                // make sure we have the minimum number of columns to process
                if (cols.Length < 4) continue;

                double a = Convert.ToDouble(cols[1]);
                Console.Write(a);
                int b = Convert.ToInt32(cols[3]);
                Console.WriteLine(b);
                Console.WriteLine();
            }
        }
    }

这里有一些注意事项：

StreamReader实现了IDisposable，因此您应该将其包装在using子句中，以便正确处理它。
不要说出像“sLine”这样的名字。这种形式的匈牙利人通常被认为是严重的不良行为。即便是微软也说不这样做。
您处理的是列，而不是行。因此应该适当地命名该变量。
在盲目访问之前，请始终进行测试以确保您拥有所需的所有列。
通常，我不会使用Convert.ToDouble或Convert.ToInt32。使用TryParse确保它能够转换更安全。如果cols [1]和cols [3]有非数字数据，那么你所拥有的代码就会被破坏。
您可以在字符串前面使用@符号告诉编译器它不需要转义。
简单地“继续”循环而不是将其包装在if语句中更清晰。
将String变量设置为空字符串然后立即将其设置为其他值会导致空白留在整个范围的内存中。换句话说，它浪费了记忆。当然，在这种情况下，它是一种微观优化，但始终使用最佳实践绝不会受到伤害。

Answer 2

您是否考虑在访问row.Length和row[1]之前检查row[3]

我怀疑你的空行是问题

Answer 3

以下是如何做到这一点的简单方法：

        string[] lines = File.ReadAllLines("d:\\TEST.txt");
        foreach (var line in lines.Where(line => line.Length > 0))
        {
            string[] numbers = line.Split(',');

            // It checks whether numbers.Length is greater than 
            // 3 because if maximum index used is 3 (numbers[3]) 
            // than the array has to contain at least 4 elements
            if (numbers.Length > 3)
            {
                double a = Convert.ToDouble(numbers[1]);
                Console.Write(a);
                int b = Convert.ToInt32(numbers[3]);
                Console.Write(b);
                Console.WriteLine();
            }
        }

Answer 4

你应该考虑使用：

if (!string.IsNullOrEmpty(sLine))

而不是

if (sLine != null)

你有这个例外，因为有些行是空的。

但是，这是一种在使用StreamReader时应该编写代码的方法：

using(var reader = new StreamReader(@"d:\\TEST.txt"))
{
    string line;
    while ((line= reader.ReadLine()) != null)
    {
        if (string.IsNullOrEmpty(line)) continue;

        var rows = line.Split(",".ToCharArray());
        var a = Convert.ToDouble(rows[1]);
        Console.Write(a);
        var b = Convert.ToInt32(rows[3]);
        Console.WriteLine(b);
        Console.WriteLine();
    }
}

此致

凯文

读取.text文件并显示文件中的数字

4 个答案: