Modul actuator_UD

Begonnen von gravieren, 24. Februar 2007, 16:58:53

Vorheriges Thema - Nächstes Thema

0 Mitglieder und 2 Gäste betrachten dieses Thema.

gravieren

Hi Hugo

Hier ist zwar kein Fehler, jedoch etwas unschön.

Es sind diverse Pasagen wie diese hier im Modul.


ZitatVAR_OUTPUT
    Yup : BOOL;                <-- BOOL   (TRUE,FALSE)
    Ydn : BOOL;                <-- BOOL   (TRUE,FALSE)
    status : BYTE;
END_VAR



IF yup AND NOT up OR ydn AND NOT dn THEN
        ydn := 0;
        yup := 0;
        last := tx;

Es wird die Zuweisung teilweise mit "TRUE und FALSE" ausgeführt.

Ebenso werden diese mit "0 und 1" ausgeführt.


Die Schreibweise ist auch teilweise   "ydn  oder  Ydn".  (Erster Buchstabe Groß b.z.w. Klein)

Ist zwar NICHT fehlerhaft, beim Überprüfen des Code für mich jedoch ungewohnt.


hugo

danke du hast recht 0 und 1 anstelle von false und true sollte nicht sein, das ein compiler das versteht bedeutet noch lange nicht das es auch ein anderer versteht.
das gleiche gilt für die meisten unserer module, schlechter programmierstiel.
werde mich bemühen das bei änderungen mit abzustellen.