[C] Separatore di stringhe

[leadergl]

Raga ho scritto questo codice per separare/estrarre, qualora mi dovesse servire, una sottostringa da una stringa originaria formattata in un determinato modo!

Es:
ho la stringa:

ciao mamma come va|oggi piove|4|cavallo

dove il mio carattere separatore è |
questo codice mi dovrebbe permettere di estrarre a piacere una delle quattro frasi/parole che si trovano tra ogni |, ma ci deve stare qualche errore subdolo:

char *separa(char *stringa, int occorrenze)
{    
 int numero; // contatore globale
 int num;    // contatore locale
 char *destinazione=NULL;
 
    destinazione=(char *)malloc(sizeof(strlen(stringa))+1);
    numero=0;
    while (occorrenze>0)
    {
        num=0;
        while(stringa[numero]!='|')
        {
            destinazione[num]=stringa[numero];
            numero++;
            num++;
        }
        destinazione[num] = '\0';
        occorrenze--;
        numero++;
    }
    return destinazione;
}

Esempio:
[code]
int main()
{
char *temp=NULL;
char *origine="ciao mamma|2|prr|";

temp=separa(origine,2); //mi restituisce 2

system("PAUSE");
return 0;
}

se però eseguo più di una volta la chiamata quella funzione si verificano anomalie nel funzionamento del programma.....perchè? dove sbaglio?

[andbin]

Originariamente inviato da leadergl

se però eseguo più di una volta la chiamata quella funzione si verificano anomalie nel funzionamento del programma.....perchè? dove sbaglio?

Ciao, non ho provato il tuo codice ma a vista direi che qualcosa che non va c'è sicuramente. La prima cosa che mi è saltata all'occhio è:

destinazione=(char *)malloc(sizeof(strlen(stringa))+1);

la sizeof non ci vuole (altrimenti allochi sempre e solo 5 bytes!).

Comunque, io l'avrei fatto così (permettimi di postare una mia soluzione ):

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

char *separa (char *str, int index)
{
    char *s = str;
    char *p;
    char *ret;
    int   len;

    /*---- Cerca l'inizio della sotto-stringa ----*/
    while (index-- > 0)
    {
        if ((p = strchr (s, '|')) != NULL)
            s = p+1;
        else
            return NULL;
    }

    /*---- Cerca la fine (e lunghezza) della sotto-stringa ----*/
    if ((p = strchr (s, '|')) != NULL)
        len = p - s;
    else
        len = strlen (s);
   
    /*---- Alloca memoria e copia ----*/
    if ((ret = (char *) malloc (len+1)) != NULL)
    {
        strncpy (ret, s, len);
        ret[len] = '\0';
    }

    return ret;
}


int main (void)
{
    char *origine = "ciao mamma|2|prr";
    char *r;

    r = separa (origine, 0);
    printf ("[%s]\n", r);   /* Stampa "ciao mamma" */
    free (r);

    r = separa (origine, 1);
    printf ("[%s]\n", r);   /* Stampa "2" */
    free (r);

    r = separa (origine, 2);
    printf ("[%s]\n", r);   /* Stampa "prr" */
    free (r);

    return 0;
}

La funzione separa() sembra più lunga ma come l'ho fatta io forse è più chiara perchè è composta da 3 parti ben precise. E ritorna NULL se non trova la sotto-stringa o se malloc ha fallito.
Ciao.

[leadergl]

perfetto, ti ringrazio..la tua soluzione funziona a meraviglia

vorresti dare uno sguardo anche qui: http://www.hwupgrade.it/forum/showthread.php?t=1109604

ancora grazie