Zählrichtung festlegen

Wie oft wird der interrupt aufgerufen, wenn die Lichtschranke 
durchlässig ist?

Dirk B. schrieb:
> Wie oft wird der interrupt aufgerufen, wenn die Lichtschranke
> durchlässig ist?

nach Oszibild sieht man ca. alle 60ms einen ca 10ms breiten negativen 
Impuls.

Wenn man die Zeile: if (AUF==1) und die Zeilen für das Herunterzählen
weg lässt, dann läuft das Zählen wie es soll bzw. im umgedrehten Fall 
wird heruntergezählt --> negative Werte

Achim S. schrieb:
> deine if-Abfrage für ZU funktioniert nicht:
>
> wolle g. schrieb:
>> if(ZU==1) zaehlerstand_anzeigen();  // Zählereingang
>>   {
>>   zaehlerstand = zaehlerstand -1;
>>   zaehlerstand_anzeigen();
>>     }

Mist.
Das kommt davon, wenn man den aktuellen Stand nicht richtig kopiert.
>D.h. der nachfolgende Block mit zaehlerstand = zaehlerstand -1; wird in
>jedem Fall ausgeführt.
So hatte ich mir das ja auch gedacht, aber leider wird gar nichts 
angezeigt, woraus ich ableiten würde, dass die Zeilen if (AUF==1) nicht 
funktionieren oder die Variable AUF nicht erzeugt wird bzw. zum 
richtigen Zeitpunkt nicht zu Verfügung steht

Walter schrieb:
>sind die Taster mit Pullup oder Pulldown?
Die Taster liegen über einen Widerstand auf Masse.

Walter schrieb:
> wenn das so zu verstehen ist dass die Taster beim Drücken den Eingang
> auf Masse ziehen, dann ist die Abfrage falsch

Der Taster liegt auf Masse (über einen R) und der Eingang des µC wird 
beim Drücken auf +3V geschaltet.

wolle g. schrieb:
> Der Taster liegt auf Masse (über einen R) und der Eingang des µC wird
> beim Drücken auf +3V geschaltet.

Interessanter Ansatz - wie machst du das?

H.Joachim S. schrieb:
> wolle g. schrieb:
>> Der Taster liegt auf Masse (über einen R) und der Eingang des µC wird
>> beim Drücken auf +3V geschaltet.
>
> Interessanter Ansatz - wie machst du das?

Ich hänge mal das Schaltbild an. Die H-Brücke hängt an der mit dem Pfeil 
gekennzeichneten Buchsenleiste.
Die Taster heißen hier Prog1 und Prog2.

Achim S. schrieb:
> zuerst war deine Fehlerbeschreibung, dass nicht gezählt wird. Jetzt
> meinst du, es wird nichts angezeigt. Kannst du das Fehlerbild mal etwas
> sauberer herausarbeiten?
>
> Der Compiler wird schon das umsetzen, was du hingeschrieben hast.
> Deshalb würde ich mir weniger Sorgen um die Verfügbarkeit der Variablen
> machen, das Problem liegt wohl eher in deiner Interpretation der
> Ausgaben des Codes.

> Wird die Funktion Blitz_gelb() denn wie erwartet ausgeführt, wenn das
> Zahnrad dreht?
a) Blitz wird erwatungsgemäß ausgeführt und ist auch ein Zeichen dafür, 
dass die Tasten und die Lichtschranke funktionieren.

b) Patricks Vorschlag habe ich getestet. Ergebnis: Der Zähler läuft nur 
rückwärts, weil offensichtlich nur die Zeile
zaehlerstand = zaehlerstand -1;
bearbeitet und dann angezeigt wird.

c) hier noch einmal meine Version mit meinen Beobachtungen:
Wenn ich es so benutze, wie folgend angegeben, dann wird aufwärts 
gezählt und der Zählerstand wird auf einer Flüssigkristallanzeige 
angezeigt, egal welche Taste gedrückt wird.
Zunächst i.O.
Wenn alle Zeilen verarbeitet werden sollen (wie nennt man das? 
kommentieren/ auskommentieren?), mit   /*  …….. */  herausnehmen
Dann wird nichts angezeigt, denn wie oben schon vermutet, werden 
wahrscheinlich die if-Schleifen nicht verarbeitet. (meine Theorie)

interrupt (PORT2_VECTOR)P2anschluss (void)
{
Blitz_gelb();
//if (AUF==1)
{
 zaehlerstand = zaehlerstand +1;
 zaehlerstand_anzeigen();
}/*
 if (ZU==1)
{
zaehlerstand = zaehlerstand -1;
zaehlerstand_anzeigen();
}*/
P2IFG&=~BIT5;
}

hoffentlich habe ich mich jetzt besser ausgedrückt

Patrick schrieb:
> Wo hast du denn "AUF" deklariert?

Die Variablen AUF, ZU wurden als globale Variable (oder wie man das 
nennt) deklariert.

So:
unsigned int AUF,ZU;
int main(void)
   {
    WDTCTL = WDTPW + WDTHOLD;     // Stop watchdog timer

    while(1)
   {
    if (P2IN&BIT0)     // Taste Ventil-AUF gedrückt
    {
       AUF=1;
       P4OUT &=~BIT0;  // LED rot   P4.0 EIN
       P2OUT &=~BIT4;  // AUF-fahren
     }
    else
     ............

Jetzt habe ich mal bei der Deklaration geschrieben:

unsigned int AUF=1,ZU;
und damit wird aufwärts gezählt
bei
unsigned int AUF,ZU=1;
wird abwärts gezählt
Also, wie könnte man es anstellen, dass den Variablen erst dann ein Wert 
zugeordnet wird, wenn sie gebraucht werden.
(hier bei Tastendruck, später zusätzlich von einem PI(D)-Regler)

Patrick schrieb:
> Versuch mal, sie in der
> main als "extern" bekannt zu machen, so vermeidest du, dass ggf. nur auf
> (wie auch immer entstandenen..) Lokalvariablen gearbeitet wird.

hier wird gleich gemeckert:

in funktion 'P2anschluss '
 AUF undeklariert

Stefanus F. schrieb:
> Das ist ja jetzt auch klein geschrieben.

dürfte m.E. keine Rolle spielen, denn bei Deklaration und Verwendung 
wurden die Variablen groß geschrieben,

Achim S. schrieb:
> Der Compiler wird schon das umsetzen, was du hingeschrieben hast.
> Deshalb würde ich mir weniger Sorgen um die Verfügbarkeit der Variablen
> machen, ...

Und dann kommt der Optimierer und macht Dir einen Strich durch die 
Rechnung.
@wolleg: Deklariere AUF und ZU doch mal als "volatile".

Mario M. schrieb:
> Und dann kommt der Optimierer und macht Dir einen Strich durch die
> Rechnung.
> @wolleg: Deklariere AUF und ZU doch mal als "volatile".

Danke für den Hinweis. auch an alle  "Mitarbeiter"

Problem gelöst.
Gehört wahrscheinlich in die Rubrik "hohe Schule" für Programmierlaien.