local: Перевод бс сумок#8532
Hidden character warning
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
Аналитический отчёт от Большого Брата. Обработка запроса завершена. Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно. Обнаружена одна рекомендация по улучшению: в файле code/game/objects/items/weapons/storage/backpack.dm на строках 83-84 используется строковый литерал в условном операторе, что нарушает стайлгайд. Пожалуйста, ознакомьтесь с комментарием к коду для получения более подробной информации. Спасибо за ваш вклад в проект!
| var/response = tgui_alert(user, "Это создаст сингулярность, которая уничтожит вас и большую часть станции. Вы уверены?", "НЕИЗБЕЖНАЯ СМЕРТЬ!", list("Нет", "Да")) | ||
| if(response != "Да") |
There was a problem hiding this comment.
🟡 Рекомендация по стилю и согласованности
Использование строковых литералов, таких как "Да", в условных операторах делает код менее гибким для будущих изменений и локализации. Согласно стайлгайду (правило 4.3, пункт 140), для строк, участвующих в проверках, следует использовать именованные константы через #define.
Предлагаю определить константы для ответов в tgui_alert (например, в code/__DEFINES/tgui.dm) и использовать их:
// В code/__DEFINES/tgui.dm
#define TGUI_ALERT_RESPONSE_YES "Да"
#define TGUI_ALERT_RESPONSE_NO "Нет"Это улучшит читаемость и упростит поддержку кода.
var/response = tgui_alert(user, "Это создаст сингулярность, которая уничтожит вас и большую часть станции. Вы уверены?", "НЕИЗБЕЖНАЯ СМЕРТЬ!", list(TGUI_ALERT_RESPONSE_NO, TGUI_ALERT_RESPONSE_YES))
if(response != TGUI_ALERT_RESPONSE_YES)
References
- Для строк, участвующих в проверках (if, switch), следует использовать именованные константы, определённые через
#define, чтобы избежать использования «магических строк» и упростить поддержку кода. (link)
|
| /obj/item/storage/backpack/holding | ||
| name = "Bag of Holding" | ||
| desc = "A backpack that opens into a localized pocket of Blue Space." | ||
| desc = "Технологичный рюкзак, в сравнении с обычными, вмещает чрезмерное количество предметов благодаря подпространственной компрессии. Этикетка предупреждает \"Не совмещать с другими блюспейс технологиями\"." |
There was a problem hiding this comment.
Не совмещать с технологиями телепортации.
Телепорт у нас не только БС, а бс предметы можно носить с другими БС предметами
| ACCUSATIVE = "блюспейс сумку хранения", | ||
| INSTRUMENTAL = "блюспейс сумкой хранения", | ||
| PREPOSITIONAL = "блюспейс сумке хранения" | ||
| NOMINATIVE = "блюспейс спорт-сумка", |
There was a problem hiding this comment.
Спорт-сумка плохое название, или пши полностью спортивная сумка или придумывай другое название типа диагонального мешка или что там в дискорде предлагали
pilygun
added
the
📜 Локализация
pilygun
left a comment
pilygun
left a comment
There was a problem hiding this comment.
Смысловое содержание в порядке
pilygun
added
☑️ Локализация одобрена
|
Проверено и одобрено |
5 participants
Что этот ПР делает
Переводит названия и описания Bag Of Holding, Satchel Of Holding и Dufelbag Of Holding
Почему это хорошо для игры
Это хорошо потому что русский
Демонстрация изменений
Демонстрации изменений
Тестирование
Да