Erstmal moechte ich mich vorstellen:

Ich mache ein Ausbildung zum Fisi seit etwa 3 Monaten und hab auch schon viel
gelernt , doch neben bei programmiere ich Zu Hause und auch etwas bei der Arbeit etwas C++.

Ich möchte mich natürlich immer weiter verbessern, und da meine Arbeitskollegen alle in Delphi programmieren und sie mir so nicht immer optimal helfen können, wollte ich hier mal in laufe meiner Karriere als Hobbyprogrammierer ein paar Quellcodes posten.

Ich hoffe das ich hier viel lernen werde

So aber nun zu meinem Rechner:

#include <iostream>
#include <math.h>

using namespace std;

double grundrechenarten(double a, double b,char x);
double wurzelziehen(double a,char x);
void textgrundrechenarten(double &a , double &b);
void textwurzelziehen(double &a);

int main() {

    double zahl1, zahl2;
    char zeichen;
    bool schleife1 {1};
    do {
        cout << "Bitte waehlen Sie aus was Sie rechen wollen!" << endl;
        cout << "Um das Programm zu beenden druecken Sie n" << endl << endl;
        cout << "+" << endl;
        cout << "-" << endl;
        cout << "*" << endl;
        cout << "/" << endl << endl;
        cout << "erweiterte Rechenarten: " << endl;
        cout << "***********************************" << endl << endl;
        cout << "Wurzel ziehen (w)" << endl << endl;
        cout << "beenden (n)" << endl;
        cin >> zeichen;
        switch (zeichen) {
        case '+' : case '-' : case '*' : case '/' :
            textgrundrechenarten(zahl1, zahl2);
             break;
        case 'w' :
            textwurzelziehen(zahl1);
            break;
        case 'n' :
            cout << "Programm wird beendet!" << endl;
            schleife1 = 0;
        default  :
            cout << "Die Eingabe ist ungültig!" << endl;
            break;
        }
        if ((grundrechenarten(zahl1,zahl2,zeichen) != 0))
            {
                cout << "das Ergebnis ist: " <<  grundrechenarten(zahl1,zahl2,zeichen) << endl;
            }
            else if ((wurzelziehen(zahl1,zeichen) !=0))
                     {
                cout << "Die Wurzel ist: " << wurzelziehen(zahl1,zeichen) << endl;
            }
    } while (schleife1);
    return 0;
}

double grundrechenarten(double a, double b,char x)
{
    double summe = 0;

    if (x == '+')
    {
        summe = a + b;
    }
    else if (x == '-')
    {

        summe = a - b;
    }
    else if (x == '*')
    {
        summe = a * b;

    }
    else if (x == '/')
    {
    if (b != 0)
    {
        summe = a / b;
    }
    else {
    cout << "Durch Null teilen ist leider nicht moeglich!" << endl << endl;
    }
    }
    return summe;
}

void textgrundrechenarten(double &a , double &b)
{
  cout << "Geben Sie bitte Zahl 1 ein" << endl;
  cin >> a;
  cout << "Geben Sie bitte Zahl 2 ein" << endl;
  cin >> b;
}
double wurzelziehen(double a,char x)
{
    double wurzel;

if (x == 'w')
{
    wurzel = sqrt (a);
}
    return wurzel;
}

void textwurzelziehen(double &a)
{
  cout << "Geben Sie die Zahl ein, von dem Sie die Wurzel haben moechten" << endl << endl;
  cin >> a;
}

Was nicht klappt

wenn die Zahl 0 als ergebnis kommt außer natürlich bei /.
endlosschleife wenn ich bei der Eingabe der 1. zahl ein Buchstabe eingebe.

naja, sonst scheint alles zu laufen.

mfg d4rksider

Was nicht klappt

wenn die Zahl 0 als ergebnis kommt außer natürlich bei /.
endlosschleife wenn ich bei der Eingabe der 1. zahl ein Buchstabe eingebe.

naja, sonst scheint alles zu laufen.

mfg d4rksider

also du kannst das mit dem buchstaben verhindern mit der "isdigit" funktion...
diese prüft ob nur zahlen in der eingabe vorhanden sind, damit solltest du z.B.

auch die eingabe von sonderzeichen oder ähnlichem abfangen können...
hier ein link dazu:
http://www.cplusplus.com/reference/cctype/isdigit/
http://en.cppreference.com/w/cpp/locale/isdigit

zu deinem anderen problem hab ich jetzt nicht gesehen warum bei eingabe einer 0 abgesehen von "/" ein fehler auftreten sollte... (hab gerade aber auch nur mal schnell drüber geschaut)

hoff ich konnte helfen
lg

danke für die Antwort.

naja, wegen der 0:

if ((grundrechenarten(zahl1,zahl2,zeichen) != 0)) 
            { 
                cout << "das Ergebnis ist: " <<  grundrechenarten(zahl1,zahl2,zeichen) << endl; 
            } 
            else if ((wurzelziehen(zahl1,zeichen) !=0)) 
                     { 
                cout << "Die Wurzel ist: " << wurzelziehen(zahl1,zeichen) << endl; 
            }

denke weil bei jeder Zahl dieser Text angezeigt wird ausßer bei der 0.
wollte damit verhindern, dass ich durch 0 teilen kann...
ich glaube ich steige gerade selber nicht mehr durch den code

wenn ich nun z.b 4 * 0 rechne kommt da nur "0 " und nicht das Ergebnis ist...

werde mal versuchen isdigit einzuarbeitet

ich hab eben nochmal geschaut ich kann nix erkennen wo eine 0 probleme machen sollte...
bitte beschreibe woran du erkennst das es "falsch" ist, um mir zu ermöglichen den fehler i.wie einzugrenzen...
so sehe ichs leider nicht und kann dir nur bedningt weiterhelfen...

hallo,

danke für die Antwort.

naja, wegen der 0:

if ((grundrechenarten(zahl1,zahl2,zeichen) != 0)) 
            { 
                cout << "das Ergebnis ist: " <<  grundrechenarten(zahl1,zahl2,zeichen) << endl; 
            } 
            else if ((wurzelziehen(zahl1,zeichen) !=0)) 
                     { 
                cout << "Die Wurzel ist: " << wurzelziehen(zahl1,zeichen) << endl; 
            }

denke weil bei jeder Zahl dieser Text angezeigt wird ausßer bei der 0.
wollte damit verhindern, dass ich durch 0 teilen kann...
ich glaube ich steige gerade selber nicht mehr durch den code

wenn ich nun z.b 4 * 0 rechne kommt da nur "0 " und nicht das Ergebnis ist...

werde mal versuchen isdigit einzuarbeitet

ja das hört sich so an als wenn dort der fehler liegt, gerade wenn diese ausgabe da sein sollte und nicht da ist...

du musst dort auch nicht die "0" beim "/" abfangen das machst du doch schon hier:

if (b != 0)
    {
        summe = a / b;
    }

oder?
wenn dir da noch das a fehlt kannst du auch so machen

if (b != 0 && a != 0 )
    {
        summe = a / b;
    }

müsste dann wenn du oben das !=0 aus deiner if-Abfrage(if((grundrechenarten(zahl1,zahl2,zeichen) != 0))... ) nimmst die du gepostet hast, müssten deine fehler erstmal behoben sein...

das Ergebnis ist:

ist vll nicht so schwer in den Griff zu kriegen aber ich verstehe es momentan nicht wie ich es hinkriegen soll.

naja und später möchte ich natürlich noch mehr rechenarten einfügen und nicht daran scheitern^^

if ((grundrechenarten(zahl1,zahl2,zeichen) != 0))
            {
                cout << "das Ergebnis ist: " <<  grundrechenarten(zahl1,zahl2,zeichen) << endl;
            }
            else if ((wurzelziehen(zahl1,zeichen) !=0))
                     {
                cout << "Die Wurzel ist: " << wurzelziehen(zahl1,zeichen) << endl;
            }

da du damit alle "ergebniss die 0" sind blockst... (also die rückgabe von grundrechenarten() darf nicht null sein...== dein ergebnis )

deswegen bekommst keine "0" als ergebnis, da die rückgabe von grundrechenarten nicht 0 sein darf... falls doch gehst du ja in den else if()...
wenn du das änderst dann solltest du auch 0 als ergebnis bekommen können sofern keine weiteren fehler existieren...

naja, das ergebnis ist richtig nur kommt der Text kommt nicht:

das Ergebnis ist:

ist vll nicht so schwer in den Griff zu kriegen aber ich verstehe es momentan nicht wie ich es hinkriegen soll.

naja und später möchte ich natürlich noch mehr rechenarten einfügen und nicht daran scheitern^^

dieser fall kann passieren wenn das ergebnis "leer" ist...
musst du mal schauen wie sowas passieren kann, soweit ich gesehen habe ist die rückgabe von wurzelziehen "leer" sein, vll auch wenn du durch 0 versuchst zu teilen...
mach mal bei deinen variablen diesen:

int x=0;
int y=0;
int a=0; ...

dann steht wenigstens da das ergebniss ist "0"... auch wenn du keine rechenoperationen durchführst...

zuudem sollte man solche fehler wie teilen durch "0" direkt abfangen, dies wäre für mich ein kritischer fehler da die rechnung so nicht möglich ist...
da solltest du die funktion definiert mit fehlermeldung beenden... ("z.B.: ACHTUNG ......" )
sonst führt das später zu fehlern die du nicht mehr identifizieren kannst, da du keine fehlermeldung bekommst... oder etwa doch?
diese fehler sind im nachhinein, geschweige denn wenn du evt. mal 10.000 zeilen code hast unmöglich oder nur mit sehr viel zeitaufwand zu finden... (musst du aber selbst entscheiden wie du mit fehlern umgehst...)-> nicht kritische fehler musst du ja nicht abbrechen ...

alternativ kann man auch mit "fstream's" schnell ein custom-log anlegen wo du einfach die aktionen loggst... -> damit du beim entwickeln und vorallem später deine fehler erkennen und lokalisieren kannst... (odewr zumindest wesentlich schneller in dieser hinsicht bist)

lg

also ist der wert 0 der da rauskommt nicht das ergebnis der rechenoption sondern ein false wert.

Also ich habe bis jetzt noch keine passende lösung dafür aber werde mich da schon reinfuchsen

danke MfG

danke für den Hinweis...
muss ich mal drüber nachdenken wie ich das in den griff bekomme.

also ist der wert 0 der da rauskommt nicht das ergebnis der rechenoption sondern ein false wert.

Also ich habe bis jetzt noch keine passende lösung dafür aber werde mich da schon reinfuchsen

danke MfG

eigentlich sollte das nicht mehr passieren wenn du deine variablen auf "0" initialisierst... -> wie oben schon geschrieben:

int x=0; etc...

ansonsten müsste ich jetzt nochmal schauen...

kannst du mir vll sagen welche funktion du meinst?

danke

hmm weiß gerade nicht welche variablen du meinst ... sorry.
habe nämlich gar keine int variablen drin nur double.

kannst du mir vll sagen welche funktion du meinst?

danke

sry ich meinte damit nich die "int" variablen sondern alle variablen die zahlen enthalten...
also auch double etc... mir gings eher ums prinzip...

double wurzelziehen(double a,char x)
{
    double wurzel;

if (x == 'w')
{
    wurzel = sqrt (a);
}
    return wurzel;
}

wenn if() greift ist deine variable "leer"...
normalerweise müsste das zwar dann NULL sein...
aber das ist je nach gültigkeit der variablen unterschiedlich...

deswegen lieber das:

double wurzelziehen(double a,char x)
{
    double wurzel=0;

if (x == 'w')
{
    wurzel = sqrt (a);
}
    return wurzel;
}

noch besser wäre:

double wurzelziehen(double a,char x)
{
    double wurzel=0;

if (x == 'w')
{
    wurzel = sqrt (a);
}else{ 
    cout << "fehler beim wurzel ziehen der Zahl: "<< a << endl; 
}
    return wurzel;
}

aber wie du es machst mit

so habe nun

if ((grundrechenarten(zahl1,zahl2,zeichen)))

verändert

und auch:

if (b != 0 && a != 0 )

auch habe ich alle zahlen initialisiert.

es kommt leider immer noch nicht : Das Ergebnis ist:

aber werde wohl noch mal einer Nacht drüber schlafen und hoffen das ich dann nichts mehr übersehen werde.

danke für deine hilfe. Der nächte quelltex(s)t kommt bestimmt

Edit: beim tippen gerade deine Antwort gesehen:

gut zu wissen das man vieles erst initialisieren muss damit alles gut läuft.
werde ich mir merken
morgen schicke ich mal die aktualisierte version meiner kleinen rechenmaschine.

bis denn

d4rksider

alles klar

so habe nun

if ((grundrechenarten(zahl1,zahl2,zeichen)))

verändert

und auch:

if (b != 0 && a != 0 )

auch habe ich alle zahlen initialisiert.

es kommt leider immer noch nicht : Das Ergebnis ist:

aber werde wohl noch mal einer Nacht drüber schlafen und hoffen das ich dann nichts mehr übersehen werde.

danke für deine hilfe. Der nächte quelltex(s)t kommt bestimmt

Edit: beim tippen gerade deine Antwort gesehen:

gut zu wissen das man vieles erst initialisieren muss damit alles gut läuft.
werde ich mir merken
morgen schicke ich mal die aktualisierte version meiner kleinen rechenmaschine.

bis denn

d4rksider

ok mach das ...
wenn es dir spaß macht und dich interessiert helfe ich gerne...

eig passiert so etwas mit der "leeren" ausgabe wenn kein wert in der variablen steht und du sie ausgibst...

eig musst du das nicht unbedingt, in den meisten fällen macht das der compiler selbst...
jedoch bei rechenprogrammen habe ich festgestellt das es einfach fehlern vorbeugt auf 0 zu initialisieren... (bei strings,chars brauchst du so etwas z.B. gar nicht!)

falls dich das interessiert mit den variablen ließ mal hier:
grundlegend:
http://www.willemer.de/informatik/cpp/var.htm

gültigkeitsbereich und alles was damit zusammenhängt (z.B. wert ohne initalisierung)
http://www.willemer.de/informatik/cpp/geltungvar.htm

hoff ich konnt dir paar tipps, anregungen, hilfen geben...
gerne wieder ...

lg

hier mal meine aktualisierte version des Taschenrechners:

#include <iostream>
#include <math.h>
#include <string>
#include <algorithm>

using namespace std;

bool grundrechenarten(double a, double b,char x, double &summe);
bool wurzelziehen(double a,char x, double &summe);
void textgrundrechenarten(double &a , double &b);
void textwurzelziehen(double &a);
void bedingung(double &a, double&b);

int main() {

    double zahl1, zahl2;
    char zeichen;
    bool schleife1 {1};
    double summe;
    do {
        cout << "Bitte waehlen Sie aus was Sie rechen wollen!" << endl;
        cout << "Um das Programm zu beenden druecken Sie n" << endl << endl;
        cout << "+" << endl;
        cout << "-" << endl;
        cout << "*" << endl;
        cout << "/" << endl << endl;
        cout << "erweiterte Rechenarten: " << endl;
        cout << "***********************************" << endl << endl;
        cout << "Wurzel ziehen (w)" << endl << endl;
        cout << "beenden (n)" << endl;
        cin >> zeichen;
        switch (zeichen)
        {
        case '+' :
        case '-' :
        case '*' :
        case '/' :
            textgrundrechenarten(zahl1, zahl2);
            break;
        case 'w' :
            textwurzelziehen(zahl1);
            break;
        case 'n' :
            cout << "Programm wird beendet!" << endl;
            schleife1 = 0;
        default  :
            cout << "Die Eingabe ist ungültig!" << endl;
            break;
        }
        if (grundrechenarten(zahl1,zahl2,zeichen,summe))
        {
            cout << "das Ergebnis ist: " <<  summe  << endl;
        }
        else if (wurzelziehen(zahl1,zeichen,summe))
        {
            cout << "Die Wurzel ist: " << summe  << endl;
        }
    } while (schleife1);
    return 0;
}

bool grundrechenarten(double a, double b,char x, double &summe)
{
    summe = 0;
    bool wahr = 1;

    if (x == '+')
    {
        summe = a + b;
    }
    else if (x == '-')
    {

        summe = a - b;
    }
    else if (x == '*')
    {
        summe = a * b;

    }
    else if (x == '/')
    {
        if (b != 0 || a !=0)
        {
            summe = a / b;
        }
        else
        {
            wahr = 0;
        }
    }
    else
    {
        cout << "Bitte geben Sie eine Zahl ein!" << endl;
        wahr = 0;
    }
    return wahr;
}

void textgrundrechenarten(double &a , double &b)
{
    do {
        cout << "Geben Sie bitte Zahl 1 ein" << endl;
        cin >> a;
        bedingung(a,b);
        cout << "Geben Sie bitte Zahl 2 ein" << endl;
        cin >> b;
        bedingung(a,b);

    } while (cin.fail() || cin.bad());
}

bool wurzelziehen(double a,char x, double &summe)
{
    summe = 0;
    bool wahr = 1;

    if (x == 'w')
    {
        summe = sqrt (a);
    }
    return wahr;
}

void textwurzelziehen(double &a)
{
    cout << "Geben Sie die Zahl ein, von dem Sie die Wurzel haben moechten" << endl << endl;
    cin >> a;
}

void bedingung(double &a, double &b)
{
    if (cin.fail() || cin.bad())
    {
        cin.clear();
        cin.ignore();
        cout << "Bitte geben Sie nur Zahlen ein" << endl << endl;
    }
}

habe jetzt mal die funktion mit einem bool wert erweitert und nun zeigt der Taschenrechner auch Ergebnisse an wenn eine Null rauskommt :-).
Nur noch die Erkennung mit den Buchstaben läuft nicht so wie ich das will...

Überleg dir im Kopf einfach immer ein Beispiel-Szenario und geh durch, wie dein Programm für dieses abläuft.

Im konkreten Beispiel: Deine bedingung-funktion überprüft zwar den stream auf Fehleingabe und weist den User darauf hin, aber lässt das Programm und die Eingabe dann in Ruhe weiterlaufen.
Desweiteren macht es keinen Sinn, dass diese Funktion die zwei double mitbekommt, wenn sie überhaupt nichts damit anstellt, meinst Du nicht?

Zudem ist dein Programm einfach ... nicht stimmig, imho. Du selber sagst schon, dass du teilweise nicht mehr durchblickst. Und das wundert nicht, wenn du aus so einem simplem Programm so ein Wirr-Warr machst.

Reagiere in den entsprechenden case-marken mit der Ausrechnung des Ergebnises.
So wie du es momentan machst ist es erstens falsch und zweitens gezwungen allgemein, was hier einfach keinen Sinn ergibt.

Ob man das Einlesen der Variabeln von der Berechnung trennen will, obwohl in den zwei unterschiedlichen Fällen eh unterschiedlicher Text ausgeben und unterschiedlich eingelesen wird ist auch zweifelhaft.

bool bedingung()
{
    bool wahr = 0;
    if (cin.fail() || cin.bad())
    {
        cin.clear();
        cin.ignore();
        wahr = 1;
        cout << "Es sind nur Zahlen gueltig!!!" << endl << endl;
    }
  return wahr;
}

und das einsetzen in eine andere funktion:

void textgrundrechenarten(double &a , double &b)
{
    do {
        cout << "Geben Sie bitte Zahl 1 ein" << endl;
        cin >> a;
        } while (bedingung());
    do {
        cout << "Geben Sie bitte Zahl 2 ein" << endl;
        cin >> b;
       } while (bedingung());

}

scheint schon gut zu klappen...
aber wenn man z.B ddd eingibt kommt die antwort 3 mal^^
bezüglich das einlesen der Variablen in der einen funktion und das ausrechnen in der Anderen, ich dachte das wäre besser, da der text immer gleich bleibt und ich den Rechner ja noch um ein paar Rechenarten erweitern möchte...
Berichtigt mich wenn ich falsch liege

Wenn ich das richtig verstanden habe, ist das ja das Problem in der grundrechen'Dings' Funktion.

LG