ATNEL tech-forum

Re: proste menu

2012-02-14T23:10:11+01:00

DidekxD napisał(a):

W książce zobacz taki rozdział jak "8.4 Wstęp do systemów czasu rzeczywistego"
.

Bardzo dobra i cenna podpowiedź tym bardziej, że kolega DidekxD już stosuje te mechanizmy w swoich kodach z powodzeniem, a co więcej widać to w kodach w jego postach więc można także podpatrzeć jak inni to robią.

Statystyki: Napisane przez mirekk36 — 14 lut 2012, o 23:10

Re: proste menu

2012-02-14T23:02:59+01:00

W książce zobacz taki rozdział jak "8.4 Wstęp do systemów czasu rzeczywistego"
I zwróć uwagę na timery programowe.

Będziesz mógł przy ich wykorzystaniu cały czas przez te cztery sekundy sprawdzać czy nie został wciśnięty któryś z guzików po czym odpali się twoja funkcja.
Zapomnij o liniowym pisaniu i o funkcji _delay warto korzystać z tych timerów. Dużo problemów odchodzi w niepamięć(albo dylematów).

Statystyki: Napisane przez DidekxD — 14 lut 2012, o 23:02

Re: proste menu

2012-02-14T22:51:51+01:00

Funkcja temp() po 4 sekundach się uruchamia ale nie mam pomysłu w jaki sposób stworzyć coś, może funkcję która rozpozna że nic się w danej chwili nie dzieje np. nie został wciśnięty żaden klawisz przez 4 sekundy i dopiero powinno nastąpić wywołanie funkcji temp(). Nie proszę o gotowy kod tylko o jakieś wskazówki w jaki sposób mogę ten problem rozwiązać

Statystyki: Napisane przez kris — 14 lut 2012, o 22:51

Re: proste menu

2012-02-14T19:41:32+01:00

No teraz lepiej

Ale po pierwsze w takim przypadku pomijając na razie to że nie wykonuje się funkcja temp, to praktycznie ciężko będzie ci w ogóle wywołać któryś klawisz chyba że przy starcie programu będziesz jakiś trzymał wciąż wciśnięty.

A druga sprawa - skoro nie wywołuje ci się funkcja temp to znaczy że masz ją źle napisaną bo to niemożliwe żeby się nie wywołała.

Po trzecie to nigdy nie warto tak pisać programu a szczególnie MENU, że pisze się po kawałku doklecając co chwilę jakąś nową dziwną możliwość. Najpierw to trzeba sobie na kartce papieru rozrysować całe MENU - jak ma działać, co po kolei i skąd wywoływane itd..... czyli prościej mówiąc przygotować sobie algorytm - a potem łatwiej to oprogramować. Tymczasem tak? to wychodzą niestety potworki

ale jak mówię - to nie ma nic wspólnego z tym że ci się rzekomo temp() nie wykonuje bo to jakieś nieporozumienie to co napisałeś.

Statystyki: Napisane przez mirekk36 — 14 lut 2012, o 19:41

Re: proste menu

2012-02-14T18:53:47+01:00

[syntax=c]int back(void){

lcd_cls();
lcd_locate(0,10);
lcd_str("Menu");
lcd_locate(1,0);
lcd_str("<");
lcd_locate(1,23);
lcd_str(">");
lcd_locate(1,8);
lcd_str("1");

while(1) {

uint8_t k1,k2,k3;

SuperDebounce(&k1, &PINB, KL1, 0, 0, menu_down, 0 );
SuperDebounce(&k2, &PINB, KL2, 0, 0, menu_up, 0 );
SuperDebounce(&k3, &PINB, KL3, 0, 0, set, 0 );
_delay_ms(4000);
temp();
}
return 0;
}[/syntax]Jeśli nie wcisne żadnego przycisku to powinno odczekać i wywołać funkcję temp(), a w moim przypadku po 4 sekundach przechodzi do funkcji temp().

Statystyki: Napisane przez kris — 14 lut 2012, o 18:53

Re: proste menu

2012-02-14T18:46:15+01:00

Oj przepraszam ale na takie pytanie w ogóle nie odpowiem. To jest wręcz prośba o napisanie programu. Panie kolego, proszę zaprezentować jakiś konkretny problem, spróbować pokazać swoje rozwiązanie i opisać co w nim konkretnie sprawia kłopot....

a nie zadawać pytania typu - życzenia: "chciałbym żeby w programie było to i to - jak to zrobić?"

sorki ale to dla mnie nie jest pytanie - OK ? (bez urazy proszę - ale pytania też trzeba umieć zadawać)

Statystyki: Napisane przez mirekk36 — 14 lut 2012, o 18:46

Re: proste menu

2012-02-14T18:05:01+01:00

Mam problem z tym kawałkiem kodu [syntax=c]int back(void){

lcd_cls();
lcd_locate(0,10);
lcd_str("Menu");
lcd_locate(1,0);
lcd_str("<");
lcd_locate(1,23);
lcd_str(">");
lcd_locate(1,8);
lcd_str("1");

while(1) {

uint8_t k1,k2,k3;

SuperDebounce(&k1, &PINB, KL1, 0, 0, menu_down, 0 );
SuperDebounce(&k2, &PINB, KL2, 0, 0, menu_up, 0 );
SuperDebounce(&k3, &PINB, KL3, 0, 0, set, 0 );
}
return 0;
}[/syntax]

Chciałbym zastosować w nim przejście do funkcji np. temp() jeśli nie zostanie wciśnięty żaden przycisk przez 4 s. W jaki sposób mogę to zrobić?

Statystyki: Napisane przez kris — 14 lut 2012, o 18:05

Re: proste menu

2012-02-13T11:38:56+01:00

Menu działa na pojedynczym przycisku, dzięki za pomoc

Statystyki: Napisane przez kris — 13 lut 2012, o 11:38

Re: proste menu

2012-02-05T15:26:09+01:00

A po co ci ta cała funkcja zmiana? po pierwsze.

A po drugie przeanalizuj sobie jak ona działa. Definiujesz w niej zmienną i tworzoną automatycznie przy wejściu i jest ustawiana zawsze na 2 ... to w ogóle niezrozumiały kod tej funkcji

utwórz sobie zmienną lokalną ale statyczną zmiana, ale w funkcji menu, i zwiększaj ją po każdym wejściu o 1, sprawdzając jednocześnie czy jest większa niż 2 i wtedy znowu 0.

[syntax=c]void menu(void){

static uint8_t zmiana;

// lcd_init(); -----------> panie kolego WYKASUJ to stąd ! czy do ciebie nie dociera że to się robi tylko RAZ przy inicjalizacji całego programu ????

if(zmiana()==0){
lcd_locate(0,0);
lcd_str("1");
}if(zmiana()==1){
lcd_locate(0,0);
lcd_str("2");
}if(zmiana()==2){
lcd_locate(0,0);
lcd_str("3");
}

if( ++zmiana > 2 ) zmiana = 0;

}[/syntax]

I na prawdę - proszę mnie zrozumieć i się nie gniewać - ale jeśli dalej w kolejnych kodach kolega będzie wciąż robił inicjalizację LCD w tak dziwnych miejscach programu - to ja się poddaję i przestanę podpowiadać.

Przecież pisałem o tym koledze już wcześniej w tym temacie nie mówiąc o tym, że książce wyłożyłem temat wyraźnie na jakie sekcje dzielimy główną funkcję main() i że tam na jej początku dokonujemy inicjalizacji wszystkich modułów. A kolega z uporem godnym podziwu wciąż wkleja w dowolne miejsca kodu tę inicjalizację LCD

Zajrzyj do swojego kodu powyżej - pokazałem ci tam wyraźnie jeszcze raz gdzie ma być sekcja inicjalizacji - może teraz kolega zapamięta na zawsze ?

Statystyki: Napisane przez mirekk36 — 5 lut 2012, o 15:26

Re: proste menu

2012-02-05T13:10:20+01:00

Stworzyłem coś takiego tylko nie działa, co może być nie tak?
[syntax=c]
#define KEY_DOWN !(PINB & KL1)
int zmiana(void){
uint8_t i;

if(KEY_DOWN)i=0;
if(KEY_DOWN)i=1;
if(KEY_DOWN)i=2;
return i;
}

void menu(void){
lcd_init();
if(zmiana()==0){
lcd_locate(0,0);
lcd_str("1");
}if(zmiana()==1){
lcd_locate(0,0);
lcd_str("2");
}if(zmiana()==2){
lcd_locate(0,0);
lcd_str("3");
}
}
int main(void) {

// **************** S E K C J A I N I C J A L I Z A C J I *************************
PORTB |= KL1;

/* Timer2 – inicjalizacja przerwania co 10ms */
TCCR2 = (1<TCCR2 = (1<OCR2 = 108;// przerwanie porównania co 10ms (100Hz)
TIMSK = (1<
lcd_init(); // -------> TU MA BYĆ INICJALIZACJA LCD - TYLKO RAZ

// **************** koniec S E K C J I I N I C J A L I Z A C J I *************************

sei();

while(1) {

uint8_t k1;

SuperDebounce(&k1, &PINB, KL1, 0, 0, menu, 0 );

}
}[/syntax]

Statystyki: Napisane przez kris — 5 lut 2012, o 13:10

Re: proste menu

2012-02-03T16:13:40+01:00

eeeh nie - skucha całkiem źle - ty wywołujesz je z tym samym klawiszem KEY1_PIN ... to trzeba całkiem inaczej...

wprowadzasz sobie jakąś zmienną, która będzie ci wskazywała poziom menu

i wywołujesz tylko raz funkcję SUperDebounce, a funkcja którą wywołujesz będzie jedynie zwiększała wartość tej zmiennej sprawdzając czy nie przekroczyła wartości 2 a jeśli tak to znowu 0

i w tej funkcji dopiero w zależności od tej zmiennej wywołujesz odpowiednią funkcję z pozycją twojego MENU

Statystyki: Napisane przez mirekk36 — 3 lut 2012, o 16:13

Re: proste menu

2012-02-03T16:11:28+01:00

Już tłumaczę, w pierwszym przypadku użyłeś jednej zmiennej k1 dla każdej z funkcji, tymczasem one muszą mieć całkiem oddzielne zmienne.

zdefiniuj sobie więc trzy:

uint8_t k1, k2, k3;

i użyj każdą do oddzielnego wywołania SuperDebounce

bez żadnego robienia dodatkowej pętli for() .... czyli:

[syntax=c] while(1){
uint8_t k1;
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testa, 0);
SuperDebounce(&k2, &PINB, KEY1_PIN, 0, 0, testb, 0);
SuperDebounce(&k3, &PINB, KEY1_PIN, 0, 0, testc, 0);
}[/syntax]

Statystyki: Napisane przez mirekk36 — 3 lut 2012, o 16:11

Re: proste menu

2012-02-03T15:37:12+01:00

A jeśli tak by wyglądał kod? [syntax=c]while(1){
uint8_t k1,i;
for(i=0;i<3;i++){
if(i==0){
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testa, 0);
}
if(i==1){
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testb, 0);
}
if(i==2){
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testc, 0);
}
}
}[/syntax]

Statystyki: Napisane przez kris — 3 lut 2012, o 15:37

Re: proste menu

2012-02-03T15:29:55+01:00

wiesz do tej pory uważałem ze do takiego celu jest wygodniejszy encoder , i zwykle to rozwiązanie stosuję
a co do powyższego , wcale mnie to nie dziwi ze wybiera przypadkowo przecież ja tu widzę 3 elementy na jednym przycisku
uruchamiane w pętli jedno po drugim wiec na co trafisz to masz

Statystyki: Napisane przez SunRiver — 3 lut 2012, o 15:29

Re: proste menu

2012-02-03T15:06:14+01:00

Może jest jakiś błąd w pętli którą zdefiniowałem? Bo po wciśnięciu przycisku wywołana zostaje dowolna funkcja, a interesuje mnie kolejno przeskakiwanie z jednej na drugą jednym przyciskiem. [syntax=c]while(1){
uint8_t k1;
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testa, 0);
SuperDebounce(&k2, &PINB, KEY1_PIN, 0, 0, testb, 0);
SuperDebounce(&k3, &PINB, KEY1_PIN, 0, 0, testc, 0);
}[/syntax]

Statystyki: Napisane przez kris — 3 lut 2012, o 15:06

Re: proste menu

2012-02-03T12:27:29+01:00

Pierwsza uwaga, proszę nie zamieszczać na forach kodów źródłowych z bibliotek z książki. Swój własny można - ale prośba - nie kody z książki OK ? ..... teraz nie mam czasu analizować - kodu ale później wrócę i sprawdzę - może coś mi się nasunie to dam znać.

Statystyki: Napisane przez mirekk36 — 3 lut 2012, o 12:27

Re: proste menu

2012-02-03T11:41:33+01:00

Stworzyłem coś takiego tylko nie wiem czemu nie działa prawidłowo. Chciałbym aby jednym przyciskiem po kolei poruszać się miedzy test1, test2, test3, proszę o wskazówki
[syntax=c]/*
* main.c
*
* Created on: 2010-03-31
* Autor: Krystian Cieśla
*/
#include
#include
#include
#include
#include "LCD/lcd44780.h"

#define KEY1_PIN (1<
volatile uint16_t Timer1, Timer2;

/************** funkcja SuperDebounce do obsługi pojedynczych klawiszy ***************/
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 testa(void) {
lcd_init();
lcd_str("Test1");
}

void testb(void) {
lcd_init();
lcd_str("Test2");
}

void testc(void) {
lcd_init();
lcd_str("Test3");
}

int main(void){

lcd_init(); //inicjalizacja wyświetlacza
DDRB &= ~KEY1_PIN;
PORTB |= KEY1_PIN;

/* Timer2 – inicjalizacja przerwania co 10ms */
TCCR2 = (1< TCCR2 = (1< OCR2 = 108;// przerwanie porównania co 10ms (100Hz)
TIMSK = (1<
sei();

while(1){
uint8_t k1;
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testa, 0);
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testb, 0);
SuperDebounce(&k1, &PINB, KEY1_PIN, 0, 0, testc, 0);
}
}

/************** 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)
* *KPIN - nazwa PINx portu na którym umieszczony jest klawisz, np: PINB
* key_mask - maska klawisza np: (1< * 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) ) {

// ..... reszta kodu funkcji w książce
}

ISR(TIMER2_COMP_vect)
{
uint16_t n;

n = Timer1;/* 100Hz Timer1 */
if (n) Timer1 = --n;
n = Timer2;/* 100Hz Timer2 */
if (n) Timer2 = --n;
}
[/syntax]

Statystyki: Napisane przez kris — 3 lut 2012, o 11:41

Re: proste menu

2012-02-02T15:09:29+01:00

dzięki za wskazówki

Statystyki: Napisane przez kris — 2 lut 2012, o 15:09

Re: proste menu

2012-02-02T15:06:55+01:00

kris napisał(a):

Co zrobić aby ta operacja była płynna?

Przeczytać do końca książkę, zrozumieć zasadę działania z przyciskami z rozdziału, który temu zagadnieniu poświęciłem w szczegółach, zobaczyć i zrozumieć opisaną tam funkcję SuperDebounce a także przeczytać rozdział:

"Wstęp do systemów czasu rzeczywistego"

A wszystko stanie się jasne jak drut i przejrzyste jak słońce. Jednocześnie opisane problemy skończą się jak toporem uciął

Statystyki: Napisane przez mirekk36 — 2 lut 2012, o 15:06

proste menu

2012-02-02T14:49:42+01:00

Stworzyłem proste menu które pozwala jednym przyciskiem przeskakiwać pomiędzy trzema wyrażeniami, a mój problem polega na tym że gdy wciskam przycisk to czasami wskakuje na wyrażenie 2 po dłuższym wciśnięciu na wyrażenie 3. Co zrobić aby ta operacja była płynna?
[syntax=c]/*
* main.c
*
* Created on: 2010-03-27
* Autor: Krystian Cieśla
*/
#include
#include
#include "LCD/lcd44780.h"

// klawisz K1
#define KEY_PIN (1<#define KEY_DOWN !(PINB & KEY_PIN)

uint8_t klawisz_wcisniety(void);

int main(void)
{
// ****** inicjalizacja *********
lcd_init();
DDRB &= ~KEY_PIN;// kierunek pinu PB1 - wejściowy
PORTB |= KEY_PIN;// podciągnięcie pinu do VCC

// ****** pętla główna programu *********
while(1)
{
if( klawisz_wcisniety() )
{
lcd_cls();
lcd_locate(0,0);
lcd_str("Test1");
_delay_ms(200);
}if (klawisz_wcisniety()){
lcd_cls();
lcd_locate(0,0);
lcd_str("Test2");
_delay_ms(200);
}if (klawisz_wcisniety()){
lcd_cls();
lcd_locate(0,0);
lcd_str("Test3");
_delay_ms(200);
}
}
}
//******************************************************** koniec main()

// definicja funkcji
uint8_t klawisz_wcisniety(void)
{
if( KEY_DOWN )
{
_delay_ms(80);
if( KEY_DOWN ) return 1;
}

return 0;
}

[/syntax]

Statystyki: Napisane przez kris — 2 lut 2012, o 14:49