Velocity Reviews

Velocity Reviews (http://www.velocityreviews.com/forums/index.php)
-   C Programming (http://www.velocityreviews.com/forums/f42-c-programming.html)
-   -   Getc or Getchar is not reading data (http://www.velocityreviews.com/forums/t440729-getc-or-getchar-is-not-reading-data.html)

mailursubbu@gmail.com 01-02-2006 10:40 AM

Getc or Getchar is not reading data
 
HI,
Below is my program. I compiled it through g++. Now strange thing is,
getc is not reading the data instead its printing the previously read
data ! Please some one let me know whats wrong.

#include<stdio.h>
int main()
{

int a;
char c;
scanf("%d",&a);
printf("%d",a);
c = getc(stdin);
printf("%c",c);
}

Thanks In advance.
Subra


Richard Heathfield 01-02-2006 10:48 AM

Re: Getc or Getchar is not reading data
 
mailursubbu@gmail.com said:

> HI,
> Below is my program. I compiled it through g++. Now strange thing is,
> getc is not reading the data instead its printing the previously read
> data ! Please some one let me know whats wrong.
>
> #include<stdio.h>
> int main()
> {
>
> int a;
> char c;
> scanf("%d",&a);


Read an int's value from the text representation given in stdin. Assuming
nothing goes wrong, that will be read correctly, and the /next/ thing in
the stream - which scanf will take a quick peek at and then quietly stick
back onto the input stream - will be a character that can't be interpreted
as part of the int. For example, it might be a newline character pressed by
the user to indicate that he's finished entering a line of information.

> printf("%d",a);
> c = getc(stdin);


Ah, let's go and collect that newline character.

> printf("%c",c);


And now let's display it.

> }
>
> Thanks In advance.


Yes. That's basically the problem here. :-)

By the way, don't forget that main returns an int, so it's best to return an
int from main. In the absence of anything better to return, I suggest 0.

--
Richard Heathfield
"Usenet is a strange place" - dmr 29/7/1999
http://www.cpax.org.uk
email: rjh at above domain (but drop the www, obviously)

mailursubbu@gmail.com 01-02-2006 11:38 AM

Re: Getc or Getchar is not reading data
 
Then how can I read the Integer and the Character ?? Any solution ???
Its happening consistantly on windows as well as on Linux...


M.B 01-02-2006 12:07 PM

Re: Getc or Getchar is not reading data
 
use one more getchar/getc (to read newline) after reading integer and
before reading the character
or
use scanf to read character also

Thanks
M.B


Emmanuel Delahaye 01-02-2006 12:13 PM

Re: Getc or Getchar is not reading data
 
mailursubbu@gmail.com a écrit :
> Below is my program. I compiled it through g++.


If you are not using a C compiler, all bets are off. Please use gcc or
the like.

> Now strange thing is,
> getc is not reading the data instead its printing the previously read
> data ! Please some one let me know whats wrong.
>
> #include<stdio.h>
> int main()
> {
>
> int a;
> char c;
> scanf("%d",&a);


scanf() is a tricky function that you are using badly:

- missing test of the returned value
- missing purge of pending characters.

> printf("%d",a);
> c = getc(stdin);


Due to the missing purge, getc() reads without waiting. Note that getc()
returns an int and not a char.

> printf("%c",c);
> }


Better to use fgets() unless you spend a long time to learn how to use
scanf() correctly.

--
A+

Emmanuel Delahaye

pemo 01-02-2006 12:36 PM

Re: Getc or Getchar is not reading data
 

<mailursubbu@gmail.com> wrote in message
news:1136201933.107953.34480@g49g2000cwa.googlegro ups.com...
> Then how can I read the Integer and the Character ?? Any solution ???
> Its happening consistantly on windows as well as on Linux...


scanf(...)

while(getchar() != '\n')
;

c = getchar();



Richard Heathfield 01-02-2006 01:04 PM

Re: Getc or Getchar is not reading data
 
mailursubbu@gmail.com said:

> Then how can I read the Integer and the Character ?? Any solution ???
> Its happening consistantly on windows as well as on Linux...


Yes, it would. Firstly, I forgot to observe last time that you are using
g++, which is a C++ compiler rather than a C compiler. I suggest you use
gcc if you wish to program in C. If you prefer to program in C++, please
ask your C++ questions in comp.lang.c++. Thank you.

Secondly, I would suggest that, to deal with text input of any kind, you
read it as a string with fgets, and then do any necessary conversions of
the data yourself using, for example, strtol or sscanf. Otherwise you'll
just litter your code with getchar calls, at the expense of clarity.

--
Richard Heathfield
"Usenet is a strange place" - dmr 29/7/1999
http://www.cpax.org.uk
email: rjh at above domain (but drop the www, obviously)

pemo 01-02-2006 04:32 PM

Re: Getc or Getchar is not reading data
 

"Richard Heathfield" <invalid@invalid.invalid> wrote in message
news:dpb8cr$avi$1@nwrdmz03.dmz.ncs.ea.ibs-infra.bt.com...
> mailursubbu@gmail.com said:
>
>> Then how can I read the Integer and the Character ?? Any solution ???
>> Its happening consistantly on windows as well as on Linux...

>
> Yes, it would. Firstly, I forgot to observe last time that you are using
> g++, which is a C++ compiler rather than a C compiler. I suggest you use
> gcc if you wish to program in C. If you prefer to program in C++, please
> ask your C++ questions in comp.lang.c++. Thank you.
>
> Secondly, I would suggest that, to deal with text input of any kind, you
> read it as a string with fgets, and then do any necessary conversions of
> the data yourself using, for example, strtol or sscanf. Otherwise you'll
> just litter your code with getchar calls, at the expense of clarity.


Wouldn't something like this work ...

/* could perhaps pass in the buffer size too */
int stdinGet(const char * format, void ** dest)
{
char buffer[100];

if(fgets(buffer, sizeof(buffer), stdin))
{
return sscanf(buffer, format, &(*dest));
}

return 0;
}

int main(void)
{
int n;
char buffer[100];

stdinGet("%d", (void *)&n);

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

stdinGet("%s", (void *)buffer);

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

return 0;
}



pemo 01-02-2006 06:41 PM

Re: Getc or Getchar is not reading data
 

"pemo" <usenetmeister@gmail.com> wrote in message
news:dpbkio$hap$1@news.ox.ac.uk...
>
> "Richard Heathfield" <invalid@invalid.invalid> wrote in message
> news:dpb8cr$avi$1@nwrdmz03.dmz.ncs.ea.ibs-infra.bt.com...
>> mailursubbu@gmail.com said:
>>
>>> Then how can I read the Integer and the Character ?? Any solution ???
>>> Its happening consistantly on windows as well as on Linux...

>>
>> Yes, it would. Firstly, I forgot to observe last time that you are using
>> g++, which is a C++ compiler rather than a C compiler. I suggest you use
>> gcc if you wish to program in C. If you prefer to program in C++, please
>> ask your C++ questions in comp.lang.c++. Thank you.
>>
>> Secondly, I would suggest that, to deal with text input of any kind, you
>> read it as a string with fgets, and then do any necessary conversions of
>> the data yourself using, for example, strtol or sscanf. Otherwise you'll
>> just litter your code with getchar calls, at the expense of clarity.

>
> Wouldn't something like this work ...
>
> /* could perhaps pass in the buffer size too */
> int stdinGet(const char * format, void ** dest)
> {
> char buffer[100];
>
> if(fgets(buffer, sizeof(buffer), stdin))
> {
> return sscanf(buffer, format, &(*dest));
> }
>
> return 0;
> }



int stdinGet(const char * format, void ** dest)
{
char buffer[100];

if(fgets(buffer, sizeof(buffer), stdin))
{
return sscanf(buffer, format, &(*dest));
}

return 0;
}

What's the type of the *dest above?

When used like this ...

int n;

stdinGet("%d", (void *)&n);

The routine's fine, but, if dest's type is changed to void *, gcc complains
about a void * dereference. This seems like a right hack to me - my code
above - but *as ever*, I'm confused: About the type of *dest, why the
compiler doesn't complain about the code 'as is', and why the call to the
routine with 'just' a void * cast isn't complained about - the expression
casts the int * to a void *, yet when this address is passed to stdinGet as
a void **, there's no complaint.




Emmanuel Delahaye 01-02-2006 07:04 PM

Re: Getc or Getchar is not reading data
 
pemo a écrit :
> Wouldn't something like this work ...


Hardly... (BTW, <stdio.h> is missing)

123456
123456

æ╣└w◄

> /* could perhaps pass in the buffer size too */
> int stdinGet(const char * format, void ** dest)


void ** ? Scary !

> {
> char buffer[100];
>
> if(fgets(buffer, sizeof(buffer), stdin))
> {
> return sscanf(buffer, format, &(*dest));


This '&(*' thingy sounds weird...

> }
>
> return 0;
> }
>
> int main(void)
> {
> int n;
> char buffer[100];
>
> stdinGet("%d", (void *)&n);


Don't use cast to hide bad pratices... It's evil and will bite you some
day...

> printf("%d\n", n);
> stdinGet("%s", (void *)buffer);


Are you a member of the random programming cult ?

> printf("%s\n", buffer);
> return 0;
> }


Devillish... Far enough to burn in Hell...

Why do you return the value of sscanf() if you never test it ?

--
A+

Emmanuel Delahaye


All times are GMT. The time now is 01:16 AM.

Powered by vBulletin®. Copyright ©2000 - 2013, vBulletin Solutions, Inc.
SEO by vBSEO ©2010, Crawlability, Inc.


1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57