Kunde *k = new Kunde();
k->setKunde("vorname", "nachname", 123456789);

cout << k << endl;

Kunde *n = new Kunde();
	n = k;
	cout << n << endl;

	cout << "K wird geloescht - n ausgegeben" << endl;
	//delete k;
	cout << n << endl;

	cin.get();

Bei cout << k schmeißt er mich raus mit unbehandelter Ausnahme... Zugriffsverletzung bei.....
Wo liegt der Fehler? Bei der COUT überladung oder... ich finde ihn nicht.
Hier der Rest... bitte um Hilfe.. hocke schon stunden daran

Kunde::Kunde()
{
	m_knr = 0;
	m_vname = NULL;
	m_nname = NULL;
}

Kunde::Kunde(const Kunde &k)
{
	setVname(k.m_vname);
	setNname(k.m_nname);
	setKnr(k.m_knr);
}

Kunde::~Kunde()
{
	m_knr = 0;
	delete [] m_vname;
	delete [] m_nname;
	m_vname = NULL;
	m_nname = NULL;
}

void Kunde::setVname(const char *vname)
{
	if( m_vname != NULL )
	{
		delete [] m_vname;
		m_vname = new char[strlen(vname)+1];
		strcpy(m_vname, vname);
	}
}

void Kunde::setNname(const char *nname)
{
	if( m_nname != NULL )
	{
		delete [] m_nname;
		m_nname = new char[strlen(nname)+1];
		strcpy(m_nname,nname);
	}
}

void Kunde::setKnr(const unsigned int knr)
{
	m_knr = knr;
}

char* Kunde::getVname() const
{
	return m_vname;
}

char* Kunde::getNname() const
{
	return m_nname;
}

int Kunde::getKnr() const
{
	return m_knr;
}

void Kunde::print()
{
	if( m_knr != 0 && m_vname != NULL && m_nname != NULL )
	{
		cout << "\n\nKnr: " << m_knr << "\nVname: " << m_vname << "\nNname: " << m_nname << endl;
	}
	else
	{
		cout << "Ungueltig" << endl;
	}
}

void Kunde::setKunde(const char *vname, const char *nname, const unsigned int knr)
{
	setVname(vname);
	setNname(nname);
	setKnr(knr);
}

Kunde& Kunde::operator=(const Kunde &k)
{
	if( this != &k )
	{
		setKnr(k.m_knr);
		setVname(k.m_vname);
		setNname(k.m_nname);
	}
	return *this;
}

ostream& operator<<(ostream &cout, const Kunde *k)
{
	if( k->getVname() != NULL && k->getNname() != NULL )
	{
		cout << "\n\nKnr: " << k->getKnr() << "\nVname: " << k->getVname() << "\nNname: " << k->getNname() << endl;
		return cout;
	}
}

Danke für jede Hilfe .. komme echt nicht mehr weiter

class Kunde
{

	private:
		unsigned int m_knr;
		char *m_nname;
		char *m_vname;

	public:
		Kunde();
		Kunde(const Kunde &k);
		~Kunde();

		void setVname(const char *vname);
		void setNname(const char *nname);
		void setKnr(const unsigned int knr);

		char* getVname() const;
		char* getNname() const;
		int getKnr() const;

		void print();

		void setKunde(const char *vname, const char *nname, const unsigned int knr);

		Kunde& operator=(const Kunde &k);
		friend ostream& operator<<(ostream &cout, const Kunde *k);

};

Und btw: std::string ist dein Freund!

Ich finde den Fehler nicht ich hocke schon stundenlang..

Und ja, wie gerne würde ich string hernehmen aber ich wir müssen hier als Aufgabe char arrays hernehmen.

Danke!

Also die Funktion bei der er abstürzt ist wenn ich cout << k mache..

Da du k als Zeiger übergibst und im Operator dereferenzierst: Mal geguckt, ob k auch einen gültigen Wert hat?

{m_knr=123456789 m_nname=0x00000000 <Schlechtes Ptr> m_vname=0x00000000 <Schlechtes Ptr> }

Dann liegts wohl an setVname und setNname... !?

bzw.

if( m_nname != NULL )

=> Es wird nicht gesetzt...

Aber gleich wird er sich beim 2.

cout << n << endl;

auf die Nase legen

if( m_vname != NULL )
    {
        delete [] m_vname;
        m_vname = new char[strlen(vname)+1];
        strcpy(m_vname, vname);
    }

Nur das delete sollte innerhalb des if-Blocks sein, das hab ich total überlesen^^
Und selbst das delete kannst du ohne if ausführen, da C++ garantiert dass ein delete nullpointer okay ist und nix macht.

Edit: XSpille, auch diesmal bezog sich mein Posting nicht auf deines xD

Programmiere mit Microsoft Visual Studio 2010 Pro.

Wisst ihr wo das ist.. habe unter Ansicht gerade schon jegliche Fenster eingeblendet.. aber irgendwie nicht dieses Debug Fenster.

Und klar gehört das aus dem if-Block heraus... Logikfehler.. ^^
Danke!

Hoffe das funktioniert jetzt

ABER

Jetzt gibt es das Problem, dass wenn ich unten k lösche und n ausgebe, das der selbe Fehler kommt. Habe nachgeschaut, n hat beim Char array die selben Adressen, also es wird nur eine Flache Kopie erstellt und nicht eine gewollte Tiefe Kopie mit verschiedenen Adressen.
Aber warum?
Ich nehme dazu meine SetterMethoden und in denen lege ich ja mit
= new char... ein neues Objekt mit eigener Adresse an.. oder?!

Also meine Setter, damit mein ich das setNname, setVname usw.
und der Operator= wird ja auch überladen.

Meine main ist auch onch die selbe.

Ich erstelle k und weise Werte zu.
Dann erstelle ich n und weise k zu.
Dann lösche ich k "delete k" und gebe n nochmal aus.

Oder was meinst du mit Codeteile verschweigen?!

Kunde *n = new Kunde();
    //n = k; // Kopiert nur die Addresse
    *n = *k; // Kopiert das Objekt.
    cout << n << endl;

    cout << "K wird geloescht - n ausgegeben" << endl;
    //delete k;
    cout << n << endl;

    cin.get();

Bin jetzt daran Stunden gehockt...

Man oh man ^^ verstehe aber iwie nicht warum das Sternchen davor sein muss.
Das Sternchen geht ja wie ich immer verstanden habe genau zum Wert. Und so nimmt er den Zeiger also einfach nur die Adresse sozusagen... oder?!

Danke !!!

Funktioniert jetzt perfekt :xmas1:

Habe es aber wieder eingeblendet mit Ansicht oben in Visual Studio.