Funktion vereinfachen

Luki Gast #4570398 
Hallo Zusammen

Ich würde gerne den Code im Anhang vereinfachen, am liebsten die 
Funktion parametrisieren. Denn diese Funktion enthält noch 6 weitere 
Taster (welche ich nun nicht im Code beigelegt habe, da sie sehr quasi 
identisch sind) abfragen die im gleich Programmiert sind, jedoch ist der 
inhalt der If verzweigungen immer leicht anderst. Ansonsten Funktioniert 
das Programm so wie es sollte. Es geht hier auch wirklich nur um diese 
Funktion.

Mein Vorschlag:

void ReadT( bool* Pushed,  long* Timeout, int PortBit, int*Mseconds )
{
  if( *Pushed)
  {
    *Pushed=false;

    if( Status==Zero)
    {
      PORTB|= _BV( PortBit);
      *Timeout= Timer()+ *Mseconds;
    }

                if (Status==One)
                {
                 ....
                }
                ...

  }
}

Was denkt ihr, bin da auf dem richtigen Weg? oder habt ihr bessere Ideen 
wie sich die Funktion vereinfachen lässt ohne das ich den code am 
schluss x-mal kopieren muss?

Vielen Dank.
Angehängte Dateien:
Antwort Zitieren Bearbeiten Thread verschieben Thread sperren Langsamer Modus Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Melden Danke
Chris D. #4570489 
Hi Luki,

> Ich würde gerne den Code im Anhang vereinfachen, am liebsten die
> Funktion parametrisieren.

.... und warum tust Du es dann nicht... weiter?
Led(Nr, X);
State(Condition, Y);
Timeout(Timer, Time);

Dein >"if(Status==One Two Three)"< Konstukt ist ziemlich kompliziert, 
versuchs mal mit switch/case.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Forist Gast #4570522 
Luki schrieb:
> jedoch ist der inhalt der If verzweigungen immer leicht anderst.

Ja verflucht noch mal, wie soll dir hier jemand verraten können, wie es 
sich vereinfachen läßt, wenn du nicht verräts "wie anders".

Normalerweise würde ich so ein Zeugs immer über Arrays laufen lassen und 
nur mit dem Index arbeiten, aber wer weiss, ob das mit dem Inhalt 
deiner geheimen If-Verzweigungen die Sache vereinfachen würde.
Antwort Zitieren Bearbeiten Löschen Melden Danke
Thread beobachten

Antwort schreiben

Eine eigene Frage? Dieser Thread ist über 10 Jahre alt. Stell sie in einem eigenen Thread – dort lesen mehr Leute mit, und sie wird öfter und schneller beantwortet.

Neuen Thread starten

Eine Lösung, Ergänzung oder Rückmeldung zu diesem Thread? Dann antworte hier.

Formatierung
  • [c]C-Code[/c]
  • [code]Code in anderen Sprachen, ASCII-Zeichnungen[/code]
  • [math]Formel in LaTeX-Syntax[/math]
  • [[Titel]] – Link zu einem Artikel
  • Verweis auf einen anderen Beitrag: Rechtsklick auf dessen Beitrags-ID (#8078628), „Adresse kopieren“, in den Text einfügen

Mehr zur Formatierung

Längeren Sourcecode als Anhang, nicht in den Text. JPG nur für Fotos und Scans; Zeichnungen und Screenshots als PNG oder GIF. Siehe Bildformate.

Dein Name und deine E-Mail-Adresse

Schon registriert? Anmelden

Wird nicht angezeigt. Wir schicken dir einen Link; sobald du damit bestätigst, erscheint dein Beitrag.

Mit dem Abschicken legst du einen kostenlosen Account an und erkennst die Nutzungsbedingungen und die Datenschutzerklärung an.