Anfänger Problem C++

06/11/2013 10:00 uebertreiber#1
hi probiere mich in c++ aus und hab so anfänger probleme,
unzwar komme ich hier bei zu keiner aussage, kann mir da jemand sagen was ich da machen muss ? :S

Meine Main:
[Only registered and activated users can see links. Click Here To Register...]

Meine Cpp Datei:
[Only registered and activated users can see links. Click Here To Register...]

und meine Header Datei:
[Only registered and activated users can see links. Click Here To Register...]

ups auf dem foto von der main fehlt oben das #include "ubungNrEins.h" das ist mit drin aber leider nicht auf dem foto drauf sry :S
06/11/2013 10:11 Schlüsselbein#2
Was soll der Bullshit? Poste den Fehler und die Zeile(n), in denen der Fehler vorkommt.

Btw. lass das using namespace std (vor allem so wie du es hier machst!) und system("pause") weg.
06/11/2013 10:14 Nightblizard#3
Abgesehen von Schlüsselbeins Punkten (die du wirklich beherzigen solltest):
Was versuchst du mit Zeile 7 in Main.cpp zu erreichen? Schau dir in deinem Buch nochmal das Thema Funktionen an. Speziell wie man sie definiert, deklariert und aufrufen tut.
06/11/2013 21:47 xxfabbelxx#4
Code:
system( "pause > nul" );
Damit pausiert man die console...
06/11/2013 21:56 Schlüsselbein#5
Quote:
Damit pausiert man die console...
Jap, und zwar genauso schlecht wie unser Threadstarter.
06/11/2013 22:40 Raz9r#6
Anstatt einfach das Programm über CTRL+F5 (ohne Debugging) zu starten und kein Pausieren irgendeiner Art zu brauchen...
06/11/2013 22:56 .SkyneT.#7
Quote:
Originally Posted by xxfabbelxx View Post
Code:
system( "pause > nul" );
Damit pausiert man die console...
Damit baut man eine 3 Sekündige Verzögerung ein:
Code:
system("ping -w 3 999.999.999.999 > nul");
Ne im Ernst jetzt:
Code:
_getch()
oder
std::cin.get() //das hier wäre übrigens die C++ Variante
06/12/2013 05:26 Delinquenz#8
Man ruft eine Funktion ohne Angabe des Return-Typs auf. Der wird ja bereits in der Funktionsdeklaration/definition definiert.

Achja, beim nächsten mal bitte Errors posten. "using namespace" hat _nichts_ in einem Header zu suchen.
06/13/2013 14:32 TIMΣ™#9
Ich verstehe ehrlich gesagt nicht, wieso du nicht einfach alles in eine Datei schreibst und dir diesen Header sparst.

Außerdem rufst du die Funktion rechnung nicht mit

Code:
void rechnung ();
auf, sondern mit

Code:
rechnung();
Aber wenn du das Problem genauer schildern könntest, währe das ziemlich hilfreich.

Außerdem ist dieser Part auch nicht richtig durchdacht.

Code:
else { if(zahl1 < zahl2) {...} }
Da reicht einfach nur

Code:
else { ... }
Du brauchst garkeine zweite If-Abfrage, denn wenn die erste Bedingung nicht erfüllt ist, muss das 'else' sowieso eintreten.

D.h. Wenn die erste Zahl kleiner ist wird das 'else' ausgeführt. (Deswegen brauchst du auch nicht extra nochmal Abfragen).

Und das

Code:
int erg = 0;
kannst die genau so sparen, bzw. wofür ist das da?

Geht der Code bei dir weiter, bzw. benutzt du den Integer? Ansonsten einfach streichen. Kann ich jetzt nicht genau beurteilen.

Zum pausieren der Kosole einfach

Code:
cin.get();
Ist schöner und schneller als das

Code:
system("pause");
Außerdem würde ich kein
Code:
using namespace std;
benutzen, sondern die std:: einfach rausschreiben.
06/13/2013 17:00 MrSm!th#10
Quote:
Originally Posted by TIMΣ™ View Post
Außerdem ist dieser Part auch nicht richtig durchdacht.

Code:
else { if(zahl1 < zahl2) {...} }
Da reicht einfach nur

Code:
else { ... }
Du brauchst garkeine zweite If-Abfrage, denn wenn die erste Bedingung nicht erfüllt ist, muss das 'else' sowieso eintreten.

D.h. Wenn die erste Zahl kleiner ist wird das 'else' ausgeführt. (Deswegen brauchst du auch nicht extra nochmal Abfragen).
Guck nochmal genau nach, das Verhalten verändert sich mit deinem Vorschlag und zwar zum Falschen. Allerdings könnte man aus dem else { if() {}} auch einfach ein else if() {} machen.
06/13/2013 18:40 TIMΣ™#11
Quote:
Originally Posted by MrSm!th View Post
Guck nochmal genau nach, das Verhalten verändert sich mit deinem Vorschlag und zwar zum Falschen. Allerdings könnte man aus dem else { if() {}} auch einfach ein else if() {} machen.
Ach hast recht, jetzt sehe ich es auch :o

Also dann lieber wohl else if() {} benutzen, sry.