setw not working as expected

Question

I'm trying to print time in hh:mm format but when the time is like 01:01 it prints as 1:1. Here's my code:

void output(int hour, int min, char ampm){
    cout << setw(2) << setfill('0') << "The time is: " << hour << ":" << min << " ";

    if(ampm == 'P'){
        cout << "PM";
    }
    else if (ampm == 'A'){
        cout << "AM";
    }
}

As I understand it, this should work. I include iomanip. Can you see anything wrong with it?

score 8 · Accepted Answer · answered Sep 26 '13 at 20:37

The width is a special formatting setting: While all other formatting flags are stick, the width() will be reset by each output operator (well, you can have user-defined output operators which don't reset the width() but doing so would not follow the normal style). That is, you need to set the width immediately prior to the output that should be affected:

std::cout << std::setfill('0')
          << std::setw(2) << hour << ':'
          << std::setw(2) << min << ' ';

how incredibly convenient, std::ostream. thank you for this answer, @Dietmar. — orion elenzil, Oct 18 '18 at 19:19

score 2 · Answer 2 · answered Sep 26 '13 at 20:32

2

Following is correct way:

 cout<<""The time is: ";
 cout << setfill('0') <<setw(2) << hour << ":" <<setw(2) << min << " ";

Ref :-this

answered Sep 26 '13 at 20:32

P0W

46,614
9
72
119

Thanks! I didn't realize it had to be right before the variable. – granra Sep 26 '13 at 20:37

setw not working as expected

2 Answers2