#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
