class a
{
public:
        funktiona(){pB->funktionb();};
private:
        b *pB;
}

class b
{
public:
       funktionb();
private:

}
b::funktionb()
{
vector<int>vectorb;
cout<<"Gibt zahl ein"<<endl;
cin>>zahl;
vectorb.pushback(zahl);
....
}

wenn ich die Funktion b mit dem Hilfe des Pointers pB aufrufe läuft die Funktion b problemlos durch doch am ende der Funktion kommt die Fehlermeldung (error: Speicherzugriffsfehler)

ich weiß das es einfacher gehn würde wenn ich in der Klasse A einfach ein objekt der klasse b erzeuge und dann die funktion aufrufe aber ich würde gern wissen wie ich das mit dem pointer lösen kann, denn irgendwie muss das ja auch funktionieren
danke im voraus

class Lotto 
{	
public:
        ...
	void ziehen();
	 Lotto();
	~Lotto();
private:
	Ziehung *pzieh;

};
void Lotto::ziehen()
{   		
	pzieh->ziehen();

}
class Ziehung
{	
public:
	void ziehen();
	void ergebnisAusgeben();
	std::vector<int>gezogene_zahlen;
private:

};
void Ziehung::ziehen()
{
//bestimmt ihre 6 Lottozahlen
int i =0;
int zahlen[49];
int zufallszahl[6];
int r;

while (i < 6)
{
	cout << "bestimme zahl" << i+1 << endl;
		do
		{
			srand(time(NULL)* time(NULL)); 
			r = rand()%48+1;

		}while (zahlen[r] == 1);
			zahlen[r] = 1;
			zufallszahl[i] = r+1;
			i++;

}
//die gezogenen Zahlen sotieren und ausgeben
for (int x=0; x<6; x++)
{
	for (int y=0; y<5; y++)
	{
		if (zufallszahl[y] > zufallszahl[y+1])
		{
			int temp = zufallszahl[y];
			zufallszahl[y] = zufallszahl[y+1];
			zufallszahl[y+1] = temp;
		}

	}
}
	for (int i=0; i<=5; i++)
	{
		cout << "Lottozahl " << i+1 <<": "<< zufallszahl[i]<< endl;

	} 

		  gezogene_zahlen.push_back(zufallszahl[0]);
		  gezogene_zahlen.push_back(zufallszahl[1]);
		  gezogene_zahlen.push_back(zufallszahl[2]);
		  gezogene_zahlen.push_back(zufallszahl[3]);
		  gezogene_zahlen.push_back(zufallszahl[4]);
		  gezogene_zahlen.push_back(zufallszahl[5]);

}

class Lotto : public Ziehung
{
...
public:
void Lottoziehen( )
{
ziehen();
}
};

Die Sache mit dem Pointer klappt auch du must jedoch dann im Konstruktor von Lotto ein Ziehung-Objekt erstellen, damit dein Pointer auch gültig ist.

MfG

verzichte hier ganz auf Zeiger und verwende einfach

Ziehung ziehung;

Und dann eben per '.' statt '->' darauf zugreifen...

P.S: Vererbung halte ich hier für den falschen Weg - m.E. reicht hier eigentlich auch eine Klasse aus (außer man würde noch andere Klassen haben, welche auch auf Ziehung zugreifen).

void Lotto::ziehen()
{   		
	Ziehung *pzieh= new Ziehung;
	pzieh->ziehe();
	delete pzieh;	
}

aber wenn ich jetzt dann das Ergebnis ausgeben möchte hab ich wieder einen Fehler
// report an out_of_range error

void Lotto::ergebnisAusgeben(void)
{
	Ziehung *pzieh = new Ziehung;
	Tippzettel *tipp = new Tippzettel;
	Tippzettel *tipp1 = new Tippzettel;
	Tippzettel *tipp2 = new Tippzettel;
	unsigned int i=0;
if(getTippzettelanzahl()==1)
{
	sort(pzieh->gezogene_zahlen.begin(),pzieh->gezogene_zahlen.end());
	sort(tipp->getippte_zahlen.begin(),tipp->getippte_zahlen.end());
	vector<int> intersec;
	set_intersection(pzieh->gezogene_zahlen.begin(), pzieh->gezogene_zahlen.end(), tipp->getippte_zahlen.begin(), tipp->getippte_zahlen.end(), back_inserter(intersec));
	cout<<tipp->adresse.at(0)<<endl;
	cout<<tipp->adresse.at(1)<<endl;
	cout<<tipp->adresse.at(2)<<endl;
	cout<<"Die getippte Zahlen waren: "<<tipp->getippte_zahlen.at(0)<<" "<<tipp->getippte_zahlen.at(1)<<" "<<tipp->getippte_zahlen.at(2)<<" "<<tipp->getippte_zahlen.at(3)<<" "<<tipp->getippte_zahlen.at(4)<<" "<<tipp->getippte_zahlen.at(5)<<" "<<endl;
	cout<<"Es stimmen folgende Zahlen ueberein: ";

		 if( i<intersec.size())
		 {
			cout << " " << intersec.at(i);
         }else
			 cout<<"Es stimmen keine Zahlen ueberein"<<endl;
}
delete pzieh;
delete tipp;
delete tipp1;
delete tipp2;
}

Kommst du von Java? Das Schlüsselwort new kannst und solltest du in C++ bis auf wenige Ausnahmen komplett vergessen. Dann ist der Code auch gleich viel, viel, viel weniger fehleranfällig.

was hat den dieser fehler zu bedeuten?

danke aber ich würde es gerne mit Pointer lösen

Und warum?
Ich würde auch gerne mit meinem Traktor zur Arbeit fahren, nur dauert das immer so lange...

Ziehung z;
Ziehung* pz = &z;
...

Wenn du unbedingt auf den Pointer bestehst mach's so.

MfG

naja interesse halber
wenn es die Möglichkeit gibt das mit Pointer zu lösen würde ich diese gerne kennen

Irgendeine Möglichkeit mit rohen Pointern gibt es schon, man muss es bloß richtig machen. Die saubere Methode, wenn man doch mal wirklich Pointer braucht, ist diese so weit wie möglich zu verstecken. In einer Klasse die nach RAII-Design aufgebaut ist. Das sollte man vielleicht einmal im Leben selber gemacht haben (also google mal, was das bedeutet), ansonsten benutzt man je nach Verwendungszweck die Container/Smartpointer aus der Standardbibliothek, die genau das machen.

Ziehung z;
Ziehung* pz = &z;
...

Wenn du unbedingt auf den Pointer bestehst mach's so.

MfG

okj hab das jetzt so geändert aber wenn ich die funktion

void Lotto::ergebnisAusgeben(void)
{
	/*Tippzettel *tipp = new Tippzettel;
	Tippzettel *tipp1 = new Tippzettel;
	Tippzettel *tipp2 = new Tippzettel;
	Ziehung *pzieh= new Ziehung;
	Ziehung z; 
	Ziehung* pzieh = &z;
	Tippzettel t;
	Tippzettel *tipp = &t;
	Tippzettel *tipp1 = &t;
	Tippzettel *tipp2 = &t;*/

	unsigned int i=0;
if(getTippzettelanzahl()==1)
{
	sort(pzieh->gezogene_zahlen.begin(),pzieh->gezogene_zahlen.end());
	sort(tipp->getippte_zahlen.begin(),tipp->getippte_zahlen.end());
	vector<int> intersec;
	set_intersection(pzieh->gezogene_zahlen.begin(), pzieh->gezogene_zahlen.end(), tipp->getippte_zahlen.begin(), tipp->getippte_zahlen.end(), back_inserter(intersec));
	cout<<tipp->adresse.at(0)<<endl;
	cout<<tipp->adresse.at(1)<<endl;
	cout<<tipp->adresse.at(2)<<endl;
	cout<<"Die getippte Zahlen waren: "<<tipp->getippte_zahlen.at(0)<<" "<<tipp->getippte_zahlen.at(1)<<" "<<tipp->getippte_zahlen.at(2)<<" "<<tipp->getippte_zahlen.at(3)<<" "<<tipp->getippte_zahlen.at(4)<<" "<<tipp->getippte_zahlen.at(5)<<" "<<endl;
	cout<<"Es stimmen folgende Zahlen ueberein: ";

		 if( i<intersec.size())
		 {
			cout << " " << intersec.at(i);
         }else
			 cout<<"Es stimmen keine Zahlen ueberein"<<endl;
}

bekomme ich folgende fehlermeldung

Unbehandelte Ausnahme bei 0x012b6dd4 in OOAD.exe: 0xC0000005: Zugriffsverletzung beim Lesen an Position 0xccccccd4.

und werde darauf hingewiesen

iterator end()
{	// return iterator for end of mutable sequence
		return (iterator(this->_Mylast, this));
		}

Möglichkeit mit rohen Pointern gibt es schon, man muss es bloß richtig machen. Die saubere Methode, wenn man doch mal wirklich Pointer braucht, ist diese so weit wie möglich zu verstecken. In einer Klasse die nach RAII-Design aufgebaut ist. Das sollte man vielleicht einmal im Leben selber gemacht haben (also google mal, was das bedeutet), ansonsten benutzt man je nach Verwendungszweck die Container/Smartpointer aus der Standardbibliothek, die genau das machen.

die Smartpointer schau ich mir jetzt mal an