針對結構數組的指針作用域和構造函數的創建

Question

問題的疑難解答詳述如下：爲什麼我得到錯誤說studentList和indexPtr未在此範圍內聲明？

我寫了一個應該創建一個結構數組並允許用戶在該數組中添加，刪除和打印項目的程序。該程序運行良好，但我無法獲得添加，刪除和打印功能自己正常工作。如果我在主函數中擁有所有的代碼，那麼一切工作都很完美。

現在我試圖重寫我的程序，並通過將所有信息發送到適當的函數來工作，但我不明白從我的主函數來回發送信息的正確方法。

這裏是我的聲明我的頭文件：

#include <iostream> 
#include <string> 
#define MAXSIZE 20; 
using namespace std; 

class Student 
{ 
private:  
    struct studentEntry 
    { 
     string FIRST; 
     string LAST; 
     string ID; 
     string CLASSIFICATION; 
     string MAJOR; 
    }; 
    int index; 

public: 
    void add(string &firstname, string &lastname, string &id, string &classification, string &major); 
    void remove(string &id); 
    void print(string &argument); 
    int commandCompare(string &command); 
    bool idCheck(string &id); 
    Student(); 
}; 

然後下面是我的一些代碼，從我與它的所有功能，即第一件事文件，該文件是一個構造函數我學生班。我試圖做的是建立學生結構的數組和指針的指數，這樣我可以在所有功能使用相同的陣列：

#include "students.h" 

//constructor for Student class to create students array and pointer to index 
Student::Student(){ 
    int *indexPtr = &index; 
    studentEntry *studentList = new student[MAXSIZE]; 
} 

//function to add entry to array of students 
void Student::add(string &firstname, string &lastname, string &id, string &classification, string &major) 
{ 
    string m_first = firstname; 
    string m_last = lastname; 
    string m_id = id; 
    string m_classification = classification; 
    string m_major = major; 
    int error = 0; 

    //check if id is all digits, print error 
    if (studentList[indexPtr]->idCheck(id) == false) 
    { 
     cout << "Error! ID can only contain digits." << endl; 
     error = 1; 
    } 

    //search match for existing ids 
    for(int i=0; i<indexPtr; i++) 
    { 
     //match found, print error, mark error true 
     if(id.compare(studentList[i]->student->ID) == 0) 
     { 
      cout << "Error! ID already exists." << endl; 
      error = 1; 
     } 
    } 

    else if(error != 1) 
    { 
     studentList[indexPtr]->FIRST = m_first; 
     studentList[indexPtr]->LAST = m_last; 
     studentList[indexPtr]->ID = m_id; 
     studentList[indexPtr]->CLASSIFICATION = m_classification; 
     studentList[indexPtr]->MAJOR = m_major; 
     indexPtr = indexPtr++; 
    } 
} 

我覺得一切都在我的計劃將有一次我工作獲得關於指針和我陣列的初始創建的信息。非常感謝您的幫助！

編輯： 這裏是我的主要功能的一個片段，只是爲了顯示我如何打電話學生構造，並試圖調用函數在學生類：

int main() 
{ 
    Student students; 
    string command; 
    int quit = 0; 

    //loop while user command != quit 
    do 
    {  
     cout << "students> "; 
     cin >> command; 

     //if user command = add 
     if(students.commandCompare(command) == 1) 
     { 
      string first, last, id, classification, major; 
      cin >> first >> last >> id >> classification >> major; 
      students.add(first, last, id, classification, major); 
     } 

Answer 1

在你的構造：

Student::Student(){ 
    int *indexPtr = &index; 
    studentEntry *studentList = new student[MAXSIZE]; 
} 

這兩條線是無用的：

int *indexPtr = &index; 
    studentEntry *studentList = new studentEntry[MAXSIZE]; 

他們創造e局部變量併爲其分配一些內容，但這些局部變量超出範圍。你也在做new student而不是new studentEntry。

我想你的意思是把studentList和indexPtr放在你的班級裏。

int index; 
    studentEntry *studentList; // add this line 
    int *indexPtr;    // add this line 

public: 
    void add(string &firstname, string &lastname, string &id, string &classification, string &major); 

，改變你的constuctor這樣的：

Student::Student(){ 
    indexPtr = &index; 
    studentList = new studentEntry[MAXSIZE]; 
} 

更妙的是 - 使用構造函數初始化語法：

Student::Student() 
    : indexPtr(&index), 
    studentList(new studentEntry[MAXSIZE]) 
{ 
} 

並確保您定義的析構函數釋放內存：

~Student() 
{ 
    delete [] studentList; 
} 

甚至更​​好 - 如果你使用std :: vector，你可以避免手動分配內存。

另一個問題。在這條線上：

#define MAXSIZE 20; 

你有一個額外的分號。它應該只是

#define MAXSIZE 20 

如果你有分號在那裏，那麼分號被放置在你的代碼的地方使用MAXSIZE，所以這行：

studentList = new studentEntry[MAXSIZE]; 

將成爲

studentList = new studentEntry[20;]; 

這是一個語法錯誤。

另一個問題 - 線路是這樣的：

studentList[indexPtr]->FIRST = m_first; 

應該是這樣的：

studentList[*indexPtr].FIRST = m_first; 

studentList是一個指針，但studentList[x]不是。另外，indexPtr是一個指針，所以你必須首先間接它才能得到它指向的內容。