我在这里写了一个快速排序:
void swap(int& a, int& b);
int mid(int lo, int hi);
// My quicksort implementation
void sort(int vec[], int lo, int hi)
{
int mid;
if (hi > lo) {
int i = lo + 1;
int j = hi;
int p = mid(lo, hi);
swap(vec[lo], vec[p]);
mid = vec[lo];
while (i < j) {
if (vec[i] <= mid) {
i++;
} else {
while (i < --j && vec[j] >= mid);
swap(vec[i], vec[j]);
}
}
i++;
swap(vec[lo], vec[i]);
sort(vec, lo, i);
sort(vec, j, hi);
}
}
void swap(int& a, int& b)
{
int temp = a;
a = b;
b = temp;
}
int mid(int lo, int hi)
{
return lo + ((hi - lo) / 2);
}
我尝试使用g++ -g -c array.cpp -o array.o
编译到目标文件
我收到这个错误:
array.cpp:24:14: error: called object type 'int' is not a function or function
pointer
int p = mid(lo, hi);
~~~^
1 error generated.
一切看起来都很正确。任何人都可以帮我弄清楚出了什么问题吗?
答案 0 :(得分:23)
您的局部变量mid
在更接近使用点的范围内声明,因此它“隐藏”mid()
函数;编译器认为您正在尝试“调用”一个无效的整数。重命名局部变量以解决此问题:
int midpoint;
if (hi > lo) {
int i = lo + 1;
int j = hi;
int p = mid(lo, hi);
swap(vec[lo], vec[p]);
midpoint = vec[lo];
...
}
注意:您也可以使用::mid(lo, hi)
而不是重命名变量,但这会让您的程序的读者感到困惑。
答案 1 :(得分:1)
int mid(int lo, int hi); // here you declared mid as function and defined
// it later
// My quicksort implementation
void sort(int vec[], int lo, int hi)
{
int mid; // but here you declared mid as local variable
if (hi > lo) { // it will shadow int mid(int lo, int hi);
int i = lo + 1;
int j = hi;
int p = mid(lo, hi); // so this is error, mid is integer not a function
您可以更改算法中变量的名称,或使用范围解析运算符::mid(lo, hi)
来访问先前在全局范围内定义的mid
函数