Autor Thema: Modul actuator_UD  (Gelesen 5839 mal)

0 Mitglieder und 1 Gast betrachten dieses Thema.

Offline gravieren

  • Entwickler
  • *****
  • Beiträge: 578
    • Profil anzeigen
Modul actuator_UD
« 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.


Offline hugo

  • Global Moderator
  • *****
  • Beiträge: 2 150
    • Profil anzeigen
Re: Modul actuator_UD
« Antwort #1 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.