Fehler bei c++ ratespiel

07/18/2013 16:44 hansewurst#1
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:
Code:
#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;






}
07/18/2013 16:49 Schlüsselbein#2
Code:
if(zahl=zufallszahl){
Nochmal genau anschauen.

Btw. ist der Code nicht gerade ästhetisch.
07/18/2013 17:19 .SkyneT.#3
Quote:
Originally Posted by hansewurst View Post
Code:
	system("pause");
Weg damit!
std::cin.get() verwenden.

Code:
using namespace std;
Kein globales using namespace.
Statdessen könntest du folgendes schreiben.
Code:
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. ;)
07/18/2013 17:49 hansewurst#4
anfänger fehler :D aber danke :)
07/18/2013 18:24 TIMΣ™#5
Quote:
Originally Posted by Schlüsselbein View Post
Code:
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.
07/18/2013 18:25 Schlüsselbein#6
Quote:
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?
07/18/2013 21:24 TIMΣ™#7
Quote:
Originally Posted by Schlüsselbein View Post
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 :p. 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.
07/18/2013 21:25 [uLow]NTX?!#8
Quote:
Originally Posted by Schlüsselbein View Post
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)
07/18/2013 21:31 .SkyneT.#9
Quote:
Originally Posted by NTX?! View Post
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".
07/18/2013 22:13 TIMΣ™#10
Quote:
Originally Posted by .SkyneT. View Post
"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.
07/18/2013 22:21 .SkyneT.#11
Quote:
Originally Posted by TIMΣ™ View Post
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 ;)