【发布时间】:2018-10-22 15:51:08
【问题描述】:
我有两个重载函数:“ChooseElements”,它从传递的数组中选择元素,以及“SortElements”,它对传递的数组的元素进行排序。一对使用 INT 数据,另一对使用 FLOAT。
int * ChooseElements(int * X, int n, int & m)
{
int * Y = NULL;
for (int i = 0; i < n; i++)
{
if (X[i] > 0)
{
if (Y == NULL)
{
m = 1;
Y = new int[1];
Y[0] = X[i];
}
else
{
m++;
Y = (int *)realloc(Y, sizeof(int) * m);
Y[m - 1] = X[i];
}
}
}
return Y;
}
float * ChooseElements(float * X, int n, int & m)
{
float * Y = NULL;
for (int i = 0; i < n; i++)
{
if (X[i] > 0)
{
if (Y == NULL)
{
m = 1;
Y = new float[1];
Y[0] = X[i];
}
else
{
m++;
Y = (float *)realloc(Y, sizeof(float) * m);
Y[m - 1] = X[i];
}
}
}
return Y;
}
和
int * SortElements(int m, int *& Y)
{
for (int i = 1; i < m; i++)
{
for (int j = 0; j < m - i; j++)
{
if (Y[j] > Y[j + 1])
{
int Temp = Y[j];
Y[j] = Y[j + 1];
Y[j + 1] = Temp;
}
}
}
return Y;
}
float * SortElements(int m, float *& Y)
{
for (int i = 1; i < m; i++)
{
for (int j = 0; j < m - i; j++)
{
if (Y[j] > Y[j + 1])
{
float Temp = Y[j];
Y[j] = Y[j + 1];
Y[j + 1] = Temp;
}
}
}
return Y;
}
我想要做的是将第一个函数作为参数传递给第二个函数。像这样:
int n, m;
int * X = NULL, * Y = NULL;
/* ...
Some code in which n and X are initialized
... */
Y = SortElements(m, ChooseElements(X, n, m));
但是,当我尝试这样做时,Visual Studio 2017 告诉我:
没有重载函数“SortElements”的实例与参数列表匹配
参数类型有:(int, int *)
如果我这样做:
Y = ChooseElements(X, n, m);
Y = SortElements(m, Y);
一切正常。
如果我删除重载并只保留 INT 对并再次尝试
int n, m;
int * X = NULL, * Y = NULL;
/* ...
Some code in which n and X are initialized
... */
Y = SortElements(m, ChooseElements(X, n, m));
我遇到了另一个问题:
int *ChooseElements(int *X, int n, int &m)
引用非常量值的初始值必须是左值
我做错了什么?我的老师要求使用另一个函数作为参数的函数。我写的东西不起作用,我不知道在这里可以做什么。
【问题讨论】:
-
从
SortElements():int * SortElements(int m, int *Y)中的数组指针中删除引用。实际上,没有必要,因为您不想在SortElements()中重新分配数组。 (数组Y的内容可能会改变,但不是它的起始地址。) -
通过上述建议的更改,它将毫无怨言地编译:ideone.
-
我对@987654333@ 中的
m感觉有点不好。如果在ChooseElements(X, n, m)中更新第二个m之前将第一个m作为参数传递怎么办?我找到了SO: Order of evaluation in C++ function parameters 关于这个。答案反而强化了我的坏感觉...... -
@Scheff 成功了!万分感谢!我想知道我是怎么错过的。
-
@Scheff 你是对的,这根本不好。我会努力弄清楚的。
标签: c++