Ich soll eine Speisekarte programmieren.
Ich habe als beispiel ein Bistro genommen.
Man soll im Programm sich sein eigenes Menü zusammenstellen können.
Und das Programm soll eine Art Warenkorb haben, in dem man nachschauen kann was man bestellt hat und wie viel die Endsumme beträgt.

Meine Aufgaben nochmal in der Zusammenfassung:
1. Es muss noch eine Lösung für den Wechsel zwischen mainMenue und subMenue gemacht werden (loop bis explizit quit durch nutzer)

2. Überprüfen ob die Nutzereingabe valide ist (Kategorie, Bestellung)

3. Warenkorb muss nach jeder Gerichts Auswahl gefüllt werden

4. Kasse muss geschrieben werden (Ausgabe aller Bestellungen im Warenkorb, plus Summe der Preise

5. Benutzer freundliches Menü muss erstellt werden.

Ich arbeit mit dem Dev-C++ 4.9.9.0.

Ich bedanke mich im voraus an ALLE, die diesen Artikel gelesen haben und mir vielleicht ein wenig weiter helfen können.

Hier ist der Code, den ich bis jetzt geschrieben habe:

#include <string>
#include <cstdlib>
#include <iostream>
#include <vector>
#include <iomanip>
#include <stdio.h>

using namespace std;

const unsigned int KARTEN_GROESSE=4;
const unsigned int KATEGORIEN=5;
char entscheidung;

	string kategorie[5]={"1. Vorspeisen","2. Salate","3. Suppen","4. Hauptspeisen","5. Nachspeisen"};

	struct speise {                           
		   int bestellnummer;
		   string name;
		   float Preis;
	};

	int currentKat; 

	speise karte[KATEGORIEN][KARTEN_GROESSE];

        vector< vector<int> > warenkorb;

do
{
	void mainMenue (){
		 int i;
		 int j;
		 int choice;

    for (i=0;i<KATEGORIEN;i++){
        cout << kategorie[i] << endl;
    }
    cout << "\n Bitte w\x84hlen sie eine Kategorie: ";
    cin >> choice;
    cout << endl << endl << "Ihre Wahl: \t" << kategorie[choice-1] << endl << endl;
    currentKat = choice;
	}

	int subMenue() {
		int i;
		//int j;
		int gericht;
		int kat = currentKat-1;

		for(i=0;i<KARTEN_GROESSE;i++){
			cout << setprecision(2) << fixed;
			cout << karte[kat][i].bestellnummer << karte[kat][i].name << karte[kat][i].Preis<< endl;                    
		}
		cout << "\n\n Bitte w\x84hlen Sie ein Gericht: ";
		cin >> gericht;
		return gericht;
		//mainMenue(gericht)
	}

	void map(int bestellnr, int kat){
		 int i;
		 for (i=0;i<KARTEN_GROESSE;i++){
			 if (karte[currentKat][i].bestellnummer == bestellnr) {
										 warenkorb.push_back(currentKat,i);
										 }
			 }
		 }

	int main() {

		karte [0][0].bestellnummer = 101;
		karte [0][0].name = " Lavas \t"; 
		karte [0][0].Preis = 3.50;
		karte [0][1].bestellnummer = 102;
		karte [0][1].name = " Dolma \t";
		karte [0][1].Preis = 3.30;
		karte [0][2].bestellnummer = 103;
		karte [0][2].name =" B\x94rek \t";
		karte [0][2].Preis = 3.40;
		karte [0][3].bestellnummer = 104;
		karte [0][3].name =" Feta \t";
		karte [0][3].Preis = 4.50;

		karte [1][0].bestellnummer = 201;
		karte [1][0].name =" Hirtensalat \t";
		karte [1][0].Preis = 4.50;
		karte [1][1].bestellnummer = 202;
		karte [1][1].name =" Bulgursalat \t";
		karte [1][1].Preis = 4.10;
		karte [1][2].bestellnummer = 203;
		karte [1][2].name =" Artischockensalat \t";
		karte [1][2].Preis = 3.80;
		karte [1][3].bestellnummer = 204;
		karte [1][3].name =" Mozarella Salat \t";
		karte [1][3].Preis = 3.10;

		karte [2][0].bestellnummer = 301;
		karte [2][0].name =" Pansensuppe \t";
		karte [2][0].Preis = 3.60;
		karte [2][1].bestellnummer = 302;
		karte [2][1].name =" Knoblauchsuppe \t";
		karte [2][1].Preis = 4.20;
		karte [2][2].bestellnummer = 303;
		karte [2][2].name =" Tarhanasuppe \t";
		karte [2][2].Preis = 3.70;
		karte [2][3].bestellnummer = 304;
		karte [2][3].name =" M\x94hrensuppe \t";
		karte [2][3].Preis = 3.20;

		karte [3][0].bestellnummer = 401;
		karte [3][0].name =" Auberginenfleischpfanne \t";
		karte [3][0].Preis = 5.60;
		karte [3][1].bestellnummer = 402;
		karte [3][1].name =" Fleischb\x84llchen \t\t";
		karte [3][1].Preis = 5.20;
		karte [3][2].bestellnummer = 403;
		karte [3][2].name =" Istanbulreis \t\t";
		karte [3][2].Preis = 4.70;
		karte [3][3].bestellnummer = 404;
		karte [3][3].name =" Sis-Kebap \t\t\t";
		karte [3][3].Preis = 4.20;

		karte [4][0].bestellnummer = 501;
		karte [4][0].name =" Tulumba \t\t";
		karte [4][0].Preis = 2.60;
		karte [4][1].bestellnummer = 502;
		karte [4][1].name =" Nachtigallennester \t";
		karte [4][1].Preis = 3.20;
		karte [4][2].bestellnummer = 503;
		karte [4][2].name =" Halwa \t\t";
		karte [4][2].Preis = 3.70;
		karte [4][3].bestellnummer = 504;
		karte [4][3].name =" Mandelcreme \t" ;
		karte [4][3].Preis = 3.20;

		mainMenue();
		int gericht = subMenue();

		cout << gericht << endl << endl;
		cout << "Wollen Sie noch etwas zu Ihrem Menu w/x84hlen (j,n)? ";
		cin >> entscheidung;
	}
}
while(entscheidung == 'j');

    system ("PAUSE");
        return EXIT_SUCCESS;                    
}

erstelle fuer jede kategorie ein objekt mit "getPrice", "getName" usw, und diese objekte wissen was sie alles anbieten,
so direkt ueber n array, und dieses fuellen, das ist doch grausig und versammt fehleranfaellig

zb

class speisen {
private:
    int _BestellNummer;
    std::string _Name;
    double _Preis;
};

class Vorspeisen {
private:
    int _nr;
    std::vector<speisen> _alleVorSpeisen;

public:
    int gibVorSpeise(int nr);
};

Vorspeisen::Vorspeisem {
    alleVorSpeisen.push_back(...);
}

- [ code]-Tags sind gut, [ cpp]-Tags sind besser. Ausserdem sind deine EInrueckungen etwas durcheinander geraten, das macht den Code schwer lesbar.

- <stdio.h> ist veraltet, nutze stattdessen <cstdio> - bzw. lass es ganz weg, ich seh naemlich nicht, dass du irgendeine Funktion daraus in deinem Code haettest (lobenswert!), die <cstdlib> kann soweit ich das sehe auch weg.

- da du schon die Konstanten KARTEN_GROESSE und KATEGORIEN definiert hast (vorbildlich), nutze sie auch: string kategorie[KATEGORIEN]... in Zeile 14

- sei konsistent in der Wahl deiner Container, du nutzt mal vector, mal simple Arrays (nur vector waere besser)

- du deklarierst auf globlaer Ebene einen do-while loop und darin dann funktionen, unter anderem die main(). Ich bezweifle dass dein COmpiler das compiliert, der loop muss in die main. ausserdem wuerde ich die Initialisierung der Karte in eine eigene Funktion auslagern.

- die meisten (oder alle) globalen Variablen kannst du mit in die main() packen.

- deklariere fuer dein struct "speise" einen Konstruktor, dann wird das initialisieren der Sepisekarte nicht so ein Tippaufwand.

Wenn du die Punkte oben bereinigt hast, poste deinen Code noch einmal und stelle vor allem eine Frage dazu - und zeig uns wie weit du bei deren beantwortung schon gekommen bist

http://de.wikipedia.org/wiki/Blog

Ich habe versucht, den Code übersichtlicher zu machen.

Allerdings komme ich nicht weiter

Ich habe es nicht geschafft,
eine Lösung für den Wechsel zwischen mainMenue und subMenue zu schreiben (loop bis explizit quit durch nutzer).

Überprüfen ob die Nutzereingabe valide ist (Kategorie, Bestellung) hab ich auch nicht hinbekommen.

Überprüfen ob die Nutzereingabe valide ist (Kategorie, Bestellung)

Die Aufagebe "3. Warenkorb muss nach jeder Gerichts Auswahl gefüllt werden"

und

die Aufgabe "4. Kasse muss geschrieben werden (Ausgabe aller Bestellungen im Warenkorb, plus Summe der Preise"

hab ich auch nicht geschafft.

Ich habe mir in den letzten Tagen viel durchgelesen, doch es hat immer irgendwie gehappert.
Doch die Hoffnung habe ich trotzdem nicht aufgegeben!

Ich danke jedem, der mir versucht zu helfen!!!
Oder der mir hilft!!!

Also zum Beispiel machst du da (in einer seperaten .cpp) eine Klasse CWarenkorb und dort so Funktionen, wie zum Beispiel AddObject (std::string, int prize) und so Zeugs.

Und Schlussendlich wirst du dann selber auf die Probleme und Lösungen kommen.

Zu den Aufgaben:
eine Lösung für den Wechsel zwischen mainMenue und subMenue zu schreiben (loop bis explizit quit durch nutzer).
-> Du beendest immer, wenn der User irgendwas anderes, als 'j' eingibt.

Überprüfen ob die Nutzereingabe valide ist (Kategorie, Bestellung) hab ich auch nicht hinbekommen.
-> Einfach mit ein paar if's prüfen (OOP ist das viel einfacher)

Überprüfen ob die Nutzereingabe valide ist (Kategorie, Bestellung)
-> Einfach mit ein paar if's prüfen (OOP ist das viel einfacher)

Die Aufagebe "3. Warenkorb muss nach jeder Gerichts Auswahl gefüllt werden"
-> Einfach mit ein paar if's prüfen (OOP ist das viel einfacher)
und

die Aufgabe "4. Kasse muss geschrieben werden (Ausgabe aller Bestellungen im Warenkorb, plus Summe der Preise"
-> OOP

Tut mir leid, wenn ich frage, aber was ist OOP?

Du hast wirklich ein bisschen wurschtelcode da.

Selsbst geschrieben?

Ich werde jetzt versuche ein wenig Ordnung in den Code zu bringen.

...Ist der Code so schlimm geschrieben?...

Ja. Beginnt schon damit das man schwer Anfang und Ende eines Scopes erkennen kann. Und noch ein Tip: vor dem Kopieren in das Forum sicherstellen das Tabulatoren durch Leerzeichen ersetzt werden (Tabulatoren haben keine genormte Breite, und du mischt in deinem Code Tabulatoren und Leerzeichen).

Eine sehr verbreitete Form für einen Übersichtlichen Programmierstil (es gibt aber mehrere Formen, ich hatte schon mehrfach in diesen Forum Beispiele gebracht, und will mich nicht wiederholen):

Sinn ist es, das man die Verschachtelungen klar sieht, und durch identische Einzüge alle zugehörigen Teile zusammengehörend erkennt

int main()
{ // Scope 1 Begin
    // Einzugshöhe innerhalb Scope 1
    int value;
    for(int i=0; i<10; ++i)
    { // Scope 2 Begin
        // Einzugshöhe innerhalb Scope 2
        TueIrgendwas();
    } // Scope 2 Ende
} // Scope 1 Ende

Ich verwende in der Regel einen Einzug von 4 Leerzeichen, dies ist aber eine
Geschmackssache. Falls ich eine Zeile umbreche so setze ich diese nur leicht versetzt, da man einerseits erkennen soll das sie zu der Vorzeile gehört, zum anderen erkennen soll das sie noch nicht zu einem anderen Scope zählt:

int main()
{
    std::string a = "Dies ist ein langer Text der um die Lesbarkeit zu "
      "erhöhen zwar etwas eingerückt ist, aber noch nicht auf höhe eines "
      "tieferliegenden Scopes liegt ";

    for(int i=0; i<10; ++i)
        TueIrgendwas(); // Auch ohne Klammern kann man hier den Scope erkennen
}

Zudem solltest du dich von deinen vielen globalen Variablen verabschieden. Und sei es das du langsam anfängst. Funktionen besitzen Rückgabewerte und Übergabeparameter, diese darfst du auch verwenden

Was mich mal interessiert ist der Wissensstand den du eigentlich haben solltest. Der Aufgabe nach zu urteilen würde ich eigentlich darauf tippen das Klassen und Methoden bereits bekannt sind. Oder weißt du wenigstens unter welchen Bereich deine Aufgaben fallen (Prozedurale Programmierung, Objektorientierte Programmierun)? Je nach dem welche Mittel dir zur Verfügung stehen sind unterschiedliche Hilfestellungen sinnvoll.

cu André

Ich bin gerade in der Ausbildung zum Fachinformatiker.
Und bin im ersten Semester.

Die erste sprache, die wir lernen meinte mein lehrer ist c++

Einfach ausgedrückt ist der "Sinn" von C++ ist die Mächtigkeit von Klassen (und natürlich Templates). Also Obejektorientierung, die du unbedingt anfangen nutzten solltest.
Eventuell solltest du mal ein Tutorial durchmachen oder zumindest die Teile dazu lesen, die was mit OOP zu tun haben. ( http://tutorial.schornboeck.net/inhalt.htm )

Ich werde mir das mal gleich anschauen und durch lesen!

Danke!

DEventuell solltest du mal ein Tutorial durchmachen oder zumindest die Teile dazu lesen, die was mit OOP zu tun haben...

Das ist jetzt nichts gegen Oguzhan, aber nach seiner scheinbaren Unsicherheit würde ich ihm ganz dringend dazu raten nicht erst mit dem OOP Bereich das Tutorial zu beginnen. Gewisse Grundlagen fehlen ihm noch, oder er ist zumindestens noch nicht sicher genug darin.

cu André

#include <string>
#include <iostream>
#include <vector>
#include <map>
#include <iomanip>

using namespace std;

const int N_KATEGORIEN = 5;
const string KATEGORIEN[N_KATEGORIEN] =
{
    "1. Vorspeisen",
    "2. Salate",
    "3. Suppen",
    "4. Hauptspeisen",
    "5. Nachspeisen"
};
enum KATEGORIE
{
    VORSPEISE,
    SALAT,
    SUPPE,
    HAUPTSPEISE,
    NACHSPEISE
};

////////////////////////////////////////////////////////////////////////////////

class Artikel
{
public:
	Artikel(size_t nr, KATEGORIE kategorie, const string &name, float preis);

    size_t  zeigeNr()        const  { return m_nr; }
    string  zeigeKategorie() const  { return KATEGORIEN[m_kategorie]; }
    string  zeigeName()      const  { return m_name; }
    float   zeigePreis()     const  { return m_preis; }

private:
	size_t      m_nr;
	KATEGORIE   m_kategorie;
	string      m_name;
	float       m_preis;
};

Artikel::Artikel(size_t nr, KATEGORIE kategorie, const string &name, float preis)
    : m_nr(nr), m_kategorie(kategorie), m_name(name), m_preis(preis)
{
}

ostream &operator<<(ostream &ausgabestrom, const Artikel &a)
{
    return ausgabestrom
        << a.zeigeNr() << "\t"
        << a.zeigeKategorie() << "\t"
        << a.zeigeName() << "\t"
        << a.zeigePreis() << "\n";
}

typedef vector<Artikel> Artikelliste;

////////////////////////////////////////////////////////////////////////////////

class Speisekarte
{
public:
	void zeigeKategorie(KATEGORIE k) const;
	Artikel gibArtikel(size_t nr) const;
	void registriereArtikel(size_t nr, KATEGORIE kategorie, const string &name, float preis);

private:
	map<KATEGORIE, Artikelliste> m_kategorisiert;
	map<size_t, Artikel *> m_inhalt;
};

void Speisekarte::zeigeKategorie(KATEGORIE k) const
{
    //TODO: Auflistung der Artikel einer Kategorie
}

Artikel Speisekarte::gibArtikel(size_t nr) const
{
    return *(m_inhalt.find(nr)->second);
}

void Speisekarte::registriereArtikel(size_t nr, KATEGORIE kategorie, const string &name, float preis)
{
    Artikel a(nr, kategorie, name, preis);

    //TODO: Gucken, ob es schon einen Artikel mit der Nummer gibt

    m_kategorisiert[kategorie].push_back(a);
    m_inhalt[nr] = &m_kategorisiert[kategorie].back();
}

////////////////////////////////////////////////////////////////////////////////

class Warenkorb
{
public:
    size_t anzahlArtikel() const
    {
        return m_inhalt.size();
    }
    const Artikel &zeigeArtikel(size_t position) const
    {
        return m_inhalt[position];
    }

	void legeRein(const Artikel &einArtikel);
	void legeRein(int nr, KATEGORIE kategorie, const string &name, float preis);

	float gesamtpreis() const;

private:
    Artikelliste m_inhalt;
};

void Warenkorb::legeRein(const Artikel &einArtikel)
{
    m_inhalt.push_back(einArtikel);
}

void Warenkorb::legeRein(int nr, KATEGORIE kategorie, const string &name, float preis)
{
    Artikel a(nr, kategorie, name, preis);
    legeRein(a);
}

float Warenkorb::gesamtpreis() const
{
    float ergebnis = 0;

    //TODO: Preise der einzelnen Artikel zusammenrechnen

    return ergebnis;
}

////////////////////////////////////////////////////////////////////////////////

class Bistro
{
public:
	Bistro(const char *name);

    void bestellung(Warenkorb &korb) const;
    KATEGORIE auswahlKategorie() const;
	void kasse(const Warenkorb &korb) const;

private:
    string m_name;
    Speisekarte m_karte;
};

Bistro::Bistro(const char *name)
    : m_name(name)
{
    //TODO: Speisekarte initialisieren

}

void Bistro::bestellung(Warenkorb &korb) const
{
    cout << "Willkommen im Bistro " << m_name << "!\n";

    //TODO: Bestellung in einer Schleife mit Menü und Submenü

    //testhalber einfach mal was in den Korb gelegt
    korb.legeRein(4711, VORSPEISE, "Rote Wurst", 2.50);
    korb.legeRein(4712, HAUPTSPEISE, "Halbe Sau", 12);
}

KATEGORIE Bistro::auswahlKategorie() const
{
    KATEGORIE ergebnis = VORSPEISE;

    //TODO: Auswahl der Kategorie

    return ergebnis;
}

void Bistro::kasse(const Warenkorb &korb) const
{
    cout << "Sie haben folgendes bestellt:\n";
    for (size_t i = 0; i < korb.anzahlArtikel(); ++i)
    	cout << korb.zeigeArtikel(i);

    cout << "Ihre Bestellung kostet " << korb.gesamtpreis() << " Euro.\n";
}

////////////////////////////////////////////////////////////////////////////////

int main()
{
    Warenkorb wk;
    Bistro b("Oguzhan");
    b.bestellung(wk);
    b.kasse(wk);
}

viel spaß noch

ich habe versucht die Aufgaben, die du mir in den Code eingebaut hast zu lösen.

Leider zeigt der Compiler mir immer noch Fehler an

Ich probiere schon eine ganze Weile herum und komme leider trotz googlen nicht weiter.

Kann mir vielleicht jemand einen denk Anstoß geben?

Danke im voraus!