BLE Supla Gateway - wyzwanie dla programistów

[Beku]

No niby tak, może nawet taniej by wyszło.

ESP32 na allegro chodzi po 30 zł. Naprawdę jest sens jeszcze bardziej oszczędzać? Tym bardziej, że mowa tutaj o czymś w rodzaju bramki, więc będzie to raczej jedno takie urządzenie "to rule them all"

linka wrzuciłem jako ciekawostka

A ja odpisałem z grzeczności i napisałem ze esp32 to rozsądny wybór

[hZkgZK]

Spoko pomysł z tą bramką. Akurat ostatnio zrobiłem sobie obsługę Xiaomi Flora https://www.aliexpress.com/item/32841210035.html. Tu mój kod do obsługi urządzenia jakbyście chcieli dodać https://github.com/pikciu/Arduino-Flora. W wolnej chwili zrobie z tego libkę do PlatformIO bo na razie mi się nie chciało
działa od wczoraj na esp32

IMG_3967.png

te labelki trzeba pozmieniac bo taki mix pol-eng słabo wygląda

[klew]

Spoko pomysł z tą bramką. Akurat ostatnio zrobiłem sobie obsługę Xiaomi Flora

Dzięki za udostępnienie projektu
W konstruktorze klasy Flora używasz mało bezpiecznego rozwiązania: https://github.com/pikciu/Arduino-Flora ... ora.cpp#L4
Przypisujesz wskaźnik do ciągu znaków. To się może dość łatwo wysypać w wielu sytuacjach. Taki ciąg znaków powinien być kopiowany

[hZkgZK]

Spoko pomysł z tą bramką. Akurat ostatnio zrobiłem sobie obsługę Xiaomi Flora

Dzięki za udostępnienie projektu
W konstruktorze klasy Flora używasz mało bezpiecznego rozwiązania: https://github.com/pikciu/Arduino-Flora ... ora.cpp#L4
Przypisujesz wskaźnik do ciągu znaków. To się może dość łatwo wysypać w wielu sytuacjach. Taki ciąg znaków powinien być kopiowany

Dzięki za code review jak powinno być poprawnie? na codzień zajmuję się programowaniem na iOS (Swift) i do c++ mi daleko

[klew]

Dzięki za code review jak powinno być poprawnie? na codzień zajmuję się programowaniem na iOS (Swift) i do c++ mi daleko

W

Code: Select all

char *

jest trzymany wskaźnik do miejsca w pamięci zawierającego jeden bajt (char). Twój aktualny kod po prostu kopiuje ten adres w pamięci. Gdy więc stworzysz instancję klasy Flora przekazując ciąg znaków w jakimś obiekcie tymczasowym, to później ta klasa może się odwoływać do pamięci, w której mogą być już inne dane.

Najprościej jest zrobić w klasie Flora zmienną

Code: Select all

char address[MAX_SIZE_OF_ADDRES];

i kopiować dane np. funkcją strncpy (http://www.cplusplus.com/reference/cstring/strncpy/)

[hZkgZK]

Dzięki za code review jak powinno być poprawnie? na codzień zajmuję się programowaniem na iOS (Swift) i do c++ mi daleko

W

Code: Select all

char *

jest trzymany wskaźnik do miejsca w pamięci zawierającego jeden bajt (char). Twój aktualny kod po prostu kopiuje ten adres w pamięci. Gdy więc stworzysz instancję klasy Flora przekazując ciąg znaków w jakimś obiekcie tymczasowym, to później ta klasa może się odwoływać do pamięci, w której mogą być już inne dane.

Najprościej jest zrobić w klasie Flora zmienną

Code: Select all

char address[MAX_SIZE_OF_ADDRES];

i kopiować dane np. funkcją strncpy (http://www.cplusplus.com/reference/cstring/strncpy/)

A czy const nie zabezpiecza mnie przed tym, że w tym miejscu w pamięci coś mi nadpisze dane? O ile dobrze rozumiem odpowiedz https://stackoverflow.com/questions/983 ... r/47300995 to właśnie tak jest. Mam wskaźnik na niemodyfikowany ciąg znaków. Wskaźnik mam zapamiętany w klasie Flora i dopóki go "nie przesunę" będzie on wskazywać na ten sam obszar w pamięci, a obszar ten jest "zarezerwowany" przez const i nie można go zmienić. Tak to rozumiem ale bardzo możliwe że jestem w błędzie
W każdym razie chyba nie ma co zaśmiecać tematu ale jak Ci się chce dalej mi to tłumaczyć to jest priv

[klew]

A czy const nie zabezpiecza mnie przed tym, że w tym miejscu w pamięci coś mi nadpisze dane? O ile dobrze rozumiem odpowiedz https://stackoverflow.com/questions/983 ... r/47300995 to właśnie tak jest. Mam wskaźnik na niemodyfikowany ciąg znaków. Wskaźnik mam zapamiętany w klasie Flora i dopóki go "nie przesunę" będzie on wskazywać na ten sam obszar w pamięci, a obszar ten jest "zarezerwowany" przez const i nie można go zmienić. Tak to rozumiem ale bardzo możliwe że jestem w błędzie
W każdym razie chyba nie ma co zaśmiecać tematu ale jak Ci się chce dalej mi to tłumaczyć to jest priv

To tak tylko dla potomnych napiszę, że nie, to Cię przed niczym nie zabezpiecza
"const" informuje kompilator, aby nie pozwalał Tobie modyfikować zawartości wskazywanej przez ten wskaźnik.
Natomiast nic nie stoi na przeszkodzie, aby inne częsci programu coś po tej pamięci pisały, gdy dostaną taki przydział (a to się może zdarzyć).
Dalej możemy ciągnąć tego offtopa na priv

[pzygmunt]

Używając strncpy zwróć uwagę na problem możliwego skopiowania ciągu znaków bez nulla na końcu. Jeśli nie użyjesz tej funkcji świadomie to możesz mieć problem z uzyskaną kopią.

Dla bezpieczeństwa użyj snprintf do skopiowaniu ciągu lub jeśli już użyjesz strncpy to po jego wystąpieniu
wykonaj
dest[sizeof(dest)-1] = 0;

[Beku]

Się dowiedziałem że te nowe kwadratowe czujniki mają zakodowane komunikaty.
Są już rozkodowane ale trzeba je inaczej obsłużyć

https://github.com/custom-components/se ... ZDCgB8EEEU

Te okrągłe podobno wysyłają czystym tekstem.

[fracz]

Podejrzewałem że cena tych prostokątnych jest podchwytliwa