Differenzen auswerten: Geht das eleganter?

[Christian J. (Gast)]

Hallo,

gibt es, obwohl das durchaus ein effizienter Code ist eine Möglichkeit 
sowas eleganter zu schreiben? Es handelt sich nur um 12 LED, die 
Temperaturen darstellen sollen, auch Zwischenwerte sollen dargestellt 
werden durch 2 LED. Die Led Setzfunktion lässt derzeit nur einen 
Parameter zu + ON, OFF oder BLINK.
Dazu setze ich einfach in einem Array die LED und ein Interrupt wertet 
diesen dann aus und betätigt das Schieberegister.

/ Bitmasken für die Aussentemperatur LES

#define HC595_LED_M10       0
#define HC595_LED_M5        1
#define HC595_LED_0         2
#define HC595_LED_5         3
#define HC595_LED_10        4
#define HC595_LED_15        5
#define HC595_LED_17        6
#define HC595_LED_20        7
#define HC595_LED_22        8
#define HC595_LED_25        9
#define HC595_LED_30        10
#define HC595_LED_35        11

/* Zustand der Aussen-Temperatur LED am 74HCT595 Nr.2 */
led_status_t atemp_led[12] = {OFF,OFF,OFF,OFF,OFF,OFF,OFF,OFF,OFF,OFF,OFF,OFF};

  if (W->Ext_Temperature < -10.5)
         HC595_SetATempLed(HC595_LED_M10,BLINK);
    else if ((W->Ext_Temperature > -10.5) && (W->Ext_Temperature < -9.5))  {
         HC595_SetATempLed(HC595_LED_M10,ON);
    } else if ((W->Ext_Temperature > -9.5) && (W->Ext_Temperature < -5.5)) {
        HC595_SetATempLed(HC595_LED_M10,ON);
        HC595_SetATempLed(HC595_LED_M5,ON);
    } else if ((W->Ext_Temperature > -5.5) && (W->Ext_Temperature < -4.5)) {
        HC595_SetATempLed(HC595_LED_M5,ON);
    } else if ((W->Ext_Temperature > -4.5) && (W->Ext_Temperature < -0.5)) {
        HC595_SetATempLed(HC595_LED_M5,ON);
        HC595_SetATempLed(HC595_LED_0,ON);
    } else if ((W->Ext_Temperature > -0.5) && (W->Ext_Temperature < 0.5)) {
        HC595_SetATempLed(HC595_LED_0,ON);
    } else if ((W->Ext_Temperature > 0.5) && (W->Ext_Temperature < 4.5)) {
        HC595_SetATempLed(HC595_LED_0,ON);
        HC595_SetATempLed(HC595_LED_5,ON);
    } else if ((W->Ext_Temperature > 4.5) && (W->Ext_Temperature < 5.5)) {
        HC595_SetATempLed(HC595_LED_5,ON);
    } else if ((W->Ext_Temperature > 5.5) && (W->Ext_Temperature < 9.5)) {
        HC595_SetATempLed(HC595_LED_5,ON);
        HC595_SetATempLed(HC595_LED_10,ON);
    } else if ((W->Ext_Temperature > 9.5) && (W->Ext_Temperature < 10.5)) {
        HC595_SetATempLed(HC595_LED_10,ON);
    } else if ((W->Ext_Temperature > 10.5) && (W->Ext_Temperature < 14.5)) {
        HC595_SetATempLed(HC595_LED_10,ON);
        HC595_SetATempLed(HC595_LED_15,ON);
    } else if ((W->Ext_Temperature > 14.5) && (W->Ext_Temperature < 15.5)) {
        HC595_SetATempLed(HC595_LED_15,ON);
    } else if ((W->Ext_Temperature > 15.5) && (W->Ext_Temperature < 16.5)) {
        HC595_SetATempLed(HC595_LED_15,ON);
        HC595_SetATempLed(HC595_LED_17,ON);
    } else if ((W->Ext_Temperature > 16.5) && (W->Ext_Temperature < 17.5)) {
        HC595_SetATempLed(HC595_LED_17,ON);
    } else if ((W->Ext_Temperature > 17.5) && (W->Ext_Temperature < 19.5)) {
        HC595_SetATempLed(HC595_LED_17,ON);
        HC595_SetATempLed(HC595_LED_20,ON);
    } else if ((W->Ext_Temperature > 19.5) && (W->Ext_Temperature < 20.5)) {
        HC595_SetATempLed(HC595_LED_20,ON);
    } else if ((W->Ext_Temperature > 20.5) && (W->Ext_Temperature < 21.5)) {
        HC595_SetATempLed(HC595_LED_20,ON);
        HC595_SetATempLed(HC595_LED_22,ON);
    } else if ((W->Ext_Temperature > 21.5) && (W->Ext_Temperature < 22.5)) {
        HC595_SetATempLed(HC595_LED_22,ON);
    } else if ((W->Ext_Temperature > 22.5) && (W->Ext_Temperature < 24.5)) {
        HC595_SetATempLed(HC595_LED_22,ON);
        HC595_SetATempLed(HC595_LED_25,ON);
    } else if ((W->Ext_Temperature > 24.5) && (W->Ext_Temperature < 25.5)) {
        HC595_SetATempLed(HC595_LED_25,ON);
    } else if ((W->Ext_Temperature > 25.5) && (W->Ext_Temperature < 29.5)) {
        HC595_SetATempLed(HC595_LED_25,ON);
        HC595_SetATempLed(HC595_LED_30,ON);
    } else if ((W->Ext_Temperature > 29.5) && (W->Ext_Temperature < 30.5)) {
        HC595_SetATempLed(HC595_LED_30,ON);
    } else if ((W->Ext_Temperature > 30.5) && (W->Ext_Temperature < 34.5)) {
        HC595_SetATempLed(HC595_LED_30,ON);
        HC595_SetATempLed(HC595_LED_35,ON);
    } else if ((W->Ext_Temperature > 34.5) && (W->Ext_Temperature < 35.5)) {
        HC595_SetATempLed(HC595_LED_35,ON);
    } else if (W->Ext_Temperature > 35.5)
        HC595_SetATempLed(HC595_LED_35,BLINK);

[Johann L. (gjlayde)]

Es genügt doch, die "Sprung"temperaturen zu speichern, die dann in einer 
Schleife abgefragt werden.

Wenn die Temperatur z.B: < -5.5 ist, werden 3 LEDs gesetzt: Die 1. im 1. 
Schleifendurchlauf, die 2. in 2. Durchlauf und die 3. im 3.  Danach wird 
die Schleife beendet weil die Temperatur zu klein ist für die 4. LED und 
alle folgenden.

[Tom (Gast)]

Christian J. schrieb:
> if (W->Ext_Temperature < -10.5)
>          HC595_SetATempLed(HC595_LED_M10,BLINK);
>     else if ((W->Ext_Temperature > -10.5) && (W->Ext_Temperature <
> -9.5))  {
>          HC595_SetATempLed(HC595_LED_M10,ON);
>     } else if ((W->Ext_Temperature > -9.5) && (W->Ext_Temperature <
> -5.5)) {

Willst Du den Fall W->Ext_Temperature == -10.5 explizit nicht behandeln?
Andernfalls wäre der Vergleich nämlich doppelt.

if (a <= -10)
   blubb1;
else if ( (a > -10) && (a < -5) )
   blubb2;

Das (a >-10) steckt schon im Wort else mit drin: Wäre a nicht > -10, 
wäre die erste Abfrage a<=-10 schon wahr gewesen und die zweite würde 
nie ausgeführt.


Ansonsten fände ich eine Lookuptable für die Grenzen der 
Temperaturbereiche und eine für die aus den Stufen resultierenden 
LED-Zustände etwas kompakter. Eleganz und Effizienz 
(Platz/Geschwindigkeit/gute Wartbarkeit?) ist natürlich wie immer 
Ansichtssache.

#include <stdio.h>

/* Temperatur wird in Stufen eingeteilt:
... -2        -> 0
-2 .. -1      -> 1
-1 .. -0. 5   -> 2
......
3.0 ...       -> 6
*/
const float temperature_range_limits[] = {-2,-1,-0.5, 0, 2.5, 3.0};
const size_t num_temperature_ranges = sizeof(temperature_range_limits) / sizeof(temperature_range_limits[0]);
const unsigned char leds[] = {0,1,2,3,4,5,6}; // irgendwelche Bitmuster.

size_t get_temperature_range(float t)
{
    for (size_t i = 0; i < num_temperature_ranges; ++i)
        if (t <= temperature_range_limits[i])
            return i;
    return num_temperature_ranges;
}

unsigned get_led_state(size_t temperature_range)
{
    // TODO: bereichsueberpreufung!
    return leds[temperature_range];
}

int main(void)
{
    for (float t = -2.5; t < 4.0; t += 0.234567)
    {
        size_t range = get_temperature_range(t);
        unsigned char leds = get_led_state(range);
        printf("%f\t%d\n", t, leds);
    }
    return 0;
}

[Christian J. (Gast)]

Hm... mir schwebt grad sowas vor... ist aber noch nicht zu Ende 
gedacht...

const float atemp[12] = {-10.0, -5.0, 0.0, 5.0, 10.0, 15.0, 17.0, 20.0, 22.0, 25.0, 30.0, 35.0};

    for (uint8_t i = 0; i < 12; i++)
    {
        if ( (W->Ext_Temperature > (atemp[i] - 0.5)) && (W->Ext_Temperature < (atemp[i] + 0.5)))
            HC595_SetATempLed(i,BLINK);

        if ( (W->Ext_Temperature > (atemp[i] + 0.5)) && (W->Ext_Temperature < (atemp[i+1] - 0.5))) {
            HC595_SetATempLed(i,ON);
            HC595_SetATempLed(i+1,ON);
        }

    }

PS: Läuft! Thread kann gelöscht werden :-)

[Sebastian S. (amateur)]

>PS: Läuft! Thread kann gelöscht werden :-)

Lang lebe der Egoismus ;-)

[Christian J. (Gast)]

Tom schrieb:
> Willst Du den Fall W->Ext_Temperature == -10.5 explizit nicht behandeln?

Float Werte kann man nicht auf Gleichheit prüfen. Trotzdem Danke, dass 
Du Dir soviel Mühe gemacht hast! Mein Vorteil war, dass alle Temps 0.5 
um ihr Zentrum liegen und alle Zwischenräume eben dazwischen. Dadurch 
war das mit der Schleife oben einfach lösbar.

const size_t num_temperature_ranges = sizeof(temperature_range_limits) / 
sizeof(temperature_range_limits[0]);

Auch noch nicht gesehen .... ich habe einfach 12 reingeschrieben.

[Tom (Gast)]

Das heißt aber nicht, dass zwei float-Werte nie gleich sein können.

[Christian J. (Gast)]

Tom schrieb:
> Das heißt aber nicht, dass zwei float-Werte nie gleich sein können.

Habe ich aber so gelernt, dass man die nicht mit == vergleichen kann.

Das KOnstrukt von Tom habe ich grad eingebaut, man lernt nie aus :-)

[Möwe (Gast)]

Christian J. schrieb:
>> Willst Du den Fall W->Ext_Temperature == -10.5 explizit nicht behandeln?
>
> Float Werte kann man nicht auf Gleichheit prüfen. Trotzdem Danke

Ich gehe davon aus das Tom ein ">=" oder "<=" meint. Prinzipiell ist es 
natürlich möglich dass zwei float/double-Werte einander gleich sind. In 
einem solchen, zugegebenermaßen eher seltenen, Fall würde das Programm 
direkt an deinem ganzen Konstrukt vorbeispringen. Deshalb fragt man in 
der Regel eine von zwei angrenzenden Bedingung abgeschlossen ab.

Grüße