Laden...

Code wird zu lang

Erstellt von DerJonny vor 13 Jahren Letzter Beitrag vor 13 Jahren 973 Views
D
DerJonny Themenstarter:in
1 Beiträge seit 2010
vor 13 Jahren
Code wird zu lang

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.

1.002 Beiträge seit 2007
vor 13 Jahren

Hallo DerJonny,

pack deine CheckBoxen 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

U
50 Beiträge seit 2009
vor 13 Jahren

Mach's mit Arrays oder so und dann mit ner Schleife. Eigentlich absolutes Grundwissen.
MfG
userid16184