17

I'm trying to do something which I think should be simple: do a blocking read from standard input, but timing out after a specified interval if no data is available.

In the Unix world this would be simple with select() but that doesn't work in Windows because stdin isn't a socket. What's the next simplest option without creating extra threads etc?

I'm using visual C++ targeting a Win32 environment.

so far I have tried:

  1. using select (doesn't work if the input is not a socket)

  2. using WaitForSingleObject(GetStdHandle(STD_INPUT_HANDLE)). - Remy's first suggestion. This always seems to return immediately when you call it if the standard input is a console (others have reported the same problem)

  3. using overlapped IO and doing a WaitForSingleObject (Remy's third suggestion). In this case the read always seems to block when the input is coming from a console - it seems that stdin does not support asynchronous I/O.

At the moment I'm thinking my only remaining option is to create a thread which will do a blocking read and then signal an event, and then have another thread which waits for the event with a timeout.

Andy
  • 10,412
  • 13
  • 70
  • 95
  • 1
    You explained your problem well, but what have you tried for that? – masoud Nov 13 '13 at 13:59
  • 1
    Your #2 attempt with WaitForSingleObject() works if you properly remove the initial events from STD_INPUT_HANDLE. In my case I have a "focus" event in the console queue from the start. Check out @Clay 's answer, it's correct. – Vladimir Sinenko Oct 26 '15 at 05:46

7 Answers7

6

I had to solve a similar problem. On Windows it is not as easy or obvious as Linux. It is, however, possible. The trick is that Windows places console events in the console input event queue. You've got to filter out the events you don't care about and only process those events you do care about (like key presses).

For further reading: see the Win32 console documentation

Here is some mostly-debugged sample code based on a socket and stdin multiplexer I was working on:

void ProcessStdin(void)
{
    INPUT_RECORD record;
    DWORD numRead;
    if(!ReadConsoleInput(GetStdHandle(STD_INPUT_HANDLE), &record, 1, &numRead)) {
        // hmm handle this error somehow...
        return;
    }

    if(record.EventType != KEY_EVENT) {
        // don't care about other console events
        return;
    }

    if(!record.Event.KeyEvent.bKeyDown) {
        // really only care about keydown
        return;
    }

    // if you're setup for ASCII, process this:
    //record.Event.KeyEvent.uChar.AsciiChar

} // end ProcessStdin

int main(char argc, char* argv[])
{
    HANDLE eventHandles[] = {
        GetStdHandle(STD_INPUT_HANDLE)
        // ... add more handles and/or sockets here
        };

    DWORD result = WSAWaitForMultipleEvents(sizeof(eventHandles)/sizeof(eventHandle[0]), 
        &eventHandles[0], 
        FALSE, 
        1000, 
        TRUE
        );

    switch(result) {
        case WSA_WAIT_TIMEOUT: // no I/O going on right now
            break;

        case WSA_WAIT_EVENT_0 + 0: // stdin at array index 0
            ProcessStdin();
            break;

        case WSA_WAIT_EVENT_0 + 1: // handle/socket at array index 1
            break;

        case WSA_WAIT_EVENT_0 + 2: // ... and so on
            break;

        default: // handle the other possible conditions
            break;
    } // end switch result
}
Clay
  • 1,159
  • 1
  • 9
  • 20
5

Using GetStdHandle + WaitForSingleObject works fine. But be sure to set the approriate flags and flush the console buffer as well before entering the loop.

In short (without error checks)

std::string inStr;
DWORD fdwMode, fdwOldMode;
HANDLE hStdIn = GetStdHandle(STD_INPUT_HANDLE);
GetConsoleMode(hStdIn, &fdwOldMode);
// disable mouse and window input
fdwMode = fdwOldMode ^ ENABLE_MOUSE_INPUT ^ ENABLE_WINDOW_INPUT;
SetConsoleMode(hStdIn, fdwMode);
// flush to remove existing events
FlushConsoleInputBuffer(hStdIn);
while (!abort)
{
    if (WaitForSingleObject(hStdIn, 100) == WAIT_OBJECT_0)
    {
         std::getline(std::cin, inStr);
    }
}
// restore console mode when exit
SetConsoleMode(hStdIn, fdwOldMode);
Jörgen
  • 316
  • 4
  • 10
  • 1
    Use `ReadConsoleInput` instead of `getline` and avoid disabling other events. – dashesy Oct 20 '16 at 01:38
  • 4
    This toggles `ENABLE_MOUSE_INPUT` and `ENABLE_WINDOW_INPUT` instead of disabling them, which is a bug. Also, that's not enough. `WaitForSingleObject` will still detect focus-changing and menu events, which will cause `ReadConsole` or `getline` to block if there's nothing in the buffer. – alexchandel Dec 06 '16 at 21:59
  • @alexchandel True. To actually disable this inputs you need to construct the mask: `SetConsoleMode(ih, cmod & ~(ENABLE_MOUSE_INPUT | ENABLE_WINDOW_INPUT));` – ScienceDiscoverer Aug 11 '23 at 05:14
3

This should do it:

int main()
{
    static HANDLE stdinHandle;
    // Get the IO handles
    // getc(stdin);
    stdinHandle = GetStdHandle(STD_INPUT_HANDLE);

    while( 1 )
    {
        switch( WaitForSingleObject( stdinHandle, 1000 ) )
        {
        case( WAIT_TIMEOUT ):
            cerr << "timeout" << endl;
            break; // return from this function to allow thread to terminate
        case( WAIT_OBJECT_0 ):
            if( _kbhit() ) // _kbhit() always returns immediately
            {
                int i = _getch();
                cerr << "key: " << i << endl;
            }
            else // some sort of other events , we need to clear it from the queue
            {
                // clear events
                INPUT_RECORD r[512];
                DWORD read;
                ReadConsoleInput( stdinHandle, r, 512, &read );
                cerr << "mouse event" << endl;
            }
            break;
        case( WAIT_FAILED ):
            cerr << "WAIT_FAILED" << endl;
            break;
        case( WAIT_ABANDONED ): 
            cerr << "WAIT_ABANDONED" << endl;
            break;
        default:
            cerr << "Someting's unexpected was returned.";
        }
    }

    return 0;
}
flux
  • 189
  • 3
  • 13
  • There is a problem with this. If events, such as mouse move, continue to arrive then WaitForSingleObject() never gets to timeout so you never get the case( WAIT_TIMEOUT ). You need another mechanism to check the time elapsed on ever loop. – flux Feb 24 '14 at 05:13
  • 1
    The `stdinHandle` can be redirected. You must check the type of the handle before call an appropriate API function: `GetFileType`. `FILE_TYPE_DISK` -> `ReadFile`; `FILE_TYPE_CHAR` -> `ReadConsoleInput` or `PeekConsoleInput`; `FILE_TYPE_PIPE` -> `ReadFile` or `PeekNamedPipe`. Otherwise `ReadConsoleInput` will fail. – Andry May 27 '21 at 18:33
2

In case anyone is writing chrome native messaging host and is looking for solution to check if there is any input on stdin without blocking then this works perfect:

HANDLE hStdin = GetStdHandle(STD_INPUT_HANDLE);
int timer = GetTickCount();
while(timer+10000 > GetTickCount())
{
    unsigned int length = 0;
    DWORD bytesAvailable = 0; 
    PeekNamedPipe(hStdin,NULL,0,NULL,&bytesAvailable,NULL);
    if(bytesAvailable > 0)
    {
        for (int i = 0; i < 4; i++)
        {
            unsigned int read_char = getchar();
            length = length | (read_char << i*8);
        }


        for (int i = 0; i < length; i++)
        {
            msg += getchar();
        }
        timer = GetTickCount();
    }
    else
    {
        // nothing to read, stdin empty
        Sleep(10);
    }
}
dejv25
  • 21
  • 1
  • Fantastic - after trying many different ways to check for available data before reading, this was the only way that worked. – user3896248 Feb 25 '19 at 15:51
1

Use GetStdHandle() to get the stdin handle. You can then either:

  1. use WaitForSingleObject() on the stdin handle itself to detect when there is console input available for reading, then read from it as needed.

  2. use GetNumberOfConsoleInputEvents() or PeekConsoleInput() on the stdin handle in a loop to determine when there is data available for reading, then read from it as needed.

  3. use ReadFile() with an OVERLAPPED structure containing an event handle, then use the event handle with WaitForSingleObject() to detect if the read times out.

In any case, be careful if stdin has been redirected. If it is redirected to something that is not console I/O, you can't use a GetStdHandle() handle with console functions. If it is redirected to a file, you have to use ReadFile().

Remy Lebeau
  • 555,201
  • 31
  • 458
  • 770
  • Thanks Remy - This is getting more complex all the time :( Actually I'm writing a google chrome native messaging host. in the real world it will be invoked by the chrome process with standard input and output connected to that, but I was assuming it would be simple to just use generic input/output functions so that I could test it from the command line. It seems that in the world of windows console applications, these things are not so straightforward as *nix! It looks like option 3 is probably the most likely to work – Andy Nov 14 '13 at 08:53
0

You'll need the GetStdHandle function to obtain a handle to the console, then you can use WaitForSingleObject to wait for an event to happen on that handle, with a timeout.

Pat
  • 1,726
  • 11
  • 18
  • D'oh! thanks. I was faffing about with Asynchronous I/O and SleepEx. this looks like what I was looking for all along :-) – Andy Nov 13 '13 at 14:18
  • hmm after some more playing around this doesn't seem work. WaitForSingleObject on stdin always seems to return WAIT_OBJECT_0 when you initially call it, even though a read from stdin would block. – Andy Nov 13 '13 at 16:06
  • According to the documentation: "A process can specify a console input buffer handle in one of the wait functions to determine when there is unread console input. When the input buffer is not empty, the state of a console input buffer handle is signaled." Maybe there is something on the console input but you are trying to read more than there is available? – Remy Lebeau Nov 13 '13 at 20:48
  • Other people seem to have had this same problem - I think it may be that there are other events (e.g. mouse events) which aren't standard input but still cause the object to appear signalled. As you'll see below, my app won't be invoked from the console in the real world (that wwas just for testing) so I'm trying to avoid this and use some more generic mechanism instead. – Andy Nov 14 '13 at 08:56
0

The existing answers don't address the case where the standard input is an anonymous pipe rather than a console. In this case functions like GetNumberOfConsoleInputEvents() will return 0x6 (bad handle) and the methods described above will not work.

In my case I am trying to use stdin and stdout to facilitate asynchronous interprocess communication, so the parent process (nodejs) opens stdin and stdout as anonymous pipes with a child process (c++). For this case, the type of stdin can be detected as follows:

 HANDLE stdinput = GetStdHandle(STD_INPUT_HANDLE);
 if (stdinput == INVALID_HANDLE_VALUE) {
    DWORD problem1 = GetLastError();
    cout << "Failed to get input handle. " << (void*) problem1 << endl;
    return(1);
 }
 DWORD fileType = GetFileType(stdinput);
 if (fileType != FILE_TYPE_PIPE) {
    cout << "Input type is not pipe. Instead: " << (void*) fileType << endl;
    return(2);
 }

Then, as to enable asynchronous reads from this input pipe, I came up with two ideas:

Method 1: Continuously poll in a loop for available input

do {
  DWORD bytesAvailable = 0;
  BOOL success = PeekNamedPipe(stdinput, NULL, NULL, NULL, &bytesAvailable, NULL );

  if (!success) {
    cout << "Couldn't run PeekNamedPipe." << endl;
    DWORD problem = GetLastError();
    cout << "Error code: " << (void*)problem << endl;
  }

  char buf[bytesAvailable+1]; //mingw allows dynamic stack allocation. In Visual studio might need to allocate on heap.

  if (bytesAvailable > 0) {
    ReadFile(stdinput, buf, additionalBytesAvailable, NULL, NULL);
    cout << "Received: " << buf << endl;
  }

  Sleep(10); //Small delay between checking for new input
} while(1);

Method 1 suffers from the issue that input can't be processed any faster than the small delay. Of course the delay could be shortened, but then the thread will consume more CPU resources. For this reason I came up with an alternate method.

Method 2: Block using ReadFile and send to different thread for processing. In the input processing thread, which will block when waiting for input:

do {
  char firstChar[2]; //we will read the first byte not sure if it is null terminated...

  //block until at least one byte is available
  ReadFile(stdinput, firstChar, 1, NULL, NULL);
  DWORD additionalBytesAvailable = 0;
  BOOL success = PeekNamedPipe(stdinput, NULL, NULL, NULL, &additionalBytesAvailable, NULL );
  if (!success) {
    cout << "Couldn't run PeekNamedPipe." << endl;
    DWORD problem = GetLastError();
    cout << "Error code: " << (void*)problem << endl;
  }
  char buf[additionalBytesAvailable+2]; //mingw allows stack allocation.

  buf[0] = firstChar[0];
  buf[1] = '\0';

  if (additionalBytesAvailable > 0) {    
    ReadFile(stdinput, buf+1, additionalBytesAvailable, NULL, NULL);
  }

  std::cout << count << " Read: " << buf << endl;

  //write some data to a different thread that is still responsive
  pthread_mutex_lock(&responsiveThreadLock);
  mutexProtectedString = std::string(buf);
  pthread_mutex_unlock(&responsiveThreadLock);
  PostThreadMessage(handleOfResponsiveThread, WM_NEWINPUT, NULL, NULL);
} while(1);

And in the thread that stays responsive:

MSG msg;
do {
  GetMessageWithTimeout(&msg, 1000);
  if (msg.message == WM_NEWINPUT) {
     std::string receivedStringCopy = "";
     pthread_mutex_lock(&responsiveThreadLock);
     receivedStringCopy  = mutexProtectedString;
     pthread_mutex_unlock(&responsiveThreadLock);
     std::cout << "Received: " << receivedStringCopy << endl;
  }
  TranslateMessage(&msg);
  DispatchMessage(&msg);
  std::cout << "Still responsive. " << endl;
} while(1);

GetMessageWithTimeout is a function designed to stay responsive (after a timeout) while waiting for a message:

//Wait upto timeoutMs milliseconds for a message.
//Return TRUE if a message is received or FALSE if the timeout occurs or there is an error.
BOOL GetMessageWithTimeout(MSG *msg, UINT timeoutMs)
{
    //Check the message queue and return immediately if there is a message available
    BOOL hasMessage = PeekMessage(msg, NULL, 0, 0, PM_REMOVE);
    if (hasMessage) {
        return(TRUE);
    }
    else {
        //Any new messages that have arrived since we last checked the message queue will
        //cause MsgWaitForMultipleObjects to return immediately.
        //otherwise this will block the thread until a message arrives or timeout occurs
        DWORD res1 = MsgWaitForMultipleObjects(0, NULL, FALSE, timeoutMs, QS_ALLINPUT);
        if (res1 == WAIT_TIMEOUT) {
            printf("!");
            return(FALSE);
        }
        if (res1 == WAIT_OBJECT_0) {
            //If we are here, there *should* be a message available. We can just get it with PeekMessage
            hasMessage = PeekMessage(msg, NULL, 0, 0, PM_REMOVE);
            return(hasMessage);
        }
        //If we get here, its because we have a WAIT_FAILED. Don't know why this would occur, but if it
        //does, lets pause for a bit, so we don't end up in a tight loop
        Sleep(100);
        return(FALSE);
    }
}

This second method will respond immediately to new inputs. For additional robustness, it may be necessary to check and make sure that the contents of the pipe ends with a \n (so incomplete messages aren't sent) and to push messages to a vector so that multiple messages don't override one another if the receiving thread can't process them fast enough.

bruceceng
  • 1,844
  • 18
  • 23