右大括号处的分段错误

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