DaniWeb Forum Index > Software Development > C

C RSS

C - Getting double output?

Please support our C advertiser: Programming Forums - DaniWeb Sister Site

•

Join Date: Apr 2005

Posts: 105

Reputation: jhdobbins is an unknown quantity at this point

Solved Threads: 3

jhdobbins

Offline

Junior Poster

C - Getting double output?

Feb 28th, 2006

I must be missing something here. I am having trouble when I try to get an input. The numbers work fine, but once you get the correct answer, the area:

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
void tryanother() {
 
     char yorn;
 
     printf("Would you like to try another one?  (y/n)\n");
     scanf("%c", &yorn);
 
     if (yorn == 'y')
        getnum();
     else if (yorn == 'n')
        return;
     else if (yorn != 'y' || yorn != 'n') {
        printf("Invalid choice\n");
        tryanother();
        }
 
}
void tryanother() {
     
     char yorn;
     
     printf("Would you like to try another one?  (y/n)\n");
     scanf("%c", &yorn);
    
     if (yorn == 'y')
        getnum();
     else if (yorn == 'n')
        return;
     else if (yorn != 'y' || yorn != 'n') {
        printf("Invalid choice\n");
        tryanother();
        }
     
}

isn't right. It will print "Would you like to try another (y/n)." "Invalid Choice." "Would you like to try another (y/n)" then wait for your input... why is it doing this?

Thank you.

Here is the rest of the code:

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
#include <stdio.h>
#include <stdlib.h>
#include <time.h>
 
 
void getnum ();
void tryagain(int, int, int);
void tryanother();
 
void getnum() {
 
    int num1, num2, solution;
    num1 = (rand() % 9);
    num2 = (rand() % 9);
 
    solution = num1 * num2;
 
    tryagain (num1, num2, solution);
}
 
 
void tryagain(int num1, int num2, int solution) {
 
    int guess;
 
    printf("How much is %d times %d ?\n", num1, num2);
    scanf("%d", &guess);
 
    if(guess == solution) {
        printf("Very Good!\n\n");
        tryanother();
        }
    else
        tryagain(num1, num2, solution);
}
 
void tryanother() {
 
     char yorn;
 
     printf("Would you like to try another one?  (y/n)\n");
     scanf("%c", &yorn);
 
     if (yorn == 'y')
        getnum();
     else if (yorn == 'n')
        return;
     else if (yorn != 'y' || yorn != 'n') {
        printf("Invalid choice\n");
        tryanother();
        }
 
}        
 
 
int main() {
 
    srand(time (NULL));
 
    getnum();
 
    return 0;
 
}
#include <stdio.h>
#include <stdlib.h>
#include <time.h>


void getnum ();
void tryagain(int, int, int);
void tryanother();

void getnum() {
    
    int num1, num2, solution;
    num1 = (rand() % 9);
    num2 = (rand() % 9);
    
    solution = num1 * num2;
    
    tryagain (num1, num2, solution);
}


void tryagain(int num1, int num2, int solution) {
    
    int guess;
    
    printf("How much is %d times %d ?\n", num1, num2);
    scanf("%d", &guess);
    
    if(guess == solution) {
        printf("Very Good!\n\n");
        tryanother();
        }
    else
        tryagain(num1, num2, solution);
}

void tryanother() {
     
     char yorn;
     
     printf("Would you like to try another one?  (y/n)\n");
     scanf("%c", &yorn);
    
     if (yorn == 'y')
        getnum();
     else if (yorn == 'n')
        return;
     else if (yorn != 'y' || yorn != 'n') {
        printf("Invalid choice\n");
        tryanother();
        }
     
}        
 

int main() {
    
    srand(time (NULL));
    
    getnum();
    
    return 0;
    
}

•

Join Date: Aug 2005

Posts: 15,438

Reputation: Ancient Dragon has a reputation beyond repute

Solved Threads: 1473

Ancient Dragon Ancient Dragon is online now

Online

Still Learning

Re: C - Getting double output?

Feb 28th, 2006

when you press 'y' followed by <Enter> key, there are two keys in the keyboard buffer -- so the second time through the code it is probably getting the '\n' key. you need to flush the keyboard buffer to remove the '\n'.

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
scanf("%c", &yorn);
getchar(); //flush the keyboard buffer of the '\n' key
scanf("%c", &yorn);
getchar(); //flush the keyboard buffer of the '\n' key

or you could use fgets() which will flush the '\n' too assuming the user types on 'y' followed by <Enter>. If anything else is typed then those keys will also be in the keyboard buffer. There is no easy standard solution to the problem. Make the buffer in the code below big enough and fgets() will probably be able to fill it with all the keys in the buffer.

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
char buf[3];
fgets(buf,sizeof(buf),stdin);
yorn = buf[0];
char buf[3];
fgets(buf,sizeof(buf),stdin);
yorn = buf[0];

•

Join Date: Apr 2005

Posts: 105

Reputation: jhdobbins is an unknown quantity at this point

Solved Threads: 3

jhdobbins

Offline

Junior Poster

Re: C - Getting double output?

Feb 28th, 2006

Alright, I throw the gatchar(); in there, but then if I hit y or n it gives me invalid choice and there is no exiting the loop...
??

•

Join Date: Aug 2005

Posts: 15,438

Reputation: Ancient Dragon has a reputation beyond repute

Solved Threads: 1473

Ancient Dragon Ancient Dragon is online now

Online

Still Learning

Re: C - Getting double output?

Feb 28th, 2006

use getchar() after each numeric input using scanf(). you need it in two places in your program.

•

Join Date: Apr 2005

Posts: 105

Reputation: jhdobbins is an unknown quantity at this point

Solved Threads: 3

jhdobbins

Offline

Junior Poster

Re: C - Getting double output?

Feb 28th, 2006

ty much dragon

it is solved!

•

Join Date: Nov 2005

Posts: 251

Reputation: dwks has a spectacular aura about

Solved Threads: 25

dwks

Offline

Posting Whiz in Training

Re: C - Getting double output?

Feb 28th, 2006

With fgets(), you risk not flushing the entire buffer, and with a single getchar() you have the same problem. The best solution is this:

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
int c;
while((c = getchar()) != '\n' && c != EOF);
int c;
while((c = getchar()) != '\n' && c != EOF);

c has to be an int because EOF is an int value.

dwk

Seek and ye shall find.

"Only those who will risk going too far can possibly find out how far one can go."
-- TS Eliot.

"I have not failed. I've just found 10,000 ways that won't work."
-- Thomas Alva Edison

"The only real mistake is the one from which we learn nothing."
-- John Powell

•

Join Date: Nov 2005

Posts: 251

Reputation: dwks has a spectacular aura about

Solved Threads: 25

dwks

Offline

Posting Whiz in Training

Re: C - Getting double output?

Feb 28th, 2006

http://faq.cprogramming.com/cgi-bin/...&id=1043284392

•

Join Date: Nov 2005

Posts: 39

Reputation: comwizz is an unknown quantity at this point

Solved Threads: 0

comwizz

Offline

Light Poster

Re: C - Getting double output?

Mar 1st, 2006

I did not understand that if the size of the buf variable is 3 then after getting input "y" and "\n" how does stdin terminate the coming input i.e. what would be buf[2]??

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
char buf[3];
fgets(buf,sizeof(buf),stdin);
yorn = buf[0];
char buf[3];
fgets(buf,sizeof(buf),stdin);
yorn = buf[0];

Also , I did not understand why

 Help with Code Tags 
 C Syntax (Toggle Plain Text) 
while((c = getchar()) != '\n' && c != EOF);
while((c = getchar()) != '\n' && c != EOF);

this is required. I have gone through faq recommended by dwks but still I am not clear . Please help,
Thanks,
comwizz.

•

Join Date: Aug 2005

Posts: 15,438

Reputation: Ancient Dragon has a reputation beyond repute

Solved Threads: 1473

Ancient Dragon Ancient Dragon is online now

Online

Still Learning

Re: C - Getting double output?

Mar 1st, 2006

lets say that instead of typing just y + '\n', you type a whole bunch of keys then '\n' at the end. fgets() will only grab from the keyboard buffer the number of keys that will fit in the buffer (sizeof the buffer - 1 for null terminator), the remaining keys will stay in the keyboard buffer until your program removes them. And the buffer returned by fgets() will contain '\n' only if there is one in the input stream.

The loop that dwks posted is probably the savest way to flush out (empty) the keyboard buffer that contains an unknown number of characters.