Ayuda con biblioteca que lee un fichero de configuracion

[snake_linux]

Hola a tod@s, pues ahora estoy haciendo una pequeña libreria que facilita la extracción de información de archivos de configuración, estos ficheros de configuración serían del tipo:

Código [Seleccionar] Expandir


campo1=valor1
campo2=valor2
#esto lo ignora, es solo un comentario
campo3=valor3
etc...


La función que se usaría tendría la sintaxys:

Código (c) [Seleccionar] Expandir


get_value("archivo.conf", "campo1", valor);


Es decir, le pasamos, como primer elemento, el archivo de configuración, como segundo elemento el campo del que va a extraer la información, y como tercer parametro, un array (creado en el programa que usa la librería) donde guardará el valor extraido (con strcpy).

El código es el siguiente:

Código (c) [Seleccionar] Expandir


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

int get_value(char [], char [], char []);
static void clear_array(char [], int);


int get_value(char file_conf[], char subject_i[], char value_i[]) {

FILE * file;

int file_line_size=50;
int subject_size=25;
int value_size=25;

char buffer[file_line_size];
char subject[subject_size];
char value[value_size];

file=fopen(file_conf, "r");
if (file==NULL) {

printf("Error al abrir el archivo\n");
return 1;

}

int x, y;
while (!feof(file))
{
clear_array(buffer, file_line_size);
clear_array(subject, subject_size);
clear_array(value, value_size);

fgets(buffer,file_line_size,file);
if (buffer[0]=='#') continue;
x=0;
for (x=0; x<subject_size; x++) {
if(buffer[x]=='=') break;
subject[x]=buffer[x];
}

for (x; x<file_line_size; x++) {

value[x]=buffer[x];

}
printf("el campo es %s\n", subject);
printf("el valor es %s\n", value);

if (strcmp(subject_i, subject)==0) { strcpy(value_i, value); return 0; }
}
printf("No existe el campo dentro del fichero\n");
return 1;
}

static void clear_array(char array[], int size) {

int x;

for(x=0;x<size;x++) {

array[x]='\0';


}

}


Los problemas que me dan son:

1) No extrae la información, ni siquiera me muestra el campo, la salida que me da es la siguiente:

el campo es
el valor es
el campo es
el valor es
el campo es
el valor es

2) El while hace un ciclo más del que debe, es decir, si el fichero de configuración tiene 3 lineas, el while da 4 vueltas... lo he comprobado con los printfs que he puesto en mitad del while.

Gracias de antemano.

Saludos.

[engel lex]

recomiendo lo siguiente...

1- siempre que se cargue la librería y se le pase un archivo, lee todo el archivo
2- todos los valores obtenidos guardalos en un array bidimensional, siendo el primer valor el nombre y el segundo el contenido
3- recomiendo siempre borrar los espacios en los bordes (trim) ya que la gente podría hacer

Código [Seleccionar] Expandir

#comentario con espacio por delante
variable = valor con un espacio de sobre aqui ->
para esto podría ser util isspace de la librería ctype.h

por otro lado tu error está probablemenet en la linea 46  donde value empieza a popularse desde el medio y sin discriminación

[snake_linux]

recomiendo lo siguiente...

1- siempre que se cargue la librería y se le pase un archivo, lee todo el archivo
2- todos los valores obtenidos guardalos en un array bidimensional, siendo el primer valor el nombre y el segundo el contenido
3- recomiendo siempre borrar los espacios en los bordes (trim) ya que la gente podría hacer

Código [Seleccionar] Expandir

#comentario con espacio por delante
variable = valor con un espacio de sobre aqui ->
para esto podría ser util isspace de la librería ctype.h

por otro lado tu error está probablemenet en la linea 46  donde value empieza a popularse desde el medio y sin discriminación

No entiendo lo del error en la linea 46, ¿me lo podrías explicar un poco más detallado?

[engel lex]

copio desde la linea 39, hablaré de los numeros de linea aquí

Código (c) [Seleccionar] Expandir

for (x=0; x<subject_size; x++) {
if(buffer[x]=='=') break;
subject[x]=buffer[x];
}

for (x; x<file_line_size; x++) {

value[x]=buffer[x];

}

supongamos que en el fichero tienes

Código [Seleccionar] Expandir

hola=mundo
el for de la linea 1 empieza en 0 y avanza hasta conseguir '=' es decir hasta 5
así que subject quedaría en memoria

Código [Seleccionar] Expandir

|h|o|l|a|0|0|0|0|0|....

cuando llegas a la linea 6 tienes un for que no inicia, sino que continúa

value está limpio y sería a nivel de memoria algo como

Código [Seleccionar] Expandir

|0|0|0|0|0|0|0|0|0|....

cuando lo llenas, empiezas a llenarlo desde la posición que quedó el ciclo anterior (en nuestro caso 5), así que

Código [Seleccionar] Expandir

|0|0|0|0|0|m|u|n|d|o|0|0....

cuando lo vas a imprimir, printf no lee cadena en en value, ya que empieza con fin de cadena (ese 0 que muestro en el string) y no imprime nada

value tienes que escribirlo desde 0, así que puedes usar x como base para buffer, pero no para value

[snake_linux]

copio desde la linea 39, hablaré de los numeros de linea aquí

Código (c) [Seleccionar] Expandir

for (x=0; x<subject_size; x++) {
if(buffer[x]=='=') break;
subject[x]=buffer[x];
}

for (x; x<file_line_size; x++) {

value[x]=buffer[x];

}

supongamos que en el fichero tienes

Código [Seleccionar] Expandir

hola=mundo
el for de la linea 1 empieza en 0 y avanza hasta conseguir '=' es decir hasta 5
así que subject quedaría en memoria

Código [Seleccionar] Expandir

|h|o|l|a|0|0|0|0|0|....

cuando llegas a la linea 6 tienes un for que no inicia, sino que continúa

value está limpio y sería a nivel de memoria algo como

Código [Seleccionar] Expandir

|0|0|0|0|0|0|0|0|0|....

cuando lo llenas, empiezas a llenarlo desde la posición que quedó el ciclo anterior (en nuestro caso 5), así que

Código [Seleccionar] Expandir

|0|0|0|0|0|m|u|n|d|o|0|0....

cuando lo vas a imprimir, printf no lee cadena en en value, ya que empieza con fin de cadena (ese 0 que muestro en el string) y no imprime nada

value tienes que escribirlo desde 0, así que puedes usar x como base para buffer, pero no para value

Gracias por la aclaración, me ha servido de mucho, he modificado un poco el código y ya obtiene tanto el campo como el valor, pero ahora tengo otro pequeño problema... ahora resulta, que solo lee una linea, es decir un campo y un valor e ignora el resto, te dejo el código que he editado:

Código (c) [Seleccionar] Expandir


y=0;
for (x=x+1; x<file_line_size; x++) {
if(buffer[x]=='\n') break;
value[y]=buffer[x];
y++;

}


¿Por que puede ser?

Gracias de antemano.

[engel lex]

Linea 52, tienes un return

[snake_linux]

Linea 52, tienes un return

Perdona, no se en que estaba pensando...

Muchísimas gracias por tu ayuda, marco como solucionado.