Автолат и патроны #6950
Автолат и патроны #6950
Conversation
Changelog status: ✔️ |
|
может лучше сделать что патроны для шотгана собираются в кучки по n патронов??? |
Может и лучше, но не проще)) |
code/game/machinery/autolathe.dm
Outdated
| var/datum/autolathe_recipe/recipe | ||
| var/obj/P = path | ||
| var/amount = 1 | ||
| var/datum/autolathe_recipe/recipe = new |
There was a problem hiding this comment.
А зачем тут создавать? Типо, ты создаешь и сразу же пересоздаёшь, если if(ispath(path, /obj/item/stack)).
Тип, почему бы не вернуть
else
recipe = new /datum/autolathe_recipe
и не убрать этот new ?
There was a problem hiding this comment.
Если путь не равен /obj/item/stack, то используется /datum/autolathe_recipe, который я сразу создаю. В противном случае создается рецепт типа /datum/autolathe_recipe/stack. Это другой класс
There was a problem hiding this comment.
Ну так я сократил код и убрал
else
recipe = new /datum/autolathe_recipe
There was a problem hiding this comment.
Чисто в теории, твой вариант медленнее. Раньше создавалась только нужная сущность, а теперь у тебя будет всегда создаваться объект от datum/autolathe_recipe и в некоторых случаях ниже будет уже ПЕРЕсоздаваться объект класса datum/autolathe_recipe/stack.
И можно сказать, в твоем случае получилось плохое сокращение кода
There was a problem hiding this comment.
Есть логика в твоих словах. Исправил.
code/game/machinery/autolathe.dm
Outdated
| else | ||
| usr.remove_from_mob(I) | ||
| I.loc = src | ||
| if(istype(I, /obj/item/ammo_box)) |
There was a problem hiding this comment.
Может лучше перенести этот иф на ту же строчку, что и этот else ?
|
Кринж У тебя еще спрайт коробки с патронами и самими патронами не совпадают. |
Я решил проблему с помощью тех спрайтов, которые смог достать. Напиши как нарисуешь спрайты которые тебе нравятся и я их добавлю. Не вопрос) |
|
^ можешь попросить в спрайтерском канале в дискорде, что бы кто нибудь помог |
|
Замечания исправил. Цвет патронов теперь совпадает с коробкой. Обновил описание ПРа. |
KIBORG04
added
Awaiting Review [Sprite Dep]
|
Ну значит зову аппруверов |
Сообщил |
|
По просьбе Dedawe добавил возможность класть коробки патронов в combat belt и security belt Пока Draft, жду новые спрайты |
|
Спасибо DedYarick за новые спрайты коробок. Добавлена ещё одна коробка для стан слагов. В протолате РнД теперь будет печататься эта коробка, а не по одному патрону. Шапку темы обновил. |
|
Вроде как опять новые спрайты, апруваем |
KIBORG04
removed
the
Awaiting Review [Sprite Dep]
Описание изменений
Убрал из автолата единичные патроны для дробовика и добавил коробки по 8 патронов. Спасибо DedYarick за спрайты коробок.
Коробки с восьмью патронами для дробовика.
Некоротые из них можно напечатать в автолате.
Цены я поменял. Напомню, один лист металла - это 3750 см в кубе. Ровно столько стоит один слаг и дробь. Ну и остальным патронам, которые есть в автолате, добавил цену.
Был баг с тем, что можно было всунуть в автолат пустую коробку из-под патронов и это давало металла столько же, сколько и полная патронов коробка. Я это исправил, теперь автолат считает цену каждого патрона, находящегося в коробке, то есть в коробке могут быть разные патроны, например slag вперемешку с beanbag, и он их правильно посчитает. Так же составляются и рецепты автолата, он смотрит не на цену коробки, а на цену патронов в ней.
Слаг-патроны теперь синие, а не белые. Я просто не смог перекрасить коробку в белый цвет, всё бы сливалось))
Дарт-патроны теперь фиолетовые. Такие спрайты уже были в билде, но не использовались, плюс коробка по цвету подошла.
В протолате РнД будет печататься коробка стан-слагов, а не один патрон, как раньше.
Добавил возможность класть коробки патронов в combat belt и security belt
Почему и что этот ПР улучшит
Вместо одного патрона для дробовика теперь коробка из восьми патронов. Что гораздо удобнее использовать и быстрее печатать.
Авторство
Ahio, DedYarick - спрайты коробок
Чеинжлог
🆑 Ahio, DedYarick