ich bin Student der Informatik an einer Uni und bin gerade dabei C++ zu lernen. Bei den "Klassen" habe ich jetzte erste Probleme mit dem Verständnis. Leider gibt es zu unseren Übungen von der Uni bisher keine Lösungsansätze.

Könnt ihr somit meine Lösung kommentieren und mir ggf. Tipps geben? Wäre super

Gegebener Code:

class C_Mitglied :{
private:
string ID;
bool Beitragbezahlt;
public:
string Vorname;
string Nachname;
};

Aufgabenstellung:
Ergänze die Klasse um die folgenden Anforderungen.
- Standardkonstruktor soll vorhanden sein, der die ID auf "0" setzt und Beitragbezahlt auf false setzt.
- Es soll ein überladener Konstruktor vorhaden sein, um die

- ID, Vornamen, Nachnamen und Beitragbezahlt als Parameter aufzunehmen.
- öffentliche Methode um das Attribut Beitragbezahlt zu lesen und zu schreiben
- öffentlich Methode um das Attribut ID zu lesen.

Deklariere eine neue Klasse C_Ehrenmitglied, die von C_Mitglied alle Attribute und Methoden als public erbt. Die neue Klasse soll als zusätzliches public-Attribut bool beitragsfrei besitzen, welches durch seinen überladenen Konstruktor auf true gesetzt wird. Dieser überladene Konstruktor soll ID, Vornamen, Nachnamen als Parameter aufnehmen und beitragbezahlt auf false setzen.

Mein Lösungsvorschlag:

#include <iostream>
#include <string>
using namespace std;

class C_Mitglied :{
	private:
		string ID;
		bool Beitragbezahlt;

	public:
		C_Mitglied();
		C_Mitglied(string pID, string pVorname, string pNachname, bool pBeitragbezahlt);
		string Vorname;
		string Nachname;
		bool get_Beitragbezahlt(){return Beitragbezahlt;};
		void set_Beitragbezahlt(bool pBeitragbezahlt){Beitragbezahlt=pBeitragbezahlt;};
		string get_ID(){return ID;};
};

C_Mitglied::C_Mitglied()
{
	ID="0";
	Beitragbezahlt=false;
}

C_Mitglied::C_Mitglied(string pID, string pVorname, string pNachname, bool pBeitragbezahlt)
{
	ID=pID;
	Vorname=pVorname;
	Nachname=pNachname;
	Beitragbezahlt=pBeitragbezahlt;
}

class C_Ehrenmitglied : public C_Mitglied
{
	public:
		bool beitragsfrei;
		C_Ehrenmitglied(string pID, string pVorname, string pNachname);
};

C_Ehrenmitglied::C_Ehrenmitglied(string pID, string pVorname, string pNachname) : C_Mitglied(pID, pVorname, pNachname, false)
{
	beitragsfrei=true;
}

Gruß.

- Steht da wirklich ein Doppeltpunkt nach class C_Mitglied ? Der darf da nicht hin.
- Alle Memberfunktionen die nur lesen und keine Membervariablen verändern sollten als const markiert sein.
- Große Objekte wie std::string übergibt man am besten als konstante Referenz. Als Rückgabewert bei get_ID() könnte man auch eine konstante Referenz nehmen.
- Persönlicher Geschmack: Bei Konstruktoren die nur ein paar Member initialisieren ziehe ich eine Initializer List dem einfachen Zuweisen, so wie du es jetzt machst, vor.

- Persönlicher Geschmack: Bei Konstruktoren die nur ein paar Member initialisieren ziehe ich eine Initializer List dem einfachen Zuweisen, so wie du es jetzt machst, vor.

Das geht über Geschmacksfragen hinaus. In vielen Fällen ist es schließlich ohne Initializer List gar nicht möglich, ein Objekt zu initialisieren, wenn es keinen Standardkonstruktor hat. Und die direkte Initialisierung ist immer mindestens gleich gut wie die Kombination aus Standardinitialisierung und Zuweisung, fast immer sogar besser. Daher würde ich empfehlen, sich diese Technik sofort anzugewöhnen.