Probleme mit Vererbung

[BasisLer (Gast)]

Hallo Leute,

habe die Dateien im Anhang mit Visualc++ 2008 kompiliert, um eine 
einfache Vererbung von Klassen auszuprobieren.
Hierbei habe ich die Klassen-Deklarationen in eigene Header-Dateien 
gepackt.
Die Klasse "CMinenleger" soll eine abgeleitete Klasse der Basisklasse 
"CRaumschiff" sein.
Wenn ich das Projekt kompiliere, erhalte ich folgende Fehlermeldungen:

1>Kompilieren...
1>CMinenleger.cpp
1>d:\sonstiges\programmierung\c++_spieleprogrammierer\listing_7.11\craum 
schiff.hpp(4)  : error C2011: 'CRaumschiff': 'class' Typneudefinition
1> 
d:\sonstiges\programmierung\c++_spieleprogrammierer\listing_7.11\craumsc 
hiff.hpp(4):  Siehe Deklaration von 'CRaumschiff'
1>d:\sonstiges\programmierung\c++_spieleprogrammierer\listing_7.11\cmine 
nleger.hpp(6)  : error C2504: 'CRaumschiff': Basisklasse undefiniert


Was mache ich falsch?

Viele Grüsse, BasisLer

[Peter II (Gast)]

BasisLer schrieb:
> Was mache ich falsch?

du musst dafür sorgen das header dateien sich mehrfach includieren 
lassen..


#ifndef __NAME_DER_DATEI
#define __NAME_DER_DATEI

class CRaumschiff
{
[...}
};

#endif

[Karl H. (kbuchegg)]

Du hast in craumschiff.hpp keine Include Guards benutzt. Dadurch wird 
der Inhalt in CMinenLeger.cpp 2 mal inkludiert (einmal weil 
CMinenLeger.cpp selber den Inlucde macht und das zweite mal weil über 
CMinenleger.hpp noch ein Include erfolgt)

#ifndef RAUMSCHIFF_HPP_INCLUDED
#define RAUMSCHIFF_HPP_INCLUDED

// Basisklasse Deklaration

class CRaumschiff
{
protected:
  int m_Farbe;
  int m_Energie;

public:
  CRaumschiff ();
  ~CRaumschiff ();
  void Starten ();
  void Landen ();
};

#endif

Gewöhn dir an, in Header Files grundsätzlich einen Include-Guard zu 
setzen, denn sowas wird dir noch öfter passieren.

[BasisLer (Gast)]

Vielen Dank für die Erklärung!

BasisLer

[Karl H. (kbuchegg)]

Alternativ könntest du natürlich auch zum Schluss kommen, dich zu fragen 
was denn hier

// Abgeleitete Klasse Definition

#include <iostream>
#include "CRaumschiff.hpp"
#include "CMinenleger.hpp"

using namespace std;

// Konstruktor
CMinenleger::CMinenleger ()
{
  m_Minen = 10;
  cout << "Neuer Minenleger erstellt" << endl;

}


// Destruktor
CMinenleger::~CMinenleger ()
{
  cout << "Minenleger zerlegt" << endl;
}


CMinenleger::void Mine_Legen ()
{
  m_Minen--;
  cout << "Mine gelegt. Es sind noch " << m_Minen;
  cout << "Minen uebrig." << endl;
}

den Include von "CRaumschiff.hpp" erzwingt. Und die Antwort drauf ist: 
Nichts. In diesem Code gibt es keinen Grund, warum dieser Include 
notwendig sein sollte und daher fliegt der da raus.

(Das ist generell eine gute Idee, keine unnötigen Includes zu machen. Du 
erzeugst damit eine künstliche, unnötige Abhängigkeit zwischen 2 Files, 
die bei großen Projekten Kreise ziehen kann und den Vorteil des 'nur das 
Notwendige bei einer Änderung kompilieren' zunichte macht. In deinem 
Fall spielt das natürlich jetzt keine große Rolle und wirkt sich nur in 
einer Fehlermeldung aus, da über die Header Files ja sowieso eine 
Abhängigkeit besteht. Das sollte aber auch kein Freibrief sein, da 
nachlässig zu sein)

[BasisLer (Gast)]

Ja, das sehe ich ein. Nochmals vielen Dank.