add/refactor: Пластитаниумные стены by Ssheya · Pull Request #8477 · ss220-space/Paradise

Ssheya · 2026-01-28T20:27:54Z

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

Меняет пластитаниумную стену на подтип усиленной стены, добавляет ей свой гирдер, меняет характеристики
Чутка рефакторит код усиленной стены, чтобы легче сделать пластитаниумную
Меняет в куче мест код пластитановой стены и самого пластитана

Почему это хорошо для игры

Очень давно была предложка, прошла, я не сделал, сделал щас, чутка поковырял дополнительно цифры, думаю всё норм

Демонстрация изменений

Демонстрации изменений

Тестирование

На локалке протестировал, поспавнил их, сделал фейковую, разобрал, потыкал.

gemini-code-assist

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена.

Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит требующей доработок.

Сводка анализа:

🔴 Критических замечаний: 0
🟡 Рекомендаций: 5
💡 Предложений: 0

🟡 Рекомендации по улучшению:

.vscode/settings.json:43: Обнаружен локальный путь к файлу в настройках проекта. Такие файлы не должны попадать в репозиторий.
code/game/objects/structures/girders.dm:296: Обнаружена опечатка в сообщении для игрока. Вместо «plasteel» должно быть «plastitanium».
code/game/objects/structures/girders.dm:277: Выявлено дублирование кода при обработке листов пластитаниума, аналогичное логике для пластали. Рекомендуется рефакторинг для избежания повторений.
code/game/turfs/simulated/walls_reinforced.dm:314: Обнаружена проблема с балансом: при разрушении пластитаниумной стены выпадает больше ресурсов, чем требуется для её постройки, что может быть использовано для их дублирования.
code/game/turfs/simulated/walls_reinforced.dm:351: Обнаружено дублирование кода в процедуре copyTurf, на что указывает комментарий //have to copypaste this code. Рекомендуется найти способ избежать этого повторения.

Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.

Спасибо за ваш вклад в проект!

I am having trouble creating individual review comments. Click here to see my feedback.

code/game/turfs/simulated/walls_reinforced.dm (314-316)

🟡 Рекомендация по балансу

Процедура devastate_wall() для пластитаниумной стены возвращает больше ресурсов, чем требуется для её постройки. Она создаёт sheet_amount (наследуется как 4) листов, а затем ещё 2, что в сумме даёт 6 листов пластитаниума. При этом для постройки стены на пластитаниумной балке требуется всего 2 листа. Это создаёт возможность для бесконечного получения ресурсов.

Рекомендуется скорректировать количество возвращаемых ресурсов. Вероятно, стоит переопределить sheet_amount для этого типа стены на 2 и убрать дополнительное создание листов в devastate_wall().

/turf/simulated/wall/r_wall/plastitanium/devastate_wall()
	new sheet_type(src, 2) // Или другое сбалансированное значение

.vscode/settings.json (43-45)

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

Этот файл содержит специфичные для пользователя настройки, включая абсолютный путь к локальным файлам (C:\Users\Администратор\...). Подобные конфигурационные файлы не должны попадать в общий репозиторий, так как они могут вызвать проблемы у других разработчиков или перезаписать их локальные настройки. Рекомендуется добавить .vscode/settings.json в файл .gitignore, чтобы избежать подобных ситуаций в будущем.

code/game/objects/structures/girders.dm (296)

🟡 Рекомендация по улучшению

В сообщении для игрока допущена опечатка: упоминается «plasteel» вместо «plastitanium». Это может ввести в заблуждение. Рекомендую исправить текст для соответствия используемому материалу.

					to_chat(user, span_warning("You need at least two sheets of plastitanium to finalize the plastitanium wall!"))

code/game/objects/structures/girders.dm (277-322)

🟡 Рекомендация по улучшению кода

Логика обработки листов пластитаниума практически полностью дублирует существующую логику для пластали. Это нарушает принцип DRY (Don't Repeat Yourself) и усложняет дальнейшее сопровождение кода.

В соответствии со стайлгайдом (правило 67), рекомендуется вынести повторяющийся код в отдельную, более универсальную процедуру, которая могла бы принимать тип материала и соответствующие ему типы стены/балки в качестве аргументов. Это сделает код чище и проще для модификации в будущем.

code/game/turfs/simulated/walls_reinforced.dm (351-370)

🟡 Рекомендация по улучшению кода

Комментарий //have to copypaste this code прямо указывает на дублирование кода в процедуре copyTurf. Согласно стайлгайду (правило 67), дублирование следует избегать.

Рекомендуется рассмотреть возможность вынесения этой логики в родительский класс /turf/simulated/wall/r_wall или найти другой способ избежать копирования, чтобы улучшить поддерживаемость кода.

Pelmenik123 · 2026-01-29T07:07:40Z

/gemini review

gemini-code-assist

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена.

Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит отлично. Выполнен масштабный и качественный рефакторинг пластитаниумных стен, который унифицирует их с усиленными стенами. Это значительное улучшение архитектуры и поддерживаемости кода.

Сводка анализа:

🔴 Критических замечаний: 0
🟡 Рекомендаций: 1
💡 Предложений: 0

🟡 Рекомендации по улучшению:

code/game/objects/structures/girders.dm:277-322: Обнаружено дублирование кода при проверке количества листов пластитаниума. Рекомендуется вынести проверку перед блоком switch для улучшения читаемости и упрощения поддержки.

Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.

Спасибо за ваш вклад в проект!

code/game/objects/structures/girders.dm

Pelmenik123 · 2026-01-29T10:03:58Z

/gemini review

gemini-code-assist

Code Review

Аналитический отчёт от Большого Брата. Обработка запроса завершена.

Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа выглядит удовлетворительно.

Сводка анализа:

🔴 Критических замечаний: 0
🟡 Рекомендаций: 1
💡 Предложений: 0

🟡 Рекомендации по улучшению:

code/game/turfs/simulated/walls_reinforced.dm:298: Обнаружена логическая несостыковка при починке пластитановых стен. Текущая реализация предполагает использование обычных листов металла, что не соответствует материалу стены. Рекомендуется переопределить логику починки для использования пластитана.

Пожалуйста, ознакомьтесь с комментариями к отдельным строкам кода для получения более подробной информации.

Спасибо за ваш вклад в проект!

code/game/turfs/simulated/walls_reinforced.dm

dageavtobusnick · 2026-03-01T15:23:59Z

code/game/objects/structures/false_walls.dm

+	smoothing_groups = SMOOTH_GROUP_PLASTITANIUM_WALLS
+
+/obj/structure/falsewall/reinforced/plastitanium/ChangeToWall(delete = 1)
+	var/turf/T = get_turf(src)


имя переменной

dageavtobusnick · 2026-03-01T15:24:23Z

code/game/objects/structures/girders.dm

+				transfer_fingerprints_to(girder)
+				girder.add_fingerprint(user)
+				qdel(src)
+				return ATTACK_CHAIN_BLOCKED_ALL


В отдельный прок

dageavtobusnick · 2026-03-01T15:25:02Z

code/game/objects/structures/girders.dm

+		switch(state)
+			if(GIRDER_DISPLACED)
+				if(plastitanium.get_amount() < 2)
+					to_chat(user, span_warning("You need at least two sheets of plastitanium to create a false wall!"))


текст не помешает перевести, но это не критично

dageavtobusnick · 2026-03-01T15:25:26Z

code/game/objects/structures/girders.dm

 				qdel(src)
 				return ATTACK_CHAIN_BLOCKED_ALL

+	if(istype(sheet, /obj/item/stack/sheet/plastitanium))


изхелпер и заменить на него все istype(переменная, /obj/item/stack/sheet/plastitanium)

dageavtobusnick · 2026-03-01T15:25:49Z

code/game/turfs/simulated/walls_reinforced.dm

 	var/d_state = RWALL_INTACT
 	var/can_be_reinforced = 1

+/turf/simulated/wall/r_wall/proc/metal_repair(obj/item/I, mob/user)


имя переменной

dageavtobusnick · 2026-03-01T15:26:02Z

code/game/turfs/simulated/walls_reinforced.dm

+	to_chat(user, span_notice("You repair the last of the damage."))
+	return ATTACK_CHAIN_SUCCESS
+
+/turf/simulated/wall/r_wall/proc/coating(obj/item/I, mob/user)


dageavtobusnick · 2026-03-01T15:28:24Z

code/game/turfs/simulated/walls_reinforced.dm

+		icon_state = "plastitanium_wall-d-[d_state]"
+		smooth = NONE
+		clear_smooth_overlays()
+	else


ранний ретурн

dageavtobusnick · 2026-03-01T15:29:07Z

code/game/objects/structures/girders.dm

+					to_chat(user, span_warning("You need at least two sheets of plastitanium to reinforce the girder!"))
+					return .
+				to_chat(user, span_notice("You start reinforcing the girder..."))
+				if(!do_after(user, 6 SECONDS * plastitanium.toolspeed, src, category = DA_CAT_TOOL) || state == GIRDER_DISPLACED || state == GIRDER_REINF || QDELETED(plastitanium) || !plastitanium.use(2))


Что значит 2? нужно дефайнизировать

dageavtobusnick · 2026-03-01T15:30:49Z

code/game/objects/structures/girders.dm

+				return ATTACK_CHAIN_BLOCKED_ALL
+
+			if(GIRDER_REINF)
+				if(plastitanium.get_amount() < 2)


скорее всего это должен быть тот же дефайн

Ssheya added 3 commits January 28, 2026 21:46

da

5d15430

da2

d969b53

da3

c802dd5

Ssheya requested a review from a team as a code owner January 28, 2026 20:27

Ssheya added 2 commits January 28, 2026 23:28

da

6bb19c0

Merge branch 'master220' into walls

99fd49e

gemini-code-assist bot reviewed Jan 28, 2026

View reviewed changes

Ssheya added 3 commits January 28, 2026 23:34

da4

72e30b4

Merge branch 'walls' of https://github.com/Ssheya/Paradise into walls

9bae00a

da5

f14e826

gemini-code-assist bot reviewed Jan 29, 2026

View reviewed changes

code/game/objects/structures/girders.dm Show resolved Hide resolved

Ssheya added 2 commits January 29, 2026 11:43

da6

b672f64

da7

e7c4fb6

gemini-code-assist bot reviewed Jan 29, 2026

View reviewed changes

code/game/turfs/simulated/walls_reinforced.dm Show resolved Hide resolved

Ssheya changed the title ~~Пластитаниумные стены~~ add: Пластитаниумные стены Jan 29, 2026

Ssheya changed the title ~~add: Пластитаниумные стены~~ add/refactor: Пластитаниумные стены Jan 29, 2026

Merge branch 'master220' into walls

8b52a63

dageavtobusnick reviewed Mar 1, 2026

View reviewed changes

Conversation

Ssheya commented Jan 28, 2026

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

Почему это хорошо для игры

Демонстрация изменений

Тестирование

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

code/game/turfs/simulated/walls_reinforced.dm (314-316)

.vscode/settings.json (43-45)

code/game/objects/structures/girders.dm (296)

code/game/objects/structures/girders.dm (277-322)

code/game/turfs/simulated/walls_reinforced.dm (351-370)

Uh oh!

Pelmenik123 commented Jan 29, 2026

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Pelmenik123 commented Jan 29, 2026

Uh oh!

gemini-code-assist bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants