[c++] problema con assegnazione

[Infernal]

Salve, ho scritto una funzione che dovrebbe prendere in input una stringa e dare in output un array di stringhe contenenti ognuna una parola della stringa data. Ho scritto dovrebbe perche il programma compila ma poi dice segmentation fault.

con il debugger ho localizzato il probema: il programma si blocca quando tento di assegnare un valore a out cioè:

char** parolizza(char* frase)
{
    char** out = (char**) malloc(sizeof(char)*(strlen(frase)+1));
    int cont = 0;
    int a = 0;
    for (int i = 0; frase[i] != '\0'; i++)
    {
        if (frase[i] == ' ' || frase[i] == '\n')
        {
            if (a == 0) continue;            
            out[cont][a] = '\0';     //     <- QUI 
            cont++;
            a = 0;
        }        
        else
        {    
            out[cont][a] = frase[i];   //   <- QUI
            a++;
        }
    }
    cont++;
    out[cont][0] = '\0';      // <- E QUI
    return out;
} 

un grazie di cuore a chiunque sappia dirmi perchè si blocca.... sto usando debian etch e g++...

edit: i punti di domanda che ci sono nel codice sono degli \0

[gp.maraldo]

forse il problema è dovuto al fatto che malloc assegna un tot di memoria,
ma non è divisa in qualcosa tipo out[0][], out[1][], out[2][] ecc...

[mondobimbi]

Originariamente inviato da Infernal
Salve, ho scritto una funzione che dovrebbe prendere in input una stringa e dare in output un array di stringhe contenenti ognuna una parola della stringa data.

siccome parliamo di c++ sarebbe meglio che tu utilizzassi la classe string che ti permette una manipolazione più agevole. Inoltre dovresti allocare la memoria con il costrutto new.

Comunque seguendo il tuo frammento di codice affronterei il problema in questa maniera.
1 - calcoli il numero di parole nella tua frase (questo lo puoi fare contando gli spazi o i \n o i \t, ci sono funzioni che controllano proprio se una carettere è spazio, \n o \t.
Attenzione che se vuoi utilizzare il ciclo for che hai riportato
for (int i = 0; frase[i] != ''; i++)
devi correggerlo in

 for (int i = 0; frase[i] != '\0'; i++)

sempre che frase sia un puntatore non nullo altrimenti anche questa versione non funziona.

2 - allochi out come puntatore di puntatore a stringa, come hai fatto e gli dai come memoria il numero delle parole trovate
3 - Ripeti il ciclo e trovi le stringhe una per una. Per ogni stringa devi allocare la memoria per contenerne una copia
out [num_della_stringa] = (char *) malloc ( lunghezza della stringa n-essima + 1);
Copi la stringa nella nuova memoria ricordandoti di terminare con \0 ( puoi usare la strncpy ).

dovrebbe essere tutto
ciao
sergio

[Infernal]

ho capito... adesso lo faccio..... una cosa: provando a seguire il consiglio di gp.maraldo sono incappato in una cosa che mi sembra strana... praticamente, allocando memoria con malloc il valore di i (il contatore del for) cambia drasticamente.... (da quello che so penso che la malloc sia andata a scrivere anche su i....) ma se i è già allocata, come può la malloc scriverci sopra?
altra cosa (se hai tempo di spiegarmi) come viene deciso dove salvare le variabili in ram?

grazie mille di tutto

[mondobimbi]

è il sistema operativo che rende disponibile la memoria, lo fa a blocchi. L'unica possibilità che hai è di suggerire di memorizzare una variabile in uno dei registri della cpu, ma è solo un suggerimento al compilatore, che può non tenerne conto.
E' impossibile che malloc ti dia disponibile della memoria già allocata, probabilmente sei tu che scrivi oltre la memoria.
Posta il codice che hai scritto.

[Infernal]

ormai l'ho sovrascritto con la tua versione.... =) cmq fa lo stesso... grazie di tutto!