CODEWITHSUNDEEP
c++coutflush
Tyler
Tyler

Reputation: 907

Calls to flush cout are ineffective

I am attempting to have the cout buffer flush to view a string before I manipulate it. Ive attempted to flush the buffer with calls to both std::flush() and std::cout.flush() but neither actually flush my output.

Only a call to std::endl has successfully flushed the buffer for me.

Here is my code

std::istringstream stm (game.date());
int day, month, year;
char delim = '/';

std::cout << "Date before: " << game.date() << std::flush; // first flush attempt
std::cout.flush();  // second flush attempt doesnt work
//std::cout << std::endl;   // if this is executed the buffer will flush

// Split date string into integers for comparison
stm >> month >> delim;
stm >> day >> delim;
stm >> year >> delim;

std::cout << "Date after: " << " | " << month << "/" << day << "/" << year << std::endl;

Here is my output
Date after: | 1/31/13
Date after: | 3/21/12
Date after: | 11/11/11
Date after: | 10/1/10
Date after: | 1/2/12

So as you can see the first call to cout isnt ever flushed but as I said before the buffer will successfully flush with endl, which calls flush itself. I am currently running Ubuntu 12.04 with VirtualBox on my host macbook pro running Mountain Lion.

Is there anything I may be doing wrong in my flush calls or is this potentially a system issue?

Upvotes: 8

Views: 11115

Answers (3)

Nicolay77
Nicolay77

Reputation: 2154

There's a system buffer on cout that will still buffer your output in modern Linux systems.

To disable it for a run of your program use the command stdbuf like this:

stdbuf -o 0 ./yourprogram --yourparams

If you need to disable the buffering in the debugger, use it like this:

stdbuf -o 0 gdb --args ./yourprogram --yourparams

Upvotes: 6

sehe
sehe

Reputation: 392843

Flush only writes the buffers for the stream to the actual device (file or tty).

It doesn't move to the next line, if that's what you expect.

This is by design.

Note, here's a cleaner version of the code, that appears to work as advertised (with or without the flush, for that matter):

See it Live on Coliru

#include <sstream>
#include <iostream>
#include <vector>
#include <cassert>

struct Game { 
    std::string _date;
    std::string date() const { return _date; }
};

int main()
{
    for (Game game : std::vector<Game> { {"01/02/1999"}, {"24/10/2013"}})
    {
        std::istringstream stm (game.date());
        int day, month, year;
        char delim = '/';

        std::cout << "Date before: " << game.date();

        stm >> month >> delim;
        assert(stm && '/' == delim);

        stm >> day >> delim;
        assert(stm && '/' == delim);

        stm >> year;

        std::cout << " Date after: " << " | " << month << "/" << day << "/" << year << std::endl;
    }
}

Upvotes: 1

Dietmar K&#252;hl
Dietmar K&#252;hl

Reputation: 153802

Both std::cout << flush; and std::cout.flush(); will flush std::cout.

It looks as if your code inserts a carriage return (\r) into the stream. Assuming you print this year, it seems you insert it as a char with value 13 which happens to be \r. The upshot of this is that your later output will just overwrite the output as it will be on the same line. You can verify this by explicitly inserting a newline (\n) before flushing the stream.

Upvotes: 11

Related Questions