ATNEL tech-forum

Re: Klawiatura analogowa

2015-10-05T13:52:54+01:00

Ciężko trochę interpretuje się Twój kod, ze względu na lekkie niechlujstwo, ale postaram się pomóc.

Po krótkiej analizie wydaje mi się, że problem leży w zmiennej a i warunku jej dotyczącym.

Po każdej wpisanej cyfrze pinu zwiększasz ją o jeden, czyli generalnie po wpisaniu czwartej cyfry zostaje wykonany warunek
[syntax=c]//Resetowanie ekranu
if (a>=4) {
_delay_ms(2000);
lcd_cls();
a=0;
i=0;
enter=0;
}[/syntax]

Twój program zostaje "ubity" na 2 sek, zmienna a się resetuje, zmienna enter również. Zatem warunek enter==1 jest niespoełniony.
Dopiero po przejściu delaya czyli te 2 sek, zmienna enter przy wcisnietym klawiszu zostaje ustawiona na 1 i wykonuje sie warunek
[syntax=c]//////////////Porównanie haseł
if ((tab1[0]==tab2[0])&&(tab1[1]==tab2[1])&&(tab1[2]==tab2[2])&&(tab1[3]==tab2[3])&&(enter==1)){
lcd_cls();
lcd_str("HASLO PRAWIDLOWE");
//_delay_ms(1000);
}[/syntax]

Gotowego rozwiązania nie będę Ci podrzucał, ale myślę że wiesz co poprawić

Pozdrawiam!

Statystyki: Napisane przez marmur — 5 paź 2015, o 13:52

Re: Klawiatura analogowa

2015-10-05T11:14:20+01:00

To jak? Pomoże ktoś?

Statystyki: Napisane przez Swoszaki — 5 paź 2015, o 11:14

Re: Klawiatura analogowa

2015-10-02T18:43:23+01:00

Witam po trochę dłuższej przerwie spowodowanej różnymi innymi zajęciami.

Temat dalej dotyczy klawiaturki. Wykonałem sobie już całą 12 przyciskowa klawiaturkę (0...9 Reset,Enter). Wszystkie cyferki pięknie wyświetlają się na lcd, przycisk kasowania ekranu też działa.

Problem jest taki....

Chciałbym, aby po wpisaniu prawidłowego hasła (1,2,3,4) oraz naciśnięciu przycisku Enter wyskakiwało "Hasło prawidłowe"....i to działa, ale tylko w przypadku jak po wpisaniu tych cyfr trzymam klawisz enter wciśnięty przez 1-2 sekundy. W innym przypadku następuje "czyszczenie ekranu".

Domyślam się, że problem jest gdzieś w tych linijkach....
[syntax=c]//Resetowanie ekranu
if (a>=4) {
_delay_ms(2000);
lcd_cls();
a=0;
i=0;
enter=0;
}[/syntax]
oraz
[syntax=c]//////////////Porównanie haseł
if ((tab1[0]==tab2[0])&&(tab1[1]==tab2[1])&&(tab1[2]==tab2[2])&&(tab1[3]==tab2[3])&&(enter==1)){
lcd_cls();
lcd_str("HASLO PRAWIDLOWE");
//_delay_ms(1000);
}[/syntax]
Czy ktoś mógłby doradzić mi co tam nie chwyta...?

Wrzucam cały kod programu
[syntax=c]#include
#include
#include "LCD/lcd44780.h"
#include

volatile uint16_t Timer1, Timer2;

uint8_t sw_on ;

//zmienna do czyszczenia ekranu lcd po wpisaniu 4 cyfr
uint8_t a=0, i=0, enter=0;
uint16_t tab2[4];
uint16_t tab1[]={1,2,3,4};

//stałe dla poszczególnych klawiszy
#define ADC_K1 340
#define ADC_K2 570
#define ADC_K3 745
#define ADC_K4 870
//stała różnica
#define ADC_RANGE 50

//definicje LED
#define LED1 (1<#define LED2 (1<
#define LED1_ON PORTD &= ~LED1
#define LED1_OFF PORTD |= LED1
#define LED1_TOG PORTD ^= LED1

#define LED2_ON PORTD &= ~LED2
#define LED2_OFF PORTD |= LED2
#define LED2_TOG PORTD ^= LED2

uint16_t pomiar(uint8_t kanal);
uint32_t pm,pm1,pm2;
uint8_t keys[4],keys1[4], keys2[4];

int main(void)
{
DDRB |= (1< PORTB |= (1<
//PortD jako wyjście
DDRD |= LED1|LED2;

//Wygaszenie LED
LED1_OFF; LED2_OFF;

// timer0 - konfig
TCCR0 |= (1< TIMSK |= (1<
//inicjalizacja LCD
lcd_init();

//napis początkowy
lcd_str("WITAJ");
_delay_ms(3000);
lcd_cls();
lcd_str("PODAJ HASLO");
_delay_ms(1000);
//lcd_cls();
lcd_locate(1,0);

//wybor napiecia odniesienia - VCC
//ADMUX |= (1< //wewnętrzne źródło napięcia 2,56V
ADMUX |=(1< ADCSRA |= (1<
//zezwolenie na przerwania
sei();

//pętla główna programu
while(1)
{
pm = pomiar(7);//pomiar z portu PA7 (ADC7)
pm1 = pomiar(6);//pomiar z portu PA6 (ADC6)
pm2 = pomiar(5);//pomiar z portu PA5 (ADC5)

///////Pierwsza kolumna. Przyciski (1,5,7,Reset)

if( pm < ADC_K1+ADC_RANGE )
{
if( keys[0]<10 ) keys[0]++;
} else keys[0]=0;
if( pm > ADC_K2-ADC_RANGE && pm < ADC_K2+ADC_RANGE )
{
if( keys[1]<10 ) keys[1]++;
} else keys[1]=0;
if( pm > ADC_K3-ADC_RANGE && pm < ADC_K3+ADC_RANGE )
{
if( keys[2]<10 ) keys[2]++;
} else keys[2]=0;
if( pm > ADC_K4-ADC_RANGE && pm < ADC_K4+ADC_RANGE )
{
if( keys[3]<10 ) keys[3]++;
} else keys[3]=0;

///////Druga kolumna. Przyciski (2,4,8,0)

if( pm1 < ADC_K1+ADC_RANGE )
{
if( keys1[0]<10 ) keys1[0]++;
} else keys1[0]=0;
if( pm1 > ADC_K2-ADC_RANGE && pm1 < ADC_K2+ADC_RANGE )
{
if( keys1[1]<10 ) keys1[1]++;
} else keys1[1]=0;
if( pm1 > ADC_K3-ADC_RANGE && pm1 < ADC_K3+ADC_RANGE )
{
if( keys1[2]<10 ) keys1[2]++;
} else keys1[2]=0;
if( pm1 > ADC_K4-ADC_RANGE && pm1 < ADC_K4+ADC_RANGE )
{
if( keys1[3]<10 ) keys1[3]++;
} else keys1[3]=0;

//////////Trzecia kolumna (3,6,9,Enter)

if( pm2 < ADC_K1+ADC_RANGE )
{
if( keys2[0]<10 ) keys2[0]++;
} else keys2[0]=0;
if( pm2 > ADC_K2-ADC_RANGE && pm2 < ADC_K2+ADC_RANGE )
{
if( keys2[1]<10 ) keys2[1]++;
} else keys2[1]=0;
if( pm2 > ADC_K3-ADC_RANGE && pm2 < ADC_K3+ADC_RANGE )
{
if( keys2[2]<10 ) keys2[2]++;
} else keys2[2]=0;
if( pm2 > ADC_K4-ADC_RANGE && pm2 < ADC_K4+ADC_RANGE )
{
if( keys2[3]<10 ) keys2[3]++;
} else keys2[3]=0;

//////////////////////////////

//klawisz 1
if( keys[0]==9 ){
lcd_int(1);
a=a+1;
tab2[i]=1;
i++;
}
//klawisz 4
if( keys[1]==9 ){
lcd_int(4);
a=a+1;
tab2[i]=4;
i++;
}
//klawisz 7
if( keys[2]==9 ){
lcd_int(7);
a=a+1;
tab2[i]=7;
i++;
}
//klawisz Reset
if( keys[3]==9 ){
lcd_cls();
a=0;
i=0;
enter=0;
}

//////////////////////////////////////

//klawisz 2
if( keys1[0]==9 ){
lcd_int(2);
a=a+1;
tab2[i]=2;
i++;
}
//klawisz 5
if( keys1[1]==9 ){
lcd_int(5);
a=a+1;
tab2[i]=5;
i++;
}
//klawisz 8
if( keys1[2]==9 ){
lcd_int(8);
a=a+1;
tab2[i]=8;
i++;
}
//klawisz Reset
if( keys1[3]==9 ){
lcd_int(0);
a=a+1;
tab2[i]=0;
i++;
}

///////////////////////////

//klawisz 3
if( keys2[0]==9 ){
lcd_int(3);
a=a+1;
tab2[i]=3;
i++;
}
//klawisz 6
if( keys2[1]==9 ){
lcd_int(6);
a=a+1;
tab2[i]=6;
i++;
}
//klawisz 9
if( keys2[2]==9 ){
lcd_int(9);
a=a+1;
tab2[i]=9;
i++;
}
//klawisz Enter
if( keys2[3]==9 ){
enter=1;
a=a+1;
//i++;
}

//Resetowanie ekranu
if (a>=4) {
_delay_ms(2000);
lcd_cls();
a=0;
i=0;
enter=0;
}

//////////////Porównanie haseł
if ((tab1[0]==tab2[0])&&(tab1[1]==tab2[1])&&(tab1[2]==tab2[2])&&(tab1[3]==tab2[3])&&(enter==1)){
lcd_cls();
lcd_str("HASLO PRAWIDLOWE");
//_delay_ms(1000);
}
}
}

//funkcja pomiaru
uint16_t pomiar(uint8_t kanal)
{
ADMUX = (ADMUX & 0xF8) | kanal;
ADCSRA |= (1< while (ADCSRA & (1< return ADCW;
}

//Procedrua przerwania
ISR( TIMER0_OVF_vect )
{
TCNT0 = 256-78; // częstotliwość 100 Hz
uint16_t n = Timer1;
if(n) Timer1=--n;
n = Timer2;
if(n) Timer2=--n;
}[/syntax]
Z góry dziękuję.

Statystyki: Napisane przez Swoszaki — 2 paź 2015, o 18:43

Re: Klawiatura analogowa

2015-05-26T21:02:43+01:00

Po pewnej liczbie kombinacji udało się A wystarczyło podejść troszkę od innej strony...
[syntax=c]if( keys[0]==9 ){
LED1_TOG;
lcd_int(1);
a=a+1;
tab2[i]=1;
i++;
}
if( keys[1]==9 ){
LED2_TOG;

lcd_int(2);
a=a+1;
tab2[i]=2;
i++;

}
if( keys[2]==9 ){
LED3_TOG;
lcd_int(3);
a=a+1;
tab2[i]=3;
i++;

}
if( keys[3]==9 ){
LED4_TOG;
lcd_int(4);
a=a+1;
tab2[i]=4;
i++;
}

if (a==4) {
_delay_ms(1000);
lcd_cls();
a=0;
i=0;

}

if ( (tab1[0] ==tab2[0]) && (tab1[1] ==tab2[1]) && (tab1[2] ==tab2[2]) && (tab1[3] ==tab2[3]) ){
LED5_OFF;
lcd_cls();
lcd_str("HASLO PRAWIDLOWE");
_delay_ms(1000);
}else LED5_ON;[/syntax]

Statystyki: Napisane przez Swoszaki — 26 maja 2015, o 21:02

Re: Klawiatura analogowa

2015-05-25T17:40:09+01:00

Zrobiłem tak, ale dalej jest to samo....czy wpisze 1234, czy 4231,czy 1342 to dioda led gaśnie....
[syntax=c]if( keys[0]==9 ){
LED1_TOG;
lcd_int(1);
a=a+1;
cy1=cy1+1;
}
if( keys[1]==9 ){
LED2_TOG;
lcd_int(2);
a=a+1;
cy2=cy2+2;

}
if( keys[2]==9 ){
LED3_TOG;
lcd_int(3);
a=a+1;
cy3=cy3+3;

}
if( keys[3]==9 ){
LED4_TOG;
lcd_int(4);
a=a+1;
cy4=cy4+4;
}

if (a>4) {
lcd_cls();
a=0;
cy1=0;cy2=0;cy3=0;cy4=0;

}
if (suma==((1000*cy1)+(100*cy2)+(10*cy3)+(cy4))) {
LED5_OFF;
}else LED5_ON;[/syntax]

Statystyki: Napisane przez Swoszaki — 25 maja 2015, o 17:40

Re: Klawiatura analogowa

2015-05-25T09:45:09+01:00

Witam

Swoszaki napisał(a):

...Sumy ...problem w tym, że jeżeli hasło było np 1234 , to jakakolwiek kolejność cyfr powodowała "zadziałanie kodu".

Nie wiem jak masz, ale sumę powinieneś porównywać:
cy1*1000 + cy2*100 + cy3*10 + cy4
W takim przypadku jest tylko jedna możliwość zgodności.

Pozdr.

Statystyki: Napisane przez kicajek — 25 maja 2015, o 09:45

Klawiatura analogowa

2015-05-24T19:41:28+01:00

Witam,

Po obejrzeniu serii poradników Pana Mirka z serii ADC stworzyłem sobie prostą klawiaturkę składającą się z 4 klawiszy (docelowo ma być 12, ale chciałem sprawdzić na początku, czy to wszystko zadziała). Wszystko pięknie działa, 4 diody led pięknie się zapalają po wciśnięciu odpowiednich klawiszy, na lcd wyświetlają się pięknie cyferki odpowiadające klawiszom. Działa też funkcja Auto Repeat (którą na chwilę obecną skasowałem).

Dalej sobie pomyślałem, aby stworzyć taki prosty zamek szyfrujący. Z początku próbowałem porównywać hasła i klawisze na zasadzie Sumy lub iloczynu (Wiem, że to banalne). I nawet do pewnego stopnia działało....problem w tym, że jeżeli hasło było np 1234 , to jakakolwiek kolejność cyfr powodowała "zadziałanie kodu".

Kolejna próba to porównanie tablic... niestety tutaj pojawia się problem z pętlą FOR, którą dodałem....Mianowicie każda cyfra pojawia się 4 razy na wyświetlaczu... "Hasło działa jedynie gdy pod "tab1" jest wpisana 4 razy ta sama cyfra.... Domyślam się, że problem jest albo z pętlą, albo z wyświetleniem cyfr na wyświetlaczu....

Dlatego zwracam się z prośba o pomoc, jak to zrobić, żeby działało.....poniżej wrzucam kod programu

[syntax=c]#include
#include
#include "LCD/lcd44780.h"
#include

volatile uint16_t Timer1, Timer2;

uint8_t sw_on ;

//zmienna do czyszczenia ekranu lcd po wpisaniu 4 cyfr
uint8_t a=0;

//tablice do porównania
uint8_t tab1[]={1,2,3,4};
uint8_t tab2[4];

//stałe dla poszczególnych klawiszy
#define ADC_K1 323
#define ADC_K2 550
#define ADC_K3 730
#define ADC_K4 850
//stała różnica
#define ADC_RANGE 70

//definicje LED
#define LED1 (1<#define LED2 (1<#define LED3 (1<#define LED4 (1<#define LED5 (1<
#define LED1_ON PORTB &= ~LED1
#define LED1_OFF PORTB |= LED1
#define LED1_TOG PORTB ^= LED1

#define LED2_ON PORTB &= ~LED2
#define LED2_OFF PORTB |= LED2
#define LED2_TOG PORTB ^= LED2

#define LED3_ON PORTB &= ~LED3
#define LED3_OFF PORTB |= LED3
#define LED3_TOG PORTB ^= LED3

#define LED4_ON PORTB &= ~LED4
#define LED4_OFF PORTB |= LED4
#define LED4_TOG PORTB ^= LED4

#define LED5_ON PORTB &= ~LED5
#define LED5_OFF PORTB |= LED5
#define LED5_TOG PORTB ^= LED5

uint16_t pomiar(uint8_t kanal);
uint32_t pm,i;
uint8_t keys[4];

int main(void)
{
DDRB |= (1< PORTB |= (1<
//Portb jako wyjście
DDRB |= LED1|LED2|LED3|LED4|LED5;

//Wygaszenie LED
LED1_OFF; LED2_OFF; LED3_OFF; LED4_OFF, LED5_ON;

// timer0 - konfig
TCCR0 |= (1< TIMSK |= (1<
//inicjalizacja LCD
lcd_init();

//wybor napiecia odniesienia - VCC
//ADMUX |= (1< //wewnętrzne źródło napięcia 2,56V
ADMUX |=(1< ADCSRA |= (1<
//zezwolenie na przerwania
sei();

//pętla główna programu
while(1)
{
pm = pomiar(7);//pomiar z portu PA7 (ADC7)

//Który przycisk został wciśnięty
if( pm < ADC_K1+ADC_RANGE )
{
if( keys[0]<10 ) keys[0]++;
} else keys[0]=0;
if( pm > ADC_K2-ADC_RANGE && pm < ADC_K2+ADC_RANGE )
{
if( keys[1]<10 ) keys[1]++;
} else keys[1]=0;
if( pm > ADC_K3-ADC_RANGE && pm < ADC_K3+ADC_RANGE )
{
if( keys[2]<10 ) keys[2]++;
} else keys[2]=0;
if( pm > ADC_K4-ADC_RANGE && pm < ADC_K4+ADC_RANGE )
{
if( keys[3]<10 ) keys[3]++;
} else keys[3]=0;

for(i = 0 ; i<4; i++)
{

if( keys[0]==9 ){
LED1_TOG;
lcd_int(1);
a=a+1;
tab2[i]=1;
}
if( keys[1]==9 ){
LED2_TOG;
lcd_int(2);
a=a+1;
tab2[i]=2;

}
if( keys[2]==9 ){
LED3_TOG;
lcd_int(3);
a=a+1;
tab2[i]=3;

}
if( keys[3]==9 ){
LED4_TOG;
lcd_int(4);
a=a+1;
tab2[i]=4;
}
}
if (a>4) {
lcd_cls();
a=0;

}

if ((tab1[0]==tab2[0])&&(tab1[1]==tab2[1])&&(tab1[2]==tab2[2])&&(tab1[3]==tab2[3])){
LED5_OFF;
}else LED5_ON;

}
}
//funkcja pomiaru
uint16_t pomiar(uint8_t kanal)
{
ADMUX = (ADMUX & 0xF8) | kanal;
ADCSRA |= (1< while (ADCSRA & (1< return ADCW;
}

//Procedrua przerwania
ISR( TIMER0_OVF_vect )
{
TCNT0 = 256-78; // częstotliwość 100 Hz
uint16_t n = Timer1;
if(n) Timer1=--n;
n = Timer2;
if(n) Timer2=--n;
}[/syntax]

Statystyki: Napisane przez Swoszaki — 24 maja 2015, o 19:41