質問
この質問とを義務付けられているウェブサイトって思ったのではないと思うまた間違いで-)
の募集でしたが こちらの
んでいきたいと思い、ギザギザにな配列をもった次のコード
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;
}
}
思いった主要なミスが東っきず助けてくれるのでworkinコード
解決
あなたは、行とセルを行をダブル読み、または多分あなたが混ざってきたように見える - 特にルックスでこのビットの本当にの奇数ます:
while ((line = rows.ReadLine()) != null)
{
String[][]rows = new String [i][]; ;
rows = rows.ReadLine();
String[][] rows = new string[S.Length][];
i++;
}
は、すなわち。再宣言するなど、行、ループごとにReadLine
するには、2つの呼び出し、私はあなたが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でこれをやるべきどのくらい十分に強調することはできません...
他のヒント
まず、ただくにはラップポリシーなどの資源の流れを記述です。
個人的には思っ LineReader クラスで読みやすくラインのテキストファイルから(またはしてなんといっても、実際).
次に、い 使用の配列 ないと思っています。 List<T>
一般的に多く快適なものにすることが可能です。今の場合は文字列です。分割は何をしたい、簡単にて 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を開発し、 プ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;
}
}
ごグループ化を行は男女別できるかもしれなくても簡単になり、特にわかれば、ジェンダーは常に"男性"や"女性"です。