miki53 napisał(a):
Mógłbym prosić o jakieś podpowiedzi jak uprościć
ten fragment kodu
[syntax=c]// **************************** USTAWIANIE WYJŚCIA NR-1 ********************************************
if( 32 == cmd && key_time == 0 ) { // klawisz pilota strzałka do DOŁU V
on_off = 0;
// beep(20);
}
if( 33 == cmd && key_time == 0 ) { // klawisz pilota strzałka do GÓRY ^
on_off = 1;
// beep(20);
}
switch (on_off)
{
case 0:
if ( 55 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY CZERWONY 55 ( R )
_1_on_h++;
beep(20);
}
if ( 54 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY ZIELONY 54 ( G )
_1_on_m++;
beep(20);
}
if ( 50 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY ŻÓŁTY 50 ( Y )
_1_on_s++;
beep(20);
}
if ( 52 == cmd && key_time ){ // -- klawisz pilota MAŁY NIEBIESKI 52 ( B )
_1_on_wy++;
beep(20);
}
if( _1_on_s>59) {
_1_on_s=0;
}
if( _1_on_m>59) {
_1_on_m=0;
}
if( _1_on_h>23) {
_1_on_h=0;
}
if( _1_on_wy>8) {
_1_on_wy=0;
}
// lcd_cls(); // czyszczenie ekranu LCD
lcd_locate(0,5);
lcd_str("WY_1 ON ");
wyswietlacz(_1_on_s, _1_on_m, _1_on_h, _1_on_wy );
break;
case 1:
if ( 55 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY CZERWONY 55 ( R )
_1_off_h++;
beep(20);
}
if ( 54 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY ZIELONY 54 ( G )
_1_off_m++;
beep(20);
}
if ( 50 == cmd && key_time ){ // -- KLAWISZ PILOTA MAŁY ŻÓŁTY 50 ( Y )
_1_off_s++;
beep(20);
}
if ( 52 == cmd && key_time ){ // -- klawisz pilota MAŁY NIEBIESKI 52 ( B )
_1_off_wy++;
beep(20);
}
if( _1_off_s>59) {
_1_off_s=0;
}
if( _1_off_m>59) {
_1_off_m=0;
}
if( _1_off_h>23) {
_1_off_h=0;
}
if( _1_off_wy>8) {
_1_off_wy=0;
}
// lcd_cls(); // czyszczenie ekranu LCD
lcd_locate(0,5);
lcd_str("WY_1 OFF ");
wyswietlacz(_1_off_s, _1_off_m, _1_off_h, _1_off_wy );
break;
}
if( 59 == cmd && key_time == 0 ) { // klawisz pilota NIEBIESKI ŚRODKOWY -> OK / zapis do pamięci
ram_wyj[0] . on_hh = _1_on_h;
ram_wyj[0] . on_mm = _1_on_m;
ram_wyj[0] . on_ss = _1_on_s;
ram_wyj[0] . on_wy = _1_on_wy;
ram_wyj[0] . off_hh = _1_off_h;
ram_wyj[0] . off_mm = _1_off_m;
ram_wyj[0] . off_ss = _1_off_s;
ram_wyj[0] . off_wy = _1_off_wy;
copy_ram_to_eem();
beep(20);
}
if( 17 == cmd && key_time == 0 ) { // klawisz pilota w LEWO <<<
pozycja = 0;
on_off = 0;
beep(20);
}
// ******************************************************************************************************
break;[/syntax]
w tym fragmencie mam 8 zmiennych do których są wpisywane wartości za pomocą pilota
i mam takich sekcji 8 jak widać w kodzie wyżej
i teraz jak za pomocą tego jednego kodu do nastawiania, ustawić wartości dla wszystkich
zmiennych niezależnie
Nawet jak wezmę tablicę to każda zmienna musi mieć swój indeks i jak ustawiać
hh,mm,ss,wy
Jak zmieniać nazwy zmiennych w kodzie do ustawiania wartości aby trafiały tam gdzie mają
np. tak dla case 0:
[syntax=c]void increment(uint8_t *data, uint8_t cmd, uint8_t cmdReceived, uint8_t key_time)
{
if(cmd == cmdReceived && key_time)
{
(*data)++;
beep(20);
}
}
void checkLimit(uint8_t *data, uint8_t limit)
{
if((*data) > limit)
{
*data = 0;
{
}
case 0:
increment(&_1_on_h, 55, cmd, key_time);
increment(&_1_on_m, 54, cmd, key_time);
increment(&_1_on_s, 50, cmd, key_time);
increment(&_1_on_wy, 52, cmd, key_time);
checkLimit(&_1_on_s, 59);
checkLimit(&_1_on_m, 59);
checkLimit(&_1_on_h, 23);
checkLimit(&_1_on_wy, 8);
lcd_cls(); // czyszczenie ekranu LCD
lcd_locate(0,5);
lcd_str("WY_1 ON ");
wyswietlacz(_1_on_s, _1_on_m, _1_on_h, _1_on_wy );
break;[/syntax]Statystyki: Napisane przez Jarecki — 8 wrz 2020, o 07:18
]]>
