ATNEL tech-forum

Re: Dziwne mruganie diody

2016-08-30T17:45:22+01:00

Bardzo Dziękuje Daro69
Przeczytałem pare razy, przeanalizowałem, z 20min mi to zajeło i juz wiem o co chodzi Teraz jeszcze pobawie się w blokowanie 3 klawisza aby działał: 'jedno kliknięcie jedno przełączenie' i zabieram się za dalsze ćwiczenia
Pozdrawiam

Statystyki: Napisane przez Maju — 30 sie 2016, o 17:45

Re: Dziwne mruganie diody

2016-08-30T16:48:16+01:00

Kolego "maju"
Przeanalizuj może ten sposób.?
Co prawda przy tym sposobie nadal będzie szybko zmieniał stan na LEDzie trzeci klawisz.
[syntax=c]#include
#include

#define LED_PIN (1<#define LED_ON PORTB &= ~LED_PIN
#define LED_OFF PORTB |= LED_PIN
#define LED_TOG PORTB ^= LED_PIN

#define KEY1_PIN (1<#define KEY2_PIN (1<#define KEY3_PIN (1<
#define PIERWSZY (!(PINB & KEY1_PIN))
#define DRUGI (!(PINC & KEY2_PIN))
#define TRZECI (!(PINC & KEY3_PIN))

uint8_t klawisz_wcisniety(uint8_t klawisz);

int main()
{
DDRB |= LED_PIN;
PORTB |= ( LED_PIN | KEY1_PIN );

DDRC &= ~( KEY2_PIN | KEY3_PIN );
PORTC |= ( KEY2_PIN | KEY3_PIN );

while(1)
{
if(klawisz_wcisniety(PIERWSZY)) LED_ON;
if(klawisz_wcisniety(DRUGI)) LED_OFF;
if(klawisz_wcisniety(TRZECI)) LED_TOG;
}
}

uint8_t klawisz_wcisniety(uint8_t klawisz)
{
if(klawisz)
{
_delay_ms(80); /*TOG-owanie tez bedzie sie wykonywac co 80 ms*/

if(klawisz) return 1;
}
return 0;
}[/syntax]
Informacyjnie- odnośnie Twojego poprzedniego kodu.
Pomijając ^, w funkcji próbowałeś sprawdzać PINC oraz PIND - Czy zauważyłeś że wciskając kl PC4 lub PC5,
Sprawdzałeś to także ze stanem PINU PB4 i PB5. a tam jest MISO i SCK . Ponieważ PC4 to 4, PC5 to 5, a PB2 to 2.
Tak więc wciskajac klawisz na PB2 funkcja sprawdzała stan także na uwaga! PINC PC2, czytając klawisz jako 2.
Niestety trzeba pilnować co funkcja może sprawdzać przypadkowo. A i niestety nie dopatrzyłem się w kodzie DDRB &= KEY1_PIN. <___ ok. jest zero od początku , ja zawsze wstawiam mimo wszystko.
Pozdrawiam

Statystyki: Napisane przez Daro69 — 30 sie 2016, o 16:48

Re: Dziwne mruganie diody

2016-08-30T14:25:18+01:00

Jazio napisał(a):

W If() jak tez w innych warunkach może być używany operator bitowy. Przecież w wyniku działania operatorem bitowym dostajemy jakąś wartość liczbową, którą sprawdzamy czy jest zerem czy nie.

Musisz jednak zrozumieć czym różnią się operatory bitowe od logicznych. Owszem można używać operatora bitowego w warnku IF ale!... ale na zmiennych/liczbach a nie jako operatora koniunkcji.

Statystyki: Napisane przez mirekk36 — 30 sie 2016, o 14:25

Re: Dziwne mruganie diody

2016-08-30T13:54:54+01:00

Tak, kolega janusz17 ma rację.to jest operator bitowy.
Jezeli to ma reagować na obia śladowe to powinno być || .

Statystyki: Napisane przez Daro69 — 30 sie 2016, o 13:54

Re: Dziwne mruganie diody

2016-08-30T13:25:31+01:00

Dziwne, że kompilator tego nie podkreśla jako błąd, bo przecież w warunku logicznym złożonym
[syntax=c]if(!(PINC & klawisz) ^ !(PINB & klawisz))[/syntax]
stosujesz operator bitowy ^. Tam może być tylko operator logiczny: &&, ||, !.
Chyba nie ma w języku C operatora logicznego ExOR.

Statystyki: Napisane przez janusz17 — 30 sie 2016, o 13:25

Re: Dziwne mruganie diody

2016-08-30T12:40:18+01:00

Faktycznie,
Pozostaje zrobić maly delay przed pętlą while żeby dać czas na podciagniecie pinu do vcc.

Statystyki: Napisane przez Daro69 — 30 sie 2016, o 12:40

Re: Dziwne mruganie diody

2016-08-30T12:22:48+01:00

KEY1_PIN jest przydzielony do portu B. Automatycznie jest ustawiany jak wejście i juz tego nie pisałem i jest podsprzegniety do vcc w 18 linijce. Przez moją niedbałość fragment kodu jest mało czytelny.
Po zamianie 17 i 18 linijki kodu na poniższe staje się bardziej czytelne. Przy zrobieniu tego zadania z 3 przyciskami na PORTC wszystko działa jak należy. A przy tym wariancie nie moge dojść dlaczego ta dioda miga przez pare sekund.
[syntax=c]DDRB |= LED_PIN;
DDRB &= (~KEY1_PIN);
PORTB |= LED_PIN | KEY1_PIN;[/syntax]

Statystyki: Napisane przez Maju — 30 sie 2016, o 12:22

Re: Dziwne mruganie diody

2016-08-30T11:39:24+01:00

Ddrc i portc key-pin1 brak.

Statystyki: Napisane przez Daro69 — 30 sie 2016, o 11:39

Dziwne mruganie diody

2016-08-30T11:12:38+01:00

Witam
Jest to mój pierwszy post na tym forum i zapowiada się, że nie ostatni.Witam wszystkich Forumowiczów . Jestem totalnym początkującym także już z góry przepraszam za moje przyszłe pytania.
Pare dni temu nabyłem BB, przeczytałem już część teoretyczną i zabrałem się za ćwiczenia praktyczne. Oprócz tych zawartych w książce robiłem różne kombinacje aby lepiej zrozumieć temat operacji bitowych i przypisywania do portów. Problem dotyczy programu do mrugania diodą przy użyciu 3 przycisków. Jednak nie podłączałem 3 przycisków do 1 portu tylko jeden do portu B a dwa do portu C. Musiałem lekko zmienić warunek if aby program zadziałał. Jednak po wgraniu programu oraz przy podłączniu napiecia do układu dioda LED przez kilka sekund mruga tak jakby przycisk LED_TOG był wciśniety. Po tych kilku sekundach dioda gasnie i program działa normalnie. Co może być przyczyną? mam jeszcze jedno pytanie czy warunek if(!(PINC & klawisz) ^ !(PINB & klawisz)) jest prawidłowo skonstruowany?(dokładnie chodzi o uzycie operatora XOR). Zauważyłem, że przy usunieciu członu PINB... z warunku if mruganie nie wystepuje(i oczywiscie jeden przycisk nie działa). Dodam jeszcze że pracuje na atmega8L 8MHz.

Z góry dziękuje za pomoc

[syntax=c]#include
#include

#define LED_PIN (1<#define LED_ON PORTB &= ~LED_PIN
#define LED_OFF PORTB |= LED_PIN
#define LED_TOG PORTB ^= LED_PIN

#define KEY1_PIN (1<#define KEY2_PIN (1<#define KEY3_PIN (1<
uint8_t klawisz_wcisniety(uint8_t klawisz);

int main()
{
DDRB |= LED_PIN;
PORTB |= (LED_PIN | (1<
DDRC &= ~(KEY2_PIN | KEY3_PIN);
PORTC |= (KEY2_PIN | KEY3_PIN);

while(1)
{
if(klawisz_wcisniety(KEY1_PIN)) LED_ON;
if(klawisz_wcisniety(KEY2_PIN)) LED_OFF;
if(klawisz_wcisniety(KEY3_PIN)) LED_TOG;
}
}

uint8_t klawisz_wcisniety(uint8_t klawisz)
{
if(!(PINC & klawisz) ^ !(PINB & klawisz))
{
_delay_ms(80);

if(!(PINC & klawisz) ^ !(PINB & klawisz)) return 1;
}
return 0;
}[/syntax]

Statystyki: Napisane przez Maju — 30 sie 2016, o 11:12