Pequeño error en ficheros

[DickGumshoe]

Hola.

Estoy intentando leer las n últimas líneas de un fichero, para, posteriormente, almacenarlas en otro.

Por ejemplo, si mi fichero tiene:

Código:

hola
adiós
duda
ficheros
punteros
estructuras

y meto el número 3 por consola, después de introducir el nombre del fichero, me debería leer:

ficheros
punteros
estructuras

Y, sin embargo, me lee:

Código:

os
punteros
estructuras

Mi código es:

Código C:

#include<stdio.h>
#include<stdlib.h>
 
typedef struct{
 
int *V;
int lineas;
 
}Estructura;
 
char* InicializarCadena();
Estructura Contar(FILE *fp);
void Almacenar(FILE *fp1,FILE *fp2,Estructura aux,int n);
 
int main (){
 
FILE *fp1;
FILE *fp2;
 
int n;
 
char *fich,c;
 
Estructura aux;
 
printf("Introduzca el nombre del fichero: ");
 
fich=InicializarCadena();
 
 
printf("Introduce un numero: ");
scanf("%d", &n);
 
 * *fp1=fopen(fich, "r");
 * *fp2=fopen("Trabajo.txt", "w");
 
 
 * *if(fp1==NULL){
 * *printf("ERROR");
 * *system("pause");
 * *exit(1);
 * *}
 
 * *aux=Contar(fp1);
 
 
 * *Almacenar(fp1,fp2,aux,n);
 
 free(aux.V);
 free(fich);
 fclose(fp1);
 fclose(fp2);
 
system("pause");
}
 
 
char* InicializarCadena(){
 * int i = 0;
 * char c, *cad;
 
 * cad = (char*) malloc(sizeof(char));
 
 * while((c = getchar())!= '\n'){
 * * *cad[i] = c;
 * * *i++;
 * * *cad = (char*)realloc(cad, (i + 1)*sizeof(char));
 * }
 * cad[i] = '\0';
 
 * return cad;
}
 
Estructura Contar(FILE *fp){
 
int i=0;
 
Estructura aux;
 
char c;
 
aux.lineas=0;
 
aux.V=(int*) malloc(sizeof(int));
 
while(feof(fp)==0){
 
c=fgetc(fp);
i++;
 
if(c=='\n'){
aux.V=(int*) realloc(aux.V,(aux.lineas+1)*sizeof(int));
aux.V[aux.lineas]=i;
 
aux.lineas++;
}
}
aux.lineas++;
return(aux);
}
 
void Almacenar(FILE *fp1,FILE *fp2,Estructura aux,int n){
 
char c;
 
fseek(fp1,aux.V[aux.lineas-n],SEEK_SET);
 
 
 
while(feof(fp1)==0){
 
c = fgetc(fp1);
fputc(c,fp2);
printf("%c", c);
 
}
 
}
 

¿Qué hago mal?

Muchas gracias.

Saludos.

[cesar_casla]

Hola! tratando de ayudart con este problema, vi algo q la verdad no tengo idea de porque pasa. Antes de responderte, primero necesito que ejecutes este codigo

Código C:

#include <stdio.h>
 
int main() {
 
    char c;
    int i=0, j=0;
    FILE *fp1=fopen("archivo.txt", "r");
 
 
    while(j<8){
    c=fgetc(fp1);
    i=ftell(fp1);
 
    printf("i:%d %d %c\n",i,c, c);
    j++;
 
}
    system("PAUSE");
    return 0;
}
 

El archivo.txt es el q vos tenes con: hola, adios........ Vas a ver q cuando lo ejecutas pasa algo(q hasta 1 min no sabia) interesant. Cuando ftell devuelve 4 se imprime 97 y el caracter 'a', hasta aca bien. Pero en el siguiente ciclo vas a notar que ftell no vale 5(como deberia ser) vale 6 y al lado se imprime el 10 y el salto de linea(ENTER)
La verdad nose xq despues de imprimir 'a', ftell devuelve 6 ¿y el 5??? es como que despues de la 'a' de "hola" hay dos "caracteres": uno nose(seria el 5) y el otro es el enter(el 6)

Sabiendo esto, te dejo modificado la parte del codigo q tendrias mal:

Código C:

while(feof(fp)==0){
 
c=fgetc(fp);
i++;
 
if(c=='\n'){
            i++;// HAY Q AGREGAR UN INCREMENTO ADICIONAL
aux.V=(int*) realloc(aux.V,(aux.lineas+1)*sizeof(int));
aux.V[aux.lineas]=i;
 
aux.lineas++;
}
}
//aux.lineas++; este incremento estaria demas
return(aux);
}
 

Agregue un i++ dentro del if, por lo mencionado anteriormente. Comente aux.lineas++; ya q esta demas. Enrealidad no esta demas solo q despues en

Código C:

fseek(fp1,aux.V[aux.lineas-n],SEEK_SET);
 

ademas de restarle n tambien tendrias q restarle 1. Acordate q los arreglos van de 0 a n-1
Otra cosa para arreglar es

Código C:

while(feof(fp1)==0){
 
c = fgetc(fp1);
fputc(c,fp2);
printf("%c", c);
}
 

tendria q ser

Código C:

if((c = fgetc(fp1))!=EOF) // tenes q comprar q no sea EOF
fputc(c,fp2);
 

ya q vas a guardar un caracter demas(basura en fp2). Tenes q poner ese if ya que en cierto modo estas haciendo mal el ciclo while, primero se lee del archivo luego se comprueba q no sea fin de archivo y se vuelve a leer ....ej:

Código C:

c=fgetc(fp1); // primero leo del archivo
while(feof(fp1)==0){ // compruebo
 
fputc(c,fp2); // guardo en fp2
printf("%c", c);
c=fgetc(fp1);  // vuelvo a leer
}
 

Bueno hasta aca llega lo q pude deducir(bastante interesant aunq me qdo un vacio ya q no entendi xq pasa eso), espero q alguien pueda responder el porque de ftell salta de 4 a 6

Saludos

PD: te respondo en este foro tambien xD por si es q alguien me ayuda a mi pregunta

[DickGumshoe]

¡Muchas gracias! xD

A ver si nos pueden decir por qué pasa eso.

Saludos.

[starfix]

Código C++:

#include <stdio.h>
 
int main() {
    char c;
    int i=0, j=0;
    FILE *fp1=fopen("archivo.txt", "r");
 
    while(j<8){
 
    c=fgetc(fp1);
    i=ftell(fp1);
 
    printf("i:%d j:%d %d %c\n",i,j,c, c);
 
    j++;
}
 
    system("PAUSE");
    return 0;
}
 

prueben ese codigo y van a ver tambien el valor de la variable j.

segun el razonamiento que segui (puedo estar equivocado), es que cuando fgetc lee el ultimo caracter de la cadena mueve el puntero 1 lugar hacia la derecha, es decir, se encuentra con el caracter de fin de cadena '\0' y lo ignora avanzando hacia el siguiente caracter y lo lee.... entonces, en ese momento, fgetc lee 2 caracteres, ignorando 1, por eso cada vez que se encuentra con el '\0' aumenta en 2 la posicion del puntero dentro del archivo, que es el numero que devuelve ftell...

espero haberme explicado bien :).
saludos

[sam90]

A me funciona bien...no me ocurre el salto que vos decis.
Si lo estan probando en windows, el salto de pagina son dos caracteres, uno es \r y el otro es el \n....

El \r conocido como retorno de carro, lo que hace es volver al principio de la linea...sin hacer un salto de linea....asi que cuando el printf con ese caracter volves al principio. Luego lees el salto de linea pisa la linea que antes habias escrito..... Para que ven como funciona prueben este ejemplo:

for(i=0;i<10;i++) printf("\rNumero de I: %d", i);

Veran que solo se ve una linea. Saludos

[DickGumshoe]

Muchas gracias a los dos, ya lo he entendido.

Saludos.

[cesar_casla]

Hola! gracias x responder a los dos pero aun tengo una pequeña duda.. Con respecto a lo q decis starfix, en primer lugar lo habia pensado de esa manera, pense q fgetc tomaba el '\0' pero despues pense que si toma '\0' deberia imprimir con el formato %d el decimal cero, ya q '\0' equivale a ese numero(sino me equivoco). De todas formas ese caracter nose si se guarda en un archivo

Sam90, probe el programa en linux y es verdad!!! ftell no hace ese salto :) (todo por no qrer abrir el virtualbox y ejecutarlo desde linux)... La duda es, siguiendo la logica de q todo es un caracter, no deberia imprimirse el decimal 13(retorno de carro)??? o es q fgetc no lo toma??
Por lo q decis entiendo esto: supongamos q fgetc toma 'a' de "hola", despues toma '\r' y el printf hace el retorno de carro, luego fgetc toma '\n' y se imprime un salto de linea. Pero esto se debe a q estoy usando %c, pero q pasa si quiero imprimir los caracteres en decimal, no deberia imprimir 97 13 y 10???

En fin suelo compilar mas en linux q windows, es por eso q nunca habia tenido este problema. De todas formas sirvio como aprendizaje sobre windows

Saludos

[sam90]

Esto no es un problema..... Si imprimes en decima te tiene uqe salir un 10 (\r) y despues un 13 \n... proba de casar del prinft la impresion del caracter %c y vas a ver....

El salto de linea se imprime pero se traduce como tal...fijate que hay una linea en blanco.

Linux considera que un salto de linea es solo el \n...de ahi viene el problema que cuando ahcesu n archivo con lin ux cuando lo ves con windows se ve mal..(si lo abris con el block de notas, otro programa un poco mejorado se da cuenta de eso)

Por ultimo un fichero del tipo texto nunca vas a encontrar un '\0'... El \0 es un caracter que se usa para indicar el fin de una cadena.... Pero es solo una convencion dentro del lenguaje. Uno podria redefinir toda las funciones de string.h para que sea otro caracterer.

Saludos

[cesar_casla]

Listo ya me quedo clarisismo, ahora lo entendi bien. Y si ya me habia pasado que un archivo creado en linux cuando lo abria en windows se veia mal, pero nunca habia pensado el porque. Muchas gracias

Saludos