c# - txt - La mejor manera de leer un archivo en la lista<string>

leer y escribir archivos en c#

Question

Estoy utilizando una lista para limitar el tamaño del archivo, ya que el destino está limitado en disco y RAM. Esto es lo que estoy haciendo ahora, pero ¿hay una manera más eficiente?

readonly List<string> LogList = new List<string>();
...
var logFile = File.ReadAllLines(LOG_PATH);
foreach (var s in logFile) LogList.Add(s);

Answer 1

¿Por qué no usar un generador en su lugar?

private IEnumerable<string> ReadLogLines(string logPath) {
using(StreamReader reader = File.OpenText(logPath)) {
string line = "";
while((line = reader.ReadLine()) != null) {
yield return line;
}
}
}

Entonces puedes usarlo como usarías la lista:

var logFile = ReadLogLines(LOG_PATH);
foreach(var s in logFile) {
// Do whatever you need
}

Por supuesto, si necesita tener una List<string> , entonces deberá guardar todo el contenido del archivo en la memoria. Realmente no hay manera de evitar eso.

Answer 2

No lo guarde si es posible. Solo léelo si tienes memoria limitada. Puedes usar un StreamReader:

using (var reader = new StreamReader("file.txt"))
{
var line = reader.ReadLine();
// process line here
}

Esto puede envolverse en un método que produzca cadenas por línea de lectura si desea usar LINQ.

Answer 3

Una pequeña actualización a la respuesta de Evan Mulawski para hacerla más corta.

List<string> allLinesText = File.ReadAllLines(fileName).ToList()

Answer 4

[Editar]

Si está haciendo esto para recortar el comienzo de un archivo de registro, puede evitar cargar todo el archivo haciendo algo como esto:

// count the number of lines in the file
int count = 0;
using (var sr = new StreamReader("file.txt"))
{
while (sr.ReadLine() != null)
count++;
}
// skip first (LOG_MAX - count) lines
count = LOG_MAX - count;
using (var sr = new StreamReader("file.txt"))
using (var sw = new StreamWriter("output.txt"))
{
// skip several lines
while (count > 0 && sr.ReadLine() != null)
count--;
// continue copying
string line = "";
while (line = sr.ReadLine() != null)
sw.WriteLine(line);
}

En primer lugar, dado que File.ReadAllLines carga todo el archivo en una matriz de string[] ( string[] ), copiar en una lista es redundante.

En segundo lugar, debe comprender que una List se implementa utilizando una matriz dinámica debajo del capó. Esto significa que CLR deberá asignar y copiar varias matrices hasta que pueda acomodar todo el archivo. Dado que el archivo ya está en el disco, puede considerar cambiar la velocidad por la memoria y trabajar directamente en los datos del disco, o procesarlo en trozos más pequeños.

1. Si necesita cargarlo completamente en la memoria, al menos intente dejarlo en una matriz:

    string[] lines = File.ReadAllLines("file.txt");

2. Si realmente necesita ser una List , cargue las líneas una por una:

    List<string> lines = new List<string>();
   using (var sr = new StreamReader("file.txt"))
   {
   while (sr.Peek() >= 0)
   lines.Add(sr.ReadLine());
   }

   Nota: la List<T> tiene un constructor que acepta un parámetro de capacidad. Si conoce el número de líneas por adelantado, puede evitar asignaciones múltiples preasignando la matriz por adelantado:

    List<string> lines = new List<string>(NUMBER_OF_LINES);

3. Aún mejor, evite almacenar todo el archivo en la memoria y procesarlo "sobre la marcha":

    using (var sr = new StreamReader("file.txt"))
   {
   string line;
   while (line = sr.ReadLine() != null)
   {
   // process the file line by line
   }
   }

Answer 5

//this is only good in .NET 4
//read your file:
List<string> ReadFile = File.ReadAllLines(@"C:/TEMP/FILE.TXT").ToList();
//manipulate data here
foreach(string line in ReadFile)
{
//do something here
}
//write back to your file:
File.WriteAllLines(@"C:/TEMP/FILE2.TXT", ReadFile);

Answer 6

string inLine = reader.ReadToEnd();
myList = inLine.Split(new string[] { "/r/n" }, StringSplitOptions.None).ToList();

También utilizo Environment.NewLine.toCharArray también, pero descubrí que no funcionó en un par de archivos que terminaron en / r / n. Prueba cualquiera de las dos y espero que te funcione bien.

Answer 7

var logFile = File.ReadAllLines(LOG_PATH);
var logList = new List<string>(logFile);

Como logFile es una matriz, puede pasarlo al constructor List<T> . Esto elimina la sobrecarga innecesaria al iterar sobre la matriz o al usar otras clases de IO.

Implementación real del constructor :

public List(IEnumerable<T> collection)
{
...
ICollection<T> c = collection as ICollection<T>;
if( c != null) {
int count = c.Count;
if (count == 0)
{
_items = _emptyArray;
}
else {
_items = new T[count];
c.CopyTo(_items, 0);
_size = count;
}
}
...
} 

Answer 8

List<string> lines = new List<string>();
using (var sr = new StreamReader("file.txt"))
{
while (sr.Peek() >= 0)
lines.Add(sr.ReadLine());
}

Sugeriría esto ... de la respuesta de Groo.