que - Pasando por un archivo de texto línea por línea en C

programa en c que lea un archivo txt

Question

He estado trabajando en un pequeño ejercicio para mi clase CIS y estoy muy confundido por los métodos que C usa para leer un archivo. Todo lo que realmente necesito hacer es leer un archivo línea por línea y usar la información recopilada de cada línea para hacer algunas manipulaciones. Intenté usar el método getline y otros sin suerte. Mi código es actualmente como sigue:

int main(char *argc, char* argv[]){
const char *filename = argv[0];
FILE *file = fopen(filename, "r");
char *line = NULL;
while(!feof(file)){
sscanf(line, filename, "%s");
printf("%s/n", line);
}
return 1;
}

Ahora mismo estoy teniendo una falla de seg con el método sscanf y no estoy seguro de por qué. Soy un total de C noob y me pregunto si había algo del panorama general que me faltaba. Gracias

Answer 1

Además de las otras respuestas, en una biblioteca de C reciente (compatible con Posix 2008), puede usar getline . Ver esta respuesta (a una pregunta relacionada).

Answer 2

Digamos que está tratando con algún otro delimitador, como una pestaña /t , en lugar de una /n nueva línea.

Un enfoque más general de los delimitadores es el uso de getc() , que capta un personaje a la vez.

Tenga en cuenta que getc() devuelve un int , para que podamos probar la igualdad con EOF .

En segundo lugar, definimos una line[BUFFER_MAX_LENGTH] matriz line[BUFFER_MAX_LENGTH] de tipo char , para almacenar hasta BUFFER_MAX_LENGTH-1 caracteres en la pila (tenemos que guardar ese último carácter para un carácter terminador /0 ).

El uso de una matriz evita la necesidad de usar malloc y free para crear un puntero de carácter de la longitud correcta en el montón.

#define BUFFER_MAX_LENGTH 1024
int main(int argc, char* argv[])
{
FILE *file = NULL;
char line[BUFFER_MAX_LENGTH];
int tempChar;
unsigned int tempCharIdx = 0U;
if (argc == 2)
file = fopen(argv[1], "r");
else {
fprintf(stderr, "error: wrong number of arguments/n"
"usage: %s textfile/n", argv[0]);
return EXIT_FAILURE;
}
if (!file) {
fprintf(stderr, "error: could not open textfile: %s/n", argv[1]);
return EXIT_FAILURE;
}
/* get a character from the file pointer */
while(tempChar = fgetc(file))
{
/* avoid buffer overflow error */
if (tempCharIdx == BUFFER_MAX_LENGTH) {
fprintf(stderr, "error: line is too long. increase BUFFER_MAX_LENGTH./n");
return EXIT_FAILURE;
}
/* test character value */
if (tempChar == EOF) {
line[tempCharIdx] = ''/0'';
fprintf(stdout, "%s/n", line);
break;
}
else if (tempChar == ''/n'') {
line[tempCharIdx] = ''/0'';
tempCharIdx = 0U;
fprintf(stdout, "%s/n", line);
continue;
}
else
line[tempCharIdx++] = (char)tempChar;
}
return EXIT_SUCCESS;
}

Si debe usar un char * , entonces puede seguir usando este código, pero strdup() la line[] array, una vez que se llena con el valor de entrada de una línea. Debe free esta cadena duplicada una vez que haya terminado con ella, o obtendrá una pérdida de memoria:

#define BUFFER_MAX_LENGTH 1024
int main(int argc, char* argv[])
{
FILE *file = NULL;
char line[BUFFER_MAX_LENGTH];
int tempChar;
unsigned int tempCharIdx = 0U;
char *dynamicLine = NULL;
if (argc == 2)
file = fopen(argv[1], "r");
else {
fprintf(stderr, "error: wrong number of arguments/n"
"usage: %s textfile/n", argv[0]);
return EXIT_FAILURE;
}
if (!file) {
fprintf(stderr, "error: could not open textfile: %s/n", argv[1]);
return EXIT_FAILURE;
}
while(tempChar = fgetc(file))
{
/* avoid buffer overflow error */
if (tempCharIdx == BUFFER_MAX_LENGTH) {
fprintf(stderr, "error: line is too long. increase BUFFER_MAX_LENGTH./n");
return EXIT_FAILURE;
}
/* test character value */
if (tempChar == EOF) {
line[tempCharIdx] = ''/0'';
dynamicLine = strdup(line);
fprintf(stdout, "%s/n", dynamicLine);
free(dynamicLine);
dynamicLine = NULL;
break;
}
else if (tempChar == ''/n'') {
line[tempCharIdx] = ''/0'';
tempCharIdx = 0U;
dynamicLine = strdup(line);
fprintf(stdout, "%s/n", dynamicLine);
free(dynamicLine);
dynamicLine = NULL;
continue;
}
else
line[tempCharIdx++] = (char)tempChar;
}
return EXIT_SUCCESS;
}

Answer 3

Para leer una línea de un archivo, debe usar la función fgets : Lee una cadena desde el archivo especificado hasta un carácter de nueva línea o EOF .

El uso de sscanf en su código no funcionaría en absoluto, ya que usa el filename como su cadena de formato para leer desde una line hasta una cadena constante literal %s .

El motivo de SEGV es que escribe en la memoria no asignada apuntada por line .

Answer 4

Tantos problemas en tan pocas líneas. Probablemente me olvide de algunos:

• argv [0] es el nombre del programa, no el primer argumento;
• Si quieres leer en una variable, tienes que asignar su memoria.
• uno nunca realiza un bucle en feof, uno hace un bucle en una función IO hasta que falla, feof sirve para determinar el motivo del fallo,
• sscanf está ahí para analizar una línea, si desea analizar un archivo, use fscanf,
• "% s" se detendrá en el primer espacio como formato para la familia? scanf
• Para leer una línea, la función estándar es fgets,
• devolviendo 1 del fallo de medios principales

Asi que

#include <stdio.h>
int main(int argc, char* argv[])
{
char const* const fileName = argv[1]; /* should check that argc > 1 */
FILE* file = fopen(fileName, "r"); /* should check the result */
char line[256];
while (fgets(line, sizeof(line), file)) {
/* note that fgets don''t strip the terminating /n, checking its
presence would allow to handle lines longer that sizeof(line) */
printf("%s", line);
}
/* may check feof here to make a difference between eof and io failure -- network
timeout for instance */
fclose(file);
return 0;
}