Система постеров для революции #3021

cyberalexix · 2020-07-30T18:30:53Z

Собственно, постеры для революции.
Имеют уникальный спрайт на стене(выглядит не очень, но в ишуе всё равно о спрайте не говорилось).

Вообще, лучше бы создать отдельную ветку для реворка революции.

Каждой area присвоены очки(сейчас во всех зонах score равняется 1, кроме мостика, для него - 4), которые начисляются каждую минуту, если в них есть революционные постеры. Это лучше позже поправить, задав разное количество очков для разных зон.

Со старта раунда с гейммодом революция, создаётся глобальный пустой список revolution_posters для хранения ссылок на развешенные революционные постеры. Хед ревам с раундстарта выдаётся по два постера(для теста).
Сами свернутые революционные постеры наследуют /obj/item/weapon/contraband/poster и находятся, как это можно было понять ранее, под путём /obj/item/weapon/contraband/poster/revolutionary.
Свернутый постер нельзя повесить на z-уровнях отличающихся от станционных, а также в космосе и в area'х, в которых уже есть повешенный революционный постер.
После того, как вешается постер, создаётся /obj/structure/sign/poster и добавляется в глобальный список revolution_posters. Если его снять - он удаляет из этого же списка.
Каждую минуту раунда происходит цикл, который проходится по всем постерам в глобальном списке revolution_posters. За каждый постер начисляется сумма очков соответствующая значению в переменной score, которая находится в area постера. Если каким-то образом постер перестаёт существовать или вовсе пропадает из станции - он удаляется из глобального списка.

Если будут какие-то вопросы, пожелания, предложения или проблемы - пишите, исправлю.

Вот пример работы. Первое число - время с начала раунда, второе - время, когда пора проходиться по списку revolution_posters. tick означает, что прошел тик.

close #975

Pull Request полностью завершен, мне не нужна помощь чтобы его закончить.
Я внимательно прочитал все свои изменения и багов в них не нашел.
Я запускал сервер со своими изменениями локально и все протестировал.
Я ознакомился c Guide to Contribute.

MrMagick2104 · 2020-07-30T18:50:59Z

Фича крутая, но постер бы сам поярче сделать. Чтобы заметить проще было.
У нас, вроде, был красный постер в виде кулака с подписью REVOLUTION, где-то?

ArmolitskiyCarpoeb · 2020-07-30T21:58:00Z

Что за очки и зачем они нужны?

cyberalexix · 2020-07-30T22:07:31Z

Что за очки и зачем они нужны?

В ишуе сказано:

Добавить возможность захватывать области карты путем расклейки революционных постеров. Чем важнее зона, тем больше условных очков получают ревы. Нужно придумать к чему прикрутить очки и вынести в отдельные issues.

Скорее всего, позже кто-нибудь придумает какой-то аплинк для хед ревов, в котором можно будет покупать вещи за эти очки. Так как этого нет в ишуе, это не касается данного пулл реквеста, поэтому я и указал в ОПе, что неплохо было бы создать отдельную ветку для реворка революции.
У меня была ещё идея насчёт Уровня сознательности, который бы увеличивал урон исходящий от ревов и добавлял
бы новые возможности хедревам.

anihillator · 2020-07-30T22:13:37Z

Выдайте кабинетам глав по 2 поинта мб? Либо бригу, как вариант.

cyberalexix · 2020-07-30T22:15:59Z

Выдайте кабинетам глав по 2 поинта мб? Либо бригу, как вариант.

Честно говоря, не знаю как геймдизайнеры/кодеры/кто-то ещё отреагируют на моё распределение очков, поэтому я и поставил только мостику количество скора больше, чем 1, и то, только для того, чтобы протестить правильную работу кода.

Pos13 · 2020-07-31T13:06:42Z

А зачем вообще хранить все постеры в глобальном списке, да ещё и проверять их каждую минуту? Просто каждый раз при размещении постера points увеличивается, и при снятии или уничтожении (в Destroy()) уменьшается.

cyberalexix · 2020-07-31T14:24:40Z

А зачем вообще хранить все постеры в глобальном списке, да ещё и проверять их каждую минуту? Просто каждый раз при размещении постера points увеличивается, и при снятии или уничтожении (в Destroy()) уменьшается.

При тесте, случилась интересная вещь.
К примеру, если поставить постер на втором з уровне станции и дропнуть бомбу прямо на постер, он телепортируется в карманное измерение на координаты 0, 0, 0 и не уничтожится. Ex_act я не решился прописать, так как боялся что-то задеть или нарушить. К тому же, спрайтов оврелеев повреждения постеров у меня нет, а просто удалять постер или оставлять от него спрайт содранного постера - не очень хорошо.
К тому же, если удалить стену под постером, то он просто провалится в опен спейс. Мы не можем знать заранее в момент написания кода что будет находится в этом опенспейсе - космос либо помещение, станция либо какой-то другой объект. Так что поэтому я и выбрал именно такой подход.

А ещё - не думаю, что сервер сильно напрягается во время проверки постеров. Функцию считаю не лишней.

Но, если при даже самом малейшем взрыве вас устроит полное удаление постера либо что-то подобное - могу дописать Ex_act и Destroy методы.

PlinHost · 2020-08-01T20:43:06Z

code/_helpers/global_access.dm

+		if("revolution_posters")
+			return GLOB.revolution_posters;


Нафиг это нужно?

Нужно для отображения в списке глобальных переменных, который находится в вкладке Debug. Делал для тестов, оставил на всякий случай.

PlinHost · 2020-08-01T20:47:48Z

code/game/antagonist/station/revolutionary.dm

+/datum/antagonist/revolutionary/proc/equip_poster(mob/living/carbon/human/player)
+
+	var/obj/item/weapon/contraband/poster/revolutionary/P = new(get_turf(player))
+	var/list/slots = list (
+		"backpack" = slot_in_backpack,
+		"left pocket" = slot_l_store,
+		"right pocket" = slot_r_store,
+		"left hand" = slot_l_hand,
+		"right hand" = slot_r_hand,
+	)
+	for(var/slot in slots)
+		player.equip_to_slot(P, slot)
+		if(P.loc == player)
+			break
+	var/obj/item/weapon/storage/S = locate() in player.contents
+	if(istype(S))
+		P.forceMove(S)


Не городи велосипеды, используй хелперы из inventory.dm. Например, equip_to_appropriate_slot и equip_to_storage_or_drop.

Спасибо, сейчас исправлю.

Заменил на equip_to_storage_or_drop, изменения загружены

Epicus7 · 2020-08-02T18:08:41Z

code/game/gamemodes/revolution/revolution.dm

+/datum/game_mode/revolution/process()
+	if(last_poster_update <= round_duration_in_ticks)


Suggested change

/datum/game_mode/revolution/process()

if(last_poster_update <= round_duration_in_ticks)

/datum/game_mode/revolution/process()

..()

if(last_poster_update <= round_duration_in_ticks)

Epicus7 · 2020-08-02T18:12:34Z

code/game/objects/effects/decals/contraband.dm

@@ -13,6 +13,7 @@
 	desc = "The poster comes with its own automatic adhesive mechanism, for easy pinning to any vertical surface."
 	icon_state = "rolled_poster"
 	var/serial_number = 0
+	var/revolution = FALSE


Здесь лучше переменную назвать не "революция", а "революционный", т.е. revolutionary

Epicus7 · 2020-08-02T18:13:29Z

code/game/objects/effects/decals/contraband.dm

+	if(istype(B, /area/space))
+		to_chat(user, SPAN_WARNING("You can't place this poster in space."))
+		return
+
+	if(!(user.z in GLOB.using_map.station_levels))
+		to_chat(user, SPAN_WARNING("You can place this poster only on station levels,"))
+		return


Почему? Пусть висят где угодно, но считать будем только размещенные в правильных зонах, не? Это как-то нелогично, что кусок бумаги мы повесить не можем где хотим.

Epicus7 · 2020-08-02T18:15:56Z

code/game/objects/effects/decals/contraband.dm

+	for(var/obj/structure/P in GLOB.revolution_posters)
+		if(istype(B, get_area(P.loc)))
+			to_chat(user, SPAN_WARNING("A poster has been placed in this area already."))
+			return


Тоже какие-то странные механоограничения вылазят. Пусть обклеивают отсек полностью, чо такого то? Просто когда считаем - надо собирать некий список зон, в который каждая зона может попасть ровно один раз. А потом по этому списку считаем очки.

Так логичнее будет, тебе так не кажется?

Epicus7 · 2020-08-02T18:16:15Z

code/game/objects/effects/decals/contraband.dm

@@ -85,16 +112,22 @@
 	var/serial_number	//Will hold the value of src.loc if nobody initialises it
 	var/poster_type		//So mappers can specify a desired poster
 	var/ruined = 0
+	var/revolutionary = FALSE


О, а здесь нормальная переменная :)

Epicus7 · 2020-08-02T18:19:00Z

code/game/objects/effects/decals/contraband.dm

+			if(revolutionary)
+				GLOB.revolution_posters -= src


Это лучше запихать в roll_and_drop

Epicus7 · 2020-08-02T18:22:00Z

code/game/area/areas.dm

@@ -7,6 +7,7 @@
 	var/global/global_uid = 0
 	var/uid
 	var/area_flags
+	var/score = 1


Лучше переименовать в strategic_importance. А то слово score ни о чем не говорит.

Epicus7 · 2020-08-02T18:24:17Z

code/game/gamemodes/revolution/revolution.dm

+				points += get_area(P.loc).score
+			else
+				GLOB.revolution_posters -= P
+		last_poster_update = round_duration_in_ticks + 1 MINUTES


Ты здесь складываешь ТИКИ и децисекунды. Нельзя так делать)

Нужно либо все считать в тиках, либо все в децисекундах.

Epicus7 · 2020-08-02T18:26:11Z

code/game/objects/effects/decals/contraband.dm

+	revolution = TRUE
+
+/obj/item/weapon/contraband/poster/revolutionary/New(turf/loc)
+	..(loc, serial_number=666)


В poster/New нет аргумента serial_number. Кажется он тут вообще не нужен.

Зачем вообще New переопределять здесь? Кажется это лишнее.

Epicus7 · 2020-08-02T18:28:50Z

code/game/antagonist/station/revolutionary.dm

+/datum/antagonist/revolutionary/equip(mob/living/carbon/human/revolutionary)
+	if(!..())
+		return 0
+
+	equip_poster(revolutionary)
+	equip_poster(revolutionary)
+
+/datum/antagonist/revolutionary/proc/equip_poster(mob/living/carbon/human/player)
+	var/obj/item/weapon/contraband/poster/revolutionary/P = new(get_turf(player))
+	player.equip_to_storage(P)


Давай это пока спилим, ибо сами постеры по себе бесполезные сейчас. Думаю их пока не стоит вводить в игру.

TobyThorne · 2020-12-14T10:16:52Z

Больше трёх месяцев ожидает автора.

Система постеров для революции

9f07460

cyberalexix requested a review from a team as a code owner July 30, 2020 18:30

Epicus7 added the 🔺 приоритет Нужно решить в первую очередь! label Jul 31, 2020

PlinHost reviewed Aug 1, 2020

View reviewed changes

cyberalexix added 3 commits August 2, 2020 01:14

Заменил кусок кода из 14 строк на одно команду

496120a

Merge remote-tracking branch 'upstream/dev' into revolution

31b1ce0

Фикс конфликта

f2661c3

Epicus7 requested changes Aug 2, 2020

View reviewed changes

TobyThorne closed this Dec 14, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Система постеров для революции #3021

Система постеров для революции #3021

cyberalexix commented Jul 30, 2020 •

edited

Loading

MrMagick2104 commented Jul 30, 2020

ArmolitskiyCarpoeb commented Jul 30, 2020

cyberalexix commented Jul 30, 2020 •

edited

Loading

anihillator commented Jul 30, 2020

cyberalexix commented Jul 30, 2020

Pos13 commented Jul 31, 2020

cyberalexix commented Jul 31, 2020 •

edited

Loading

PlinHost Aug 1, 2020

cyberalexix Aug 1, 2020

PlinHost Aug 1, 2020

cyberalexix Aug 1, 2020

cyberalexix Aug 1, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

Epicus7 Aug 2, 2020

TobyThorne commented Dec 14, 2020

		/datum/game_mode/revolution/process()
		if(last_poster_update <= round_duration_in_ticks)

Система постеров для революции #3021

Система постеров для революции #3021

Conversation

cyberalexix commented Jul 30, 2020 • edited Loading

MrMagick2104 commented Jul 30, 2020

ArmolitskiyCarpoeb commented Jul 30, 2020

cyberalexix commented Jul 30, 2020 • edited Loading

anihillator commented Jul 30, 2020

cyberalexix commented Jul 30, 2020

Pos13 commented Jul 31, 2020

cyberalexix commented Jul 31, 2020 • 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

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

TobyThorne commented Dec 14, 2020

cyberalexix commented Jul 30, 2020 •

edited

Loading

cyberalexix commented Jul 30, 2020 •

edited

Loading

cyberalexix commented Jul 31, 2020 •

edited

Loading