C++ (Fehler in Script) || Variabelnwerte werden nicht verändert.

JaffaCoder

Professional

Posts: 211

Saturday, May 21st 2011, 5:56pm

C++ (Fehler in Script) || Variabelnwerte werden nicht verändert.

Huhu

Weiß jemand, wieso in diesem Script:
1. Die Variablen Nahrung, Gold, Erholung, LP im Startmenü verändert werden, wenn ich eine Arbeit mache, aber nicht wenn ich handel.?
2. Die IF-bedinungen ausgeführt werden wegen zu geringer Nahrung?

Suche schon seit 2 Stunden.
Vielen Danke

JaffaCoder has attached the following file:

Script.txt (16.47 kB - 35 times downloaded - latest: Apr 9th 2012, 2:48am)

uwe

Unregistered

Saturday, May 21st 2011, 7:12pm

C++ ist kein Skriptsprache und dieser Quelltext ist ja wohl ein schlechter Scherz. Die Krönung des Blödsinns ist zweifelsohne, hinter jede Klammer eine Kommentar zu setzen dass hier eine Klammer einen Block aufmacht bzw. schließt.

bcc-fan

Professional

Posts: 384

Saturday, May 21st 2011, 7:16pm

Zu deinem Quelltext: kein Wunder das du nix findest

Grauenhaft formatiert und Spaghetticode

Zum formatieren - schreibe die zusammengehörenden geschweiften Klammern untereinander - also die schliessende unter die dazu gehörige öffnende - ich weiss es gibt noch andere Formatierungen - aber die erscheint mir am übersichtlichsten.

Und geh das mit dem Spaghetticode an - gewöhn dir das gar nicht erst an so zu schreiben - handelst dir jetzt schon Ärger ein - bei grösseren Projekten wird das noch heftiger :!:

Wahrscheinlich bist du irgendwo mit grösser und kleiner durcheinander.

MfG bcc-fan

JaffaCoder

Professional

Posts: 211

Saturday, May 21st 2011, 7:16pm

Quoted from "uwe"

C++ ist kein Skriptsprache und dieser Quelltext ist ja wohl ein schlechter Scherz. Die Krönung des Blödsinns ist zweifelsohne, hinter jede Klammer eine Kommentar zu setzen dass hier eine Klammer einen Block aufmacht bzw. schließt.

Das mit des Klammern habe ich dazu während der Fehlersuche. Da am Anfang mein Compiler die Klammern als Fehler benutzt hat. Das war nur zur Orientierung, wo der Fehler ist und es hat funktioniert.

Quoted from "bcc-fan"

Zu deinem Quelltext kein Wunder das du nix findest
Grauenhaft formatiert und Spaghetticode

Zum formatieren - schreibe die zusammengehörenden geschweiften Klammern untereinander - also die schliessende unter die dazu gehörige öffnende - ich weiss es gibt noch andere Formatierungen - aber die erscheint mir am übersichtlichsten.

Und geh das mit dem Spaghetticode an - gewöhn dir das gar nicht erst an so zu schreiben - handelst dir jetzt schon Ärger ein - bei grösseren Projekten wird das noch heftiger

Wahrscheinlich bist du irgendwo mit grösser und kleiner durcheinander.

MfG bcc-fan

Zu Formatierung, okey. Werde ich

Aber was meinst du mit Spaghetticode?

bcc-fan

Professional

Posts: 384

Saturday, May 21st 2011, 7:19pm

Hat deine IDE oder dein Editor eine automatische Formatierungsmöglichkeit?
Wenn ja, nutze die.

Spaghetticode -> du packst alles in die main -> versuche den Quelltext so umzubauen das du maximal 20 Zeilen pro Funktion hast, besser wäre noch weniger als 10 Zeilen. Ausserdem die Abfragen möglichst einfach halten. Das verringert die Fehlerquote.

MfG bcc-fan

JaffaCoder

Professional

Posts: 211

Saturday, May 21st 2011, 7:23pm

Quoted from "bcc-fan"

Hat deine IDE oder dein Editor eine automatische Formatierungsmöglichkeit?
Wenn ja, nutze die.

MfG bcc-fan

Leider nicht, hab schonmal nachgesucht. Benutze den Dev-C++

Kanst du mir einen Compiler empfehlen, der soetwas besitzt?

bcc-fan

Professional

Posts: 384

Saturday, May 21st 2011, 7:49pm

Der DEV ist die IDE und der Compiler darunter ist der MinGW :!:

Nicht der Compiler ist für die Formatierung zuständig, sondern die IDE oder der Editor. Der Compiler schluckt auch sehr schwer lesbaren Quelltext solange der halbwegs richtig ist.

Versuche mal Code::Blocks 10.05 mit dem Compiler zu laden und zu testen.

Hier mal ein Codesnipsel:

C/C++ Quellcode

if(Auswahl=='b')
        {
            system("cls");
            Goldmine = 1 + rand()%(10-1+1);  // wo initialisierst du rand() ?
 
            if(Goldmine<6)
            {
                cout << "Du hast" << Goldmine << "Gold gefunden.";
                Gold=Gold+Goldmine;
            }
	    else
            {
                GMSchaden = 1 + rand()||(250-1+1);
                cout << "Dir bricht die Mine \x81ber dem Kopf zusammen, aber kannst grade noch entkommen." << endl
                     << "Du verlierst jedoch " << GMSchaden << "Lebenspunkte.";
 
                LP=LP-GMSchaden;
            }
        }