C++ Programm tut nicht was es soll

[wulga (Gast)]

Hallo zusammen,

ich habe ein Programm geschrieben, welches Eingaben entgegennehmen und 
diese dann auswerten soll. Die Eingabe soll z.B. so erfolgen: 2,3p; 
daraufhin wird die Permutation aus 2 und 3 errechnet. Ob die Berechnung 
richtig ausgeführt wird, soll erstmal nicht wichtig sein. Mir gehts in 
erster Linie um eine Fehlerbehandlung die scheinbar ignoriert wird. Ich 
prüfe ob die erste Zahl größer als die zweite ist, ist dies der Fall, 
dann soll das Programm eine Fehlermeldung ausgeben und beendet werden.

Hier der Code, ich hoffe ihr könnt mir da helfen:

/*
Das Programm soll:
    den Benutzer auffordern zwei Zahlen einzugeben
    abfragen, ob die Anzahl der möglichen Permutationen oder Kombinationen berechnet werden soll
    und schließlich das Ergebnis ausgeben.

Diese Aufgabe besteht aus mehreren Teilen:
    Zuerst analysieren Sie die obigen Anforderungen.
    Schreiben Sie genau auf, was das Programm tun muss.

Gehen Sie dann zur Entwurfsphase über:
    Schreiben Sie Pseudocode für das Programm und zerlegen Sie ihn in Teilkomponenten.

Das  Programm  sollte  auch  über  eine  vernünftige  Fehlerbehandlung  verfügen:
    Stellen  Sie sicher, dass für fehlerhafte Eingaben nützliche Fehlermeldungen ausgegeben werden
*/
#include "../../std_lib_facilities.h"

class PermComb {
public:
    PermComb();
    bool getInput();
    int calculate();
    void printResult();
    int faculty(int num1, int num2);

    // Konstanten
    const char comma;
    const char semicolon;
    const char permutation;
    const char combination;

    // Variablen
    int num1, num2, result;
    char ch1, ch2, ch3;
};

PermComb::PermComb():num1(0), num2(0), ch1('_'), ch2('_'), ch3('_'), comma(','), semicolon(';'), permutation('p'), combination('c')
{
    // entweder über die Memberinitialiererliste initialisieren oder direkt hier drin, beide richtig
    /*
    num1 = num2 = 0;
    ch1 = ch2 = ch3 = '_';
    comma = ',';
    semicolon = ';';
    permutation = 'p';
    combination = 'c';
    */
}

bool PermComb::getInput()
{
    cin >> num1 >> ch1 >> num2 >> ch2 >> ch3; // note that >> skips whitespace (space, newline, tab, etc.)
    if(!cin) error("Fehler: Zahl erwartet");    // ein char akzeotiert sowohl Zahlen als auch Ziffern, daher greift die Abfrage if(!cin) nur bei num1 und num2
    else if(ch1 != comma) error("Fehler: ',' zwischen erster und zweiter Zahl erwartet");
    else if(ch2!='p' && ch2 != 'c')  error("Fehler: 'p' oder 'c' als Eingabe erwartet");
    else if(ch3 != semicolon) error("Fehler: ';' als Eingabe erwartet");
    return 1;
}

int PermComb::calculate()
{
    // Vorbedingung num1 muss größer als num2 sein
    if(num1>num2) error("Fehler: Zahl1 < Zahl2");
    cout << num1 << "   " << num2 << '\n';

    if(ch2=='p') result=faculty(num1, num2);
    //else result=combination(num1, num2);

}

void PermComb::printResult()
{
    cout << num1 << ch1 << num2 << ch2 << ch3;
    cout << '\n' << result << '\n';
}

int PermComb::faculty(int num1, int num2)
{
    int sum = num1-num2;
    for(int i=sum; i>0; --i) {
        sum = sum*(sum-1);
    }
    return sum;
}

int main()
try{
    cout << "Bitte zwei ganze Zahlen eingeben (getrennt mit Komma),\n";
    cout << "gefolgt von 'p' oder 'm'. Mit ';' Eingabe beenden. (Bsp.: 3,2;): ";
    PermComb t;
    while(t.getInput())
    {
        //t.getInput(); // speichert zwei Zahlen in der Klasse
        t.calculate(); // Frage ob Permutation oder Kombination berechnen + Berechnung(Funktionsaufruf)
        t.printResult(); // Ausgabe d. Ergebnisses
    }
}
catch (runtime_error e) {  // this code is to produce error messages
  cout << e.what() << '\n';
  //keep_window_open("0");  // For some Windows(tm) setups
}
catch (...) {
    cerr << "Komische Ausnahmeauslösung";
}

[Rainer V. (rudi994)]

Compilieren von o.g. Code liefert hier Warnungen, weil comma, semicolon 
usw. in Klasse PermComb als Konstanten definiert sind und erst später 
initialisiert werden. Ich habe das in private-Variablen geändert, die im 
Konstruktor auf ihre Sollwerte gesetzt werden. Dann funktioniert auch 
die Eingabe-Fehlerabfrage. Zudem: Wertrückgabe in PermComb::calculate() 
fehlt. std_lib_facilities.h habe ich von stroustrup.com kopiert, kann 
ich nicht gebrauchen wg. Fehlern, die ich nicht beseitigen kann. Für 
exception handling includiere ich Header <stdexcept> und habe auch mal 
in der Eingabe-Abfrage anstatt Ausgaben mit error() Ausnahmen mit throw 
(char*)"Fehlermeldung" geworfen und mit catch (char *e){...} abgefangen. 
Funktioniert ebenfalls. LG

[Wilhelm M. (wimalopaan)]

Ist das eine Übungsaufgabe eines C++-Kurses?

Neben einigen Fragwürdigkeiten ist schlecht, das SRP (Single 
Responsibility Principle) nicht eingehalten wird: was hat die Klasse 
PermComb mit der Benutzereingabe zu tun? Nichts.

Der Kommentar im ctor

// entweder über die Memberinitialiererliste initialisieren oder direkt hier drin, beide richtig

ist falsch, denn initialisieren macht man für primitive DT (am besten) 
als in-class-initializer, oder (wie hier) in der 
Elementinitialisierungsliste. Aber zuweisen (wie ctor-Rumpf als 
Kommentar) ist keine Initialisierung und bei const-Membern nicht 
möglich! Bei primitiven DT mag der Unterschied klein erscheinen, bei UDT 
kann er gewaltig sein bis hin zu schlicht unmöglich (wenn der DT nicht 
kopier-/verschiebzuweisbar ist.

Und noch ein Tipp: liefere kleinere und vollständige Beispiele, die auch 
direkt so compiliert werden können, ohne weitere Files zu benötigen.

[wulga (Gast)]

Hallo Rainer, habe es jetzt auch ohne "const" gemacht und alles im 
Konstruktor initialisiert, jetzt funktioniert die Fehlerbehandlung auch, 
mir ist allerdings immernoch unklar warum das mit dem "const" ein 
Problem war. Wenn ich z.B. folgendes mache (siehe Code), dann 
funktioniert das Programm zwar, aber der Compiler gibt Warnmeldungen 
aus:

...
    // Konstanten
    const char comma = ',';
    const char semicolon = ';';
    const char permutation = 'p';
    const char combination = 'c';
...

Wanrmeldung des Compilers:
"non-static data-memebr initializer only available with C++11"



Hallo Wilhelm, so in der Art, es handelt sich um eine Übungsaufgabe aus 
dem Buch "Einführung in die Programmierung mit C++ - Stroustrup".

Dass die Klasse nicht sinnvoll aufgebaut ist, hat damit zu tun, dass ich 
Anfänger bin und wie es aussieht Übung brauche um besser zu werden, an 
dieser Stelle danke für den Hinweis, habe es mir zu Herzen genommen.

Mein Kommentar im Konstruktor ist in der Tat unsinnig, da habe ich die 
Memberinitialisierungsliste mit ganz normaler Initialisierung von 
Variablen gleichgesetzt. Aber ich muss Konstanten doch einen Wert 
zuweisen können, wie wird das dann richtig gemacht? 
Memberinitialisierungsliste scheint ja nicht zu funktionieren und wenn 
ich die Konstanten sofort bei der deklaration definiere - also einen 
Wert übergebe - dann bekomme ich Warnmeldungen vom Compiler...

[wulga (Gast)]

wulga schrieb:
> Aber ich muss Konstanten doch einen Wert
> zuweisen können, wie wird das dann richtig gemacht?

Ups da habe ich jetzt blödsinn geschrieben, wenn ich die Konstanten nur 
deklariere und dann im Konstruktor initialisiere, dann funzt es 
einwandfrrei, habe es eben getestet.

[Mark B. (markbrandis)]

Wozu jetzt einen zweiten Thread?

[beric (Gast)]

wulga schrieb:
> Diese Aufgabe besteht aus mehreren Teilen:
>     Zuerst analysieren Sie die obigen Anforderungen.
>     Schreiben Sie genau auf, was das Programm tun muss.
>
> Gehen Sie dann zur Entwurfsphase über:
>     Schreiben Sie Pseudocode für das Programm und zerlegen Sie ihn in
> Teilkomponenten.

Dein Code sieht nicht danach aus, dass du diesen Teil der Aufgabe schon 
bewältigt hast.

[wulga (Gast)]

Mark B. schrieb:
> Wozu jetzt einen zweiten Thread?

Ist nur ein wenig dumm gelaufen,  in  diesem Beitrag sollte nur 
behandelt werden warum die Fehlerbehandlung nicht funktioniert, 
letztendlich sind wir dann bei den Konstanten gelandet die ich in einem 
anderen Beitrag erwähnt habe, das Problem wurde dort ja aber behoben. 
Von dem her ist dieser Beitrag also abgeschlossen.

 @beric
 Analyse und Programmablauf mache ich im Kopf. Pseudocode macht man doch 
nur wenn das Problem wirklich komplex und unübersichtlich wird? Aus 
meiner Sicht alles eingehalten.