所以我正在尝试使用排序功能(类似于气泡)并将其传递给它。如果该对象更大(按字母顺序),则切换然后返回true并将其与之前切换。虽然在mySort()
里面的if语句里面说“在arr [j]中没有匹配operator []”,但我仍然收到错误但是根据我的理解我正在传递一个对象数组吗?为什么会发生这种情况,我该如何解决?
这是驱动程序
#include <iostream>
#include <fstream>
#include <string>
#include "phoneEntry.h"
using namespace std;
void mySort(PhoneEntry &arr, int size)
{
bool inOrder = false;
string temp;
for (int i = size - 1; i > 0 && !inOrder; i--)
{
inOrder = true;
for (int j = 0; j < i; j++)
{
if(arr.alphaGreater(arr[j]))
{
inOrder = false;
temp = arr[j];
arr[j] = arr[j + 1];
arr[j + 1] = temp;
}
}
}
};
int main()
{
const int MAXNUM = 500;
PhoneEntry entry[MAXNUM];
ifstream filezilla;
filezilla.open("phone.txt");
int count = 0;
if(filezilla)
{
while(count < MAXNUM && entry[count].readEntry(filezilla))
{
count++;
mySort(entry[count], count);
}
for(int i = 0; i < count; i++)
{
entry[i].writeEntry(cout) << endl;
}
}
else
{
cout << "404" << endl;
}
return 0;
}
排序文本(http://pastebin.com/HE8Rsmbg)
答案 0 :(得分:7)
arr
应该是一个数组,而不是引用,例如PhoneEntry arr[]
您应该将整个数组传递给排序,而不是单个元素,如下所示:mySort(entry, count);
除此之外,您的代码显示正常。
我应该补充一点,这不是C ++-ish解决方案:在C ++中管理数组的首选方法是使用标准库中的std::vector<T>
容器。关于向量的好处是你不需要“在侧面”传递它们的大小。
答案 1 :(得分:2)
您可以使用左指针表示法 - mySort(PhoneEntry * arr, int size)
或数组表示法 - mySort(PhoneEntry arr[], int size)
。
如果要在调用函数时传递整个数组,只需执行mySort(entry, count)
。
答案 2 :(得分:1)
arr
不是您方法中的数组。
将您的方法签名更改为
void mySort(PhoneEntry *arr, int size)
并使用
调用您的方法 mySort(entry[count], count);
答案 3 :(得分:1)
从我的理解我正在传递一个对象数组吗?
不,您没有传递对象数组。您正在将引用(由函数标头中的&
表示)传递给位于PhoneEntry
位置count
的{{1}}元素1}}数组。您可能在entry
的标头中意味着PhoneEntry* arr
- 这需要指向mySort
实例的指针,并且因为数组的名称可以被解释为指向第一个元素的指针在该数组中,您只需将PhoneEntry
作为第一个参数传递给entry
。
答案 4 :(得分:0)
代替:
void mySort(PhoneEntry * arr, int size)
而不是:
// Wrong
mySort(entry[count], count);
...执行其中一项(视情况而定):
// Always passes the start of the array, "entry[0]":
mySort(entry, count);
// Passes a pointer to a particular entry, onwards:
mySort(&entry[count], count);