Unbehandelte Ausnahme nach Laden einer Datei

314159265358979

Sorex schrieb:

Da hab ich einfach mal dem Autor des Buches vertraut, aber gut es jetzt besser zu wissen.

Welches Buch ist das? Hat der Autor die Initialen J.W.?

Sorex schrieb:

Den Binärdatentyp habe ich in der aktuellen Version nicht mehr drin

Das hat damit nichts zu tun. Sobald du .read() aufrufst, liest du hier Binär.
Mit >> liest man formatiert.

Hast du auch die Speicher-Methode mit angepasst? Die beiden müssen natürlich Hand in Hand arbeiten, poste optimalerweise nochmal deine jetzige Fassung beider Methoden.

Das ist das Problem, wenn man Unreg ist, man kann nicht editieren.

Dir ist schon klar, dass du beim Speichern eine Datei mit Namen "Chrakter.ch" und NICHT "Charakter.ch" erstellst?

Sorex

Aktuell sehen sie so aus:

void Game::saveGame()
{
	std::cout << "\nWirklich speichern? Alter Spielstand wird dabei überschrieben\n"
		      << "Bestätigen durch Eingabe von (J)a bzw (N)ein: ";
	char str;
	std::cin >> str;
	if (str == 'j' || str == 'J')
	{
		std::ofstream Output("hero.ch");
		if (Output == NULL)
		{
			std::cout << "\nDatei zum Speichern konnte nicht geöffnet werden!\n";
		}
		else
		{

		   Output.write((char*) &hero.attackPower_, sizeof(hero.attackPower_));
		   Output.write((char*) &hero.defensePower_, sizeof(hero.defensePower_));
		   Output.write((char*) &hero.hitPoints_, sizeof(hero.hitPoints_));
		   Output.write((char*) &hero.hpMax_, sizeof(hero.hpMax_));
		   Output.write((char*) &hero.mana_, sizeof(hero.mana_));
		   Output.write((char*) &hero.manaMax_, sizeof(hero.manaMax_));
		   Output.write((char*) &hero.level_, sizeof(hero.level_));
		   Output.write((char*) &hero.exp_, sizeof(hero.exp_));
		   Output.write((char*) &hero.expMax_, sizeof(hero.expMax_));
		   Output.write((char*) &hero.money_, sizeof(hero.money_));
		   Output.write((char*) &hero.creditPoints_, sizeof(hero.creditPoints_));
		   Output << hero.name_;

		   std::cout << "\nSpielstand erfolgreich gesichert\n";
		}
	    Output.close();
	}
	else if (str == 'n'|| str == 'N')
	{
		std::cout << "\nSpeichern abgebrochen\n";
	}
	else
	{
		std::cout << "\nFalsche Eingabe\n";
	}
}

void Game::loadGame()
{
    std::cout << "\nWirklich laden? Bisherige Erfolge werden gelöscht\n"
              << "Bestätigen durch Eingabe von Ja bzw Nein: ";
    char str;
    std::cin >> str;

    if (str == 'j' || str == 'J')
    {
        std::ifstream Input("hero.ch");
        if(Input == NULL)
        {
            std::cout << "\nDatei konnte nicht geöffnet werden!\n";
        }
        else
        {  

           Input.read((char*) &hero.attackPower_, sizeof(hero.attackPower_));
           Input.read((char*) &hero.defensePower_, sizeof(hero.defensePower_));
           Input.read((char*) &hero.hitPoints_, sizeof(hero.hitPoints_));
           Input.read((char*) &hero.hpMax_, sizeof(hero.hpMax_));
           Input.read((char*) &hero.mana_, sizeof(hero.mana_));
           Input.read((char*) &hero.manaMax_, sizeof(hero.manaMax_));
           Input.read((char*) &hero.level_, sizeof(hero.level_));
           Input.read((char*) &hero.exp_, sizeof(hero.exp_));
           Input.read((char*) &hero.expMax_, sizeof(hero.expMax_));
           Input.read((char*) &hero.money_, sizeof(hero.money_));
           Input.read((char*) &hero.creditPoints_, sizeof(hero.creditPoints_));
           Input >> hero.name_;

            std::cout << "\nSpielstand erfolgreich geladen\n";
        }
        Input.close();

        if(Input.good())
           std::cout << "JA\n";
        else
           std::cout << "NEIN\n";
    }
    else if (str == 'n'|| str == 'N')
    {
        std::cout << "\nLaden abgebrochen\n";
    }
    else
    {
        std::cout << "\nFalsche Eingabe\n";
    }
}

Welches Buch ist das? Hat der Autor die Initialen J.W.?

Das Buch nennt sich C++ für Spieleprogrammierer und ist von Heiko Kalista

Sorex

bitmap.button schrieb:

Das ist das Problem, wenn man Unreg ist, man kann nicht editieren.

Dir ist schon klar, dass du beim Speichern eine Datei mit Namen "Chrakter.ch" und NICHT "Charakter.ch" erstellst?

Ja, das war die Version von gestern.. hab sie danach auf hero.ch umgetauft (entstehen weniger Tippfehler xD)

CStoll

Wie schon mehrfach erwähnt wurde: Verzichte lieber auf die read()/write() und verwende für alle Daten die Stream-Operatoren.

@Pi: Hat der eof-Test bei deinem Code eigentlich einen tieferen Sinn?

314159265358979

CStoll schrieb:

@Pi: Hat der eof-Test bei deinem Code eigentlich einen tieferen Sinn?

Wenn das Ende erreicht wurde, ist alles glatt gelaufen. Wenn nicht, nicht. Ist doch logisch?

CStoll

Erstens erinnere ich mich düster, daß eof erst nach einem erfolglosen Leseversuch gesetzt wird. Und zweitens: Was machst du, wenn in der Datei mehr steht als dein Lese-Operator erfasst hat?

314159265358979

CStoll schrieb:

Erstens erinnere ich mich düster, daß eof erst nach einem erfolglosen Leseversuch gesetzt wird. Und zweitens: Was machst du, wenn in der Datei mehr steht als dein Lese-Operator erfasst hat?

Im Normalfall lese ich alles aus. Will man das nicht, muss man anders vorgehen.
Aber du hast wohl Recht, wenn du meinst, dass die Überprüfung mit .fail() besser ist, da immer funktioniert.

Sorex

CStoll schrieb:

Wie schon mehrfach erwähnt wurde: Verzichte lieber auf die read()/write() und verwende für alle Daten die Stream-Operatoren.

@Pi: Hat der eof-Test bei deinem Code eigentlich einen tieferen Sinn?

Hab das so gemacht, funktioniert auch wunderbar, nur das Ergebnis, was Input.good() liefert bleibt auch hier false.
Aber warum ist das so? Ist das überhaupt von Belang oder kann ich das getrost ignorieren?