Conversation
zalupa экшены оболочки для мода оболочка для мода перенесена в мехфаб маркер. удаление устаревшего переопределённого прока get_equipped_items, замена передаваемых аргументов на битфлаги, где этого ещё не было. компоненту лазера добавлен кд на использование 1 секунда
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена.
Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок. Обнаружено несколько проблем, включая критическую ошибку, которая может привести к падению сервера, а также несколько несоответствий стайлгайду проекта.
Сводка анализа:
- 🔴 Критических замечаний: 1
- 🟡 Рекомендаций: 6
- 💡 Предложений: 0
🔴 Критические замечания (требуют обязательного исправления):
code/modules/projectiles/projectile/energy.dm:574: Обнаружена некорректная перегрузка процедурыprehit, что приведёт к ошибке выполнения. Процедура определена для неверного типа объекта и пытается изменить несуществующие переменные.
🟡 Рекомендации по улучшению:
code/game/gamemodes/clockwork/clockwork_items.dm:1667-1669: Отсутствует проверкаisclocker(user), что позволяет не-культистам использовать предмет культа.code/game/gamemodes/clockwork/clockwork_workshop.dm:61: Название предмета «brass cradle» написано в нижнем регистре, что нарушает единообразие с другими названиями в списке.code/game/mecha/equipment/tools/mining_tools.dm:147: Описание предмета (desc) на английском языке, в то время как стайлгайд требует, чтобы тексты для игроков были на русском.code/game/mecha/equipment/tools/other_tools.dm:192: Описание предмета (desc) на английском языке.code/game/mecha/equipment/tools/other_tools.dm:879: Название предмета «brass cradle» написано в нижнем регистре.code/game/mecha/equipment/tools/work_tools.dm:116: Описание предмета (desc) на английском языке.
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.
Спасибо за ваш вклад в проект!
dageavtobusnick
force-pushed
the
master220
branch
from
10b38fa to
54187f7
Compare
pilygun
left a comment
pilygun
left a comment
There was a problem hiding this comment.
Очень много забытых ру неймов, так же описания старайся делать не конкретно под спец роль а общие, если хочешь под спец роль адаптировать то есть специальный флаг для этого
| integrated = TRUE | ||
|
|
||
| /obj/item/mecha_parts/mecha_equipment/drill/brass | ||
| name = "Brass exosuit drill" |
| return TRUE | ||
|
|
||
| /obj/item/mecha_parts/mecha_equipment/brass_armor_booster | ||
| name = "Brass Armor Booster Module" |
|
|
||
| /obj/item/mecha_parts/mecha_equipment/drill/brass | ||
| name = "Brass exosuit drill" | ||
| desc = "Оборудование для инженерных и боевых экзокостюмов. Это благословлённая версия бура." |
There was a problem hiding this comment.
Лучше поменять на то что это покрытый латунью механизм можно еще описать шестеренки или то что он светиться. Предмет могут осматривать не только игроки с спец ролью ратвра
|
|
||
| /obj/item/mecha_parts/mecha_equipment/cage/brass | ||
| name = "Brass cradle" | ||
| desc = "Модуль для экзокостюмов, используемый для задержания заблудших." |
There was a problem hiding this comment.
также более общее описание нужно. Если хочешь что бы игроки со спец ролю видели дополнительное описание есть отдельный флаг для этого
| return TRUE | ||
|
|
||
| /obj/item/mecha_parts/mecha_equipment/hydraulic_clamp/brass | ||
| name = "Brass clamp" |
| size = 1 | ||
|
|
||
| /obj/item/mecha_parts/mecha_equipment/weapon/energy/brasstaser | ||
| name = "\"Spark of the Righteous\" Mounted Taser" |
|
|
||
| /obj/item/mecha_parts/mecha_equipment/weapon/energy/brasslaser | ||
| equip_cooldown = 0.3 SECONDS | ||
| name = "\"Ray of Faith\" Laser" |
pilygun
added
the
📜 Локализация
5 participants
Что этот ПР делает
Добавляет возможность конвертировать мехи странным осколком с 10-ти секундным прогрессбаром
Добавляет несколько новых модулей что можно сделать:
Латунный усилитель брони - Снижает весь получаемый урон на 25%, Умножает текущий шанс отражения в 1.3 раза
Латунный бур - Почти идентичен алмазному буру, но сверлит быстрее.
Может бурить сквозь укрепленные стены.
Имеет 30% шанс оторвать конечность.
Имеет кулдаун в полсекунды между действиями. Прокрут длится 0.25 секунды.
Латунная колыбель - Работает как mecha_equipment/cage
Навесной тазер "Искра праведника" - оглушает на 0.5 секунд и наносит 30 урона по стамине.
Латунная клешня - Может поднимать любые не прикрученные к полу объекты.
Может погружать контейнеры в свой грузовой отсек.
Не может поднимать живых существ.
Может спихивать людей с пути в зеленом интенте или бить их в харме, нанося 25 единиц урона от травм.
Вместимость хранилища - 25 объектов
(Названия могли немного поменяться)
Почему это хорошо для игры
Больше контента для культа
Тестирование
Всё работало на момент проверки.