add: [bounty] death book #6770

Pelmenik123 · 2025-04-08T21:58:58Z

Описание

Добавляет предмет death_book. Добавляет компонент phantom_component, он нужен чтобы выпиливать из жизни любой предмет по сигналу. dynamic_outfit, точно такой же outfit только хранит уже готовые объекты и обладает спец функционалом. Наследник от outfit - radial_outfit будет полезен для радиальных менюшек.

Причина создания ПР / Почему это хорошо для игры

https://discord.com/channels/617003227182792704/755125334097133628/1350044689989894175

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

Прикольный предмет для антага, не оставляет улик.

Тесты

Работает хорошо, плохо не работает

…into master220

Co-authored-by: Den109G <[email protected]>

Курва апдейт

Pelmenik123

ыфвфывыф

…nto bounty-knigga

Pelmenik123 · 2025-05-16T17:24:17Z

@dageavtobusnick я так и не понял где этот ранний ретурн ставить и зачем

dageavtobusnick · 2025-05-16T17:31:53Z

@dageavtobusnick я так и не понял где этот ранний ретурн ставить и зачем

Я тебе уже 100 раз сказал. Для читаемости. Когда весь смысловой код на одинаковых отступах его проще читать.

Pelmenik123 · 2025-05-16T17:35:13Z

да я не знаю где его впихнутьь для читаемости нужно

littleboobs · 2025-05-16T19:55:21Z

Плохой мистер Пельмень, что насчёт остального ревью?
Про баллуны на 20 слов я не верю. Показывай пруфы или сокращай, ты пытаешься меня обмануть.
Половину ревью ты тупо проигнорил или скипнул. Исправляйся!

Co-authored-by: littleboobs <[email protected]>

Pelmenik123 · 2025-05-16T20:01:25Z

баллуны видно нормально

Co-authored-by: littleboobs <[email protected]>

littleboobs · 2025-05-16T20:07:40Z

Не может быть, чтобы такой балун было видно нормально.
user.balloon_alert(user, "Вас наполняет решимость. Вы готовы прочесть книгу вновь.") и такой user.balloon_alert(user, "Ваша экипировка теряет очертания медленно растворяясь в воздухе"). Их смысл в лаконичности, информативности и быстроте. Паста из двух предложений, а уж тем более с большой буквы и точками - точно не соответствует всему раннее перечисленному.
Если на то пошло, то прочти данный содержательный скриншот от 29.10.24 Вани Пилюгана.

Co-authored-by: littleboobs <[email protected]>

…es.dm Co-authored-by: littleboobs <[email protected]>

Co-authored-by: littleboobs <[email protected]>

code/game/gamemodes/wizard/death_book/death_book.dm

dageavtobusnick

Вложенность кода ухудшает его читаемость. И не надо закрывать conversation, если ты не исправил то, что написано в нем.

dageavtobusnick · 2025-05-16T21:51:20Z

code/datums/diseases/viruses/transformation.dm

+		prom.mind.objectives += new /datum/objective/xeno_genocide
+		var/list/messages = prom.mind.prepare_announce_objectives()
+		to_chat(prom, chat_box_red(messages.Join("<br>")))
+	return prom


Тут нужно сделать ранний ретурн.

dageavtobusnick · 2025-05-16T21:55:22Z

code/game/gamemodes/wizard/death_book/death_book.dm

+		var/datum/outfit/radial_outfit/death_book/prom_obj = new prom_outfit()
+		if(prom_obj.can_choise(user))
+			desc_to_outfit[prom_obj.descr] = prom_obj
+			radial_look[prom_obj.descr] = prom_obj.get_image()


Тут ранний continue. Вложенность кода ухудшает его читаемость.

dageavtobusnick · 2025-05-16T21:56:46Z

code/game/gamemodes/wizard/death_book/death_outfits.dm

+	if(H.dna.species.name == SPECIES_VOX)
+		var/obj/item/tank/internals/oxygen/red/prom = H.s_store
+		prom.air_contents.oxygen = 0
+		prom.air_contents.nitrogen = (6 * ONE_ATMOSPHERE) * prom.volume / (R_IDEAL_GAS_EQUATION * T20C)


Тут ранний ретурн, если условие не выполняется. Можно вообще стелать if(H.dna.species.name != SPECIES_VOX) return, а дальше то, что делается в случае выполнения без отступа.

dageavtobusnick · 2025-05-16T21:59:21Z

code/modules/antagonists/terror_spiders/terror_spider_objectives.dm

+	if(!length(possible_spiders))
+		explanation_text = "Вы остались без стаи и единой цели. Но вы знаете что вы созданы убивать и сеять хаос."
+	else
+		explanation_text = "Помогите вашему гнезду отложить яйцо Императрицы Ужаса. Это могут сделать: [possible_spiders.Join(", ")]. Защищайте их и помогите им набрать силу, чтобы они смогли это сделать."


Тут без if Можно. используй:
(length(possible_spiders))? Помогите вашему гнезду отложить яйцо Императрицы Ужаса. Это могут сделать: [possible_spiders.Join(", ")]. Защищайте их и помогите им набрать силу, чтобы они смогли это сделать." : "Вы остались без стаи и единой цели. Но вы знаете что вы созданы убивать и сеять хаос."

тринарные операторы еще хуже воспринимаются

Хмм. Вообще да. Перенеси текст в дефайны, тогда будет нормально восприниматься.

Тот, что с possible_spiders, через макрос.

ооо май гад это что подготовка к двум языкам???!?!

dageavtobusnick · 2025-05-16T22:00:29Z

code/modules/reagents/chemistry/reagents/disease.dm

+	if(volume >= 1)
+		var/datum/disease/virus/transformation/xeno/phantom/D = new
+		D.Contract(M)
+	return ..()


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

какой ранний ретурн, там же всеравно в ..() упрется

code/game/gamemodes/wizard/death_book/death_book.dm

dageavtobusnick · 2025-05-16T22:38:14Z

code/game/gamemodes/wizard/death_book/components/phantom_component.dm

+		return COMPONENT_INCOMPATIBLE
+	if(deep)
+		var/atom/prom = parent
+		phantomification(prom.contents)


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

dageavtobusnick · 2025-05-16T22:45:15Z

code/game/gamemodes/wizard/death_book/death_book.dm

+	var/list/desc_to_outfit = list()
+	for(var/prom_outfit in subtypesof(/datum/outfit/radial_outfit))
+		var/datum/outfit/radial_outfit/death_book/prom_obj = new prom_outfit()
+		if(prom_obj.can_choise(user))


Не нужно каждый раз искать subtypesof и создавать новые датумы. По идее достаточно будет единожды сделать это при инициализации и закешировать в статику для всех /obj/item/death_book.

сделал кеширование при инициализации

dageavtobusnick · 2025-05-16T22:55:37Z

code/modules/reagents/chemistry/reagents/disease.dm

+	metabolization_rate = 1
+
+/datum/reagent/terror_eggs/on_mob_life(mob/living/carbon/M)
+	if(volume >= 5 && !M.get_int_organ(/obj/item/organ/internal/body_egg/terror_eggs/phantom))


Для сравнения с volume числа перенеси в дефайны. Магические числа - плохо.

И тут и в /datum/reagent/xenomicrobes/phantom

так тут же volume количество, количество реагента

сделано

Pelmenik123 and others added 12 commits February 19, 2025 15:25

fix: Оптимизация оповещений ПДА

c160092

Merge branch 'master220' of https://github.com/Pelmenik123/Pelmen-prs …

a21ea80

…into master220

мда

f248958

Update code/modules/pda/PDA.dm

989bb10

Co-authored-by: Den109G <[email protected]>

Update code/modules/pda/PDA.dm

955eda3

Co-authored-by: Den109G <[email protected]>

Update code/modules/pda/PDA.dm

402f649

Co-authored-by: Den109G <[email protected]>

Merge branch 'ss220-space:master220' into master220

86911ee

Update Accounts.dm

4ba91e1

Курва апдейт

Merge branch 'ss220-space:master220' into master220

1f2335c

промежуточные изменения

71e7bc4

промежудочный изменения

21e776f

уэээ еще чуть чуть

da35315

github-actions bot added Merge Conflict This doesn't seem right DMI Affects DMI files and removed Merge Conflict This doesn't seem right labels Apr 8, 2025

Pelmenik123 changed the title ~~Bounty knigga~~ [Bounty] add: death knigga Apr 9, 2025

угар, вроде ничего не забыл

4926bf3

Pelmenik123 marked this pull request as ready for review April 9, 2025 12:17

Pelmenik123 added 3 commits April 9, 2025 15:18

упс

71676d4

Merge branch 'ss220-space:master220' into master220

dd36cdb

fix

4291153

Pelmenik123 changed the title ~~[Bounty] add: death knigga~~ [Bounty] add: death book Apr 9, 2025

Pelmenik123 changed the title ~~[Bounty] add: death book~~ [bounty] add: death book Apr 9, 2025

Pelmenik123 changed the title ~~[bounty] add: death book~~ [bounty] add: death knigga Apr 9, 2025

Pelmenik123 changed the title ~~[bounty] add: death knigga~~ [bounty] add: death book Apr 9, 2025

Pelmenik123 commented Apr 9, 2025

View reviewed changes

Merge branch 'master220' of https://github.com/ss220-space/Paradise i…

76d8f9e

…nto bounty-knigga

github-actions bot removed the Merge Conflict This doesn't seem right label Apr 9, 2025

ыфвфвыфвы

24df218

ё АПДЕЙТ, Б

72d6118

Pelmenik123 requested a review from littleboobs May 16, 2025 17:23

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

d275a04

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

f67a3f4

Co-authored-by: littleboobs <[email protected]>

Pelmenik123 and others added 12 commits May 16, 2025 23:26

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

668a1a1

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

6c4c9b7

Co-authored-by: littleboobs <[email protected]>

Update code/modules/reagents/chemistry/reagents/disease.dm

c32e483

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

0a11e16

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

0e21830

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

5ac6e2c

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

a5e4cf4

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_book.dm

d400d9a

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

41ad5b5

Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

c1537dc

Co-authored-by: littleboobs <[email protected]>

Update code/modules/antagonists/terror_spiders/terror_spider_objectiv…

588d1fc

…es.dm Co-authored-by: littleboobs <[email protected]>

Update code/game/gamemodes/wizard/death_book/death_outfits.dm

b5f0178

Co-authored-by: littleboobs <[email protected]>

dageavtobusnick requested changes May 16, 2025

View reviewed changes

code/game/gamemodes/wizard/death_book/death_book.dm Outdated Show resolved Hide resolved

у меня 3 по русскому не бить палками

4fea4b5

Pelmenik123 requested a review from dageavtobusnick May 16, 2025 21:03

Merge branch 'master220' into bounty-knigga

b45bb56

dageavtobusnick requested changes May 16, 2025

View reviewed changes

dageavtobusnick reviewed May 16, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

add: [bounty] death book #6770

add: [bounty] death book #6770

Pelmenik123 commented Apr 8, 2025 •

edited

Loading

Pelmenik123 left a comment

Pelmenik123 commented May 16, 2025

dageavtobusnick commented May 16, 2025

Pelmenik123 commented May 16, 2025

littleboobs commented May 16, 2025

Pelmenik123 commented May 16, 2025

littleboobs commented May 16, 2025

dageavtobusnick left a comment

dageavtobusnick May 16, 2025

dageavtobusnick May 16, 2025

dageavtobusnick May 16, 2025

dageavtobusnick May 16, 2025

Pelmenik123 May 16, 2025

dageavtobusnick May 16, 2025

dageavtobusnick May 16, 2025 •

edited

Loading

Pelmenik123 May 17, 2025

dageavtobusnick May 16, 2025

Pelmenik123 May 17, 2025

dageavtobusnick May 16, 2025

Pelmenik123 May 17, 2025

dageavtobusnick May 16, 2025

Pelmenik123 May 17, 2025

dageavtobusnick May 16, 2025

dageavtobusnick May 16, 2025

Pelmenik123 May 17, 2025

Pelmenik123 May 17, 2025

add: [bounty] death book #6770

Are you sure you want to change the base?

add: [bounty] death book #6770

Conversation

Pelmenik123 commented Apr 8, 2025 • edited Loading

Описание

Причина создания ПР / Почему это хорошо для игры

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

Тесты

Pelmenik123 left a comment

Choose a reason for hiding this comment

Pelmenik123 commented May 16, 2025

dageavtobusnick commented May 16, 2025

Pelmenik123 commented May 16, 2025

littleboobs commented May 16, 2025

Pelmenik123 commented May 16, 2025

littleboobs commented May 16, 2025

dageavtobusnick left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

dageavtobusnick May 16, 2025 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Pelmenik123 commented Apr 8, 2025 •

edited

Loading

dageavtobusnick May 16, 2025 •

edited

Loading