ATNEL tech-forum 2018-10-24T09:22:21+01:00 https://forum.atnel.pl/feed.php?f=58&t=21469&mode 2018-10-24T09:22:21+01:00 2018-10-24T09:22:21+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212938#p212938 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> kalani napisał(a):

Toteż piszę: zostawić superdebounce w spokoju i dane przekazywać do callbacka przez zm. globalne o zasiegu plikowym:



Wiem o co biega tak jak piszesz będzie najprościej.

------------------------ [ Dodano po: 2 minutach ]

mirekk36 napisał(a):

i w ogóle nie ingerujesz w kod SuperDebounce() .... a wszystko działa tak jak chcesz - i może na tym przykładzie zaskoczysz - do czego są callbacki.



Czaję, poszedłem nie tą drogą co trzeba.

Statystyki: Napisane przez Krzycho69 — 24 paź 2018, o 09:22

]]> 2018-10-24T09:20:04+01:00 2018-10-24T09:20:04+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212937#p212937 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> Krzycho69 napisał(a):

mam strukturę temperatura jak ją przekazać do SuperDebynceP?

Kolega na razie jak widzę KOMPLETNIE nie rozumie callbacków :( .... Po jasny gwint przekazywać do tej funkcji superdebounce jakieś dodatkowe argumenty - toż to nieporozumienie praktycznie ... Callback tu jest po to aby nie ruszać kodu funkcji a reszta ma być robiona w funkcji callbackowej czyli tej wywołanej ....

To w niej jak kolega wyżej pisze możesz operować DOWOLNYMI zmiennymi - jakie tylko zechcesz (globalne - jakaś struktura itp) ... zapomnij najlepiej o przekazywaniu dodatkowych argumentów do superdebounce bo za chwilę wyjdzie ci z tego takie SPAGHETTI - że sam się nie połapiesz o co chodzi.

Krótko, Superdebounce wywoła twoją funkcję np:

[syntax=c]void klawisz_plus( void );[/syntax]

no toż po co w niej jakieś argumenty skoro wiesz, że ona ma zwiększać "coś tam" - nie ważne czy temperaturę czy co innego - masz jakąś gdzieś zmienną np: temperatura to w tej funkcji napiszesz sobie:

[syntax=c]void klawisz_plus( void ) {
temperatura++;
// ew wyświetlisz tę temperaturę tutaj
}[/syntax]

chyba nie muszę mówić, że analogicznie możesz sobie zrobić funkcję:

[syntax=c]void klawisz_minus( void );[/syntax]

i w ogóle nie ingerujesz w kod SuperDebounce() .... a wszystko działa tak jak chcesz - i może na tym przykładzie zaskoczysz - do czego są callbacki.

Statystyki: Napisane przez mirekk36 — 24 paź 2018, o 09:20

]]> 2018-10-24T08:35:25+01:00 2018-10-24T08:35:25+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212932#p212932 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> int x1;
int max;
int min;
}t_ustawienia;

t_ustawienia temperatura;[/syntax]

mam strukturę temperatura jak ją przekazać do SuperDebynceP?

[syntax=c]void SuperDebounceP(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(int x1,int max,int min), void (*rep_proc)(int x1,int max,int min) ) {

enum {idle, debounce, go_rep, wait_rep, rep};


if(!rep_time) rep_time=20;
if(!rep_wait) rep_wait=150;

uint8_t key_press = !(*KPIN & key_mask);

if( key_press && !*key_state ) {
*key_state = debounce;
Timer1 = 15;
} else
if( *key_state ) {

if( key_press && debounce==*key_state && !Timer1 ) {
*key_state = 2;
Timer1=5;
} else
if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc(x1,max,min);/* KEY_UP */
*key_state=idle;
} else
if( key_press && go_rep==*key_state && !Timer1 ) {
*key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==*key_state && !Timer1 ) {
*key_state = rep;
} else
if( key_press && rep==*key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc(x1, max, min);/* KEY_REP */
}
}
if( *key_state>=3 && !key_press ) *key_state = idle;
}[/syntax]

Statystyki: Napisane przez Krzycho69 — 24 paź 2018, o 08:35

]]> 2018-10-24T08:27:12+01:00 2018-10-24T08:27:12+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212930#p212930 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> kalani napisał(a):

Superdebounce musi "znać" parametry wywoływanej funkcji. Jeśli są one zewnętrzne wobec funkcji debounce to trzeba je podać w parametrach tejże funkcji. Żeby nie zrujnować uniwersalności funkcji debounce, parametry te należy zamknąć w strukturze, i przekazywać ją do debounce jako wskaźnik. Oczywiście dalsze przekazanie parametru do funkcji zwrotnej także musi być przez wskaźnik. W ten sposób funkcja debunce nie musi nawet znać wewnętrznej budowy parametru i może pozostać uniwersalna dla dowolnego zestawu parametrów.



Sam tego nie ogarnę nie mój poziom :(.
Właśnie o to chodzi żeby SuperDebunce Była uniwersalna, ponieważ w trakcie programu chciałem parametry zmieniać.

- trzeba utworzyć strukturę
- ale jak wstawić parametry do SuberDebunce i funkcji zwrotnej ? - tego nie potrafię.

Statystyki: Napisane przez Krzycho69 — 24 paź 2018, o 08:27

]]> 2018-10-24T08:08:55+01:00 2018-10-24T08:08:55+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212928#p212928 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]>
[syntax=c]/*
* main.c
*
* Created on:
* Author: Krzysztof
*/

/* DDRx - rejestr kierunku 0-wejście, 1-wyjście np DDRC |=(1<<PC7) - PC7 wyjście
* DDRC &=~(1<<PC6) - PC6 jako wejście
* PORTx - rejestr wyjściowy ustawiamy 1 lub 0 na pinie np PORTC |=(1<<PC6) na wejściu 1 podciagniecie
* pinu do VCC
* PORTC&=~(1<<PC7) na wyjściu 0
*PINx - rejestr wejściowy do sprawdzania stanu Pinu np if !(PINC&=(1<<PC6))
* jeżeli do PC6 podłaczymy 0 to wyrażenie w
* nawisie da 0 a ! zmieni je na 1 czyli TRUE
*/


#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <util/delay.h>
#include "funkcje.h"
#include "LCD/lcd44780.h"// w ustawieniach wyzwietlacza sprawdzic czy RW do GND czy procesora

#define KL1 (1<<PD0)// definicje klawiszy do Super Debunce wejście do menu
#define KL2 (1<<PD1)// definicja klawisza do Superdebunce zwiększenie godzin
#define KL3 (1<<PC2)// definicja klawisza do SuperDebunce zwiekszenie minut


uint8_t k1, k2, k3, k4, k5, k6; // zmienne pomocnicze do obsługi Superdebunce


uint8_t znak_buzka[] = {14,17,27,17,17,21,17,14};// wzór znaku buźki w pamięci FLASH
uint8_t znak_termo[] = {4,10,10,10,17,31,31,14};// wzór znaku termometru w pamięci FLASH

void zwiekszenie(int zmienna, int max, int min);
void zwiekszenie_menu1(void);

int main(void){
PORTD |= KL1 | KL2; // podciaganie wejśc do 1, DDRD nie trzeba ustawiac na 0 ponieważ domyslnie
PORTC |= KL3;

lcd_init();//inicjalizacja wyświetlacza
timer2_init();


lcd_defchar(0x80, znak_buzka);//zaladowanie znaku buzka do pamieci CGRAM wyswietlacza
lcd_locate(0,1);//locate dac pomiedzy zaladowaniem znaku a wyswietleniem aby byl czas na zaladowanie
lcd_str("\x80");//wyswietlenie znaku buzka na wyswietlaczu

_delay_ms(1000);

lcd_defchar(0x81, znak_termo);
lcd_locate(0,4);
lcd_str("\x81");


lcd_locate(0,6);
lcd_str("proba");
_delay_ms(1000);
//lcd_cls();
x1=5;
min = 1;
max = 7;


sei();//włączenie przerwań zaremowane

//uint8_t a=0;
//uint16_t b=0;
while(1){

SuperDebounce(&k2, &PIND, KL2, 20, 50, zwiekszenie_menu1, zwiekszenie_menu1);
SuperDebounceP(&k1, &PIND, KL1, 20, 50, zwiekszenie, zwiekszenie);

lcd_locate(1,1);
lcd_int(x1);




}

}

void zwiekszenie(int x1, int max, int min) {
lcd_cls();
x1++;
bip();
if (x1 > max){
x1 = min;
}

}

void zwiekszenie_menu1(void) {
lcd_cls();
x1++;
bip();
//if (x > 3){
//x=0;
//}
}[/syntax]

funkcje.h

[syntax=c]/*
* funkcje.h
*
* Created on: 6 sty 2016
* Author: Krzysiek
*/

#ifndef STEROWNIK_CZASOWY_FUNKCJE_H_
#define STEROWNIK_CZASOWY_FUNKCJE_H_

#define BIP (1<<PC3) // Pin jako BIP
#define BIP_ON PORTC |= BIP // właczenie Buzera ustawienie na 1
#define BIP_OFF PORTC &=~BIP // wylaczenie Buzera ustawienie na 0
// deklaracje zmiennych

extern volatile uint16_t Timer1, Timer2, Timer3, Timer4, mstick; // deklaracja timerów programowych (extern - dostepne w innych plikach)

int x1, max, min;



// deklaracje funkcji


void timer2_init (void); // deklaracja funkcji ustawiającej timer2

void SuperDebounce(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) );

void SuperDebounceP(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(int x1,int max,int min), void (*rep_proc)(int x1,int max,int min) );

void bip (void); // deklaracja funkcji Buzera




#endif /* STEROWNIK_CZASOWY_FUNKCJE_H_ */[/syntax]

funkcje.c

[syntax=c]/*
*
*
* Created on: 6 sty 2016
* Author: Krzysztof Gładysz
*/

/* DDRx - rejestr kierunku 0-wejście, 1-wyjście np DDRC |=(1<<PC7) - PC7 wyjście
* DDRC &=~(1<<PC6) - PC6 jako wejście
* PORTx - rejestr wyjściowy ustawiamy 1 lub 0 na pinie np PORTC |=(1<<PC6) na wejściu 1 podciagniecie
* pinu do VCC
* PORTC&=~(1<<PC7) na wyjściu 0
*PINx - rejestr wejściowy do sprawdzania stanu Pinu np if !(PINC&=(1<<PC6))
* jeżeli do PC6 podłaczymy 0 to wyrażenie w
* nawisie da 0 a ! zmieni je na 1 czyli TRUE
*/

// dołączone pliki biblioteczne
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <util/delay.h>

// dołaczone moje pliki nagłówkowe

#include "funkcje.h"


// definicje zmiennych

volatile uint16_t Timer1, Timer2, Timer3, Timer4, mstick; // definicja timerów
//Timer1 w Superdebynce
//Timer4 co 1 s odczyt temperatury i ADC
int x1, max, min;



ISR(TIMER2_COMP_vect) // przerwanie do obsługi timerow programowych
{
uint16_t n;

n = Timer1;/* 100Hz Timer1 */
if (n) Timer1 = --n;
n = Timer2;/* 100Hz Timer2 */
if (n) Timer2 = --n;
n = Timer3;/* 100Hz Timer3 */
if (n) Timer3 = --n;
n = Timer4;/* 100Hz Timer4 */
if (n) Timer4 = --n;
mstick++;

}









/************** funkcja SuperDebounce do obsługi pojedynczych klawiszy ***************
* AUTOR: Mirosław Kardaś
* ZALETY:
* - nie wprowadza najmniejszego spowalnienia
* - posiada funkcję REPEAT (powtarzanie akcji dla dłużej wciśniętego przycisku)
* - można przydzielić różne akcje dla trybu REPEAT i pojedynczego kliknięcia
* - można przydzielić tylko jedną akcję wtedy w miejsce drugiej przekazujemy 0 (NULL)
*
* Wymagania:
* Timer programowy utworzony w oparciu o Timer sprzętowy (przerwanie 100Hz)
*
* Parametry wejściowe:
*
* *key_state - wskaźnik na zmienną w pamięci RAM (1 bajt) - do przechowywania stanu klawisza
* *KPIN - nazwa PINx portu na którym umieszczony jest klawisz, np: PINB
* key_mask - maska klawisza np: (1<<PB3)
* rep_time - czas powtarzania funkcji rep_proc w trybie REPEAT
* rep_wait - czas oczekiwania do przejścia do trybu REPEAT
* push_proc - wskaźnik do własnej funkcji wywoływanej raz po zwolenieniu przycisku
* rep_proc - wskaźnik do własnej funkcji wykonywanej w trybie REPEAT
**************************************************************************************/

void SuperDebounce(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(void), void (*rep_proc)(void) ) {

enum {idle, debounce, go_rep, wait_rep, rep};

if(!rep_time) rep_time=20;
if(!rep_wait) rep_wait=150;

uint8_t key_press = !(*KPIN & key_mask);

if( key_press && !*key_state ) {
*key_state = debounce;
Timer1 = 15;
} else
if( *key_state ) {

if( key_press && debounce==*key_state && !Timer1 ) {
*key_state = 2;
Timer1=5;
} else
if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc();/* KEY_UP */
*key_state=idle;
} else
if( key_press && go_rep==*key_state && !Timer1 ) {
*key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==*key_state && !Timer1 ) {
*key_state = rep;
} else
if( key_press && rep==*key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc();/* KEY_REP */
}
}
if( *key_state>=3 && !key_press ) *key_state = idle;
}

void SuperDebounceP(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(int x1,int max,int min), void (*rep_proc)(int x1,int max,int min) ) {

enum {idle, debounce, go_rep, wait_rep, rep};


if(!rep_time) rep_time=20;
if(!rep_wait) rep_wait=150;

uint8_t key_press = !(*KPIN & key_mask);

if( key_press && !*key_state ) {
*key_state = debounce;
Timer1 = 15;
} else
if( *key_state ) {

if( key_press && debounce==*key_state && !Timer1 ) {
*key_state = 2;
Timer1=5;
} else
if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc(x1,max,min);/* KEY_UP */
*key_state=idle;
} else
if( key_press && go_rep==*key_state && !Timer1 ) {
*key_state = wait_rep;
Timer1=rep_wait;
} else
if( key_press && wait_rep==*key_state && !Timer1 ) {
*key_state = rep;
} else
if( key_press && rep==*key_state && !Timer1 ) {
Timer1 = rep_time;
if(rep_proc) rep_proc(x1, max, min);/* KEY_REP */
}
}
if( *key_state>=3 && !key_press ) *key_state = idle;
}


//inicjalizacja Timear2 wykonana w postaci funkcji
void timer2_init (void){
/* Timer2 – inicjalizacja przerwania co 1ms */
TCCR2 |= (1<<WGM21);// tryb pracy CTC
TCCR2 |= (1<<CS22)|(1<<CS20)|(1<<CS21);// preskaler = 1024
OCR2 = 155;// przerwanie porównania co 10ms (100Hz)
TIMSK |= (1<<OCIE2);// Odblokowanie przerwania CompareMatch

}

void bip (void){
BIP_ON;
_delay_ms(15);
BIP_OFF;
}[/syntax]

Przesyłam pliki, chodzi o to, aby po wciśnięciu klawisza KL1 wykonała się funkcja SuperDebunceP, do której przekazuję funkcję zwiększenie z 3 parametrami, x1, max i min, funkcja zwiekszenie ma tylko zwiększać x1 w zadanym przedziele pomiędzy min a max.

W tej chwili SuperDebunce wywołuje funkcję zwiększenie ponieważ czyści się ekran i słychać bip() ( buzer), ale nie zwiększa mi wartości x1, przynajmniej na wyświetlaczu tego nie widzę.

Statystyki: Napisane przez Krzycho69 — 24 paź 2018, o 08:08

]]> 2018-10-24T07:56:58+01:00 2018-10-24T07:56:58+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212927#p212927 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> W tej funkcji:[syntax=c]void zwiekszenie_menu2(int x, int max, int min) {
lcd_cls();
x++;
bip();
if (x > max){
x=min;
}
}[/syntax]zmienne x, min i max są znane. Natomiast nic mi nie wiadomo o tym, czy zmienne te są lokalne czy globalne.
Wstawisz cały kod? Będzie można coś więcej powiedzieć.

Statystyki: Napisane przez Marhef — 24 paź 2018, o 07:56

]]> 2018-10-24T06:55:09+01:00 2018-10-24T06:55:09+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212923#p212923 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> lcd_cls();
x++;
bip();
if (x > max){
x=min;
}
}[/syntax]

Ok, mam funkcję jak powyżej jak ją odpalić za pomocą SuperDebunce?

Statystyki: Napisane przez Krzycho69 — 24 paź 2018, o 06:55

]]> 2018-10-23T21:34:33+01:00 2018-10-23T21:34:33+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212917#p212917 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> Statystyki: Napisane przez Marhef — 23 paź 2018, o 21:34

]]> 2018-10-23T21:05:46+01:00 2018-10-23T21:05:46+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212915#p212915 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> jez2000 napisał(a):

W definicji masz ze przekazujesz 3 argumenty a przy wywolaniu funkcji nic nie przekazujesz zadnego. Tak jakbys mial void


No tak, jak wstawię liczby
[syntax=c]if(push_proc) push_proc(1,2,3);[/syntax]

to się kompiluje, ale nie o to chodzi


myślałem, że może tak

[syntax=c]if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc(x,max,min);/* KEY_UP */
*key_state=idle;[/syntax]

ale też źle
mam błąd

../funkcje.c:145:28: error: 'x' undeclared (first use in this function)
if(push_proc) push_proc(x,max,min); /* KEY_UP */

Statystyki: Napisane przez Krzycho69 — 23 paź 2018, o 21:05

]]> 2018-10-23T20:47:44+01:00 2018-10-23T20:47:44+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212911#p212911 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> Statystyki: Napisane przez jez2000 — 23 paź 2018, o 20:47

]]> 2018-10-23T20:32:34+01:00 2018-10-23T20:32:34+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212909#p212909 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> [syntax=c]void SuperDebounceP(uint8_t * key_state, volatile uint8_t *KPIN,
uint8_t key_mask, uint16_t rep_time, uint16_t rep_wait,
void (*push_proc)(int x,int max,int min), void (*rep_proc)(int x,int max,int min) )[/syntax]


ale teraz w treści funkcji:

[syntax=c]if( !key_press && *key_state>1 && *key_state<4 ) {
if(push_proc) push_proc();/* KEY_UP */
*key_state=idle;[/syntax]

mam błąd

../funkcje.c:157:26: error: too few arguments to function 'push_proc'
make: *** [funkcje.o] Błąd 1

Statystyki: Napisane przez Krzycho69 — 23 paź 2018, o 20:32

]]> 2018-10-23T19:50:43+01:00 2018-10-23T19:50:43+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212906#p212906 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> Statystyki: Napisane przez jez2000 — 23 paź 2018, o 19:50

]]> 2018-10-23T19:45:43+01:00 2018-10-23T19:45:43+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212905#p212905 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]> Statystyki: Napisane przez Krzycho69 — 23 paź 2018, o 19:45

]]> 2018-10-23T19:11:04+01:00 2018-10-23T19:11:04+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212901#p212901 <![CDATA[Re: SuperDebunce - funkcja z parametrami]]>
[syntax=c]SuperDebounce(&k2, &PIND, KL2, 20, 50, zwiekszenie, zwiekszenie)[/syntax]

Tyle że wcześniej musisz ją zadeklarować z przynajmniej typami argumentów

Np
[syntax=c]1.SuperDebounce(int PIN, int PIND, int KL, int V_1, int V_2, (void)(*zwiekszenie)(int x,int max,int min), (void)(*zwiekszenie)(int x,int max,int min));[/syntax]

Taki przykład z tymi zmiennymi

Statystyki: Napisane przez jez2000 — 23 paź 2018, o 19:11

]]> 2018-10-23T18:51:01+01:00 2018-10-23T18:51:01+01:00 https://forum.atnel.pl/viewtopic.php?t=21469&p=212898#p212898 <![CDATA[SuperDebunce - funkcja z parametrami]]>
Np :
[syntax=c]SuperDebounce(&k2, &PIND, KL2, 20, 50, zwiekszenie(x,max,min), zwiekszenie(x,max,min));[/syntax]

Statystyki: Napisane przez Krzycho69 — 23 paź 2018, o 18:51

]]>