class Kunstwerke
{
public:
	virtual ~Kunstwerke();
	virtual void drucke(ostream&)const=0;
protected:
	Kunstwerke(string,double,double*);
	Kunstwerke::Kunstwerke(const Kunstwerke&);
	Kunstwerke& operator=(const Kunstwerke&);
	string name;
	double preis;
	double *rabatt;
};

class FlacheObjekte:public Kunstwerke

class Gemaelde:public FlacheObjekte

class Graphiken:public FlacheObjekte

//Kunstwerke

Kunstwerke::Kunstwerke(string n,double p,double* r)
{
	name=n;
	preis=p;
	if(rabatt!=0)delete[]rabatt;
	rabatt=new double[sizeof(r)];
	rabatt=r;
}

//FlacheObjekte

Problem:

FlacheObjekte::FlacheObjekte(int l,int b):Kunstwerke("See",39.9,10.0)
{
	laenge=l;breite=b;
}

//Gemaelde

//Graphiken
Graphiken::~Graphiken(){cout<<"Gr"<<endl;}
Graphiken::Graphiken(int t):FlacheObjekte(50,50)
{
	typ=t;
}

int main()
{
	Kunstwerke* feld[2];
	feld[0]=new Gemaelde(1876);
	feld[1]=new Graphiken(7);

	for(int i=0;i<2;i++)
		cout<<feld[i]<<endl;

	cin.get();
	return 0;
}

• korrektem Sourcecode (bis auf deinen Fehler),
• der wörtlichen Fehlermeldung,
• nur dem für das Problem relevanten Teil?

Sonst wird das hier wieder eine Rumraterei.

FlacheObjekte::FlacheObjekte(int l,int b):Kunstwerke("See",39.9,10.0)        // ... und hier eben ein double und kein Zeiger auf double
{
	laenge=l;breite=b;
}

main:

Kunstwerke* feld[2];
     feld[0]=new Gemaelde("See",39.9,10.0,30,40,1876);

hab an den Elementinitialisiererlisten noch etwas verbessert:

Graphiken::Graphiken(string n,double p,double* r,int l,int b,int t):FlacheObjekte(n,p,r,l,b),typ(t){}

in der abstrakten BK soll das Feld rabatt dynamisch angelegt werden:

Kunstwerke::Kunstwerke(string n,double p,double* r)
{
	name=n;
	preis=p;
	if(rabatt!=0)delete[]rabatt;
	rabatt=new double[];
	rabatt=r;
}

ich nehme an im Konstrukor ist was falsch...

Grüssli

Kunstwerke::Kunstwerke(string n,double p,double* r) 
{ 
    name=n; 
    preis=p; 
    if(rabatt!=0)delete[]rabatt;  // Unnötig und gefährlich. (1)
    rabatt=new double[sizeof(r)]; // Wird immer von der Grösse 4 oder 8 sein. (2)
    rabatt=r;                     // Erzeugt ein Speicherleck. (3)
}

(1) Wenn du ein Objekt konstruierst, dann zeigt rabatt ja noch auf keinen Speicher, daher muss du keinen löschen. Was aber sein kann ist, dass rabatt irgendeinen Wert hat, da er uninitializiert ist, wodurch du am Ende versuchen könntest, irgendwo Speicher freizugeben, der dir gar nicht gehört. Folge: Undefiniertes Verhalten.
(2) r ist ein Zeiger und auf einem x86 System ist dieser 4, bzw. auf einem x64 System 8, Bytes gross. Und genau das ist es, was sizeof dir zurückgeben wird. Wenn du die Grösse des Arrays wissen willst, dann musst du die Grösse im Konstruktor mitübergeben.
(3) Durch new double[...] hast du Speicher reserviert und den Zeiger darauf in rabatt gespeichert. Nun überschreibst du diesen Zeiger mit dem Zeiger r. Dadurch verlierst du jeglichen Zeiger auf deinen gleich davor reservierten Speicher und hast ein Speicherleck, da du den nicht mehr freigeben kannst. Wahrscheinlich wolltest du hier das Array, welches sich dahinter versteckt, kopieren. Nur weiss der Kompiler nichts davon, er sieht nur einen Zeiger auf einen einzelnen Double-Wert.

Allgemein: Du solltest nochmals den Teil über Zeiger & Arrays durchlesen. Dringend!

Zu deiner Rabattstaffel an sich:
Wie soll die denn genau implementiert werden? Also wie entscheided das Objekt, wann welcher Rabatt genommen werden soll?

In der Intialisierungsliste kann man kein Array mit Defaultwerten erstellen und übergeben. Also müsste man dies sowieso ein wenig anders lösen, als wie du es vorgehabt hast.

Grüssli

Ich würde zum besseren Vertändniss den Variablennamen mit angeben. Das wird ungeahnte Vorteile für den späteren Endbenutzer/Leser haben wenn er mit deinem Code arbeiten möchte.

gypsygirl schrieb:

wie find ich denn die Größe von r raus?

Wie bereits unter (2) gesagt wurde: Gar nicht! Musst du mit übergeben.