Erraten der Zufallszahl funktioniert nicht

[DirkB]

[quote="silent_max"]Ja, habe ich jetzt schon zig Male ausprobiert...

Auch wenn du beim ersten mal die richtige Zahl eingibst?
Was hast du geändert?

[quote="silent_max"]JDas soll der Lehrer entscheiden...

Mit einer 4 hat man bestanden. Bestanden ist gut. Gut ist 2.

[Michael E.]

silent_max schrieb:

daddy_felix schrieb:

Funktioniert dein Programm denn jetzt so, wie es soll?

Ja, habe ich jetzt schon zig Male ausprobiert...

Es wurde dir ein Fall genannt, in dem dein Programm nicht funktioniert, nämlich wenn beim ersten Versuch richtig geraten wird.

Was ist mit dieser Anforderung:

Die Überprüfung der Zufallszahl soll in einem Unterprogramm erfolgen. Das Unterprogramm liefert einen Enumwert zurück, der angibt ob die Zahl RICHTIG, ZU_GROSS oder ZU_KLEIN war.

Das soll der Lehrer entscheiden...[/quote]
Ich dachte, du willst etwas lernen. Du hast doch in den letzten Tagen in Threads geschrieben, wie fleißig und hartnäckig du lernst. Und jetzt ignorierst du einfach Teile der Aufgabenstellung? Das passt nicht zusammen.

Auf die falschen Includes bist du bereits mehrfach hingewiesen worden. Bitte noch einmal nachlesen. Außerdem sind deine Kommentare falsch. rand() erzeugt keine Zufallszahl zwischen 2^0 und 2^15, sondern eher zwischen 2^0 - 1 und 2^15 - 1. der %-Operator ist nicht dazu da, Kommazahlen zu vermeiden, bitte auch hier noch einmal die Bedeutung nachschlagen.

[Skym0sh0]

Achtung Spoiler

[silent_max]

[quote="DirkB"][quote="silent_max"]Ja, habe ich jetzt schon zig Male ausprobiert...

Auch wenn du beim ersten mal die richtige Zahl eingibst?
Was hast du geändert?

Also ich habe in einer zweiten CPP Datei "Überprüfen" folgenden Code geschrieben:

#include <iostream>
using namespace std;

int main()
{
	int einlesen, zahl;
	enum {RICHTIG, ZU_GROSS, ZU_KLEIN};

	if(einlesen<zahl)
	{
		cout << "ZU_KLEIN";
	}

	if(einlesen == zahl)
	{
		cout << "RICHTIG";
	}

	if(einlesen > zahl)
	{
		cout << "ZU_GROSS";
	}

	return 0;
}

Nun hat sich bei mir die Frage eingeschlichen, wie ich diesen Code in die bereits bestehende CPP Datei mit einbinde??

[Belli]

copy & paste

[silent_max]

Hier einmal der gesamte Quellcode:

#include <ctime>													//Header-Datei für time bei srand
#include <cstdlib>													//Header-Datei für rand und srand
#include <iostream>													//Header-Datei für "cin" und "cout"
using namespace std;

int main(void)
	{
		int zahl, einlesen;											//Variablen "Zahl" und "Einlesen" als Integer(da Zahlen) deklarieren
		int versuche;												//Versuche ist auch Integer da die Versuche gezählt werden

		srand((unsigned)time(NULL));
		zahl=rand();												//Zahl über Zufallsgenerator erzeugt
		zahl=zahl%1000+1;											//"rand() erzeugt eine Zufallszahl zwischen 2^0 und 2^15, deswegen 
																	//%1000+1; %=Modulo -> keine Kommazahlen; 0 bis 1000

		cout<<"Zufallszahl zwischen 1 und 1000 eingeben";
		cin>>einlesen;

		for(versuche=0; zahl!=einlesen; versuche++)					//Versuche muss mit =0 initialisiert werden
		{
				if( einlesen<zahl)
					{
						cout<<"Eingebene Zahl ist zu klein!\n";
						cout<<"Erneute eingabe\n";
						cin>> einlesen;
					}

				else if (einlesen>zahl)
					{
						cout<<"Eingebene Zahl ist zu gross!\n";
						cout<<"Erneute eingabe!\n";
						cin>> einlesen;
					}

				if(einlesen==zahl)
				{
					if(versuche<10)
					{
						cout<<"Weniger als 10 Versuche, Glueck gehabt\n";
						cout<<"Anzahl der Versuche:\n\n"<<versuche<<endl;
					}
					if(versuche==10)
					{

							cout<<"Prima, 10 Versuche, sie sind ueberlegt an das Problem herangegangen\n";
					}		cout<<"Anzahl der Versuche:\n\n"<<versuche<<endl;

					if(versuche>10)
					{
						cout<<"Mehr als 10 Versuche, sie haben mehr oder weniger im Nebel gestochert\n";
						cout<<"Anzahl der Versuche:\n"<<versuche<<endl;

					}
				}
		}
    enum {RICHTIG, ZU_GROSS, ZU_KLEIN};

    if(einlesen<zahl)
    {
        cout << "ZU_KLEIN";
    }

    if(einlesen == zahl)
    {
        cout << "RICHTIG";
    }

    if(einlesen > zahl)
    {
        cout << "ZU_GROSS";
    }
		return 0;
}

Er funktioniert zumindest bei mir noch nicht ganz... Ich weiß nicht, wie ich "ZU_GROSS", "ZU_KLEIN" und "RICHTIG" einbauen soll, damit das eingebungen wird...

Sollte ich dazu eine Funktion schreiben??

[DirkB]

silent_max schrieb:

Er funktioniert zumindest bei mir...

Das hat mit der Aufgabenstellung nichts zu tun.

Du hast vergessen dein Hirn zu benutzen.

[Skym0sh0]

noch nie mit Enumerationen gearbeitet?

weil so ist die nutzung falsch.

hat da nicht jemand en guten link?

[silent_max]

DirkB schrieb:

silent_max schrieb:

Er funktioniert zumindest bei mir...

Das hat mit der Aufgabenstellung nichts zu tun.

Du hast vergessen dein Hirn zu benutzen.

Ich habe nur den Quellcode gepostet, bevor ich mein Hirn eingeschaltet habe.

Ich hoffe, dass folgender Code jetzt passt:

#include <ctime>													//Header-Datei für time bei srand
#include <cstdlib>													//Header-Datei für rand und srand
#include <iostream>													//Header-Datei für "cin" und "cout"
using namespace std;

enum {RICHTIG, ZU_GROSS, ZU_KLEIN};

int main(void)
	{
		int zahl, einlesen;											//Variablen "Zahl" und "Einlesen" als Integer(da Zahlen) deklarieren
		int versuche;												//Versuche ist auch Integer da die Versuche gezählt werden

		srand((unsigned)time(NULL));
		zahl=rand();												//Zahl über Zufallsgenerator erzeugt
		zahl=zahl%1000+1;											//"rand() erzeugt eine Zufallszahl zwischen 2^0 und 2^15, deswegen 
																	//%1000+1; %=Modulo -> keine Kommazahlen; 0 bis 1000

		cout<<"Zufallszahl zwischen 1 und 1000 eingeben";
		cin>>einlesen;

		for(versuche=0; zahl!=einlesen; versuche++)					//Versuche muss mit =0 initialisiert werden
		{
				if( einlesen<zahl)
					{
						cout<<"Eingebene Zahl ist zu klein!\n";
						cout << "ZU_KLEIN\n";
						cout<<"Erneute eingabe\n";
						cin>> einlesen;
					}

				else if (einlesen>zahl)
					{
						cout<<"Eingebene Zahl ist zu gross!\n";
						cout << "ZU_GROSS\n";
						cout<<"Erneute eingabe!\n";
						cin>> einlesen;
					}

				if(einlesen==zahl)
				{
					cout << "RICHTIG\n";
					if(versuche<10)
					{
						cout<<"Weniger als 10 Versuche, Glueck gehabt\n";
						cout<<"Anzahl der Versuche:\n\n"<<versuche<<endl;
					}
					if(versuche==10)
					{

							cout<<"Prima, 10 Versuche, sie sind ueberlegt an das Problem herangegangen\n";
					}		cout<<"Anzahl der Versuche:\n\n"<<versuche<<endl;

					if(versuche>10)
					{
						cout<<"Mehr als 10 Versuche, sie haben mehr oder weniger im Nebel gestochert\n";
						cout<<"Anzahl der Versuche:\n"<<versuche<<endl;

					}
				}
		}

		return 0;
}

[ScottZhang]

Was passiert wenn du Zeile 6 löschst?
Glaubst du wirklich du hast die enums benutz, oder nur dahingeschrieben?

[DirkB]

Skym0sh0 schrieb:

... hat da nicht jemand en guten link?

Ich habe da gerade einen gesehen. Dunkelblau Schrift auf hellblauen Hintergrund.

[Skym0sh0]

silent_max schrieb:

Ich hoffe, dass folgender Code jetzt passt:

Nein, ist an der Aufgabenstellung vorbei.

[silent_max]

Skym0sh0 schrieb:

Nein, ist an der Aufgabenstellung vorbei.

Wie würdest Du denn das mit dem Enum machen??

[ScottZhang]

@silent_max: Mach uns ma nen Ablaufplan (pseudocode) wie du dir dein Programm vorstellst.

[Skym0sh0]

DirkB schrieb:

Skym0sh0 schrieb:

... hat da nicht jemand en guten link?

Ich habe da gerade einen gesehen. Dunkelblau Schrift auf hellblauen Hintergrund.

Hab ich gepostet gehabt. Aber ist halt ein Spoiler, sprich eine Komplettlösung zu der Aufgabe und das sollte nicht in deinem Sinne sein.

Zumal ich eine mathematische Funktion benutzt habe um es mir ein wenig zu vereinfachen...

[DirkB]

silent_max schrieb:

Wie würdest Du denn das mit dem Enum machen??

Auf Seite 4 von diesem Thread hat Skym0sh0 einen Link gepostet unter dem Text Achtung Spoiler. (Ein Link erscheint hier in dunkelblauer Schrift)

[silent_max]

ScottZhang schrieb:

@silent_max: Mach uns ma nen Ablaufplan (pseudocode) wie du dir dein Programm vorstellst.

Ich kann nur schreiben, was ich aus der Aufgabenstellung heraus verstehe...

-> Ausgabe von "Zufallszahl zwischen 1 und 1000 eingeben"
-> Zahl einlesen
-> Falls zu klein => Ausgabe von "Die eingegebene Zahl ist zu klein! ZU_KLEIN Erneute Eingabe"
Falls zu gross => Ausgabe von "Die eingegebene Zahl ist zu gross! ZU_GROSS Erneute Eingabe"
Falls richtig => Ausgabe von " RICHTIG. Sie haben ... Versuche gebraucht!"

[DirkB]

Da steht nichts davon, dass der Text "ZU_KLEIN" oder "RICHTIG" auf dem Schirm erscheinen soll.

Du sollst eine Funktion schreiben die den Vergleich für dich erledigt.
Statt

if( einlesen<zahl)

soll dann

if( my_check(einlesen, zahl) == ZU_KLEIN) //  ohne "" um ZU_KLEIN. Das ist ein enum.
...

Du musst jetzt noch die Funktion my_check schreiben die den Vergleich macht und entsprechend die Werte zurück gibt.

[silent_max]

DirkB schrieb:

Da steht nichts davon, dass der Text "ZU_KLEIN" oder "RICHTIG" auf dem Schirm erscheinen soll.

Du sollst eine Funktion schreiben die den Vergleich für dich erledigt.
Statt

if( einlesen<zahl)

soll dann

if( my_check(einlesen, zahl) == ZU_KLEIN) //  ohne "" um ZU_KLEIN. Das ist ein enum.
...

Du musst jetzt noch die Funktion my_check schreiben die den Vergleich macht und entsprechend die Werte zurück gibt.

Ist denn mein Code so vorbei an der Aufgabe??

Also muss ich für ZU_GROSS und RICHTIG auch eine Funktion schreiben??

Edit: Da muss ich den ganzen Code umschreiben oder wie verstehe ich das??
Edit²: mycheck muss doch schon vom Typ int sein?
Edit³: Wenn ich jetzt folgenden Code hinzufüge

int my_check();
...
...
...
int my_check (int Ergebnis)
{
	if(my_check (einlesen, zahl) == ZU_KLEIN)
		cout << "Zu Klein\n";
	if(my_check (einlesen, zahl) == RICHTIG)
		cout << "Richtig\n";
	if(my_check (einlesen, zahl) == ZU_GROSS)
		cout << "Zu gross\n";
	return ergebnis;
}

dann sind my_check , einlesen , zahl , RICHTIG und Ergebnis rot unterringelt....

[DirkB]

silent_max schrieb:

Also muss ich für ZU_GROSS und RICHTIG auch eine Funktion schreiben??

Nein.
Das soll die eine Funktion erledigen

Aufgabe schrieb:

Die Überprüfung der Zufallszahl soll in einem Unterprogramm erfolgen. Das Unterprogramm liefert einen Enumwert zurück, der angibt ob die Zahl RICHTIG, ZU_GROSS oder ZU_KLEIN war.

silent_max schrieb:

Edit: Da muss ich den ganzen Code umschreiben oder wie verstehe ich das??
Edit²: mycheck muss doch schon vom Typ int sein?

Nein, musst du nicht.
Ein enum ist ein int. Da haben die Zahlen nur andere Namen.

silent_max schrieb:

Edit³: Wenn ich jetzt folgenden Code hinzufüge

int my_check();
...
...
...
int my_check (int Ergebnis)
{
	if(my_check (einlesen, zahl) == ZU_KLEIN)
		cout << "Zu Klein\n";
	if(my_check (einlesen, zahl) == RICHTIG)
		cout << "Richtig\n";
	if(my_check (einlesen, zahl) == ZU_GROSS)
		cout << "Zu gross\n";
	return ergebnis;
}

dann sind my_check , einlesen/[c],[c]zahl , RICHTIG und Ergebnis rot unterringelt....

- Der Prototyp (Zeile 1) passt nicht zur Definition (Zeile 5)
- Den Paramter Ergebnis benutzt du nicht in der Funktion.
- Du machst einen rekursiven Aufruf mit anderen Parametern (Zeile 7, 9, 11)
- Die Variablen einlesen, zahl und ergebnis sind in der Funktion nicht
definiert.

Die Zeilen 7 bis 12 gehören in dein main in deine for-Schleife (oberer Teil).
Was bisher in deiner for-Schleife war (der Vergleich auf kleiner oder größer) kommt in die Funktion.
Achte auf die Parameter.

Natürlich musst du da noch Sachen anpassen.
Nochmal mit eingeschaltetem Hirn überlegen.