[FD-Assets] Pure evil and Bugfixes | MAIN #48
Conversation
| playsound(src, 'mods/_fd/fd_assets/sounds/prime_soul/Mp_intro2.ogg', 60, 0, extrarange = 13, falloff = 4) | ||
| playsound(src, 'mods/_fd/fd_assets/sounds/prime_soul/intro_music.ogg', 60, 0, extrarange = 13, falloff = 4) | ||
| ISay("Ah... Free... At last.") | ||
| sleep(52) |
There was a problem hiding this comment.
На самом деле, лучше использовать spawn() вместо sleep(). Он и удобнее и меньше нагружает код
Он, фактически, делает ровно то же самое, но при этом весь процесс не встаёт намертво
There was a problem hiding this comment.
И да, у нас же вроде как есть вычисление в СЕКУНДАХ. С точки зрения читабельности кода куда лучше использовать [proc_name](1 SECOND), так как не каждый может понять "а скока биёндовских тиков вмещается в секунду???"
There was a problem hiding this comment.
И да, у нас же вроде как есть вычисление в СЕКУНДАХ. С точки зрения читабельности кода куда лучше использовать [proc_name](1 SECOND), так как не каждый может понять "а скока биёндовских тиков вмещается в секунду???"
Ну. Тут четко 5 секунд и 2 миллисекунды. (10 тиков = 1 секунде) По этому тут изменить НИКАК. Разве что делать 5,2 SECONDS
Насчет SPAWN(). Я в ближайшее время гляну в бионд-рефах о нем и подумаю как это реализовать
| switch(rand(1,2)) | ||
| if(1) | ||
| natural_weapon = /obj/item/natural_weapon/punch/minos/uppercut | ||
| forceMove(get_step(target,get_dir(target,src))) |
There was a problem hiding this comment.
Это, конечно, вкусовщина, но мне лично никогда не нравился его мгновенный телепорт в упор к тебе. Может быть, сделать как у теле-панча или дэша с оружием? Так, по крайней мере, будет видно след от перемещения Сурова
Впрочем, необязательно! Просто совет!
| var/pick_attack = pick("PREPARE THYSELF!", "THY END IS NOW!", "DIE!", "JUDGEMENT!") | ||
| switch(pick_attack) | ||
| if("PREPARE THYSELF!") | ||
| if(stamina - 3 < 0) |
There was a problem hiding this comment.
Немного непонятно, чего мы здесь хотим добиться. "Если [N] - [стоимость атаки] меньше чем [0] - повторяем цикл". Данное условие попросту не может быть достигнуто, в большинстве случаев. Мне кажется что куда лучше было бы написать сюда if(stamina < 3), аналогично со всеми остальными атаками
There was a problem hiding this comment.
Судя по всему - здесь должна быть стамина в промежутке от (0; +Бесконечности). Т.е. Стамина не должна быть НОЛЬ и НИЖЕ. Однако причину этого я не понимаю. Так что последую ревьюву.
There was a problem hiding this comment.
Т.е. Стамина не должна быть НОЛЬ и НИЖЕ
У нас выше идёт проверка на это, отдельная. Здесь проверяется именно то, достаточно ли у нас стамины на юз конкретной способности. Нет? Тогда пытаемся сделать следующую
ПР делает что-то умное, например копирует #47