Anfänger Problem C++

[uebertreiber]

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:


Meine Cpp Datei:


und meine Header Datei:


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

[Schlüsselbein]

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.

[Nightblizard]

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.

[xxfabbelxx]

system( "pause > nul" );

Damit pausiert man die console...

[Schlüsselbein]

Damit pausiert man die console...

***, und zwar genauso schlecht wie unser Threadstarter.

[Raz9r]

Anstatt einfach das Programm über CTRL+F5 (ohne Debugging) zu starten und kein Pausieren irgendeiner Art zu brauchen...

[.SkyneT.]

Originally Posted by xxfabbelxx

system( "pause > nul" );

Damit pausiert man die console...

Damit baut man eine 3 Sekündige Verzögerung ein:

system("ping -w 3 999.999.999.999 > nul");

Ne im Ernst jetzt:

_getch()
oder
std::cin.get() //das hier wäre übrigens die C++ Variante

[Delinquenz]

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.

[TIMΣ™]

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

void rechnung ();

auf, sondern mit

rechnung();

Aber wenn du das Problem genauer schildern könntest, währe das ziemlich hilfreich.

Außerdem ist dieser Part auch nicht richtig durchdacht.

else { if(zahl1 < zahl2) {...} }

Da reicht einfach nur

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

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

cin.get();

Ist schöner und schneller als das

system("pause");

Außerdem würde ich kein

using namespace std;

benutzen, sondern die std:: einfach rausschreiben.

[MrSm!th]

Originally Posted by TIMΣ™

Außerdem ist dieser Part auch nicht richtig durchdacht.

else { if(zahl1 < zahl2) {...} }

Da reicht einfach nur

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.

[TIMΣ™]

Originally Posted by MrSm!th

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

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