[C] scanf e ciclo do.while -> Non si aggiorna scanf!

[vincenzo2342]

Ciao a tutti.
Il ciclo incriminato è il do-while

void add_price(float * prezzi){
    float prezzo = 0.0;
    unsigned i = 0;
    int a;
  

   do{
   printf("Inserisci il numero dell'elemento a cui vuoi mettere il prezzo ed il prezzo:\n");
   printf("\n");
   scanf("%u%*c%f",&i,&prezzo);
   } while (!i && !prezzo);

Non funziona, va all'infito. Praticamente io mi aspetto (e sbaglio) che scanf, ad ogni iterazione chieda l'input.

[oregon]

Che vuol dire "non si aggiorna scanf" ? Che dati inserisci ?

E poi non ha senso usare il ! per il prezzo (dato che non è un valore intero). Semmai

while (!i || prezzo<=0.0);

[seby.91]

Il primo errore è sicuramente all'interno dello scanf... infatti alla varibile prezzo, non devi mettere '&' perchè è già passata per riferimento....
però non ho capiuto una cosa, quante volte si deve ripetere ciò che c'è nel do??

[oregon]

Originariamente inviato da seby.91
Il primo errore è sicuramente all'interno dello scanf... infatti alla varibile prezzo, non devi mettere '&' perchè è già passata per riferimento....

Guarda che prezzo e' una variabile locale, quindi va bene il '&'

prezzi è un puntatore apparentemente non usato

[vincenzo2342]

Sulla &: devo usarla perchè passo l'indirizzo della var locale, come dice oregon
prezzi lo userò una volta che sono soddisfatto dell'input. (faccio fare i controlli al programma e poi lo inserisco)

Il fatto è che l'input ideale è:

4 15.22

E se l'utente inserisce solo una delle due? il ciclo do-while dovrebbe dirgli di reinserire i dati.

All'inizio avevo pensato a:

   do{
   printf("Inserisci il numero dell'elemento a cui vuoi mettere il prezzo ed il prezzo:\n");
   printf("\n");

   } while (   scanf("%u%*c%f",&i,&prezzo) != 2);

perchè scanf ritorna teoricamente il numero di parametri letti.
Invece, una volta inserito l'input sbagliato, il programma esegui infinite volte il corpo del ciclo, senza richiedere all'utente di dargli in input. (Scanf, non dovrebbe entrare in azione ad ogni fine iterazione?).

Sono un po' confuso.

[seby.91]

Originariamente inviato da oregon
Guarda che prezzo e' una variabile locale, quindi va bene il '&'

prezzi è un puntatore apparentemente non usato

scusami hai ragione... avevo confuso prezzo con prezzi..

[oregon]

Adesso hai cambiato le condizioni del while ... così non ci capiamo ...

Stiamo parlando della prima versione del while (per cui ti ho dato il codice funzionante) o di un'altra ?

[XAlbeX]

Originariamente inviato da vincenzo2342
E se l'utente inserisce solo una delle due? il ciclo do-while dovrebbe dirgli di reinserire i dati.

comunque se a scanf gli passi "%u%*c%f" lui aspetterà ignorando gli a capo e spazi fino a che non ottiene i due valori.. quindi se l'utente digita soltanto un valore scanf non ritorna..

[n00b_new]

do{
printf("Inserisci il numero dell'elemento a cui vuoi mettere il prezzo ed il prezzo:\n");
printf("\n");

} while ( scanf("%u%*c%f",&i,&prezzo) != 2);

se ho capito...

do{
i=0;
prezzo = 0.0;
printf("Inserisci il numero dell'elemento a cui vuoi mettere il prezzo ed il suo prezzo:\n");
//formato 1 13.5
scanf("%u %f\n", &i,&prezzo);
}while(i!=0 || prezzo!=0.0);

anche se teoricamente scanf()!=2 dovrebbe andare bene...

[oregon]

Come ti avevo scritto

Originariamente inviato da n00b_new
}while(i!=0 || prezzo<=0.0);

dato che ci si aspetta un prezzo positivo.

anche se teoricamente scanf()!=2 dovrebbe andare bene...

No, a meno che tu non voglia accettare valori come

0 0