#1
25.08.2021 um 10:27 UhrHallo,
die Idee ist gut und das Programm liefert auch das richtige Ergebnis.
Zwei Anregungen:
1. Umwandung von Zahlen in Strings kostet viel Rechenzeit. Hier lieber mit Modulo arbeiten.
2. Der Code kann vereinfacht werden. Hier mit Refactoring:
C#-Code
Gruß Jens
die Idee ist gut und das Programm liefert auch das richtige Ergebnis.
Zwei Anregungen:
1. Umwandung von Zahlen in Strings kostet viel Rechenzeit. Hier lieber mit Modulo arbeiten.
2. Der Code kann vereinfacht werden. Hier mit Refactoring:

private void BnStart_Click(object sender, EventArgs e) { var zahlerg = 0; _ = int.TryParse(TbZahl.Text, out var z1); _ = int.TryParse(TbZahl2.Text, out var z2); var ergebnis = ((double)z1 / z2).ToString(); var check = new int[10]; for (int i = 3; i < ergebnis.Length - 1; i++) { for (int j = 0; j < 10; j++) { if (ergebnis[i] == j + 48 && check[j] != 1) { zahlerg++; check[j]++; } } } LbErg.Text = zahlerg.ToString(); }
Gruß Jens
