ich habe ein kleines Programm geschrieben um den Real- und Imaginärteil von komplexen Zahlen zu addieren.
Wenn ich kompiliere bekomme ich 0 Fehler und 0 Warnungen aber beim Ausführen bekomme ich eine "Debug Assertion Faailed".

Hier mal mein kleiner Code!

Hauptprogramm:

#include "MyComplex.h"

int main()
{
	complex x(1.0,2.0);
	complex y(2.0,4.0);
	complex z(0.0,0.0);

	x.print();
	y.print();

	z.add(x,y);

	z.print();

	return 0;
}

hier die Klasse:

#include <iostream>
using namespace std;

class complex
{
	private:

		float *re, *im;

	public:

		complex(float r, float i);
		~complex();

		float	real	();
		float	imag	();
		void	add		(complex a, complex b);
		void	print	();
};

complex::complex(float r, float i)
{
	re =new float;
	*re = r;

	im = new float;
	*im = i;
}

complex::~complex()
{
	delete re;
	re = 0;

	delete im;
	im = 0;
}

float complex::real()
{
	return *re;
}

float complex::imag()
{
	return *im;
}

void complex::add(complex a, complex b)
{
	*re = a.real() + b.real();
	*im = a.imag() + b.imag();
}

void complex::print()
{
	cout << "Realteil: " << *re << "			Imaginaerteil: " << *im << endl << endl;
}

und hier die Fehlermeldung:

http://www8.picfront.org/picture/s4lFbXE5/thb/DBGASS.PNG

Vielen Dank für eure Hilfe und Mühe !

Regards Duckler

Vielen Dank,
Duckler

Wollte aber paar Anmerkungen zu deinem Code machen:

- Definitionen kommen nicht in die Headerdatei, außer es handelt sich um inline oder templates. In der Headerdatei solltest du nur die Methoden deklarieren und sie in einer .cpp Datei definieren.

- using - Direktiven sollte man in Headerdateien vermeiden, könnte unterwünschte Namenskonflikte verursachen.

- Wieso benutzt du Zeiger und new ?

Danke fur Deine schnelle Antwort Dweb!

Viele Grüße Duckler!

Dann ist eurer Dozent gelinde gesagt einfach nur schlecht, oder du hast etwas missverstanden. Grundsätzlich braucht deine Klasse so wie du sie mit Zeigern und new verwendest aber noch mindestens den Zuweisungsoperator und Kopierkonstruktor (oder diese explizit müssen verboten werden, in dem sie privat deklariert, aber nicht implementiert werden)!

Kann es sein dass ich die Objekte erstelle dann zerstöre aber in add() auf die beiden erstellten *und* zerstörten Objekte a und b zugreifen will und das ganze deshalb verrutscht?

Nein, die 3Objekte (x,y und z) werden erst beim verlassen des Güligkeitsbereichs zerstört (Destruktor wird aufgerufen), und das ist ...

#include "MyComplex.h"

int main()
{
    complex x(1.0,2.0);
    complex y(2.0,4.0);
    complex z(0.0,0.0);

    x.print();
    y.print();

    z.add(x,y);

    z.print();

    return 0;
} // ... erst hier der Fall.

ahh moment mal.^^

z.add(x,y);

Du erzeugst hier nur eine flache-Kopie, d.h. es werden nur die Zeiger kopiert, nicht aber die Objekte auf die die Zeiger zeigen.
Das heißt wenn die add-Funktion abgearbeitet ist, wird der Destruktor für 'a' und 'b' aufgerufen. --> Die Objekte, auf die die Zeiger zeigen, werden dann zerstört, weil diese, wie gesagt, nicht kopiert werden.

z.print();

Hier willst du wieder auf die Objekte zugreifen, sie wurden aber schon zerstört.

complex::complex(const complex& ori)
{
	this->re = new float( ori.real() );
	this->im = new float( ori.imag() );
}

und mache die real() und imag() Methoden konstant:

float    real    () const; // Deklaration

float complex::real() const // Definition
{
    return *re;
}

( und das gleiche noch bei imag() )

vielen, vielen, vielen, vielen, vielen, vielen, vielen, DANK Dweb es funktioniert tadellos

Viele Grüße,
Duckler

Viele Grüße,
Duckler

Was mich nur wundert, dass die Meldung bei VS 2010 nicht kommt, hmm^^

grüßli

- Die Dreierregel wurde verletzt.
- re und im brauchen aber keine Zeiger sein. Das ist -- mit Verlaub -- völliger Schwachsinn.

Viele Grüße,
Markus

Viele Grüße,
Duckler

P.S.: Dweb geht das irgendwie mit der PN ?

Viele Grüße,
Duckler

hmm "PN" ?^^

Viele Grüße,
Duckler

duckler schrieb:

P.S.: Dweb geht das irgendwie mit der PN ?

Viele Grüße,
Duckler

hmm "PN" ?^^

PN gibts nicht, aber Email an die registrierte Email Adresse.

Ansonsten kannst du mir auch icqNR oder so geben oder mit schicken.

da mußt Du nochmal nachgucken, da steht leider nicht ;-)!

Viele Grüße,
Duckler

Hi dweb,

da mußt Du nochmal nachgucken, da steht leider nicht ;-)!

Viele Grüße,
Duckler

nu event. ?

hab mir Pidgin gezogen dann können wir mal chatten, meld mich morgen nochmal!

Viele Grüße,
Duckler

Viele Grüße,
Duckler

Als Grnud steht hier: [ duckler1911@web.de Benutzer nicht bekannt/User unknown]

hab Dir ne Mail mit ner Erklärung geschickt!

Viele Grüße,
Duckler