1. We have moved from vBulletin to XenForo and you are viewing the site in the middle of the move. Though the functional aspect of everything is working fine, we are still working on other changes including the new design on Xenforo.
    Dismiss Notice


Discussion in 'C' started by jose_peeterson, Jun 27, 2011.

  1. jose_peeterson

    jose_peeterson New Member

    //hi experts,
      //              i can't get the following code to read in  multiple times after the 
    "more(y/n)" printout. it does not scan a second time it just exists the loop after just once 
    //can you please fix the bug in this simple program.
    char op = 'y';
    while(op == 'y') 
      printf("Enter formula: ");
      l = strlen(molecule);
     printf("More? (y/n) : "); // THIS DOES NOT WORK???
     scanf("%c",&op);       //      WHY???????
    return 0;    
    Last edited by a moderator: Jun 27, 2011
  2. shabbir

    shabbir Administrator Staff Member

    Use fflush before the scanf and that should work.
  3. priyatendulkar

    priyatendulkar New Member

    The issue is in scanf
    scanf reads the input after enter/return key is pressed.But the newline generated by it is not consumed by scanf.Hence next time when scanf reads the input it reads newline instead of your input.

    I tried using fflush on my Linux system..2.6.9-22.EL..But it doesnt seem to work

    Alternative solution could be as under:
    U can use scanf as
    In this case scanf will read the newline and discard it
  4. xpi0t0s

    xpi0t0s Mentor

    scanf is a crap way of getting user input. Use fgets instead and parse the string.
  5. jose_peeterson

    jose_peeterson New Member

    hi thanks for replies BUT priyatendulkers method reads a second time but when it reaches the condition in the while loop it fails and exits the loop even after typing y.
    please help me
  6. xpi0t0s

    xpi0t0s Mentor

    After using fgets you can still use sscanf if you really want to. But writing parsing code yourself is better; user input is usually not rigidly formatted enough for the scanf family to handle it. You'll need to know about the family of functions that includes atoi() (ascii to integer).
  7. priyatendulkar

    priyatendulkar New Member

    It fails as u might have not modified

    Even here u'll need to consider the same


    Use the above method if u still wish to use scanf...

    Else u can use fgets instead as suggested by xpi0t0s which would be a better approach

Share This Page