feat: begin work on #7 and #10 #14

Manually merged
nm17 merged 1 commit from device_managment into master 2024-06-01 23:24:31 +03:00
Owner
No description provided.
nm17 force-pushed device_managment from 3e684d093f to 956905e6e8 2024-06-01 22:31:35 +03:00 Compare
nm17 changed title from WIP: feat: begin work on #7 and #10 to feat: begin work on #7 and #10 2024-06-01 22:31:44 +03:00
Author
Owner

Сейчас от сокет сервера только файл заглушка. Пока только сделал разделение кода заливки сущностей в отдельный файл. Нужно это залить сейчас чтобы я смог наконец-то настроить форматирование, пока не поздно.

Сейчас от сокет сервера только файл заглушка. Пока только сделал разделение кода заливки сущностей в отдельный файл. Нужно это залить сейчас чтобы я смог наконец-то настроить форматирование, пока не поздно.
requested review from DarkCat09 2024-06-01 22:33:25 +03:00
Author
Owner

И ещё сделал заливку команд в базу. Чтение будет потом в сокет сервере.

И ещё сделал заливку команд в базу. Чтение будет потом в сокет сервере.
DarkCat09 reviewed 2024-06-01 22:50:56 +03:00
@ -86,0 +110,4 @@
// TODO: Добавить гистерезис
// Отчёт совместимости: отсутствует
if device_time > now {
Owner

Можно сделать:

let device_time = self.time
  .map(|t| if t > now {
    now
  } else {
    t
  })
  .unwrap_or(now)
Можно сделать: ```rust let device_time = self.time .map(|t| if t > now { now } else { t }) .unwrap_or(now) ```
Author
Owner

Можно и наверное так канонично делать в расте, но это не читабельно. Уж извини, но этот вариант мне не нравится в данном случае.

Можно и наверное так канонично делать в расте, но это не читабельно. Уж извини, но этот вариант мне не нравится в данном случае.
Owner

А точно let mut плохо для производительности не будет?

Если считаешь более читабельным – окей, пусть так будет. Мне наоборот кажется, что странно выглядит.

А точно `let mut` плохо для производительности не будет? Если считаешь более читабельным – окей, пусть так будет. Мне наоборот кажется, что странно выглядит.
DarkCat09 marked this conversation as resolved
@ -111,6 +111,10 @@ pub async fn parse_nm_qs_format(input: &str) -> Result<NMDeviceDataPacket, QSPar
lon: parse_decimal_if_exists(&mut parsed, "lon")?,
alt: parse_decimal_if_exists(&mut parsed, "alt")?,
time: parse_epoch_if_exists(&mut parsed, "time")?,
// TODO: Выяснить можно ли передавать команды по QS и можно ли их отличить от
Owner

Скорее всего нет, или как минимум никто в здравом уме не будет через кверистринг слать список команд, я боюсь представить чем там кодируется список (типа &cmd1=...&cmd2=... ?).

В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?

Скорее всего нет, или как минимум никто в здравом уме не будет через кверистринг слать список команд, я боюсь представить чем там кодируется список (типа `&cmd1=...&cmd2=...` ?). В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?
Author
Owner

Я это добавил так, на будущее. Вдруг там каким-то магическим образом можно.

В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть?

Да там в доках большинство не написано, мб где-то так делают на проде. Хз, я и не такое видел в коде некоторых библиотек работы с narodmon.

Я это добавил так, на будущее. Вдруг там каким-то магическим образом можно. > В доках нету => а как юзеры-то об этой фиче узнают, даже если она есть? Да там в доках большинство не написано, мб где-то так делают на проде. Хз, я и не такое видел в коде некоторых библиотек работы с narodmon.
Owner

А, весело, получается у разработчиков и админов выпытывают незадокументированные апи либо сами перебирают варианты)

А, весело, получается у разработчиков и админов выпытывают незадокументированные апи либо сами перебирают варианты)
DarkCat09 marked this conversation as resolved
DarkCat09 approved these changes 2024-06-01 23:03:16 +03:00
DarkCat09 left a comment
Owner

LGTM

LGTM
nm17 manually merged commit 9617f5ffbe into master 2024-06-01 23:24:31 +03:00
nm17 deleted branch device_managment 2024-06-01 23:41:29 +03:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: iotishnik/server#14
No description provided.