带指针/非指针的C ++分段错误

Question

它告诉我这里存在分段错误：

class Atelier  /* Director qui va executer les méthodes du monteur */
{

private:
MonteurVoiture *_monteur;

public:
Voiture* doTheJob();
Atelier(MonteurVoiture *mont);

};

int main()
{

MonteurVoiture *monteur=MonteurVoiture::instanceVoiture("luxe");
Atelier a(monteur);
cout << a.doTheJob() << endl;

return 0;
}

没有cout << a.doTheJob() << endl;就行了。如果我用cout替换它＆lt;＆lt; ＆＃34; OK＆＃34 ;;没关系。 因此分割错误来自这种方法。

Atelier::Atelier(MonteurVoiture *mont) { _monteur=mont; }

Voiture* Atelier::doTheJob()
 {
 _monteur->createVoiture();
 _monteur->addPortiere();
 _monteur->addSiege();
 return _monteur->getVoiture();
}

班级Voiture

class Voiture
{

private:
std::string portiere;
std::string siege;

public:
void setPortiere(std::string p);
void setSiege(std::string s);
friend std::ostream &operator<<(std::ostream &o,const Voiture *v);
};

和班级MonteurVoiture

class MonteurVoiture  
{

protected:
Voiture *v;

public:
~MonteurVoiture();
Voiture* createVoiture();
Voiture* getVoiture();
virtual void addPortiere() =0;
virtual void addSiege() =0;
static MonteurVoiture* instanceVoiture(std::string type);
};

Voiture* MonteurVoiture::createVoiture() { return new Voiture(); }

Voiture* MonteurVoiture::getVoiture() { return v; }

提前致谢

Answer 1

我用了Valgrind，它说：

==19269==    at 0x38B0D: std::string::assign(std::string const&) (in /usr/lib/libstdc++.6.0.9.dylib)
==19269==    by 0x100001E02: Voiture::setPortiere(std::string) (in ./main.exe)
==19269==    by 0x1000018AA: MonteurVoitureLuxe::addPortiere() (in ./main.exe)
==19269==    by 0x10000120C: Atelier::doTheJob() (in ./main.exe)
==19269==    by 0x100002029: main (in ./main.exe)

MonteurVoitureLuxe的addPortiere方法

void MonteurVoitureLuxe::addPortiere() { v->setPortiere("diamand"); }

来自Voiture的setPortiere

void Voiture::setPortiere(std::string p) { this->portiere=p; }

Answer 2

快速修复

好的，首先要解决您的问题：

class MonteurVoiture  
{

protected:
       Voiture *v; // how is this being set??? 

public:
            ~MonteurVoiture();
    Voiture* createVoiture();
    Voiture* getVoiture();
    virtual void addPortiere() =0;
    virtual void addSiege() =0;
    static MonteurVoiture* instanceVoiture(std::string type);
};

Voiture* MonteurVoiture::createVoiture() { return MonteurVoiture::v = new Voiture(); }
                                         // ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^  you need this
Voiture* MonteurVoiture::getVoiture() { return v; }

问题出现是因为你没有设置Voiture * v值，只是你在doTheJob（）中使用它：

Voiture* Atelier::doTheJob()
 {
     _monteur->createVoiture();
     _monteur->addPortiere();
     _monteur->addSiege();
     return _monteur->getVoiture(); /* this is undefined without the fix in createVoiture() above */
}

总体设计问题

你的对象设计非常奇怪。你有一辆汽车（Voiture级）然后你在MonteurVoiture中对它进行修改是控制汽车的吗？

您可以考虑以下结构：

class Option {
public:
      virtual std::string name() = 0; /* name of the option */
      void add(Voiture &v) {
             v.addOption(this);
      }   
      virtual std::string getDetails() = 0;  /* details of the option */
};

class optionDeSeiges: public Option {
public:
      virtual inline std::string name() { return "Sieges"; }
};

virtual std::string getDetails() {

}

class Voiture {
      std::List<Option *> options; /* an array of options */
      void addOption(Option *o) {
           options.add(o);
      }
}

注意：上面大部分是伪代码，它应该向您展示替代概念，从中获取想法不要逐字复制。