Skip to content

Реворк воксов рейдеров#9277

Open
dageavtobusnick wants to merge 38 commits into
ss220-space:master220from
dageavtobusnick:vox_raider
Open

Реворк воксов рейдеров#9277
dageavtobusnick wants to merge 38 commits into
ss220-space:master220from
dageavtobusnick:vox_raider

Conversation

@dageavtobusnick

@dageavtobusnick dageavtobusnick commented May 18, 2026

Copy link
Copy Markdown
Collaborator

Что этот ПР делает

Портирован и почищен код отсюда ss220club/Paradise-SS220#900 и отсюда ss220club/Paradise-SS220#1423

Список изменений

🆑
add: Переработаны воксы рейдеры, их цели и возможности и способ спавна.
add: Добавлена новая валюта для воксов, а так же товары за нее.
refactor: Датумизированы воксы рейдеры.
del: Удален режим воксов рейдеров.
/:cl:

@dageavtobusnick dageavtobusnick requested a review from a team as a code owner May 18, 2026 22:54
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. Изменение карты PR содержит изменения в файлах карт. Спрайты PR содержит изменения в файлах DMI. TGUI PR содержит изменения в файлах TGUI. Звук PR изменяет звуковые эффекты. Инструменты PR вносит изменения в инструменты разработки, конфигурации IDE или конфигурацию AI помощника. Refactor PR проводит рефакторинг кода. Removal PR удаляет какой-то код. Feature PR добавляет новые механики или изменения в игровом процессе. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels May 18, 2026
@dageavtobusnick dageavtobusnick added Testmerge PR в данный момент проходит тестирование на серверах. Testmerge Candidate PR скоро будет в тестмёрже. labels May 18, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена. Данный пулл-реквест вносит масштабные изменения, заменяя устаревший режим «Heist» на новую систему антагонистов «Vox Raiders». Реализован полный набор снаряжения, специализированное вооружение (био-пушки, шипометы) и уникальная экономическая система с торговыми терминалами. В ходе ревью было выявлено 8 замечаний, из которых 5 классифицированы как критические ошибки. Основные проблемы связаны с потенциальными рантаймами из-за отсутствия проверок на null и длину списков, использованием неопределенных переменных и некорректной логикой обновления визуальных слоев. Также даны рекомендации по рефакторингу крупных процедур и улучшению надежности таймеров взаимодействия. Работа требует обязательного исправления указанных ошибок перед слиянием.

Comment thread code/datums/status_effects/magic_disguise.dm Outdated
Comment thread code/modules/antagonists/vox_raiders/antag_items/biogun_ammo.dm Outdated
Comment thread code/datums/mind.dm Outdated
Comment thread code/modules/antagonists/vox_raiders/antag_objects/vox_shop.dm Outdated
Comment thread code/modules/antagonists/vox_raiders/packs/vox_shop_pack_goods.dm
Comment thread code/game/objects/structures/beds_chairs/plastic_chair.dm
Comment thread code/modules/antagonists/vox_raiders/antag_items/biogun_ammo.dm
playsound(get_turf(src), 'sound/weapons/contractorbatonhit.ogg', 25, TRUE)
flick("trader-beam", src)

/obj/machinery/vox_trader/proc/get_value(mob/user, list/items_list, is_visuale_only = FALSE)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

🟡 Рекомендация по архитектуре

Проблема: Процедура get_value() слишком велика (почти 200 строк), что затрудняет её поддержку и чтение.

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

References
  1. Разбивайте крупные процедуры на более мелкие, логически разделенные для лучшей поддержки.

@github-actions github-actions Bot added the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label May 18, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels May 18, 2026
Comment thread code/datums/status_effects/magic_disguise.dm Outdated
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels Jun 1, 2026
@github-actions github-actions Bot added the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label Jun 1, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels Jun 2, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels Jun 18, 2026
@github-actions github-actions Bot added the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label Jun 18, 2026
@github-actions github-actions Bot added Merge Conflict PR содержит изменения, конфликтующие с master-веткой. and removed Merge Conflict PR содержит изменения, конфликтующие с master-веткой. labels Jun 19, 2026
@dageavtobusnick dageavtobusnick removed the Merge Conflict PR содержит изменения, конфликтующие с master-веткой. label Jun 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature PR добавляет новые механики или изменения в игровом процессе. Refactor PR проводит рефакторинг кода. Removal PR удаляет какой-то код. Testmerge Candidate PR скоро будет в тестмёрже. Testmerge PR в данный момент проходит тестирование на серверах. TGUI PR содержит изменения в файлах TGUI. Звук PR изменяет звуковые эффекты. Изменение карты PR содержит изменения в файлах карт. Спрайты PR содержит изменения в файлах DMI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants