C - leggere una stringa di lunghezza indefinita da stdin

[p002244]

Ciao ragazzi, dovrei leggere una stringa di lunghezza indefinita da stdin e salvarla in un'array.


Ho dichiarato l'array usando "malloc" per allocare una quantità di memoria iniziale e vorrey usare la funzione "realloc" per poter riallocare la memoria nel caso in cui la lunghezza dichiarata inizialmente non dovesse bastare.

Per la lettura dei caratteri ho usato la funzione "getchar" all'interno di un ciclo while, nel quale controllo la lunghezza raggiunta dalla stringa per mezzo di un contatore e, nel caso in cui la lunghezza raggiunta supera quella dichiarata richiamo la realloc ma così come è il codice ora non funzione e mi ritorna un errore.

Ho fatto deverse prove ma nessun miglioramento.

Qualcuno sa aiutarmi?

[MItaly]

Posta il codice che hai scritto.

[p002244]

void get_string()
{

int i = 0;
char *word = malloc(sizeof(char) * 4);


printf ("Scrivi la parola:\n");

do
{
word[i] = getchar();
i++;
if ((i % 4) == 0)
{
if (!(realloc (word, sizeof(char) * 4)))
{
printf("Errore nella riallocazione della memoria. Il programma sarà chiuso.");
}
}

} while (word[i-1] != '\n');

word[i] = '\0';


printf("\n%i\n%s\n\n",i,word);


}

[MItaly]

realloc (word, sizeof(char) * 4)

Qui stai riallocando la stessa quantità di memoria che era già allocata, mentre devi ovviamente allocarne di più.

[MItaly]

(doppio)

[p002244]

Ho provato a raddoppiare il numero ma mi restiturisce sempre lo stesso errore.

*** glibc detected *** c_prog/pe/prog_estivo_00/ordina: double free or corruption (fasttop): 0x0915c038 ***
======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6[0xb7f29454]
/lib/tls/i686/cmov/libc.so.6(cfree+0x96)[0xb7f2b4b6]
c_prog/pe/prog_estivo_00/ordina[0x804857d]
c_prog/pe/prog_estivo_00/ordina[0x8048602]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe5)[0xb7ed0685]
c_prog/pe/prog_estivo_00/ordina[0x8048481]
...
...
più una serie di riche in cui mi elenca le posizioni in cui ho salvato il sorgente.

Ho letto da qualche parte di liberare il puntatore prima di riallocare la memoria con (free) ma l'errore è sempre questo.

[oregon]

Ma il codice completo di main qual e'? (Usa i tag code altrimenti si fa fatica a comprenderlo e a risponderti)

[p002244]

Questa è la main:

int main ()
{

	char 	*word1 = malloc(sizeof(char) * 15),
			*word2 = malloc(sizeof(char) * 15);

	get_string();


	return (0);
}

Questa è la funzione che vorrei usare per leggere la stringa:

void get_string()
{

	int i = 0;		/* Indice ciclo for */
	char *word = malloc(sizeof(char) * 4);


	printf ("Scrivi la parola:\n");

	do
	{
		word[i] = getchar();
		i++;
		if ((i % 4) == 0)
		{
			if (!(realloc (word, sizeof(char) * 8))) /* dopo suggerimento di MItaly */
			{
				printf("Errore nella riallocazione della memoria. Il programma sarà chiuso.");
			}
		}
		
	} while (word[i-1] != '\n');

	word[i] = '\0';


	printf("\n%i\n%s\n\n",i,word);
}

Ps. Grazie per la dritta sul codice!!

[MItaly]

Il punto è che devi aumentare ad ogni iterazione la quantità di memoria allocata; ecco una mia soluzione (che evita la getchar in favore della fgets):

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

char * GetLine(FILE * File);

int main(void)
{
    char * stringa;
    stringa = GetLine(stdin);
    printf("Hai digitato: %s\n",stringa);
    free(stringa);
    return 0;
}

char * GetLine(FILE * File)
{
    const size_t blockSize=128;
    size_t actualSize = blockSize;
    char * ret = NULL;
    for(;;)
    {
        ret = realloc(ret, actualSize);
        fgets(ret+actualSize-blockSize, blockSize+1, File);
        if(ret[strlen(ret)-1]=='\n')
            break;
        actualSize+=blockSize;
    }
    return ret;
}

.

[oregon]

Ame sembra che la realloc debba essere scritta cosi'

if (!(word = (char *)realloc (word, sizeof(char) * (i+4))))

E la word1 e word2 non sembrano essere utilizzate ...