Синди мыло стирает фибры и отпечатки с объекта #13711
Conversation
Changelog status: ✔️ |
| if(user.client && (target in user.client.screen)) | ||
| to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>") |
There was a problem hiding this comment.
Проверять на клиентский screen не очень корректно - итем на нем может оказаться даже если сам предмет не в инвентаре.
Тут несколько вариантов, самое простое наверно проверять, что итем находится на тайле, а не где-то еще
| if(user.client && (target in user.client.screen)) | |
| to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>") | |
| if(!isturf(target.loc)) | |
| to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>") |
| if(user.client && (target in user.client.screen)) | ||
| to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>") | ||
| else if(istype(target,/obj/effect/decal/cleanable)) | ||
| to_chat(user, "<span class='notice'>You scrub \the [target.name] out.</span>") | ||
| qdel(target) | ||
| if(target.fingerprints) | ||
| target.fingerprints = null | ||
| if(target.suit_fibers) | ||
| target.suit_fibers = null | ||
| target.clean_blood() | ||
| to_chat(user, "<span class='notice'>You FULLY clean \the [target.name].</span>") | ||
| return |
There was a problem hiding this comment.
Я еще рекомендую не строить лесенки из if-else, и воспользоваться ранним возвратом, так код будет читаймее. Что-то вроде
if(!isturf(target.loc))
to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>")
return
if(!istype(target,/obj/effect/decal/cleanable))
to_chat(user, "<span class='notice'>....</span>")
return
...тут остальной код...
| if(target.suit_fibers) | ||
| target.suit_fibers = null | ||
| target.clean_blood() | ||
| to_chat(user, "<span class='notice'>You FULLY clean \the [target.name].</span>") |
There was a problem hiding this comment.
Сейчас тут нет задержки в действии, и два сообщения
You scrub \the [target.name] out
You FULLY clean \the [target.name]
просто сразу появятся друг за другом. Если хочется добавить задержку - посмотри в коде билда примеры использования do_after()
| to_chat(user, "<span class='notice'>You need to take that [target.name] off before cleaning it.</span>") | ||
| else if(istype(target,/obj/effect/decal/cleanable)) | ||
| to_chat(user, "<span class='notice'>You scrub \the [target.name] out.</span>") | ||
| qdel(target) |
There was a problem hiding this comment.
это точно что-то лишнее, тут ты удаляешь итем который чистишь из игры.
|
Предлагаю все же оставить хотя бы сообщение, мол "заметны следы затирки", иначе детектив вообще бедолага. И просто, и не обесценивает профу. |
Описание изменений
Синди мылу добавлено свойство убирать фибры и отпечатки при клике
Почему и что этот ПР улучшит
Добавит новое применение мылу, расширит стелс возможности
Авторство
hyda
Чеинжлог
🆑 hyda