SoloCodigo

Tengo un problema al ejecutar el siguiente código.
El nombre del archivo se obtiene bien por el diálogo pero después algo falla con el stream...

private StreamReader sr; // variable de clase

private void miFich_Click(object sender, System.EventArgs e)
{         
   if(ofdFich.ShowDialog()==System.Windows.Forms.DialogResult.OK && ofdFich.FileName!="")
   {
      string nom = ofdFich.FileName;
      FPrincipal.ActiveForm.Text+= nom;         
      sr= new StreamReader(nom);         
      if(ap.suPista().estaLibre())
         leerAvion();
   }
}
public void leerAvion()
{         
   string cad=null;
   do
   {
      cad=sr.ReadLine();         
   }
      while(cad==null);         
   string[] cpo= cad.Split(',');
   Avion av= null;
--   if( cpo[1].Trim().ToUpper().StartsWith("COM"))// ERROR INDICE
                            // FUERA DE LIMITES                      
         
{...}   
}

La excepción se produce porque el Split solo guarda un string en cad y no existe cad[1].
En el debugger el contenido de cad al producirse la excepción es algo así:
      " \0\0\0\0\0 \0\0 \0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\02 \0\0\0\0\0\0... (basura)

El contenido del fichero abierto es:
   Boeing 747, comer, Valencia, Madrid, 20:45, 1234
Boeing 747, com, Valencia, BCN, 20:56, 1234
Boeing 747, pasa, Valencia, Paris, 20:59, 123, 25

Algo debe estar mal pero no lo veo...

He probado a hacer simplemente cad=sr.ReadLine() en lugar de hacerlo dentro del do...while ( teoricamente la primera linea del archivo es válida porque tiene comas) y sigue dando el mismo error. Como podria arreglarlo? No entiendo muy bien a que te refieres...

el metodo leerAvion debe leer UNA SOLA LINEA, guardar los campos separados por comas en un array y despues procesarlos para crear un objeto con esos argumentos.
el do...while lo puse para asegurarme de que leia una linea no vacia pero realmente solo debe leer una linea pero MANTENIENDO EL STREAM para en llamadas posteriores leer la siguiente. Es decir:
1- definir el stream al archivo del dialogo
2- leerAvion
    - leer una linea
    - separar campos
    - procesar campos
El problema sigue estando en leer una linea ya que lee basura y no hay comas...
pero...Por qué???

Hola

Debes arreglar varias cosas:

En primer lugar el null es un indicador de final de fichero y no de linea vacia, en tal caso para capturar la linea vacia puedes hacer cad.count y mirar si es igual a cero.

Por este motivo debes cambiar el bucle de lectura de lineas y poner algo asi:


Para procesar cada linea quitandole las comas y meterla en una estructura, yo me haria una funcion a la cual le pasas la linea como string y dentro de ella te separe cada substring y los monte en la estructura y te lo devuelva ya montado



Haciendo esto no deberias tener problemas

un saludo  

Gracias Diodo pero el problema está a la hora de definir el StreamReader.
Me interesa un SR global que debo definir dentro de un método que sólo se ejecuta una vez y después quiero hacer uso de ese SR dentro de otro método LeerAvion.
En LeerAvion sólo hay que leer UNA línea y el problema esta aquí. Después el tratamiento de la línea no hay problema.

//METODO INICIAL
if(ofdFich.ShowDialog()==System.Windows.Forms.DialogResult.OK && ofdFich.FileName!="")
{            
   sr= new StreamReader(ofdFich.FileName);
....
//METODO LEERAVION
string cad;
string[] cpo;
cad=sr.ReadLine();//ERROR NULL REFERENCE
cpo= cad.Split(',');   
Avion av= null;
            
if( cpo[1].Trim().ToUpper().StartsWith("COM"))// ERROR INDICE ARRAY FUERA LIMITES

Con lo de global queria decir que es una variable de la clase que me interesa definir al principio y una sola vez, para leer la siguiente linea cada vez que sea necesario, ya que no quiero leer en un bucle sino a traves de un metodo al que se invocará en determinadas circunstancias.
No se si me explico bien, pero lo decia por si podia ser un problema definir el Stream en un metodo y utilizarlo en otro posteriormente siempre dentro de la misma clase, aunque yo creo que no deberia ser problema...
Gracias de todos modos.