Is there a better way for scanf?

[QQ]

Hello I am running this code
int main(void)
{
char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);
}

I get output if I type A more than 2 chars.

../a.out
Please input A:
asdgc
Please input B:
A is as,B is dg

I know the way to avoid this to use
scanf("%s",A);
However, people ususally don't use in this way
so is it a better way to avoid it?

Thanks a lot!

[Mike Wahler]

"QQ" <ju****@yahoo.c om> wrote in message
news:11******** *************@g 44g2000cwa.goog legroups.com...

Hello I am running this code
int main(void)
{
char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);
}

I get output if I type A more than 2 chars.

./a.out
Please input A:
asdgc
Please input B:
A is as,B is dg

I know the way to avoid this to use
scanf("%s",A);
However, people ususally don't use in this way
The reason is that there's no protection from
overflowing the array.
so is it a better way to avoid it?

Limit the input as you're doing above, and simply
throw away any unwanted characters:

#include <stdio.h>

void discard(void)
{
int c = 0;
while((c = getchar()) != EOF && c != '\n')
;
}

int main(void)
{
char A[3] = {0};
char B[3] = {0};

printf("Please input A: \n");
scanf("%2s", A);
discard();

printf("Please input B: \n");
scanf("%2s", B);
discard();

printf("A is %s, B is %s\n",A,B);
return 0;
}

-Mike

[Robert Gamble]

QQ wrote:

Hello I am running this code
int main(void)
{
char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);
}

I get output if I type A more than 2 chars.

./a.out
Please input A:
asdgc
Please input B:
A is as,B is dg

I know the way to avoid this to use
scanf("%s",A);
No, that's not the way to avoid it. In addition to exposing the
program to buffer overflows, this probably won't do what you want. If
the user entered three words, the first one would be read into A, the
second would then be handled by the above statement, but the last word
would still be on the input stream waiting to be picked up by the next
call to scanf.
However, people ususally don't use in this way
so is it a better way to avoid it?

You weren't completely clear on what it is you are trying to avoid. If
you just want to discard the rest of the line after your scanf, try
something like this:

scanf("%*[^\n]%*1[\n]");

You might also consider using fgets and sscanf instead.

Robert Gamble

[Peter Nilsson]

Robert Gamble wrote:

... If you just want to discard the rest of the line after your
scanf, try something like this:

scanf("%*[^\n]%*1[\n]");
Note that %[ must match _at least_ one character before scanf will
swallow a subsequent newline. Hence, if the remaining text on the
line is _just_ the newline, this scanf call will leave it there.
Better is something like...

if (scanf("%*[^\n]") != EOF) getchar();
You might also consider using fgets and sscanf instead.

--
Peter

[russfink]

Easiest method is use the * modifier which means to scan, but do not
assign, the input. The format string becomes " %2s%*s": the leading
space means skip whitespace (including newlines); %2s means read a
string of 2 characters (ending with null); %*s means read all further
characters on the line, but don't assign. The positional parameter is
NULL in this case (though it can be anything).

#include <stdio.h>

int main(void)
{
char A[3], B[3];
printf("Please input A: \n");
scanf(" %2s%*s", A, NULL);
printf("Please input B: \n");
scanf(" %2s%*s", B, NULL);
printf("A is %s, B is %s\n", A, B);
return 0;
}

Output is -
Please input A:
qwertyuiop
Please input B:
asdfghjkl;
A is qw, B is as

-- Russ

[Robert Gamble]

ru******@yahoo. com wrote:

Easiest method is use the * modifier which means to scan, but do not
assign, the input. The format string becomes " %2s%*s": the leading
space means skip whitespace (including newlines);
%2s means read a string of 2 characters (ending with null);
scanf reads characters, not strings, from the standard input stream and
does not expect null character termination.
%*s means read all further characters on the line, but don't assign.
No, it doesn't. The %*s means read and discard a sequence of
"non-white-space characters". If the string in question is "this
doesn't work", the " %2s" will consume the "th" and the subsequent
"%*s" will consume the "is", leaving the rest of the input on the
stream.
The positional parameter is
NULL in this case (though it can be anything).

No, the positional parameter is not NULL, nor can it "be anything".
There must not be a parameter provided at all for an assignment
suppression, doing so will invoke undefined behavior if there is
another conversion later in the same format string that does expect a
corresponding parameter. (It may cause undefined behavior in every
case, I don't feel like looking up the details)

Robert Gamble

[Robert Gamble]

Peter Nilsson wrote:

Robert Gamble wrote:

... If you just want to discard the rest of the line after your
scanf, try something like this:

scanf("%*[^\n]%*1[\n]");
Note that %[ must match _at least_ one character before scanf will
swallow a subsequent newline. Hence, if the remaining text on the
line is _just_ the newline, this scanf call will leave it there.

I didn't even think about that, good catch, thanks.
Better is something like...

if (scanf("%*[^\n]") != EOF) getchar();

Robert Gamble

[okcozyit]

char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);

Solution,buffer stream out
addition code ,fflush(stdin)

char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
fflush(stdin);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);

[Richard Bos]

"okcozyit" <ok******@gmail .com> wrote:

Solution,buffer stream out
addition code ,fflush(stdin)

That's a perfect solution for the problem of not getting enough
undefined behaviour: you can't fflush() input streams.

Richard

[Lawrence Kirby]

On Wed, 08 Jun 2005 14:23:28 -0700, QQ wrote:

Hello I am running this code
int main(void)
{
char A[3],B[3];
printf("Please input A: \n");
scanf("%2s",A);
printf("Please input B: \n");
scanf("%2s",B);
printf("A is %s,B is %s\n",A,B);
}

I get output if I type A more than 2 chars.

./a.out
Please input A:
asdgc
Please input B:
A is as,B is dg

I know the way to avoid this to use
scanf("%s",A);
However, people ususally don't use in this way
so is it a better way to avoid it?

The subject of the thread implies the use of scanf() but scanf() is the
wrong tool for reading line based input. Use fgets() instead. Once you
have read a line you have all of C's string handling functions, including
sscanf(), to interpret it.

Lawrence