Teilprogramm kann nur einmal pro Programmstart ausgeführt werden

[JoeJoke]

@manni66 sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Wie oft wird das ausgeführt?

Ich gehe jetzt davon aus, dass int correct = 0; nur einmal ausgeführt wird.
Hab jetzt zum testen mal ganz am anfang der while schleife mal

std::cout << correct;
		system("pause");

gesetzt. Beim ersten mal ausführen wird mir der Wert korrekt mit 0 ausgegeben, beim 2. mal ausführen tritt das oben beschriebene Problem auf. Gehe also jetzt mal davon aus, dass die Schleife beim 2. mal gar nicht erst ausgeführt wird sondern ignoriert wird aus irgendeinem Grund.

@manni66 sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Warum benutzt du globale Variablen?

Du meinst du globalen Variablen im Menü, also Gen(); und AddKey();? Die verwende ich, da ich den Zufallsgen wie auch den Programmteil zum Key hinzufügen in eine extra .cpp ausgelagert habe.

@manni66 sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Warum benutz du char Arrays statt std;;string?

Die Char Arrays nutze ich, da bei nem std::string keiner Leerzeichen übernommen wurden. Wenn ich zu dem Key auch den Namen mit einbinden will ist es ohne Leerzeichen bissel blöd zu lösen, gerade da dann die nächste Abfrage direkt übersprungen wird. Durch die Char Arrays kriege ich auch die Leerzeichen in die Variable.

[Swordfish]

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Die Char Arrays nutze ich, da bei nem std::string keiner Leerzeichen übernommen wurden.

std::getline(). operator>>() liest nur bis zum nächsten Whitespace.

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Du meinst du globalen Variablen im Menü, also Gen(); und AddKey();? Die verwende ich, da ich den Zufallsgen wie auch den Programmteil zum Key hinzufügen in eine extra .cpp ausgelagert habe.

Lokale Variablen und als Parameter übergeben. Globale Variablen ohne wirklich wirklich guten Grund sind Gaga.

[JoeJoke]

@Swordfish sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

std::getline(). operator>>() liest nur bis zum nächsten Whitespace.

Bisher hat es bei mir mit dem einlesen der Wörter mit mehreren Leerzeichen wunderbar funktioniert. Bloß das 2. mal aufrufen des Codes macht probleme.

@Swordfish sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Lokale Variablen und als Parameter übergeben. Globale Variablen ohne wirklich wirklich guten Grund sind Gaga

Dann werd ich das ganze nochmal überarbeiten bei Zeiten. Wie schon erwähnt bin ich ziemlich neu in der Materie und das war so jetzt erstmal die einfachste Lösung die ich jetzt gefunden habe um alles auf verschiedene Dateien auszulagern.

[Swordfish]

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Bisher hat es bei mir mit dem einlesen der Wörter mit mehreren Leerzeichen wunderbar funktioniert.

Maybe. Nimm std::getline() und std::string.

[hustbaer]

@JoeJoke
Schnapp dir einfach deine globalen Variablen und verschiebe sie in die Funktionen rein. Mehr sollte nicht nötig sein. Du verwendest die Variablen ja eh nur lokal. Der einzige effektive Unterschied wird dabei sein dass correct dann bei jeder Ausführung von AddKey neu mit 0 initialisiert wird. Und genau dieser Unterschied behebt deinen Bug.

Um die restlichen Dinge kannst du dich später kümmern.

[hustbaer]

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Ich gehe jetzt davon aus, dass int correct = 0; nur einmal ausgeführt wird.

Korrekt

Hab jetzt zum testen mal ganz am anfang der while schleife mal

std::cout << correct;
		system("pause");

gesetzt. Beim ersten mal ausführen wird mir der Wert korrekt mit 0 ausgegeben, beim 2. mal ausführen tritt das oben beschriebene Problem auf. Gehe also jetzt mal davon aus, dass die Schleife beim 2. mal gar nicht erst ausgeführt wird sondern ignoriert wird aus irgendeinem Grund.

Na was meinst du was der Grund sein könnte?
Pack das

std::cout << correct;
		system("pause");

mal vor die while Schleife, dann siehst du den Grund.

[JoeJoke]

@hustbaer sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

globalen Variablen und verschiebe sie in die Funktionen rein

Also nur das ich das jetzt grad richtig verstehe. Du meinst den Code zurück in die Haupt .cpp Datei zurück schieben?

@hustbaer sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

mal vor die while Schleife, dann siehst du den Grund.

Erwischt. Ja, dann ist jetzt klar wo der Bug war. Jetzt funktioniert das sogar mit dem aktuellen setup

void AddKey()
{
	int correct = 0;
	char GameName[256]; 
	char NewKey[256];
	
	std::cout << correct;
	system("pause");

	while (correct != 1)

War n schlauer Denkfehler meinerseits.

[Swordfish]

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

Also nur das ich das jetzt grad richtig verstehe. Du meinst den Code zurück in die Haupt .cpp Datei zurück schieben?

Pack' bitte die globalen Variablen in die Funktionen rein in denen sie verwendet werden. So nah wie möglich dort deklarieren wo sie verwendet werden. Und nimm um himmelswillen bitte keine char-Arrays sondern std::string. Danke.

[JoeJoke]

@Swordfish std::string ist geändert.

[hustbaer]

@JoeJoke sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

@hustbaer sagte in Teilprogramm kann nur einmal pro Programmstart ausgeführt werden:

globalen Variablen und verschiebe sie in die Funktionen rein

Also nur das ich das jetzt grad richtig verstehe. Du meinst den Code zurück in die Haupt .cpp Datei zurück schieben?

Nein, ich meine genau das was du hier gemacht hast:

void AddKey()
{
	int correct = 0;
	char GameName[256]; 
	char NewKey[256];