Il problema specifico che si sta avendo è che una stringa di formato scanf
di %f
salterà spazi bianchi (compresa fine riga) finché non trova un carattere effettivo scansione. Dalla serie C99:
Una specifica di conversione viene eseguita nei seguenti passi:
- ingresso spazi vuoti (come specificato dalla funzione isspace
) vengono ignorati, a meno che la specifica comprende a '['
, 'c'
o 'n'
specificatore.
e, altrove, descrivendo isspace()
:
Gli spazi vuoti standard sono i seguenti: spazio ' '
, forma mangimi '\f'
, new-line '\n'
, ritorno carrello '\r'
, scheda orizzontale '\t'
, e scheda verticale .
La cosa migliore è quella di utilizzare fgets
per ottenere la linea (e questo può essere protetto da buffer overflow molto facilmente), quindi utilizzare sscanf
sulla linea risultante.
La funzione scanf
è una di quelle che dovresti guardare con molta cautela. Il seguente pezzo di codice è uno che uso spesso per gestire l'ingresso di linea:
#include <stdio.h>
#include <string.h>
#define OK 0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
int ch, extra;
// Get line with buffer overrun protection.
if (prmpt != NULL) {
printf ("%s", prmpt);
fflush (stdout);
}
if (fgets (buff, sz, stdin) == NULL)
return NO_INPUT;
// If it was too long, there'll be no newline. In that case, we flush
// to end of line so that excess doesn't affect the next call.
if (buff[strlen(buff)-1] != '\n') {
extra = 0;
while (((ch = getchar()) != '\n') && (ch != EOF))
extra = 1;
return (extra == 1) ? TOO_LONG : OK;
}
// Otherwise remove newline and give string back to caller.
buff[strlen(buff)-1] = '\0';
return OK;
}
// Test program for getLine().
int main (void) {
int rc;
char buff[10];
rc = getLine ("Enter string> ", buff, sizeof(buff));
if (rc == NO_INPUT) {
// Extra NL since my system doesn't output that on EOF.
printf ("\nNo input\n");
return 1;
}
if (rc == TOO_LONG) {
printf ("Input too long [%s]\n", buff);
return 1;
}
printf ("OK [%s]\n", buff);
return 0;
}
test con diverse combinazioni:
pax> ./prog
Enter string>[CTRL-D]
No input
pax> ./prog
Enter string> a
OK [a]
pax> ./prog
Enter string> hello
OK [hello]
pax> ./prog
Enter string> hello there
Input too long [hello the]
pax> ./prog
Enter string> i am pax
OK [i am pax]
Quello che vorrei fare è usare questa funzione per ottenere una linea in modo sicuro, quindi utilizzare semplicemente:
sscanf (buffer, "%f %f", &real, &img)
per ottenere i valori effettivi (e controllare il conteggio).
In realtà, ecco un programma completo, che è più vicino a ciò che si vuole:
#include <stdio.h>
#include <string.h>
#define OK 0
#define NO_INPUT 1
#define TOO_LONG 2
static int getLine (char *prmpt, char *buff, size_t sz) {
int ch, extra;
// Get line with buffer overrun protection.
if (prmpt != NULL) {
printf ("%s", prmpt);
fflush (stdout);
}
if (fgets (buff, sz, stdin) == NULL)
return NO_INPUT;
// If it was too long, there'll be no newline. In that case, we flush
// to end of line so that excess doesn't affect the next call.
if (buff[strlen(buff)-1] != '\n') {
extra = 0;
while (((ch = getchar()) != '\n') && (ch != EOF))
extra = 1;
return (extra == 1) ? TOO_LONG : OK;
}
// Otherwise remove newline and give string back to caller.
buff[strlen(buff)-1] = '\0';
return OK;
}
int main (void) {
int i = 1, rc;
char prompt[50], buff[50];
float real, imag;
while (1) {
sprintf (prompt, "\nEnter real and imaginary for #%3d: ", i);
rc = getLine (prompt, buff, sizeof(buff));
if (rc == NO_INPUT) break;
if (*buff == '\0') break;
if (rc == TOO_LONG) {
printf ("** Input too long [%s]...\n", buff);
}
if (sscanf (buff, "%f %f", &real, &imag) == 2) {
printf ("Values were %f and %f\n", real, imag);
i++;
} else {
printf ("** Invalid input [%s]\n", buff);
}
}
return 0;
}
insieme ad una corsa di prova:
pax> ./testprog
Enter real and imaginary for # 1: hello
** Invalid input [hello]
Enter real and imaginary for # 1: hello there
** Invalid input [hello there]
Enter real and imaginary for # 1: 1
** Invalid input [1]
Enter real and imaginary for # 1: 1.23 4.56
Values were 1.230000 and 4.560000
Enter real and imaginary for # 2:
pax> _
Nel caso in cui si aveva bisogno di più motivi per evitare di utilizzare 'scanf': http://c-faq.com/stdio/scanfprobs.html – jamesdlin