Parsing input con scanf in C

Question

Ho avuto un sacco di problemi cercando di capire come usare scanf(). Sembra funzionare bene con numeri interi, essendo abbastanza diretto scanf("%d", &i).

In cui sono in esecuzione problemi con scanf() nei loop che tentano di leggere l'input. Per esempio:

do { 
    printf("counter: %d: ", counter); 
    scanf("%c %c%d", &command, &prefix, &input); 
} while (command != 'q'); 

1. Quando entro in una validamente ingresso strutturato come c P101, sembra ciclo di nuovo davanti a me chiedere conferma. Questo sembra accadere anche con un singolo:

   scanf("%c", &c) 
   

   in un ciclo while. Farà il ciclo due volte prima di richiamarmi di nuovo. Che cosa lo rende loop due volte e come lo interrompo?

2. Quando inserisco una quantità inferiore di input che a livello di programmazione non avrebbe un altro carattere o numero come q, premere Invio sembra richiedere di inserire altro. Come posso ottenere scanf() per elaborare le voci sia a singolo che a doppio carattere?

Answer 1

Quando si immette "c P101", il programma riceve effettivamente "c P101\n". La maggior parte degli specificatori di conversione salta gli spazi bianchi iniziali inclusi i newline ma non lo è lo %c. La prima volta intorno a tutto fino a "\n" viene letto, la seconda volta il "\ n" viene letto in command, "c" viene letto in prefix e "P" è lasciato che non è un numero quindi la conversione fallisce e "P101\n" rimane nello stream. La prossima volta che "P" viene memorizzato nel comando, "1" viene memorizzato nel prefisso e 1 (dal rimanente "01") viene memorizzato in input con "\n" ancora sullo streaming per la volta successiva. Puoi risolvere questo problema inserendo uno spazio all'inizio della stringa di formato che salterà qualsiasi spazio bianco iniziale, inclusi i newline.

Una cosa simile sta accadendo per il secondo caso, quando si entra "q", "q\n" viene inserito nel flusso, la prima volta intorno al "q" viene letto, la seconda volta la "\n" viene letto , solo la terza chiamata è la seconda lettura "q", è possibile evitare nuovamente il problema aggiungendo un carattere spazio all'inizio della stringa di formato.

Un modo migliore per farlo sarebbe utilizzare qualcosa come fgets() per elaborare una riga alla volta e quindi utilizzare sscanf() per eseguire l'analisi.

Answer 2

Per la domanda 1, ho il sospetto che hai un problema con il printf(), dal momento che non v'è alcuna terminazione "\ n".

Il comportamento predefinito di printf consiste nel buffer di output fino a quando non ha una linea completa. Questo a meno che non si modifichi esplicitamente il buffering su stdout.

Per la domanda 2, hai appena raggiunto uno dei maggiori problemi con scanf(). A meno che il tuo input corrisponda esattamente alla stringa di scansione che hai specificato, i tuoi risultati non saranno come quelli che ti aspetti.

Se si dispone di un'opzione, si otterranno risultati migliori (e meno problemi di sicurezza) ignorando scanf() e facendo la propria analisi. Ad esempio, utilizzare fgets() per leggere un'intera riga in una stringa, quindi elaborare i singoli campi della stringa —, magari utilizzando anche sscanf().

Answer 3

scanf fa schifo, per le ragioni che hai già scoperto. È molto meglio usare qualcosa per ottenere l'intera linea e poi analizzarla.

Answer 4

È davvero rotto! Non sapevo che

#include <stdio.h> 

int main(void) 
{ 
    int counter = 1; 
    char command, prefix; 
    int input; 

    do 
    { 
     printf("counter: %d: ", counter); 
     scanf("%c %c%d", &command, &prefix, &input); 
     printf("---%c %c%d---\n", command, prefix, input); 
     counter++; 
    } while (command != 'q'); 
} 

counter: 1: a b1 
---a b1--- 
counter: 2: c d2 
--- 
c1--- 
counter: 3: e f3 
---d 21--- 
counter: 4: ---e f3--- 
counter: 5: g h4 
--- 
g3--- 

L'uscita sembra adattarsi con la risposta di Robert.

Answer 5

Una volta ottenuta la stringa che contiene la linea. Ad esempio "C P101", è possibile utilizzare le abilità di parsing di sscanf.

See: http://www.cplusplus.com/reference/clibrary/cstdio/sscanf.html

Answer 6

Forse usando un ciclo while, non una do ... while ciclo sarà di aiuto. In questo modo la condizione viene testata prima dell'esecuzione del codice. Provate il seguente frammento di codice:

while(command != 'q') 
    { 
     //statements 
    } 

Inoltre, se si conosce la lunghezza della stringa prima del tempo, 'per' i loop possono essere molto più facile da lavorare rispetto 'mentre' loop. Ci sono anche alcuni modi più complicati per determinare dinamicamente anche la lunghezza.

Come rant finale: scanf() non "succhiare". Fa quello che fa e questo è tutto. fgets() è molto pericoloso (sebbene conveniente per le applicazioni senza rischi), dal momento che non esegue alcun controllo sull'input in modo nativo. È MOLTO comunemente noto come punto di exploit, in particolare attacchi di overflow del buffer, sovrascrittura dello spazio nei registri non allocati per quella variabile. Quindi, se scegli di usarlo, dedica un po 'di tempo a sistemare alcuni corretti errori di correzione/correzione.

Spero che questo aiuti qualcuno in futuro, dal momento che presumo di averlo già gestito! ;)