A
Я сегодня весь день ворчу, конечно, но вот нахера они выносят куски кода в отдельные функции, которые только один раз используются?
Чтоб чистенько было и логика была понятна без вчитывания в лапшу кода
Size: a a a
2020 March 16
AS
AS
Вот здесь чисто тело Look перенести как есть в Decide
N
Я сегодня весь день ворчу, конечно, но вот нахера они выносят куски кода в отдельные функции, которые только один раз используются?
им платят за строки
AS
Ъуъ
MG
вкусовщина
M
Я сегодня весь день ворчу, конечно, но вот нахера они выносят куски кода в отдельные функции, которые только один раз используются?
Я кстати так же сделал, ризбил чут-ли не каждую строчку в метод обернул, и каждый по смыслу метод назвал. И знаете что, я таким образом разобрал что за дерьмо у меня там творилось.
AS
Мне кажется, это ухудшает читабельность
AF
с одной стороны да, с другой стороны тебе не нужно править большой метод, правишь маленький)
MG
decide on look, а могло быть decide on smth else, логику понять можно, читабельность имхо не пострадала. и когда я хожу-брожу по коду, я бы заглянул в Decide и сокрее всего не полез в Look, а так пришлось бы вчитываться в реализацию
N
с одной стороны да, с другой стороны тебе не нужно править большой метод, правишь маленький)
и не увидишь цельной связи с остальным кодом и сломаешь вообще всё
MG
но и обратную позицию могу принять. дело вкуса
AF
и не увидишь цельной связи с остальным кодом и сломаешь вообще всё
увижу)
AF
но и обратную позицию могу принять. дело вкуса
согласен
AS
decide on look, а могло быть decide on smth else, логику понять можно, читабельность имхо не пострадала. и когда я хожу-брожу по коду, я бы заглянул в Decide и сокрее всего не полез в Look, а так пришлось бы вчитываться в реализацию
На практике из-за такого часто приходится скакать по вызовам, чтобы понять, что там происходит, и на четвёртом переходе уже забываешь, откуда пришёл
KE
На практике из-за такого часто приходится скакать по вызовам, чтобы понять, что там происходит, и на четвёртом переходе уже забываешь, откуда пришёл
ctrl + alt + leftarrow
KE
Для меня было проблемой вспомнить откуда я пришел пока не прознал про киллерфичу - скакнуть назад
AS
ctrl + alt + leftarrow
Ну этим я пользовался сколько себя помню)
A
enum для реализации конечного автомата — точно лишнее
покажи плз если не лень(можешь в лс) реализацию, не совсем понимаю как без enum
AS
Вот решение от Юнитеков https://learn.unity.com/tutorial/5c515373edbc2a001fd5c79d?language=en
Ну вот же только что скидывали реализацию без енама)