LED-Leiste mit switch-case

[Thomas (Gast)]

Hallo,
ich möchte auf einem Atmel-Controller die Temperatur über eine 8er 
LED-Leiste ausgeben.
Ich wollte die Temperatur (welcher °-Bereich wird später eingestellt) in 
Acht Bereiche einteilen und dann mit switch-case abarbeiten.
Irgendwie komme ich aber nicht mehr weiter. Ich weiss dass der Block mit 
den Defines noch nicht korrekt ist, es geht mir nur um die 
switch-case-Geschichte
Könnte das mal jemand durchchecken? Kann man das so gebrauchen? Oder ist 
das grob falsch? (syntaktische Fehler bitte verzeihen, ist noch in der 
Bastelphase).

Danke schonmal, ich verzweifle hier bald

#define Bereich_1 voltage_conversion <= 100mV;
#define Bereich_2 voltage_conversion <= 200mV;
#define Bereich_3 voltage_conversion <= 300mV;
#define Bereich_4 voltage_conversion <= 400mV;
#define Bereich_5 voltage_conversion <= 500mV;
#define Bereich_6 voltage_conversion <= 600mV;
#define Bereich_7 voltage_conversion <= 700mV;
#define Bereich_8 voltage_conversion <= 800mV;

void LED_Anzeige(uint16_t voltage)
{
  
  uint16_t voltage_conversion;
  
  Vref=5;
  
  voltage_conversion = (ADC_value*Vref)/1024;  //Datenblatt S.325
  
  
  switch (LED_Anzeige)
  {
    case Bereich_1:
      while(Bereich_1)          //Eine LED an
      {
        PORTD == 0x01;
        OSTimeDlyHMSM(0,0,1,0);      
      }
      break;
      
    case Bereich_2:
      while(Bereich_2)          //Zwei LEDs an
      {
        PORTD == 0x03;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_3:
      while(Bereich_3)          //Drei LEDs an
      {
        PORTD == 0x07;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_4:
      while(Bereich_4)          //Vier LEDs an
      {
        PORTD == 0x0F;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_5:
      while(Bereich_5)          //Fuenf LEDs an
      {
        PORTD == 0x1F;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_6:
      while(Bereich_6)          //Sechs LEDs an
      {
        PORTD == 0x3F;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_7:
      while(Bereich_7)          //Sieben LEDs an
      {  
        PORTD == 0x7F;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
    case Bereich_8:
      while(Bereich_8)          //Acht LEDs an
      {
        PORTD == 0xFF;
        OSTimeDlyHMSM(0,0,1,0);
      }
      break;
      
  }

[Dumdi D. (dumdidum)]

Thomas schrieb:
> Kann man das so gebrauchen? Oder ist
> das grob falsch?

Leider letzteres. Switch-case kann keine Ungleichungen. Mach einfach 8 
if Abfragen hintereinander.

[Thomas (Gast)]

Danke für die schnelle Antwort.

Wenn ich die Bereiche bzw. Grenzspannungen vorher definiert habe, ist 
doch beim switch-case keine Ungleichung drin oder seh ich das irgendwie 
falsch?

[Dumdi D. (dumdidum)]

#define  ist eine einfache Textersetzung. Du koenntest den Text auch per 
copy und paste einfuegen.

[Thomas (Gast)]

Ok, das bedeutet also, wenn ich mit Grenzspannungen arbeiten will ist 
switch-case nicht möglich? Oder gibt es anstelle von #define eine 
bessere Möglichkeit?

[Stefan E. (sternst)]

Thomas schrieb:
> Ok, das bedeutet also, wenn ich mit Grenzspannungen arbeiten will ist
> switch-case nicht möglich?

Klar geht das. Man müsste nur die eigentliche Bereichsprüfung 
"auslagern" und einen dem gefundenen Bereich zugeordneten Index 
ermitteln, den man dann per switch auswerten könnte.

Aber bei einem derart simplen linearen Zusammenhang ist das alles 
komplett überflüssig. Man kann das Ganze auch mit nur einer Zeile Code 
erledigen.
Angenommen voltage_conversion ist in mV (was es bei dir nicht ist):

PORTD = (1 << (voltage_conversion/100)) - 1;

[Pfitsche (Gast)]

Vielleicht ist ein anderer Ansatz sinnvoll.

z.B. etwas mehr Mathematik : Modulo ?

[Thomas (Gast)]

Naja es soll so simpel wie möglich sein. Ich möchte einfach nur abhängig 
von der Temperatur (also der Spannung in mV) 1 bis 8 LEDs leuchten 
lassen.
Ich dachte switch-case wäre das richtige Werkzeug dafür.

Wie müsste den der oben genannte Codeschnippsel eingebaut werden
 (PORTD = (1 << (voltage_conversion/100)) - 1;) ?

[g457 (Gast)]

> Naja es soll so simpel wie möglich sein. Ich möchte einfach nur abhängig
> von der Temperatur (also der Spannung in mV) 1 bis 8 LEDs leuchten
> lassen.

Noch einfacher gehts nicht. Z.B. so:

[pseudocode]

uint8_t gibMichAnzahlOderAuchGleickBitPatternDerLedsDieWoLeuchtenSollen(T spannung)
{
    if (spannung < 42mV)
        return 1;
    if (spannung < 105mV)
        return 2;
    if (spannung < 170mV)
        return 3;
    [..]
    return uups;
}

void LED_Anzeige(uint16_t voltage)
{

    uint16_t voltage_conversion;
    Vref=5;
    voltage_conversion = (ADC_value*Vref)/1024;  //Datenblatt S.325

    uint8_t ui8AnzahlOderAuchGleickBitPatternDerLedsDieWoLeuchtenSollen = gibMichAnzahlOderAuchGleickBitPatternDerLedsDieWoLeuchtenSollen(voltage_conversion);

    tuWas(ui8AnzahlOderAuchGleickBitPatternDerLedsDieWoLeuchtenSollen);
[/pseudocode]

[Stefan E. (sternst)]

Thomas schrieb:
> Wie müsste den der oben genannte Codeschnippsel eingebaut werden
>  (PORTD = (1 << (voltage_conversion/100)) - 1;) ?

Was meinst du mit "einbauen"? Es "ersetzt" das komplette Geraffel.

Was nur zusätzlich noch fehlt, ist eine sinnvolle Vorbereitung von 
voltage_conversion. Die Formel dafür in deinem Code liefert nämlich kein 
wirklich sinnvolles Ergebnis.

[Thomas (Gast)]

Das klingt gut.
Dann verstehe ich die Bedeutung deiner Code-Zeile irgendwie falsch, 
könntest du mir die kurz erläutern?
Das mit der Formel ist mir bewusst und wird später geändert.
Es ist nur eine Weile her dass ich programmiert habe und ich quäl mich 
schon ewig mit diesem einfachen Problem...

[Stefan E. (sternst)]

Thomas schrieb:
> Dann verstehe ich die Bedeutung deiner Code-Zeile irgendwie falsch,
> könntest du mir die kurz erläutern?

Nö, denn so kompliziert ist die ja nun auch nicht. Sorry, aber wenn du 
die Zeile nicht selbst nachvollziehen kannst, dann fehlen dir eh ein 
paar Grundlagen, die du erst mal nachholen müsstest.


Übrigens müsstest du die Zeile ggf. auch noch ein wenig modifizieren. In 
ihrer jetzigen Form erzeugt sie ein Verhalten, wie ich es von einer 
Balkenanzeige mit 100mV-Schritten erwarten würde. Das weicht ein wenig 
davon ab, was du in deinem Pseudo-Code oben als beabsichtigtes Verhalten 
offenbart hast.