für eine Einsendeaufgabe soll ich ein einfaches Programm für eine Hotelzimmervewaltung schreiben. Das Programm soll objektorientiert sein, dafür habe ich eine Oberklasse "Zimmer" und zwei Unterklassen "EinzelZimmer" & "DoppelZimmer". Es gibt 10 Einzel- und 10 Doppelzimmer, also erstelle ich für beide Typen je ein Feld - und zwar so:

/* Felder der Einzel- & Doppelzimmer erzeugen
	--------------------------------------------- */
		/* Einzelzimmer */
		EinzelZimmer EinzelZimmerArray[10];
		for (intZaehler = 0; intZaehler < 10; intZaehler++)
		{
			EinzelZimmerArray[intZaehler].ZimmernummerErzeugen(intZaehler);
		}

		/* Doppelzimmer */
		DoppelZimmer DoppelZimmerArray[10];
		for (intZaehler = 0; intZaehler < 10; intZaehler++)
		{
			DoppelZimmerArray[intZaehler].ZimmernummerErzeugen(intZaehler);
		}

Eine Funktion des Programms soll sein dass der Benutzer die Zimmer mit allen Daten anzeigen lassen kann - die freien, die belegten, oder alle.
Nun habe ich über ein Menü die Auswahl bereit gestellt und wenn der User sich entscheidet die Zimmer abzufragen, schicke ich ihn in ein weiteres Menü - und zwar so:

switch (charMenuepunkt[0])
			{
				case '1': MenueZimmerstati(EinzelZimmerArray[10], DoppelZimmerArray[10]);
							system("Pause");
							break;

Der Prototyp der Funktion sieht so aus:

void MenueZimmerstati(EinzelZimmer, DoppelZimmer);

Ist er im neuen Menü, kann er sich entscheiden ob eben die freien, die belegten oder alle angezeigt werden sollen.
Momentan habe ich testhalber nur die ersten beiden geschrieben - und das sieht so aus:

- Untermenü:

void MenueZimmerstati(EinzelZimmer EinzelZimmerArray[10], DoppelZimmer DoppelZimmerArray[10])
{
...
switch (charMenuepunkt[0])
			{
				case '1': strSuchKriterium = "frei";
							StatiFreieBelegteZimmer(strSuchKriterium, EinzelZimmerArray[10], DoppelZimmerArray[10]);
							system("Pause");
							break;

- anzeigen der gefragten Zimmer:

void StatiFreieBelegteZimmer(string strSuchKriterium, EinzelZimmer EinzelZimmerArray[10], DoppelZimmer DoppelZimmerArray[10])
{
	/* Hilfsvariablen */
	int intZaehler; /* zum Durchlaufen der Zimmer-Felder */

	/* Durchlauf des EinzelZimmer-Feldes und Ausgabe der gefragten Daten */
	for (intZaehler = 0; intZaehler < 10; intZaehler++)
	{
		/* Stati des aktuellen Elementes prüfen... */
		if (EinzelZimmerArray[intZaehler].RueckgabeStatus() == strSuchKriterium)
		{
			/* ...und Ausgabe veranlassen, falss das aktuelle Zimmer den gefragten Wert enthällt */
			EinzelZimmerArray[intZaehler].EinzelZimmerDatenAusgeben();

			/* Übersichtszeile */
			cout << "-----------------------------------------------\n";
		}
	}

	/* Durchlauf des DoppelZimmer-Feldes und Ausgabe der gefragten Daten */
	for (intZaehler = 0; intZaehler < 10; intZaehler++)
	{
		/* Stati des aktuellen Elementes prüfen... */
		if (DoppelZimmerArray[intZaehler].RueckgabeStatus() == strSuchKriterium)
		{
			/* ...und Ausgabe veranlassen, falss das aktuelle Zimmer den gefragten Wert enthällt */
			DoppelZimmerArray[intZaehler].DoppelZimmerDatenAusgeben();

			/* Übersichtszeile */
			cout << "-----------------------------------------------\n";
		}
	}
}

Der Prototyp der Funktion "StatiFreieBelegteZimmer" ist dieser:

void StatiFreieBelegteZimmer(string, EinzelZimmer, DoppelZimmer);

Folgendes Problem tritt nun auf:
- wenn ich die Funktionen OHNE die Übergabe der Arrays laufen lasse (dann natürlich nur zum Test und ohne die Funktionalitäten) geht es
- rufe ich die Funktionen MIT der Übergabe aufrufe, also wie ihr sie hier seht, bekomme ich diesen Fehler:

1>------ Erstellen gestartet: Projekt: Aufbabe_2, Konfiguration: Debug Win32 ------

1>EA2_main.obj : error LNK2019: Verweis auf nicht aufgelöstes externes Symbol
""void __cdecl MenueZimmerstati(class EinzelZimmer,class DoppelZimmer)" (?MenueZimmerstati@@YAXVEinzelZimmer@@VDoppelZimmer@@@Z)" in Funktion "_main".

1>MenueZimmerstati.obj : error LNK2019: Verweis auf nicht aufgelöstes externes Symbol
""void __cdecl StatiFreieBelegteZimmer(class std::basic_string<char,struct std::char_traits<char>,class std::allocator<char> >,class EinzelZimmer,class DoppelZimmer)"
(?StatiFreieBelegteZimmer@@YAXV?$basic\_string@DU?$char_traits@D@std@@V?$allocator@D@2@@std@@VEinzelZimmer@@VDoppelZimmer@@@Z)" in Funktion
""void __cdecl MenueZimmerstati(class EinzelZimmer * const,class DoppelZimmer * const)" (?MenueZimmerstati@@YAXQAVEinzelZimmer@@QAVDoppelZimmer@@@Z)".

1>C:\Users\sinnlos\Documents\My Dropbox\Studium\C++\Programme_Kurs\Programme_VisualStudio\_Einsendeaufgaben\Aufbabe_2\Debug\Aufbabe_2.exe :
fatal error LNK1120: 2 nicht aufgelöste externe Verweise.

========== Erstellen: 0 erfolgreich, Fehler bei 1, 0 aktuell, 0 übersprungen ==========

Wie ihr vielleicht merkt bin ich noch neu bei C++. Im Internet habe ich schon einige ähnliche Bsp gelesen, mit Zeigern, Vektoren usw., das hatten wir im Kurs aber noch nicht und leider tu ich mich schwer die ähnlichen Bsp. auf meins umzulegen.

Könnt ihr mir vielleicht sagen was ich beim Übergeben der Felder falsch mache..??

Danke schon mal im Voraus..!!

bei der Übergabe eines Arrays darfst du nicht die Größe mitangeben (da es sonst ein Zugriff auf das x. Element bedeutet), also

MenueZimmerstati(EinzelZimmerArray, DoppelZimmerArray);
// sowie
StatiFreieBelegteZimmer(strSuchKriterium, EinzelZimmerArray, DoppelZimmerArray);

Im Sinne von OOP wäre es aber besser du hättest nur eine Liste von Zimmern (z.B. std::vector<Zimmer*> ) und die Funktionen würden nur mit den virtuellen Funktionen von Zimmer arbeiten.
Haben denn die Klassen Einzel- und Doppelzimmer unterschiedliche Member, weil sonst würde ja auch einfach ein Member "AnzahlBetten" in der Basisklasse "Zimmer" reichen?

EDIT: Zu langsam

@ Th69
Einfach die Größenangabe weglassen geht nicht, da bekomme ich schon bei schreiben einen Hinweis dass das ungültig ist (Visual C++) und kompilieren ist dann natürlich auch nicht drin...
Hast du vielleicht einen Tip wo das mit der Liste und den Vektoren gut erklärt ist..? ..vielleicht mit Codebsp...?
Fällt das auch unter die Kategorie Zeiger..?

@ Jonas OSDever
Hab ich was falsches geschrieben, oder hast du da was falsch interpretiert..? EinzelZimmer ist ja die Klasse, also der Datentyp vom Feld EinzelZimmerArray, da kommen doch auf keinen Fall die [] hin...??

@ SeppJ
Du müsstest mal die Schulungsunterlagen sehen..da waren teilweise auch die Codebeispiele falsch und haben nicht funktioniert..und das Buch das uns empfohlen wurde enthält NICHTS (!!) über Klassen... ..also bitte entschuldige mir/uns dass wir Schwierigkeiten haben...

@ SeppJ
Du müsstest mal die Schulungsunterlagen sehen..da waren teilweise auch die Codebeispiele falsch und haben nicht funktioniert..und das Buch das uns empfohlen wurde enthält NICHTS (!!) über Klassen... ..also bitte entschuldige mir/uns dass wir Schwierigkeiten haben...

Welches Buch wurde euch denn empfohlen?

Aus deinem letzten Beitrag schließe ich, dass sich das mit meiner 'falschen Interpretation' wohl erledigt hat, oder?
Ich bezog mich auf den Prototyp

void MenueZimmerstati(EinzelZimmer, DoppelZimmer);

der mit

void MenueZimmerstati(EinzelZimmer EinzelZimmerArray[10], DoppelZimmer DoppelZimmerArray[10])

nicht zusammenpasst.

Das ganze beschäftigt mich jetzt schon ein paar Tage und ich stelle fest dass ich einfach noch keine Ahnung vom C++ programmieren habe, wenn ich hier im Netz so rumsurve... ..und ich wusste einfach nicht dass man die [] auch an den Datentyp schreiben kann/muss.
Kannst du mir das vielleicht erklären..? Was passiert da..?

Das Buch heißt C-Programmierung lernen, von André Willms - und zu meiner Schande muss ich gestehen da steht schon was, nur über Strukturen und wie ich mittlerweile rausgefunden habe gibt´s Klassen erst seit C++...

Das nächste Mal schmeiß ich das Skript weg und mach´s gleich aus´m Buch, dann kann ich´s auch leister lesen wenn fortlaufende Bsp. drin sind...

Also nochmals Danke..!!

Kompiler Dev C++.

oO Na dann viel Spaß.
Wie viel Zeit bleibt denn noch für diese Aufgabe? Ich meine man kann schon gut erst C und dann C++ lernen, aber auch für die Grundlagen sollte man etwas Zeit einplanen.

Abgeben muss ich die Aufgabe am Mo., das ist noch zu schaffen..das einzige Problem war jetzt wirklich die Übergabe der Zimmer, bei mir jetzt eben in Arrays, an andere Funktionen.

Klassen/Unterklassen definieren, die Vererbung, Funktionen auslagern, Menüs erstellen usw habe ich kapiert, ziemlich sicher , das war auch relativ leicht zu recherchieren. Auch "normale" Variablen mit Zeigern an Funktionen übergeben hab ich schon mal mit Erfolg versucht..also den Rest bekomme ich wohl hin..und ich hab halt auch mega Spaß dabei... :p

Foo(Bar);

und

Foo(Bar[]);

nicht das gleiche sind. Der Compiler kann so unterscheiden, ob du nur eine Variable vom Typ Bar oder ein Feld vom Typ Bar haben willst. Wär ja auch schlimm wenn nicht. Was sollte z.B.

int Quadrat(int Zahl) { return Zahl * Zahl; }

machen, wenn ihm ein Array übergeben wird (sofern der Compi das durchgehen lassen würde)? Du siehst: Es hat seinene Gründe auch bei Parametern zwischen Array-Parametern und normalen Parametern zu unterscheiden.

EDIT: Die Frage von cooky541 stell ich mir aber auch: Warum C Buch, wenn C++ gelehrt wird?

Die Empfehlung hab ich übergangen und nutze Visual C++, dass ist zum Lernen und Fehlersuchen deutlich effektiver...

Eisenspitze schrieb:

Abgeben muss ich die Aufgabe am Mo., das ist noch zu schaffen..

Die Frage ist halt wie viel du verstanden hast..

Eisenspitze schrieb:

und ich hab halt auch mega Spaß dabei

Das ist immer ein gutes Zeichen dafür, dass du das auch packst.

Wenn du Pointer verstanden hast vielleicht noch ein Tipp zu Arrays:

void foo(int i[10]); // daraus macht der Compiler:
void foo(int *i)
{
  i[5] = 3; // Daraus macht der Compiler:
  *(i + 5) = 3;
}

Nur zum Verständnis. Das bedeutet nämlich dass Arrays so nicht kopiert werden, und das bedeutet, dass du die Dinge, die du in der Funktion am Array änderst, auch außerhalb änderst. (Und dass sich sizeof nicht so verhält wie man es vielleicht erwartet.)

Weiter ist die Verwendung von system("pause") im Code etwas fragwürdig, da das nicht überall funktioniert. Vielleicht postest du mal etwas mehr von der Aufgabe wenn du weiter fortgeschritten bist, aber bitte ohne jede Zeile einzeln zu kommentieren, das bringt einen total aus dem Lesefluss.

EDIT: Die Frage von Pi stell ich mir aber auch: Warum C Buch, wenn C++ gelehrt wird?

Warum halten mich alle für ne Zahl? -.-

@cooky451
Auch dir danke für deinen Hinweis, das denke ich ist mir klar...

Wegen dem Code posten, kann ich auch Dateien hochladen..? Es ist schon ziemlich viel Code... ..alles peinlich genau kommentiert, da gibt´s Punkte drauf... ..aber dass lösch ich dann halt raus...

Wie macht ihr das mit den Kommentaren..? Teilweise hab ich schon Code ganz ohne gesehen, ist das nur schwierig weil ich Neuling bin..??

Gute Nacht, morgen geht´s weiter...

Wie macht ihr das mit den Kommentaren..?

Den meisten Anfängern wird leider nur gesagt: Kommentiert viel und alles! Kommentare sind toll! (Könnte ein Zitat von meinem Informatiklehrer sein..)
Aber wie und was man kommentiert wird völlig vernachlässigt. Denn viel wichtiger ist: Kommentieren warum man etwas macht und nicht was man macht, denn das steht ja im Code. Es gibt nichts Schlimmeres als:

int main()
{
  // Datei öffnen
  std::ifstream file("blubb.txt");
  // Aus Datei lesen
  std::string s;
  file >> s;
  // ...
}

Das ist schon mal die Grundregel. Weiter geht es mit: Großflächig kommentieren. Vielleicht vor eine lange Funktion/Klasse schreiben was sie macht, aber nicht jede Zeile kommentieren. Du darfst ruhig davon ausgehen, dass derjenige der den Code liest C++ kann, sonst hilft auch kein Kommentar mehr. Einzelne Zeilen nur dann kommentieren, wenn du denkst dass sie schwer verständlich sind. Und auch dann an die erste Regel denken: Kommentieren warum man etwas macht und nicht was man macht.

Und nein, man kann hier nichts hochladen. Solange du nicht schon wie deine Kollegen 600 Zeilen Code hast, poste es einfach hier. Ansonsten lade es halt irgendwo hoch. Oder nutze gleich Seiten wie http://pastebin.com .

Die Dateien habe ich hochgeladen, sorry dass ich auch hier ein Neuling bin, ich hab´s bei Megaupload hochgeladen:

main: http://www.megaupload.com/?d=9MUL27GV

und außerdem komme ich mit Megaupload nicht zurecht...

main: http://www.loaditup.de/677835-s2k64rptp2.html

O-Klasse Zimmer: http://www.loaditup.de/677836-8sq56q8cks.html
U-Klasse Einzel: http://www.loaditup.de/677837-7n5zn6wx55.html
U-Klasse Doppel: http://www.loaditup.de/677838-26xxm9hkt5.html

Header Funktionen: http://www.loaditup.de/677844-qfavzvmw3k.html

Menü Zimmerstati: http://www.loaditup.de/677839-wdba8vwpxx.html
Stati gefragte Zimmer: http://www.loaditup.de/677841-5vtr89n2df.html
Menü Zimmerbelegen: http://www.loaditup.de/677840-ueu3vqb6n7.html
Zimmer belegen: http://www.loaditup.de/677842-gtravuvtsv.html

Zimmernummer eingeben: http://www.loaditup.de/677843-c42nxr4834.html

So läuft bisher alles, fertig bin ich noch nicht ein paar Punkte fehlen noch.

• system("cls");
• Zu viel system("Pause") (warum brauchst du das eig.)?
• Cahr-Array mit der Länge 1 benutzt (das '\0'-Zeichen am Ende eines Strings benötigit dieses Byte, in das variabel lange Strings eingelesen werden, d.h. schon wenn charMenupunkt[0] != '\0' ist hast du in Speicher geschrieben, der dir nicht gehört),
• die Umwandlung von char[] nach std::string ist völlig sinnlos und
• deine Plausiblitätsprüfung ist auch nicht unbedingt geignet. Das Problem gekürzt sieht so aus:

string strMenupunkt;
cin >> strMenupunkt;

//entweder
switch(strMenupunkt[0])
{
    //...
}

//oder
if(strMenupunkt == "1") { }
else if(strMenupunkt == "2") { } //...
else cout << "Fehlerhafte Eingabe\n";

Kritikpunkte Zimmer.h:

• NIEMALS using namespace std; in Header-Dateien
• Für alles Strings zu verwenden ist nicht immer sinnvoll, stdrBelegt könnte man z.B. zu bool bBelegt; machen
• belegen könnte man folgendermaßen verbessern:

void belegen()
{
	bool bEingabeOk = false;

	bBelegt = true; //nur wenn du Tipp2 beachtest, ansonsten wie gehabt

	while (!bEingabeOk)
	{
		cout << "Bitte den Vor- und Nachname des Gastes eintragen.\n";
		cout << "   > ";
		getline(cin, strGast);

		if (strGast.length() < 5)
		{
			cout << "Bitte geben Sie einen gültigen Namen ein\n";
		}
		else bEingabeOk = true;
	}
}

• Zimmerdatenausgeben könnte man virtual machen, um es in der Abgeleiteten Klasse zu überschreiben
• Die Funktionen für die Zimmernummererzeugung könnte man pure vritual machen

Kritikpunkte zu Einzelzimmer.h (und damit auch zu Doppelzimmer.h, ich gucks mir nicht erst an, weils wahrscheinlich sehr änlich ist):

• Die Funktion Zimmernummer erzeugen funktioniert nur von 1 bis 10. Bei Stringstream kann man doch die Breite und das Füllzeichen einstellen.
• Für eine Bessere OO nehm dir die Tipps von Zimmer.h zu herzen und bastel so die Klasse mit den Überschreibungen um

Headerfunktionen: Bis auf using namespace erstmal keine Kritik. Aber mal Zwischenfrage, hat es einen Grund, warum du hier nie Parameternamen mitgiebst?

MenuZimmerstati.cpp, MenuZimmerbelegen.cpp, Zimmernummereingeben.cpp: siehe Kritik zu main

StatiGefragteZimmer.cpp: system("cls"); und müsste natürlich angepasst werden wenn Änderung von Zimmer-Klasse durchgeführt wird.

Zimmerbelegen.cpp: soweit keine Kritik

Hab die Aufgabe jetzt schon abgeben, weil´s heute einfach drunter und drüber ging und ich eh schon knapp geworden bin.
Trotzdem will ich deine Tipps natürlich haben..!

Also:
- wie bekomme ich es anders hin die Konsole wieder zu löschen außer mit system("cls")..? (hab ich aus dem Netz übrigens) Gemacht hab ich´s damit sich die Ausgabe schöner liest.

- genauso bei system("Pause"), welche Alternative habe ich (return 0;..?), und warum geht das nicht überall..?

- dass strMenuepunkt[0] ein char ist habe ich leider erst kapiert als ich mal probiert habe auf einzelne Zeichen in einem String zuzugreifen..da hast du natürlich Recht..!

NIEMALS using namespace std; in Header-Dateien

- das klingt nach Gefahr, also bitte sag mir warum nicht..?

Für alles Strings zu verwenden ist nicht immer sinnvoll...

- da hast du auch Recht, ich war da nur so beschäftigt mit den Klassen und den anderen Problemen dass ich darüber weniger nachgedacht hab. In den Klasen wollte ich zuerst die Variablen strRechnungGestellt und strStatus als bool deklarieren, hab ich dann aber wegen der Werteausgabe dagegen entschieden.

- mit virtuell geht´s wieder in Richtung Zeiger, oder..? Das sagt mir leider noch zu wenig, virtual/pure virtual gar nichts. In den Semesterferien würde ich gern ein Buch durch machen, in Ruhe, hast du da vielleicht eine Empfehlung..?
Dazu bitte ich dich auch noch um deine Meinung: muss ich C können um C++ verstehen zu können..? Ist C noch so präsent oder wird es von C++ abgelöst..?
Mein Studiengang ist Produktionstechnik und logischerweise ist mir noch nicht >ganz< klar wohin ich mich spezialisieren will, da werden sich im Laufe des Studiums mit Sicherheit noch Themen ergeben von denen ich bis jetzt noch nicht mal gehört habe.
Programmieren macht mir Spaß und ich saug es richtig in mich hinein (wenn die Unterlagen was gescheites sind ), also das werde ich auf jeden Fall machen...
Bisher habe ich einige Excel Makros geschrieben, auch größere..durch meine damalige Arbeit war da nur die Zeit für´s oberflächliche, "funktionale" Wissen - das hat mich da schon geärgert, auch deswegen noch mal die Bitte um den Buchtip damit ich´s hier besser machen kann.

- Zum Zimmernummer erstellen: meinst du damit ein Variables Feld nehmen..? (Stringstream ist auch aus dem Netz, nicht aus dem Kurs...) Vorgabe waren 10 Zimmer, deswegen habe ich nicht mehr Zeit investiert..das es geht habe ich im Netz schon mal nebenbei gelesen, aber das war nicht Thema des Kurses.

- Zu OOP: mal im Ernst, die Zimmer in Ober- & Unterklassen aufzuteilen war hier doch der totale Mist, oder..? Gemacht hab ich´s weil ich dachte es ist ne gute Übung zum Thema Vererbung und zur Verinnerlichung des ganzen Themas, aber hier habe ich doch eine Menge Zeit verplempert die Methoden der ganzen Klassen zu schreiben - bei einer einzigen Klasse Zimmer wäre das doch übersichtlicher und kürzer gewesen..?? Noch dazu bin ich total skeptisch ob ich das nicht sowieso alles zu kompliziert und durcheinander programmiert habe...

Headerfunktionen: ... Aber mal Zwischenfrage, hat es einen Grund, warum du hier nie Parameternamen mitgiebst?

- wie meinst du das..? So wie ich´s gemacht habe, habe ich´s im Netz recherchiert (Funktionen in andere Dateien auslagern war auch nicht Teil des Kurses, weil im Netz einfach ne Menge steht und ich´s da gelesen habe, im Bezug auf Header allgemein, hat´s mich einfach interessiert...)