0

它告诉我这里有一个分段错误：

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 << "ok"; 没关系。所以分段错误来自这种方法。

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; }

提前致谢

4

2 回答 2

0

我使用了 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; }
于 2013-07-28T17:26:59.353 回答
0

快速解决

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

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);
      }
}

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

于 2013-07-28T17:27:18.307 回答