Grobe Fehler sind, soweit ich es prüfen konnte, keine mehr drin.

Aber:

Was ist daran sehr unschön bzw. kompliziert gelöst?

Was mich am meisten stört ist die Benutzerverifikation. Sollte mein Programm für mehr als 3 User arbeiten, hätte ich für jeden ein weiteres else if...

#include <iostream> 
#include <string>

using namespace std;

struct konto
{
	string User;
	string Pin;
	int Kontostand;
};

void show(int*);
void give(int*);
void choose(int*);

int main() 
{
	konto a, b, c;
	a.User = "Tim";
	a.Pin = "1";
	a.Kontostand = 1000;
	b.User = "Karl";
	b.Pin = "2";
	b.Kontostand = 2000;
	c.User = "Hugo";
	c.Pin = "3";
	c.Kontostand = 3000;
	Anmeldung:
	string User;
	cout<<"Benutzername: ";
	cin>>User;
	string Pin;
	cout<<"PIN: ";
	cin>>Pin;
	if(User == a.User && Pin == a.Pin)
	{
		choose(&a.Kontostand);
	}
	else if(User == b.User && Pin == b.Pin)
	{
		choose(&b.Kontostand);
	}
	else if(User == c.User && Pin == c.Pin)
	{
		choose(&c.Kontostand);
	}
	else
	{
		cout<<"\nFehler bei der Anmeldung!!!\n\n";
	}
	goto Anmeldung;
	cin.clear(); cin.ignore(cin.rdbuf()->in_avail()); cin.get();
}

void show(int* money)
{
	cout<<"\nIhr aktueller Kontostand ist: "<<*money<<" Euro \n\n";
}
void give(int* money)
{
	int out;
	outagain:
	cout<<"\nBitte geben Sie den gew\201nschten Auszahlungsbetrag ein: ";
	cin>>out;
	if(*money-out < 0)
	{
		cout<<"\nDer gew\204hlte Betrag ist nicht verf\201gbar!!! \n";
		goto outagain;
	}
	else
	{
	*money-=out;
	cout<<"\nIhnen wurden "<<out<<" Euro ausgezahlt.";
	show(money);
	}
}
void choose(int* money)
{
	cout<<"\nBitte treffen Sie eine Auswahl: \n1 = Kontostand \n2 = Auszahlung \n3 = Neuanmeldung\n\nAuswahl:";
	int Auswahl;
	cin>>Auswahl;
	if(Auswahl == 1)
	{
		show(money);
	}
	else if(Auswahl == 2)
	{
		give(money);
	}
	else if(Auswahl == 3)
	{
		cout<<"\n";
	}
	else
	{
		cout<<"\nAuswahl nicht m\224glich!!!\n\n";
	}
}

Dank und Gruß, SchlippSchlapp

P.S. Sollte man den Quelltext nur als Link verknüpfen?
Gibt es eine Forumsordnung die ich nicht finde?

zu a): goto ist echt unbeliebt (anscheinend aufgrund seiner Fehleranfälligkeit) ^^

Den Rest habe ich verstanden, da ich aber erst das Onine-Tutorial "Programmieren in C++: Einführung" durch habe, ist mir vektor noch unbekannt (Ich sehe es mir natürlich gerne an).

Schleifen: do - while oder nur while solltest du auch noch mal üben, dann sollt das goto entbehrlich sein.

MfG f.-th.

void choose(int* money)
{
	cout<<"\nBitte treffen Sie eine Auswahl: \n1 = Kontostand \n2 = Auszahlung \n3 = Neuanmeldung\n\nAuswahl:";
	int Auswahl;
	cin>>Auswahl;
	switch(money)
	{
	case 1:
		show(money);
		break;
	case 2:
		give(money);
		break;
	case 3:
		cout<<"\n";
		break;
	default:
		cout<<"\nAuswahl nicht m\224glich!!!\n\n";
		break;
}

Zeile 29 habe ich jetzt einfach durch

while(1)
	{

ersetzt und in Zeile 52 die Schleife geschlossen.

if (Auswahl==1)show(money);
    else if (Auswahl==2)give(money);
    else if (Auswahl==3)cout<<"\n";
    else cout<<"\nAuswahl nicht m\224glich!!!\n\n";

Hier ein Link:
http://www.c-plusplus.net/forum/viewtopic-var-p-is-916613.html

MfG f.-th.

Schau wegen switch in dein Tutorial Kap. 4.1

...einfach nur ein Fehler...

@ f.-th.

Danke für die genauen Tipps (du machst dir sogar die Mühe und stimmst sie auf mein Tutorial ab). Switch habe ich eigentlich schon verstanden ^^.
Es ging bei deinem Hinweis doch hoffentlich nur um den von Athar beschriebenen Fehler oder?

switch(*money)
{
...
}

Zumal ich nicht wirklich verstehe warum du also Funktionsargument Pointer nimmst. Willst du dir dadurch das Kopieren der Argumente sparen? Dann hätte es eine const-Referenz auch getan, also

void choose(const int& money)
{
...
}

Für solche Beispiele sind Referenzen denke ich besser als Pointer.

Das ist nur das was mir noch aufgefallen ist.

Bei dem switch hätte ich einfach nur Auswahl, also den richtigen Wert, einfügen sollen, dann hätte alles geklappt.

Da die referenzierte Variable geändert werden soll funktioniert doch const nicht oder?
Eine einfache Referenz könnte ich aber anscheinend verwenden.

Zu deiner Frage warum ich das tue: Ich finde Zeiger sehr sympatisch gegenüber Referenzen.

Wenn ich einer Funktion eine Referenz gebe und diese dann weitergegeben wird und das dann noch 20x passiert, dann stelle ich mir da eine endlose Schlange an Verknüpfungen auf Verknüpfungen vor (auch wenn das eventuell nicht so ist).
Wenn ich hingegen eine Adresse weitergebe, dann bleibt das immer die selbe.

Und ja, es geht in der Programmierung vermutlich nicht um persönliche Präferenzen, sondern eher um sinnvoll und nützlich.

Dank und Gruß, SchlippSchlapp

Hat beides wie gesagt Vor- und Nachteile.
Mit Referenzen kann man meiner Meinung nach halt weniger falsch machen ;).