einzelne bits ändern in einem Array oder komplett anders ?

Frank schrieb:
> ich hatte mit das so vergestellt das ich meinem Code
> LED_1(an);
> bzw LED_1(aus);
> Schreiben kann

Einfacher wirds, wenn Du statt einer im Funktionsnamen codierten Nummer 
die Koordinaten der LED angibst:

  LedAn(int x, int y);

  LedAus(int x, int y);

(x € [0..7], y € [0 .. 4])

Mit y wählst Du das Arrayelement aus, mit x das Bit im Arrayelement.

Damit kommt sowas dabei raus:

LedAn(int x, int y)
{
  LED_Array[y] |= 1 << x;
}

und

LedAus(int x, int y)
{
  LED_Array[y] &= ~ (1 << x);
}

... jeweils ohne Bereichsüberprüfung für x und y, was Du natürlich 
nachpflegen solltest.

Frank schrieb:
> Also das man die Funktion mit:
>
> LedAn(Heizdraht_LED);
>
> LedAus(Heizdraht_LED);


Da Du bei Deiner Hardware die Hardwareadresse der LED kennen musst, also 
wissen musst, in welcher Spalte und Zeile diese angeordnet sind, auch 
genau diese Information in zwei #defines unterbringen:

#define SPALTE_HEIZDRAHT_LED 1
#define ZEILE_HEIZDRAHT_LED 3

LedAn(SPALTE_HEIZDRAHT_LED, ZEILE_HEIZDRAHT_LED);

Wenn Dir das zu viel Tipperei ist, lässt sich das wiederum mit einem 
Macro erledigen:

#define LedAnMacro(a) LedAn(SPALTE_##a##_LED, ZEILE_##a##_LED)

so daß Du

LedAnMacro(HEIZDRAHT);

aufrufen musst.


Natürlich kannst Du auch mit einer indexbasierten Lösung arbeiten, aber 
dann musst Du entweder den Index von Hand berechnen ... oder wieder ein 
Macro verwenden, das den aus den "Koordinaten" der Led bestimmt.

>

1

> #define SPALTE_HEIZDRAHT_LED 1

2

> #define ZEILE_HEIZDRAHT_LED 3

3

>

Da kannst du aber dann auch gleich für jede LED einfach seine 
Indexnummer und seine Bitnummer innerhalb des Bytes angeben lassen. Das 
ist dann auch schon egal.

1

#define HEIZDRAHT_INDEX 0    // Index 0 im Array

2

#define HEIZDRAHT_BIT   0    // und dort das 0-te Bit

3

4

#define KUEHLUNG_INDEX  0    // Index 0

5

#define KUEHLUNG_BIT    1    // und dort das 1-te Bit

Noch ein bischen Makro Magie rundherum, wie du das angedeutet hast und 
fertig.

Karl Heinz Buchegger schrieb:
> Da kannst du aber dann auch gleich für jede LED einfach seine
> Indexnummer und seine Bitnummer innerhalb des Bytes angeben lassen.

Und worin besteht da jetzt der Unterschied? Das ist, von den Namen 
"Spalte" und "Zeile" abgesehen exakt dasselbe.


Die Index-Variante, die hier 
Beitrag "Re: einzelne bits ändern in einem Array oder komplett anders ?" erwähnt wurde, hat 
nicht Bitnummern in Indexwerten, sondern numeriert alle LEDs durch.

Rufus Τ. Firefly schrieb:
> Karl Heinz Buchegger schrieb:
>> Da kannst du aber dann auch gleich für jede LED einfach seine
>> Indexnummer und seine Bitnummer innerhalb des Bytes angeben lassen.
>
> Und worin besteht da jetzt der Unterschied?

Dass sich der TO nicht mit der Umsetzung von Spalten und Zeilen in 
Arrayindizes und Bitnummern gedanklich quälen muss :-)

> Das ist, von den Namen
> "Spalte" und "Zeile" abgesehen exakt dasselbe.

Ja. Sollte auch für den TO nur ein Denkanstoss sein, dass er seine 
Matrix in diesem Sinne nicht wortwörtlich als Matrix ansehen muss, 
sondern einfach für eine LED die Ansteuerinformation so festlegen und 
damit arbeiten kann, wie es ihm am leichtesten fällt.

Ob er eine LED als 'sitzt im Schnittpunkt von Zeile z und Spalte s' 
ansieht, oder ob er sie als 'das zuständige Bit ist das Bit Nr b im 
i-ten Eintrag des Arrays' ansieht ist gehupft wie gesprungen. Mir schien 
nur, dass er die 2-te Sichtweise überhaupt nie in Betracht gezogen hätte 
sondern durch den Begriff der 'Matrix' sich selbst unnötigerweise auf 
Zeilen und Spalten festgenagelt sah.

Frank schrieb:
> Wenn ich das LedAn vor der Whileschleife aufrufe dann geht die richtige
> LED an und wenn ich das in der Whileschleife aufrufe dann geht sie nicht
> an ?!

Die Funktionen LedAn und LedAus schalten nicht die LEDs selbst, sondern 
setzen nur im Array die korrespondierenden Bits.

Wo ist Dein Code, der den Array-Inhalt per Multiplex an die LEDs 
ausgibt?

>   TIMSK    |= 1<<TOIE0;          // Time 0 Initialisieren

Und für den Timer 0 hast du auch eine ISR?


Zeig bitte den ganzen Code. Wenn du denkst, dass das zuviel Code ist, 
dann specke ihn auf das Notwendige ab, probier nochmal ob sich auch 
wirklich derselbe Fehler zeigt, und poste das dann.

Für uns Helfer gibt es nichts lästigeres, als wie wenn wir uns aus 
Codeschnipseln die möglichen Fehlerursachen zusammensuchen müssen und 
dann ständig irgendwelche Nachfragen stellen müssen, die sich ganz 
leicht dadurch verhindern lassen, indem du einfach alles zeigst.

Die meisten 'Dauerhelfer' hier sind ziemlich gut darin, auch etwas 
umfangreicheren Code in kurzer Zeit zu verstehen und auf versteckte 
Fehler abzuklopfen. Dazu gehören zb vergessene volatile oder auch 
freigegebene Interrupts, für die es keine ISR gibt. Beides ist mit 
deinen Codeschnipseln nicht beantwortbar.