C++ - セグメンテーションエラーバイナリファイルを読み込む

Question

char* professeurをバイナリファイルで読み込もうとすると、問題が発生します。これは失敗し、read()関数でセグメント化エラーが発生します。奇妙なのは他のクラスの他のクラスのload関数を読み取ると、char*のメンバーはうまく動作しますが、この場合はprofesseurが正しく書かれていても、segフォルトが発生します。だからここ

はコードです：Cours.cpp

Cours.h

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
#include <iostream> 

using namespace std; 

#include "Liste.h" 
#include "Event.h" 
#include "Professeur.h" 

class Cours: public Event 
{ 
    private: 
     char* professeur; 
     Liste<int> groupes; 
    public: 
     void save(ofstream&) const; 
     void load(ifstream&); 
}; 

void Cours::save(ofstream& o) const 
{ 
    int n=strlen(professeur); 
    char buff[60], *buff2; 

    o.write((char *)&n, sizeof(int)); 
    strcpy(buff, getProfesseur()); 
    o.write(buff, n+1); 

    groupes.save(o); 
    Event::save(o); 
} 

void Cours::load(ifstream& i) 
{ 
    int n; 
    char buff[60]; 

    i.read((char *)&n, sizeof(int)); 
    cout<<"n: "<<n<<endl; 
    if(i.read(buff, n+1))//seg fault 
    { 
     strcpy(professeur, buff); 
     cout<<"prof: "<<professeur<<endl;  
    } 
    else 
     cout<<"erreur read prof cours"<<endl; 
    groupes.load(i); 
    Event::load(i); 
} 

Answer 1

nは、それがバッファより大きくわから取得していないにするためにチェックする必要があります

。 save()で

：

int n=strlen(professeur); 

nがここに最大59でなければなりませんが - をチェックする必要があります。 load()で

：

i.read((char *)&n, sizeof(int)); 

より良いここにチェックnすぎ（最大59）。

はまた：strlen(professeur)、その後getProfesseur()：

int n=strlen(professeur); 
char buff[60], *buff2; 

o.write((char *)&n, sizeof(int)); 
strcpy(buff, getProfesseur()); 
o.write(buff, n+1); 

2つの異なる値は、データを書き込むために使用されています。

professeurのメモリも割り当てられません（少なくとも図示のコードには含まれていません）。だからload()のstrcpy(professeur, buff);も失敗します。あなたはallocateとdeallocateメモリに自分自身を持っていない道

private: 
    char* professeur; 

を

private: 
    char professeur[60]; 

へ：

あなたが同様に変更される場合があります。

そして、すべての文字列がNULLで終了することを確認してください。もちろんの

、運動がそれを可能にする場合は、代わりに（string professeur;）stringを使用し、中と<<と>>を使用してデータをストリーミングすることができます。

Answer 2

まず、名前の長さを読んでください。 Allocat as many 名前は/0の長い+1です。ファイルから読み込み、末尾に\0を追加してください。

void Cours::load(ifstream& i) 
{ 
    int n; 
    i.read((char *)&n, sizeof(int)); 
    cout<<"n: "<<n<<endl; 
    if (n <= 0) 
     return; 

    professeur = new char[n+1];    // allocate professeur 
    if (i.read(professeur, n))   // read directly to professeur 
    { 
     professeur[ n ] = ´\0`;    // write \0 at end of name 
     cout<<"prof: "<<professeur<<endl; 
    } 
    else 
    { 
     delete [] professeur; 
     professeur = nullptr; 
     cout<<"erreur read prof cours"<<endl; 
    } 

    groupes.load(i); 
    Event::load(i); 
}