Ich habe von meinem Lehrer folgende Aufgabe bekommen:
`

Schreiben sie ein Programm, bei dem sie eine zufällig erzeugte Zahl zwischen 1 und 1000 erraten sollen. Die Zufallszahl wird mit der rand() – Funktion im Hauptprogramm erzeugt. Anschließend wird der Spieler solange zur Eingabe einer Zahl aufgefordert, bis er die Zahl erraten hat. Die Anzahl der Rateversuche wird mitgezählt, wurde die Zahl mit weniger als 10 Versuchen erraten geben sie aus:

Weniger als 10 Versuche, Glueck gehabt

Wurden genau 10 Versuche benötigt:

Prima, 10 Versuche, sie sind ueberlegt an das Problem herangegangen

Wurden mehr als 10 Versuche benötigt:

Mehr als 10 Versuche, sie haben mehr oder weniger im Nebel gestochert

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.

Die Definition des Enumtyps sieht folgendermaßen aus:

enum ergebnis {RICHTIG, ZU_GROSS, ZU_KLEIN};

Beispiel für die erzeugte Bildschirmausgabe:

Versuch 1: Geratene Zahl zwischen 1.. 1000 eingeben: 500

Die geratene Zahl ist zu gross

Versuch 2: Geratene Zahl zwischen 1.. 1000 eingeben: 250

Die geratene Zahl ist zu klein

Versuch 3: Geratene Zahl zwischen 1.. 1000 eingeben: 375

Die geratene Zahl ist zu klein

Versuch 4: Geratene Zahl zwischen 1.. 1000 eingeben: 437

Die geratene Zahl ist zu gross

Versuch 5: Geratene Zahl zwischen 1.. 1000 eingeben: 407

Die geratene Zahl ist zu gross

Versuch 6: Geratene Zahl zwischen 1.. 1000 eingeben: 391

Die geratene Zahl ist zu klein

Versuch 7: Geratene Zahl zwischen 1.. 1000 eingeben: 399

Weniger als 10 Versuche, Glueck gehabt

Press any key to continue

`

Dazu habe ich folgenden Code aufgesetzt:
In der Header-Datei zufallszahl.h habe ich folgendes geschrieben:

#include <iostream>
using namespace std;

int zufallszahl(int min, int max);

Im Main habe ich geschrieben:

#include "zufallszahl.h"
#include <time.h>													//Header-Datei für time bei srand
#include <stdlib.h>													//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;
					}

				else (einlesen==zahl);
				{
					if(versuche<10)
					{
						cout<<"Weniger als 10 Versuche, Glueck gehabt/n";
						cout<<"Anzahl der Versuche:/n/n"<<versuche<<endl;
					}
					else 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;
}

Bei mir ist jedoch die Zeile

#include <zufallszahl.h

rot unterringelt mit der Meldung

Error: Die Datei "Quelle" kann nicht geöffnet werden: "zufallszahl.h".

Wo ist mein Denkfehler?

Danke für die Antworten im Voraus.

Gruß

Max

also bei mir im vs2012 sind 2 fehler, beide in der allerersten zeile deines codes. und beide locker zu beheben

so, vom prinzip her klappt es dann, wenn diese beiden fehler ausgemerzt sind. aber guck doch mal in deine strings was bedeutet "/n" und "\n" ?

achja, an deinen lehrer: wenn man binär an das lösen von der zufallszahl rangeht (also per binäre suche), schafft man das in unter 10 versuchen (denn 2^10 = 1024 > 1000)

was ist der unterschied zwischen #include <...> und #include "..." ?

also bei mir im vs2012 sind 2 fehler, beide in der allerersten zeile deines codes. und beide locker zu beheben

Sorry, war wir wirklich ein Leichtsinnsfehler *Kopf gegen die Wand schlag*

Jetzt funktioniert jedoch die Ausgabe nicht so, wie ich mir das vorstelle.

Jetzt kommt bei jeder eingegebenen Zahl die Meldung "Sie haben weniger als 10 Versuche gebraucht.Glück gehabt."

Edit: Ich editiere einmal den Code richtig.

zufallszahl.h

geschrieben?

else (einlesen==zahl);

Du wolltest Dir doch eigentlich bei jeder Zeile Gedanken darüber machen, wozu Du sie schreibst?
Sieht aber gar nicht danach aus ...

Jetzt kommt bei jeder eingegebenen Zahl die Meldung "Sie haben weniger als 10 Versuche gebraucht.Glück gehabt."

ist mir auch grad aufgefallen, und das sieht man auf den ersten blick nicht. man muss schon genau gucken -> zeile 37

Ja, schlag mal ordentlich vor die Wand.
Und dann bitte beantworte:
Warum hast Du überhaupt eine

zufallszahl.h

geschrieben?

Mein Lehrer sagt immer, dass Funktionsaufrufe/Funktionsprototypen in eine Headerdatei gehören

Edit:

#include <stdlib.h> //Header-Datei für rand und srand

Aber du brauchst die Datei quasi eh nicht, da du diese Funktion weder geschrieben noch genutzt hast...

edit: merke: jegliche Standard Header die eine "xXx.h" Endung haben, machst du zu "cxXx"

in deinem fall also:

#include <cstdlib>
#include <ctime>
// etc.

Und wenn du so gut bist, dass du gleich die Zahl geraten hast, passiert gar nichts.

Mein Lehrer sagt immer, dass Funktionsaufrufe/Funktionsprototypen in eine Headerdatei gehören

Das ist dann sinnvoll, wenn die Definition der Funktion (der Code) in einer andern Datei (.cpp oder .c) steht.

Man sollte auch nur die Header einbinden, die man wirklich in dem Code braucht.
Gibt es eine zufallszahl.c oder zufallszahl.cpp?
Wenn ja brauchst du da iostream?
Hast du diese auch im Projekt eingebunden?

Finde den Fehler:

else (einlesen==zahl);

Wenn ich das Semmikolon weglassen, dann bekomme ich folgende Fehlermeldung:

error C2143: Syntaxfehler: Es fehlt ';' vor '{'

Hast du diese auch im Projekt eingebunden?

Ja, habe ich.

Da sind noch 16 andere (in der Zeile).

Shade Of Mine schrieb:

Finde den Fehler:

else (einlesen==zahl);

Wenn ich das Semmikolon weglassen, dann bekomme ich folgende Fehlermeldung:

error C2143: Syntaxfehler: Es fehlt ';' vor '{'

und warum?

vergleiche diese zeilen

if( einlesen<zahl) {}
else if (einlesen>zahl) {}
else (einlesen==zahl) {} // ohne semikolon hitner den runden klammern fehler!

Belli schrieb:

Ja, schlag mal ordentlich vor die Wand.
Und dann bitte beantworte:
Warum hast Du überhaupt eine

zufallszahl.h

geschrieben?

Mein Lehrer sagt immer, dass Funktionsaufrufe/Funktionsprototypen in eine Headerdatei gehören

Du hast aber weder eine Funktion noch einen Aufruf einer Funktion namens zufallszahl, die Du da in der Header-Datei deklarierst.

Hier einmal der richtige Code...

#include "zufallszahl.h"
#include <time.h>													//Header-Datei für time bei srand
#include <stdlib.h>													//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;
					}

				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;

					}
				}
		}
		return 0;
}

#include "zufallszahl.h"

weg und nimm bitte statt time.h ctime und statt stdlib.h cstdlib.
Und dann lösch Deine zufallzahl.h - Headerdatei wieder.

zahl = 2;

Und dann gibst du auch gleich 2 ein.

Schreib mal in Zeile 16:

zahl = 2;

Und dann gibst du auch gleich 2 ein.

Also bei mir bleibt dann die Ausgabe leer... es erscheint gleich die Meldung "Drücken Sie eine beliebege Taste..."

Also bei mir bleibt dann die Ausgabe leer... es erscheint gleich die Meldung "Drücken Sie eine beliebege Taste..."

Ist das wirklich das gewünschte Verhalten?

Nie wieder arbeite ich bei mehreren if-Anweisungen mit else ...

Und jetzt bau das else wieder ein und zwar korrekt.

silent_max schrieb:

Nie wieder arbeite ich bei mehreren if-Anweisungen mit else ...

Und jetzt bau das else wieder ein und zwar korrekt.

Quatsch. Das braucht er nicht.

Shade Of Mine schrieb:

silent_max schrieb:

Nie wieder arbeite ich bei mehreren if-Anweisungen mit else ...

Und jetzt bau das else wieder ein und zwar korrekt.

Quatsch. Das braucht er nicht.

sollte er aber. Und sei es nur zu Übungszwecken - irgendwann wird er es brauchen.

Shade Of Mine schrieb:

silent_max schrieb:

Nie wieder arbeite ich bei mehreren if-Anweisungen mit else ...

Und jetzt bau das else wieder ein und zwar korrekt.

Quatsch. Das braucht er nicht.

Shade Of Mine ... bitte verstehe mich nicht falsch, aber wozu brauche ich das else, wenn es mit nur if - Anweisungen klappt??

Shade Of Mine schrieb:

silent_max schrieb:

Nie wieder arbeite ich bei mehreren if-Anweisungen mit else ...

Und jetzt bau das else wieder ein und zwar korrekt.

Quatsch. Das braucht er nicht.

Doch, das braucht er. Wann soll er es sonst lernen?