Новые предметы для растяжек + камера в автолат#8849
Conversation
снаряды дробовиков + оружие + огнетушитель + рецепт камеры в автолат
There was a problem hiding this comment.
Code Review
Аналитический отчёт от Большого Брата. Обработка запроса завершена. Приветствую! Я проанализировал предложенные Вами изменения. С технической точки зрения, работа требует доработок. В пулл-реквесте реализована возможность установки огнетушителей, оружия и патронов на растяжки, а также добавлен чертеж камеры в автолат. Выявлены критические замечания: ошибка в определении свойств чертежа камеры (дублирование build_path вместо build_type) и потенциальный runtime error при выстреле оружия из-за передачи объекта вместо моба в процедуру shoot_live_shot. Также рекомендуется исправить логику освобождения растяжки огнетушителем при срабатывании без предохранителя и убрать избыточную проверку can_trigger_gun для механической ловушки.
…Parashiza into rastyazhki_hotfixes
|
там почти все завязано на /mob, я умру пытаясь это исправить( |
|
ай каст перезапуск проверки ч3 |
…яжке где надо (#9389) ## Что этот ПР делает Черрипикает фиксы с ПРа #8849. Уменьшает время до взрыва гранаты до 0.4с ибо иначе с растяжками работают только с осколочными или мощными гранатами. Так будет хоть как то вредить от детонации более слабых гранат ## Почему это хорошо для игры Бубс попросил (а вообще багфиксы круто) ## Демонстрация изменений <details><summary>Изображения и видео</summary> <p> </p> </details> ## Список изменений :cl: bugfix: Исправлен рецепт растяжки. refactor: Перенос дефайна TRIPWIRE_GRENADE_DETONATION_TIME к гранатам и его использование. balance: Уменьшено время детонации гранат на растяжках (1с -> 0.4с). /:cl:
|
плашки что давали надежду были ради 4х строчек фиксов?.. УЭЭЭЭЭ |
|
попроси Эшель о ревью |
| var/is_tk_grab = (living_user && !isnull(living_user.tkgrabbed_objects[src])) | ||
| var/is_pacifist = (living_user && (HAS_TRAIT(living_user, TRAIT_PACIFISM) || GLOB.pacifism_after_gt)) | ||
| var/is_buckled = (living_user && living_user.buckled) | ||
| var/is_left_hand = (living_user && living_user.l_hand == src) |
There was a problem hiding this comment.
Это бы все вообще в старт файр вынести
| bonus_spread += living_user.get_fracture_spread_bonus(is_left_hand) | ||
| add_fingerprint(living_user) | ||
|
|
||
| if(is_buckled) |
There was a problem hiding this comment.
А это в сетап буллет акураси вообще
| BB.preparePixelProjectile(target, user, modifiers, spread) | ||
| if(isliving(user)) | ||
| BB.preparePixelProjectile(target, user, modifiers, spread) | ||
| else |
There was a problem hiding this comment.
А равзве если у тебя он ливинг, то оригин атомом не будет юзер?
| /obj/item/ammo_casing/proc/throw_proj(atom/target, turf/targloc, mob/living/user, list/modifiers, spread, atom/firer_source_atom) | ||
| var/turf/curloc = get_turf(firer_source_atom) | ||
| /obj/item/ammo_casing/proc/throw_proj(atom/target, turf/targloc, atom/movable/user, list/modifiers, spread, atom/firer_source_atom) | ||
| var/turf/curloc = get_turf(firer_source_atom) || get_turf(src) || get_turf(user) |
There was a problem hiding this comment.
Не уверен, что эти get_turf - хорошая идея. Особенно src
| distro += variance | ||
| for(var/i = max(1, pellets), i > 0, i--) | ||
| var/targloc = get_turf(target) | ||
| if(!get_turf(user) && !get_turf(firer_source_atom) && !get_turf(src)) |
There was a problem hiding this comment.
А точно ли эти проверки нужно делать внутри цикла?
| if(is_tk_grab) | ||
| rotate_to_target(target) | ||
| if(living_user) | ||
| SEND_SIGNAL(living_user, COMSIG_MOB_GUN_FIRE, src) |
There was a problem hiding this comment.
А этот сигнал тут нужен?
Что этот ПР делает
Добавляет возможность цеплять патроны для дробовиков (с 50% урона), все пушки, огнетушители (они могут взрываться, да) и однабаллонки. Ну и добавляет рецептик камеры (которая делает фотки) в автолат.
Почему это хорошо для игры
Люди накинули пару крутых идей. Побегал сам, поделал фотомины, на станции кончились камеры
Демонстрация изменений
Изображения и видео
Список изменений
🆑
add: Пушки, патроны 12g, огнетушители, однобаллонки для растяжек. Добавляет рецепт фотоаппарата в автолат.
tweak: Спрайты направляются в сторону куда смотрит растяжка.
refactor: Убирает жесткую необходимость в мобе-стрелке для основных проках стрельбы.
/:cl: