Kod: Markera allt
flag=0;
RC3=0;
while(1){
if(RA3==0 && flag==0){
RC3=1;
flag=1;
}
if(RA3==0 && flag==1){
RC3=0;
flag=0;
}
}
Kod: Markera allt
#define true 1
#define false 0
#define Key RA3
#define LED RC3
unsigned char Previous;
while(true)
{
if(!Key && !Previous)
{
// Key has been pressed
Previous = true; // Mark as "Got it!"
if(LED) Led = false;
else LED = true;
}
else Previous = false;
}
Har suttit å försökt lösa de där med timern som du föreslog vilket verkar som en mycket bra idé. Tyvärr har jag inte fått till det så förklara gjärna mer utförligt hur du menar!Icecap skrev: Jag föredrar att lösa detta i mjukvara och då är lösningen att läsa fler gångar och när två (eller fler) avläsningar med ett visst tidintervall mellan (50ms?) är identiska är det brytarens status.
Nu börjar det komma in en massa "använd __delay_ms()"-råd men mitt råd är att starta en timer-interrupt med kanske 20-30Hz, då blir resten enkelt.
Jag har som regel att i varje interrupt läsa porten en enda gång! Ska man använda resultatet fler gångar i samma rutin sparar jag det lästa värde i en variabel. Då undviker man att brytaren studsar mellan första och andra avläsningen. Ska det hela vara 100% bergsäkert använder jag 3 avläsningar efter varandra.
Sedan sparar jag även en variabel med "förra avläsningen" i och först när de två är identiska kollar jag om man har tryckt eller släppt knappen. Efter det är det bara att kolla vad som händer.
Det där hjälpte inte ett dugg.sodjan skrev:Vad händer med den ursprungliga koden om knappen hålls intryckt?
(Jo *jag* vet, det är en hemläxa till dig).
Sen så är den fördröjning på 1 ms "ingenting" i sammanhanget
och gör ingen skillnad alls.
Tillbaka till ritbordet och tänkt till lite mer...
Kod: Markera allt
if(RA3==0 && flag==0){
RC3=1;
flag=1;
}
---- Vi antar att RA3 är 0 och flaggan var 0
Flaggan har ändrats till 1 när programmet kommer hit
if(RA3==0 && flag==1){
---- RA3 är fortfarande 0 och din flagga är 1
RC3=0;
flag=0;
---- Nu är din flagga 0 igen
---- Resultatet blir att RC3 växlar mellan 1 och 0 så länge som RA3 = 0
}
Jo att de blir sådär har jag förstått nu! Nu vill jag bara förstå hur jag avhjäper bouncen!Swech skrev:Kod: Markera allt
if(RA3==0 && flag==0){ RC3=1; flag=1; } ---- Vi antar att RA3 är 0 och flaggan var 0 Flaggan har ändrats till 1 när programmet kommer hit if(RA3==0 && flag==1){ ---- RA3 är fortfarande 0 och din flagga är 1 RC3=0; flag=0; ---- Nu är din flagga 0 igen ---- Resultatet blir att RC3 växlar mellan 1 och 0 så länge som RA3 = 0 }
Swech
Kod: Markera allt
#include <pic.h>
#include <htc.h>
__CONFIG (FOSC_INTRCCLK&WDTE_OFF&MCLRE_OFF&CP_OFF);
#define FOSC 8000000L
#define _XTAL_FREQ 8000000
#define ON 1
#define OFF 0
//LED:S
#define LED_BSPEGEL RC3
//BRYTARE
#define SW_BSPEGEL RA3
int button;
int flag;
int clock;
int bounce;
Init(void)
{
OSCCON = 0x70;
TMR0 = 0; //Clear the TMR0 register
OPTION_REG = 0B00000010;
}
main(void)
{
Init();
//CONFIG PORTS
TRISC=0x00;
ANSEL=0x00;
PORTC=0x00;
TRISA=0xff;
RC3 = 0;
//GRUNDVÄRDEN
flag=0;
bounce = 200; //TID för DEBOUCE
while(1) //LOOP FOREVER
{
if(T0IF && SW_BSPEGEL==1){ //NÄR TIMER OWERFLOW ÖKA CLOCK MED 1
clock=clock+1;
T0IF=0;
}
if(clock>bounce && flag==OFF && SW_BSPEGEL==0){ //TÄND LED FÖR BACKSPEGEL
flag=ON;
LED_BSPEGEL = ON;
clock=0;
}
if(clock>bounce && flag==ON && SW_BSPEGEL==0){ //SLÄCK LED FÖR BACKSPEGEL
LED_BSPEGEL = OFF;
flag=OFF;
clock=0;
}
}
}För att se om jag förstod dig rätt, du såg fel förut men inte nu?sodjan skrev:> Nu vill jag bara förstå hur jag avhjäper bouncen!
Det viktiga här är ju att du faktiskt förstog vad som var det *stora*
problemet med din ursprungliga kod! Det har inte med "de-bounce" att göra.
Om du inte vid det här laget ser det klart och tydligt så har du sannolikt
andra problem på gång.
Ett annat sätt att se på det är att en så kort kod som det var från början
nästan måste vara fel, det går inte att få önskad funktion med så
lite kod...
