1

Tengo que leer un fichero línea a línea utilizando llamadas al sistema en C pero no sé cómo detectar el final del fichero y estoy obteniendo todo el rato Segmentation Fault porque en la última línea no hay \n y se queda infinitamente leyendo el último carácter.

Esta es mi función leer linea: (fd es el descriptor de fichero)

int leerLinea(int fd, char linea[]){
    char c;
    int leidos=0;
    int i=0;
    read(fd,&c,1);
    while(c!='\n' && c!=NULL){
        linea[i]=c;
        i++;
        leidos++;
        read(fd,&c,1);
    }
    linea[i] = '\n'; //por si quieres tener el \n en lo leido
    return leidos;
}
1
  • Lo que hace es leer una línea pero cuando llega a la última línea de un fichero se queda infinitamente porque no hay un \n al final de esa línea, supongo que te habrá funcionado porque habrás añadido una línea en blanco al final Commented el 19 jul. 2020 a las 17:32

2 respuestas 2

1

El problema estaba en que sumaba 1 a leídos cada vez que entraba en el bucle en lugar de sumarlo después de comprobar si la llamada al sistema había funcionado bien y mi programa principal tenía la condición de que si leídos > 0 invocaba otra vez a leerLinea() por lo que se creaba un bucle infinito ya que leídos siempre era > 0, con su respectivo Segmentation Fault. Tampoco hacía falta comprobar que el carácter no fuese NULL.

int leerLinea(int fd, char linea[]){
    char c;
    int leidos=0;
    int fin=1;
    int i=0;
    read(fd,&c,1);
    while(c!='\n' && fin !=0){
        linea[i]=c;
        i++;
        if(fin = read(fd,&c,1)>0) leidos++;
    }
    linea[i] = '\n'; //por si quieres tener el \n en lo leido
    return leidos;
}
0

Prueba añadiendo en la condicional si llegó al final del fichero:

int leerLinea(int fd, char linea[]){
char c;
int leidos=0;
int fin = 1;
int i=0;
read(fd,&c,1);
while(c != '\n' && c!=NULL && fin != 0){
    linea[i]=c;
    i++;
    leidos++;
    fin = read(fd,&c,1);
}
linea[i] = '\n'; //por si quieres tener el \n en lo leido
return leidos;

}

6
  • No me funciona tampoco con el EOF Commented el 19 jul. 2020 a las 17:33
  • Puedes mostrar el código de la función read. Commented el 19 jul. 2020 a las 17:41
  • 1
    Read es una llamada al sistema de UNIX Commented el 19 jul. 2020 a las 17:45
  • Pues ya está, si devuelve cero es que llegó al final del archivo. Commented el 19 jul. 2020 a las 17:59
  • 1
    Esto es incorrecto. EOF no es un valor que se puede representar en un char, por lo tanto c != EOF siempre es cierto. c!=NULL tampoco tiene sentido. NULL es un puntero, y c no. Commented el 19 jul. 2020 a las 20:41

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.