2

我正在学习动态记忆，但有些事情不太对劲。我有一个接受数字作为输入的函数，并且应该创建一个该大小的数组。

class Doctor {
public:
    Doctor();
    void fillOut();
    void listPatients();
    void patientReset();
    ~Doctor();
private:
    string name;
    int numPatients;
    string *patientList;
};

Doctor::Doctor() {
    name = "";
    numPatients = 0;
    patientList = new string[numPatients];
}

（第三个代码块中最相关的代码）。

void Doctor::fillOut() 
{
    string buffer = "";
    string buffer2 = "";
    size_t found;
    bool valid = false;
    int numP = 0;
    int tester = 0;
    bool validNum = false;

    while(!valid) 
    {
        cout << "Enter doctor name: ";
        getline(cin, buffer);
        found = buffer.find_first_of("1234567890!@#$%^&*()-=_+/<>?;':][");
        if(string::npos == found) 
        {
            name = buffer;
            valid = true;
        }
    }

    while (!validNum) 
    {
        cout << "\nEnter number of patients: ";
        buffer = "";
        getline(cin, buffer);
        buffer2 = buffer;
        stringstream ss(buffer);
        if(ss >> tester) 
        {
            stringstream ss2(buffer2);
            ss2 >> numP;
            validNum = true;
        }
        else 
        {
            cout << "Not a number. Please try again." << endl;
        }
    }

    patientList = new string[numP];
    cout << patientList->size() << endl;
    for(int i = 0; i < (numP + 0); i++) 
    {
        valid = false;
        while(!valid) 
        {
            cout << "\nEnter patient " << (i + 1) << ": ";
            getline(cin,buffer);
            found = buffer.find_first_of("1234567890!@#$%^&*()-=_+,./<>?;':][");
            if(string::npos == found) 
            {
                *(patientList + i - 0) = buffer;
                //patientList[i-1] = buffer;
                valid = true;
            }
            else 
            {
                valid = false;
            }
        }
    }
}

然后我尝试显示列表的内容。

void Doctor::listPatients() 
{
    cout << "size: " << patientList->size() << endl;
    cout << "\nDoctor: " << name << endl;
    for(int i = 0; i < (patientList->size() - 1); i++) 
    {
        cout << "Patient " << (i+1) << ": " << patientList[i] << endl;
    }
    cout << "end patients" << endl;
}

但由于某种原因，我提交的大小不是数组的大小。例如，在fillOut()我有函数输出大小。它每次输出大小为0。然后在 中listPatients()，我有一些东西可以再次打印尺寸，以验证我做对了。如果我最初输入 3，它在此函数中输出 5。

我完全迷惑了。

4

3 回答 3

6

该行patientList->size()等效于，数组patientList[0].size()中初始字符串的长度。patientList在您刚刚分配数组时，结果始终为零；在其他情况下，它是第一个字符串的长度。

在 C++ 中制作容器的首选方法是std::vector或std::array。在您的情况下使用std::vector更合适，因为您的代码是patientList动态分配的。

于 2012-07-14T00:36:42.430 回答
2

好的，我发现了你的问题。您正在声明一个字符串数组。但是原始数组实际上只是指向数组内容的指针。当您调用 ->size 时，它​​会取消引用该指针，该指针指向数组中的第一个字符串并告诉您该字符串的大小。

如果您想要一个实际知道其大小的数组，请使用 std::array。

于 2012-07-14T00:24:06.153 回答
2

好的，所以您将 patientList 声明为 string* 但是当您对此调用 size() 时，实际上是在您创建的数组中的第一项上调用它。编译代码的唯一原因是因为 string 具有 size() 函数，并且由于这是数组中包含的类型，因此您可以摆脱它，因为 patientList 实际上指向数组开头的项目（字符串）。通常，数组没有 size() 函数。

您将遇到的另一个问题是在您的循环中

for(int i = 0; i < (patientList->size() - 1); i++) {
                 ^
                 ^  
    cout << "Patient " << (i+1) << ": " << patientList[i] << endl;
}

这将导致您的循环在 PatientList结束之前终止一个。你应该使用

for(int i = 0; i < patientList->size(); i++) {                                      
    cout << "Patient " << (i+1) << ": " << patientList[i] << endl;
}

你最好在这里使用 std::vector 来放入你的字符串。在这里也要小心

   if(string::npos == found) {
        *(patientList + i - 0) = buffer;
        //patientList[i-1] = buffer;
        valid = true;
    }
    else {
        valid = false;
    }

因为你只会在你的 if 语句为真的情况下分配给你的数组 - 当它为假时数组中的索引未初始化。老实说，我会从头开始并使用字符串向量。

于 2012-07-14T00:38:10.857 回答