# Modul-Code zur Anpassung **URL:** \ **Category:** Bestehende Module / Existing Modules\ **Created:** [28. Februar 2007 um 17:18 UTC](https://oscat.discourse.group/t/modul-code-zur-anpassung/83 "2007-02-28T17:18:56Z")\ **Posts on this page:** 4\ **Page:** 1
**Author:** ![gravieren](https://avatars.discourse-cdn.com/v4/letter/g/54ee81/32.png) [@gravieren](https://oscat.discourse.group/u/gravieren)\ **Post date:** [28. Februar 2007 um 17:18 UTC](https://oscat.discourse.group/t/modul-code-zur-anpassung/83/1 "2007-02-28T17:18:56Z")
Hi Hugo Ich teste momentan die OSCAT-S7 Version. Mir fallen einige Code-Stellen auf, die nachgebessert werden könnten. > Orginal: C\_to\_F := celsius \* 1.8 + 32 > > Vorschlag: C\_to\_F := (celsius \* 1.8 ) + 32.0 32 → (INT-Wert) Punkt vor Strich kann der Compiler, jedoch wird die lesbarkeit erhöht. * * * check\_parity FUNCTION check\_parity : BOOL TITLE = ‘check\_parity’ // // this function checks for an even partity for a dword and partity bit. // // Uses: even // VERSION : ‘1.0’ AUTHOR : hugo NAME : CHECKP FAMILY : GATE VAR\_INPUT in : DWORD; \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\* FEHLER ? Sollte BYTE sein (Laut Handbuch) p : BOOL; \*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\*\* FEHLER ? Sollte BOOL sein (Laut Handbuch) END\_VAR Zudem fehlt im Handbuch die Erklärung/Beschreibung für den Paramter “p” ---
**Author:** ![hugo](https://yyz2.discourse-cdn.com/free1/user_avatar/oscat.discourse.group/hugo/32/133_2.png) [@hugo](https://oscat.discourse.group/u/hugo)\ **Post date:** [28. Februar 2007 um 22:03 UTC](https://oscat.discourse.group/t/modul-code-zur-anpassung/83/2 "2007-02-28T22:03:56Z")
danke für die inputs zu punkt 1 32.0 ja vielleicht besser lesbar, aber durch typecast errechnet der compiler automatisch den richtigen wert. 32.0 wird als real gespeichert und 32 als in spart also 2 byte speicherplatz und ist effizienter dword für input. das ist volle absicht, dword kann durch typecast des compilers mit allen kleineren bit typen geladen werden, damit kann wenn in einem dword das unterste byte bearbeitet werden soll dieses direkt auf den eingang gelegt werden. die funktion nist also flexibler. wir wollen dies in der lib verstärkt nutzen um universeller zu sein. p (parity sollte natuerlich beschrieben sein und wird unmittelbar berichtigt. vielen dank ---
**Author:** ![gravieren](https://avatars.discourse-cdn.com/v4/letter/g/54ee81/32.png) [@gravieren](https://oscat.discourse.group/u/gravieren)\ **Post date:** [1. März 2007 um 15:41 UTC](https://oscat.discourse.group/t/modul-code-zur-anpassung/83/3 "2007-03-01T15:41:15Z")
Hi > dword für input. das ist volle absicht, dword kann durch typecast des compilers mit allen kleineren bit typen geladen werden Jo, klingt logisch, sollte aber auch dokumentiert sein ;) ---
**Author:** ![hugo](https://yyz2.discourse-cdn.com/free1/user_avatar/oscat.discourse.group/hugo/32/133_2.png) [@hugo](https://oscat.discourse.group/u/hugo)\ **Post date:** [4. März 2007 um 03:06 UTC](https://oscat.discourse.group/t/modul-code-zur-anpassung/83/4 "2007-03-04T03:06:24Z")
doku wird verbessert. danke, hugo