[C] - Problemas con archivos - SOLUCIONADO

[cbug]

Hola, resulta que estaba practicando algo con archivos, y se me ocurrió hacer 2 procedimientos para leer línea a línea y char a char... En fin, este es el code:

Código (c) [Seleccionar] Expandir

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void Lee_Caracter(char *RUTA);
void Lee_Lineas(char *RUTA);
void Menu(void);
char *itoa(int val, int base);

void Limpiar_Buffer(void)
{
 while(getchar() != '\n');
}


int main()
{
 
 int opt;
 char *ruta_local = "texto.txt";
 Menu();
 scanf("%d", &opt);
 Limpiar_Buffer();
 while(opt != 0)
 {
   switch(opt)
   {
     case 1:
     {
Lee_Caracter(ruta_local);
break;
     }
     case 2:
     {
Lee_Lineas(ruta_local);
break;
     }      
     default:
break;
   }
   Menu();
   scanf("%d", &opt);
   Limpiar_Buffer();
 }
 return 0;
}


char* itoa(int val, int base){
 static char buf[32] = {0};
 int i = 30;
 for(; val && i ; --i, val /= base)
   buf[i] = "0123456789abcdef"[val % base];
 return &buf[i+1];
}

void Menu(void)
{
 puts("\t**** MENU ****\t");
 puts("1- Leer caracter a caracter");
 puts("2- Leer linea a linea");
 puts("0- Salir");
 puts("Ingrese Opcion>");
}

void Lee_Caracter(char *RUTA)
{
 FILE *fp;
 char buffer;
 fp = fopen(RUTA, "r");
 if(!fp)
   puts("Error al intentar abrir archivo");
 else
   while((buffer = getc(fp)) != EOF)
     printf("%c", buffer);
 putchar(buffer);
 fclose(fp);
}

void Lee_Lineas(char *RUTA)
{
 FILE *fp;
 char *buffer;
 char *mi_eof = NULL;
 mi_eof = atoi(EOF, 10);
 fp = fopen(RUTA, "r");
 if(!fp)
   puts("Error al intentar abrir archivo");
 else
 {
   buffer = (char *)(malloc(sizeof(char) * 100));
   fgets(buffer, sizeof(char) * 100, fp);
   
   while(strstr(buffer, mi_eof) != NULL);
   {
     puts(buffer);
     free(buffer);
     buffer = (char *)(malloc(sizeof(char) * 100));
     fgets(buffer, sizeof(char) * 100, fp);
   }
 }
 free(buffer);
 fclose(fp);
}

El problema está en mi función atoi, al compilar:

Código [Seleccionar] Expandir

lectura.c: In function 'Lee_Lineas':
lectura.c:89: warning: passing argument 1 of 'atoi' makes pointer from integer without a cast
lectura.c:89: error: too many arguments to function 'atoi'
lectura.c:89: warning: assignment makes pointer from integer without a cast


No puedo resolverlo.

Ahora bien, tengo una duda más, está bien la forma en que busco el EOF del archivo, en la función Lee_Lineas()?


EDITADO: La primera duda resuelta, estaba dormido y cambié itoa por atoi  
Ahora el problema es que la función Lee_Lineas no funciona  

CORREGIDO: while(strstr(buffer, mi_eof) == NULL)

Creo que el problema está en que no incremento el puntero para leer el archivo.

[leogtz]

Código [Seleccionar] Expandir

int atoi ( const char * str );
No concuerda con tu llamada, estás enviando EOF una macro o un indicador.
Sin contar que la función solo trabaja con un argumento.

malloc() no necesita un casting.

Y para cada clausula dentro del switch(), no son necesarias las llaves

Código [Seleccionar] Expandir

case ALGO:
/* codigo */
break;

Saludos.

[cbug]

Código (c) [Seleccionar] Expandir

void Lee_Lineas(char *RUTA)
{
 FILE *fp;
 char *buffer;
 char *mi_eof = NULL;
 mi_eof = itoa(EOF, 10);
 fp = fopen(RUTA, "r");
 if(!fp)
   puts("Error al intentar abrir archivo");
 else
 {
   buffer = malloc(sizeof(char) * 100);
   fgets(buffer, sizeof(char) * 100, fp);
   
   while(strstr(buffer, mi_eof) == NULL);
   {
     puts(buffer);
     buffer = malloc(sizeof(char) * 100);
     fgets(buffer, sizeof(char) * 100, fp);
   }
 }
 free(buffer);
 fclose(fp);
}

Aqui tengo el problema en que sólo lee la primera línea y sale.

SOLCUCIONADO:

No se necesita comparar, o buscar el EOF, leyendo el man de fgets:

Código [Seleccionar] Expandir

Upon successful completion, fgets() and gets() return a pointer to the
     string.  If end-of-file occurs before any characters are read, they
     return NULL and the buffer contents remain unchanged.  If an error
     occurs, they return NULL and the buffer contents are indeterminate.  The
     fgets() and gets() functions do not distinguish between end-of-file and
     error, and callers must use feof(3) and ferror(3) to determine which
     occurred.[/b]

Código (c) [Seleccionar] Expandir

void Lee_Lineas(char *RUTA)
{
  FILE *fp;
  char *buffer;
  fp = fopen(RUTA, "r");
  if(!fp)
    puts("Error al intentar abrir archivo");
  else
  {
    buffer = malloc(sizeof(char) * 100);
   
    while(fgets(buffer, sizeof(char) * 100, fp));
    {
      puts(buffer);
      buffer = malloc(sizeof(char) * 100);
    }
  }
  free(buffer);
  fclose(fp);
}

[leogtz]

Es que no entiendo por qué usar itoa() y strstr() si tu objetivo solo es leer el archivo línea línea.

Yo hice algo así y funciona:

Código (c) [Seleccionar] Expandir

void Lee_Lineas(char *ruta)
{
 FILE *archivo = fopen(ruta, "r");
 if(archivo == NULL)
 {
     perror("Problema abriendo archivo.");
 } else {
     char linea[500];
     while(fgets(linea, 500, archivo) != NULL)
         printf("%s", linea);
 }
   fclose(archivo);
}


[cbug]

Cierto, justo había leído el man. Solucionado

[leogtz]

No utilices un array dinámico, puesto que el valor por defecto es 100, así que puedes crearlo directamente:

Código [Seleccionar] Expandir

char vector[LIM]

Así te evitas el uso de malloc(), free() y la inclusión de stdlib.h.

saludos.

[cbug]

Es que ahora pienso modificarlo, puesto que como dijiste, sólo lee 100 caracteres, y muchas veces el .txt posee más en una sóla línea, y necesitaré prontamente un realloc()