tweak: баллун при сообщении от ревенанта#8490
tweak: баллун при сообщении от ревенанта#8490SCPfollower wants to merge 1 commit intoss220-space:master220from
Conversation
|
Пока побудет в драфте, ибо все же думаю сделать через алерты (которые медленно на пол экрана появляются) |
![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
Аналитический отчёт от Большого Брата. Обработка запроса завершена.
Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно.
Сводка анализа:
- 🔴 Критических замечаний: 0
- 🟡 Рекомендаций: 1
- 💡 Предложений: 0
🟡 Рекомендации по улучшению:
code/game/gamemodes/miniantags/revenant/revenant_abilities.dm:159: Для улучшения поддержки кода рекомендуется вынести текстовую строку в константу.
Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.
Спасибо за ваш вклад в проект!
| log_say("(REVENANT to [key_name(M)]) [msg]", user) | ||
| to_chat(user, "[span_revenboldnotice("Вы передаёте [M]:")] [span_revennotice(msg)]") | ||
| to_chat(M, "[span_revenboldnotice("Голос из ниоткуда раздаётся вокруг...")] [span_italics(msg)]") | ||
| M.balloon_alert(M, "голос в голове...") |
There was a problem hiding this comment.
🟡 Рекомендация по улучшению читаемости и поддержки кода
В соответствии с лучшими практиками, для улучшения читаемости и поддержки кода рекомендуется выносить строковые литералы в именованные константы. Это упрощает их дальнейшее изменение и переиспользование.
Предлагаю определить константу в начале файла:
#define REVENANT_WHISPER_BALLOON_TEXT "голос в голове..."И затем использовать её в коде.
M.balloon_alert(M, REVENANT_WHISPER_BALLOON_TEXT)
References
- Стайлгайд (пункт 23) рекомендует придерживаться лучших практик для улучшения читаемости и поддержки кода. Использование именованных констант вместо 'магических строк' является одной из таких практик. (link)
1 participant
Что этот ПР делает
Тот, кому шепчет ревенант, получает баллун о том, что он слышит голос... в голове...
Почему это хорошо для игры
Теперь игрок хоть как то понимает, что получил сообщение от ревенанта
Демонстрация изменений
Демонстрации изменений
Тестирование
Проверял на локалке