Fehler bei c++ ratespiel

[hansewurst]

Hey lieber programmierer, habe versucht ein ratespiel versucht in c++ zu schreiben, scheitere aber daran das die zufallszahl 44 ist, ich die zahl 5 eingegeben hab und trozdem "Richtig!!" erscheind. Hier mal mein source:

#include <iostream>
#include <cstdlib>
#include <string>
#include <ctime>

using namespace std;

int main(void){
	int zahl;
	int wert = 0;
	int runden;
	int zufallszahl;
	int max = 100;
	cout<<"Wie oft willst du spielen?:\n";
	cin>>runden;
	srand((unsigned int )time(NULL));
		zufallszahl = rand() % 100 + 1;

	while(wert<runden){
		cout<<"Geb eine Zahl ein:\n";
		cin>>zahl;
		wert++;
		if(zahl=zufallszahl){
			cout<<"Richtig!!\n";
		}
		else{
			cout<<"Falsch!!\n";
		}

	
		
		
		
		cout<<zufallszahl<<"\n";
	
	

	}
	system("pause");
	return 0;






}

[Schlüsselbein]

if(zahl=zufallszahl){

Nochmal genau anschauen.

Btw. ist der Code nicht gerade ästhetisch.

[.SkyneT.]

Originally Posted by hansewurst

	system("pause");

Weg damit!
std::cin.get() verwenden.

using namespace std;

Kein globales using namespace.
Statdessen könntest du folgendes schreiben.

using std::cout;
using std::cin;
using std::endl;
//...

Die Lösung zum eigentlichen Problem wird dir hoffentlich
durch Schlüsselbeins Post noch auffallen, falls
dies noch nicht geschehen ist.

[hansewurst]

anfänger fehler aber danke

[TIMΣ™]

Originally Posted by Schlüsselbein

if(zahl=zufallszahl){

Nochmal genau anschauen.

Btw. ist der Code nicht gerade ästhetisch.

Es ist nicht jeder ein super professioneller Programmierer. Glaube kaum, dass es bei dir anfangs anders war. Man fängt immer klein an und arbeitet sich mit der Zeit in neue Funktionen, Header etc. ein.

[Schlüsselbein]

Es ist nicht jeder ein super professioneller Programmierer. Glaube kaum, dass es bei dir anfangs anders war. Man fängt immer klein an und arbeitet sich mit der Zeit in neue Funktionen, Header etc. ein.

Aha, und was willst du mir damit jetzt sagen?
Dass man Anfänger nicht darauf hinweisen soll, wenn etwas nicht stimmt?

[TIMΣ™]

Originally Posted by Schlüsselbein

Aha, und was willst du mir damit jetzt sagen?
Dass man Anfänger nicht darauf hinweisen soll, wenn etwas nicht stimmt?

Nö, ich will damit einfach nur sagen, dass du nicht gleich bei jedem "unschönen" Code noch einen expliziten Kommentar darunter schreiben musst. Mein Code dazu sähe z.B. auch nicht viel besser aus . Wie gesagt, man arbeitet sich mit der Zeit ein und dann kann man seinen Code auch noch entsprechend optimieren. Für den Anfang ist das doch nicht wichtig.

Was bringt ihm das den, wenn du zu ihm meinst, dass sein Code von der Struktur her besser sein könnte?

Richtig, garnichts, da er denke mal nicht weiß wie man das ganze ohne gleich 200.000 Zeilen an Code zu schreiben verbessern könnte. Sonst hätte er denke mal nochmal darüber geschaut.

Naja, nur mal so am Rande.

[[uLow]NTX?!]

Originally Posted by Schlüsselbein

Aha, und was willst du mir damit jetzt sagen?
Dass man Anfänger nicht darauf hinweisen soll, wenn etwas nicht stimmt?

Nein das denke ich nicht, er will einfach sagen, dass es evtl ein bissen hart geklungen hat..

Ich meine das kennt doch jeder... man coded rum denkt alles läuft, der compiler meckert evtl auch nicht aber es geht nicht...

dann schaut ein anderer drüber und findet den fehler schnell.


Ist doch normal.. (jedenfalls bei mir)

[.SkyneT.]

Originally Posted by NTX?!

Nein das denke ich nicht, er will einfach sagen, dass es evtl ein bissen hart geklungen hat..

"Btw. ist der Code nicht gerade ästhetisch."
Kling für mich nicht grade "hart".
Sowas wie "Schreib den Müll erst mal neu, der Stil ist grässlich",
wäre eher in die Richtung "hart".

Außerdem war es auch nur "by the way".

[TIMΣ™]

Originally Posted by .SkyneT.

"Btw. ist der Code nicht gerade ästhetisch."
Kling für mich nicht grade "hart".
Sowas wie "Schreib den Müll erst mal neu, der Stil ist grässlich",
wäre eher in die Richtung "hart".

Außerdem war es auch nur "by the way".

Hmm..., haste auch irgendwie recht. Nur mal so eine Frage am Rande: Nutzt du eigentlich BackTrack? Denn du hast ja das Logo und diesen Spruch dazu in deiner Sigi.

[.SkyneT.]

Originally Posted by TIMΣ™

Hmm..., haste auch irgendwie recht. Nur mal so eine Frage am Rande: Nutzt du eigentlich BackTrack? Denn du hast ja das Logo und diesen Spruch dazu in deiner Sigi.

Ja, ist aber schon ne Zeit lang her, das ich es das letzte Mal verwendet habe.
Und da ich nicht grade der Gfx-Künstler bin, lass ich das einfach drin