ATNEL tech-forum

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-25T17:56:02+01:00

dayer41 napisał(a):

Przeczytałem BB od deski do deski, teraz biorę się za praktyczną część. Chcę jednym switchem obsługiwać jedną diodę led,

Zacznij od tego co chcesz robić
Potem zrób kolejny krok -

dayer41 napisał(a):

tak aby po pierwszym wciśnięciu migała co np. 0,5s, po drugim wciśnięciu do 1s, po trzecim np. co 2s, a po czwartym gasła.

Wiem, wiem klawisz to liczba nie większa od 255(uint8_t).
Przypuszczam, że klawisz ma być flagą przechowującą ilość wciśnień KEY_... - brakuje tego w kodzie.

Statystyki: Napisane przez Wirnick — 25 maja 2015, o 17:56

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:37:10+01:00

No faktycznie. Dzięki za szybkie odpowiedzi, oj jeszcze dłuuga droga przede mną

Statystyki: Napisane przez dayer41 — 24 maja 2015, o 15:37

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:33:21+01:00

zobacz że w trakcie delaya przy ledzie nie będzie reakcji na przycisk i koło się zamknie ... przeglądnij / oglądnij poradniki Mirka i wtedy polecam zacząć program od nowa.

Statystyki: Napisane przez majster — 24 maja 2015, o 15:33

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:34:09+01:00

O szok, ale idiotyczne błędy..

Na razie obsługę klawisza robię dość prymitywnie na delayach, bo chodziło mi o realizację samego migania, ale fakt, zmienię ją na taką nieopóźniającą.
A o poradnikach do migania LEDami nie wiedziałem, umknęły mi Zaraz się wezmę za oglądanie.

skalarro, funkcję off umieściłem po to, żeby wyzerować zmienną klawisz w funkcji miganie, chociaż teraz widzę, że to jest bez sensu.. zapewne trzeba użyć wskaźnika?

Statystyki: Napisane przez dayer41 — 24 maja 2015, o 15:30

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:28:10+01:00

ogólnie na dzien dobry kilka poprawek:

[syntax=c]if (miganie (2))[/syntax] zmień na [syntax=c]if (miganie()==2)[/syntax]

i wywal te off; z 43 linii. Po co to??

Statystyki: Napisane przez skalarro — 24 maja 2015, o 15:28

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:25:00+01:00

Obejrzyj sobie kilka moich poradników na YT o miganiu diodami LED
http://mirekk36.blogspot.com/2014/05/mi ... rawki.html

i podobny, poza tym poczytaj o lepszej obsłudze klawiszy w C na blogu:
http://mirekk36.blogspot.com/2012/10/ob ... w-cd2.html

a najlepiej to połącz te poradniki z książką:
http://atnel.pl/mikrokontrolery-avr-jezyk-c.html

wtedy dasz radę

Statystyki: Napisane przez mirekk36 — 24 maja 2015, o 15:25

Re: Jeden switch - kilka akcji, warn "statement with no effe

2015-05-24T15:24:13+01:00

Co to jest ....

If(miganie (x))

Powinno być

If (miganie == x){

}

Czytałeś bb to pozbadz sie delay.
Czytałeś bb to zerknij do niego nt warunków if.

Namazane ze smarta.

Statystyki: Napisane przez majster — 24 maja 2015, o 15:24

Jeden switch - kilka akcji, warn "statement with no effect"

2015-05-24T15:18:31+01:00

Witam, mam nadzieję, że mnie nie zabijecie ale już nie mam pomysłów.
Przeczytałem BB od deski do deski, teraz biorę się za praktyczną część. Chcę jednym switchem obsługiwać jedną diodę led, tak aby po pierwszym wciśnięciu migała co np. 0,5s, po drugim wciśnięciu do 1s, po trzecim np. co 2s, a po czwartym gasła. Naskrobałem coś takiego:
[syntax=c]#include
#include

#define LED_PIN (1<#define LED_ON PORTC &= ~(LED_PIN)
#define LED_OFF PORTC |= (LED_PIN)
#define LED_TOG PORTC ^= LED_PIN
#define KEY_PIN (1<#define KEY_CHK !(PINC & KEY_PIN)

uint8_t miganie(uint8_t klawisz);
uint8_t off(uint8_t klawisz);

int main(void)
{
DDRC |= LED_PIN;
PORTC |= LED_PIN;
DDRC &= ~KEY_PIN;
PORTC |= KEY_PIN;

while(1)
{
if (miganie (1))
{
LED_TOG;
_delay_ms(500);
}
if (miganie (2))
{
LED_TOG;
_delay_ms(1000);
}
if (miganie (3))
{
LED_TOG;
_delay_ms(2000);
}
if (miganie (4))
{
LED_OFF;
off;
}
else
{
LED_OFF;
}
}
}

uint8_t miganie(uint8_t klawisz)
{
if(!(PINC & klawisz))
{
klawisz++;
return klawisz;
}
return klawisz;
}

uint8_t off(uint8_t klawisz)
{
klawisz=0;
LED_OFF;
return klawisz;
}[/syntax]
W linijce 43 eclipse krzyczy, że "statement with no effect [-Wunused-value]", czyli chyba, że funkcja nic nie robi.
Gdzie tkwi błąd? I czy w ogóle ten kod ma sens?

Statystyki: Napisane przez dayer41 — 24 maja 2015, o 15:18