Pytałem wcześniej o pewne podpowiedzi dotyczące tworzone przeze mnie prostego menu w celu uczenia się języka C.
Koledzy mi dużo podpowiedzieli i coś tam się udało, ale chciałbym się dowiedzieć jak mogłem ten program napisać w sposób bardziej profesjonalny bo ten na pewno jest banalny 
Największym problemem jest zła obsługa klawisza , klawisz nie zawsze działa poprawnie tzn. kierunek menu się zmienia ale naciśnięcie krótkie powoduje złą reakcje programu.
Jak mogę spróbować to lepiej obsłużyć ?
Pokaże program:
[syntax=c]// ######################################################################################################################################
// ATMEGA 32 F 16000000 MHZ
//
//
// ######################################################################################################################################
#include <avr/io.h>
#include <avr/interrupt.h>
#include <avr/pgmspace.h>
#include <util/delay.h>
#include <avr/eeprom.h>
#include "LCD/lcd44780.h"
//#include "I2C_TWI/i2c_twi.h"
#include "EEKRANY/ekrany.h"
uint8_t klawisz_wcisniety(uint8_t klawisz);
// timery programowe
volatile uint16_t Timer1,Timer ;
uint8_t sw1=0,flagg;
uint8_t stan_klawisz1,stan_klawisz2,t,poz,ypoz;
char tablica[20];
void miganie (void);
void FunkcjaDoModyfikacjiStringa (char * twojnowystring);
void FunkcjaDoWyswietlaniaStringa (char * tu_tez_twojstring, uint8_t pozycja_x,uint8_t pozycja_y);
#define KEY1_PIN (1<<PD3) // piny wejsciowe do obslugi menu
#define KEY2_PIN (1<<PD4) // piny wejsciowe do obslugi menu
uint8_t key_lock,x,kier_menu; // kkier,menu_kier,stan_menu,sw;
uint16_t kier=0; // zmienna potrzebna do zmiany pokazywanych komórek
uint16_t y=0;
// --------------------- główna funkcja programu main() -----------------------
int main(void) {
DDRA |= (1<<PA7);
PORTA |=(1<<PA7);
DDRD &= ~(KEY1_PIN); // JAKO WEJSCIE
DDRD &= ~(KEY2_PIN); // JAKO WEJSCIE
PORTD |= KEY1_PIN; // DODCIAGNIECIE DO VCC
PORTD |= KEY2_PIN; // PODCIAGNIECIE DO VCC
/* timer2 - konfiguracja silnika timerów programowych */
TCCR2 |= (1<<WGM21); // tryb pracy CTC
TCCR2 |= (1<<CS22)|(1<<CS21)|(1<<CS20); // preskaler = 1024
OCR2 = 78; // przerwanie porównania co 10ms (100Hz)
TIMSK = (1<<OCIE2); // Odblokowanie przerwania ComareMatch
lcd_init(); // inicjalizacja lcd
lcd_cls(); // kasowanie ekranu
sei(); // globalne odblokowanie przerwania
// *************** pętla główna programu
while(1) {
if (kier>8191) kier=0; // wartośc 8191 okresla rozmiar pamieci Eprom i ilosc nacisniec przycisku jest powyzej zerowana
if (kier_menu>3) kier_menu=0;
if( klawisz_wcisniety ( KEY1_PIN ) ){
kier++;
kier_menu++;
}
if( klawisz_wcisniety ( KEY2_PIN ) ){
kier--;
kier_menu--;
}
if( kier_menu==0 ) {
if( klawisz_wcisniety ( KEY1_PIN ) ){
FunkcjaDoModyfikacjiStringa("Kopiow");
poz=0;
ypoz=0;
FunkcjaDoWyswietlaniaStringa(tablica,poz,ypoz);
}
}
if( kier_menu==1 ) {
if( klawisz_wcisniety ( KEY1_PIN ) ){
FunkcjaDoModyfikacjiStringa("Odczyt");
poz=1;
ypoz=0;
FunkcjaDoWyswietlaniaStringa(tablica,poz,ypoz);
}
}
if( kier_menu==2 ) {
if( klawisz_wcisniety ( KEY1_PIN ) ){
FunkcjaDoModyfikacjiStringa("Testow");
poz=0;
ypoz=8;
FunkcjaDoWyswietlaniaStringa(tablica,poz,ypoz);
}
}
if( kier_menu==3 ) {
if( klawisz_wcisniety ( KEY1_PIN ) ){
FunkcjaDoModyfikacjiStringa("Parametr");
poz=1;
ypoz=8;
FunkcjaDoWyswietlaniaStringa(tablica,poz,ypoz);
}
}
miganie();
lcd_locate(3,12);
lcd_int(kier_menu);
}
}
uint8_t klawisz_wcisniety(uint8_t klawisz)
{
if(!key_lock && ! ( PIND & klawisz) )
{
key_lock=1;
_delay_ms(80); // To jest niefahowe ale narazie tak wstawiłem
if( ! ( PIND & klawisz) ) return 1;
}else if ( key_lock && (PIND & klawisz)) key_lock++;
return 0;
}
//*** przerwanie Timer2 CompareM
ISR(TIMER2_COMP_vect) {
uint16_t n;
n = Timer1; /* 100Hz Timer */
if (n) Timer1 = --n;
n = Timer; /* 100Hz Timer */
if (n) Timer = --n;
}
void miganie (void){
if (!Timer) {
lcd_locate(0,0);
lcd_str ("Kopiow");
lcd_locate(1,0);
lcd_str ("Odczyt");
lcd_locate(0,8);
lcd_str ("Testow");
lcd_locate(1,8);
lcd_str ("Parametr");
if( !sw1 ){
}
else { lcd_locate(poz,ypoz);
lcd_str (" ");
}
sw1^=1;
Timer =40;
}
}
void FunkcjaDoModyfikacjiStringa (char * twojnowystring) { // Tablica zawsze przekazywana jako wskaznik
strcpy(tablica, twojnowystring); // Dla uproszczenia twojnowystring[] <= tablica[]
}
void FunkcjaDoWyswietlaniaStringa (char * tu_tez_twojstring, uint8_t pozycja_x,uint8_t pozycja_y) {
lcd_cls();
lcd_locate(pozycja_x,pozycja_y);
lcd_str(tu_tez_twojstring);
lcd_str (" ");
}[/syntax]
Próbowałem wysłać wideo ale coś nie poszło.
Dodatkowo w czasie przełączania wyświetlanie nie jest stabilne .Statystyki: Napisane przez AKSELINEK — 27 lut 2021, o 20:12
]]>