[C] Accodare stringhe da file di testo

[magnus]

Ciao Ragazzi,
ho la situazione seguente:

/* Apertura file SQL */
FILE *pfile;
char* testo;
char* testoQRY;

testo="select sysdate from dual ";

if((pfile = fopen(argv[1], "rb")) == NULL){
printf("Sorry, can't open %s", argv[1]);
return;
}

while(!feof(pfile))
{

if (fgets(testo,10,pfile)){
printf("%s", testo);
/*strncat(testoQRY,testo,10);*/
}
}

fclose(pfile);

printf("%s %s\n", "QUERY: ", testo);

Nella variabile testo ho solo una parte del file letto

Vorrei accodare man mano quello che leggo in testoQRY ma quell'istruzione commentata mi da:

24640 Memory fault(coredump)

Come mai?

Come posso accodare in una stringa il valore del testo letto?

Ciao!

[magnus]

Sostituendo l'istruzione:

char *testoQRY;

con:

char testoQRY[10000];

funziona... però così ho un limite di 10000 bytes. Come posso ovviare? :master:

[Xaratroom]

Prima di tutto dovresti specificare il linguaggio, che presumo sia c.
Ti consiglio di allocare dinamicamente la memoria che ti serve, di volta in volta.
Fatti un'idea con questi link:
http://www.cplusplus.com/reference/c...ib/calloc.html
http://www.cplusplus.com/reference/c...ib/malloc.html
http://www.cplusplus.com/reference/c...b/realloc.html
http://www.cplusplus.com/reference/c...dlib/free.html

Il problema alla base è che fai la strcat su un puntatore che punta a null.
Nel secondo caso il puntatore passato alla strcat non punta a null ma a 10000 char.
Per questo motivo non ti da errore.
Prima di strcat(testoQRY,testo,10); esegui una realloc.

[magnus]

Originariamente inviato da Xaratroom
Prima di tutto dovresti specificare il linguaggio, che presumo sia c.
Ti consiglio di allocare dinamicamente la memoria che ti serve, di volta in volta.
Fatti un'idea con questi link:
http://www.cplusplus.com/reference/c...ib/calloc.html
http://www.cplusplus.com/reference/c...ib/malloc.html
http://www.cplusplus.com/reference/c...b/realloc.html
http://www.cplusplus.com/reference/c...dlib/free.html

Il problema alla base è che fai la strcat su un puntatore che punta a null.
Nel secondo caso il puntatore passato alla strcat non punta a null ma a 10000 char.
Per questo motivo non ti da errore.
Prima di strcat(testoQRY,testo,10); esegui una realloc.

Uhm... giusto. Quindi provo ad allocare direttamente la memoria che mi serve.

[magnus]

Ho provato in questo modo:

/* Apertura file SQL */
   FILE *pfile;   
   char* testo;
   char *testoQRY;
   int cont;
   
   testo="select sysdate from dual ";
   
   if((pfile = fopen(argv[1], "rb")) == NULL){
       printf("Sorry, can't open %s", argv[1]);
       return;
   }

   while(!feof(pfile))
   {
          
     if (fgets(testo,10,pfile)){
         printf("%s", testo);
         cont++;
         testoQRY = realloc (testoQRY, cont * 10);
         strcat(testoQRY,testo);
     }
   }
      
   fclose(pfile); 

Ma mi da comunque:

26440 Memory fault(coredump)

Come mai? :master:

[oregon]

Perche' utilizzi il puntatore testo senza avere allocato lo spazio in memoria ...

[magnus]

Ho provato in questo modo ed adesso funziona:

/* Apertura file SQL */
   FILE *pfile;   
   char* testo;
   char* testoQRY;
   
   testo=(char*) malloc (1);;
   testoQRY=(char*) malloc (1);

   if((pfile = fopen(argv[1], "rb")) == NULL){
       printf("Sorry, can't open %s", argv[1]);
       return;
   }

   while(!feof(pfile))
   {
          
     if (fgets(testo,10,pfile)){
         
         printf("%s", testo);
         
         strncat(testoQRY,testo,10);
     }
   }
      
   fclose(pfile);
   
   printf("%s %s\n", "QUERY: ", testoQRY); 

Grazie a tutti

[Xaratroom]

Effettivamente qui non la usa http://www.cplusplus.com/reference/c...b/realloc.html

[oregon]

Originariamente inviato da Xaratroom
Effettivamente qui non la usa http://www.cplusplus.com/reference/c...b/realloc.html

Non usa ... cosa?

[oregon]

Originariamente inviato da magnus
Ho provato in questo modo ed adesso funziona:

No ... il programma ha errori gravi ...

Non puoi leggere 10 caratteri in memoria se ne hai allocato 1 ...