Taschenrechner

[Grapefruit97]

Hallo, ich bin ein noch ziemlicher Anfänger in C++. Heute habe ich einen Taschenrechner programmiert( er kann nur zwei zahlen bearbeiten). :

#include <iostream>

using namespace std;
int Zahl1, Zahl2, Eingabe, Ergebnis;
char Antwort;

int main()
{

cout << "Geben sie die Art der Rechnung ein :\n [1]: Addition\n [2]: Subtraktion\n [3]: Multiplikation\n [4]: Division\n" << '\n';
if ((cin >> Eingabe) == false)
{
cerr << "FEHLER" << '\n';
}
else
{
switch (Eingabe)
{
case 1:
cout << "Sie haben Addition gewählt. Geben sie die 2 Zahlen ein die sie addieren möchten" << '\n';
if ((cin >> Zahl1) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
else
{
if ((cin >> Zahl2) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
else
{
Ergebnis = Zahl1 + Zahl2;
cout << "Dass Ergebnis beträgt : " << Ergebnis << '\n';
}
}
break;

case 2:
cout << "Sie haben Subtraktion gewählt. Geben sie die 2 Zahlen ein die sie subtrahieren möchten" << '\n';
if ((cin >> Zahl1) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
if ((cin >> Zahl2) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
else
{
Ergebnis = Zahl1 - Zahl2;
cout << "Dass Ergebnis beträgt : " << Ergebnis << '\n';
}
break;

case 3:
cout << "Sie haben Multiplikation gewählt. Geben sie die 2 Zahlen ein die sie multiplizieren möchten" << '\n';
if ((cin >> Zahl1) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
if ((cin >> Zahl2) == false)
{
cerr << "Es muss eine Zahl eingegeben werden!" << '\n';
}
else
{
Ergebnis = Zahl1 * Zahl2;
cout << "Dass Ergebnis beträgt : " << Ergebnis << '\n';
}
break;
case 4:
cout << "Sie haben Division gewählt. Geben sie die 2 Zahlen ein die sie dividieren möchten(Keine Nullen)" << '\n';
if ((cin >> Zahl1) == false || Zahl1 == 0)
{
cerr << "Fehler bei Eingabe" << '\n';
}
if ((cin >> Zahl2) == false || Zahl2 == 0)
{
cerr << "Fehler bei Eingabe" << '\n';
}
else
{
Ergebnis = Zahl1 / Zahl2;
cout << "Dass Ergebnis beträgt : " << Ergebnis << '\n';
}
break;
}cout << "Willst du jetzt nochmal was rechnen oder das Programm schließen??" << '\n';
}
return 0;
}

Ich würde mich sehr über verbesserungsvorschläge etc freuen! (PS: er ist noch nicht ganz fertig)

Danke im Voraus

[マルコ]

Vorschlag 1) benutz die [CODE] Tags
2) Benutz functions, um den Code übersichtlicher und besser wiederverwendbar zu machen
3) Mach dann noch ne TR-Klasse, so dass es wirklich sauber wiederverwendbar ist.

Natürlich mein ich mit "wiederverwendbar", dass du es üben sollst. Weil brauchen wirst du das wahrscheinlich nicht mehr^^

Desweiteren empfehle ich, cin, so wie scanf in C, immer zu flushen.

Wie das alles geht? GOOGLE ; )
Oder frag dein Buch, das du sicherlich benutzt, um C++ zu lernen.

[Grapefruit97]

Meinst du mit ner Tr klasse ein template?

[マルコ]

Nein, eine Klasse.

class CCalculator
{
private:
  int num1,num2;
public:
  CCalculator();
  int multiply();
  void setNumbers(int, int);
};

[Hg(CNO)2]

Warum kennt er wohl keine Klassen?

Hallo, ich bin ein noch ziemlicher Anfänger in C++.

Verbesserungsvorschläge kann man hier nicht geben. Ein Anfänger sollte erstmal auf seiner Weise die Sprache lernen, da ist es momentan egal, welchen Programmierstil du verwendest.(Solang du schlechte Funktionen dir nicht angewöhnst.)


Over. And out. Hg(CNO)2

[.SkyneT.]

Da geb ich dem Knallquecksilber über mir Recht.
OOP kann man gleich mitlernen muss man aber nicht.

Stil sollte man aber von Anfang an so gut wie möglich lernen,
sonst gewöhnt man sich noch irgend nen Schwachsinn an...

[Grapefruit97]

Was muss ich denn eigl als grundlage in c++ alles wissen? was ist wichtig und was nicht unbedingt???

[dowhile]

Originally Posted by Grapefruit97

Was muss ich denn eigl als grundlage in c++ alles wissen? was ist wichtig und was nicht unbedingt???

Originally Posted by マルコ

Nein, eine Klasse.

class CCalculator
{
private:
  int num1,num2;
public:
  CCalculator();
  int multiply();
  void setNumbers(int, int);
};

Eine Klasse die ... multipliziert?!

Schreibst du statt

int frameWidth = rectWidth * rectCount

dann auch

Calculator calc();
calc.setNumbers(rectWidth, rectCount);
int frameWidth = calc.multiply();

? Und für eine Addition zweier Zahlen verwendest du dann kein "+" sondern schreibst dir die Klasse CAdder?

[マルコ]

Originally Posted by dowhile

Eine Klasse die ... multipliziert?!

Schreibst du statt

int frameWidth = rectWidth * rectCount

dann auch

Calculator calc();
calc.setNumbers(rectWidth, rectCount);
int frameWidth = calc.multiply();

? Und für eine Addition zweier Zahlen verwendest du dann kein "+" sondern schreibst dir die Klasse CAdder?

Ich glaub, du verfehlst das Topic -.-
Es war ein Beispiel, wie eine Klasse aussehen kann. Vollkommen ungeachtet der Funktionen. Flame also nicht rum.


Und ja, er hat nach Verbesserungsvorschlägen gefragt. Und eine Klasse ist eine Verbesserung. Wie/wann/wo ers lernt - darüber hab ich keine Aussage getroffen. Wobei ich empfehle, gerade als Anfänger möglichst früh sich das Klassenprinzip anzueignen und zu verwenden.

[Grapefruit97]

Was ist aber grundlage in C++? was muss ich beherrschen?

[MrSm!th]

Was muss ich denn eigl als grundlage in c++ alles wissen? was ist wichtig und was nicht unbedingt???

Alles ist wichtig.

Und ja, er hat nach Verbesserungsvorschlägen gefragt. Und eine Klasse ist eine Verbesserung. Wie/wann/wo ers lernt - darüber hab ich keine Aussage getroffen. Wobei ich empfehle, gerade als Anfänger möglichst früh sich das Klassenprinzip anzueignen und zu verwenden.

OOP ist kein göttliches Sprachmittel, das immer und überall am besten passt und deshalb jedem Projekt aufgezwungen werden sollte.

[ehauser]

OOP ist kein göttliches Sprachmittel, das immer und überall am besten passt und deshalb jedem Projekt aufgezwungen werden sollte.

Das stimmt!