Defines für Port / Pins

[x5 (Gast)]

Guten Morgen zusammen,

ich bin momentan meinen Code am durchwühlen - und da ist mir eine Stelle 
im Code aufgefallen der einfach nicht schön aussieht.

Kurz zum Aufbau ich habe einen Pin am Controller der einen Status 
einliesst, ist der Pin HIGH ist alles OK wenn er LOW ist liegt eine 
störung vor. Da zu gibt es eine DUO LED (grün/rot) an einem Ausgang für 
alles OK/Störung


in meinem Code sieht es dann so aus:

if(PIN_IN & (1<<Status_Temperatur))   {PORT_LED   |=   (1<<LED_Status);}
        else    {PORT_LED  &=  ~(1<<LED_Status);}

und die Header

#define Status_Temperatur      PF0    //
#define LED_Status      PE0    //LED /

so kann ich es nun elegant lösen das das PIN_IN und PORT_LED aus dem 
Code rausfliegt?

zb Beispiel das es einfach heisst
if(Status_Temperatur) LED_Status=1;
else LED_Status=0;

schematisch geschrieben.

wisst ihr was ich meine ? ansonsten muss ich es vieleicht noch anders 
erklären

gruß
x5

[Peter D. (peda)]

http://www.avrfreaks.net/index.php?name=PNphpBB2&file=viewtopic&p=823879#823879


Peter

[Oliver (Gast)]

x5 schrieb:
> so kann ich es nun elegant lösen das das PIN_IN und PORT_LED aus dem
> Code rausfliegt?

Man kann Peters Lösung verwenden, oder einfach akzeptieren, daß 
Bitbehandlung in Standard-C diese Befehle braucht. Nach einer Weile sind 
die dann gar nicht mehr unelegant.

Oliver

[Peter D. (peda)]

Oliver schrieb:
> Nach einer Weile sind
> die dann gar nicht mehr unelegant.

An dieser AND/OR-erei hatte mich vor allem gestört, daß ich für jeden 
Pin 2 Symbole definieren und mitschleppen muß (Byteadresse, Bitnummer).

Die Idee zu dem SBIT-Macro hat mal jemand hier im Forum gepostet, ist 
also nicht von mir.


Peter

[Tom K. (ez81)]

x5 schrieb:

> so kann ich es nun elegant lösen das das PIN_IN und PORT_LED aus dem
> Code rausfliegt?
>
> zb Beispiel das es einfach heisst
> if(Status_Temperatur) LED_Status=1;
> else LED_Status=0;
>

In C gibt es für soetwas ein höchst lesbares, flexibles und sehr gut 
wartbares Feature, nämlich Funktionen:

bool Status_Temperatur(void)
{
    return ! (PIN_IN & (1<<STATUS_TEMPERATUR));
}

void LED_Status(uint8_t on)
{
    if (on)
        PORT_LED  |=  (1<<LED_STATUS);
    else
        PORT_LED  &=  ~(1<<LED_STATUS);
}

if(Status_Temperatur())
    LED_Status(1);
else
    LED_Status(0);

Nachtrag: Abstrakte Business-Logik und Implementierungsdetails (wie 
Bitschubsen) gehören IMO nicht zusammen in eine Zeile.