ATNEL tech-forum

Re: Dziwne zachowanie się kodu

2016-05-18T07:13:57+01:00

Jeżeli korzystasz z if() to nawet dla jednej funkcji/polecenia używaj nawiasów {}. Po pierwsze program jest czytelniejszy. Po drugie, gdy trzeba dodać coś nie ma problemu, bo wstawiasz funkcje/polecenia w ten nawias (a przy braku {} często się o tym zapomina i program nie działa, tak jak chce się). Po trzecie, gdy wyrobisz sobie odruch, że po każdym if() dajesz {, to unikasz kolejnego częstego błędu, jakim jest wstawianie średnika po if() - wtedy warunek działa tylko dla średnika. Mało tego, jeśli nawet na końcu linijki z if(){ wstawisz średnik to program będzie działał dobrze, gdyż średnik (czyli pusta instrukcja) będzie w bloku obsługi polecenia if() - ot przykład:
[syntax=c]if(a=0); //tu if sprawdza warunek i wykonuje gdy a=0 instrukcję pustą oraz w każdym przypadku LED_ON- program nieprawidłowo działa
LED_ON;

if(a=0){; //tu if sprawdza warunek i wykonuje gdy a=0 grupę instrukcji - pustą i LED_ON - program działa prawidłowo
LED_ON;
}[/syntax]

Statystyki: Napisane przez PJS — 18 maja 2016, o 07:13

Re: Dziwne zachowanie się kodu

2016-05-18T05:03:52+01:00

W przypadku if jeśli masz tylko jedną czynność do wykonania to nie musisz używać nawiasów klamrowych, jeśli masz więcej niż jedną to już musisz inaczej program pomimo nie zgodnego warunku wykona te czynności poza pierwszą leżącą za ifem. Co do #define (przepraszam za mój nie profesjonalny język itp.) po prostu w define możesz dać kilka linijek kodu działa to jak makro. Tak jak masz w przypadku z BB, dzięki czemu za jednym zamachem możesz ustawiać bity na kilku portach co ułatwia sprawę, musisz je oddzielać średnikiem poza ostatnią inaczej program zakomunikuje błąd. Średnik daje informację, że za nim jest kolejna funkcja. A przynajmniej ja to tak rozumuje. Generalnie Define ułatwia bardzo życie

Statystyki: Napisane przez k_lukasik — 18 maja 2016, o 05:03

Re: Dziwne zachowanie się kodu

2016-05-17T21:21:44+01:00

Dziękuje o to chodziło. Mam jeszcze jedno pytanie odnośnie używania nawiasów klamrowych:

np. Jak zapisuje zmienne to:

int a, b, c; mogę oddzielić je przecinkami
np. jak mam if (x==2&&y==1)
{LED;
LED2;
}
jak mogę zapisać to bez nawiasów klamrowych? Jak bym miał tylko samo LED to bym zapisał:
if (x==2&&y==1) LED;
Oddzielenie przecinkami nie działa.

Pan Mirek w BB użył takiej makrodefinicji:
#define DC_LEWO PORTB &= ~WE_A; PORTB |= WE_B
Po WE_B nie powinno być; Czemu w zmiennych żeby oddzielić zmienne używamy przecinków a tutaj średników?

Statystyki: Napisane przez elektronikq — 17 maja 2016, o 21:21

Re: Dziwne zachowanie się kodu

2016-05-17T14:05:39+01:00

Dokładnie to co powiedział kolega Jacek, aby to co chcesz zrobić zadziałało powinieneś wg mnie dodać jakąś zmienną którą byś zmieniał w przerwaniu i sprawdzał dodatkowo w warunku np [syntax=c]uint8_t zapalona = 0;
if(x==4 && !zapalona) {LED2; zapalona = 1;}[/syntax]
, a w przerwaniu np.
[syntax=c]ISR(TIMER1_COMPA_vect)
{
x++;
if(zapalona) zapalona = 0;
}[/syntax]
Wtedy dioda się zapali i nie zgaśnie dopiero przy następnym x==4 dioda zgaśnie czyli za ok 251 sekund

Statystyki: Napisane przez k_lukasik — 17 maja 2016, o 14:05

Re: Dziwne zachowanie się kodu

2016-05-17T14:00:04+01:00

W czasie gdy x ma wartość równą cztery to stan diody w pętli while nie zmieni się jeden raz tylko tysiące razy. Zmienna x wartość równą cztery będzie miała na czas jednej sekundy. W tym czasie pętla while przekręci się tysiące razy. I za każdym razem zmieni się stan diody led. Czyli dioda tak naprawdę nie świeci światłem ciągłym tylko bardzo szybko miga.

Podejrzewam, że w momencie gdy x zmienia swą wartość z 4 na 5 dioda jest akurat wygaszona. Dlatego wydaje ci się, że dioda przez sekundę świeci a potem gaśnie.

Dla testu wstaw do pętli while opóźnienie np. 100ms aby to sprawdzić.

Statystyki: Napisane przez jacekk232 — 17 maja 2016, o 14:00

Dziwne zachowanie się kodu

2016-05-17T13:37:01+01:00

[syntax=c]#include
#include

#define LED (1<#define LED2 PORTB ^=LED

volatile uint8_t x;
int main (void)
{
TCCR1B |= (1<TCCR1B |= (1<OCR1A = 31249;
TIMSK |= (1<sei();

DDRB |= WE_A|WE_B|PWM| LED | LED1;
PORTB |= KEY| LED;
while(1)
{
if (x==4) LED2;

}
}

ISR(TIMER1_COMPA_vect)
{
x++;
}[/syntax]

W tym kodzie przerwanie występuje co 1 sek. I po 4 sekundach ma się zapalić dioda i powinna świecić aż x się wyzeruje i doliczy do 4 sekund i powinna zgasnąć. A u mnie dioda zapala się po 4 sekundach czyli dobrze i świeci ok 1 sekund i gaśnie.
Natomiast jak zrobię tak:
[syntax=c]#include
#include

#define LED (1<#define LED2 PORTB &=~LED

volatile uint8_t x;
int main (void)
{
TCCR1B |= (1<TCCR1B |= (1<OCR1A = 31249;
TIMSK |= (1<sei();

DDRB |= WE_A|WE_B|PWM| LED | LED1;
PORTB |= KEY| LED;
while(1)
{
if (x==4) LED2;

}
}

ISR(TIMER1_COMPA_vect)
{
x++;
}[/syntax]
Dioda się zapali po 4 sekundach i świeci cały czas.
PYTANIE:
Dlaczego w 1 przypadku dioda gaśnie po 1 sekundzie?

Statystyki: Napisane przez elektronikq — 17 maja 2016, o 13:37