[C]doppio puntatore char in struct

[valakas92]

Sto scrivendo una funzione che ha lo scopo di separare gli argomenti da un url e salvarli separatamante in un'apposita struttura dati.
es:
http://miosito/index.html&usn=123&psw=abc

gli argomenti possono essere infiniti quindi uso una realloc allo scopo






typedef struct{
char **arguments;
}bho;


void call(bho *t,char *msg){


printf("arguments: %s\n",msg);
char *k = strtok(msg,"&"); //separo in chunk gli argomenti delimitati da &

int c=0;
char **tmp; //appoggio per realloc

while (k){


tmp=realloc(t->arguments,(c+2)*sizeof(char*));
if (tmp==0) printf("errore realloc\n");
else t->arguments=tmp;


t->arguments[c]=calloc(strlen(k)+1,1);

strcpy(t->arguments[c],k);

printf("loaded: %s\n",t->arguments[c]);
c++;

k=strtok(0,"&");

}
printf("chiudo in %d\n",c);

free(tmp);
free(k);


}


int main(){


bho *tmp=malloc(sizeof(bho));

char *msg=calloc(18,1);
strcpy(msg,"psw=ciao&name=abc");


call(tmp,msg);

printf("primo: %s\n",tmp->arguments[0]);
}





il codice funziona perfettamente se interrogo la struct dentro la funzione, ma se la interrogo nel main il primo record e NULL e gli altri vanno bene.. cos'è che nn va ?

[valakas92]

k=strtok(0,"&");


http://www.cplusplus.com/reference/cstring/strtok/
Il primo parametro è la stinga da troncare, se nella nuova chiamata (quella nel while) la stringa immessa è NULL allora continua a scandire la stringa già precedentemente inserita quindi è giusto.
Riguardo al controllo se il ritorno è NULL viene fatto nel while (k) che equivale a while (k != NULL), quindi mi estromette dal ciclo.

[valakas92]

riguardo a strtok posso assicurarti che funziona benissimo, il passaggio del puntatore a NULL se leggi il link dice che continua a scandire la stringa precedentemente passata.


se k ritorna NULL significa che ho letto immediatamente NULL, non entro nel while terminando

char *k = strtok(msg,"&");




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



int main(){

char *t=calloc(33,1);
strcpy(t,"12/34/56/78/90");

char *k=strtok(t,"/");

while (k){
printf("%s\n",k);
k=strtok(k+3,"/"); equivale a scrivere k=strtok(0,"/");
}

free(t);
}



per k hai ragione fare free() è sbagliato

[valakas92]

In realta il programma che sto facendo non è poi tanto semplice, questa è la parte preposta ad analizzare gli argomenti inclusi in una richiesta HTTP in un server web, gli argomenti poi verranno passati all'interprete per il linguaggio da server.
quindi se non uso strtok cosa mi proponi che abbia tale comportamento?
Provero al piu presto un debbugger, pero ne devo cercare uno sotto linux

[Daniels118]

Questa riga da sola non va bene:
tmp=realloc(t->arguments,(c+2)*sizeof(char*));
realloc si aspetta un puntatore ottenuto da malloc, realloc, oppure null, ma t->arguments non è inizializzato al primo loop.

[Daniels118]

Ho analizzato più a fondo il codice, ci sono un po' di cose che non vanno:
1) manca l'inizializzazione di t->arguments;
2) free(tmp) è sbagliato in quel contesto;
3) free(k) è sbagliato in quel contesto;
4) strtok(msg,"&") ritorna null se msg non contiene "&".
Detto questo, ti consiglierei di usare questa funzione (l'ho riscritta in maniera più compatta):
int split(char *s, char delimiter, int argc, const char **argv) {
int i = 0, c = 1;
argv[0] = s;
while (s[i] != 0) {
if (s[i] == delimiter) {
s[i] = 0;
argv[c++] = &s[i + 1];
if (c >= argc) break;
}
i++;
}
return c;
}
s è la stringa da dividere; viene modificata, se vuoi conservare l'originale devi farne una copia;
delimiter è il carattere da utilizzare come separatore;
argc è il numero massimo di sottostringhe da estrarre;
argv deve essere un array di char*, ciascun elemento riceverà il puntatore ad una sottostringa.
argc deve essere pari al massimo al numero di elementi in argv (almeno 2).
La funzione ritorna il numero di sottostringhe estratte, ovvero il numero di elementi valorizzati in argv.
Non viene fatta reallocazione, argv deve essere sufficientemente capiente, ma quanti parametri puoi inserire in un url? E' più efficiente sovradimensionarlo sprecando (temporaneamente) un po' di memoria che chiamare realloc per ogni parametro.
Se poi vuoi maggiore flessibilità puoi aggiungere l'allocazione dinamica.

[valakas92]

Grazie mille, davvero ottimo
Effettivamente è uno spreco riallocare parametro per parametro, avevo pensato di allocarne 5 alla volta ma ho preferito concentrarmi prima sul realloc

[Daniels118]

Prego :)
Piccola nota riguardo la funzione che ho postato: se il risultato è minore di argc vuol dire che la stringa è stata divisa tutta, se invece è uguale c'è la possibilità che l'ultima sottostringa contenga altri separatori; in questo caso basta richiamare la funzione sull'ultima sottostringa per dividere la parte restante. Inserendo questa condizione in un loop ed usando realloc su argv è possibile ottenere un algoritmo dinamico ed efficiente.