ich habe vor ein paar Tagen damit angefangen C++ zu lernen.

Momentan bin ich dabei ein kleines Quizprogramm zu schreiben.
Dabei bin ich auf ein Problem gestoßen und ich habe einfach keine Ahnung warum das passiert was passiert.
Natürlich sind meine C++ kenntnisse noch sehr beschränkt und Ihr werdet euch herzlich über meinen Quelltext amüsieren aber was solls:

Wie ihr seht habe ich unten mittels Zufallsgenerator einen Datensatz aus einer externen *.txt datei ausgewählt.
Nachdem ich eine Frage egal ob Falsch oder Richtig beantworte beginnt das script von vorne... so oft wie Datensätze vorhanden sind.

Allerdings wird danach die Frage nicht mehr korrekt gestellt. Will heißen es wird kein Land mehr abgefragt sondern eine Leerstelle. Ich kann leider den fehler nicht erkennen. Sicherlich könnt ihr mir dabei helfen.

#include <iostream>
#include <fstream>
#include <string>
#include <cstdlib>
#include <ctime>
using namespace std;

main()
{
	ifstream dat_ein;
	string dateiname = ("daten.txt.");

	dat_ein.open(dateiname.c_str(), ios_base::in);

		if(!dat_ein)
		{
			cout << "Die Datei konnte nicht geöffnet werden!";
			cout << "endl";
			return -1;
		}

	int anzahl;
	int datensatz;

	string land, stadt;

	int loop;
	int loop1;

	string antwort;

	dat_ein >> anzahl;

for(loop1 = 1; loop1 <= anzahl; ++loop1)
{
	srand( (unsigned)time( NULL) );
	datensatz = rand() % anzahl + 1;

	loop = 2 * datensatz;

	while(loop >= 2)
	{
	dat_ein >> land >> stadt;
	loop -= 2;
	}

	cout << "Wie heisst die Hauptstadt von " << land << "? ";
	cin >> antwort;
	cout << endl << endl;

	if(antwort == stadt)
	{
		cout << "\tRichtig!" << endl << endl;
	}
	else
	{
		cout << "\tFALSCH!" << endl << "Die richtige Antwort lautet " << stadt << "!" << endl << endl << endl;
	}
}

dat_ein.close();

return 0;
}

mfg Fusle

PS: Sorry das ich euch mit solchen trivialitäten die Zeit stehle

Und normal heißt es

int main ()
{
.....
}
------------

cout << endl; statt cout << "endl";

------------

So kleiner Tipp am Rande. Definiere deine Variablen am besten an Funktionsanfang.
Ist leichter zu verstehen und zu warten.

Also dein Problem ist das -> dat_ein >> anzahl;
Denke mal das soll dafür stehen wie viel Datensätze vorhanden sind.
Bei mir kommt es bei 4 zeilen in daten.txt zu der Zahl 4487064.

Und da deine Schleife "for(loop1 = 1; loop1 <= anzahl(4487064); ++loop1)" ist, wird die auch nicht so schnell beendet.

Was du besten tun solltest. Alle Datensätze einlesen zuerst. Und dann haste deine Anzahl.

Glaub geht mit einer while schleife mit EOF und wen da alle Zeilen Zählst das durch 2.

Ps.: bin auch noch Anfänger. Aber hoffe hilft dir trotzdem.

mfg isi

9
Angola Luanda
Kolumbien Bogota
USA Washington
Schweiz Bern
Schweden Stockholm
Senegal Dakar
Niederlande Amsterdam
Deutschland Berlin
Japan Tokio

Und zu der sache mit INT main () << danke hab ich wohl geschlampt.

-----------------------------------------------------------

EDIT: nachdem ich jetzt sicher 30 mal das Programm getestet habe ist mir aufgefallen das nicht immer das land in der frage fehlt. Manchmal kommen 2 - 3 "korrekte" fragen bis der Fehler wieder auftaucht.
Klingt das nach einem fehler in der Datenbank?

---------------------------------------------------------

EDIT2 : Also die "for(loop1 = 1; loop1 <= anzahl; ++loop1)"
wird nach exakt neun durchgängen beendet.

Die Frage: "Wie lautet die Hauptstadt von ?" kann ich ja immernoch falsch beantworten und nach 9 falsch beantworten Fragen ist das Programm dann auch beendet...

Nur wie gesagt verstehe ich immernoch nicht aus welchem Grund auf einmal kein Land mehr in der Frage vorkommt

Ist ja auch nicht allzu wichtig... Sollte ja nur eine Übung sein vielleicht kommt mir die erleuchtung ja wenn ich mich etwas mehr auskenne
Und das mit den Variablen werd ich mir angewöhnen.

PS: wo siehst du cout << "endl"; ?

Was mir auffällt.

Bei Mir wird sind meist 2-3 fragen richtig. Und nach der letzten Frage folgt nur noch Japan Tokio, weil da wohl das Dateiende erreicht ist. Das Random arbeitet auch richtig. Ich denke du müsstest nach dem Daten einlesen deine Datei schliessen. Damit die wieder bei 0 anfängt. Da ich leider noch nichts mit fstream geschrieben hab, musst du wohl google nutzen oder warten.

Ich denke eben das zb
datensatz 8 = Deutschland
datensatz 7 = Japan
dann und dann bleibt er da stehen.

Als würde er nicht einen direkten Sprung machen, sondern Addieren.

mfg

ps:

dat_ein.open(dateiname.c_str(), ios_base::in);

        if(!dat_ein)
        {
            cout << "Die Datei konnte nicht geöffnet werden!";
     >>>>>  cout << "endl";      <<<<<<<
            return -1;

Weite ich diese aber aus und Wiederhole den gesamten Queltext funktioniert das ganze. Warum genau weiss ich jetz leider auch nicht aber vielleicht wirds mir irgenwdann klar.

Hier nochmal der korrekte Code:

// quiz.cpp Ein kleines Haupstädtequiz
//
#include <iostream>
#include <fstream>
#include <string>
#include <cstdlib>
#include <ctime>
#include <cstdio>
using namespace std;

int main()
{
	int anzahl;
	int datensatz;
	int richtig;
	int falsch;
	int loop;
	int zaehler;

	richtig = 0;
	falsch = 0;

for(zaehler = 1; zaehler <= 9; ++zaehler)
{

	ifstream dat_ein;
	string dateiname = ("daten.txt.");

	dat_ein.open(dateiname.c_str(), ios_base::in);

		if(!dat_ein)
		{
			cout << "Die Datei konnte nicht geöffnet werden!";
			cout << endl;
			return -1;
		}

	string land, stadt;

	string antwort;

	dat_ein >> anzahl;

	srand( (unsigned)time( NULL) );
	datensatz = rand() % anzahl + 1;

	loop = 2 * datensatz;

	while(loop >= 2)
	{
	dat_ein >> land >> stadt;
	loop -= 2;
	}

	cout << "Wie heisst die Hauptstadt von " << land << "? ";
	cin >> antwort;
	cout << endl << endl;

	if(antwort == stadt)
	{
		cout << "\tRichtig!" << endl << endl;
		richtig++;

	}
	else
	{
		cout << "\tFALSCH!" << endl << "Die richtige Antwort lautet " << stadt << "!" << endl << endl << endl;
		falsch++;
	}
if(zaehler == anzahl)
{
cout << "Das Quiz endet hier!" << endl;
cout << "Du hast " << richtig << " Fragen Richtig und " << falsch << " Falsch beantwortet!" << endl;
cout << "Eingabetaste um Programm zu beenden";
fflush(stdin);
getchar();
}

dat_ein.close();

}
return 0;
}

Zusätzlich habe ich noch eine "Statistik" mit den Richtigen und Falschen antworten eingebaut.

Ohne deine Denkanstöße währe ich wohl nicht drauf gekommen danke isi.

PS:
Keine Ahnung wie umfangreich das ist aber ich werde versuchen das das Programm bereits gestellte Fragen ausklammert damit sie in einem Durchlauf nicht wiederholt werden.

Ausserdem währe eine art "highscore" noch interessant für mich!

Vielleicht hat ja jemand ein paar Denkhilfen für mich

MFG Fusle

EDIT:

Alles klar ich hab gleich mal etwas weiter Probiert um rauszukriegen wie ich verhindern kann das dass Programm mehrmals die selbe Frage stellt.

Meine idee war eigentlich die richtigen antworten in einem Array zu speichern um dann beim nächsten Durchlauf der for Schleife, vor der Ausgabe der Frage die antworten abzugleichen und evtl die schleife wieder von vorn laufen zu lassen.

Nun weiss ich nichtmal ansatzweise wie ich strings in einem Array speichern soll.
Ich weiss nichtmal ob ich mit dem Gedanken auf dem richtigen weg bin.

Über Google hab ich was von Strings in char Arrays gelesen nur leider steig ich da nicht ganz durch. Sieht mir alles sehr fortgeschritten aus.

Vielleicht könnt ihr mir da helfen.

Danke schonmal!

MFG Fusle

PS: Ich habe das jetzt in meiner Laiensprache gehalten da ich mit den begriffen Deklarieren Initialisieren usw noch nicht ganz sicher bin und da nur sachen durcheinanderbringen würde!

mfg Datalos

//Einlesen
vector<pair<string,string> > laender;
ifstream dat_ein("daten.txt");
do
{
  string land,hstadt;
  dat_ein>>land>>hstadt;
  if(dat_ein) laender.push_back(make_pair(land,hstadt));
}
while(dat_ein);

//verwenden
datensatz = rand()%lander.size();
cout<<"Wie heißt die Hauptstadt von "<<laender[datensatz].first<<"?\n";
cin>>antwort;
if(antwort==laender[datensatz].second)
  ...

Noch einige Anmerkungen:
srand() mußt du nicht in die Schleife packen - das muß nur einmal beim Start aufgerufen werden
fflush(stdin) ist böse - nimm lieber cin.ignore(numeric_limits<streamsize>::max(),'\n');