1

Estoy intentando leer un archivo txt pero me aparece el error "indice fuera de la matriz"

if (nFile.Length > 0)
{
    StreamReader rd = new StreamReader(directorio + "\\log.txt");

    while (rd.Peek() != -1)
    {
        string s = rd.ReadLine();
        string[] arr = s.Split(';');
        string[] arrLog = arr[0].Split(',');

        fw.mdbcmd.Parameters.Clear();
        fw.mdbcmd.CommandText = "GuardaEvento";
        fw.mdbcmd.CommandType = CommandType.StoredProcedure;
        fw.mdbcmd.Parameters.Add("@ID", OleDbType.VarChar, 50).Value = arrLog[0].ToString().Trim();
        fw.mdbcmd.Parameters.Add("@code", OleDbType.VarChar, 50).Value = arrLog[1].ToString().Trim();
        fw.mdbcmd.Parameters.Add("@bando", OleDbType.VarChar, 50).Value = arrLog[2].ToString().Trim();
        fw.mdbcmd.Parameters.Add("@observacion", OleDbType.VarChar, 100).Value = arrLog[3].ToString().Trim();
        fw.mdbcmd.Parameters.Add("@user", OleDbType.VarChar, 50).Value = arrLog[4].ToString().Trim();
        fw.mdbcmd.ExecuteNonQuery();
    }

    rd.Close();
    File.WriteAllText(directorio + "\\log.txt", "");

}

Esto es lo que guardo en el archivo log.txt:

public void guardarLogTxt(string codigo, string numParte, string descripcion, string user)
  {
 string directorio = System.IO.Directory.GetCurrentDirectory();
StreamWriter escribirLog = File.AppendText(directorio + "\\log.txt");
 String contenido = Properties.Settings.Default.id.ToString() + "," + code + "," + bando + "," + observacion + ","+ user + ";" + Environment.NewLine;
                escribirLog.Write(contenido.ToString());
                escribirLog.Flush();
                escribirLog.Close();
            }
2
  • 1
    Hola Ronald, bienvenido a Stack Overflow en español. Por favor, lee el recorrido de bienvenida para aprender el funcionamiento básico del sitio. Enseñanos qué texto hay en log.txt porque parece que estas accediendo a un indice tras el split que no existe. Recuerda leer Cómo preguntar y ejemplo mínimo verificable para más info en como mejorar la pregunta. Un saludo.
    – lois6b
    Commented el 25 abr. 2018 a las 15:23
  • Enseña el contenido del fichero, no cómo lo generas.
    – lois6b
    Commented el 25 abr. 2018 a las 15:33

1 respuesta 1

1

El error es debido a que no siempre se cumple la condición de hacer el split a 5 posiciones del arreglo. Sugiero que uses Linq en conjunto con una clase para hacer un mapeo de datos más limpio y además para que asegures que siempre obtendrás el resultado esperado.

Clase Datos:

public class Datos {
    public string ID { get; set; }
    public string code { get; set; }
    public string bando { get; set; }
    public string observacion { get; set; }
    public string user { get; set; }    
}

Código:

List<Datos> listaArchivo = new List<Datos>();

listaArchivo = (from p in File.ReadAllLines(directorio + "\\log.txt")
        let parts = p.Split(new[] { ',' }, StringSplitOptions.RemoveEmptyEntries) // <-- Aquí se hace el split
        where parts.Length == 5 // <-- Aquí siempre aseguras que se hará el split a 5 posiciones, las líneas del archivo que no cumplan con la condición se omiten
        select new Datos
        {
            ID = parts[0].ToString().Trim(),
            code = parts[1].ToString().Trim(),
            bando = parts[2].ToString().Trim(),
            obseracion = parts[3].ToString().Trim(),
            user = parts[4].ToString().Trim()
        }).ToList();

foreach(Datos item in listaArchivo) 
{
    fw.mdbcmd.Parameters.Clear();
    fw.mdbcmd.CommandText = "istl.SP_PRIMASYS_GuardaEvento";
    fw.mdbcmd.CommandType = CommandType.StoredProcedure;
    fw.mdbcmd.Parameters.Add("@ID", OleDbType.VarChar, 50).Value = item.ID;
    fw.mdbcmd.Parameters.Add("@code", OleDbType.VarChar, 50).Value = item.code;
    fw.mdbcmd.Parameters.Add("@bando", OleDbType.VarChar, 50).Value = item.bando;
    fw.mdbcmd.Parameters.Add("@observacion", OleDbType.VarChar, 100).Value = item.obsercacion;
    fw.mdbcmd.Parameters.Add("@user", OleDbType.VarChar, 50).Value = item.user;
    fw.mdbcmd.ExecuteNonQuery();
}

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.