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。