OSCAT Forum

oscat.lib => Bestehende Module / Existing Modules => Thema gestartet von: gravieren am 24. Februar 2007, 16:58:53

Titel: Modul actuator_UD
Beitrag von: gravieren am 24. Februar 2007, 16:58:53

Hi Hugo

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

Es sind diverse Pasagen wie diese hier im Modul.

Zitat

VAR_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.

Titel: Re: Modul actuator_UD
Beitrag von: hugo am 24. Februar 2007, 18:47:42

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.