PDA: Messenger program system #48

TokDenis · 2023-05-31T19:12:34Z

PDA: Messenger program system

Добавил мессенджер программу для cartridge системы.
MessengerCartridgeComponent - хранит полученные сообщения и информацию о пользователе программы
MessengerCartridgeSystem - использует DeviceNetworkSystem для общения между мессенджерами, подробнее можно почитать в комментариях самой системы

Проблемы

Хранение в MessengerCartridgeComponent только полученных сообщений выглядит логично, что бы не задваивать данные. Но не слишком удобно для получения от туда данных.
Есть идея заменить\дополнить MessengerCartridgeComponent чем то вроде MessengerServerComponent который и будет выступать в роле хранилища сообщений. Его можно будет поставить на карте, можно будет выключить, украсть или взять данные по чужим чатам и т.д.
Как первый этап Canvas говорил что можно использовать за сервер TelecomServer.
Есть проблема с получением имени члена экипажа, ид карта не инициализирована при OnInstall программы с мессенджером.
Мы хотим вообще использовать DeviceNetworkSystem или работать без этого?
Может ли пользователь менять имя? Локально для себя можно будет менять имя контактов, в перспективе.

TODO

Добавление нового контакта при прилете с шатла. т.е. смене network сети.
Интерфейс тестовый, сделаем красиво как с функционалом определимся.
Пиликать рингтоном при получение сообщения. (главная фича в общем то)

Медиа

Проверки

PR полностью завершён и мне не нужна помощь чтобы его закончить.
Я внимательно просмотрел все свои изменения и багов в них не нашёл.
Я запускал локальный сервер со своими изменениями и всё протестировал.
Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Изменения

add: MessengerCartridgeComponent
add: MessengerCartridgeSystem

TokDenis · 2023-06-01T16:36:25Z

Обсуждение по месенджеру:

Храним сообщения централизовано в сервере типа TelecomServer, назовем как нибудь MessendgerServer.
Храним так же информацию о пользователе что бы если сломается пда, мы смогли достать чат.
Враги прилетающие на станцию не будут подключаться к серверу, посмотреть на использование FactionComponent.
а. проверить не будет ли атаковать турель такой pda.
b. вынести FactionComponent в общий, а не только для NPC.
Карта - аутенфицирует пользователя в месенджере, можно вставить в чужой пда и прочитать свои сообщения.

rincew1nd · 2023-06-07T12:39:52Z

Немного дополнительной информации.
Посчитал эту задачи интересной и полезной в освоении базового UI, обмена сообщениями и эвентами.
В итоге уделил часов 10 своего времени и накидал рабочий прототип который можно допилить:
https://github.com/rincew1nd/space-station-14/tree/messenger-cartridge-system

Что было сделано:

Базовый UI
Регистрация пользователей в системе при наличии Id карты в PDA
Онлайн статус пользователя
Отправка сообщений пользователям, получение истории сообщений
Кеш сообщений работает только в пределах одной станции

Что нужно сделать:

Красивый UI
Кеш сообщений на стороне пользователя для сокращения трафика Server-Client
Оповещение получателей о новых сообщениях
Оповещение отправителей о неудачной доставке (пользователь офлайн)
Обновление имени при его изменении на Id карте (если такое вообще возможно)
Получение истории переписки с других станций, если она была
Сброс истории сообщений в конце раунда
Возможно: терминал на станции через который можно посмотреть переписку, требует доступы
Возможно: добавить проверку рейнджа до ближайшего терминала что бы не давать смотреть историю сообщений в дали от станции

Redict · 2023-06-07T12:41:52Z

@rincew1nd @TokDenis синканитесь по задаче тогда

rincew1nd · 2023-06-07T13:48:01Z

@rincew1nd @TokDenis синканитесь по задаче тогда

Мы уже. Я ему отдал исходники 3 дня назад. Просто решил сюда продублировать.

github-actions · 2023-06-07T18:06:48Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

…tem and component, init messenger cartridge UI

TokDenis · 2023-06-20T23:22:42Z

Переделал UI
Звук рингтона при новом сообщение
Локализовал
Стейт +- эффективно отправляется. При первом открытии отправляется весь стейт, далее только отсутствующие.
Аутентификация ID картой. Можно расширить.
Авторизация системой AccessReader
Добавил два прототипа сервера Nanotrasen и синдиката
Мессенджер основан на чатах и их участниках. Можно расширить чаты до групп, каналов и тд

Общий вид (не прочитанное сообщение выделяется желтым)

Поиск по чатам и сообщениям

Избранное (чат с самим собой)

Redict · 2023-06-21T21:20:07Z

Ready for review?

TokDenis · 2023-06-21T23:09:22Z

Ready for review?

Yes

rincew1nd · 2023-06-23T04:38:09Z

Resources/Prototypes/Entities/Objects/Devices/Circuitboards/Machine/production.yml

+  description: A machine printed circuit board for a messenger server.
+  components:
+    - type: MachineBoard
+      prototype: CrewMonitoringServer


Зачем добавлять CrewMonitoringServer компонент к мессенджер серверу?

Копипаста, поправлю

rincew1nd · 2023-06-23T04:41:07Z

Resources/Prototypes/Entities/Objects/Devices/cartridges.yml

+    - type: StationLimitedNetwork
+    - type: DeviceNetwork
+      deviceNetId: Wireless
+      receiveFrequencyId: MessengerClient


Нельзя заюзать уже существующие волны для PDA?

Я пробовал, но тк сам клиент работает как отдельный компанент я решил не засорять пда и тем более можно накинуть клиент на любое entity и прописать логику работы. Как пример - консоль с которой можно отправить сообщения или вендинговый автомат который отправляет всем рекламу...

rincew1nd · 2023-06-23T04:44:19Z

Resources/Prototypes/Entities/Objects/Devices/pda.yml

@@ -63,6 +63,7 @@
    softness: 5
    autoRot: true
  - type: Ringer
+  - type: StationLimitedNetwork


Ты ограничиваешь работу всего ПДА до одной станции. Если я не ошибаюсь, у тебя может быть несколько станций за один раунд, что приведёт к тому, что твой ПДА будет картошкой на других станциях.

Да, ограничение работает на текущю станцию (как я понял, можешь посмотреть), т.е. если ты перейдешь в другую станцию и там будет другой сервер, то пда свяжется с ним, а не будет отправлять сообщения на все сервера на карте.

Посмотрел по коду. Да ограничивает в пределах текущей станции на которой находишься. т.е. перейдешь на другую сеть сменится на сеть станции на которую перешел.

rincew1nd · 2023-06-23T04:47:14Z

Resources/Prototypes/Entities/Structures/Machines/messenger_server.yml

+      range: 500
+    - type: MessengerServer
+      serverName: "Nanotrasen Messenger Server"
+    - type: AccessReader


Любой человек может зааксессить данные ислючая синдикат?

Да, (если у него тег !NuclearOperative) сейчас как я понял из вражеских доступов только NuclearOperative.

rincew1nd · 2023-06-23T04:47:38Z

Resources/Prototypes/Entities/Structures/Machines/messenger_server.yml

+      denyTags: [ "NuclearOperative" ]
+
+- type: entity
+  id: PDASyndicateMessengerServer


Зачем отдельный сервер для синдиката?

Как пример. Вроде у них есть свой корабль, не? Туда установить можно.

rincew1nd · 2023-06-23T05:59:32Z

Content.Server/CartridgeLoader/Cartridges/MessengerClientCartridgeSystem.cs

+        BroadcastCommand(uid, args.Loader, MessengerClientCommandCheckServer);
+    }
+
+    private void BroadcastCommand(EntityUid senderUid, EntityUid loaderUid, string command)


Я правильно понял что ты оповещаешь все сервера о том что игрок поставил картридж?
Это в какой-то степени не правильно, так как игрок находится только на какой-то станции и может быть вне ренджа сервера.

Есть компонент StationLimitedNetwork он это отслеживает. Ренж отслеживает вроде WirelessNetworkConnection

rincew1nd · 2023-06-23T06:04:28Z

Content.Server/Messenger/MessengerServerSystem.cs

+            case MessengerClientCartridgeSystem.MessengerClientCommandCheckServer:
+            {
+                // check authorization to server before registration
+                if (!AccessCheck(uid, args.Data))


Заведи классы, которые будут соответствовать данным для каждого события, клади их в args.Data с каким-нибудь нужным тебе ключём, конверти в нужный тип на сервере и не мучайся с парсерами.
if (args.Data.ContainsKey("payload") && args.Data["payload"] is PayloadType payload)

Немного не понял. Можешь подробнее? Это обязательно нужно в текущей итерации?

В шарпе все классы это object. Object можно конвертнуть обратно в классы, так как object знает какой он был классом.
Если сделать ‘’’if(object is NewClass classObj)’’’, то код зайдёт в if только если объект нужного типа и конвертнулся нормально, а classObj будет заполнен инстансом.

args.Data это объект, который можно сделать отдельным классом на каждый тип ивентов. А потом смотреть на тип ивента и понимать в какой класс нужно конвертнуть данные. Если конвертация упала, то что-то пошло не так.

rincew1nd · 2023-06-23T06:13:39Z

Content.Server/Messenger/MessengerServerSystem.cs

+                var message = new MessengerMessage(chatKey.Id, contactKey.Id, _gameTiming.CurTime, messageText);
+                var messageKey =
+                    component.AddMessage(message);
+                message.Id = messageKey.Id;


Айдишник должен заполняться внутри AddMessage. Передай с ref message

До записи в компонент ид не известно.

rincew1nd · 2023-06-23T06:19:30Z

Content.Server/Messenger/MessengerServerSystem.cs

+        // and chats with every entity client
+        var chatsList = new List<ChatKey>();
+
+        foreach (var (_, contact) in component.GetClientToContact())


Огромное колличество мусорных объектов в памяти. Зачем создавать чаты которых пока ещё даже нет.

Не такое уж и огромное. Так проще и расширять понятнее. Вот есть контакты, вот есть чаты, в чатах есть их члены и сообщения. Сообщения тоже отдельно хранятся. В общем небольшая база данных.

Окей. Видимо нужно учитывать что это не высоконагруженный сервак и у нас не десятки тысяч человек, наверно сможем выделить лишнюю память на кеш под раунд.

rincew1nd · 2023-06-23T06:24:02Z

Content.Server/CartridgeLoader/Cartridges/MessengerClientCartridgeSystem.cs

+        if (!Resolve(uid, ref component))
+            return;
+
+        switch (command)


rincew1nd · 2023-06-23T06:29:03Z

Посмотрел всё что касается сервера. UI не смотрел, не осталось свободного времени.

rincew1nd · 2023-06-23T09:10:07Z

Ещё тестов нет и они падают

KashRas · 2023-06-26T21:07:22Z

Выдаёт постоянно ошибку, мол, сервер не найден.

TokDenis · 2023-06-26T21:47:43Z

Выдаёт постоянно ошибку, мол, сервер не найден.

Я на карту не добавлял. В поиске entity найди мессенджер и установи, клиент подключится при открытие UI

KashRas · 2023-06-27T04:39:41Z

Выдаёт постоянно ошибку, мол, сервер не найден.

Я на карту не добавлял. В поиске entity найди мессенджер и установи, клиент подключится при открытие UI

Всё равно не видит сервер. К тому же, я его добавлял.

github-actions · 2023-07-15T07:06:29Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

github-actions · 2023-07-16T14:26:49Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

aw-c · 2023-07-17T19:14:44Z

@TokDenis привет, есть что доделывать и мёржить или закрывать?

TokDenis · 2023-07-17T19:34:47Z

@TokDenis привет, есть что доделывать и мёржить или закрывать?

Нужно доделать правки на которые указал rincew1nd, я напишу в дискорде когда будет готово. Думаю на неделе выделю несколько часиков

github-actions · 2023-07-29T08:15:01Z

This pull request has conflicts, please resolve those before we can evaluate the pull request.

TokDenis · 2023-08-03T08:09:23Z

Инструкция как поставить сервер что бы работало

Ищем сервер для сообщений и ставим в нужное место

Проверяем работает открывая приложение в пда

refactore nyarlathotep

PDA: Messenger program system

1196512

github-actions bot added Changes: Localization Изменение затронуло файлы ".ftl" Changes: UI Изменение затронуло файлы ".xaml" labels May 31, 2023

TokDenis changed the title ~~PDA: Messenger program system~~ DRAFT: PDA: Messenger program system May 31, 2023

TokDenis changed the title ~~DRAFT: PDA: Messenger program system~~ PDA: Messenger program system May 31, 2023

Redict marked this pull request as draft June 6, 2023 16:51

github-actions bot added the Merge Conflict Some content has conflicts. label Jun 7, 2023

TokDenis added 3 commits June 21, 2023 02:53

init messenger server system and component, init messenger client sys…

2e82995

…tem and component, init messenger cartridge UI

merge master

ccb0e00

del netsync false

e9af167

github-actions bot removed the Merge Conflict Some content has conflicts. label Jun 20, 2023

Redict requested a review from rincew1nd June 22, 2023 10:05

github-actions bot added the Status: Needs Review label Jun 22, 2023

Redict marked this pull request as ready for review June 22, 2023 10:06

rincew1nd suggested changes Jun 23, 2023

View reviewed changes

github-actions bot added the Merge Conflict Some content has conflicts. label Jul 15, 2023

github-actions bot added Merge Conflict Some content has conflicts. and removed Merge Conflict Some content has conflicts. labels Jul 16, 2023

aw-c force-pushed the master branch from bacad85 to beb0c2a Compare July 25, 2023 00:32

TokDenis added 2 commits July 29, 2023 01:30

code review

ba92c74

merge master

08b96ef

github-actions bot removed the Merge Conflict Some content has conflicts. label Jul 28, 2023

fix

312a905

github-actions bot added the Merge Conflict Some content has conflicts. label Jul 29, 2023

merge master

c626b55

github-actions bot removed the Merge Conflict Some content has conflicts. label Jul 30, 2023

submodule update

2608ab4

aw-c merged commit 0fd29da into SerbiaStrong-220:master Aug 3, 2023
12 checks passed

stalengd pushed a commit to stalengd/space-station-14 that referenced this pull request Oct 27, 2024

Merge pull request SerbiaStrong-220#48 from FunTust/Cult!Cult!Cult!

21d4869

refactore nyarlathotep

PDA: Messenger program system #48

PDA: Messenger program system #48

Conversation

TokDenis commented May 31, 2023

PDA: Messenger program system

TokDenis commented Jun 1, 2023

rincew1nd commented Jun 7, 2023

Redict commented Jun 7, 2023

rincew1nd commented Jun 7, 2023

github-actions bot commented Jun 7, 2023

TokDenis commented Jun 20, 2023

Redict commented Jun 21, 2023

TokDenis commented Jun 21, 2023

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

rincew1nd commented Jun 23, 2023

rincew1nd commented Jun 23, 2023

KashRas commented Jun 26, 2023

TokDenis commented Jun 26, 2023

KashRas commented Jun 27, 2023 • edited Loading

github-actions bot commented Jul 15, 2023

github-actions bot commented Jul 16, 2023

aw-c commented Jul 17, 2023

TokDenis commented Jul 17, 2023

github-actions bot commented Jul 29, 2023

TokDenis commented Aug 3, 2023

KashRas commented Jun 27, 2023 •

edited

Loading