1

下面的代码给了我一个编译错误,但我不明白我做错了什么。抱歉问了这么愚蠢的问题。

$ cat swapcstrings.cc
#include <iostream>

void swap(char*& c, char*& d) {
    char* temp = c;
    c = d;
    d = temp;
}

int main() {
    char c[] = "abcdef";
    char d[] = "ghijkl";
    std::cout << "[" << c << "," << d << "]\n";
    swap(c, d);
    std::cout << "[" << c << "," << d << "]\n";
}
$ g++ swapcstrings.cc
swapcstrings.cc: In function ‘int main()’:
swapcstrings.cc:13: error: invalid initialization of non-const reference of type ‘char*&’ from a temporary of type ‘char*’
swapcstrings.cc:3: error: in passing argument 1 of ‘void swap(char*&, char*&)’
$
4

2 回答 2

8

数组不能被修改,它们只是衰减为临时指针,它们不是真正的指针,不能交换。数组的地址无法更改,当您尝试将从数组获得的临时指针绑定到非const引用时,编译器会出错,这违反了语言规则。

声明数组,然后交换两个指向它们的指针。

char a[] = "abcdef";
char b[] = "defghi";

char* aptr = a, *bptr = b;

std::cout << "[" << aptr << "," << bptr << "]\n";
swap(aptr, bptr);
std::cout << "[" << aptr << "," << bptr << "]\n";

或者,如果您可以更改函数的原型,const char*请首先使用:

void swap(const char*& c, const char*& d) {
    const char* temp = c;
    c = d;
    d = temp;
}

const char* c = "abcdef", // These must be const char* because the arrays are
          * d = "ghijkl"; // const char[N]

std::cout << "[" << c << "," << d << "]\n";
swap(c, d);
std::cout << "[" << c << "," << d << "]\n";
于 2012-10-07T18:55:57.327 回答
3

c并且d是数组。它们将自动转换为需要指针的指针。这是你的编译器输出所说的“临时”。这样想:

char c[] = "abcdef", d[] = "ghijkl";
char *cp = (char*)c, *dp = (char*)d;
swap(cp, dp);

以上将编译,但仅交换cpand dp,而不是原始cand d。由于上面的代码为这些指针命名,您现在也可以引用它们。但是在您的原始代码中,临时对象没有名称,对它们的任何修改都表明可能存在错误。所以编译器不会让你这样做,而是会抱怨。

如果你想在它们的数组中交换 C 风格的字符串,你必须一次换一个字符:

template<size_t n> void swap(char (&a)[n], char (&b)[n]) {
  for (size_t i = 0; i != n; ++i)
    std::swap(a[i], b[i]);
}

此模板确保两个参数都是长度相同的数组。

于 2012-10-07T18:59:22.640 回答