MaBarker's "CalcTool"

[MaBarker]

Hey,
Ich habe mich insgesamt jetzt zirka 2-3 Stunden mit C++ befasst da ich dies nun anfange zu lerne.
Ich habe nun eine KonsolenAnwendung geschrieben die +, -, *, / und ggf. den Notendurchschnitt errechnet.

Ich release es da ich gerne vllt. ein wenig Arbeit teilen möchte und euch dies Presentieren möchte und ich warte immer auf Verbesserungs Vorschläge o.ä.


Also Gut.
In den 2-3 Stunden habe ich genug Wissen erlangt um diesen Code selbstständig zu schreiben.
Aus dieser Source ist nix C&P.

 Spoiler

#include <iostream>

using namespace std;

void addition();
void subtraktion();
void multiplikation();
void division();
void Notenrechner();

int main(void)
{
//////////MODES AUSWAHL///////////////////
cout << "Waehle einen Modus:\n\n"
<< "1 = Addition\n"
<< "2 = Suptraktion\n"
<< "3 = Multiplikation\n"
<< "4 = Division\n"
<< "5 = Notenrechner\n"
<< "6 = Abbrechen\n\n"
<< "Auswahl: ";
//////////////////////////////////////////

char auswahl;
cin >> auswahl;

switch( auswahl )
{
case '1':
cout << "Sie haben die Option Addition ausgewaehlt.\n\n" << endl;
addition();
break;
case '2':
cout << "Sie haben die Option Subtraktion ausgewaehlt.\n\n" << endl;
subtraktion();
break;
case '3':
cout << "Sie haben die Option Multiplikation ausgewaehlt\n\n" << endl;
multiplikation();
break;
case '4':
cout << "Sie haben die Option Division ausgewaehlt.\n\n" << endl;
division();
break;
case '5':
cout << "Sie haben die Option Notenrechner ausgewaehlt.\n\n" << endl;
Notenrechner();
case '6':
return 0;

default:
cout << "Ungueltige Eingabe\nProgramm wird Beendet." << endl;
break;
////////////////////////////////////////////////////////////////
}
cin.sync();
cin.get();
return 0;
}

void addition()
{
float zahl1, zahl2;

cout << "Bitte Zahl 1 Angeben." << endl;
cin >> zahl1;
cout << "Nun bitte Zahl 2 Angeben." << endl;
cin >> zahl2;
cout << "Die Rechnung war: \n" << zahl1 << "+" << zahl2 << endl;
cout << "\n\n\n Das Ergebnis ist:\n\n " << zahl1 + zahl2 << endl;
}

//////////////////////////SUBTRAKTION///////////////////////////////////////////
void subtraktion()
{
float Subzahl1, Subzahl2;

cout << "Bitte Zahl 1 Angeben." << endl;
cin >> Subzahl1;
cout << "Nun bitte Zahl 2 Angeben." << endl;
cin >> Subzahl2;
cout << "Die Rechnung war: \n" << Subzahl1 << "-" << Subzahl2 << endl;
cout << "\n\n\n Das Ergebnis ist:\n\n " << Subzahl1 - Subzahl2 << endl;
}
////////////////////////////////////////////////////////////////////////////////
////////////////////////////////////////MULTIPLIKATION//////////////////////////

void multiplikation()
{
float Mulzahl1, Mulzahl2;

cout << "Bitte Zahl 1 Angeben." << endl;
cin >> Mulzahl1;
cout << "Nun bitte Zahl 2 Angeben." << endl;
cin >> Mulzahl2;
cout << "Die Rechnung war: \n" << Mulzahl1 << "*" << Mulzahl2 << endl;
cout << "\n\n\n Das Ergebnis ist:\n\n " << Mulzahl1 * Mulzahl2 << endl;
}
////////////////////////////////////////////////////////////////////////////////
////////////////////////////////Division////////////////////////////////////////

void division()
{
float Mulzahl1, Mulzahl2;

cout << "Bitte Zahl 1 Angeben." << endl;
cin >> Mulzahl1;
cout << "Nun bitte Zahl 2 Angeben." << endl;
cin >> Mulzahl2;
cout << "Die Rechnung war: \n" << Mulzahl1 << "*" << Mulzahl2 << endl;
cout << "\n\n\n Das Ergebnis ist:\n\n " << Mulzahl1 * Mulzahl2 << endl;
}

////////////////////////////////////////////////////////////////////////////////
////////////////////////////////Noten////////////////////////////////////////

void Notenrechner()
{
float NotenAnzahl, Note1, Note2, Note3, Note4, Note5, Note6;

cout << "Bitte geben sie die Anzahl ihrer Noten ein die sie insgesamt hatten." << endl;
cin >> NotenAnzahl;
cout << "Geben sie bitte ein wieviele sie von der Note 1 haben" << endl;
cin >> Note1;
cout << "Gben sie bitte ein wieviele sie von der Note 2 haben." << endl;
cin >> Note2;
cout << "Geben sie bitte ein wieviele sie von der Note 3 haben." << endl;
cin >> Note3;
cout << "Geben sie bitte ein wieviele sie von der Note 4 haben." << endl;
cin >> Note4;
cout << "Geben sie bitte ein wieviele sie von der Note 5 haben." << endl;
cin >> Note5;
cout << "Geben sie bitte ein wieviel sie von der Note 6 haben." << endl;
cin >> Note6;
cout << "Note1 = " << Note1 << endl;
cout << "Note2 = " << Note2 << endl;
cout << "Note3 = " << Note3 << endl;
cout << "Note4 = " << Note4 << endl;
cout << "Note5 = " << Note5 << endl;
cout << "Note6 = " << Note6 << endl;
cout << "Dein Durchschnitt ist: " << ((Note1 * 1) + (Note2 * 2) + (Note3 * 3) + (Note4 * 4) + (Note5 * 5) + (Note6 * 6)) / NotenAnzahl << endl;

cin.sync();
cin.get();
}

Mein Programm gibt es im Anhang zum Downloaden.


Also ich freue mich auf euer Feedback und eure Verbesserungsvorschläge.


Dazu noch der AV Link:



~MaBarker

[adistoe]

Sieht ganz gut aus, gehört aber eher hier hin:

[.Marcel']

schreibe deinen Quellcode ohne den namespace std;
Bsp:

PHP Code:

std::cout << "hello, world" << std::endl; 


Aber sonst siehts ok aus.

[MaBarker]

Originally Posted by .Marcel'

schreibe deinen Quellcode ohne den namespace std;
Bsp:

PHP Code:

std::cout << "hello, world" << std::endl; 


Aber sonst siehts ok aus.

Ok ich werd mal gucken mit dem namespace std; ^^
Aber schonmal danke



Und schuldigung wegen Falsche Sec ^^
Hab hier vorher noch nix gepostet ^^

[alpines]

Sollte wohl aussagekräftig genug sein.

Dann auch noch Ausdrücke wie Zahl 1 und Zahl 2, das nennt man bei einer Addition: 1. Summand und 2. Summand und das Ergebnis ist eine Summe, wie dem auch sei.
Ebenfalls kann man auch nur 2 Zahlen verrechnen. So etwas schaffe ich in einer neuen Programmiersprache innerhalb von 15 Minuten. Vom Errorhandling will ich gar nicht sprechen, wenn man als Dezimalkomma ein , statt . nimmt dann crasht es instant.

Dennoch halt dich ran, vielleicht kannst du ja bald was anständiges releasen aber bis dahin immer schön in den Sammelthread ansonsten wird die Release Section nur geflooded.

[Mostey]

Mit Objektorientierung solltest du dich auch mal befassen, wird deinen Code sehr viel verständlicher und lesbarer machen.

[iSam34]

***Sorry falls sinnlos***

Aber bin ich wirklich der einzige, der sich fragt warum alle ne Konsolenanwendung machen wollen? Ich meine als WinForm wäre das ganze doch viel Praktischer.

Btw als Verbesserungsvorschlag : Du könntest 2 verschiedene Modes in das Programm einbauen :

1. Windows Form
2. Konsole (Mit Lime-Green Forecolor damit's cool aussieht :P )

[MaBarker]

Originally Posted by alpines

Sollte wohl aussagekräftig genug sein.

Dann auch noch Ausdrücke wie Zahl 1 und Zahl 2, das nennt man bei einer Addition: 1. Summand und 2. Summand und das Ergebnis ist eine Summe, wie dem auch sei.
Ebenfalls kann man auch nur 2 Zahlen verrechnen. So etwas schaffe ich in einer neuen Programmiersprache innerhalb von 15 Minuten. Vom Errorhandling will ich gar nicht sprechen, wenn man als Dezimalkomma ein , statt . nimmt dann crasht es instant.

Dennoch halt dich ran, vielleicht kannst du ja bald was anständiges releasen aber bis dahin immer schön in den Sammelthread ansonsten wird die Release Section nur geflooded.

Naja ich beschäftige mich ja halt noch nicht ganz so lange damit ^.^
Und das war halt mehr oder weniger mein erster Versuch was "vernünftiges" zu coden bzw. es war als kleiner test gedacht
Ich bin vorher aber auch nur von Normalen Zahlen ausgegangen ^^ also nicht von kommazahlen.

Aber danke fürs Feedback

Originally Posted by iSam34

***Sorry falls sinnlos***

Aber bin ich wirklich der einzige, der sich fragt warum alle ne Konsolenanwendung machen wollen? Ich meine als WinForm wäre das ganze doch viel Praktischer.

Btw als Verbesserungsvorschlag : Du könntest 2 verschiedene Modes in das Programm einbauen :

1. Windows Form
2. Konsole (Mit Lime-Green Forecolor damit's cool aussieht :P )

Ja mit der Farbe kann ich mal gucken haha ist aber eher ein wenig überflüssig meiner meinung aber danke
Und naja wie du liest hab ich mich damit noch nicht sooo viel befasst und bisher halt nur mit Konsolenanwendungen beschäftigt.

Aber danke fürs Feed schonma

[iSam34]

Originally Posted by MaBarker

Naja ich beschäftige mich ja halt noch nicht ganz so lange damit ^.^
Und das war halt mehr oder weniger mein erster Versuch was "vernünftiges" zu coden bzw. es war als kleiner test gedacht
Ich bin vorher aber auch nur von Normalen Zahlen ausgegangen ^^ also nicht von kommazahlen.

Aber danke fürs Feedback



Ja mit der Farbe kann ich mal gucken haha ist aber eher ein wenig überflüssig meiner meinung aber danke
Und naja wie du liest hab ich mich damit noch nicht sooo viel befasst und bisher halt nur mit Konsolenanwendungen beschäftigt.

Aber danke fürs Feed schonma

Kein Problem. Mach einfach weiter so^^

[th0rex]

Originally Posted by iSam34

***Sorry falls sinnlos***

Aber bin ich wirklich der einzige, der sich fragt warum alle ne Konsolenanwendung machen wollen? Ich meine als WinForm wäre das ganze doch viel Praktischer.

Btw als Verbesserungsvorschlag : Du könntest 2 verschiedene Modes in das Programm einbauen :

1. Windows Form
2. Konsole (Mit Lime-Green Forecolor damit's cool aussieht :P )

Vielleicht weil es grauenvoll ist native Fenster mit der Windows API zu erstellen ? Der Topic Ersteller benutzt ja nicht .NET so wie du. Außerdem wenn er noch nicht lange c++ kann soll er ruhig erstmal bei konsolenanwendungen bleiben. Er braucht ja am Ende keine super GUI, wenn das Programm uneffizient ist.

[snow]

Originally Posted by iSam34

Aber bin ich wirklich der einzige, der sich fragt warum alle ne Konsolenanwendung machen wollen? Ich meine als WinForm wäre das ganze doch viel Praktischer.

Primär soll das ganze wohl zur Übung dienen & tatsächlich ist das wichtigste beim Programmieren, dass man die Prinzipien versteht. Und das geht ohne Windows Forms oder ähnliches Zeugs einfach besser.

@MaBarker:
1) Schau nochmal auf deinen Code - alle Funktionen (multiplikation, division etc.) haben einen sehr ähnlichen Ablauf. Beim Programmieren sollte man auch darauf achten, so wenig Code wie möglich zu wiederholen - evtl. kannst du ja auch eine Funktion "calculation" schreiben, die als Parameter 1, 2, 3 oder 4 erhält & nach dem Einlesen je nachdem dividiert, subtrahiert etc. In deiner switch-Anweisung kannst du das dann so kürzen:
switch 1:
switch 2:
switch 3:
switch 4:
calculation(auswahl);
break;

2) Schau mal bei Notenrechner rein - hier wiederholst du dich auch. Evtl. sagen dir ja for-Schleife und Arrays etwas, dann kannst du aus den ganzen Wiederholungen eine Schleife machen

3) wie hier bereits erwähnt wurde: using namespace std; weglassen, das ist böse

@Mostey: Zu diesem Zeitpunkt wäre OOP für den TE wohl noch zu "viel" & nicht so sinnvoll.

[Mostey]

Originally Posted by snow911

@Mostey: Zu diesem Zeitpunkt wäre OOP für den TE wohl noch zu "viel" & nicht so sinnvoll.

Das sehe ich anders. Natürlich fängt man klein an und startet nicht sofort mit OOP voll durch, allerdings kann es nicht schaden, sich das so früh wie möglich mal anzuschauen. Vorallem weil wie gesagt der Code dadurch noch übersichtlicher wird, wie wenn du die Methoden konkret zusammenfasst.

Btw. weißt du ja gar nicht ob der TE nicht schon mit anderen OOP Sprachen gearbeitet hat? Falls das nämlich so ist, sollte man sich das wirklich umbedingt anschauen.

[MaBarker]

Originally Posted by Mostey

Das sehe ich anders. Natürlich fängt man klein an und startet nicht sofort mit OOP voll durch, allerdings kann es nicht schaden, sich das so früh wie möglich mal anzuschauen. Vorallem weil wie gesagt der Code dadurch noch übersichtlicher wird, wie wenn du die Methoden konkret zusammenfasst.

Btw. weißt du ja gar nicht ob der TE nicht schon mit anderen OOP Sprachen gearbeitet hat? Falls das nämlich so ist, sollte man sich das wirklich umbedingt anschauen.

Bisher gearbeitet nur mit VB.net ^^

[MrSm!th]

Ja, iSam, rate nem Anfänger zu C++/CLI. Super Idee.

Originally Posted by Mostey

Das sehe ich anders. Natürlich fängt man klein an und startet nicht sofort mit OOP voll durch, allerdings kann es nicht schaden, sich das so früh wie möglich mal anzuschauen. Vorallem weil wie gesagt der Code dadurch noch übersichtlicher wird, wie wenn du die Methoden konkret zusammenfasst.

Btw. weißt du ja gar nicht ob der TE nicht schon mit anderen OOP Sprachen gearbeitet hat? Falls das nämlich so ist, sollte man sich das wirklich umbedingt anschauen.

C++ ist nicht rein OO. An einem Taschenrechner gibt es nicht viel, was sich für OOP eignet.

Außerdem:



#closed