Re: AT+WSCAN, parsowanie dostępnych sieci

2017-04-12T20:10:17+01:00

jacky_cy napisał(a):

Czy w ogóle zapis sieci do tablicy dwuwymiarowej to dobry pomysł.

Tablice wielowymiarowe to najczęściej nietrafiony pomysł - szczególnie, że w języku C mamy do dyspozycji struktury i można ładnie przygotować definicję struktury a później zrobić tablicę struktur - i zero potrzeby jakichś tablic wielowymiarowych ... z tego bym z marszu zrezygnował

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

druga sprawa po co zapisywać takie mega nadmiarowe dane stringowe do tablic/struktur? toż jak będzie w okolicy nagle 58 wykrytych sieci to się może nawet ci RAM chlasnąć ...

toż można sobie ładnie przygotować typami wyliczeniowymi enum definicje takich danych jak AES, TKIP, AES/TKIP, i tym podobne i do struktury zapisywać jedynie liczbowy typ wyliczeniowy. A co najwyżej nazwę sieci i hasło jako string. Później z typu wyliczeniowego łatwo odtworzyć takie stringi

------------------------ [ Dodano po: 3 minutach ]

to tak na gorąco co mi przychodzi do głowy i co mogę podpowiedzieć bo nie mam nawet jak teraz takiego kodu sprawdzać

Statystyki: Napisane przez mirekk36 — 12 kwi 2017, o 20:10

AT+WSCAN, parsowanie dostępnych sieci

2017-04-12T19:42:36+01:00

Chciałbym poprosić Was o wskazówki odnośnie parsowania danych z modułu wifi232 po wysłaniu komendy AT+WSCAN.

Na podstawie funkcji z poradnika zrobiłem funkcję parsującą stringa pomiędzy
liczbą (nr kanału) oraz przecinkiem, który jest zaraz po nazwie sieci. Niestety nie wiem co robię źle, ale przecinek po nazwie sieci wyświetla się zawsze, a i czasami dodatkowo do tablicy zapisują się jakieś strzępy np."AES", "TKIPAES"

Mimo iż ostatni element stringa czyli ten przecinek zeruję.
[syntax=c]wpstr->ssidbuf[ssid_index][str_end] = 0;[/syntax]
ssidbuf[ssid_index][str_end] na pewno jest przecinkiem, bo sprawdziłem wyświetlając na LCD

Poniżej funkcja str_parse_ssid:
[syntax=c]uint8_t str_parse_ssid (int sign, TPSTR_SSID * wpstr)
{
static uint8_t idx1;
static uint8_t str_end;
char mzn = sign;
uint8_t mzncon;//zmienna na potrzeby nr kanału
uint8_t flag;

if(wpstr->status < 0 || sign < 0){
return 0;//jezeli status rowny -1 lub brak znaku > return
}

if(!wpstr->status)//jezeli status = 0
{
mzncon = HexCharToInt(mzn);//konwersja char do uint8_t
if(((mzncon>=1)&&(mzncon<15))) flag=1; //pierwszy lancuch 2 znaki - ","
if(flag==1){//zwiekszanie indeksu
if(mzn==','){ //jesli index rowny 2
wpstr->status=1;
flag=0;
idx1=0;
}
} else
{
idx1 = 0;
}
} else if(wpstr->status){
//jezeli natrafiono na ciag
wpstr->ssidbuf[ssid_index][idx1++] = mzn;//zapis do bufora znak po znaku
if(idx1>16) idx1=0;

//sprawdzanie czy jest przecinek
if(mzn == ','){
if(wpstr->status==1){
str_end = idx1 - 1;
}
//jezeli natrafiono na ciąg WPA to nie zapisuj
if((wpstr->ssidbuf[ssid_index][0]=='W') && (wpstr->ssidbuf[ssid_index][1]=='P') && (wpstr->ssidbuf[ssid_index][2]=='A')){ //jeżeli znaleziony ciag to WPA to nie wpisuj do tablicy sieci

} else{
ssid_index++;
}
idx1=0;
wpstr->status = -1;//zatrzymujemy wyszukiwanie strumienia

wpstr->ssidbuf[ssid_index][str_end] = 0; //zerujemy ostatni znak - przecinek

return 1;

} else{//jezeli okaze sie ze nie sprawdzamy drugiego lancucha
wpstr->status = 1;
}
}

return 0;
}[/syntax]

oraz wywołanie w main:
[syntax=c]if (str_parse_ssid (cn, &wifiscan))// || cn!=-1)
{
lcd_locate(0,0);
lcd_str(wifiscan.ssidbuf[6]); //siódma nazwa sieci
wifiscan.status=0;

}
if(key(sw_ok, 20, 150) == SHORTPRESS) uart_puts("AT+WSCAN\r\n"); // wyslanie komendy at+wscan[/syntax]

Bardzo proszę o pomoc / wskazówki co jest nie tak z tą funkcją parsującą. Czy w ogóle zapis sieci do tablicy dwuwymiarowej to dobry pomysł. A może ktoś z Was rozwiązał już kwestie łączenia się z dowolną siecią z poziomu użytkownika?
Wielkie dzięki za pomoc i pomysły!!!

Statystyki: Napisane przez jacky_cy — 12 kwi 2017, o 19:42

ATNEL tech-forum

Re: AT+WSCAN, parsowanie dostępnych sieci

AT+WSCAN, parsowanie dostępnych sieci