C++宏（min max）无法正常工作

Question

为什么下面的代码不工作？它打印INT_MAX。但是如果我取消注释循环内部的两行，那么它工作正常（打印2）。我不能把这两个宏组合起来？不知道是否需要进一步的细节...很自我解释。

谢谢。周围的术语

#include <iostream> 
#include <limits.h> 

using namespace std; 

#define min(a,b) a<b?a:b 
#define max(a,b) a>b?a:b 

int main(int argc, char **argv) 
{ 
    int N = 100; 
    int *drop = new int[N+1]; 
    drop[0] = 0; drop[1] = 1; drop[2] = 1; 
    for(int i=3; i<=N; i++) 
    { 
     drop[i] = INT_MAX; 
     for(int start=1; start<=i; start++) 
     { 
      drop[i] = min(drop[i], max(start, drop[i-start]+1)); 
      //int x = max(start, drop[i-start]+1); 
      //drop[i] = min(drop[i], x); 
     } 
    } 
    cout<<drop[3]<<endl; 
    return 0; 
} 

Answer 1

括号括起来，在您的宏：

#define min(a,b) ((a)<(b)?(a):(b)) 
#define max(a,b) ((a)>(b)?(a):(b)) 

正因为如此，这样的：

drop[i] = min(drop[i], max(start, drop[i-start]+1)); 

是扩大到这个（不带括号）：

drop[i] <start> drop[i-start]+1 ? start: drop[i-start]+1 ? drop[i] : start > drop[i-start]+1 ? start: drop[i-start]+1; 

可能无法按照您的顺序评估d。使用括号会强制执行正确的操作顺序。

如注释中所述，如果宏参数多次计算，则不应使用具有副作用的表达式的宏。

Answer 2

C++已经在<algorithm>中定义了std::min和std::max。您可以将您的代码更改为纯C++版本

#include <iostream> 
#include <algorithm> 
#include <limits> 
using namespace std; 

int main(int argc, char ** argv) { 
    int N = 100; 
    int * drop = new int[N + 1]; 
    drop[0] = 0; 
    drop[1] = drop[2] = 1; 
    for (int i = 3; i <= N; ++i) { 
     drop[i] = numeric_limits<int>::max(); // <limits> 
     for(int start = 1; start <= i; ++start) 
      drop[i] = min(drop[i], max(start, drop[i - start] + 1)); // <algorithm> 
    } 
    cout << drop[3] << endl; 
    return 0; 
} 

Answer 3

不是一个答案，这是一个恳求所有开发人员在那里：请不要使用这样的宏。 C++为这些目的提供了模板函数。请记住，宏只是替代参数，而不是预先评估它们。即使你像samgak解释的那样添加圆括号，这只能解决一半的问题。考虑这样的代码：

int x = 5; 
int y = max(++x, 0); 

主叫方期望之后x=6和y=6;然而，宏将得到消耗到

int y = (++x > 0)? ++x : 0; 

造成x=7和y=7。