WPF Taschenrechner -- Crash nach Backspace

Kirk1701A · Verfasst: Di 10.10.17 07:31

Hey Leute,

Ich bin Azubi im ersten Jahr zum Fachinformatiker. Ich bin grade dabei, ein Taschenrechner zu bauen. Allerdings Crashed der, wenn ich mit Backspace ein Zeichen lösche und ein anderes wieder einsetzen möchte (hauptsächlich bei Operatoren).
Hier mein XAML-Code:

markieren

XML-Daten

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:

<Window x:Class="WpfApplication2.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:Taschenrechner_3"
        mc:Ignorable="d"
        Title="Rechner" Height="350" Width="300" KeyDown="Window_KeyDown" MinWidth="300" MinHeight="350" MaxWidth="600" MaxHeight="650">
    <Grid>
        <Grid.RowDefinitions>
            <!--Zeilen-->
            <RowDefinition Height="16*"/>
            <RowDefinition Height="32*"/>
            <RowDefinition Height="10*"/>
            <RowDefinition Height="40*"/>
            <RowDefinition Height="10*"/>
            <RowDefinition Height="40*"/>
            <RowDefinition Height="10*"/>
            <RowDefinition Height="40*"/>
            <RowDefinition Height="10*"/>
            <RowDefinition Height="40*"/>
            <RowDefinition Height="10*"/>
            <RowDefinition Height="40*"/>
            <RowDefinition Height="10*"/>
        </Grid.RowDefinitions>
        <Grid.ColumnDefinitions>
            <ColumnDefinition Width="28*"/>
            <ColumnDefinition Width="40*"/>
            <ColumnDefinition Width="10*"/>
            <ColumnDefinition Width="40*"/>
            <ColumnDefinition Width="10*"/>
            <ColumnDefinition Width="40*"/>
            <ColumnDefinition Width="10*"/>
            <ColumnDefinition Width="40*"/>
            <ColumnDefinition Width="30*"/>
        </Grid.ColumnDefinitions>
        <Button x:Name="eins" Content="1" FontSize="16" FontWeight="Bold" Grid.Column="1" Grid.Row="3" Click="eins_click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="zwei" Content="2" FontSize="16" FontWeight="Bold" Grid.Column="3" Grid.Row="3" Click="zwei_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="drei" Content="3" FontSize="16" FontWeight="Bold" Grid.Column="5" Grid.Row="3" Click="drei_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="plus" Content="+" FontSize="16" FontWeight="Bold" Grid.Column="7" Grid.Row="3" Click="plus_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="vier" Content="4" FontSize="16" FontWeight="Bold" Grid.Column="1" Grid.Row="5" Click="vier_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="fünf" Content="5" FontSize="16" FontWeight="Bold" Grid.Column="3" Grid.Row="5" Click="fünf_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="sechs" Content="6" FontSize="16" FontWeight="Bold" Grid.Column="5" Grid.Row="5" Click="sechs_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="minus" Content="-" FontSize="16" FontWeight="Bold" Grid.Column="7" Grid.Row="5" Click="minus_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="sieben" Content="7" FontSize="16" FontWeight="Bold" Grid.Column="1" Grid.Row="7" Click="sieben_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="acht" Content="8" FontSize="16" FontWeight="Bold" Grid.Column="3" Grid.Row="7" Click="acht_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="neun" Content="9" FontSize="16" FontWeight="Bold" Grid.Column="5" Grid.Row="7" Click="neun_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="mal" Content="*" FontSize="16" FontWeight="Bold" Grid.Column="7" Grid.Row="7" Click="mal_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="clean" Content="C" FontSize="16" FontWeight="Bold" Grid.Column="1" Grid.Row="9" Click="clean_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="null" Content="0" FontSize="16" FontWeight="Bold" Grid.Column="3" Grid.Row="9" Click="null_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="komma" Content="." FontSize="16" FontWeight="Bold" Grid.Column="5" Grid.Row="9" Click="komma_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="teil" Content="/" FontSize="16" FontWeight="Bold" Grid.Column="7" Grid.Row="9" Click="teil_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="gleich" Content="=" FontSize="16" FontWeight="Bold" Grid.Column="1" Grid.ColumnSpan="3" Grid.Row="11" Cursor="Hand" Click="gleich_Click" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <Button x:Name="lösch" Content="f" FontSize="16" FontWeight="Bold" FontFamily="Wingdings 3" Grid.Column="5" Grid.ColumnSpan="3" Grid.Row="11" Click="lösch_Click" Cursor="Hand" BorderBrush="Black" IsTabStop="False" Focusable="False"/>
        <TextBox x:Name="rechner" BorderBrush="Black" Background="Cyan" FontSize="17" HorizontalContentAlignment="Right" Grid.Column="3" Grid.ColumnSpan="5" Grid.Row="1" TextAlignment="Right" MaxLines="1" FontFamily="Verdana" FontWeight="Bold" IsTabStop="False" Focusable="False"/>
        <TextBox x:Name="op" BorderBrush="Black" Background="Cyan" FontSize="17" HorizontalContentAlignment="Right" Grid.Column="3" Grid.ColumnSpan="5" Grid.Row="1" TextAlignment="Right" MaxLines="1" FontFamily="Verdana" FontWeight="Bold" IsTabStop="False" Focusable="False" Visibility="Hidden"/>
        <TextBox Name="p" Grid.Column="1" Grid.Row="1" Background="#FFF3F3F3" FontSize="10" TextWrapping="Wrap" TextOptions.TextHintingMode="Fixed" TextOptions.TextFormattingMode="Display" IsEnabled="False" />
    </Grid>
</Window>

Und hier mein C#:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:
61:
62:
63:
64:
65:
66:
67:
68:
69:
70:
71:
72:
73:
74:
75:
76:
77:
78:
79:
80:
81:
82:
83:
84:
85:
86:
87:
88:
89:
90:
91:
92:
93:
94:
95:
96:
97:
98:
99:
100:
101:
102:
103:
104:
105:
106:
107:
108:
109:
110:
111:
112:
113:
114:
115:
116:
117:
118:
119:
120:
121:
122:
123:
124:
125:
126:
127:
128:
129:
130:
131:
132:
133:
134:
135:
136:
137:
138:
139:
140:
141:
142:
143:
144:
145:
146:
147:
148:
149:
150:
151:
152:
153:
154:
155:
156:
157:
158:
159:
160:
161:
162:
163:
164:
165:
166:
167:
168:
169:
170:
171:
172:
173:
174:
175:
176:
177:
178:
179:
180:
181:
182:
183:
184:
185:
186:
187:
188:
189:
190:
191:
192:
193:
194:
195:
196:
197:
198:
199:
200:
201:
202:
203:
204:
205:
206:
207:
208:
209:
210:
211:
212:
213:
214:
215:
216:
217:
218:
219:
220:
221:
222:
223:
224:
225:
226:
227:
228:
229:
230:
231:
232:
233:
234:
235:
236:
237:
238:
239:
240:
241:
242:
243:
244:
245:
246:
247:
248:
249:
250:
251:
252:
253:
254:
255:
256:
257:
258:
259:
260:
261:
262:
263:
264:
265:
266:
267:
268:
269:
270:
271:
272:
273:
274:
275:
276:
277:
278:
279:
280:
281:
282:
283:
284:
285:
286:
287:
288:
289:
290:
291:
292:
293:
294:
295:
296:
297:
298:
299:
300:
301:
302:
303:
304:
305:
306:
307:
308:
309:
310:
311:
312:
313:
314:
315:
316:
317:
318:
319:
320:
321:
322:
323:
324:
325:
326:
327:
328:
329:
330:
331:
332:
333:
334:
335:
336:

using System.Windows;
using System.Windows.Input;

namespace WpfApplication2
{
    /// <summary>
    /// Interaktionslogik für MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {

        //Variablen zum Rechnen
        double erg1;
        double erg2;

        public MainWindow()
        {
            InitializeComponent();
        }

        //Die nächsten 18 Methoden sind die Buttons, die 1, 2, 3, ... + - * / ... ist-gleich, Backspace, ... schreiben
        private void eins_click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "1";
            p.Text += "1";
        }

        private void zwei_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "2";
            p.Text += "2";
        }

        private void drei_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "3";
            p.Text += "3";
        }

        private void plus_Click(object sender, RoutedEventArgs e)
        {
            op.Text = "+";
            p.Text += "+";
            erg1 += double.Parse(rechner.Text);
            rechner.Clear();
        }

        private void vier_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "4";
            p.Text += "4";
        }

        private void fünf_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "5";
            p.Text += "5";
        }

        private void sechs_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "6";
            p.Text += "6";
        }

        private void minus_Click(object sender, RoutedEventArgs e)
        {
            if (op.Text == "")
            {
                p.Text += "-";
                rechner.Text += "-";
                op.Text = "-";
            }
            else
            {
                p.Text += "-";
                op.Text = "-";
                erg1 += double.Parse(rechner.Text);
                rechner.Clear();
            }
        }

        private void sieben_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "7";
            p.Text += "7";
        }

        private void acht_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "8";
            p.Text += "8";
        }

        private void neun_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "9";
            p.Text += "9";
        }

        private void mal_Click(object sender, RoutedEventArgs e)
        {
            op.Text = "*";
            p.Text += "*";
            erg1 += double.Parse(rechner.Text);
            rechner.Clear();
        }

        private void clean_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text = "";
            p.Text = "";
        }

        private void null_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += "0";
            p.Text += "0";
        }

        private void komma_Click(object sender, RoutedEventArgs e)
        {
            rechner.Text += ",";
            p.Text += ",";
        }

        private void teil_Click(object sender, RoutedEventArgs e)
        {
            op.Text = "/";
            p.Text += "/";
            erg1 += double.Parse(rechner.Text);
            rechner.Clear();
        }

        private void lösch_Click(object sender, RoutedEventArgs e)
        {
            if (rechner.Text == "")
            {
                p.Text = p.Text.Remove(p.Text.Length - 1, 1);
            }
            else
            {
                rechner.Text = rechner.Text.Remove(rechner.Text.Length - 1, 1);
                p.Text = p.Text.Remove(p.Text.Length - 1, 1);
            }
        }

        private void gleich_Click(object sender, RoutedEventArgs e)
        {
            //Rechnungsfunktion
            if (op.Text == "+")
            {
                erg2 = erg1 + double.Parse(rechner.Text);
                rechner.Text = erg2.ToString();
                erg1 = 0;
            }
            else if (op.Text == "-")
            {
                erg2 = erg1 - double.Parse(rechner.Text);
                rechner.Text = erg2.ToString();
                erg1 = 0;
            }
            else if (op.Text == "*")
            {
                erg2 = erg1 * double.Parse(rechner.Text);
                rechner.Text = erg2.ToString();
                erg1 = 0;
            }
            else if (op.Text == "/")
            {
                erg2 = erg1 / double.Parse(rechner.Text);
                rechner.Text = erg2.ToString();
                erg1 = 0;
            }
        }

        private void Window_KeyDown(object sender, KeyEventArgs e)
        {
            //Tastenschreiben...
            if (e.Key == Key.NumPad1)
            {
                rechner.Text += "1";
                p.Text += "1";
            }

            else if (e.Key == Key.NumPad2)
            {
                rechner.Text += "2";
                p.Text += "2";
            }

            else if (e.Key == Key.NumPad3)
            {
                rechner.Text += "3";
                p.Text += "3";
            }

            else if (e.Key == Key.NumPad4)
            {
                rechner.Text += "4";
                p.Text += "4";
            }

            else if (e.Key == Key.NumPad5)
            {
                rechner.Text += "5";
                p.Text += "5";
            }

            else if (e.Key == Key.NumPad6)
            {
                rechner.Text += "6";
                p.Text += "6";
            }

            else if (e.Key == Key.NumPad7)
            {
                rechner.Text += "7";
                p.Text += "7";
            }

            else if (e.Key == Key.NumPad8)
            {
                rechner.Text += "8";
                p.Text += "8";
            }

            else if (e.Key == Key.NumPad9)
            {
                rechner.Text += "9";
                p.Text += "9";
            }

            else if (e.Key == Key.NumPad0)
            {
                rechner.Text += "0";
                p.Text += "0";
            }

            else if (e.Key == Key.Add)
            {
                p.Text += "+";
                op.Text = "+";
                erg1 += double.Parse(rechner.Text);
                rechner.Clear();
            }

            else if (e.Key == Key.Subtract)
            {
                if (op.Text == "")
                {
                    p.Text += "-";
                    rechner.Text += "-";
                    op.Text = "-";
                }
                else
                {
                    p.Text += "-";
                    op.Text = "-";
                    erg1 += double.Parse(rechner.Text);
                    rechner.Clear();
                }
            }

            else if (e.Key == Key.Multiply)
            {
                p.Text += "*";
                op.Text = "*";
                erg1 += double.Parse(rechner.Text);
                rechner.Clear();
            }

            else if (e.Key == Key.Divide)
            {
                p.Text += "/";
                op.Text = "/";
                erg1 += double.Parse(rechner.Text);
                rechner.Clear();
            }

            else if (e.Key == Key.Decimal)
            {
                rechner.Text += ",";
                p.Text += ",";
            }

            else if (e.Key == Key.Back)
            {
                if (rechner.Text == "")
            {
                op.Text = op.Text.Remove(op.Text.Length - 1, 1);
            }
            else
            {
                rechner.Text = rechner.Text.Remove(rechner.Text.Length - 1, 1);
                    op.Text = op.Text.Remove(op.Text.Length - 1, 1);
                }
            }

            else if (e.Key == Key.Delete)
            {
                rechner.Text = "";
                p.Text = "";
            }

            else if (e.Key == Key.Return)
            {
                if (op.Text == "+")
                {
                    erg2 = erg1 + double.Parse(rechner.Text);
                    rechner.Text = erg2.ToString();
                    erg1 = 0;
                }
                else if (op.Text == "-")
                {
                    erg2 = erg1 - double.Parse(rechner.Text);
                    rechner.Text = erg2.ToString();
                    erg1 = 0;
                }
                else if (op.Text == "*")
                {
                    erg2 = erg1 * double.Parse(rechner.Text);
                    rechner.Text = erg2.ToString();
                    erg1 = 0;
                }
                else if (op.Text == "/")
                {
                    erg2 = erg1 * double.Parse(rechner.Text);
                    rechner.Text = erg2.ToString();
                    erg1 = 0;
                }
            }
        }

    }
}

Danke für eure Hilfe.

Euer Kirk

Ehemaliges Mitglied Erhaltene Danke: 1

- Nachträglich durch die Entwickler-Ecke gelöscht -

Kirk1701A · Verfasst: Di 10.10.17 08:18

Hi,

Der Cursor ist im unteren Beispiel [CURSOR].

[CURSOR]erg1 += double.Parse(rechner.Text);

Egal ob "+ - * /" der bleibt immer an der Stelle beim Debuggen (und Backspace) stecken und crashed.

Euer Kirk

Moderiert von Th69: C#-Tags hinzugefügt

Ehemaliges Mitglied Erhaltene Danke: 1

- Nachträglich durch die Entwickler-Ecke gelöscht -

Kirk1701A · Verfasst: Di 10.10.17 08:29

Das is bei jedem Fragment so.

Also:

Wenn ich "3+3+" eingegeben habe, möchte ich das letzte + löschen (mit Backspace) und ein z.B. - einsetzen, also "3+3-...". Wenn ich das + gelöscht habe "3+3" und nun ein (in diesem Beispiel) - einsetzen möchte, crashed das Programm. Das is bei jeder Rechenart so und unabhängig davon, ob ich es mit Button oder Tastatur mache. Der stürzt immer ab. Am besten ist es, wenn du "3+3+" eingibst, mit Backspace-Button oder -Taste, das letzte + löschst, und stattdessen ein + oder - oder * oder / eingibst (auch egal ob mit Button oder Taste).

Hoffe, du verstehst das.

jaenicke · Verfasst: Di 10.10.17 08:42

Es gibt auch Double.TryParse, womit du nicht einfach in eine Zahl umwandelst, sondern nachschauen kannst, ob das überhaupt geht.

Denn wenn der Wert z.B. leer ist, schlägt die Umwandlung in einen Zahlenwert fehl.

Th69 · Verfasst: Di 10.10.17 08:45

Hallo und

wenn du beim Debuggen einen Fehler erhältst, dann deutet dies auf eine Exception hin (s.a. Ausnahmen und Ausnahmebehandlung).
In deinem Fall ist dies die double.Parse-Methode, welche eine Exception auslöst, weil wohl keine (einzelne) Zahl in der übergebenen Variablen rechner steht.
Sobald der Debugger eine (unbehandelte) Exception anzeigt, kannst du dir im "Überwachen"-Fenster (engl. "Watch window") die Variableninhalte anzeigen lassen (s. Fenster "Überwachen" und "Schnellüberwachung").

Verwende besser die double.TryParse-Methode, um evtl. Fehler abzufangen.

Kirk1701A · Verfasst: Di 10.10.17 08:51

Danke.

Ich probiers gleich mal aus und gebe euch dann eine Rückmeldung. :zustimm:

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Kirk1701A · Verfasst: Di 10.10.17 09:40

Langsam, aber sicher bin ich mit der Sache überfordert.

Ich habe das erstmal am "plus_Click" probiert, komme aber irgendwie nicht weiter. :lupe:

Hier der "plus_Click"-Code, nach aktuellem Stand.

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:

private void plus_Click(object sender, RoutedEventArgs e)
        {
            double t;
            op.Text = "+";
            p.Text += "+";
            erg1 = double.TryParse("+", out t);
            rechner.Clear();
        }

VS sagt mir: "Der Typ 'bool' kann nicht implizit in 'double' konvertiert werden."
Ich habe es, bevor ich das TryParse genommen habe, damit versucht, dass, wenn Backspace gemacht wird, dass op(also da, wo die Operatoren sitzen) gereinigt wird. Das hat der auch gemacht, aber als ich dann einen Operatoren einsetzen wollte, hat es trotzdem nicht funktioniert. :autsch:

Was allerdings komisch ist, ist, dass es vor 2 Tagen funktioniert hat. Und da sah der Code wie gaanz oben aus (Eröffnungsbeitrag). :autsch:

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Th69 · Verfasst: Di 10.10.17 10:13

Die Methode funktioniert etwas anders als die Parse: sie gibt einen bool-Wert zurück, der angibt, ob die Umwandlung erfolgreich war:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:

if (double.TryParse(rechner.Text, out erg1))
{
  // Umwandlung erfolgreich und in 'erg1' steht der Zahlenwert
}
else
{
  // Umwandlung nicht erfolgreich -> z.B. Fehler ausgeben
}

Der untere Fall sollte natürlich bei deinem Programm niemals auftreten, aber so kannst du nach und nach die logischen Fehler bei deinem Programmcode korrigieren. ;-)

Kirk1701A · Verfasst: Di 10.10.17 10:27

Wow, Danke.
Allerdings wirft der mir was ganz anderes raus, was ich will. z.B. "3/2" ist eigentlich 1,5; aber der zeigt mir "2" an.

Ich weiss nicht, ob Ihr das wisst (bitte nicht persönlich nehmen, ok? :eyes:

), aber "rechner" ist das Textfeld bzw. die TextBox, in der hineingeschrieben wird und das Ergebnis am Ende steht.
Ich zeig mal schnell, was ich für Mist fabriziert habe.

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:

 private void teil_Click(object sender, RoutedEventArgs e)
        {
            op.Text = "/";
            p.Text += "/";

            if (double.TryParse(rechner.Text, out erg1))
            {
                erg1 += double.Parse(rechner.Text);
                rechner.Clear();
            }
            else
            {
                rechner.Text = "ERROR!";
            }
        }

Ich weiss echt nicht mehr weiter. :autsch:

Moderiert von Christian S.: Beiträge zusammengefasst

oder habe ich das falsch verstanden?

Moderiert von Christian S.: Beiträge zusammengefasst

Was ich vergessen habe, ist, dass wenn ich den oben beschriebenen Fehler mal ausführe (also bei "3+3+" das letzte + wegmachen und hier ein / einsetze), kommt "ERROR!".

D.h. der kann das nicht machen, oder?

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Ralf Jansen · Verfasst: Di 10.10.17 10:38

Zitat:

Allerdings wirft der mir was ganz anderes raus, was ich will. z.B. "3/2" ist eigentlich 1,5; aber der zeigt mir "2" an.

Parse/TryParse rechnen nicht. Es wandelt nur einen string in einen anderen Typ zum Beispiel double um. Wenn du irgendwo 3/2 gerechnet hast und dabei 2 raus kommt dann hast du eine Ganzzahl Division durchgeführt und keine Floatingpoint Division. Das Problem liegt dann aber vorher und nicht beim Parse bzw. TryParse.

Kirk1701A · Verfasst: Di 10.10.17 10:45

Kurioser Weise hat mein Taschenrechner erst so gut gerechnet, seit ich die parse-Methode habe. Ich habe auch noch eine andere Version eines Taschenrechners, an der ich gearbeitet habe, die allerdings keine längeren Terme (Rechnungen) zulässt. Also: "3+3" geht, aber "3+3+3" geht schon nicht mehr.

Alte Version eines Taschenrechners:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:
40:
41:
42:
43:
44:
45:
46:
47:
48:
49:
50:
51:
52:
53:
54:
55:
56:
57:
58:
59:
60:
61:
62:
63:
64:
65:
66:
67:
68:
69:
70:
71:
72:
73:
74:
75:
76:
77:
78:
79:
80:
81:
82:
83:
84:
85:
86:
87:
88:
89:
90:
91:
92:
93:
94:
95:
96:
97:
98:
99:
100:
101:
102:
103:
104:
105:
106:
107:
108:
109:
110:
111:
112:
113:
114:
115:
116:
117:
118:
119:
120:
121:
122:
123:
124:
125:
126:
127:
128:
129:
130:
131:
132:
133:
134:
135:
136:
137:
138:
139:
140:
141:
142:
143:
144:
145:
146:
147:
148:
149:
150:
151:
152:
153:
154:
155:
156:
157:
158:
159:
160:
161:
162:
163:
164:
165:
166:
167:
168:
169:
170:
171:
172:
173:
174:
175:
176:
177:
178:
179:
180:
181:
182:
183:
184:
185:
186:
187:
188:
189:
190:
191:
192:
193:
194:
195:
196:
197:
198:

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using System.Windows;
using System.Windows.Controls;
using System.Windows.Data;
using System.Windows.Documents;
using System.Windows.Input;
using System.Windows.Media;
using System.Windows.Media.Imaging;
using System.Windows.Navigation;
using System.Windows.Shapes;

namespace Taschenrechner
{
    /// <summary>
    /// Interaktionslogik für MainWindow.xaml
    /// </summary>
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
        }

        //Schreiben mit den Buttons ("eins_Click" schreibt 1, ...)
        private void eins_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "1";
        }

        private double erg1;
        private double erg2;
        private void zwei_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "2";
        }

        private void plus_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "+";
        }

        private void drei_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "3";
        }

        private void vier_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "4";
        }

        private void fünf_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "5";
        }

        private void sechs_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "6";
        }

        private void minus_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "-";
        }

        private void sieben_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "7";
        }

        private void acht_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "8";
        }

        private void neun_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "9";
        }

        private void mult_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "*";
        }

        private void komma_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += ",";
        }

        private void null_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "0";
        }

        private void clean_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text = "";
        }

        private void teil_Click(object sender, RoutedEventArgs e)
        {
            this.Rechnerbox.Text += "/";
        }
        //Einzelne Zeichen löschen -> wie Backspace
        private void lösch_Click(object sender, RoutedEventArgs e)
        {
            if (Rechnerbox.Text.Length > 0)
            {
                Rechnerbox.Text = Rechnerbox.Text.Substring(0, Rechnerbox.Text.Length - 1);
            }
        }
        //Klammern...
        private void kla_Click(object sender, RoutedEventArgs e)
        {
            //...auf
            this.Rechnerbox.Text += "(";
        }

        private void klz_Click(object sender, RoutedEventArgs e)
        {
            //...zu
            this.Rechnerbox.Text += ")";
        }

        //Die Ist-Gleich-Funktion -> wenn's nicht funktioniert, kommt "Error!"
        private void gleich_Click(object sender, RoutedEventArgs e)
        {
            try
            {
                result(); //Verweist zum private void unten...
            }
            //Jetzt kommt Error
            catch (Exception exc)
            {
                Rechnerbox.Text = "Error!";
            }
        }
        //...Eingabe von + - * /
        private void result()
        {
            //Prüfung, wo sich die Operatoren befinden
            String op;
            int iOp = 0;
            if (Rechnerbox.Text.Contains("+"))
            {
                iOp = Rechnerbox.Text.IndexOf("+");
            }
            else if (Rechnerbox.Text.Contains("-"))
            {
                iOp = Rechnerbox.Text.IndexOf("-");
            }
            else if (Rechnerbox.Text.Contains("*"))
            {
                iOp = Rechnerbox.Text.IndexOf("*");
            }
            else if (Rechnerbox.Text.Contains("/"))
            {
                iOp = Rechnerbox.Text.IndexOf("/");
            }
            else
            {
                //Error
            }
            

            //Rechnung nachdem = gedrückt wurde
            op = Rechnerbox.Text.Substring(iOp, 1);
            double op1 = Convert.ToDouble(Rechnerbox.Text.Substring(0, iOp));
            double op2 = Convert.ToDouble(Rechnerbox.Text.Substring(iOp + 1, Rechnerbox.Text.Length - iOp - 1));
            
            
            //Operatorenprüfung
            if (op == "+") //Operator "Plus"
            {
                Rechnerbox.Text = "" + (op1 + op2);
            }
            else if (op == "-") //Operator "Minus"
            {
                Rechnerbox.Text = "" + (op1 - op2);
            }
            else if (op == "*") //Operator "Mal"
            {
                Rechnerbox.Text = "" + (op1 * op2);
            }
            else //Operator "Geteilt"
            {
                Rechnerbox.Text = "" + (op1 / op2);
            }
           
        }
    }
}

markieren

XML-Daten

1:
2:
3:
4:
5:
6:
7:
8:
9:
10:
11:
12:
13:
14:
15:
16:
17:
18:
19:
20:
21:
22:
23:
24:
25:
26:
27:
28:
29:
30:
31:
32:
33:
34:
35:
36:
37:
38:
39:

<Window x:Class="Taschenrechner.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        xmlns:local="clr-namespace:Taschenrechner"
        mc:Ignorable="d"
        Title="Taschenrechner" Height="420.784" Width="372.209" Background="#FF00D1FF" ResizeMode="CanMinimize" BorderBrush="Black" Foreground="Black">
    <Grid Margin="0,0,0,-21">
        <Grid.Background>
            <SolidColorBrush Color="#FF00E8FF"/>
        </Grid.Background>
        <TextBlock x:Name="Text" Height="35" Margin="9,10,0,0" HorizontalAlignment="Left" VerticalAlignment="Top" FontWeight="Bold" FontStyle="Italic" FontSize="22" Width="319"><Run Text="Taschenrechner (Ver. 1."/><Run Text="1"/><Run Text=")"/><Run Text=" Beta"/></TextBlock>
        <TextBox Name="Rechnerbox" Height="35" Margin="10,63,10,0" VerticalAlignment="Top" TextAlignment="Right" FontSize="19" BorderBrush="Black" MaxLines="1" Cursor="IBeam" FontWeight="Bold" SelectionBrush="{x:Null}" Background="White" Foreground="Black"/>
        <Grid Margin="22,121,29,31" VerticalAlignment="Center" HorizontalAlignment="Center">
            <Button x:Name="eins" Content="1" Height="40" Width="40" Margin="8,20,234,155" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="eins_Click" Cursor="Hand"/>
            <Button x:Name="zwei" Content="2" Height="40" Width="40" Margin="50,13,186,149" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="zwei_Click" Cursor="Hand"/>
            <Button x:Name="drei" Content="3" Height="40" Width="40" Margin="95,13,141,149" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="drei_Click" Cursor="Hand"/>
            <Button x:Name="plus" Content="+" Height="40" Width="40" Margin="135,0,90,136" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="plus_Click" Cursor="Hand"/>
            <Button x:Name="vier" Content="4" Height="40" Width="40" Margin="5,59,231,103" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="vier_Click"/>
            <Button x:Name="fünf" Content="5" Height="40" Width="40" Margin="50,59,186,103" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="fünf_Click"/>
            <Button x:Name="sechs" Content="6" Height="40" Width="40" Margin="98,66,144,109" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="sechs_Click"/>
            <Button x:Name="minus" Content="-" Height="40" Width="40" Margin="135,46,90,90" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="minus_Click" Cursor="Hand"/>
            <Button x:Name="sieben" Content="7" Height="40" Width="40" Margin="8,110,234,65" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="sieben_Click" Cursor="Hand"/>
            <Button x:Name="acht" Content="8" Height="40" Width="40" Margin="53,110,189,65" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="acht_Click" Cursor="Hand"/>
            <Button x:Name="neun" Content="9" Height="40" Width="40" Margin="98,110,144,65" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="neun_Click" Cursor="Hand"/>
            <Button x:Name="mult" Content="*" Height="40" Width="40" Margin="151,126,106,81" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="mult_Click" Cursor="Hand"/>
            <Button x:Name="null" Content="0" Height="40" Width="40" Margin="50,149,186,13" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="null_Click" Cursor="Hand"/>
            <Button x:Name="clean" Content="C" Height="40" Width="40" Margin="8,156,234,19" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="clean_Click" Cursor="Hand"/>
            <Button x:Name="teil" Content="/" Height="40" Width="40" Margin="151,172,106,35" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="teil_Click" Cursor="Hand"/>
            <Button x:Name="lösch" Content="f" FontFamily="Wingdings 3"  Height="40" Margin="191,20,7,155" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Width="84" Click="lösch_Click" Cursor="Hand"/>
            <Button x:Name="gleich" Content="=" Height="84" Margin="191,66,51,65" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Width="40" Click="gleich_Click" Cursor="Hand"/>
            <Button x:Name="Komma" Content="." Height="40" Width="40" Margin="106,172,151,35" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="komma_Click" Cursor="Hand"/>
            <Button x:Name="kla" Content="(" Height="40" Width="40" Margin="242,126,15,81" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="kla_Click" Cursor="Hand" Command="Close"/>
            <Button x:Name="klz" Content=")" Height="40" Width="40" Margin="242,82,15,125" FontSize="18" FontWeight="Bold" VerticalAlignment="Center" HorizontalAlignment="Center" Click="klz_Click" Cursor="Hand" Command="Close"/>
        </Grid>
        <TextBlock x:Name="Text2" Height="35" Margin="10,351,0,0" HorizontalAlignment="Left" VerticalAlignment="Top" FontSize="18" Width="319" Text="Dies ist eine ungetestete Beta-Version"/>
    </Grid>
</Window>

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Kirk1701A · Verfasst: Di 10.10.17 10:46

Hier lag es am "IndexOf".

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Kirk1701A · Verfasst: Di 10.10.17 10:50

Sry, bin vom Thema abgewichen.
Ähmm, ich weiss aber nicht, wie ich dann eine Rechenfunktion stellen soll. :nixweiss:

Aber, wie gesagt, der Taschenrechner gaaanz oben hat erst so gut gerechnet, seit dieses "double.Parse(...)" drin ist.

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Th69 · Verfasst: Di 10.10.17 12:23

Gibst du "3+3+" etc. direkt in die TextBox ein oder aber über die Buttons?
In der TextBox darf nur eine einzelne Zahl stehen (aber sonst keine anderen Zeichen), wenn du Parse/TryParse aufrufst - sonst erhältst du eben eine Exception bzw. false als Ergebnis)!!!

Halte dich an die Funktionalität eines realen Taschenrechners, d.h. immer bei jeder Operation die vorherige Operation ausführen, d.h.

markieren

Quelltext

1:
2:
3:
4:

3
+
3
/

Jetzt 3+3 rechnen und das Ergebnis intern in einer Variablen merken und in die TextBox zurückschreiben - und als Operation '/' merken.
Und nach

markieren

Quelltext

1:
2:

2
=

dann wieder die letzte Operation ('/') ausführen und das Ergebnis in die TextBox zurückschreiben.
Mit Backspace solltest du nur die letzte Ziffer einer Zahl löschen (aber nicht die Operation). Setze daher am besten (ersteinmal) "readonly" bei der TextBox, so daß der Anwender komplett die Buttons benutzen muß.

Einen Parser mit beliebigen Ausdrücken zu schreiben, ist etwas komplizierter (u.a. wegen unterschiedlicher Operatorprioriäten und Klammern) - dafür habe ich Parser für mathematische Formeln entwickelt. ;-)

PS: Der nächste Schritt wäre es, wenn du dein Programm auf MVVM (Model View ViewModel) umschreibst, denn dies ist die übliche Vorgehensweise bei WPF. So kannst du dann die Anwendungslogik komplett unabhängig von der UI entwickeln und testen!

Kirk1701A · Verfasst: Di 10.10.17 13:15

Die TextBox ist so eingestellt, dass man nicht hineinklicken kann (hast du das nicht ausprobiert? :?:

).

Es gibt aber 2 Möglichkeiten, die Zahlen einzugeben:
1. per Buttons
2. per Tastatur (genauer: NumPad)

Rechnen tut der schon ganz gut. Längere Rechnungen, Minuszahlen, Dezimalzahlen, verschiedene Rechenarten in einer Rechnung, ...
Mir ist nur wichtig, dass ich, wenn ich einen Operatoren lösche, dass ich (ohne Crash) einen neuen einsetzen kann.

Hinweis: Es hat schon einmal so funktioniert!

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Ralf Jansen · Verfasst: Di 10.10.17 13:50

Zitat:

Die TextBox ist so eingestellt, dass man nicht hineinklicken kann (hast du das nicht ausprobiert? :?:

).

Üblich ist sein Problem soweit herunter zu brechen das ausprobieren für uns eher unnötig sein sollte. Es ist also nicht hilfreich wenn man seinen ganzen Code hier einfach reinwirft oder schlimmer noch den ganzen Code verschiedener Versionen seines Entwicklungsstands und dann noch von diversen Änderungen spricht die man daran vorgenommen hat weiß irgendwann keiner mehr was genau jetzt dein Problem womit ist. Du solltest es Helfern einfach machen und nicht verwirren. Dann verlieren Helfer potentiell nur ihre LUst am helfen. Heißt selber Problemstellen ausmachen und diese zeigen ohne den ganzen nicht problemrelevanten Boilerplate Code. Wenn mehr nötig wird werden Helfer dich schon darauf hinweisen und nach mehr fragen. Wenn dann etwas zum ausprobieren nötig ist dann auch eher als angehängtes Projekt (z.B. als Zip ohne Executeable oder Objektdateien) so das man das einfach bei sich ausprobieren kann und nicht erst aufwendig ein Projekt basteln muß.

Zitat:

Mir ist nur wichtig, dass ich, wenn ich einen Operatoren lösche, dass ich (ohne Crash) einen neuen einsetzen kann.

Dann mal kurz Hilfe zur Selbsthilfe

a.) Etwas crasht nicht einfach so. Das System spricht mit dir und sagt dir welche Exception geworfen wurde etc. Diese Details sind wichtig. Auch wenn dir das noch nichts sagt uns könnten sie was sagen. Also was für eine Exception bekommst du?
b.) Was steht in den beteiligten Variablen? Wenn es irgendwo knallt solltest du einfach mal in die beteiligten Variablen schauen und deine Annahmen gegen das checken was da wirklich drin steckt. Z.B. was steht den zum Zeitpunkt an dem es knallt in der Variablen die du in Parse/TryParse reingibst drin?

Kirk1701A · Verfasst: Di 10.10.17 14:11

Okay, ich seh ein, dass ich mein Problem nicht korrekt geschildert habe. Also mache ich das jetzt richtig:

Ich baue gerade an einem ganz einfachen Taschenrechner, der die Grundrechenarten, negative -, und Dezimalzahlen beherrschen soll. Die Eingabe geschieht über Buttons oder NumPad. Als Beispiel zur Schilderung nehme ich die Rechnung "3+3/3".

Ich gebe "3+3+" ein. Ich habe mich vertippt und möchte nun das letzte + korrigieren. Also drücke ich die Backspace-Taste oder -Button (extra angelegt). Nun steht "3+3" in einer separaten TexBox. Wenn ich nun "/" eingebe für "3+3/...", stürzt mein Programm ab und setzt die Ausnahme auf meine double.Parse (Code kommt gleich). Das ist mit [b]allen[/b] Rechenarten so. Egal ob "3-3*..." oder etc.

Hier nun mein Code:

markieren

C#-Quelltext

1:
2:
3:
4:
5:
6:
7:

 private void teil_Click(object sender, RoutedEventArgs e)
        {
            op.Text = "/";
            p.Text += "/";
            erg1 += double.Parse(rechner.Text);
            rechner.Clear();
        }

Wenn ich debugg mache, geht der bis zu "erg1" und zeigt die Exception. Ich habe mich über die "double.Parse" Methode echt gefreut, weil erst dadurch mein Taschenrechner rechnen konnte.

Der "rechner" oben im Code ist eine TextBox, in die nicht hineingeklickt werden kann.

Ich hoffe, ihr ... verzeiht mir wegen meiner bescheidenen Erläuterung vorhin.

Lebt lang und in Freiden

Euer Kirk

_________________
"Ich war es und werde es immer bleiben... Ihr Freund!"
@Spock -> Star Trek II: Der Zorn des Khan

Ralf Jansen · Verfasst: Di 10.10.17 14:15

Zitat:

Wenn ich debugg mache, geht der bis zu "erg1" und zeigt die Exception. Ich habe mich über die "double.Parse" Methode echt gefreut, weil erst dadurch mein Taschenrechner rechnen konnte.

Der "rechner" oben im Code ist eine TextBox, in die nicht hineingeklickt werden kann.

Zu dem Zeitpunkt wo die Exception angezeigt wird schau was in rechner.Text drinsteht. Und bedenke double.Parse rechnet nicht es wandelt einen string in eine Zahl und 3+3 ist keine Zahl. Das Ergebnis wäre eine Zahl der Ausdruck selbst aber nicht.

	Mitgliederliste
	Gruppen
	Das Team
	Richtlinien
	Synonyme