Arduino Merkwürdiges verhalten

[Enrico S. (key-board)]

Hallo
Ich will mir mit LEDs eine Ladezustandsanzeige basteln

Ansteuern will ich die mit:

byte Zustand =0;
byte anz=0;
anz = (etwas zwischen 0 und 8);
Zustand = byte(pow(2,anz)-1);

Wenn ich anz direkt etwas zwischen 0 und 8 setze leuchten die leds so 
wie sie sollen.
z.B.

anz = 4;
println(anz,DEC); ->4 angezeigt
println(Zustand,DEC); -> 15 angezeigt(leds leuchten entsprechend dem Binärmuster)

Fange ich jetzt an anz zu berechnen aus einem SOC

anz=byte(map(SOC,0,100,0,Aufloesung));

oder

anz=byte(SOC*Aufloesung/100);

wobei mir Aufloesung angibt ob ich 4 oder 8 oder... LEDs für die Anzeige 
zur Verfügung habe.
kommt die falsche anzeige bei raus obwohl ich an der Zustand berechnung 
nichts ändere kommt z.B. bei SOC 50 folgendes raus
println(anz,DEC); ->4 angezeigt
println(Zustand,DEC); -> 14 angezeigt(leds leuchten entsprechend dem 
Binärmuster)

kann mir mal einer sagen warum einmal
2^4-1=15
und einmal
2^4-1=14
rauskommt?????
und das nur weil die 4 einmal fest geschrieben und einmal ausgerechned 
wird?

beim ausrechnen entsteht ein fehler bei jedem SOC

PS:
Der fehler tritt auch auf wenn ich die einzelnen Schwellen mit if und 
else if bedingungen mache... und immer setze z.B. if(SOC > 45) anz = 4;
nur wenn ich wieder ganz kurz vor dem ausrechnen des Zustandes anz auf 
einen wert z.B. 4 setze, dass wird alles richtig angezeigt...

es ist zum verrückt werden...

[Karl H. (kbuchegg)]

Enrico S. schrieb:

> kann mir mal einer sagen warum einmal
> 2^4-1=15
> und einmal
> 2^4-1=14
> rauskommt?????
> und das nur weil die 4 einmal fest geschrieben und einmal ausgerechned
> wird?

Da ist mit Sicherheit irgendwo wieder eine Floating Point Rechnung im 
Spiel. Die Details, wie zb Datentypen oder realen Code, hast du ja nicht 
gezeigt.

Dein Ergebnis ist eben nicht 4, sondern zb. 3.999999.
Bei der Ausgabe wird gerundet. Wenn du aber rechnest, wird tatsächlich 
mit 3.999999 gerechnet. Und das ist nun mal nicht dasselbe wie 4.0


Wo auch immer du mit Floating Point rechnest, füge beim Übergang zu int 
eine Rundungskorrektur ein

  int intWert;
  float floatWert;

  floatWert = 3.99999;

  // Rundungskorrektur
  intWert = floatWert + 0.5;

[Karl H. (kbuchegg)]

Abgesehen davon, ist die Verwendung von pow hier mit Kanonen auf Spatzen 
geschossen.

Du hast 8 mögliche Ergebnisse.
Die kannst du auch ganz einfach als Fix und Fertiges Ergebnis in einem 
Array vorhalten.

  byte LedPattern[] = { 0b00000001,    // 0
                        0b00000010,    // 1
                        0b00000100,    // 2
                        0b00001000,    // 3
                        0b00010000,    // 4
                        0b00100000,    // 5
                        0b01000000,    // 6
                        0b10000000,    // 7
                        0b11111111,    // 8 -> zu gross
                      };

...
  if( anz > 7 )
    anz = 7;

  Zustand = LedPattern[anz];
...

keiner muss gross rumrechnen und du hast den Bonus, das Led Muster 
problemlos so anzupassen, wie du das möchtest. Möchtest du den Level als 
einzelne LED angeben, dann ist das kein Problem. Möchtest du ein 
LED-Band dann ist das auch kein Problem. Möchtest du den Level als 2 
nebeneinander liegende LED anzeigen, dann änderst du einfach das Muster 
im Array. Möchtest du anstatt 8 Levels deren 16, indem du zb Abwechselnd 
nur 1 und 2 LED leuchten lässt, dann geht auch das (wenn man das Array 
entsprechend größer macht und die Rechenfaktoren anpasst).

Alles in allem: win win

[Enrico S. (key-board)]

OK Danke

die floating point operation kahm durch pow();
und habs jetzt gelößt mit
if (SOC > 90)
Zustand = 255;
usw

und die Anpassung an meine Auflösung mit
if (Aufloesung !=8)
SOC=SOC*Auflösung/8;

funktioniert mit alles Auflösungen die für meinen Einsatzzweck sinnvoll 
sind (3 bis 8 LEDs)

[Samuel C. (dragonsam)]

Es wäre deutlich einfacher gewesen, einfach das

pow(2, anz)

durch

(1<<anz)

zu ersetzen.