[TM ONLY] Вендоры для ОБР#5709
Conversation
Добавлены вендоры для ОБР Blue, Amber, RED, SRT (попытка в аналог БЩ для ЦКшников) и вендор с оборудованием для мехов
|
Добавил еще к SRT Mod paiter и баллончик с краской в раздел accessories |
|
Обнови ветку пожалуйста |
угу. Я заодно тогда изменения некоторые внесу. Новасектор снова поменяли названия пушек |
|
Claude finished @AnywayFarus's task in 3m 20s —— View job Ревью PR: [TM ONLY] Вендоры для ОБР
Общее впечатлениеХорошая идея и полезный контент — вендоры для ОБР действительно полезны при серьёзных угрозах. Спрайты выглядят аккуратно. Есть несколько проблем, которые нужно исправить перед мержем. Критическая ошибка — неверный путь к типуФайл: /obj/item/gun/energy/laser/cybersun/black_market_trader = 2,Реальный путь в кодобазе — Именно на это и указывает предложение от @Kocma-san — нужно исправить путь. Проблемы форматирования1. Отступы пробелами вместо табуляций в /obj/machinery/vending/ertvend/Initialize()
. = ..()
all_products_free = TRUE
qdel(GetComponent(/datum/component/payment))Код DM использует табуляцию. Здесь используются 4 пробела, что нарушает конвенции кодобазы. 2. Пропущены пробелы вокруг
Замечания по дизайну1. Граната в категории "Ammo" (RED вендор, строки 220–221) /obj/item/grenade/c4 = 3,
/obj/item/grenade/frag = 3,C4 и осколочные гранаты логичнее вынести в отдельную категорию (например, 2. Зелья скорости из ксенобиологии тематически не вписываются в снаряжение спецотряда охраны. Возможно, лучше заменить на что-то вроде стимулятора ( 3. Дублирование списка аксессуаров Все четыре специализированных вендора (Blue, Amber, Red, SRT) полностью копируют один и тот же 4. Закомментированный контроль доступа (строки 14–16) // scan_id = 1
// контроль доступов у вендоров не работает
// req_access = list("cent_general")Это означает, что вендоры бесплатны и доступны всем. Для TM-контента это может быть приемлемо, но при дальнейшем использовании на картах стоит разобраться с проблемой доступа, особенно с учётом RED вендора (hellgun, C4, фраги). 5. Заметка в коде (строка 336) // Вообще стоит сделать отдельные "preloaded" мехи в кодеЭто внутренняя заметка для разработчиков. Лучше завести issue или TODO в трекере, а не оставлять в рабочем коде. Что работает хорошо
Главное, что нужно исправить перед мержем: путь к типу cybersun (критический баг), отступы в |
|
лан, одного раза достаточно |
|
Он кстати смотрит не по коду в мастере, а по коду в текущей ветке. Потому что этот путь уже давно изменен: |
ты че |
может через ревью.мд ему можно дать команду смотреть в мастере? |
4 participants
О Pull Request
Добавлены вендоры для ОБР Blue, Amber, RED, SRT (попытка в аналог БЩ для ЦКшников) и вендор с оборудованием для мехов. Собственноручно перерисованные спрайты для каждого вендора. Оборудование приблизительно соответствует тому, что было представлено в темплейте модульной оружейки. В связи с тем, что неплохо было бы пересмотреть в дальнейшем список оружия для разных типов угроз, необходим ТМ. Надеюсь, временно
Как это может улучшить/повлиять на игровой процесс/ролевую игру
Может быть полезным при ремапе ЦК. Невероятно полезно уже сейчас при отправке ОБР на какую-то более-менее серьезную угрозу, а не на подравшихся в баре ассистентов.
Доказательства тестирования
Скриншоты/Видео
Generic вендор
ERT Blue
ERT Amber
ERT Red
ERT Mech Equipment
ERT - SRT(Blueshield)
Пополняшка и меню маяка призыва
Спрайты вендоров в редакторе (анимированы)
Спрайты вендоров в игре
Changelog
25/11/25 - релиз
🆑
add: Добавлены вендоры со снаряжением ОБР под все уровни угрозы (Blue, Amber, Red) и оперативников SRT (аналог БЩ)
add: Добавлен вендор со снаряжением для МЕХов ОБР
add: Добавлен маяк призыва(спавна) вендоров и resupply canister
/:cl: