ATNEL tech-forum

Re: Funkcja zmiany stanu na wyjściu led_on(1);

2021-05-10T08:52:24+01:00

To tylko przykład. Nie ma obowiązku stosowania ani struktur ani typów wyliczeniowych. Chodzi o ideę - pozbycie się kłopotliwych ifów prostymi technikami. W pierwszym przypadku numer LEDa traktujemy jako indeks do tablicy, która zawiera paskudną część kodu i co najważniejsze skupia ją w jednym miejscu. W razie modyfikacji robi się to nie w n funkcjach tylko w jednej tablicy. Przypadek drugi podobnie tyle że obrzydliwości ukrywamy pod stałą symboliczną.

Statystyki: Napisane przez fofex — 10 maja 2021, o 08:52

Re: Funkcja zmiany stanu na wyjściu led_on(1);

2021-05-08T22:23:19+01:00

Ło Matko...

Na pierwszy rzut oka nic z tego nie rozumiem, ale jest późno i może dlatego... Jutro ze świeżą głową się temu przyjrzę... ale najpierw muszę zgłębić jeszcze raz tematy struktur i typów wyliczeniowych...

Dziękuję!

Statystyki: Napisane przez argentrol — 8 maja 2021, o 22:23

Re: Funkcja zmiany stanu na wyjściu led_on(1);

2021-05-08T21:37:02+01:00

Zależy czy chcesz by numer diody był kolejną liczbą czy też wystarczy odwoływanie się przez nazwę symboliczną. W pierwszym przypadku można zastosować tablicę "tłumaczącą" nr na konkretny pin i port. W drugim przypadku można zrobić zwykłą operację na porcie. W najbardziej podstawowym rozwiązaniu jest ograniczenie, że LEDy muszą się znajdować na jednym porcie. Po pewnej rozbudowie można to obejść.

przypadek 1.

[syntax=c]struct led {
uint8_t volatile *port;
uint8_t pin;
_Bool inverted; //true dla Anoda-V+, Katoda-pin, false dla Anoda-pin, katoda-GND
}
static struct led led[LED_NUMBER] = {
{&PORTA, 0,1},
{&PORTB,0,0},
{&PORTC,0,1}
}

static inline void led_hi(uint8_t ledno)
{
*led[ledno].port |= 1<}

static inline void led_low(uinnt8_t ledno)
{
*led[ledno].port &= ~(1<}

void led_on(uint8_t ledno)
{
led[ledno].inverted) ? led_low(ledno) : led_hi(ledno);
}

void led_of(uint8_t ledno)
{
led[ledno].inverted) ? led_hi(ledno) : led_low(ledno);
}

void led_toggle(uint8_t ledno)
{
*led[ledno].port ^= 1<}

int main()
{
led_toggle(0);
led_toggle(2);
led_on(0);
led_on(1);
led_on(2);
led_off(0);
led_off(1);
led_off(2);
}[/syntax]

Przypadek drugi

[syntax=c]#define LED1NR 6
#define LED2NR 5
#define LED3NR 4
#define LED_PORT PORTD
#define LED_INVERTED ((1<
enum led {
LED1 = (1< LED2 = (1< LED3 = (1<};

void led_on (enum led led){
LED_PORT &= ~led;
LED_PORT |= (LED_INVERTED & led) ^ led;
}

void led_off (enum led led){
LED_PORT |= led;
LED_PORT &= ~((LED_INVERTED & led) ^ led);
}

void led_tog (enum led led){
LED_PORT ^ led;
}

int main()
{
led_on(LED1);
led_toggle(LED2);
led_toggle(LED3);
led_off(LED1);
}[/syntax]

Statystyki: Napisane przez fofex — 8 maja 2021, o 21:37

Re: Funkcja zmiany stanu na wyjściu led_on(1);

2021-05-08T21:04:48+01:00

Jeśli chcesz mieć zupełną niezależność od pinu to tylko zabawa z else if w takim przypadku

Statystyki: Napisane przez mirekk36 — 8 maja 2021, o 21:04

Funkcja zmiany stanu na wyjściu led_on(1);

2021-05-08T20:04:57+01:00

Witam Was.

Szczerze mówiąc, to przypuszczam, że odpowiedź na moje pytanie gdzieś na forum jest, tylko że ja nie wiem jak to pytanie dobrze zadać. Zaczynam z językiem C, uczę się programować na Atmedze i nie umiem jeszcze nazwać niektórych rzeczy "po imieniu".

Mam taki mały kod, ale ugrzęzłem na takim czymś:

Chciałbym w main.c użyć funkcji np. led_on(x); gdzie (x) ma oczywiście oznaczać nr diody (ale nie nr pinu!). W tej chwili mam to zrobione tak, że funkcja sprawdza to na zasadzie:

[syntax=c]#define LED1 (1<#define LED2 (1<#define LED3 (1<
#define LED1_ON PORTD |= LED1
#define LED1_OFF PORTD &= ~LED1
#define LED1_TOG PORTD ^= LED1
#define LED2_ON PORTD |= LED2
#define LED2_OFF PORTD &= ~LED2
#define LED2_TOG PORTD ^= LED2
#define LED3_ON PORTD |= LED3
#define LED3_OFF PORTD &= ~LED3
#define LED3_TOG PORTD ^= LED3

void led_on (uint8_t x){
if(x==1) LED1_ON;
else if(x==2)LED2_ON;
else if(x==3)LED3_ON;
}

void led_off (uint8_t x){
if(x==1) LED1_OFF;
else if(x==2)LED2_OFF;
else if(x==3)LED3_OFF;
}

void led_tog (uint8_t x){
if(x==1) LED1_TOG;
else if(x==2)LED2_TOG;
else if(x==3)LED3_TOG;
}[/syntax]

To na pewno da się zrobić jakoś mądrzej, żeby w przypadku konieczności użycia np. czwartej diody nie musieć dopisywać do każdej funkcji kolejnego "else if", dyrektyw preprocesora (wiem, że one akurat nie są konieczne) itd.

Kombinowałem z jakimiś składniami typu: zamiast LED1 to LED(x), próbowałem użyć tych makr dostępu do portów, kombinowałem jak użyć tablic... nie umiem nic mądrego wymyślić...
Naprowadzi mnie ktoś na jakiś azymut?

Tylko bardzo proszę, jak najprostszym, lub chociaż w miarę prostym językiem... Dziękuję za wyrozumiałość...

Bartek

Statystyki: Napisane przez argentrol — 8 maja 2021, o 20:04