4. C Code Korrekturlesen bitte, besonders Menüstruktur! einfacher "Taschenrechner"

C Code Korrekturlesen bitte, besonders Menüstruktur! einfacher "Taschenrechner"

  • C

    vielen lieben Dank! 🙂 Ich freu mich wirklich riesig hier so klasse Unterstützung zu kriegen! Ich finde die Idee mit diesem Code weiterzuarbeiten gar nicht so schlecht, einfach das Programm erweitern etc. Habe auch wieder ein paar neue Ideen was ich so machen könnte. 🙂 Herzlichen Dank!

    Grüße

    0
  • C

    soo, ich habe mich nochmal ans Werk gemacht und wollte jetzt erreichen, dass ich eine Funktion auswähle (addieren, subtrahieren usw.) und dann beliebig viele Zahlen eingeben kann und die Eingabe mit der Eingabe von 0 beende. Ich habe bis jetzt nur diese Programmschnipsel:

    #include <stdio.h>
#include <stdlib.h>              

int Hauptmenue();
double Addition();
double Eingabe();

int main()
{
    printf("\tTaschenrechner Version 4\n\n");

    while(Hauptmenue() != 0)
    {
      Hauptmenue();
    }

    printf("Auf Wiedersehen");

    system("PAUSE");
    return 0;
}

int Hauptmenue()
{
    char wahl;

    printf("Hallo, welche Rechenoperation wollen Sie ausfuehren?\n\n");
    printf("****************************************************\n\n");
    printf("\t[a] - Addition\n");
    printf("\t[b] - Subtrakion\n");
    printf("\t[c] - Multiplikation\n");
    printf("\t[d] - Division\n");
    printf("**************************\n");
    printf("\t[x] - Beenden\n\n");
    printf("Ihre Wahl: ");

    scanf("%c",&wahl);
    getchar();

    printf("\n\nBitte Zahlen eingeben, wenn Eingabe vollendet, 0 eintippen\n\n");

    switch (tolower(wahl))
    {
           case 'a' : Addition();
                    break;
           case 'b' : 
                    break;
           case 'c' :
                    break;
           case 'd' :
                    break;
           case 'x' : return 0;

           default  : printf("Falsche Eingabe!!\n\n");
                      return 1;
    }

}

double Eingabe()
{
       double zahl=0;

       printf("Zahl eingeben: ");
       scanf("%f",&zahl);
       getchar();

       printf("\nZahl = %f\n",zahl);

       return zahl;
}

double Addition()
{
       double ergebnis1=0;
       double ergebnis2=0;

       do {
           ergebnis1 = Eingabe();
           ergebnis2 += ergebnis1;

           printf("Zwischenergebnis: %f\n",ergebnis2);
           }       
       while (ergebnis1 !=0);

       printf("\nDas Ergebnis lautet: %f\n\n",ergebnis2);

       return 0;
}

    Allerdings scheint die Eingabe der Zahl nicht zu gelingen, er speichert sie nicht oder sowas in der Art. Erkennt jemand meinen Fehler? Vielleicht ist es aber auch einfach nur zu spät grade 😃 Kann ich das ganze so mit der do-while schleife überhaupt anstellen oder ist das wieder viel zu umständlich?! 🙂

    Grüße

    0
  • ?

    chmbw schrieb:

    Allerdings scheint die Eingabe der Zahl nicht zu gelingen, er speichert sie nicht oder sowas in der Art

    %f ist für float. Für double musst du %lf nehmen.

    chmbw schrieb:

    Kann ich das ganze so mit der do-while schleife überhaupt anstellen oder ist das wieder viel zu umständlich?! 🙂

    Ich find das nicht umständlich. Klarer wird's halt immer, wenn die Namen der Variablen was mit iherer Funktion zu tun haben.

    while(Hauptmenue() != 0)
    {
      Hauptmenue();
    }

    Jetzt kannst du nur jedes zweite Mal abbrechen. Sonst wird's schon passen. Bis auf Eingabefehler natürlich, aber die sind langweilig. Andererseits bräuchtest du die nur in Eingabe() abfangen. Bei Interesse zB hier weiterlesen:
    http://www.c-plusplus.net/forum/viewtopic-var-t-is-39349-and-start-is-1.html
    🙂

    0
  • C

    µngbd schrieb:

    Jetzt kannst du nur jedes zweite Mal abbrechen. Sonst wird's schon passen. Bis auf Eingabefehler natürlich, aber die sind langweilig. Andererseits bräuchtest du die nur in Eingabe() abfangen. Bei Interesse zB hier weiterlesen:
    http://www.c-plusplus.net/forum/viewtopic-var-t-is-39349-and-start-is-1.html
    🙂

    int c; // int c, nicht char c.
while ((c = getchar()) != EOF && c != '\n'); // kein {} block, stimmt s

    also damit funktionierts wunderbar! 🙂 nur irgendwie habe ich den Ausdruck noch nicht ganz verstanden:

    ich definiere mit ein INTEGER c, warum? c ist doch 99 im ASCII-Code. Der Sinn erschließt sich mir nicht. Wieso hat die while-schleife keinen Block? In ihrer Bedigung wird c solange ein character zugewiesen (aber als integer gespeichert?!) solange != EOF (EOF = End Of File, aber was bedeutet das hier genau?? Ende der Funktion?) Ich wäre sehr dankbar wenn mir das einer ein wenig weiter aufdröseln könnte 🙂 Danke im Voraus!

    Grüße

    0
  • G

    EOF = End of file. Das wird oft durch einen negativen Wert dargestellt, daher ist es gefährlich da einen char zu benutzen, der oft unsigned ist.

    0
  • ?

    chmbw schrieb:

    Wieso hat die while-schleife keinen Block?

    Das sollte man sicherheitshalber so schreiben:

    while ((c = getchar()) != EOF && c != '\n')
    ;

    Dann kann man sich den Kommentar sparen, weil das ; ins Auge springt. Das ist eine leere Anweisung, die tut einfach nichts. Die muss dort sein, weil die nächste Anweisung (oder der nächste Block) eben zum while gehört, so ist die Syntax. Genauso solltest du sagen:

    while(Hauptmenue() != 0)
    ;
// und weil Vergleiche auf != 0 redundant sind, sagt man oft stattdessen
while(Hauptmenue())
    ;

    chmbw schrieb:

    EOF = End Of File, aber was bedeutet das hier genau?? Ende der Funktion?

    EOF heisst immer End Of File. Die Standard-Eingabe ist auch nur eine Datei. Damit fängt man den Fall ab, dass die Eingabe zu Ende ist. Das kann passieren, wenn man eine Datei als Eingabe an das Programm leitet (zB type file.txt | program auf Windows), oder wenn man explizit die Eingabe beendet, das geht auf den meisten Terminals mit Strg-D oder Strg-Z oder so.

    chmbw schrieb:

    aber als integer gespeichert?!

    Ein char ist auch nur eine Ganzzahl, aber der Wertebereich ist meistens kleiner. 'A' ist nur eine andere Weise, um 65 zu sagen (ASCII vorausgesetzt). EOF darf aber kein gültiges Zeichen sein, deshalb muss man für die Rückgabe von getchar() einen grösseren Wertebereich nehmen.
    🙂

    0
  • ?

    EOF darf aber kein gültiges Zeichen sein, deshalb muss man für die Rückgabe von getchar() einen grösseren Wertebereich nehmen.

    Da fällt mir ein, dass fricky mal erzählt hat, dass auf 8-Bit-Dingern ein int trotzdem 16 Bit hat. Ich hab mir damals gedacht, dass 8 Bits für eine Zahl wohl doch recht wenig wären, und mir das so erklärt. Aber wenn ein char so gross wie ein int wäre, würde getchar() nicht mehr funktionieren, oder?

    0
  • ?

    µngbd schrieb:

    Da fällt mir ein, dass fricky mal erzählt hat, dass auf 8-Bit-Dingern ein int trotzdem 16 Bit hat.

    bezogen darauf, dass ich noch nie 'nen compiler gesehen habe, der 8-bittige ints hat (wohl aber einen, wo man das einstellen konnte, default war aber auch bei dem 16-bit int). im C-std findeste etwa sowas:

    — minimum value for an object of type int
INT_MIN -32767 // -(2^15 - 1)
— maximum value for an object of type int
INT_MAX +32767 // 2^15 -1

    und wieder mal sind die standard auswendigkenner gefragt: ist ein 8-bit int laut C-standard theoretisch möglich?
    🙂

    0
  • V

    zder davebardsley in http://www.linuxforums.org/forum/linux-programming-scripting/44480-sizeof-int.html bestätigt dich und der link http://en.wikipedia.org/wiki/C_variable_types_and_declarations#Size sagt auch "An int must be at least 16 bits long.".

    0
  • G

    Nicht wenn ich mit 8Bit ~65000 Zahl darstellen kann! Ich nehm einfach einen komprimierungsalgorithmus... 🤡

    0
  • V

    Genmutant schrieb:

    Nicht wenn ich mit 8Bit ~65000 Zahl darstellen kann! Ich nehm einfach einen komprimierungsalgorithmus... 🤡

    Viel Erfolg http://en.wikipedia.org/wiki/Minimum_message_length

    0
  • ?

    Genmutant schrieb:

    Nicht wenn ich mit 8Bit ~65000 Zahl darstellen kann!

    musste passend runden, 0...256...512...768... usw., dann klappt das *fg*
    🙂

    0
  • V

    ;fricky schrieb:

    Genmutant schrieb:

    Nicht wenn ich mit 8Bit ~65000 Zahl darstellen kann!

    musste passend runden, 0...256...512...768... usw., dann klappt das *fg*
    🙂

    Geht sogar noch komprimierter. Sozusagen am komprimiertesten:

    int passendRunden(int x){
   if(x==0)
      return 0;
   else
      return x/2;
}

    oder etwas langsamer aber nicht rekursiv:

    int passendRunden(int x){
   while(x*=3);
      x*=5;
}
    0
Anmelden zum Antworten