Question

这是我的第一篇文章。

我试图记住这些说明，但如果我错过了某些内容，我会提前道歉。

该课程是课程作业，我们将使用三个不同的链接列表（学生，课程和交易，为了更好的单词）制作成绩簿。

我发布的块将搜索列表（假设有一个，查找具有相同信息的现有对象，这是csearch函数的作用。

如果找不到，那么它会创建一个新对象，并设置将它放在列表中，总是在尾部。

所有这一切都有效，直到它到达函数的右括号，然后它就会出现分段错误。

我花了大部分时间试图弄清楚为什么会这样，我希望你能帮助我。

    void classes::addcourse(classes section)
   {
   int courseid;
   char  title[81];
   course *temp= NULL, *search=NULL;

   cout<<"Please enter the course id number."<<endl;
   cin>>courseid;getchar();

   cout<<"Please enter a course name."<<endl;
   cin>>title;

   search=section.csearch(courseid);               
   if (search != NULL)
      {                                                  
      cout<<"This course has already been entered."<<endl;  
      return;                                            
      }

   temp=new course(courseid);
   if (head==NULL)
      {
      sethead(temp);
      settail(temp);
      }
   tail->setnext(temp);
   settail(temp);
   section.setcnum(section.getcnum()+1);
   cout<<section.getcnum()<<endl;
   temp->setname(title);
   }

如果它有用，这里是对象和容器的类定义，它们分别存储在.h文件中。

class course {
   private:
   int cid;
   int average;
   course *next;
   char name[81];

   public:
   course(int);
   void saveyourself(FILE *write);
   void loadyourself(FILE *read);

   //accessor functions
   int getcid() {return cid;}
   int getaverage() {return average;}
   course* getnext() {return next;}

   //mutator functions
   void setnext(course *val) {next=val;}
   void getname() {cout<<name;}
   void setcid (int newcid) {cid=newcid;}
   void setaverage (int newav) {average=newav;}
   void setname (char word[]) {strcpy(name, word);}
};

class classes {
   private:
   course *head;
   course *tail;
   int numcourse;

   public:
   classes();
   ~classes();
   course* csearch (int course);
   void addcourse(classes section);
   void classaverage(enrollment semester, classes section);
   void save();
   void load();

   //accessor functions
   int getcnum() {return numcourse;}
   course* gethead() {return head;}
   course* gettail() {return tail;}

   //mutator functions
   void setcnum(int num) {numcourse=num;}
   void sethead(course *val) {head=val;}
   void settail(course *val) {tail=val;}   

};

如果它出现，我使用字符串数组作为字符串，因为这是我厌倦来回切换时最终确定的。

编辑：抱歉没有添加构造函数，我忘了它们，这里是：

course::course(int id=-1)
   {
   cid=id;
   average=0;
   strcpy(name, "name");
   next=NULL;
   }


classes::classes()
   {
   numcourse=0;
   head=NULL;
   tail=NULL;
   }

我一直对未初始化的字符串感到偏执，所以这就是为什么它已经被设置为某种东西了。我通常将它们初始化为“\ 0”，但这让我担心文件I / O.

Answer 1

我不相信这是你想要的：

   cin>>title;

如果您打算使用C风格的字符数组，您可能希望使用cin.getline()，如下所示：

   cin.getline(title, 81);

这将确保您不会溢出title[]，并且它会在结尾处放置一个NUL终止符。它也会在最后吃换行，而不是放在title[]。

此处提供更多信息：http://www.cplusplus.com/reference/istream/istream/getline/?kw=istream%3A%3Agetline

闭合支撑处的分段故障

1 个答案: