Printable View
Hi there! I'm working with strcat but I have some problems. I want to make a function to read a line from a file. At the begining, I don't know how long the line is, because of that, I want to start with a small array and read in small slices. These slices I will attach to a final line containing the line of the text. If I use an statically reserved array 'aux', this works properly. But if I use calloc to reserve memory for that array it strcat doesn't work. Anybody knows WHY???
THANK YOU!
Yes I do. There is a bug in your code. If you'd post some code we might be able to find that bug.Quote:
Originally Posted by bartleby84
Kurt
Here is the code SORRY :P
the variable 'longitud' is a globlal variable that is use to determine the minimum size of reading buffer. If there's something you don't understand ask for it.Code:
char *leer(char *line,FILE *fp){
char *aux;
int length,i;
char *r;
aux = calloc(longitud,sizeof(char));
r=fgets(aux,longitud,fp);
if(r!=NULL){
strcpy(line,aux);
largo=longitud;
while (strrchr(aux,10)==NULL){
r=fgets(aux,longitud,fp);
if (r==NULL)
break;
largo = largo+longitud;
realloc(linea,largo*sizeof(char));
strcat(linea,aux);
}
}
free(aux);
return r
}
I can't be shure because you call realloc on a variable linea but you pass line into that function. If your intention is to modify the passed in pointer then this doesn't work this way because you modify a copy. You have to pass a pointer to a pointer.
Kurt
But if I statically reserve memory for aux, all works right! This is the real problem. I realloc line bcause I want to keep in that array (that is reserved out of this function) the entire line.
Code:char *leer(char *line,FILE *fp){
char aux[10];
int length,i;
char *r;
longitud=10;
r=fgets(aux,longitud,fp);
if(r!=NULL){
strcpy(line,aux);
largo=longitud;
while (strrchr(aux,10)==NULL){
r=fgets(aux,longitud,fp);
if (r==NULL)
break;
largo = largo+longitud;
realloc(line,largo*sizeof(char));
strcat(line,aux);
}
}
return r
As I said
Does not change the address of the passed in pointer.Code:realloc(line,largo*sizeof(char));
I don't understand why you say it works if you don't dynamically allocate aux.
Post a compilable example that shows how you use that function.
kurt
EDIT: Yes I do unterstand why it appears to work if you use a statically allocated buffer for aux. If you use a static buffer the realloc propably just expands the allocation and returns the same pointer. if you dynamically allocate aux then realloc cannot expand that buffer and moves the data to another location.
Here is the code you can try first with aux dinamically allocated and then statically.
Code:#include <stdio.h>
#include<unistd.h>
#include <string.h>
#include <stdlib.h>
int longitud;
// You have to add the leer function!!!!
main(){
static char *arg[]={"write here the file you want to create"};
static char *argori[]={"write here the file you want to read from"};
char *linea;
FILE *fp, *fpori;
char *r;
fpori = fopen(argori[0],"r");
fp = fopen(arg[0],"w");
printf("Set the variable longitud: ");
scanf("%d",&longitud);
linea = calloc(longitud,sizeof(char));
r = leer (linea,fpori);
while (r!=NULL){
fputs(linea,fp);
free(linea);
linea=calloc(longitud,sizeof(char));
r=leer(linea,fpori);
}
fclose(fp);
fclose(fpori);
free(linea);
}
But why happen that if I allocate aux and the pointer I use in reallocate is another one?Quote:
Originally Posted by ZuK
You allocate some memory for line then you allocate the buffer aux, your memorymanager will propably allocate the space for aux just after the space for line. When you now call realloc then the memorymanager cannot expand line ( because the spcae is occupied by aux ) so it has to move the data to another location and return anoter pointer that you assigne to line in leer, but this line is not the line from main ( it's a copy ).Quote:
Originally Posted by bartleby84
Kurt
Thank you very much for your clear explanation. I'm gona work on that.
Split from this OLD THREAD
Read the forum rules.