Foro de elhacker.net

Hola buenas!!

He realizado una series de funciones que compilándolas con el GCC funcionan sin problema, pero necesito pasarlo al entorno de desarrollo Visual Studio 2012.

En una de las funciones, tengo que leer un fichero.
Preguntas:

• El fichero no se el tamaño que puede tener una línea, de modo que abro el fichero una 1º vez , lo recorro entero y guardo el tamaño de la línea de mayor tamaño (tamMax) y lo igualo a N, pasando de tener un buffer gigantesco a uno apropiado.
  Este procedimiento compilandolo en GCC funciona, pero en Visual Studio no.
  
  error C2057: se esperaba una expresión constante   en linea[N]
  
  Como solución he visto que me plantean usar:
  #define int N;
  
  El problema es que al ser define no puedo cambiar su valor y por tanto, en la línea 30 salta error.
  
  
• ¿Cómo hago para tratar usar la ruta relativa en el fichero? Desde el directorio actual quiero crear un directorio y dentro tener el .txt
  

#include "librerias.h"
 
int N;
#define CRT_SECURE_NO_WARNINGS
 
int leer_fichero_operComp()
{ 
	N=20000;
	FILE *p_fichero=NULL;
	char linea[N];
	int i=0,j=0,tam,tamMax=0;
	char *ficheros[] = {"hola.txt","adios.csv"};
 
	if((p_fichero=fopen(ficheros[0],"r"))==NULL)
		{
			printf("Error al abrir el fichero");
			exit(-1);
		} 
	else{
		while( fgets(linea,N,p_fichero) != NULL)
		{
			if( linea[0] != '\0')
			{		
			tam=strlen(linea);
			if(tam > tamMax)
				tamMax = tam;	
			}
		}//while
		fclose(p_fichero);
		N=tamMax;
 
		p_fichero=fopen(ficheros[0],"r");	
		while( fgets(linea,N,p_fichero) != NULL)
		{
			if( strlen(linea) > 1)
			{	
			tam=strlen(linea);
			sscanf(linea,"%[^\n]",linea);	
			printf("\n\t Linea %d :%s ",j,linea);
			j++;
			}
		}//while
	fclose(p_fichero);	
	}//else	
return 0;
}
 
 
 

PD: No se si debería haber puesto este tema en otro foro no de C

Compila mas no funciona como lo describiste, el tamaño de la variable local linea (almacenada en la pila) es constante: siempre es 20000. Que posteriormente cambies el valor de N no tiene ninguna incidencia sobre linea (si le indica fgets que el buffer tiene menos caracteres realmente). Yo me preocuparia mas por no duplicar el acceso a disco, abriendo, leyendo y cerrando de nuevo, mejor seria que usaras memoria dinamica y si fgets no devuelve el salto de linea - y no llego a EOF - leyeras nuevamente ...


Dir\file.txt

Buenas!
Gracias por tu respuesta.

¿Entonces si de esta forma no altero el tamaño N de línea cómo lo planteo?
Aunque use memoria dinámica para guardar cada línea del fichero, en la función fgets hay que indicar una cantidad de caracteres a leer en mi caso N.

Por eso tratar de obtenerlo antes. Así no lee 200000 caracteres sino tamMax

¿Y si defino la variable char linea[N] después de obtener N?


Lo de la ruta entendido! :)

Declarar un array cuyo numero de elementos se indica mediante un valor calculado en tiempo de ejecución (VLA, array de longitud variable) solo es posible si el compilador soporta el estándar C99 (en C90 no existe y en C11 (el ultimo) es opcional), si vas por esa aproximación solo queda revisar la documentación del compilador.

Otra opción ya la indico Eternal Idol, al utilizar las funciones para reserva dinámica de memoria (malloc, callo y realloc) te olvidas de ese problema.

Ya por ultimo no es necesario utilizar fgets y strlen para conocer la longitud de la cadena mas larga y verificar que el primer elemento del array no sea cero es innecesario ya que si fgets no retorna NULL significa que leyó y almaceno por lo menos un carácter en el array (mas el carácter '\0').

Una forma mas simple de obtener la longitud máxima incluyendo el carácter '\n' es mediante fgetc, mas o menos así:

FILE *p_fichero;
int ch;
long max_len;
long len;
 
/* ... */
 
max_len = len = 0;
while ((ch = fgetc(p_fichero)) != EOF){
   len++;
 
   if (ch == '\n'){
      if (len > max_len)
         max_len = len;
 
      len = 0;
   }
}

Un saludo

Olvidate del tamaño de la linea mas larga, no te sirve absolutamente para nada, lees todo el archivo para despues usar eso nuevamente para leer el archivo PERO aca esta la clave no tenes el archivo en memoria, tenes UNA sola linea al mismo tiempo asi que no tiene ningun sentido hacer ese calculo.

Lo mejor que podes hacer es una funcion que lea una linea completa usando memoria dinamica y llamarla dentro de un bucle.

He aplicado los cambios que me habéis comentado y ya funciona sin problema =)

Al final he usado ambas ideas. Aplicar fgetc a la par que reservada memoria con malloc y realloc.

Así que muchas gracias rir3760 y Eternal Idol
Un saludo!

De nadas  ::)