Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Сегвеи на станции #157

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Kernstall
Copy link
Contributor

@Kernstall Kernstall commented Aug 30, 2018

segway
Собственно вот скрины, есть анимация движущихся колес, спрайты мне дали вроде бы с тг. Пока что есть только один вид сегвеев, потому что их надо бы отбалансить по-хорошему.
🆑 Trogwar

  • rsсadd: В роботехнику добавлена возможность создавать новые сегвеи. Не забудьте вставить батарею!

@Gamma-Bot
Copy link
Contributor

Warning! Invalid changelog detected.

Reason: unknown classes detected. Next should be changed or removed: rsadd.

@Kernstall Kernstall changed the title Сигвеи на станции Сегвеи на станции Aug 30, 2018
@Shudyn
Copy link
Contributor

Shudyn commented Aug 30, 2018

Грифить с сегвея можно?
Надо бы еще добавить сегвеи для нюки с энергощитами и пулеметами.
Для СБ тоже с прицепом для скамбэгов.
И складные сегвеи, чтобы можно было из кармана достать и поехать.
Роботай, кодить будешь ты.

@Kernstall
Copy link
Contributor Author

Можно

if(istype(AM, /obj/vehicle))
var/obj/vehicle/V = AM
if(density)
if(V.airlock_automatic_opening && V.load && (src.allowed(V.load) || emergency))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src. здесь может быть чутка лишним.

name = "Segway"
id = "segway_common"
build_type = MECHFAB
build_path =/obj/vehicle/segway
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(!)Куда-то пропавший пробел.

mob_offset_y = 5
health = 100
maxhealth = 100
on = 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on может получать значения, отличные от 1 и 0? Если нет, то почему это не оформлено под булевую переменную?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

health = 100
maxhealth = 100
on = 1
can_pull = 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Читать комментарий выше.

code/modules/vehicles/segway.dm Show resolved Hide resolved
. = ..()
if( . )
M.layer = layer - 0.9
return
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ого это выход из функции в конце функции?

turn_off()

/obj/vehicle/segway/relaymove(mob/user, direction)
if(user.stat || user.stunned || user.weakened || user.paralysis)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

user.incapacitated() anyone?

user.visible_message("[user] inserts a cell into the [name].", "You insert a cell into the [name].")
turn_on()
return
if(istype(W, /obj/item/weapon/crowbar))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

else if.

user.drop_item()
W.forceMove(src)
powercell = W
user.visible_message("[user] inserts a cell into the [name].", "You insert a cell into the [name].")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

if(istype(W, /obj/item/weapon/stock_parts/cell) && !powercell && open)
user.drop_item()
W.forceMove(src)
powercell = W
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я пока не нашёл дестрой прока который бы чистил ссылку, и удалял саму батарейку по ссылке.

@Skippeer12
Copy link
Contributor

Skippeer12 commented Sep 10, 2018

В чём их смысл, если это спейс байки только для станции то не нужны.
Какие их дебафы/бафы? Кроме батарейки

@SanecMan
Copy link
Contributor

SanecMan commented Sep 16, 2018

Можно прикрутить покраску,использование для этаких профессий (У варианта СБ будет с тайзером,у варианта клоуна - кожуромёт) а так-же не надо забывать о святом емаге,который
может быть будет менять тайзер на лазер или кожуромёт на арбалет синдиката.
(Нужное додумайте)

@LudwigVonChesterfield
Copy link
Contributor

кому нужно.

обычно "хорошие" идеи пишут во флудилке, а не на гите.

@Miracler112
Copy link
Contributor

Если хочешь продолжать работать со своим ПРом и надеется на мерж, то поправь чейнджлог, он у тебя отвалился и неверен

@Kernstall
Copy link
Contributor Author

Верен

@Miracler112
Copy link
Contributor

Бот так не считает, пошатай его всё же. Может у тебя там где-то лишний пробел затерялся

@Kernstall
Copy link
Contributor Author

Я этот чл исправил в первую минуту после того, как сделал пр. Больше смотри на ботов

@Miracler112
Copy link
Contributor

Ну тогда мержа не будет вовсе

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants