ローカルメンバーとしてstd :: mapを使用したメモリアクセス違反

Question

のメソッドv_init()の呼び出し中に、m_Lights=tmp;行のメモリアクセス違反エラーが発生しました。 m_Lightsをm_LightsManagerのインスタンスで作成する必要はありませんか？このエラーが発生するのはなぜですか？

class LightManager 
{ 
private: 
    std::map<sint32,Light> m_Lights; 
    public: 
    LightManager (void); 
    ~LightManager (void); 
    void v_init(); 
}; 

LightManager ::LightManager (void) 
{ 

} 



LightManager ::~LightManager (void) 
{ 
} 

void LightManager ::v_init() 
{ 
    Light tL; 
    std::memset(&tL,0,sizeof(Light)); 
    std::map<sint32,Light> tmp; 
    tmp.insert(std::pair<sint32,Light> (-1,tL)); 
    m_Lights=tmp; 
} 

class Algo 
{ 
private:  
    LightsManager m_LightsManager; 
.... 
public: 
Algo(); 
void Init(); 
}; 

    Algo::Algo() 
    { 
    Init(); 
    } 


void Algo::Init() 
{ 

    m_LightsManager.v_init(); 

} 

Answer 1

Lightがtrivially-copyableない場合、

std::memset(&tL,0,sizeof(Light)); 

は未定義の動作です。これはおそらくエラーの原因です。

Answer 2

Vittoriosに加えて、答え：外部memsetコールに頼るのではなく、Lightのコンストラクタですべての初期化を行います。そして、C++でstd::memsetを使用して、あるステートメントのオブジェクトのすべてのメンバー変数を初期化しないでください。すべての変数に対して明示的に行います（通常、単純な代入を使用します。それはIMHOが属する）。

理由：Lightが導出され、基本クラスが独自のデータを定義し、オブジェクト全体を無謀に上書きして上書きします。

Answer 3

私は、エラーを解決し、問題が間違っているから来た `Light`、なぜあなたはこれをやっているとは何