从文本文件读取到锯齿状数组

时间:2008-12-12 08:41:10

标签: c# jagged-arrays

我在你的网站上的一个老问题中发现了这个问题所以我认为我可以做到但我认为我错了: - )

旧帖子是here

我是锯齿状阵列的新手,所以做了以下代码

StreamReader rows = new StreamReader("c:\\practice.txt");
            string line;
            int i;
            i=1;
            while ((line = rows.ReadLine()) != null)
            {
                String[][]rows = new String [i][]; ;
                rows = rows.ReadLine();
                String[][] rows = new string[S.Length][];
                i++;
            }
            for (int i; i < S.Length; i++)
            {

                row[i] = S[I].Split(',');

            }

            int totalCounter = 0, totalSum = 0;
            // etc
            foreach(string[] row in rows)
            {    
                int m1 = int.Parse(row[3]);
                totalCounter++;
                totalSum += m1;
                switch(row[2])
                {        
                    case "male":
                    maleCount++;            
                    maleSum += m1;            
                        break;        
                    case "female":            
                        femaleCount++;            
                        femaleSum += m1;            
                        break;    
                }
            }

我知道我犯了重大错误但在东方我试过可以帮助我把它变成一个工作代码

2 个答案:

答案 0 :(得分:2)

首先,确保在using语句中包含非流量资源(如流)。

就我个人而言,我喜欢我的LineReader类,它可以轻松地从文件中读取文本行(或其他任何内容)。

接下来,除非你真的需要,否则我会avoid using arrays。使用List<T>通常更愉快。现在,如果string.Split可以满足您的需求,您可以轻松拥有List<String[]>。或者,你可以使用LINQ做很多工作:

var query = from line in new LineReader("c:\\practice.txt")
            let parts = line.Split(',')
            select new { Gender=parts[2], Amount=int.Parse(parts[3]) };

在“正常”LINQ中从单个数据流中获取多个聚合是很棘手的(这就是为什么Marc Gravell和我开发Push LINQ)。但是,您可以使用正常的foreach语句:

int totalCounter = 0, totalSum = 0;
int maleCount = 0, maleSum = 0, femaleCount = 0, femaleSum = 0;
foreach (var row in query)
{
    totalCounter++;
    totalSum += row.Amount;
    switch (row.Gender)
    {
        case "male":
            maleCount++;
            maleSum += Amount;
            break;
        case "female":
            femaleCount++;
            femaleSum += Amount;
            break;
    }
}

如果按性别对行进行分组,您可以使这更简单,尤其是如果您知道性别始终是“男性”或“女性”。

答案 1 :(得分:2)

你似乎是双读行,或者你可能混合了行和单元格 - 这一点特别看起来真的奇数:

        while ((line = rows.ReadLine()) != null)
        {
            String[][]rows = new String [i][]; ;
            rows = rows.ReadLine();
            String[][] rows = new string[S.Length][];
            i++;
        }

即。重新声明行,每循环两次调用ReadLine等等。我怀疑你的意思是string.Split?无论哪种方式,要么使用File.ReadAllLines,要么查看昨天提出的一些选项。如果你急于使用数组,核心可能看起来像:

using System;
using System.IO;
static class Program
{
    static void Main()
    {
        string[] lines = File.ReadAllLines("foo.txt");
        string[][] grid = new string[lines.Length][];
        for (int i = 0; i < lines.Length; i++)
        {
            grid[i] = lines[i].Split(',');
        }

        int totalCount = 0, maleCount = 0, femaleCount = 0,
            m1Total = 0, m2Total = 0, m3Total = 0,
            m1MaleTotal = 0, m1FemaleTotal = 0;
        foreach (string[] line in grid)
        {
            totalCount++;
            int m1 = int.Parse(line[3]),
                m2 = int.Parse(line[4]),
                m3 = int.Parse(line[5]);
            m1Total += m1;
            m2Total += m2;
            m3Total += m3;
            switch (line[1].Trim())
            {
                case "male":
                    maleCount++;
                    m1MaleTotal += m1;
                    break;
                case "female":
                    femaleCount++;
                    m1FemaleTotal += m1;
                    break;
            }
        }
        Console.WriteLine("Rows: " + totalCount);
        Console.WriteLine("Total m1: " + m1Total);
        Console.WriteLine("Average m1: " + ((double)m1Total)/totalCount);
        Console.WriteLine("Male Average m1: " + ((double)m1MaleTotal) / maleCount);
        Console.WriteLine("Female Average m1: " + ((double)m1FemaleTotal) / femaleCount);
    }
}

再一次 - 我不能强调你应该用LINQ而不是手动循环来做多少...