84

So I was trying to get valid integer input from cin, and used an answer to this question.

It recommended:

#include <Windows.h> // includes WinDef.h which defines min() max()
#include <iostream>
using std::cin;
using std::cout;

void Foo()
{
    int delay = 0;
    do
    {
        if(cin.fail())
        {
            cin.clear();
            cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
        }
        cout << "Enter number of seconds between submissions: ";
    } while(!(cin >> delay) || delay == 0);
}

Which gives me an error on Windows, saying that the max macro doesn't take that many arguments. Which means I have to do this

do
{
    if(cin.fail())
    {
        cin.clear();
#undef max
        cin.ignore(std::numeric_limits<std::streamsize>::max(), '\n');
    }
    cout << "Enter number of seconds between submissions: ";
} while(!(cin >> delay) || delay == 0);

To get it to work. That's pretty ugly; is there a better way to work around this issue? Maybe I should be storing the definition of max and redefining it afterward?

5

6 Answers 6

127

Define the macro NOMINMAX:

This will suppress the min and max definitions in Windef.h.

1
98

Just wrap the function name in parenthesis:

(std::numeric_limits<size_type>::max)()

No need for the NOMINMAX macro in this case, plus you won't get compiler warnings

7
  • 2
    std::max<size_type>(a,b) works for me and is more readable. The template parameter suppresses the macro. Am I missing something? Feb 3, 2017 at 22:02
  • 4
    @DaleWilson Yes, you're missing something. Your call will get the larger of two values. std::numeric_limits<size_type>::max() will return the maximum value that can be stored by size_type.
    – Ben
    Apr 25, 2017 at 18:08
  • 2
    Wrapping the function name in parens is taking the function pointer & invoking it as though implementing a strategy pattern via function pointers. That defies the advice of expressing ideas directly in code since this other syntax is rather an attempt to work around the definition of the min & max macros (not an implementation of strategy pattern via function pointer). The Clang/LLVM infrastructure suggests "#define NOMINMAX before any Windows header". Which is better advice IMO. Sep 3, 2017 at 16:01
  • 2
    @LouisLangholtz With VS projects there is common stdafx.h and when you place NOMINMAX in it, there can be errors in other files, which use the macros.
    – ggurov
    Jul 30, 2018 at 12:37
  • 1
    When working with a project that includes legacy code (which might use the old max and min macros), this is your best option. The ugly syntax is a trade-off.
    – jakar
    Dec 7, 2018 at 16:52
13

If you don't know whether somebody else might have included windows.h without NOMINMAX, you might define a dummy macro which can be used to suppress function-like macro invocations without changing the definition:

#define DUMMY
...
std::numeric_limits<std::streamsize>::max DUMMY ()

Not really pretty either, but works and is non-intrusive.

When working with the Windows header file, I prefer to hide it as much as I can by including it only in specialized code and header files (using pimpl if necessary), because it throws just too much garbage into the global namespace.

2
  • 3
    Interesting. I will definitely be following advice to peel out windows.h wherever possible.
    – Almo
    Jul 18, 2012 at 21:53
  • This is an excellent way to do things when writing a header for use by others and you cannot control whether the user #includes windows.h before your header Jan 12, 2021 at 13:31
11

Are you just trying to flush the cin buffer? I always just used:

cin.ignore(cin.rdbuf()->in_avail());
1
  • This is a good answer, though I have accepted the other one as it directly answers the question, while yours addresses the source of the problem in another way. :)
    – Almo
    Jul 18, 2012 at 15:07
7

If you happen to use GDI+, the approach with NOMINMAX won't work for you, because headers of GDI+ require min or max in global namespace.

And the simplest workaround in this case is to undefine min/max when they are no longer needed.

The code sample to illustrate the approach:

//#define NOMINMAX - this won't work
#include <Windows.h>
#include <gdiplus.h>
#undef max
#undef min
...
#include <cxxopts.hpp>
1
  • 1
    Potentially reverses the problem, though: if name clash happens in a header, solved as described here, and you include the this header somewhere requiring min/max yet to be defined - bad luck...
    – Aconcagua
    Jan 31, 2018 at 13:11
2

I came here looking for a list of common things that windows.h defines, and after much scouring, I found min, max, small, near, far. Hopefully this helps someone else, too.

I realize this question is 10 years old, but nobody answered with the obvious solution, so here's my way of dealing with this issue.

All OS specific stuff goes in one translation unit (one CPP source file) which provides a cleaner API to the rest of the project, and thus windows.h never pollutes the rest of the project. Any handles to OS stuff is just a "NativeHandle" which is typedef'd to void*; some things are heap allocated this way that wouldn't otherwise need to be possibly, but that's not much of a downside, considering this also provides a single source point for porting to other platforms.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.