[C] Problema con funzione malloc

[cldbaldini]

Salve a tutti,
come potete vedere questo esercizio chiede di inserire un determinato numero di stringhe e di concatenarle. Il problema è che non ci sono errori di compilazione ma nell'esecuzione a volte restituisce la stringa concatenata correttamente, altre volte con numeri e/o simboli tra le stringhe e altre volte si blocca, il tutto varia (a quanto pare) in base alle stringhe che l'utente inserisce. Qualcuno sa dirmi il perché? Grazie

S.O.: Windows 7 Professional; Compilatore: Dev-C++ 4.9.9.2

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

const int LEN = 1000;

main(){
       int N=0;
       int i=0;
       char buffer[LEN];

       char *stringa;
       stringa = (char *) malloc(sizeof(char));
       *stringa = '\0';

       while(N<=0){
             printf("QUANTE STRINGHE VUOI INSERIRE? ");
             scanf("%d", &N);
       }
       
       printf("\nINSERISCI LE STRINGHE:\n\n");
       for(i=1;i<=N;i++)
       {
            *buffer='\0';
            printf("STRINGA NUMERO %d: ",i);
            scanf("%s", buffer);
            stringa = realloc(stringa, (strlen(stringa) + strlen(buffer)));
            stringa = strcat(stringa,buffer);
            //printf("\n%s\n", stringa);
       }
       
       printf("\nSTRINGHE CONCATENATE: '%s'", stringa);
       
       
printf("\n\n");
system("PAUSE");
free(stringa);       
}

[AllerITA]

Guarda che hai sbagliato il Titolo del thread quello che stai usando è C puro, non C#.
Io non userei realloc in modo così disinvolto, la funzione tende a non funzionare se non trova spazio a sufficenza e
ritorna un puntatore NULL.
Guarda le funzionalità di realloc.
io farei così:

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

const int LEN = 1000;

main(){
       int N=0;
       int i=0;
       char buffer[LEN];

       char *stringa;
       stringa = (char *) malloc(sizeof(char));
       *stringa = '\0';

       while(N<=0){
             printf("QUANTE STRINGHE VUOI INSERIRE? ");
             scanf("%d", &N);
       }
       
       printf("\nINSERISCI LE STRINGHE:\n\n");
       for(i=1;i<=N;i++)
       {
            *buffer='\0';
            printf("STRINGA NUMERO %d: ",i);
            scanf("%s", buffer);
 
           char * tmpstringa=(char*) realloc(stringa, (strlen(stringa) + strlen(buffer));
           if(tmpstringa==NULL)
            {
                        printf("\nERRORE:Non e' possibile ridimensionare la stinga");
                       system("PAUSE");
                       break;

            }
            stringa=tmpstringa;
            stringa = strcat(stringa,buffer);
            //printf("\n%s\n", stringa);
       }
       
       printf("\nSTRINGHE CONCATENATE: '%s'", stringa);
       
       
printf("\n\n");
system("PAUSE");
free(stringa);       
}

penso così dovrebbe andare meglio.

[Vincenzo1968]

Quello è codice C non C#.

[sottovento]

Manca lo spazio per il terminatore.
Inoltre i controlli di allocazione andrebbero fatti anche sulla prima malloc().
Infine, e' meglio usare fgets() per leggere lo standard input, cosi' da controllare che l'utente non inserisca un numero eccessivo di caratteri

[AllerITA]

Ora che me lo fai notare hai ragione il realloc e' fatto su buffer e quindi solo sulla lunghezza della stringa e questo valore non comprende il terminatore quindi il realloc va rivisto così:

            ...Codice....
            int len=strlen(stringa) + strlen(buffer)+1
            char * tmpstringa=(char*) realloc(stringa, len);
            if(tmpstringa==NULL)
            {
                        printf("\nERRORE:Non e' possibile ridimensionare la stinga");
                       system("PAUSE");
                       break;

            }
            stringa=tmpstringa;
            stringa = strcat(stringa,buffer);
            *(stringa+len)='\0';
           ...Codice...