[C con Clase] problemas con la posicion del puntero en un fichero, funciones fread, frwite, fseek etc.
Albert Montolio
albert.montolio en gmail.com
Lun Mar 5 23:46:02 CET 2012
Hola Salvador, he aplicado el cambio que me has dicho, y sigue pasando lo
mismo, no lee la primera palabra, lee la segunda, y la ultima palabra la
escribe dos veces :S
la i esta inicializada a 0, y con la funcion fseek deberia ponerse el
contador al principio de todo, para leer la primera palabra, xo nose xk no
lo hace, me lee la segunda. y luego la ultima me la vuelve a repetir :S
gracias por las ideas para optimizarlo, una vez consiga k funcione me
dedicare a mejorarlo
#include <stdio.h>
#include <string.h>
struct t_palabra { // t_palabra es una estructura que
contiene castellano ingles y contadores de aciertos y fallos
char castellano[30];
char ingles[30];
int cont_aciertos;
int cont_errores;
};
int main (void)
{
FILE *fichero; // apunta al fichero donde se
guardaran las palabras y sus aciertos/fallos
struct t_palabra palabra; // definim la variable paraula.
char respuesta[50]; // l'usuari entra la seva resposta
int i; // recorrer el ficher
struct t__virt_palabra { // l'utilitzo per augmentar els
contadors cada cop k l'usuari practica 1 paraula. (nose una altre manera de
fer-ho)
char virt_castellano[30];
char virt_ingles[30];
int virt_cont_aciertos;
int virt_cont_errores;
};
struct t__virt_palabra virt_palabra; // definim la variable
virt_palabra
fichero = fopen ("datos.dat", "rb+"); // s'obre el ficher
datos.dat. rb+ es correcte????? al ficher hi ha tant caracters
// castellano,
ingles, com enters (els contadors) x tant te que ser binari, no?
if (fichero == NULL) // salta en cas
que no existeixi l ficher datos.dat
printf ("Error: No se ha podido abrir el fichero datos.dat.");
else
{
i=0;
strcpy (respuesta, "hola");
virt_palabra.virt_cont_aciertos = 0;
virt_palabra.virt_cont_errores = 0;
fseek(fichero, i*sizeof(struct t_palabra), SEEK_SET); // se pone el
cursor al inicio de todo xk i=0
while (!feof(fichero)) // preguntar al
usuari paraules, fins k sarrivi a la ultima paraula,
{ // despres
d'aquesta tindria que parar!!!!! i no ho fa :s, tornar a preguntar
// sempre la
ultima :(
if(fread(&palabra, sizeof(palabra), 1, fichero)) {
//fread(&palabra, sizeof(palabra), 1, fichero); // en la
primera iteracio, llegirm la primera paraula gracies a i=0
printf ("\nEscribe la traduccion de...: "); // preguntem
la paraula
printf ("\n%s: ", palabra.castellano);
gets(respuesta); // l'usuari
entra la paraula
// analitzar si l'usuari la encertat, o fallat, i incrementar
els contadors corresponents x cada paraula
if (strcmp(respuesta, palabra.ingles) == 0) //
compara resposta amb palabra.ingles si es correcte...
{
printf ("\nCorrecto!"); // felicitem
al usuari x cortesia, tenim que incrementar el contador d'encerts
// com que no es pot modificar nomes 1
apartat de la estructura paraula, tenim que introduir de nou tot...
// copiem a la variable virtual (nomes
serveix x traspar informacio) la paraula i la seva traduccio
strcpy (virt_palabra.virt_castellano,
palabra.castellano);
strcpy (virt_palabra.virt_ingles,
palabra.ingles);
// incrementem el contador una unitat
//palabra.cont_aciertos a la primera
iteració val 0 (aixi sa inicialitzat)
// i ho guardem a la virtual tb, x tal d
copiar tot again.
virt_palabra.virt_cont_aciertos =
palabra.cont_aciertos + 1;
// important: com que hem utilitzat
fread, el curso esta darrera la paraula llegida ara!!
// la variable virtual conte les noves
dades, es a dir, amb el contador incrementat,
// x tant, tenim que ficar el cursor al
inici de la paraula k estavem treballant, xk
// escrigui la informacio just a sobre
de la paraula antiga
// ara la i estava a la posicio dspres
de size of (equivalent a i=0), es a dir, dspres de la primera paraula
// en la primera iteració, ja que la i
val 0, amb aquesta comanda tornem a la posicio inical
fseek(fichero, i*sizeof(struct
t_palabra), SEEK_SET);
strcpy (palabra.castellano,
virt_palabra.virt_castellano);
strcpy (palabra.ingles,
virt_palabra.virt_ingles);
palabra.cont_aciertos =
virt_palabra.virt_cont_aciertos;
// i podem sobreescriure la paraula que
voliem
fwrite (&palabra, sizeof(palabra), 1,
fichero);
// important, el cursor torna a estar
darrera de la primera paraula (en primera iteracio)
} // sino es correcte...
else
{
printf ("\nIncorrecto.");
virt_palabra.virt_cont_errores =
palabra.cont_errores + 1; // mateixa estrategia
strcpy (virt_palabra.virt_castellano,
palabra.castellano);
strcpy (virt_palabra.virt_ingles,
palabra.ingles);
fseek(fichero, i*sizeof(struct
t_palabra), SEEK_SET);
strcpy (palabra.castellano,
virt_palabra.virt_castellano);
strcpy (palabra.ingles,
virt_palabra.virt_ingles);
palabra.cont_errores =
virt_palabra.virt_cont_errores;
fwrite (&palabra, sizeof(palabra), 1,
fichero);
// important, ara el cursor esta despres
de la paraula treballada
}
// hem acabat la primera iteracio
i++; // incrementem la i, per tal danar a la propera paraula.
fseek(fichero, i*sizeof(struct t_palabra), SEEK_SET); //
coloquem el cursor a la seguent posicio, es a dir, a la posicio 1,
// es a
dir, a la segona paraula. nem cap a dalt, ja que al ficher
//
encara hi ha paraules, i tooorneeem a fer el while.
}
else {
printf ("\nfin!");
}
}
fclose (fichero);
}
fflush(stdin);
printf("\n\nPulse Intro para finalizar...");
getchar();
}
El 5 de marzo de 2012 23:12, Salvador Pozo <salvador en conclase.net> escribió:
> El pasado 2012-03-05 21:30:06, Albert_Munich escribió:
>
> A> Hola, he intentado siguiendo tus indicaciones solucionar el problema, he
> A> creado un contador, que recorre el fichero, y ahora lee todo ordenador,
> a
> A> mas con la funcion feof nueva ya no se va hasta el infinito. el problema
> A> ahora esta en la primera palabra que lee y la ultima.
> A> la primera palabra no la lee, va directo a la segunda, y la ultima
> palabra
> A> no la repite infinitamente, xo la repite 1 vez mas, es decir, la pone
> dos
> A> veces. no se como solucionarlo, ya que el contador con i=0 indica
> clarament
> A> que lea la primera palabra, y con la funcion feof, en teoria, no
> tendria k
> A> volver a entrar en el while, por tanto, no entiendo porque repite la
> ultima
> A> palabra antes de acabar.
>
> Hola:
>
> Esta vez es más fácil. El error está aquí:
> ----8<------
> if(fread(&palabra, sizeof(palabra), 1, fichero)) {
> fread(&palabra, sizeof(palabra), 1, fichero);
> ...
> ----8<------
>
> El fallo es leer dos veces la palabra, con la lectura del interior del
> "if" es suficiente. En la primera iteración, la segunda lectura anula la
> lectura de la primera palabra, por eso no sale la primera.
>
> En la última iteración la segunda lectura falla, pero se mantiene la
> palabra leída en la primera (la del if).
>
> Puedes compactar mucho tu programa:
> - No necesitas la estructura t__virt_palabra para nada. Ya que lees una
> estructura "palabra" desde el fichero, para actualizar el fichero bastará
> con incrementar palabra.errores o palabra.aciertos, y volver a usar la
> misma "palabra" para escribirla en el fichero, en la posición en la que
> estaba. Esto te evita asignar valores que no cambian: las palabras y el
> contador que no modificas.
> - Evita repetir el código, por ejemplo, cuando verificas si el usuario ha
> acertado o no la respuesta, muchas de las sentencias son comunes tanto al
> if como al else, saca esas sentencias del bloque if..else.
> - No necesitas asignar un valor inicial a la respuesta, ya que la primera
> vez que se evalúa es después de leerla desde el teclado.
> - Evita usar la función gets, en su lugar usa fgets, por ejemplo,
> fgets(respuesta, 30, stdin);
>
> Hasta pronto.
>
> --
> Salvador Pozo (Administrador)
> mailto:salvador en conclase.net
> _______________________________________________
> Lista de correo Cconclase Cconclase en listas.conclase.net
> http://listas.conclase.net/mailman/listinfo/cconclase_listas.conclase.net
> Bajas: http://listas.conclase.net/index.php?gid=2&mnu=FAQ
>
------------ próxima parte ------------
Se ha borrado un adjunto en formato HTML...
URL: <http://listas.conclase.net/pipermail/cconclase_listas.conclase.net/attachments/20120305/6b5f4790/attachment.html>
Más información sobre la lista de distribución Cconclase