wie Code verbessern?

[Robin F. (gehacktes)]

Schönen guten Abend,

ich habe mir einen Code geschrieben, um aus einem Taster, einen Schalter 
zu machen.
Kann man diesen Code noch irgendwie verbessern?

#include <avr/io.h>
#include <util/delay.h>

int main()
{
    DDRB=0xff;
    DDRC=0x00;
    PORTB=0x00;
    PORTC=0x00;

    while(1)
    {
        if( PINC &(1<<PC0)){
             PORTB=PORTB ^ 1;
            _delay_ms(50);
            _delay_ms(50);
        }
    }

        return 0;
    }

Das Programm funktioniert soweit...aber manchmal blinkt die LED am 
Ausgang! Kann man das irgendwie umgehen per Software oder mithilfe von 
Hardware?

Gruß Robin

[Turbotoni (Gast)]

Ja, in dem man den Taster enrprellt.

[Ulrich Radig (Gast)]

Hallo,

füge zwischen deinen beiden Delays while(PINC &(1<<PC0)); ein. Das ist 
noch nicht optimal sollte dir aber weiter helfen.

Gruß
Uli

[Robin F. (gehacktes)]

Danke Uli!

Meinst du es so?

#include <avr/io.h>
#include <util/delay.h>

int main()
{
    DDRB=0xff;
    DDRC=0x00;
    PORTB=0x00;
    PORTC=0x00;

    while(1)
    {
        if( PINC &(1<<PC0)){
            _delay_ms(5);
            PORTB=PORTB ^ 1;
            _delay_ms(50);
            while(2)
            {
                (PINC &(1<<PC0));
            }
            _delay_ms(50);
            }
    }

        return 0;
}
Wenn ja, dann passt das nicht! So bleibt die LED nach dem Tasten druck 
dauerhaft leuchten und lässt sich nicht mehr beim zweiten drücken  aus 
schalten.

Gruß Robin

[Jonathan S. (joni-st)]

Klar, Du hast ja auch eine Endlosschleife gebaut. In den Klammern hinter 
While steht nämlich die Bedingung, unter der die Schleife wiederholt 
wird - und alles außer 0 (=FALSCH) ist WAHR. Also wird deine 
While(2)-Schleife nie verlassen. Und das in der While-Schleife ist auch 
ziemlicher Käse. Schau dir doch mal an, was ein Flag und was Entprellung 
ist. Und was ein Timer ist.


Gruß
Jonathan

[Jonathan S. (joni-st)]

Jonathan Strobl schrieb:
> Und das in der While-Schleife ist auch
> ziemlicher Käse.

Zumindest dort, wo Du ihn hingeschrieben hast - nicht, dass sich Ulrich 
Radig noch auf den Schlips getreten fühlt ;)

So hat er das gemeint:

#include <avr/io.h>
#include <util/delay.h>

int main()
{
    DDRB=0xff;
    DDRC=0x00;
    PORTB=0x00;
    PORTC=0x00;

    while(1)
    {
        if( PINC &(1<<PC0)){
            PORTB=PORTB ^ 1;
            _delay_ms(50);

           while(PINC&(1<<PC0)) {}

            _delay_ms(50);
        }
    }

        return 0;
    }

[Robin F. (gehacktes)]

Okay danke!

Ich bin gerade dabei Entprellung besser zu verstehen

Melde mich später nochmal ob es geklappt hat oder nicht ;).

Grüße und schönen Abend noch
Robin

[Karl H. (kbuchegg)]

Nimm dein Originalprogramm

    while(1)
    {
        if( PINC &(1<<PC0)){
             PORTB=PORTB ^ 1;
            _delay_ms(50);
            _delay_ms(50);
        }
    }

und teste es:
Drück auf die Taste und bleib mit dem Finger auf der Taste und halte sie 
gedrückt. Deine LED wird wie verrückt blinken.

Warum?

Weil dein Code innerhalb des then-Zweiges vom if ausgeführt wird SOLANGE 
eine Taste gedrückt ist. Das willst du aber nicht. Du willst einen 
Tastendruck erkennen. Wenn also die Taste den Übergang von nicht 
gedrückt zu gedrückt macht.

Dazu gibt es 2 Möglichkeiten
* entweder du lässt das Programm aus dem if-Zweig solange nicht herraus, 
solange die Taste immer noch gedrückt ist
* oder aber, du vergleichst den Pinzustand mit demselben Pinzustand im 
unmittelbar vorhergehenden Durchlauf durch die while Schleife. Dazu 
musst du dir diesen Zustand natürlich merken. Wenn du in einer Zimmer 
reinschaust und da brennt das Licht drinnen und du WEISST, dass es eine 
halbe STunde vorher (als du das letzte mal nachgesehen hast) nicht 
gebrannt hat, dann muss in der Zwischenzeit wohl jemand das Licht 
aufgedreht haben.

[Robin F. (gehacktes)]

Ah okay danke Jonathan so klappt es.
Ich werde trotzdem die 3 Punkte die du angesprochen hast, mir zu gemüte 
führen.

Euch allen einen schönen Abend

[Robin F. (gehacktes)]

Genau Karl Heinz das war der Fehler.

Ich werde mal ein wenig weiter daran rum spielen und versuchen es mit 
beiden Methoden hin zu bekommen :)

Vielen Dank von allen

[Martin (Gast)]

Wenn du mit dem µC noch andere Dinge vorhast, als ihn in den 
Delay-Zeittotschlageschleifen vollzeit mit dem Taster zu beschäftigen, 
solltest du über einen Timer nachdenken, der per IRQ regelmäßig einen 
Blick auf die Tasten wirft und die Tasten auch gleich entprellt.
http://www.mikrocontroller.net/articles/Entprellung#Interrupt-Verfahren_.28nach_Peter_Dannegger.29
Beitrag "Tasten entprellen - Bulletproof"