C++派生クラスのコールベースクラスのメソッドは、セグメンテーションフォールトに

Question

を与える、私は次のような基底クラスと派生クラスがあるとします。

#include "Sprite.h" 
class Base{ 
    private: 
    int X; //location 
    Sprite* sprite; 
    public: 
    Base(Sprite* sprite){ 
     sprite = new Sprite(some parameter); 
    } 
    int getLocation(){ 
      return X; 
    } 
    int getWidth(){ 
     return sprite->getWidth(); 
    } 
} 


class Derived : public Base{ 
    private: 
     Sprite* sprite; // here I have to redefine it in constructor 
    public: 
     Derived(Sprite* sprite); // it's different that base constructor 
{ sprite = new Sprite(some parameter); 
    sprite->setPriperty(some parameter); 
} 
} 

私が派生呼び出されると::のgetLocation（）他の場所から、エラーはありません。しかし、私はDerived :: getWidth（）を呼び出すと、私にセグメント化エラーが発生しました。私は、フォールトを避けるために、ベースクラスから同じコードをコピーする必要があります（つまり、getWidthのコードを派生クラスにコピーする必要があります）。私も "Base :: getWidth;を使って"しようとしましたが、まだ、私にセグメンテーションフォールトを与えました。基本クラスメソッドにポインタが含まれていると思われる場合

Answer 1

まず最初に、セグメンテーションフォールトとは何ですか？ Javaの場合はNullPointerException、C＃の場合はNullReferenceExceptionと類似しています。これは、割り当てられていないメモリーにアクセスしようとしたときに発生します。

どうしてですか？ spriteポインタのためにメモリを割り当てるのを忘れてしまったので、それはあらかじめ用意されていないメモリチャンクを指しています。あなたは危険な住みたい場合は、以下のんでした

Base::Base(Sprite* sprite) { 
    this->sprite = new Sprite(sprite); 
} 

：

Base::Base(Sprite* sprite) { 
    this->sprite = sprite; 
} 

しかし、それは安全ではないですが、スプライトため、このようなnew演算子を使用して、コンストラクタでそれを初期化する必要がありますスタックに割り当てることができ、そのようなコードは未定義の動作を引き起こし、あなたのプログラムに大混乱を台無しことができます：

Base * getBase() { 
    Sprite sprite; 
    //some code operating on sprite 
    return new Base(&sprite); 
} 

それが原因で終了した後、危険なのですgetBase機能メモリにはもうspriteがありません。

更新：

だから、基本的に、私はあなたのコメントから結論何があなたのBaseクラスの適切なコンストラクタを呼び出すことはありませんということです。あなたのDerivedコンストラクタは次のようになります。これを行わない場合

Derived(Sprite* sprite): Base(sprite) { 
    ... 
} 

は、あなたの基本クラスのspriteの値を設定しないでください。派生クラスのspriteフィールドは、Baseクラスのspriteフィールドをシャドーイングしています。それは影にしたため、グローバルiが変更されていない例で

int i = 0; 
void fun() { 
    int i = 20; 
} 

：それはようなものです。

Derivedクラスのspriteフィールドにアクセスしていたので、コードはDerivedクラスのgetWidth()メソッドを複製した後に機能します。明確にするには、getWidth()メソッドをオーバーライドしないでBaseクラスのspriteフィールドにアクセスしています。オーバーライドすると、Derivedクラスのフィールドにアクセスします。

Answer 2

おそらく、spriteメンバ変数をコンストラクタ内に設定することを忘れている可能性があります。次のようなことをしてください：Base(Sprite * s): sprite(s) {}またはBase(Sprite * s) { this->sprite = s; }物事はBaseコンストラクタをスタンドとしてがDerivedのインスタンス上で動作するようにgetLocation()ためDerivedコンストラクタから有効なSprite*を渡さなければならないことをDerived(Sprite* sprite): Base(sprite) {};

注：

Derived(Sprite* sprite)コンストラクタは、このような親コンストラクタにspriteを渡す必要があります。