为什么我的setter方法会产生错误的访问错误

时间:2018-10-01 06:59:21

标签: c++ memory memory-leaks

错误访问意味着我正在尝试访问不存在的内存,我曾经尝试并尝试为此类分配内存,但是到处都失败了。我不知道错误的根源。它仅告诉我,setter方法是程序崩溃时。在setFName()方法中会发生错误。但主要方法是它实际发生的地方。 Nurse.hpp

#ifndef Nurse_hpp
#define Nurse_hpp

#include <stdio.h>
#include <string>
#include <stdlib.h>
using namespace std;

class nurse{
private:
    string firstName;
public:
    nurse() {
        firstName = "jim";
    }
    string getFName() {return firstName;}
    void setFName(string fName) {firstName = fName;} // Thread 1: bad access 0x0
};

#endif /* Nurse_hpp */

这是实际发生错误的地方 main.cpp

#include <cstdint>  // ::std::uint64_t type
#include <cstddef> // ::std::size_t type
#include <iostream>
#include <fstream>
#include <stdlib.h>
#include "nurseHolder.hpp"
using namespace std;

nurseHolder *l = new nurseHolder();

int main() {


 return 0;
}

最后是导致问题的类 NurseHolder.hpp

#ifndef Nurses_hpp
#define Nurses_hpp

#include <stdio.h>
#include <vector>
#include <stdlib.h>
#include "Nurse.cpp"
using namespace std;

class nurseHolder{
private:
    int nurse_cnt;
    int nurse_cap;
    vector<nurse> nurse_list;
public:


    nurseHolder()  {
    nurse_cnt = 0;
    nurse_cap = 10;
    for(int i= 0; i < 11; i++){
        nurse_list[i].setFName("na");
      }
    }
    vector<nurse> &getNurseList() { return nurse_list;}

};

#endif /* Nurses_hpp */

如果有很多代码,我试图使它尽可能紧凑。

为了使代码正常工作,我做了以下更改:

nurseHolder()  {
        nurse_cnt = 0;
        nurse_cap = 10;
        for(int i= 0; i < 11; i++){
            nurse l;
            nurse_list.pushback(l);
          }
        }

这是正确的方法吗?

1 个答案:

答案 0 :(得分:4)

您的向量nurse_list的大小为0。因此,不能使用[]运算符来设置名称。 有两种方法可以纠正此问题:

  1. 设置矢量的初始大小,然后使用[]设置名称。
  2. 使用push_back将元素添加到向量中。

第一种方法。

nurse_list.resize(noOfTotalNurses). 
nurse_list[i].setFName("name");

第二种方法。

nurse tNurse; //local nurse object
tNurse.setFName("name");
nurse_list.push_back(tNurse);