C# - Wndows-Anwednung (GUI)

[delphinhawe]

Danke!

1. Pardon, ja das ist wie bei Dateiname ohne Umlaute zu speichern.
2. Camel-Casting finde ich echt super gut. Das werde ich noch ändern.
3. Kannst Du mir bitte dazu ein paar Beispiele aufschreiben,
damit die Wartbarkeit des Codes sich nicht erhöht? Ich stehe auf dem Schlauch.
4. Ja, ist nicht optimal.

Gruß delphinhawe

 

[Norbert Eder]

Wartbarkeit des Codes:
Nun ganz einfach: Hast du zwei Methoden die das haargleiche machen und es ist eine Änderung notwendig, musst du diese in beiden Methoden nachziehen. Wird von zwei Stellen aber nur _eine_ Methode aufgerufen, ist die notwendige Änderung nur in dieser einen durchzuführen. Daher im Falle eine Änderung wesentlich weniger Aufwand.

Hier nun ein Beispiel:

private void VergibEinenBesserenMethodenNamen()
{
  TEXTFELDER01.Location = new Point(50, 160);
  TEXTFELDER01.Size = new Size(290, 20);
  TEXTFELDER01.Text = "Bitte geben Sie die Stromstärke in Ampere ein:";
  Controls.Add(TEXTFELDER01);
  EINGABEFELDER01.Location = new Point(50, 190);
  EINGABEFELDER01.Size = new Size(280, 20);
  EINGABEFELDER01.Text = "";
  Controls.Add(EINGABEFELDER01);
  TEXTFELDER02.Location = new Point(50, 220);
  TEXTFELDER02.Size = new Size(280, 20);
  TEXTFELDER02.Text = "Bitte geben Sie den Widerstand in Ohm ein:";
  Controls.Add(TEXTFELDER02);
  EINGABEFELDER02.Location = new Point(50, 250);
  EINGABEFELDER02.Size = new Size(280, 20);
  EINGABEFELDER02.Text = "";
  Controls.Add(EINGABEFELDER02);
  SCHALTFLÄCHE0A.Location = new Point(135, 280);
  SCHALTFLÄCHE0A.Size = new Size(100, 30);
  SCHALTFLÄCHE0A.Text = "Berechnen";
  Controls.Add(SCHALTFLÄCHE0A);
}

protected void ANLEGENKLICK01(object sender, EventArgs e)
{
    VergibEinenBesserenMethodenNamen();
    // Wenn hier schon ein Click-Handler drauf sitzt sollte dieser
    // zuvor mit SCHALTFLÄCHE0A.Click -= ... entfernt werden
    // Zusätzlich diese bösen Variablen-Namen ändern, da bekommt man ja
    // Augenkrebs
    SCHALTFLÄCHE0A.Click += new EventHandler(WIDERSTANDCLICK);
}

protected void ANLEGENKLICK02(object sender, EventArgs e)
{
    VergibEinenBesserenMethodenNamen();
    // Wenn hier schon ein Click-Handler drauf sitzt sollte dieser
    // zuvor mit SCHALTFLÄCHE0A.Click -= ... entfernt werden
    // Zusätzlich diese bösen Variablen-Namen ändern, da bekommt man ja
    // Augenkrebs
    SCHALTFLÄCHE0A.Click += new EventHandler(SPANNUNGCLICK);
}