la lecture d'un fichier texte dans un tableau en dents de scie
-
21-08-2019 - |
Question
Je trouve cette question dans une ancienne question sur votre site, donc je pensais que je peux le faire, mais je pense que je me suis trompé: -)
Je suis nouveau à la chose en dents de scie tableau fait donc le code suivant
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;
}
}
Je sais que je l'ai fait des erreurs majeures, mais à l'est, j'ai essayé quelqu'un peut me aider à faire un code Workin
La solution
Vous semblez être à double lecture des lignes, ou peut-être que vous avez mélangé des lignes et des cellules - ce bit en apparence particulier vraiment bizarre:
while ((line = rows.ReadLine()) != null)
{
String[][]rows = new String [i][]; ;
rows = rows.ReadLine();
String[][] rows = new string[S.Length][];
i++;
}
i.e.. re-lignes déclarant, deux appels à la boucle par ReadLine
, etc. Je suppose que vous voulez dire string.Split
? De toute façon, que ce soit l'utilisation File.ReadAllLines
, ou regarder quelques-unes des options présentées hier. Si vous êtes désespéré d'utiliser des tableaux, le noyau pourrait ressembler à:
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);
}
}
Encore une fois - je ne peux pas insister assez sur combien vous devriez faire cela avec LINQ au lieu de mise en boucle manuelle ...
Autres conseils
Tout d'abord, assurez-vous d'emballer des ressources non gérés tels que les flux en utilisant les déclarations.
Personnellement, j'aime classe mon LineReader qui le rend facile à lire des lignes de texte à partir d'un fichier (ou quoi que ce soit d'autre, en fait).
Ensuite, je voudrais évitez d'utiliser des tableaux sauf si vous avez vraiment. Est généralement List<T>
beaucoup plus agréable de travailler avec. Maintenant, si string.split fait ce que vous voulez, vous pouvez facilement avoir un List<String[]>
. Sinon, vous pouvez probablement faire beaucoup de travail en utilisant 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]) };
En prenant plusieurs agrégats à partir d'un seul flux de données est difficile dans « normal » LINQ (ce qui est la raison pour laquelle Marc Gravell et moi avons développé Push LINQ). Cependant, vous pouvez utiliser une instruction foreach normale:
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;
}
}
Si vous avez regroupé les lignes par sexe vous pourriez être en mesure de rendre cela encore plus simple, surtout si vous savez que le sexe est toujours « mâle » ou « femelle ».