Sweeps and spins. #3755

LudwigVonChesterfield · 2019-08-06T08:24:36Z

Описание изменений

Теперь кроме как шваброй можно махать-пихать различными вещами, а так же:

Щитами можно махать, и если кто-то упрётся в твой машущийся щит своим махком, его застанит, опрокинет.

Если имея что-то, что может пихать ты ударишь по руке с щитом - ты можешь опрокинуть щитоносца.

Хоткеи такие же как у швабры, только вещи которые можно "крутить", если они имеют какое-либо взаимодействие при "использовании" крутить можно на среднее колёсико мышки.

По конкретным предметам:

Топором можно крутиться и махать, но он довольно медленный. Он упирается об щит, и стены.
Копьём можно пихать, и даже атаковать через один тайл(Или притягивать и атаковать).
Энерго-милишные оружия можно махать и крутить, и они не упираются ни об что кроме *столов, мусоро, полочек.
Двойную саблю можно крутить.

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

Откатывает #3974

Есть возможность эти все финты проворачивать с помощью мышки, если вы выбрали тайл рядом с вами, и начали его перетаскивать на тайл не рядом с вами - вы отпихнёте.

Если перетащили тайл не рядом с вами, на тот который рядом - притянете.

Если все "выделенные" вами тайлы будут рядом с вами - вы сделаете свайп по ним.

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

Почему и что этот ПР улучшит

Добавит чуть больше механа нашей боёвочке.

🆑 Luduk

rscadd[link]: Крутить, махать, пихать можно не только шваброй.
rscadd: Пихание щитом кого-то об стену - опрокинет их.
tweak: Швабра теперь засовывается в тележку не альт-кликом а перетаскиванием её на тележку.

TauKitty · 2019-08-06T08:24:37Z

Changelog status: ✔️

Now you can sweep and spin different weapons. Shields stun on sweep. Energy weapons don't care about hitting something, etc.

Snailcat · 2019-08-06T13:44:42Z

А лазерный двойной, когда крутишь, во все стороны снаряды отражает ?

LudwigVonChesterfield · 2019-08-06T19:09:31Z

Я в недоумении от этого вопроса.

SomeUselessGuy · 2019-08-06T19:32:17Z

А лазерный двойной, когда крутишь, во все стороны снаряды отражает ?

https://media.giphy.com/media/Ddab9zJPtaEmI/giphy.gif

LudwigVonChesterfield · 2019-08-06T19:33:11Z

Учитывая что я это нигде не упоминул в описании ПР-а, вероятно, нет.

SpaiR · 2019-08-08T20:04:27Z

code/game/objects/weapons.dm

 /obj/item/weapon
 	name = "weapon"
 	icon = 'icons/obj/weapons.dmi'

+	var/__interupt_on_sweep_hit_types = list(/atom) // By default we interupt on any hit.


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

Ещё мне не нравится то что ты добавил инстанс список каждому инстансу от вепона, при этом подавляющее большинство объектов его никогда использовать не будут. Посмотри в сторону ленивых списков и ленивой инциализации в целом. За референсом подсмотри сюда tgstation/tgstation#21245

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

Посмотрю на ленивые листы, да.

У нас практикуют такое немного с переменными, но исключительно в аргументах чтобы не прибегать к src или другому наименованию. В переменных класса если уж очень хотелось как-то обозначить приватку (правда не ясно зачем все равно), то можно было специфический нейм префикс или суфикс сделать вида p_varname / private_varname.

SpaiR · 2019-08-08T20:08:16Z

code/game/objects/weapons.dm

+/obj/item/weapon/proc/on_sweep_push(atom/target, turf/T, mob/user)
+	return
+
+/obj/item/weapon/proc/on_sweep_push_success(atom/target, mob/user)


Может я чего и упускаю, но на первый взгляд on_sweep_push_success, как и on_sweep_pull_success, везде одинаковые. Где-то есть проверки на интент, но где они есть - там тоже всё идентично. У копья есть +2 строки, но остальное такое же. И то что ты везде, по факту, продублировал код - не выглядит хорошим дизайном.

Сделал переменную которая отличает копьё от остольного, убрал копипасту

SpaiR · 2019-08-08T20:15:34Z

code/game/objects/weapons.dm

+
+	var/__can_sweep = FALSE
+	var/__can_spin = FALSE
+	var/spin_on_middleclick = FALSE


Не совсем ясно, почему где-то мидлклик есть, а где-то нету?

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

Что, кстати, было описано в шапке ПР-а.

stale · 2019-09-19T12:41:39Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

stale · 2019-09-26T13:00:59Z

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

…ssic into spin_and_slash

…sult Weaken.

LudwigVonChesterfield · 2019-09-26T16:19:41Z

Сделал что просили, перенёс текущий механ щита на то что есть в этом ПР-е, по сути откатил #3974

Откройте пожалуйста

ZVee · 2019-09-26T17:32:11Z

ПР в нотисах засветился, поэтому решил заглянуть.
На мой взгляд тут компоненты былиб очень кстати к мидлклику, а куда сигнал пихать? Хммм, например в мидл клик там где смена рук.

LudwigVonChesterfield · 2019-09-26T17:45:16Z

@volas Сделал управление мышью c:

ZVee · 2019-09-26T17:48:27Z

code/game/objects/weapons.dm

+
+	var/list/turfs = getline(dropping_turf, target_turf)
+	var/list/directions = list()
+	world.log << "[turfs.len] [target.name] [dropping.name]"


ZVee · 2019-09-26T17:50:26Z

code/game/objects/weapons.dm

+
+/obj/item/weapon/attack(mob/living/M, mob/living/user, def_zone)
+	. = ..()
+	if(. && can_push())


Да, вот такие тоже моменты на компоненты в идеале надо закидывать

А можно поподробнее?

Вместо выдачи тонны переменных куда-то внизы и проков на низкие уровни, чтобы просто потом не тайпкастить и т.д (как это делали раньше часто, а сейчас так), можно выдавать компонент объектам нужным, и расставлять сигналы на те или иные действия куда надо. За подсказками можно вот к ТГ слазать, понять на что они в принципе бывают - https://github.com/tgstation/tgstation/blob/master/code/__DEFINES/components.dm#L64

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

…ssic into spin_and_slash

LudwigVonChesterfield · 2019-09-29T16:06:14Z

(в этом ПР-е поправил конфликты)

TauKitty · 2019-09-30T22:04:22Z

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/discussion-kombo-orientirovannaya-boyovka/12387/59

TauKitty · 2019-10-17T14:51:36Z

This pull request has been mentioned on Tau Ceti Station Forum. There might be relevant details there:

https://forum.taucetistation.org/t/discussion-kombo-orientirovannaya-boyovka/12387/102

ZVee · 2019-10-19T10:13:42Z

code/game/objects/weapons.dm

@@ -132,6 +132,7 @@
 			break

 	QDEL_IN(sweep_image, sweep_delay)
+	sweep_image = null


У меня коммит тут просто высветился, поэтому заглянул.

Не совсем понятно, это ведь локальная переменная, что ее нуль дает?

…d cooldown for mouse-pull and mouse-push.

…ssic into spin_and_slash

volas · 2019-11-02T08:19:49Z

Сделал управление мышью c:

Швабрам бы еще. Вообще было бы замечательно, будь отдельный ПР с управлением мышкой, включенным для швабр, и отдельно уже оружие.
Не известно, зайдет ли нам боевое махание, а вот швабры и управление к ним точно нужны.

LudwigVonChesterfield · 2019-11-02T08:44:19Z

У швабр механ боевого махания, все вещи которые «машутся» машутся в том числе мышкой

volas · 2019-11-02T08:42:13Z

code/game/objects/weapons.dm

+	var/spin_on_middleclick = FALSE
+	var/sweep_step = 4
+
+/obj/item/weapon/proc/can_push()


стена кода без существенных комментариев :/

Пояснения бы пригодились, как пример

TauCetiClassic/code/_onclick/drag_drop.dm

Line 2 in 7c882ad

MouseDrop:

volas · 2019-11-02T08:43:19Z

code/game/objects/weapons.dm

@@ -1,7 +1,345 @@
+/obj/effect/effect/weapon_sweep


Стоит свайп-специфичные вещи (это и определения can_* + сами проки) вынести в отдельный файл, что бы не складывать сюда в одну кучу.

И еще соглашусь по поводу компонентов, с ними всё было бы аккуратнее.

stale · 2019-12-02T09:53:33Z

Данный ПР автоматически отмечен как застоявшийся по причине длительного отсутствия обновлений. Он будет закрыт через 7 дней, если никакой активности не будет проявлено. Если вы считаете, что ПР еще актуален, или что я (злобный робот) пристаю к вам зря - просто напишите любой комментарий. Спасибо за ваш вклад.

stale · 2019-12-09T10:29:22Z

ПР закрыт из-за длительного отсуствия активности. Для переоткрытия ПРа, пожалуйста, обратитесь к кому-либо из мейнтейнеров. Вы можете призвать их в комментарии слапнув @TauCetiStation/maintainers.

LudwigVonChesterfield · 2020-01-06T17:19:25Z

Откройте ПР пожалуйста. Переделал всё на компоненты.

TauKitty added the Feature Новая фича label Aug 6, 2019

LudwigVonChesterfield force-pushed the spin_and_slash branch 6 times, most recently from ae23865 to 8ed6ec1 Compare August 6, 2019 12:07

Sweeps and spins.

6d903c5

Now you can sweep and spin different weapons. Shields stun on sweep. Energy weapons don't care about hitting something, etc.

LudwigVonChesterfield force-pushed the spin_and_slash branch from 8ed6ec1 to 6d903c5 Compare August 6, 2019 12:29

SpaiR requested changes Aug 8, 2019

View reviewed changes

stale bot added the Stalled PR label Sep 19, 2019

stale bot closed this Sep 26, 2019

LudwigVonChesterfield added 3 commits September 26, 2019 16:48

Merge branch 'master' of https://github.com/TauCetiStation/TauCetiCla…

7990a4d

…ssic into spin_and_slash

Weird syntax removal

3b4dcff

Making pushing and pulling more abstract, shield pushes into walls re…

f202fd0

…sult Weaken.

volas reopened this Sep 26, 2019

stale bot removed Stalled PR labels Sep 26, 2019

ZVee reviewed Sep 26, 2019

View reviewed changes

LudwigVonChesterfield force-pushed the spin_and_slash branch from 79edc0e to 1e27b27 Compare September 26, 2019 17:51

MouseDrop controls for swipes.

44580bf

LudwigVonChesterfield force-pushed the spin_and_slash branch from 1e27b27 to 44580bf Compare September 26, 2019 18:38

Merge branch 'master' of https://github.com/TauCetiStation/TauCetiCla…

27e12f1

…ssic into spin_and_slash

LudwigVonChesterfield requested a review from SpaiR September 30, 2019 21:16

ZVee reviewed Oct 19, 2019

View reviewed changes

Added the ability to do spins with a mouse, by a diagonal swipe, fixe…

5e61c6a

…d cooldown for mouse-pull and mouse-push.

LudwigVonChesterfield force-pushed the spin_and_slash branch from 878b825 to 5e61c6a Compare October 19, 2019 10:17

Merge branch 'master' of https://github.com/TauCetiStation/TauCetiCla…

7c882ad

…ssic into spin_and_slash

volas assigned volas and unassigned volas Nov 2, 2019

volas self-requested a review November 2, 2019 08:43

volas requested changes Nov 2, 2019

View reviewed changes

stale bot added the Stalled PR label Dec 2, 2019

stale bot closed this Dec 9, 2019

LudwigVonChesterfield mentioned this pull request Jan 6, 2020

Sweeps and spins. #4458

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Sweeps and spins. #3755

Sweeps and spins. #3755

LudwigVonChesterfield commented Aug 6, 2019 •

edited

TauKitty commented Aug 6, 2019

Snailcat commented Aug 6, 2019

LudwigVonChesterfield commented Aug 6, 2019

SomeUselessGuy commented Aug 6, 2019

LudwigVonChesterfield commented Aug 6, 2019

SpaiR Aug 8, 2019

LudwigVonChesterfield Aug 9, 2019

ZVee Aug 20, 2019

SpaiR Aug 8, 2019

LudwigVonChesterfield Sep 26, 2019

SpaiR Aug 8, 2019

LudwigVonChesterfield Aug 9, 2019

stale bot commented Sep 19, 2019

stale bot commented Sep 26, 2019

LudwigVonChesterfield commented Sep 26, 2019

ZVee commented Sep 26, 2019 •

edited

LudwigVonChesterfield commented Sep 26, 2019

ZVee Sep 26, 2019

LudwigVonChesterfield Sep 26, 2019

ZVee Sep 26, 2019

LudwigVonChesterfield Sep 26, 2019

ZVee Sep 26, 2019 •

edited

ZVee Sep 26, 2019

LudwigVonChesterfield commented Sep 29, 2019

TauKitty commented Sep 30, 2019

TauKitty commented Oct 17, 2019

ZVee Oct 19, 2019

volas commented Nov 2, 2019

LudwigVonChesterfield commented Nov 2, 2019

volas Nov 2, 2019

volas Nov 2, 2019

stale bot commented Dec 2, 2019

stale bot commented Dec 9, 2019

LudwigVonChesterfield commented Jan 6, 2020

Sweeps and spins. #3755

Sweeps and spins. #3755

Conversation

LudwigVonChesterfield commented Aug 6, 2019 • edited

Описание изменений

Почему и что этот ПР улучшит

TauKitty commented Aug 6, 2019

Changelog status: ✔️

Snailcat commented Aug 6, 2019

LudwigVonChesterfield commented Aug 6, 2019

SomeUselessGuy commented Aug 6, 2019

LudwigVonChesterfield commented Aug 6, 2019

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

stale bot commented Sep 19, 2019

stale bot commented Sep 26, 2019

LudwigVonChesterfield commented Sep 26, 2019

ZVee commented Sep 26, 2019 • edited

LudwigVonChesterfield commented Sep 26, 2019

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

ZVee Sep 26, 2019 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

LudwigVonChesterfield commented Sep 29, 2019

TauKitty commented Sep 30, 2019

TauKitty commented Oct 17, 2019

Choose a reason for hiding this comment

volas commented Nov 2, 2019

LudwigVonChesterfield commented Nov 2, 2019

Choose a reason for hiding this comment

Choose a reason for hiding this comment

stale bot commented Dec 2, 2019

stale bot commented Dec 9, 2019

LudwigVonChesterfield commented Jan 6, 2020

LudwigVonChesterfield commented Aug 6, 2019 •

edited

ZVee commented Sep 26, 2019 •

edited

ZVee Sep 26, 2019 •

edited