Thread: sscanf weirdness

try

Code:

int _extended=0, _high_low=0, _intel_motorola=0;

Hello,

Firstly you are making a very big mistake of treating a Character array as an Integer. You cannot expect the whole string "1 0 1 1 0" to be the same in integer data-type.

If you try to find out the Integer substitute of that string, you would have to indivitually convert each characters into ASCII and calculate it. And the answer what you get would get stored in first variable in your sscanf list i.e., _key.

Now you can do it by two methods. You can use Type Casting or you can directly take Character data-type for extended, _high_low, _intel_motorola.

Either way it will work...So the point is you need to work on it as a STRING, you cannot directly expect a string to get stored into integer from a stream.

Hope it helps
Chirag Chhatbar.

Originally Posted by Chirag

Firstly you are making a very big mistake of treating a Character array as an Integer.

Where?

The original poster's usage of sscanf is correct. The first paramter should be char *
As already pointed out the problem is with the format specifier for long and short

%ld and %hd may be appropriate

Okay i think you don't understand my point. When sscanf is storing the value into _key...it is storing the ASCII substitute of the whole String.

Let me just post what i think is a Solution to the problem, yes use of sscanf is CORRECT, but is not what he wanted...

Code:

int pgmvalidity_check(char checkline[256])
{
   char _key=0;
   char _extended=0, _high_low=0, _intel_motorola=0;
   char _baudrate=0;
   
   puts(checkline);
   getch();

   sscanf(checkline,"%c\t%c\t%c\t%c\t%c", &_key, &_extended, &_high_low, &_baudrate, &_intel_motorola);

   printf("high_low: %c\n", _high_low);

....

}

Originally Posted by Chirag

Okay i think you don't understand my point. When sscanf is storing the value into _key...it is storing the ASCII substitute of the whole String.

nah, if you want sscanf to interpret the string as integers you can use %d of course. but you have to make sure you pass the correct pointers to the variables that will receive the values.

he wants to get 1 and 0 in his variables and not '1' and '0'

Originally Posted by Chirag

Okay i think you don't understand my point. When sscanf is storing the value into _key...it is storing the ASCII substitute of the whole String.

Let me just post what i think is a Solution to the problem, yes use of sscanf is CORRECT, but is not what he wanted...

Code:

int pgmvalidity_check(char checkline[256])
{
   char _key=0;
   char _extended=0, _high_low=0, _intel_motorola=0;
   char _baudrate=0;
   
   puts(checkline);
   getch();

   sscanf(checkline,"%c\t%c\t%c\t%c\t%c", &_key, &_extended, &_high_low, &_baudrate, &_intel_motorola);

   printf("high_low: %c\n", _high_low);

....

}

i did this

Code:

#include<stdio.h>

int main() {
  char s[10];

  fgets(s, 10, stdin);

  int a;
  int b;

  printf("%s\n", s);

  sscanf(s, "%d %d", &a, &b);

  printf("%d %d\n", a, b);

  return 0;
}

and it works just fine when I put some integer value on the fgets...

I try to do the same, but in my case what happens whatever i provide as INPUT (i.e., 10110) it gets stored into only variable a and none of it even gets stored into b. i don't know why...

rite... u have to put spaces "1 0 1 1 0"