Классы, наследственность Опции

[Account]

Есть вот такая программа.

#include <iostream.h>
#include <conio.h>
#include <stdlib.h>
#include <dos.h>
#include <graphics.h>
#include <string.h>
#include <math.h>
void move(int i);
// è½áßß öêâôÉÇ
class Figure
{
protected:
  int Color;
  int CenterX;
  int CenterY;
public:
  Figure(int iCenterX, int iCenterY)
  {
    Color = RED;
    CenterX = iCenterX;
    CenterY = iCenterY;
  }
  void SetNewColor(int NewColor)
  {
    Hide();
    Color=NewColor;
    Show();
  }
  virtual void Show() = 0;
  virtual void Hide()
  {
    int prev_col = Color;
    Color=getbkcolor();
    Show();
    Color = prev_col;
  }
  void Move(int DeltaX, int DeltaY)
  {
    Hide();
    CenterX+=DeltaX;
    CenterY+=DeltaY;
    Show();
  }
  virtual ~Figure() {};
};

class Butterfly: public Figure
{
private:
int Radius;
  int RadiusX;
  int RadiusY;
  int Stangle;
  int Endangle;
public:
  Butterfly(int iCenterX, int iCenterY, int iStangle,int iEndangle,int iRadiusX,int iRadiusY,int iRadius) :
    Figure(iCenterX, iCenterY)
  {
   Stangle=iStangle;
   Endangle=iEndangle;
   Radius=iRadius;
     RadiusX = iRadiusX;
     RadiusY = iRadiusY;
  }
  void Show();
};

class Hexe: public Figure
{
private:
  int RRadius;
  int rRadius;
public:
  Hexe(int iCenterX, int iCenterY, int iRadius) :
    Figure(iCenterX, iCenterY)
  {
     RRadius = iRadius;
     rRadius = (sqrt(3)/2)*RRadius;
  }
  void Show();
};

class Combo:public Hexe,public Butterfly
{
public:
  Combo(int iCenterX, int iCenterY, int iStangle,int iEndangle,int iRadiusX,int iRadiusY,int iRadius) :
    Hexe( iCenterX, iCenterY,  iRadius),Butterfly(iCenterX, iCenterY, iStangle,iEndangle, iRadiusX, iRadiusY,iRadius)
{}
  void Show();
  void Hide();
  void Move(int x);
   void SetNewColor(int colr);
  ~Combo(){};
};

void Hexe::Show()
{
  int prev_color=getcolor();
  setcolor(Color);
  line(CenterX, CenterY+RRadius,CenterX-rRadius,CenterY+RRadius/2);
  line(CenterX-rRadius,CenterY+RRadius/2,CenterX-rRadius,CenterY-RRadius/2);
  line(CenterX-rRadius,CenterY-RRadius/2,CenterX,CenterY-RRadius);
  line(CenterX,CenterY-RRadius,CenterX+rRadius,CenterY-RRadius/2);
  line(CenterX+rRadius,CenterY-RRadius/2,CenterX+rRadius,CenterY+RRadius/2);
  line(CenterX+rRadius,CenterY+RRadius/2,CenterX, CenterY+RRadius);
  setcolor(prev_color);
}

void Butterfly::Show()
{
  int prev_color=getcolor();
  setcolor(Color);
  circle(CenterX, CenterY-RadiusY, Radius/5);
  ellipse(CenterX, CenterY+Radius/5, Stangle,Endangle,RadiusX,RadiusY);
  line(CenterX-(sqrt(3)/2*Radius),CenterY-Radius/2,CenterX-20,CenterY);
  line(CenterX-20,CenterY,CenterX-(sqrt(3)/2*Radius),CenterY+Radius/2);
  line(CenterX-(sqrt(3)/2*Radius),CenterY-Radius/2,CenterX-(sqrt(3)/2*Radius),CenterY+Radius/2);
  line(CenterX+(sqrt(3)/2*Radius),CenterY-Radius/2,CenterX+20,CenterY);
  line(CenterX+20,CenterY,CenterX+(sqrt(3)/2*Radius),CenterY+Radius/2);
  line(CenterX+(sqrt(3)/2*Radius),CenterY-Radius/2,CenterX+(sqrt(3)/2*Radius),CenterY+Radius/2);
  setcolor(prev_color);
}

 void Combo::Show()
{
  Butterfly::Show();
  Hexe::Show();
  }

  void Combo::Hide()
 {
   Butterfly::Hide();
   Hexe::Hide();
 }

 void Combo::Move(int x)
{ 
  Butterfly::Move(15*x,15*x);
  Hexe::Move(15*x,15*x);
 }

void Combo::SetNewColor(int colr)
 {
   Butterfly::SetNewColor(colr);
   Hexe::SetNewColor(colr);
 }

int main()
{   int n_menu;
while (1)
  {
    clrscr;
   cout<<"1-Butterfly\n2-Hexe\n3-Combo\n4-Exit\n";
   cin>>n_menu;
  int gdriver = DETECT, gmode, errorcode;
  initgraph(&gdriver, &gmode,
     "../BGI");
  errorcode = graphresult();
  if (errorcode != grOk)
  {
     cerr<<"Graphics error";
     cerr<<grapherrormsg(errorcode);
     exit(1);
   }
  Butterfly F1(100,100,0,360,20,75,100);
  Hexe F2(100,100,100);
  Combo F3(100,100,0,360,20,75,100);
  switch(n_menu)
  {
  case 1: cleardevice();F1.Show();break;
  case 2: cleardevice();F2.Show();break;
  case 3: cleardevice();F3.Show();break;
  case 4: cout<<"Bay! Bay!\n";delay(750);return 0;
  }
  getch();
  int maxcolor = getmaxcolor();
  int direction = 1;
  for (int color = 0; !kbhit(); color++)
  {
     if (color>maxcolor)
       {color=0; direction*=-1;}

  switch (n_menu)
  {
  case 1: F1.SetNewColor(color);F1.Move(10*direction,10*direction);break;
  case 2: F2.SetNewColor(color);F2.Move(10*direction,10*direction);break;
  case 3: F3.SetNewColor(color);F3.Move(direction);break;
  }
  delay(50);
  }
  closegraph();
  }
}

По меню по очереди можно вывести бабочку, потом шестиугольник правильный, потом комбинированно бабочку в шестиугольнике. Бабочки и шестиугольник наследники от общего класса фигуры, а вот комбо от бабочки и шестиугольника. Не пойму вот следующее по идее класс Combo должен ведь унаследовать методы Move и SetNewColor, так как он наследник от Butterfly и Hex, которые в свою очередь наследуют данный метод от Figure. Но если выкинуть эти методы как сделано в выше представленном коде и напрямую обращаться например F3.SetNewColor(color);F3.Move(10*direction,10*direction); естественно компилятор материться. Так как правильно оформить наследственность и добраться до данных методов, не прописывая их в классе Combo и естественно не описывая так

void Combo::Move(int x)
{ 
  Butterfly::Move(15*x,15*x);
  Hexe::Move(15*x,15*x);
 }

void Combo::SetNewColor(int colr)
 {
Butterfly::SetNewColor(colr);
Hexe::SetNewColor(colr);
 }

P.S. Как в спойлер код скрывать, а то как то не красиво длинный пост получается?

[IUnknown]

но компилятор ругается в конструкторе Combo на то что не найден конструктор для инициализации базового класса Figure

Я знал, что так и будет... Вас что, не учили виртуальному наследованию?

Сначала - как это происходит при наследовании обычном:

class base {
   // ...
};

class A : public base {
   // ... тут инициализируется базовый класс
};
class B : public base {
   // ... и тут инициализируется базовый класс
};

class C : public A, public B {
   // Здесь достаточно инициализировать прямых предков, но не ИХ предков
   C() : A(), B() {}
   // ...
};

, то есть, тебе было достаточно в списке инициализации написать только инициализацию тех классов, от которых ты наследовался, а уже они сами будут вызывать конструктор СВОИХ предков... Более того, тебе было запрещено писать в списке инициализации C() конструктор base, потому что ты от него напрямую не наследовался...

В случае с виртуальным базовым классом такой вариант не проходит. Класс C должен содержать одну копию base, поэтому надо инициализировать виртуальный базовый класс в наиболее удаленном от него потомке (чтоб потом не спрашивать себя, а откуда же был инициализирован базовый класс, из A или из B?). У тебя наиболее удаленный - не Hexe и не Butterfly, а Combo. Компилятор при виртуальном наследовании ждет инициализации не только Hexe + Butterfly, но и инициализации Figure. Если ты не записываешь его конструктор в список инициализации - будет вызван дефолтный конструктор (без параметров). У тебя и его нет, компилятор старый, не создает конструкторов самостоятельно. Надо добавить инициализацию ручками (все равно ты и в Hehe и в Butterfly одинаково вызываешь конструктор Figure):

class Combo : public Hexe, public Butterfly
{
public:
    Combo(int iCenterX, int iCenterY, int iStangle, int iEndangle, int iRadiusX, int iRadiusY, int iRadius) :
        Hexe(iCenterX, iCenterY,  iRadius),
        Butterfly(iCenterX, iCenterY, iStangle, iEndangle, iRadiusX, iRadiusY, iRadius),
        Figure(iCenterX, iCenterY)
        {}
// ...

, теперь компилятор знает, чем инициализировать базовый класс, и проблем не будет...

Сообщение отредактировано: IUnknown - 5.07.2011 18:03