Analoger Eingang - Bargraph Anzeige

[delaware]

Hallo, habe eine VU / Bargraph Anzeige programmiert.
An Pin 0 PORT C ( ADC) hängt ein Poti zwischen GND & 5V (mit Vorwiderstand).

Je nach Stellung des Potis soll eine LED-Licht wandern.

Ich beschäftige mich seit 3 Wochen mit ATmels.
Die Schaltung läuft auf einem STK500 mit ATmega8.

Vielleich kann mir jemand eine Meinung zu dem C- Code geben, evtl. geht es ja auch viel einfacher.

christoph

[sternst]

Wie wäre es erstmal mit "richtiger" statt "einfacher"?
Oder ist es Absicht, dass keine LED an ist, wenn result genau 4 ist (oder genau 20, oder 30, ...)?
Ich fände es auch einfacher lesbar, wenn man die beiden Vergleiche tauschen würde, also so:

    if (result>0  && result<=4)   PORTD = 0b11111110;
    if (result>4  && result<=20)  PORTD = 0b11111101;
    if (result>20 && result<=30)  PORTD = 0b11111011;
    ...

[delaware]

Hast recht. Viel gar nicht auf, da der Port bei z.b. genau 4 nicht gelöscht wurde sondern einfach der vorige Wert erhalten blieb.

[Ceos]

ohne in das programm gesehen zu haben würde ich folgende zeile vorschlagen

PORTD = ~(1<<((result*8)/1024)); // messwert / 10bit->1024 / 8Lampen = bitposition

wobei das ergebnis aus der rechnung die nte lampe ist, die leuchten soll und die binäre zahl 1 dann um soviele positionen nach links geschoben wird, im falle von 1023 (maximalwert) ist das ergebnis 7,99~ die nachkommastellen werden einfach abgeschnitten im falle von weniger als 128 wäre das 0

ACHTUNG: ich hab hier result*8 geklammert, damit auch definitiv DAS zuerst berechnet wird, wenn ich erst das ergebnis durch 1024 teile, klommt IMMER 0 bei raus, bei ganzzahldivision ignoriert man alles was nach dem komma kommt

bei 0 ist die erste lampe an und bei 1023 die achte lampe ... da die LEDs low active sind, hab ich mit '~' die zahl abschliessend invertiert


PS: bei result == 0 würde garkeine lampe bei dir leuchten!