Hallo leute.
Wollte mal fargen ob ihr mir vieleicht auch ein paar antworten auf meine Frage geben könnt. Ich bin völliger anfänger. Aber könnt ihr mir vieleicht sagen wie ich meinen Quelltext verschönern bzw vereinfachen kann damit mein quelltext bisschen weniger wird? Vielen dank im Voraus.
//********************************
// Wiederstandserrechnung 1
//********************************
if (chBWied1.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung1 = double.Parse(textBoxSpann1.Text);
double Strom1 = double.Parse(textBoxStrom1.Text);
double wiederstand1 = Spannung1 / Strom1 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand1.Text = Convert.ToString(wiederstand1);
}
//********************************
// Wiederstandserrechnung 2
//********************************
if (chBWied2.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung2 = double.Parse(textBoxSpann2.Text);
double Strom2 = double.Parse(textBoxStrom2.Text);
double wiederstand2 = Spannung2 / Strom2 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand2.Text = Convert.ToString(wiederstand2);
}
//********************************
// Wiederstandserrechnung 3
//********************************
if (chBWied3.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung3 = double.Parse(textBoxSpann3.Text);
double Strom3 = double.Parse(textBoxStrom3.Text);
double wiederstand3 = Spannung3 / Strom3 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand3.Text = Convert.ToString(wiederstand3);
}
//********************************
// Wiederstandserrechnung 4
//********************************
if (chBWied4.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung4 = double.Parse(textBoxSpann4.Text);
double Strom4 = double.Parse(textBoxStrom4.Text);
double wiederstand4 = Spannung4 / Strom4 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand4.Text = Convert.ToString(wiederstand4);
}
//********************************
// Wiederstandserrechnung 5
//********************************
if (chBWied5.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung5 = double.Parse(textBoxSpann5.Text);
double Strom5 = double.Parse(textBoxStrom5.Text);
double wiederstand5 = Spannung5 / Strom5 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand5.Text = Convert.ToString(wiederstand5);
}
//********************************
// Wiederstandserrechnung 6
//********************************
if (chBWied6.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung6 = double.Parse(textBoxSpann6.Text);
double Strom6 = double.Parse(textBoxStrom6.Text);
double wiederstand6 = Spannung6 / Strom6 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand6.Text = Convert.ToString(wiederstand6);
}
//********************************
// Wiederstandserrechnung 7
//********************************
if (chBWied7.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung7 = double.Parse(textBoxSpann7.Text);
double Strom7 = double.Parse(textBoxStrom7.Text);
double wiederstand7 = Spannung7 / Strom7 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand7.Text = Convert.ToString(wiederstand7);
}
//********************************
// Wiederstandserrechnung 8
//********************************
if (chBWied8.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung8 = double.Parse(textBoxSpann8.Text);
double Strom8 = double.Parse(textBoxStrom8.Text);
double wiederstand8 = Spannung8 / Strom8 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand8.Text = Convert.ToString(wiederstand8);
}
//********************************
// Wiederstandserrechnung 9
//********************************
if (chBWied9.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung9 = double.Parse(textBoxSpann9.Text);
double Strom9 = double.Parse(textBoxStrom9.Text);
double wiederstand9 = Spannung9 / Strom9 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand9.Text = Convert.ToString(wiederstand9);
}
//********************************
// Wiederstandserrechnung 10
//********************************
if (chBWied10.Checked == true)
{
//Veriablenvergabe und TextBox text Umwandeln
double Spannung10 = double.Parse(textBoxSpann10.Text);
double Strom10 = double.Parse(textBoxStrom10.Text);
double wiederstand10 = Spannung10 / Strom10 * 1000;
//Ergebiss Umwandeln in eine Zeichenfolge(string) und ins ergebnissfenster schreiben
textBoxWiederstand10.Text = Convert.ToString(wiederstand10);
}
}
private void checkBox1_CheckedChanged(object sender, EventArgs e) { }
private void picBSpann_Click(object sender, EventArgs e) { }
private void btnBeispiel_Click(object sender, EventArgs e)
{
//********************
//Bei ausführen dieses Buttons werden alle Felder mit einem wert eingetragen
//********************
chBSchaltkreisAnz.Checked = true;
// Wiederstandswerte 1
chBWied1.Checked = true;
textBoxStrom1.Text = "139,1";
textBoxSpann1.Text = "14";
// Wiederstandswerte 2
chBWied2.Checked = true;
textBoxStrom2.Text = "64,8";
textBoxSpann2.Text = "14,3";
// Wiederstandswerte 3
chBWied3.Checked = true;
textBoxStrom3.Text = "43,9";
textBoxSpann3.Text = "14,5";
// Wiederstandswerte 4
chBWied4.Checked = true;
textBoxStrom4.Text = "31,06";
textBoxSpann4.Text = "14,6";
// Wiederstandswerte 5
chBWied5.Checked = true;
textBoxStrom5.Text = "21,52";
textBoxSpann5.Text = "14,7";
// Wiederstandswerte 6
chBWied6.Checked = true;
textBoxStrom6.Text = "14,72";
textBoxSpann6.Text = "14,8";
// Wiederstandswerte 7
chBWied7.Checked = true;
textBoxStrom7.Text = "1,5";
textBoxSpann7.Text = "14,8";
// Wiederstandswerte 8
chBWied8.Checked = true;
textBoxStrom8.Text = "0,71";
textBoxSpann8.Text = "14,8";
// Wiederstandswerte 9
chBWied9.Checked = true;
textBoxStrom9.Text = "0,19";
textBoxSpann9.Text = "14,8";
// Wiederstandswerte 10
chBWied10.Checked = true;
textBoxStrom10.Text = "0,05";
textBoxSpann10.Text = "14,8";
}
Ihr seht schon es ist alles ziemlich lang.
Hallo DerJonny,
pack deine CheckBox
en in ein Array und iteriere in einer Schleife darüber. So musst du den jeweiligen Code nur einmal schreiben.
Außerdem ...*verwende besser double.TryParse() anstelle von double.Parse()!
*teile nicht ohne Prüfung durch Strom1
(der Wert könnte 0 sein)!
*s. [Tipp] Anfängerfehler == true / == false ...
*Variablen innerhalb einer Methode beginnen laut Konvention mit einem Kleinbuchstaben (Capitalization Styles).
m0rius
Mein Blog: blog.mariusschulz.com
Hochwertige Malerarbeiten in Magdeburg und Umgebung: M'Decor, Ihr Maler für Magdeburg
Mach's mit Arrays oder so und dann mit ner Schleife. Eigentlich absolutes Grundwissen.
MfG
userid16184