Verbesserungsvorschläge

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 07:26:42 GMT

Flyerz1934 — Tue, 15 May 2012 07:26:42 GMT

#include 
#include 
using namespace std;

int main()
{
    int ichoice;
    int ichoice1;
    int choice;
    int choice1;
    int choice2;
    int choice3;
    const double PI = 3.14;
    double umfang_kreis;
    double radius_kreis;
    double flaeche_kreis;
    double grundflaeche_pyramide;
    double hoehe_pyramide;
    double volumen_pyramide;
    double seite_dreieck;
    double hoehe_dreieck;
    double flaeche_dreieck;
    double seite_rhomboid;
    double hoehe_rhomboid;
    double flaeche_rhomboid;
    double flaeche_rhombus;
    double hoehe_rhombus;
    double seite_rhombus;
    double flaeche_quadrat;
    double seite_quadrat;
    double laenge_rechteck;
    double breite_rechteck;
    double tiefe_rechteck;
    double volumen_rechteck;
    double flaeche_rechteck;
    double kantenlaenge_wuerfel;
    double volumen_wuerfel;
    float num1;
    float num2;
    float sum1;
    char multi = '*';
    char ccount;
    char cyesno;
    char durch = '/';

    Menu:
         system("cls");
         cout << "Waehle: " << endl;
         cout << "1. Mathematik" << endl;
         cout << "2. Geometrie" << endl;
         cout << "3. Exit" << endl;

         cin >> ichoice;
         system("cls");

         if(ichoice == 1) {goto Mathematik; }
         if(ichoice == 2) {goto Geometrie;}
         if(ichoice == 3) {goto Exit; }
         else {
              cout << "Falsche Eingabe!" << endl;
              cerr << "Druecke eine Taste um fortzufahren" << endl;
              getchar();
              goto Menu;
              }

    Mathematik:
               system("cls");

               cout << "1 Zahl: ";
               cin >> num1;

               cout << "Operationszeichen (+, -, *, /):";
               cin >> ccount;

               cout << "2 Zahl: ";
               cin >> num2;
               system("cls");

               if(ccount =='+') { sum1 = num1 + num2; };
               if(ccount =='-') { sum1 = num1 - num2; };
               if(ccount =='*') { sum1 = num1 * num2; };
               if(ccount =='/') { sum1 = num1 / num2; };

               cout << "Summe: " << endl;
               cout << num1 << " " << ccount << " " << num2 << " = " << sum1 << endl << endl;

               cout << "Nochmal rechnen?" << endl << "[Y/N]: ";
               cin >> cyesno;
               if(cyesno =='y') {goto Mathematik; }
               if(cyesno =='n') {goto Menu;}
               else { goto confirm_calc; }

    confirm_calc:
                 cout << endl << endl << endl << "Falsche Eingabe!" << endl;
                 cout << "Nochmal rechnen?" << endl << "[Y/N]: ";
                 cin >> cyesno;
                 if(cyesno =='y') {goto Mathematik; }
                 if(cyesno =='n') {goto Menu;}
                 else { goto confirm_calc; }

    Geometrie:
                 system("cls");

                 cout << "Waehle: " << endl;
                 cout << "1. Rechteck, Quadrat, Wuerfel" << endl;
                 cout << "2. Rhombus, Rhomboid" << endl;
                 cout << "3. Dreieck, Pyramide" << endl;
                 cout << "4. Kreis, Kugel" << endl;

                 cin >> ichoice1;
                 system("cls");

                 if(ichoice1 == 1){goto Rechteck_Quadrat_Wuerfel; }           
                 if(ichoice1 == 2){goto Rhombus_Rhomboid; }
                 if(ichoice1 == 3){goto Dreieck_Pyramide; }
                 if(ichoice1 == 4){goto Kreis_Kugel; }
                 else {
                       cout << "Falsche Eingabe!" << endl;
                       cout << "Druecke eine Taste um fortzufahren" << endl;
                       getchar();
                       goto Menu;
                       }

    Rechteck_Quadrat_Wuerfel:
                               system("cls");

                               cout << "1. Flaeche: Rechteck" << endl;
                               cout << "2. Flaeche: Quadrat" << endl;
                               cout << "3. Volumen: Rechteck(Quader)" << endl;
                               cout << "4. Volumen: Wuerfel" << endl;

                               cin >> choice;
                               system("cls");

                               if(choice == 1){goto Flaeche_Rechteck; }
                               if(choice == 2){goto Flaeche_Quadrat; }
                               if(choice == 3){goto Volumen_Rechteck; }
                               if(choice == 4){goto Volumen_Wuerfel; }
                               else {
                                    cout << "Falsche Eingabe!" << endl;
                                    cout << "Druecke eine Taste um fortzufahren" << endl;
                                    getchar();
                                    goto Menu;
                                    }

    Flaeche_Rechteck:
                      system("cls");

                      cout << "Geben Sie die Länge ein: ";
                      cin >> laenge_rechteck;

                      cout << "Geben Sie die Breite ein: ";
                      cin >> breite_rechteck;

                      flaeche_rechteck = ( laenge_rechteck * breite_rechteck );

                      cout << laenge_rechteck << " " << multi << " " << breite_rechteck << " = " << flaeche_rechteck << "cm2" << endl << endl;

                      cout << "Noch eine Rechtecksflaeche berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Rechteck; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Volumen_Rechteck:
                     system("cls");

                     cout << "Geben Sie die Länge ein: ";
                     cin >> laenge_rechteck;

                     cout << "Geben Sie die Breite ein: ";
                     cin >> breite_rechteck;

                     cout << "Geben Sie die Tiefe ein: ";
                     cin >> tiefe_rechteck;

                     volumen_rechteck = ( laenge_rechteck * breite_rechteck * tiefe_rechteck );

                     cout << laenge_rechteck << " " << multi << " " << breite_rechteck << " " << multi << " " << tiefe_rechteck << " = " << volumen_rechteck << "ccm" << endl << endl;

                     cout << "Noch ein Rechtecksvolumen berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Volumen_Rechteck; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; } 

    Flaeche_Quadrat:
                    system("cls");

                    cout << "Geben Sie eine Seitenlänge ein: ";
                    cin >> seite_quadrat;

                    flaeche_quadrat = ( seite_quadrat * seite_quadrat );

                    cout << seite_quadrat << " " << multi << " " << seite_quadrat << " = " << flaeche_quadrat << "cm2" << endl << endl;

                    cout << "Noch eine Quadratflaeche berechnen berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Quadrat; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Volumen_Wuerfel:
                    system("cls");

                    cout << "Geben Sie die Kantenlänge ein: ";
                    cin >> kantenlaenge_wuerfel;

                    volumen_wuerfel = ( kantenlaenge_wuerfel * kantenlaenge_wuerfel * kantenlaenge_wuerfel );

                    cout << kantenlaenge_wuerfel << " " << multi << " " << kantenlaenge_wuerfel << " " << multi << " " << kantenlaenge_wuerfel << " = " << volumen_wuerfel << "ccm" << endl << endl;

                    cout << "Noch eine Würfel berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Volumen_Wuerfel; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Rhombus_Rhomboid:
                     system("cls");

                               cout << "1. Flaeche: Rhombus" << endl;
                               cout << "2. Flaeche: Rhomboid" << endl;

                               cin >> choice1;
                               system("cls");

                               if(choice1 == 1){goto Flaeche_Rhombus; }
                               if(choice1 == 2){goto Flaeche_Rhomboid; }
                               else {
                                    cout << "Falsche Eingabe!" << endl;
                                    cout << "Druecke eine Taste um fortzufahren" << endl;
                                    getchar();
                                    goto Menu;
                                    }

    Flaeche_Rhombus:
                    system("cls");

                    cout << "Geben Sie irgendeine Seite ein: ";
                    cin >> seite_rhombus;

                    cout << "Geben Sie die Hoehe h ein: ";
                    cin >> hoehe_rhombus;

                    flaeche_rhombus = ( seite_rhombus * hoehe_rhombus );

                    cout << seite_rhombus << " " << multi << " " << hoehe_rhombus << " = " << flaeche_rhombus << endl << endl;

                    cout << "Noch eine Rhombus berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Rhombus; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Flaeche_Rhomboid:
                     system("cls");

                     cout << "Geben Sie eine Seite ein: ";
                     cin >> seite_rhomboid;

                     cout << "Geben Sie die dazugehoerige Hoehe ein: ";
                     cin >> hoehe_rhomboid;

                     flaeche_rhomboid = ( seite_rhomboid * hoehe_rhomboid );

                     cout << seite_rhomboid << " " << multi << " " << hoehe_rhomboid << " = " << flaeche_rhomboid << endl << endl;

                     cout << "Noch eine Rhomboid berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Rhomboid; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Dreieck_Pyramide:
                     system("cls");

                               cout << "1. Flaeche: Dreieck" << endl;
                               cout << "2. Volumen: Pyramide" << endl;

                               cin >> choice2;
                               system("cls");

                               if(choice2 == 1){goto Flaeche_Dreieck; }
                               if(choice2 == 2){goto Volumen_Pyramide; }
                               else {
                                    cout << "Falsche Eingabe!" << endl;
                                    cout << "Druecke eine Taste um fortzufahren" << endl;
                                    getchar();
                                    goto Menu;
                                    }

    Flaeche_Dreieck:
                    system("cls");

                    cout << "Geben Sie eine Seite ein: ";
                    cin >> seite_dreieck;

                    cout << "Geben Sie die dazugehoerige Hoehe ein: ";
                    cin >> hoehe_dreieck;

                    flaeche_dreieck = ( seite_dreieck * hoehe_dreieck / 2 );

                    cout << seite_dreieck << " " << multi << " " << hoehe_dreieck << " " << durch << " " << " 2 " <<  " = " << flaeche_dreieck << endl << endl;

                    cout << "Noch ein Dreieck berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Dreieck; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Volumen_Pyramide:
                    system("cls");

                    cout << "Geben Sie die Grundflaeche ein: ";
                    cin >> grundflaeche_pyramide;

                    cout << "Geben Sie die Hoehe der Pyramide ein: ";
                    cin >> hoehe_pyramide;

                    volumen_pyramide = ( grundflaeche_pyramide * hoehe_pyramide / 3 );

                    cout << grundflaeche_pyramide << " " << multi << " " << hoehe_pyramide << " " << durch << " " << " 2 " << " = " << volumen_pyramide << endl << endl;

                    cout << "Noch eine Pyramide berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Volumen_Pyramide; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Kreis_Kugel:
                system("cls");

                               cout << "1. Flaeche: Kreis" << endl;
                               cout << "2. Umfang: Kreis" << endl;

                               cin >> choice3;
                               system("cls");

                               if(choice3 == 1){goto Flaeche_Kreis; }
                               if(choice3 == 2){goto Umfang_Kreis; }
                               else {
                                    cout << "Falsche Eingabe!" << endl;
                                    cout << "Druecke eine Taste um fortzufahren" << endl;
                                    getchar();
                                    goto Menu;
                                    }

    Flaeche_Kreis:
                  system("cls");

                  cout << "Geben Sie den Radius ein: ";
                  cin >> radius_kreis;

                  flaeche_kreis = ( radius_kreis * radius_kreis * PI );

                  cout << radius_kreis << " " << multi << " " << radius_kreis << " " << multi << " " << PI << " = " << flaeche_kreis << endl << endl;

                  cout << "Noch einen Kreis berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Flaeche_Kreis; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Umfang_Kreis: 
                  system("cls");

                  cout << "Geben Sie den Radius ein: ";
                  cin >> radius_kreis;

                  umfang_kreis = ( radius_kreis * 2 * PI );

                  cout << radius_kreis << " " << multi << " " << 2 << " " << multi << " " << PI << " = " << umfang_kreis << endl << endl;

                  cout << "Noch einen Umfang berechnen?" << endl << "[Y/N]: ";
                      cin >> cyesno;
                      if(cyesno =='y') {goto Umfang_Kreis; }
                      if(cyesno =='n') {goto Menu;}
                      else { goto confirm_calc; }

    Exit:
                 return 0;
                 }

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 07:55:09 GMT

[[global:guest]] — Tue, 15 May 2012 07:55:09 GMT

Alles neu! Haste bei Jürgen Wolf gelernt oder was!? Das ist unglaublich hässlich, Variablennamen immer so lokal wie möglich halten, niemals den Standard-Namensraum ausschütten, keine OS-spezifischen Konsolenbefehle benutzen, und dazu ist das alles noch so imperativ programmiert.....

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 08:05:38 GMT

arghonaut — Tue, 15 May 2012 08:05:38 GMT

Hacker schrieb:

Alles neu! Haste bei Jürgen Wolf gelernt oder was!? Das ist unglaublich hässlich, Variablennamen immer so lokal wie möglich halten, niemals den Standard-Namensraum ausschütten, keine OS-spezifischen Konsolenbefehle benutzen, und dazu ist das alles noch so imperativ programmiert.....

Das ist doch alles relativ wurscht.
Das eigentliche Problem ist mMn., dass alles in der main gemacht wird, und anstatt die Funktionalität in Funktionen/Klassen auszulagern, wird alles hinter Sprungmarken (goto) versteckt. Das wird unheimlich unübersichtlich und schwer erweiterbar.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 08:09:11 GMT

SeppJ — Tue, 15 May 2012 08:09:11 GMT

Abstraktion! Du wiederholst 20x fast den gleichen Code. Versuche dein Problem in Funktionen zu zerlegen, die du mehrmals benutzen kannst. Optimalerweise^* hast du in deinem Code keine zwei Zeilen Code jemals doppelt.

Werde den Spaghettisalat mit den ganzen gotos los! Die Zerlegung in Funktionen wird dir dabei helfen. Im Prinzip dürfte jedes goto bei dir stattdessen ein Funktionsaufruf werden, denn deine jetzigen Codeblöcke entsprechen schon fast Funktionen. Vorteil der Funktionen ist aber gewaltig, denn dann ist garantiert, dass die Codeblöcke auch wirklich unanbhängig sind und ein normaler Mensch (d.h. jeder außer dir und du selber in ein bis zwei Wochen) kann das Programm nachvollziehen, ohne neue Hirnwindungen zu bekommen.

Das sind erst einmal die allerwichtigsten zwei Punkte. Wenn das gemacht ist, kann man weitergucken, es gibt noch viel mehr Verbesserungspotential.

^*: Naja, wirklich optimal ist das nicht, aber es ist eine gute Anfängerübung, wenn du dies anstrebst.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 09:46:52 GMT

[[global:guest]] — Tue, 15 May 2012 09:46:52 GMT

arghonaut schrieb:

Hacker schrieb:

Alles neu! Haste bei Jürgen Wolf gelernt oder was!? Das ist unglaublich hässlich, Variablennamen immer so lokal wie möglich halten, niemals den Standard-Namensraum ausschütten, keine OS-spezifischen Konsolenbefehle benutzen, und dazu ist das alles noch so imperativ programmiert.....

Das ist doch alles relativ wurscht.
Das eigentliche Problem ist mMn., dass alles in der main gemacht wird, und anstatt die Funktionalität in Funktionen/Klassen auszulagern, wird alles hinter Sprungmarken (goto) versteckt. Das wird unheimlich unübersichtlich und schwer erweiterbar.

Hmmm? Das war ein Punkt von mir.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:14:51 GMT

Flyerz1934 — Tue, 15 May 2012 10:14:51 GMT

Vielen Dank für die Vorschläge.

Nun ich mach das Programm für die Schule als Projekt.
Öhm ich übe das Ganze mit einem Buch (jetzt lerne ich C++ , Dirk Louis)
Ausserdem werde ich das alles noch in der Ausbildung lernen ;).

Das Buch hab ich noch ned fertig deshalb kenn ich das mit den Funktionen nicht.

Ich werde das sicher noch versuchen zu ändern.
Könnt ihr mir sagen wie man hoch 2 macht?

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:16:52 GMT

volkard — Tue, 15 May 2012 10:16:52 GMT

p ist zwei hoch x
als double:
double p=std::pow(2,x);
als int:
int p=1<

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:19:16 GMT

SeppJ — Tue, 15 May 2012 10:19:16 GMT

Wie wäre es allgemein mit x*x ?

~edit: so ein kurzer Satz, so viele Rechtschreibfehler, *schäm*.~

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:21:00 GMT

314159265358979 — Tue, 15 May 2012 10:21:00 GMT

@volkard: Er will doch x^2, nicht 2^x.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:22:52 GMT

volkard — Tue, 15 May 2012 10:22:52 GMT

Flyerz1934 schrieb:

Das Buch hab ich noch ned fertig deshalb kenn ich das mit den Funktionen nicht.

Draußen scheint die Sonne. Setz eine Sonnebrille auf und geh mit dem Buch in den Garten und hau es Dir flugs in den Kopf. Ganz oberflächlich, so daß Du es bis morgen durch hast.

Und dann mach weiter wie gehabt, aber mit einem besseren Blick fürs Große. Im Endeffekt sparst Du damit sicherlich einiges an Zeit. (Benutze für dieses Projekt keine selbergebauten Klassen.)

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:31:01 GMT

arghonaut — Tue, 15 May 2012 10:31:01 GMT

Hacker schrieb:

Hmmm? Das war ein Punkt von mir.

Auch ohne goto wäre das Programm noch imperativ gehalten.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:31:09 GMT

Flyerz1934 — Tue, 15 May 2012 10:31:09 GMT

Aber wenn ich das Buch nur oberflächlich lese (d.h ohne üben), habe ich es morgen oder übermorgen schon wieder vergessen

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:33:12 GMT

[[global:guest]] — Tue, 15 May 2012 10:33:12 GMT

Volkard!!! Du bist zurück!!

@argonhaut: hab ich das gesagt? Ich hab einfach gesagt, es wurde imperativ programmiert, mehr nicht (auch nicht woraus ich das schließe).

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 10:49:24 GMT

berniebutt — Tue, 15 May 2012 10:49:24 GMT

Irgendwie erinnert mich der Code mit den vielen goto an früheres BASIC (vor 1980?), das auch noch keine Funktionen kannte. Was soll man daran verbessern? Wegschmeissen und ohne goto neu machen!

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 11:35:11 GMT

arghonaut — Tue, 15 May 2012 11:35:11 GMT

Hacker schrieb:

hab ich das gesagt? Ich hab einfach gesagt, es wurde imperativ programmiert, mehr nicht (auch nicht woraus ich das schließe).

Selbst wenn du die goto durch Funktionen ersetzt, bleibt das Programm eine Aneinanderreihung von "Anweisungen" -> immer noch imperativ. Demnach hast du nicht das selbe ausgedrückt. Und selbst wenn es das selbe gewesen wäre, der OP kann mit "kein goto" sicher mehr anfangen als mit "das ist imperativ programmiert -> böse".

Aber diese Nebendiskussion ist unnötig, da mittlerweile das goto schon so oft angemeckert wurde, dass es klar ist, was zuerst geändert werden sollte.

Reply to Verbesserungsvorschläge on Tue, 15 May 2012 13:33:03 GMT

volkard — Tue, 15 May 2012 13:33:03 GMT

Flyerz1934 schrieb:

Aber wenn ich das Buch nur oberflächlich lese (d.h ohne üben), habe ich es morgen oder übermorgen schon wieder vergessen

Du sollst es überfliegen, damit Du fürderhin in die richtigen Richtungen vorausüberlegst. Und dann da weiterlesen, wo Du gerade bist. Mit Üben.