fmt e' un array di formati che passi alla funzione, e sui quali il tuo codice itera, controllando il tipo di parametri passati. Nel tuo caso tu controlli se fmt e' "s", che corrisponde secondo il tuo check a "stringa" (la tua printf stampa la stringa con il modifier %s). Ma poi quando chiami la funzione (nel primo caso), non passi "s" come primo parametro (che serve a dire al tuo check che i parametri seguenti sono stringhe). A mio parere Il tuo check e' inutile per quello che vuoi fare (tu vuoi sempre passare dei comandi di tipo stringa, quindi che senso ha controllare il formato ?).
Il secondo problema è che non effettui un ciclo di iterazione sui parametri effettivamente passati: il risultato è che viene stampato solo il primo parametro (o meglio il secondo, visto che il primo è il formato
fmt).
Il terzo problema è che non costruisci la tua lista parametri in modo tale da far capire al tuo codice quando i parametri sono effettivamente finiti: se il numero è variabile, il tuo codice non può inventarsi il numero!
Puoi semplificare e ottimizzare il codice così:
Codice: Seleziona tutto
#include <stdarg.h>
#include <stdio.h>
void funzione(char *fmt,...);
main (){
funzione("sdc" , "stringa", "ciao", 0);
funzione("s", "stringa2", 0);
}
void funzione(char *first, ...)
{
va_list ap;
char *s = first;
va_start(ap, first);
do{
printf("string %s\n", s);
} while( s = va_arg(ap, char *) );
va_end(ap);
}
Ovvero:
1) eliminiamo completamente il parametro inutile fmt (il formato sarà sempre e solo "stringa"), ed l'altrettanto inutile variabile char c, e sfruttiamo in maniera produttiva il primo parametro, che diventa il primo argomento della lista passata (qui, se hai necessità di ottimizzare (anche se di poco) ulteriormente il tutto, potresti passare come primo parametro un intero indicante il numero di elementi nella lista che segue, e quindi iterare con un
for-next su quello invece di fare un accesso alla memoria ad ogni iterazione del
do-while per verificare di aver raggiunto la fine)
2) ottimizziamo il codice facendo si che il primo parametro sia già un elemento della lista (first), e quindi iteriamo sulla lista dei parametri finché non incontriamo lo 0-byte finale che indica che i parametri sono finiti (in quel momento il check che fa ciclare il
do-while fallisce e si esce dal loop)
3) naturalmente, tale 0 byte deve essere incluso in ogni lista di argomenti (nota il modo in cui ho invocato
funzione).
Spero di essere stato abbastanza chiaro.
Ciao,
nuu
Learn to pause -- or nothing worthwhile can catch up to you.
