ATNEL tech-forum

Re: Zamiana "if" na "case"

2014-12-01T22:19:49+01:00

mokrowski napisał(a):

Powiem wprost.. proszę ... nie pisz takiego kodu! Co z tego że inni piszą jeszcze gorszy (np. tasiemiec switch-case który wyłącza i włącza diody... )

W programowaniu obowiązuje zasada że kod który się powtarza, łamie regułę DRY (ang. Don't Repeat Yourself). U Ciebie jest ,,stado swirch-case" a możesz to zrobić bardzo prosto definiując tablicę z wartościami każdego z warunków i sprawdzając czy dane mieszczą się w zakresie. Indeks dopasowanej komórki tablicy to numer klawisza. Prosto, łatwo i skutecznie bez ,,tasiemca" z switch-case. Zwykłe for po tablicy z 1 if a w środku break po dopasowaniu! I kod mniejszy i szybciej będzie działało niż to.. coś....

PS. Jak miałbym wskazać jaką konstrukcję usunąć z C, to było by to switch-case. W 99% można zastąpić ją czytelniejszym if lub tablicą. Twój przypadek nie należy do tego 1%

PS2. No i jeszcze kwiatki ze zmienną globalną.. eh..

Jaka jest objętość wsadu do MCU teraz? Jak poprawisz na tablicę i przemyślisz zwracanie klawisza w metodzie sprawdz, to zobaczysz że kod _bardzo_ zmaleje

Bardzo dziękuję za uwagę i inną propozycję rozwiązania mojego problemu. Powiem szczerze, że nie podobało mi się wersja z poleceniem case ale nie wpadłem na inne rozwiązanie
[syntax=c]/*
* main.c
*
* Created on: 29-11-2014
* Author: Michał Szymocha
*/

#include
#include
#include
#include

#include "LCD/lcd44780.h"

#define x 3

uint16_t pomiar(uint8_t kanal);
uint8_t sprawdz();

uint16_t warunek[16] = {743, 990, 927, 967, 472, 600, 415, 536, 297, 371, 241, 327, 158, 199, 140, 179};
uint16_t raw;
uint8_t key, i;

int main( void )
{

lcd_init();
ADMUX |= (1<ADCSRA |= (1<

while(1)
{

key = sprawdz();

lcd_locate(0,0);
lcd_int( pomiar(7) );
lcd_str(" ");
lcd_locate(1,0);
lcd_int( key );
lcd_str(" ");

}
}
uint16_t pomiar(uint8_t kanal )
{
ADMUX = (ADMUX & 0xF8) | kanal;
ADCSRA |= (1<while( ADCSRA & (1<return ADCW;
}
uint8_t sprawdz()
{
raw = pomiar(7);

for(i = 0; i < 16; i++)
{
if(raw >= (warunek[i] - x) && raw <= (warunek[i] + x) ) //(raw >= 743 - x && raw <= 743 + x )
{
key = i+1;
break;
}else key = 0;
}
return key;
}[/syntax]

Kod z instrukcją case zajmował 1646b, a kod z tablicą i instrukcją for 1438b.

Proszę jeszcze o komentarz odnośnie schematu w eagle'u

Statystyki: Napisane przez michal98 — 1 gru 2014, o 22:19

Re: Zamiana "if" na "case"

2014-12-01T20:42:31+01:00

[syntax=c]Case 1 , 2:

Case 1 . 2:

Case (1 . 2):

Case (1 , 2):

Case 1 2:[/syntax]

Statystyki: Napisane przez michal98 — 1 gru 2014, o 20:42

Re: Zamiana "if" na "case"

2014-12-01T20:38:27+01:00

michal98 napisał(a):

działa prawidłowo, a zapisany w każdy inny sposób eclipse podkreśla na pomarańczowo z błędem "Syntax error"

"Każdy inny sposób" tzn. jak?

Statystyki: Napisane przez kaka0204 — 1 gru 2014, o 20:38

Re: Zamiana "if" na "case"

2014-12-01T20:22:01+01:00

Dzięki za odpowiedź.

Zmieniłem typ funkcji z uint8_t na void i usunąłem "return 0;" nie potrzebuje nic zwracać, ponieważ ta funkcja zapisuje wynik do zmiennej globalnej "key"

Warunek
[syntax=c]case 743 - x ... 743 + x:[/syntax]
działa prawidłowo, a zapisany w każdy inny sposób eclipse podkreśla na pomarańczowo z błędem "Syntax error"

Statystyki: Napisane przez michal98 — 1 gru 2014, o 20:22

Re: Zamiana "if" na "case"

2014-12-01T20:09:29+01:00

Napisałeś

[syntax=c]uint8_t sprawdz();[/syntax]

Źle musisz podać typ argumentu. Ta funkcja będzie korzystała z argumentu typu void. To po pierwsze. Po drugie w funkcji sprawdz() napisałeś

[syntax=c]return 0;[/syntax]

A więc funkcja zawsze zwróci 0. Napisz tak:

[syntax=c]return key; // zapewne o tą wartość ci chodzi :)[/syntax]

Po trzecie.

[syntax=c]case 743 - x ... 743 + x:[/syntax]

Ten warunek jest błędy. Kolega jacekk232 wielokropek podał dla przykładu. Musisz to zmienić

Pozdrawiam.

Statystyki: Napisane przez kaka0204 — 1 gru 2014, o 20:09

Re: Zamiana "if" na "case"

2014-12-01T19:18:53+01:00

Dziękuję za pomoc.

Potrzebowałem tego gdyż programowałem obsługę keypada 4x4 nota katalogowajednym pinem procesora za pomocą ADC. Potrzebowałem użyć zakresu liczb, gdyż jak wiadomo rezystory mogą minimalnie zmieniać swoją rezystancję np. pod wpływem temperatury.

Kod który napisałem :
[syntax=c]/*
/*
* main.c
*
* Created on: 29-11-2014
* Author: Michał Szymocha
*/

#include
#include
#include
#include

#include "LCD/lcd44780.h"

#define x 8

uint16_t pomiar(uint8_t kanal);
uint8_t sprawdz();

uint16_t raw;
uint8_t key;
int main( void )
{

lcd_init();
ADMUX |= (1<ADCSRA |= (1<

while(1)
{

sprawdz();

lcd_locate(0,0);
lcd_int( pomiar(7) );
lcd_str(" ");
lcd_locate(1,0);
lcd_int( key );
lcd_str(" ");

}
}
uint16_t pomiar(uint8_t kanal )
{
ADMUX = (ADMUX & 0xF8) | kanal;
ADCSRA |= (1<while( ADCSRA & (1<return ADCW;
}
uint8_t sprawdz()
{
raw = pomiar(7);
switch(raw)
{
case 743 - x ... 743 + x:
key = 1;
break;
case 988 - x ... 988 + x:
key = 2;
break;
case 921 - x ... 921 + x:
key = 3;
break;
case 966 - x ... 966 + x:
key = 4;
break;
case 476 - x ... 476 + x:
key = 5;
break;
case 601 - x ... 601 + x:
key = 6;
break;
case 415 - x ... 415 + x:
key = 7;
break;
case 542 - x ... 542 + x:
key = 8;
break;
case 281 - x ... 281 + x:
key = 9;
break;
case 359 - x ... 359 + x:
key = 10;
break;
case 233 - x ... 233 + x:
key = 11;
break;
case 318 - x ... 318 + x:
key = 12;
break;
case 153 - x ... 153 + x:
key = 13;
break;
case 195 - x ... 195 + x:
key = 14;
break;
case 135 - x ... 135 + x:
key = 15;
break;
case 176 - x ... 176 + x:
key = 16;
break;
default:
key = 0;
break;
}
return 0;
}

}[/syntax]

A oto schemat stworzony w Eagle'u

p1 : Czy kod w C jest napisany poprawnie, bez jakichś błędów których ja nie widzę z uwagi na to, że jestem początkujący ?
p2 : Czy połączenia na schemacie są wykonane prawidłowo ?

Statystyki: Napisane przez michal98 — 1 gru 2014, o 19:18

Re: Zamiana "if" na "case"

2014-12-01T01:12:05+01:00

W gcc można użyć takiej konstrukcji
[syntax=c]switch(x)
{
case 1 ... 5:
funkcja();
case 10 ... 25:
funkcja2();
}[/syntax]

Statystyki: Napisane przez jacekk232 — 1 gru 2014, o 01:12

Re: Zamiana "if" na "case"

2014-12-01T00:18:21+01:00

michal98 napisał(a):

(...) ale z tego co widzę to zamiast 16 linijek ten warunek miałby 160.

Możesz jeszcze spróbować napisać taką funkcję:
[syntax=c]int czy_jest_w_zakresie(int zmienna_testowana, int xmin, int xmax)
{
for (int i = xmin; i < xmax; i++)
{
if (zmienna_testowana == i)
{
return 0; // jest w zakresie
}
else return 1; // nie jest w zakresie
}[/syntax]

ewentualnie taką:
definicja:
[syntax=c]int czy_jest_w_zakresie_tab(zmienna_testowana2, int *tab_zakres)
{
const int c = (sizeof(tab_zakres) / sizeof(tab_zakres[0]));
for (int i = 0; i < c; i++)
{
if (zmienna_testowana2 == tab_zakres[i])
{
// jakieś funkcje jak jest w zakresie
return 0;
}
else
{
// jakieś funkcje jak nie jest w zakresie
return 1;
}
}[/syntax]

zastosowanie w kodzie dla funkcji czy_jest_w_zakresie_tab():
[syntax=c]int zakres_1[] = { 0, 1, 2, 3 }; // możliwe wartości stałych dcb
int zakres_2[] = { 2, 5, 10, 12, 18 };

czy_jest_w_zakresie_tab(3, int zakres_1); // jest
czy_jest_w_zakresie_tab(-2, int zakres_1); // nie jest
czy_jest_w_zakresie_tab(3, int zakres_2); // nie jest
czy_jest_w_zakresie_tab(12, int zakres_2); // jest[/syntax]

Mam nadzieję, że nic nie pomieszałem.

Pozdrawiam! j23 Jarek

Statystyki: Napisane przez j23 — 1 gru 2014, o 00:18

Re: Zamiana "if" na "case"

2014-11-30T23:31:54+01:00

Dzięki za szybką odpowiedź

Ja mam takich warunków if 16 i porostu myślałem że mogę jakoś uprościć mój kod, ale z tego co widzę to zamiast 16 linijek ten warunek miałby 160.

Statystyki: Napisane przez michal98 — 30 lis 2014, o 23:31

Zamiana "if" na "case"

2014-11-30T23:22:02+01:00

Czy można zapisać poniższy kod z użyciem polecenia case ?

[syntax=c]if (x > 1 && x < 5)
{
funkcja;
}[/syntax]

Statystyki: Napisane przez michal98 — 30 lis 2014, o 23:22