station goal to centcom #1034
station goal to centcom #1034
Conversation
WalkthroughВ системе StationGoalPaperSystem.cs добавлена зависимость от CentcommSystem. Логика метода SendStationGoal изменена: сначала проверяется, существует ли grid у факс-машины, затем удостоверяются, что он является наибольшим или относится к Centcomm. При успешной отправке факса осуществляется спавн сущностей с последующей установкой флага. В файлах centcomm.yml и holopad.yml внесены корректировки в прототипы, компоненты и параметры сущностей, включая обновление суффиксов, позиций и меток. Changes
Sequence Diagram(s)sequenceDiagram
participant S as StationGoalPaperSystem
participant F as FaxMachine
participant G as Grid
participant E as EntitySpawner
S->>F: Запуск SendStationGoal()
F->>G: Получение grid
alt grid == null
F->>S: Пропуск итерации
else grid != null
F->>S: Проверка: наибольший или Centcomm?
S->>F: Отправка факса
S->>E: Спавн сущностей
E-->>S: Установка флага wasSent=true
end
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
github-actions
bot
added
S: Untriaged
Changes: Map
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
Content.Server/Corvax/StationGoal/StationGoalPaperSystem.cs (2)
2-2: Удалите неиспользуемый импорт.Импорт
Content.Server.Backmen.Arrivalsне используется в текущем файле.-using Content.Server.Backmen.Arrivals;
117-117: Улучшите читаемость условия проверки сетки.Сложное условие проверки сетки можно сделать более читаемым, разбив его на отдельные проверки.
-if(!(largestGrid == grid.Value || _centcomm.CentComGrid == largestGrid)) +var isLargestGrid = largestGrid == grid.Value; +var isCentCommGrid = _centcomm.CentComGrid == largestGrid; +if (!isLargestGrid && !isCentCommGrid)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
Content.Server/Corvax/StationGoal/StationGoalPaperSystem.cs(3 hunks)Resources/Maps/centcomm.yml(5 hunks)Resources/Prototypes/_Backmen/Entities/Structures/Machines/holopad.yml(2 hunks)
⏰ Context from checks skipped due to timeout of 300000ms (3)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: Test Packaging
🔇 Additional comments (10)
Resources/Prototypes/_Backmen/Entities/Structures/Machines/holopad.yml (3)
13-15: Проверка обновления суффикса для HolopadCentCommLongRange.
В данной сущности суффикс изменён на "Centcomm". Убедитесь, что данное изменение соответствует ожиданиям всей системы и что обработка нового значения не вызывает конфликтов в логике отображения или идентификации объекта.
19-23: Проверка обновления суффикса для HolopadCentCommUnlimitedRange.
Суффикс изменён на "CentComm, Operator", что отражает дополнительную информацию об операторе. Рекомендуется убедиться, что это значение корректно интегрируется с остальной логикой прототипов и не приводит к несоответствиям в работе системы.
47-50: Уточнение для currentLabel у HolopadCentCommEvac.
Изменённое значение currentLabel теперь установлено как "holopad-centomm-evac". Проверьте, что это изменение обеспечивает требуемое форматирование, в частности, если предполагается наличие завершающего символа новой строки, то это условие корректно выполняется.Resources/Maps/centcomm.yml (5)
21768-21780: Проверка корректности удаления свойств прототипа
В данном хунке удаляются строки, задающие прототипы: первая строка с "proto: ComputerWage" и строка с "proto: ConveyorBelt". Согласно описанию pull request, сущность с uid 207 должна быть повторно добавлена под новым прототипом (в описании упоминается переопределение), поэтому удаление "proto: ComputerWage" требует проверки. Аналогично, убедитесь, что удаление "proto: ConveyorBelt" соответствует новой логике задания сущностей.
23533-23540: Добавление компонента Pullable
Добавлен новый компонент "Pullable" с параметром prevFixedRotation, равным True, для сущности с uid 3746. Изменение соответствует описанию новых требований. Рекомендуется перепроверить, что это изменение совместимо с остальными компонентами и не приводит к конфликтам в физическом поведении объекта.
28316-28322: Проверка обновления прототипа GunSafe
Удаляется строка "proto: GunSafeSubMachineGunVector". Согласно описанию pull request, прототип для GunSafe должен быть обновлён с GunSafeSubMachineGunDrozd на GunSafeSubMachineGunVector. Убедитесь, что удаление данной строки не приводит к тому, что сущность оказывается без необходимого назначения прототипа. Возможно, требуемое значение прототипа должно быть задано в другом месте.
28534-28548: Проверка обновления прототипа Holopad
В этом хунке удаляется строка, задающая "proto: HolopadCentCommUnlimitedRange". Согласно описанию, именно этот прототип должен использоваться, а также должен быть добавлен компонент "Label" с currentLabel: "ЦК - Оператор". Проверьте, что удаление строки прототипа не нарушает логику определения сущности и что конфигурация holopad соответствует требованиям новой функциональности.
34377-34383: Обновление позиции и удаление прототипа RubberStampDenied
Здесь изменяется позиция компонента Transform для uid 7057, что соответствует обновлённой позиции, а также удаляется строка "proto: RubberStampDenied". Убедитесь, что удаление данного прототипа намеренно и что сущность больше не должна использовать его для определения своей логики или внешнего вида.Content.Server/Corvax/StationGoal/StationGoalPaperSystem.cs (2)
111-126: Одобрено: улучшена логика проверки и отправки.Изменения улучшают логику работы системы:
- Добавлена проверка на null для сетки
- Добавлена валидация против сетки CentComm
- Создание сущностей перенесено после успешной отправки факса
121-124: Добавьте обработку ошибок при создании сущностей.Создание сущностей может завершиться неудачей, но текущая реализация не обрабатывает возможные ошибки.
Рекомендуется добавить обработку ошибок и логирование:
foreach (var spawnEnt in goal.Spawns) { - SpawnAtPosition(spawnEnt, Transform(faxUid).Coordinates); + try + { + var entity = SpawnAtPosition(spawnEnt, Transform(faxUid).Coordinates); + if (entity == null) + { + Log.Warning($"Failed to spawn entity {spawnEnt} for station goal"); + } + } + catch (Exception ex) + { + Log.Error($"Error spawning entity {spawnEnt} for station goal: {ex}"); + } }
🆑
Summary by CodeRabbit
Рефакторинг
• Улучшена обработка отправки целей станции через факс, обеспечивая более точное и последовательное выполнение операций.
Новые возможности
• Оптимизированы объекты на карте: переработаны прототипы устройств, обновлены их физические свойства, добавлены новые интерактивные элементы и метки.
Стиль
• Обновлены текстовые подписи и оформление некоторых терминалов для повышения единообразия и удобства восприятия.