為什么這種反向算法不起作用？

Question

#include <iostream>

void intswap ( int * pa, int * pb )
{
    int temp = *pa;
    *pa = *pb; 
    *pb = temp;
}

void reverse ( int * arr, size_t n )
{
    int * end = arr + n;
    while (arr != end && arr != --end) intswap(arr++,end);
}

int main ( )
{
    int myArray [] = {1, 69, 0, -123, 1377};
    size_t n = sizeof(myArray)/sizeof(int);
    reverse(myArray, sizeof(myArray, n));
    for ( size_t i = 0; i < n; ++i ) std::cout << myArray[i] << " ";

    return 0;
}

產生

-123 0 69 1 1377

我究竟做錯了什么？ 畢竟，它是與此處所示相同的算法： http : //www.cplusplus.com/reference/algorithm/reverse/

Answer 1

您的問題是，由於逗號運算符sizeof(myArray, n)與sizeof(n)完全相同，在您的情況下顯然為4個字節。 您可能只想說n 。

Answer 2

Mark B已經在您的代碼中指出了問題。 這是提高代碼可讀性的建議。 您可以更改reverse到：

void reverse ( int * arr, size_t n )
{
   int * end = arr + n;
   while ( arr < end )
   {
      --end;
      intswap(arr, end);
      ++arr;
   }
}

使其更具可讀性。