ATNEL tech-forum

Re: Zamek na hasło

2014-10-12T10:27:48+01:00

Powalczyłem troche z tym przez sobotę i udało się. Oczywiście trzeba jeszcze kod nieco doszlifować, żeby wszystko zachowywało się tak jak potrzeba ale na tę chwilę porównanie tablic działa poprawnie i zapala diode. Poniżej fragment dotyczący samego porównywania i zapalania diody.

[syntax=c]klawiatura_analog();
if(pomiar_klawiatura(PC4)>40)
{
tab[i] = stan_przycisku;
UART_putc(tab[i]);//sprawdzam co zostało wpisnae do tablicy
stan_przycisku = 0; //zeruje stan przycisku, żeby tylko raz odczytać stan przycisku
// Mamy już wszystkie 3 znaki hasła
if (i == 2)
{
//Porównaj hasło
if (tab[0] == haslo[0] && tab[1] == haslo[1] && tab[2] == haslo[2])
{
LED_2_ON;
}
// ustaw wartości początkowe
i = 0;
tab[0] = 0; tab[1] = 0; tab[2] = 0;
}
i++;
}[/syntax]

Trzeba dorobić teraz kiedy ma się kasować i dać przycisk do potwierdzenia i kasowania wpisanego hasla.

Statystyki: Napisane przez fotograf9 — 12 paź 2014, o 10:27

Re: Zamek na hasło

2014-10-11T13:23:38+01:00

Cześć, ja miałem podobny problem przy zamku na RFID ,gdzie zapisywałem znaki kolejno do tablicy i porównywałem ze wzorcem.
Udało się rozwiązać i pięknie działa.

Tu co prawda wklepujesz cyfry ręcznie ,ale sama idea porównywania tablic może ci się przyda.

forum.atnel.pl/topic8255.html?hilit=zamek%20rfid

Statystyki: Napisane przez mac34 — 11 paź 2014, o 13:23

Re: Zamek na hasło

2014-10-11T12:51:59+01:00

Definicja zmiennych:

[syntax=c]//swiatlo zielone
#define LED_Z (1<#define LED_Z_ON PORTB &= ~LED_Z
#define LED_Z_OFF PORTB |= LED_Z
//swiatlo czerwone
#define LED_C (1<#define LED_C_ON PORTB &= ~LED_C
#define LED_C_OFF PORTB |= LED_C
//przycisk zapisu pomiaru mostka
#define KEY (1<//mruganie diodą dla sprawdzenia programu
#define LED (1<#define LED_ON PORTB &=~LED
#define LED_OFF PORTB |= LED
//syganlizacja diody jako otwarcie bramy
#define LED_2 (1<#define LED_2_ON PORTC &=~LED_2
#define LED_2_OFF PORTC |= LED_2
//klawiatura analogowa definicja przycisku
#define PRZYC_S1 0x31 // 1
#define PRZYC_S2 0x32 // 2
#define PRZYC_S3 0x33 // 3
#define PRZYC_S4 0x34 // 4
#define PRZYC_S5 0x35 // 5
#define PRZYC_S6 0x36 // 6
#define PRZYC_S7 0x37 // 7
#define PRZYC_S8 0x38 // 8
#define PRZYC_S9 0x39 // 9
#define PRZYC_S10 0x2A //*
#define PRZYC_S11 0x30 //0
#define PRZYC_S12 0x23 //#[/syntax]

A tu główny program, nie przesyłam bibliotek UARTA i ADC bo w tym przypadku są kompletnie zbędne.

[syntax=c]/*
* main.c
*
* Created on: 21-08-2014
* Author: B?a?ej
*/

#include
#include
#include
#include
#include "UART/UART.h"
#include "ADC/ADC.h"
#include "DEFINICJE/DEFINICJE.h"

uint8_t klawisz_wcisniety ();

uint8_t odebrano;
uint16_t wynik, a;
unsigned char stan_przycisku;

char haslo[]={0x31,0x33,0x35};
char tab[3];
int x=0,y=0,z=0;
int main (void)
{
DDRB |= LED_C | LED_Z | LED;
PORTB |= LED_Z;
DDRD &= ~KEY;
PORTD |= KEY;
DDRC |= LED_2;
PORTC |= LED_2;

ADC_init (); //inicjalizacja ADC
UART_init( __UBRR ); //inicjalizacja UART
UART_puts("Mostek tenosmetryczny\r\n");
UART_puts("Zmiany napiecia:\r\n");
sei();

while(1)
{

//sprawdzanie jaki przycisk jest wciśnięty
// chętnie wywaliłbym to do jakiejś podfunkcji żeby nie zabierać miejsca w pętli głównej programu.
if (pomiar_klawiatura(PC4)>980) stan_przycisku=PRZYC_S1;
else if (pomiar_klawiatura(PC4)>850) stan_przycisku=PRZYC_S2;
else if (pomiar_klawiatura(PC4)>770) stan_przycisku=PRZYC_S3;
else if (pomiar_klawiatura(PC4)>700) stan_przycisku=PRZYC_S4;
else if (pomiar_klawiatura(PC4)>600) stan_przycisku=PRZYC_S5;
else if (pomiar_klawiatura(PC4)>500) stan_przycisku=PRZYC_S6;
else if (pomiar_klawiatura(PC4)>400) stan_przycisku=PRZYC_S7;
else if (pomiar_klawiatura(PC4)>350) stan_przycisku=PRZYC_S8;
else if (pomiar_klawiatura(PC4)>270) stan_przycisku=PRZYC_S9;
else if (pomiar_klawiatura(PC4)>180) stan_przycisku=PRZYC_S10;
else if (pomiar_klawiatura(PC4)>100) stan_przycisku=PRZYC_S11;
else if (pomiar_klawiatura(PC4)>50) stan_przycisku=PRZYC_S12;

a=pomiar_mostek(PC5);

//dioda świeci - sprawdznei czy działa pętla.
LED_ON;
_delay_ms(100);
LED_OFF;
_delay_ms(100);

//otwieranie bramy na hasło, przyspianie przycisku do zmiennej

for(int i=0;i<3;i++)
{
if(stan_przycisku)
{
tab[i]=stan_przycisku;
for(int j=0;j<3;j++)
{
if(tab[j]==haslo[0]) x=1;
if(tab[j]==haslo[1]) y=1;
if(tab[j]==haslo[2]) z=1;
if(x+y+z==3) LED_2_ON;
}
}
}
if (klawisz_wcisniety())
{
LED_C_OFF;
LED_Z_ON;
UART_puts("\r\nZapisano pomiar! \r\n");
}

if(a>850)
{
LED_Z_OFF;
LED_C_ON;
}
}

}

ISR (USART_RXC_vect) //przerwanie odczytu danych
{
odebrano=odbierz();
if(odebrano==0x3F) //jeżeli odebrano znak ?
{
wynik=pomiar_mostek(PC5);//przypisywanie od zmiennej znakow przesłanych
UART_puts ("\r\n Wartosc ADC: ");
UART_putlong (wynik, 10); //wywołanie zmiennej wynik i zapis w postaci dziesiętnej
UART_puts (" Przycisk ADC: ");
UART_putlong (b, 10);
UART_puts ("\r\n");
UART_putc(stan_przycisku);
UART_puts ("\r\n");
}
}

uint8_t klawisz_wcisniety ()
{
if (!(PIND & KEY))
{_delay_ms(20);
if (!(PIND & KEY)) return 1;
}
return 0;
}[/syntax]
Jak w gram te program to zaświeca diodę jeżeli wciśniemy po prostu te trzy cyfyr obojetnie w jakiej kolejności i obojętnie czy naciśniemy w trakcie inny przycisk

Statystyki: Napisane przez fotograf9 — 11 paź 2014, o 12:51

Re: Zamek na hasło

2014-10-11T12:38:35+01:00

fotograf9 napisał(a):

Zasugerowałem się zrobieniem dwóch pętli,

A moglbys pokazac caly kod ? Byloby na pewno latwiej cos Ci podpowiedziec

Statystyki: Napisane przez RafPe — 11 paź 2014, o 12:38

Re: Zamek na hasło

2014-10-11T12:00:25+01:00

Zasugerowałem się zrobieniem dwóch pętli, jednak teraz jeżeli nadusimy obojętnie który przycisk z tablicy haslo zapali nam się dioda, kolejność wciskania też może być obojętna.

[syntax=c]char haslo[]={0x31,0x33,0x35};
char tab[3];

for(int i=0;i<3;i++)
{
if(stan_przycisku)
{
tab[i]=stan_przycisku;
for(int j=0;j<3;j++)
{
if(tab[j]==haslo[j]) LED_2_ON;
}
}
}[/syntax]
Na pewno kod który tu napisałem jest zbyt posty żeby to mogło zadziałać w dany sposób. Domyślam się że brakuje to jakieś zmiennej, która mogła by trzymać w pamięci kolejne porównania przycisków. Kombinowałem, żeby program sprawdzał czy przypisany przycisk do tab[j] równał się z wartością w tablicy haslo[]

[syntax=c]char haslo[]={0x31,0x33,0x35};
char tab[3];
int x,y,z;

for(int i=0;i<3;i++)
{
if(stan_przycisku)
{
tab[i]=stan_przycisku;
for(int j=0;j<3;j++)
{
if(tab[j]==haslo[0]) x=1;
if(tab[j]==haslo[1]) y=1;
if(tab[j]==haslo[2]) z=1;
if(z+y+x==3) LED_2_ON;
}
}
}[/syntax] Jedank w tym przypadku pętla zapamiętuje zmienną tylko raz x,y lub z.

Statystyki: Napisane przez fotograf9 — 11 paź 2014, o 12:00

Re: Zamek na hasło

2014-10-08T21:03:08+01:00

Witam

Na moje oko musisz mieć co najmniej dwie pętle for.

Pierwsza - zapis klawiszy do tablicy. Druga - porównanie i dopiero wynik.

Jak pomysł Ci podejdzie to coś skrobnę.

Pomysł z ADC dobry, ale może da się to zrobić łatwiej.

Statystyki: Napisane przez Lex_ — 8 paź 2014, o 21:03

Re: Zamek na hasło

2014-10-08T17:10:24+01:00

To jak tak na dziko wskocze z NIETESTOWANYM pomyslem

[syntax=c]uint8_t moj_pomiar=769;// to bylby pomiar z ADC

uint8_t war[4] = {980,880,780,680}; // Wartosci ADC dla klawiszy
char klaw_war[4] = {"1","2","3","4"};// dostepne wartosci kodu
char entered[4];// wprowadzony kod
char unlock[4] ="4132";// unlock

for (int var = 0; var < sizeof( war ); var++)
{
if( moj_pomiar > war[ var ])
{
entered[ var ] += klaw_war[var];
break;
}
}

// porwnianie entered i unlock ?!
// uzywajac funkcji wbudopwanej do porownywania...[/syntax]

A cos takiego ? Robisz sobie ADC a nastepnie masz tablice war z wartosciami ADC dla klawiszy. Pozniej masz klaw_war ktora ma odpowiednio wartosci char dla klawiszy.
Nastepnie entered czyli to co zostaje wprowadzone. I docelowo unlock czyli to co ma byc wprowadzone jako kod zabezpieczajacy

Po przejsciu przez tablice wartosci mozesz porownac entered oraz unlock i jesli sa takie same to cos robisz a jak nie to robisz cos innego ?!

Jesli kod zawiera jakis blad ... sory ale nie mialem czasu wrzucic na ATB :/

Statystyki: Napisane przez RafPe — 8 paź 2014, o 17:10

Re: Zamek na hasło

2014-10-08T16:32:25+01:00

Kolego ale tak jak pisałem wyżej bez twojego kodu mało co pomogę ... więc tylko podpowiadam jak wyżej.

Statystyki: Napisane przez majster — 8 paź 2014, o 16:32

Re: Zamek na hasło

2014-10-08T16:12:30+01:00

Nic dobrego nie przychodzi mi do głowy, jakby to poprawnie rozwiązać

Próbowałem z funkcją strcmp jednak ona porównuje stringi a nie pojedyncze znaki w tabeli. Wstawiając nową zmienną która liczy ile przycisków użyłem mogę wrzucić w pętle, ale zwiększy mi to tylko raz argument.

W funkcji warunkowej if mogę porównywać tylko jeden argument z tablicy.

Statystyki: Napisane przez fotograf9 — 8 paź 2014, o 16:12

Re: Zamek na hasło

2014-10-08T10:15:44+01:00

Kolego jak na moje oko to mało kodu dałeś więc nie mogę pomóc.

Ale ale doradzić zawsze mogę.

Powiedzmy że kod ma 6 cyfr dołożyłbym zmienną która liczyła by od 0 do 6 ... dlaczego ma liczyć wciśnięć klawisza ? A no dlatego że wbojamy

123456 cyfr i na końcu zatwierdzamy # i tutaj odpowiednio zmienna biegnie ...
012345 zmiennych i na końcu mamy 6 zmienną i po osiągnięciu zmienna = 6 dopiero porównywałbym czy cyfry się zgadzają z tymi zapisanymi w pamięci eeprom

Ta linijka jest zbędna:
[syntax=c]else {};[/syntax]

No i jeszcze to:

[syntax=c]if(stan_przycisku==haslo[i]) LED_2_ON; //tu ma porównywać[/syntax]

tutaj i jest numerem obiegu pętli for więc led zapali się zawsze i to od samego początku obiegu pętli.

Statystyki: Napisane przez majster — 8 paź 2014, o 10:15

Zamek na hasło

2014-10-08T09:58:35+01:00

Ostatnio zmagam się z problemem kodu, który będzie porównywał naciśnięte przyciski z tablicą.

Otwarcie zamku ma początkowo symbolizować zapalona dioda. Do programu wykorzystuję klawiaturę analogową z racji tego, że potrzebuje mieć do dyspozycji większą liczbę portów do czego innego.

Identyfikacja przycisków i tablica kodu wygląda następująco:
[syntax=c]if (pomiar_klawiatura(PC4)>980) stan_przycisku=PRZYC_S1;
else if (pomiar_klawiatura(PC4)>850) stan_przycisku=PRZYC_S2;
else if (pomiar_klawiatura(PC4)>770) stan_przycisku=PRZYC_S3;
else if (pomiar_klawiatura(PC4)>700) stan_przycisku=PRZYC_S4;
else if (pomiar_klawiatura(PC4)>600) stan_przycisku=PRZYC_S5;
else if (pomiar_klawiatura(PC4)>500) stan_przycisku=PRZYC_S6;
else if (pomiar_klawiatura(PC4)>400) stan_przycisku=PRZYC_S7;
else if (pomiar_klawiatura(PC4)>350) stan_przycisku=PRZYC_S8;
else if (pomiar_klawiatura(PC4)>270) stan_przycisku=PRZYC_S9;
else if (pomiar_klawiatura(PC4)>180) stan_przycisku=PRZYC_S10;
else if (pomiar_klawiatura(PC4)>100) stan_przycisku=PRZYC_S11;
else if (pomiar_klawiatura(PC4)>50) stan_przycisku=PRZYC_S12;
else {};
unsigned char haslo[]={0x32,0x35,0x39};[/syntax]

Rozumiem, że do porównania najlepiej trzeba użyć pętli for, która to będzie za każdym razem porównywać wciśnięty przycisk z tablicą. Jednak to nie za bardzo się tu sprawdza.
[syntax=c]for(i=0;i<3;i++) //trzy obiegi pętli bo trzy znaki w tablicy haslo[]
{
if(stan_przycisku==PRZYC_S10) break; //wciśnięta * przerwać pętle
if(stan_przycisku==haslo[i]) LED_2_ON; //tu ma porównywać
}[/syntax]
jednak pętla zapali diode po naciśnięciu 2 lub 5 lub 9. Rozumiem, że brakuje mi jakiegoś znaku, który dopiero po wcisnieciu przycisku pozwalał by na rozruch pętli i przyrównywanie. Czy może ktoś jakoś na to pomóc, nakierować?

Statystyki: Napisane przez fotograf9 — 8 paź 2014, o 09:58