Question

我正在尝试传递Student

的数组

进入函数processStudent(string myFilename, Student* myArray, int &mySize)。但它给了我不同的错误。

Student（）什么都不做，但我尝试给它们分配某种值，它仍然给出完全相同的错误信息：

主要是我有这个：

// Create an array of students, with the size of theMax (256)
Student* arrayOfStudent= new Student[theMax];

// An integer that will keep track of actually how many students
// Because when we loop, we want to loop within the cell
// that actually have data or student.
int actualSize = 0;

// Invoke the helper function to set up the array of students
// It passed the arryOfStudent by reference, so the changes
// inside of the function will be reflected when it returns
processStudent(filename, arrayOfStudent, actualSize);

功能如下：

void processStudent(string myFilename, Student* myArray, int& mySize)
{
    // Something here, but removed still gives that error
}

//在班级学生的cpp文件

中

Student::Student() 
{
    // Nothing here
}

错误消息：

new-host-2:csci135p1 george$ g++ -Wall -o csci135p2main csci135p2main.cpp
Undefined symbols for architecture x86_64:
  "Student::Student()", referenced from:
      _main in cc3fTXti.o
ld: symbol(s) not found for architecture x86_64
collect2: ld returned 1 exit status

我一直在剥离和删除我的代码，但这个错误只是不会消失。我想创建这个数组，并将它传递给processStudent函数，这样它就可以在读取文件时设置每个数组。

Answer 1

错误消息告诉您答案：

学生*'到'学生**'

更改myArray中的processStudent()参数的类型：

void processStudent(string myFilename, Student** myArray, int& mySize)
{
}

当一个数组传递给一个函数时，它会衰减到一个指针：

void some_func(char* b) {...}

char buf[100];
some_func(buf);

当您将数组衰减的arrayOfStudent传递给指针时，恰好是数组是一个指针数组，因此**。

链接器抱怨它找不到Student的默认构造函数的定义。该定义包含在Student.cpp文件中，因此编译所有源文件以解决链接器错误：

g ++ -Wall -o csci135p2main csci135p2main.cpp Student.cpp

Answer 2

第一个错误（自从我写这个以来似乎已从问题中删除）告诉你processStudent的第二个参数是一个指向指针，Student**，而不是指针Student*。这是奇怪的，因为你用Student*参数显示你的声明;你确定这是你代码中的实际声明吗？你在某个地方有不同的声明吗？

第二个错误可能是因为您没有链接包含构造函数定义的单元。你只用一个源文件（csci135p2main.cpp）调用编译器，我猜你在不同的源文件中定义了构造函数。

Answer 3

你应该问自己一些可以帮助你的问题：

“如何创建学生的新实例？”
好吧，我这样做：Student* s = new Student(); - 它创建新对象并将引用存储为指针（Student*）

“那么如何创建10个Student的数组呢？”
好吧，它可能是一个指向新Student的指针数组，我可能不得不多次调用new ...通过这样思考你很容易就会得到类似这样的东西：

Student** students = new Student*[10];
for (int i = 0; i < 10; ++i)
    students[i] = new Student();

...这意味着，当您清理它时，您必须在每个delete上调用Student*加上您必须清理阵列本身：致电{{1}在delete[]上 - 当你意识到丑陋的内存管理与指针数组连接时，它应该让你找到更简单，更好的方法来实现它，因此你应该最终使用Student**而不是数组，对象而不是指针，如果可能的话，如果不可能，那么至少使用智能指针而不是裸指针。

Answer 4

你最好使用Student的向量（更喜欢只使用基本类型的数组（int，char ..）和其他所有的向量）。

#include <vector>

std::vector<Student*> arrayOfStudent = new vector<Student*>();
/* ... */
processStudent(filename, arrayOfStudent, actualSize);

和

void processStudent(std::string& filename, vector<Student*>& arrayOfStudent, int& actualSize) {
/* ... */
}

我不确定这是否会解决您的问题，但至少可以更好地使用......

Answer 5

您的代码容易出错，难以维护异常不安全的C ++。在现代C ++中，您应该使用std::vector<Student>，Student类实现移动语义，或vector<shared_ptr<Student>>或vector<unique_ptr<Student>>基于所有权（共享或你的Student个实例。

如果学生数组是输入/输出参数，那么你的函数原型应该是这样的：

void processStudent(const std::string & filename, std::vector<Student> & students);

相反，如果processStudent函数创建了学生数组（例如，基于文件的内容），只需将其作为返回值返回（由于移动语义，这非常有效）：

std::vector<Student> processStudent(const std::string & filename);

尝试将对象数组传递给函数时获取错误

5 个答案: