[C#] Taschenrechner-Konsole

[Snees]

Nabend, Saad hat mich gefragt ob ich ihm ein einfaches Beispiel für einen Taschenrechner als Konsolenanwendung schreiben kann.
Das habe ich gerade gemacht, vielleicht kann ja noch jemand diesen Code gebrauchen

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Taschenrechner
{
    class Program
    {
        static void Main(string[] args)
        {
            // Taschenrechner-Konsole Beispiel für Saad

            //Der Konsole einen Titel zuweisen
            Console.Title = "Taschenrechner-Konsole für Saad";
            //Dem Benutzer sagen, dass er die Rechenmethode wählen soll
            Console.WriteLine("Bitte wähle die Rechenmethode (+,-,*,/)");
            //Die Rechenmethode auslesen und in einem String speichern
            string rechenmethode = Console.ReadLine();
            //Nun erwarten wir von dem Benutzer der Eingaben der ersten Zahl
            Console.WriteLine("Bitte gib die erste Zahl an.");
            //Eingabe in einem String speichern 
            string input = Console.ReadLine();
            //und in einen Integer konvertieren
            //würde man Convert.ToInt64 benutzen, würde man die Zahl in einem Long speichern &
            //würde man Convert.ToInt16 benutzen, würde man die Zahl in einem Short speichern
            // dabei bin ich mir aber nicht ganz sicher
            //Convert.ToInt32 speichert die Zahl auf jeden Fall in einem Integer
            //Man müsste noch prüfen ob wirklich eine Zahl eingegeben wurde um mögliche auftretende Fehler zu behandeln
            //das lasse ich aber hier weg, weil es klar ist, dass man eine Zahl eingeben muss
            int zahl1 = Convert.ToInt32(input);
            //Nun die zweite Zahl eingeben lassen, ansonsten gilt das gleiche wie bei der ersten Zahl
            Console.WriteLine("Bitte gib die zweite Zahl an.");
            string inp = Console.ReadLine();
            int zahl2 = Convert.ToInt32(inp);
            //Nun noch dem Benutzer ausgeben, was gerechnet wird. Muss man nicht machen, kann man aber
            Console.WriteLine("Rechnung: " + zahl1 + " " + rechenmethode + " " + zahl2);
            //Integer Ergebis erstmal auf 0 setzen, wir weisen später einen Wert zu
            int ergebnis = 0;
            //switch-Block um nun die ausgewählte Rechenmethode anzuwenden
            //Man KÖNNTE auch if-Abfragen benutzen, so ist es aber einfacher und schöner
            switch (rechenmethode)
            {
                //case(EN) = Fall(DE)
                    //Sollte also der Fall eintreten, dass der Benutzer + als Rechenmethode angegeben hat
                    //führen wir folgenden Code aus
                case "+":
                    //Hier weisen wir dem Integer ergebnis einen Wert zu, nämlich zahl1 + zahl2
                    ergebnis = zahl1 + zahl2;
                    //und schließen mit break; ab
                    break;
                    //Genau das gleiche gilt auch für die anderen Fälle

                case "-":
                    ergebnis = zahl1 - zahl2;
                    break;

                case "*":
                    ergebnis = zahl1 * zahl2;
                    break;

                case "/":
                    ergebnis = zahl1 / zahl2;
                    break;

                    //zum Schluss benutze ich noch default
                    //sollte der Benutzer als weder +, -, * oder / angegeben haben wird der nachfolgende Code aufgerufen
                default:
                    Console.WriteLine("Falsche Rechenmethode ausgewählt.");
                    break;
            }
            //Das Ergenis ausgeben
            Console.WriteLine("Ergenis: " + ergebnis);
            //Und auf das Drücken einer Warten
            Console.ReadKey();
        }
    }
}

[Brainy]

Warum so umständlich für den Benutzer?
Es sollte eine Eingabe wie 2 - 4 + 9 - 12 + 7 möglich sein.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;

namespace Minirechner {
    class Program {
        static int operation;
        static void Main(string[] args) {
            Console.Write("Aufgabe: ");
            Console.WriteLine(Calculate(Console.ReadLine()));
            Console.Write("<Drücke eine Taste, um die Konsole zu schließen..>");
            Console.ReadKey(true);
        }

        static double Calculate(string input) {
            operation = -1;
            double res = 0;
            int index;
            while ((index = input.IndexOf(' ')) != -1) {
                double num;
                string p = input.Substring(0, index);
                if (!double.TryParse(p, out num)) {
                    switch (p) {
                        case "+":
                            operation = 1;
                            break;

                        case "-":
                            operation = 2;
                            break;

                        case "*":
                            operation = 3;
                            break;

                        case "/":
                            operation = 4;
                            break;
                    }
                }
                else {
                    if (operation == -1) {
                        res = num;
                    }
                    else {
                        switch (operation) {
                            case 1: res += num; break;
                            case 2: res -= num; break;
                            case 3: res *= num; break;
                            case 4: res /= num; break;
                        }
                    }
                }

                input = input.Substring(index + 1);
            }

            double last = double.Parse(input);

            switch (operation) {
                case 1: return res + last;
                case 2: return res - last;
                case 3: return res * last;
                case 4: return res / last;
                default: return -1;
            }
        }
    }
} 

Hab ich mal eben geschrieben, ist aber meiner Meinung nach nicht die sauberste Lösung, aber es geht.

MfG

[Mr. White]

Hab ich mal eben geschrieben, ist aber meiner Meinung nach nicht die sauberste Lösung, aber es geht.

http://www.abload.de/img/unbenanntq5zml.png
Richtig wäre 35.

Ich habe mal einen Recursive Descent Parser für mathematische Ausdrücke in C++ geschrieben, der u.A die Regeln (Klammern, Punkt vor Strich) einhält und auch mit Konstanten zurecht kommt (PI, e, ...). Wenn ich Zeit finde, portiere ich das nach C# und poste es hier.