Ich möchte einen set-Container in einer h-Datei mit meinem eigenen Datentypen "person" deklarieren. Laut compiler bekomme ich da aber einige Fehler, unter anderem, dass ; vor dem Operator < fehlt. Ich weiß ehrlich gesagt nicht was ich da so falsch gemacht haben soll:

#ifndef VERWALTUNG__H
#define VERWALTUNG__H

#include<set>
#include"person.h"

class verwaltung
{
private:
	char* faName;
	set<person> mitarbeiter;

public:
	verwaltung(void);
	~verwaltung(void);

	void addMitarbeiter(person p);
	/*void entlasseMitarbeiter();
	void gehaltMitarbeiter();
	void druckeMitarbeiter();
	void berechneKosten();*/
};

#endif

Naja jetzt mosert der Compiler jedenfalls, dass person nicht deklariert sei.

Hier ist die Person.h:

#ifndef PERSON__H
#define PERSON__H
#include "verwaltung.h"

class person
{
public:
	char* name;
	char* wohnort;
	unsigned int gehalt;

	person(void);
	~person(void);

	void setName(char* inputName);
	void setWohnort(char* inputWohnort);
	void setGehalt(int inputGehalt);
};

#endif

Fehler 6 error C2676: Binärer Operator '<': 'const person' definiert diesen Operator oder eine Konvertierung in einen für den vordefinierten Operator geeigneten Typ nicht c:\program files (x86)\microsoft visual studio 10.0\vc\include\xfunctional 125

Aber warum genau willst du denn eigentlich ein std::set benutzen? Wieso reicht ein vector nicht aus?

void verwaltung::addMitarbeiter(person pers)
{
	mitarbeiter.insert(pers);
}

Wenn ich nun in der main mehrere personen anlege und diese bspw. 3 personen in der verwaltung "zusammenfassen" will, dann funktioniert das soweit, bis eine zweite person zur set hinzugefügt wird. wenn das passieren soll, dann kackt mir das programm mit den Worten "invalid < operator" ab. Ich denke, die Überladung hat dann nicht so funktioniert. Die string.h hab ich natürlich inkludiert. Hier die Überladung:

int operator < (const person p1, const person p2)	//Überladung, damit die set sortieren kann
{
	return strcmp(p1.name, p2.name);
}

Das kommt eben vom Mischen von C und C++. Wurde dir schon oft genug gesagt.

int operator < (const person p1, const person p2)	//Überladung, damit die set sortieren kann
{
	//Wert größer als 0 zeigt an, dass der Wert des ersten Zeichens im String p1.name welches nicht passt,
	//größer ist als der Wert des Zeichens im String p2.name.
	//Wert gleich 0 zeigt an, dass beide Strings gleich lang sind.
	if(strcmp(p1.name, p2.name) < 0)
	{
		return true;	//Wenn Ordnung erfüllt ist, gib true zurück
	}
	else
	{
		return false;	//Ansonsten false
	}
}

Wie hätte ich denn dann den Operator überladen sollen wenn ich mit der C-Bibliothek string.h in C++ nicht arbeiten soll?

bool foo(bool b)
{
  if (b == true)
    return true;
  else if (b == false)
    return false; 
}

Wie hätte ich denn dann den Operator überladen sollen wenn ich mit der C-Bibliothek string.h in C++ nicht arbeiten soll?

Du sollst eben gleich std::string benutzen. Dann sparst du dir auch den komplizierten Konstruktorrumpf deiner Klassen, den nicht-default Destruktor sparst du ganz ein und es ist außerdem kein Fehler mehr, dass du, wie bisher, den Kopierkonstruktor und den Zuweisungsoperator weggelassen hast.

Du sollst eben gleich std::string benutzen.

Hm, darauf bin ich jetzt gar nicht gekommen. C++ bietet ja in der Tat eine Datentypen string an. Da brauch ich ja dann gar nicht mit char-Pointern arbeiten, oder? Das würde aber dann bedeuten, dass ich meine zwei Klassen in denen ich char-Pointer benutzt habe, auf string umstellen muss, oder?

Das würde aber dann bedeuten, dass ich meine zwei Klassen in denen ich char-Pointer benutzt habe, auf string umstellen muss, oder?

Ja, wenn man von char* auf string umstellen möchte, dann muss man dazu von char* auf string umstellen.

Den Vergleichsoperator für Person müsstest du natürlich trotzdem schreiben (oder einen kleinen Lambda-Ausdruck benutzen). Dies geht aber sehr einfach, da string bereits alle Vergleichsoperatoren (mit den in C++ üblichen Konventionen) kennt.

person myPerson1, myPerson2, myPerson3;
	verwaltung myVerwaltung;

	myPerson1.setName("aaa");
	myPerson1.setWohnort("bbb");
	myPerson1.setGehalt(3500);
	myVerwaltung.addMitarbeiter(myPerson1);

Ich denke ihr versteht was ich meine, oder?

Nein. Selbstverändernde Variablennamen? Macht keinen Sinn. Was möchtest du erreichen? Deine Äußerungen deuten darauf hin, dass du irgendwelche grundlegenden Dinge völlig falsch verstanden hast.