[C] strcpy, strcat, sprintf

[Dany_]

Grazie

[MItaly]

Tra l'altro, se devi concatenare tante stringhe la sprintf è anche più efficiente: se fai:

char buf[1024]="asd";
strcat(buf, var1);
strcat(buf, var2);
strcat(buf, var3);
strcat(buf, var4);
strcat(buf, var5);

ciascuna strcat deve scorrersi la stringa fino in fondo per sapere da dove iniziare a copiare; al contrario, una sprintf

char buf[1024];
sprintf("asd%s%s%s%s%s", var1, var2, var3, var4, var5);

tiene traccia internamente di dove è arrivata, per cui in ultima analisi risulta più efficiente.

[Dany_]

Era esattamente questo che volevo capire inizialmente! Grazie mille

[signoredeltempo]

@Dany_ occhio che sprintf di per sé può portare a buffer overflow. Il codice sopraesposto non controlla che le var* entrino in buf. Usa snprintf per maggiore sicurezza:

snprintf(buff, sizeof(buf), format, ...);

e controlla che abbia successo. man sprintf(3)

[Dany_]

Io ho fatto così:

char *buf;
buf = calloc(lunghezzaDesiderata + 1, sizeof(char));
sprintf(buf, ".....", ...);

Va bene lo stesso? Nel senso che avendo calcolato prima la dimensione di buf in base a quello che ci devo mettere dentro, quando faccio la sprintf dovrebbe essere tutto ok immagino!

[signoredeltempo]

Quella è parte della soluzione. O usi snprintf o controlli la lunghezza di ogni membro (cosa ripetitiva per tipi non-stringa).
Es:

char buf[5];
sprintf(buff, "%d", 123456789); // buffer overflow

[Dany_]

Cioè sarebbe più corretto così?

char *buf;
buf = calloc(lunghezzaDesiderata + 1, sizeof(char));
snprintf(buf, sizeof(buf), ".....", ...);

[signoredeltempo]

Sì. Citando man sprintf

Because sprintf() and vsprintf() assume an arbitrarily long string, callers must be careful not to overflow the actual space; this is often
impossible to assure. Note that the length of the strings produced is
locale-dependent and difficult to predict. Use snprintf() and
vsnprintf() instead (or asprintf(3) and vasprintf(3)).

[Dany_]

Facendo così però ho un warning che dice:
"argument to ‘sizeof’ in ‘snprintf’ call is the same expression as the destination; did you mean to provide an explicit length?"

[Dany_]

Grazie, gentilissimo