В видео намётки по рефакторингу, после 2000 лайков выпущу отдельный видос где уже более комплексно пройдёмся по коду и решим многие проблемы на корню. Также при записи я совершил пару ошибок, предлагаю внимательным их все найти

Однажды Билла Гейтса спросили как ему удалось достичь таких успехов в программировании операционных систем на что он ответил: Если код работает, значит не надо его трогать

Я просто хотел посмотреть как дерутся кубики :(

Пацаны! У Романа хороший и честный курс!!! Я прошел его и меня устроили на работу, я мою полы у них в офисе, всем советую, полов на всех хватит)

я думаю перфоманс просел в разы, столько замыканий с переборами в замыканиях с переборами с linq да еще и в апдейте, что ну его нафиг ) в TransformExtensions в 29-30 строке ошибка (нарушена логика, поломан код) и лучше завести переменную для closestDistance что бы её каждый раз не пересчитывать конечно смотрится эстетически красиво, но между говнокодом вначале и перегибом получившимся в результате где-то лежит истина

Ура. Вот сейчас честно. Если бы все последние ролики были бы такими, критика - объяснение - показал решение, претензий бы не было. И сейчас их(претензий) нет. Адекватно.

Очень интересно наблюдать, как у него в цикле сочетается три условия с разной стилистикой, это шикарно)

Кстати, почему бы не выкладывать заранее какой-нибудь код и предлагать подписчикам его отрефакторить, а потом их закидывать говном (ну или говорить, что они молодцы)? А?

Видео просто супер, намного лучше когда рефакторишь в прямом эфире, думаю со мной согласятся...

Джва года ждал подобный разбор, если не больше. Спасибо! Давай ещё.

Мое перфекто выдавило мне глаза после того как Роман так и не применил поле _speed с магическим числом 0.1f в классе cube. Это как минимум попахивает дилдом в глаз

Во-первых, я бы прикопалась к твоей оценке денег фрилансеров. В отличии от офисных работяг, они также занимаются поиском заказов Во-вторых, если есть конкретная КОНЕЧНАЯ цель, то в чём смысл тратить время на качественный код и расширяемость? Если дальше над ним работать не нужно, то расширяемость не имеет смысла, за которую ты очень сильно цепляешься В третьих, я, как и многие другие люди, ждала, что хотя бы продемонстрируется работа этого кода, но хрен там плавал В четвёртых, многие комментаторы жалуются на ошибки в коде, и это выглядит вдвойне нелепо, что "плохой" код работает, в отличии от "хорошего" В пятых, нафига в методе TakeDamage было кидать исключение, если значение полученного урона меньше нуля? Почему бы не написать просто return? Это в юнити считается нормальной практикой, или во всём шарпе? Исключения - это отличный способ ловить серьёзные ошибки, но... тут же её нет. К тому же, если и происходит какая-то такая ситуация, то ловить её нужно здесь и сейчас, а не где-то извне

Спасибо, что ты есть! Благодаря тебе начал копать документации по неймингу и стилистике кода

37:39 в коде четверное перечисление IEnumerable это не очень хорошо, т.к. IEnumerable не означает, что его несколько раз можно перечислить. :) Такой контракт либо явно через IList надо задавать, либо проверочки как-то по другому делать... А в коде метода достаточно дернуть `source.Min(x => Vector3.Distance(x.position, to.Position))` вместо цикла. Если уж одно перечисление хочется оставить, то там и эксепшны все нужные выпадут. Останется проверить если result == to => throw InvalidOperationException() (В теле тоже, кстати, опечатка, в коде сравнивается another и another)

Вот такой формат лучше всего! Кто-то написал криво и хрен с ним - нечего его хейтить... А ты показал как надо писать и объяснил почему да как... Вот за такое не лень Любо ставить

Я взял заказ на бирже , нужно было спарсить сайт.За 500руб, я написал код,парсил сутки,оказалось спарсились около 60%товаров,не понял что за дела,но потом оказывается на сайте верстка разных блоков отличается.Решил парсить по категориям, написал код под каждую из различий верстки,парсер работал оочень медленно, и нужно было чтобы ПК наверное 3 суток работал без перерывно,чтобы спарсить сайт.В итоге через часов 5 подумал "зачем оно мне надо за 3000р(заказчик предложил поднять цену)", и слился. Андрей 17 лет.

На мой взгляд, подобные ролики это лучшая реклама ваших курсов. Наглядно видно чему можно научиться и как у вас пишут.

Еще вопрос, можно ли было вместо event Action использовать event Func, для того, чтобы сигнатура подходила под метод Remove? Там же возвращаемое значение имеет тип bool.

А какова была точная формулировка задачи для фрилансера? Если чтобы работало, то задача выполнена :) Я как-то оооочень давно на 1-м курсе писал софтинку для анализа геометрии GPS-карт для работы. Сейчас я в ужасе от написанного мной тогда кода, однако софтина работает и используется уже 13 лет и никто не жалуется, т.к. она закончена и расширение и развитие её не планировалось изначально. имхо расширяемость и поддерживаемость КОДА должны быть в постановке задачи, иначе тратить время на соблюдение всех принципов ООП просто незачем.

36:00 метод Transform Closest 1. почему сначала поиск element == to, а следующей проверкой уже Count() == 0? Разве не логичнее наоборот, если в списке ничего нет то и элементов там не будет 2. почему не использовать вместо x.Count() == 0 !x.Any() ?

Я думаю, что использовать LINQ в Unity должно быть запрещено. Потом всё равно придётся их выпиливать из-за бесконечных стартеров из-за GC. В идеале код вообще не должен выделять памяти в основном цикле, только на загрузке. Ну и то что дистанция вычисляется не квадратом длины и не кэшируется для самого близкого - кровь из глаз.

Очень классно разобрал и собрал код)) Жду следующего ролика

Может уже писали, а в методе Closest если мы нашли элемент, не лучше break ставить, что бы остановить перебор обьектов??

Классный выпуск, спасибо!

Роман)) не знаю куда еще написать, но можно ли у вас попросить демо доступ или подробное описание вашего курса?)

Самый лучший ролик на канале

Спасибо за разбор У меня вопрос. Я LINQ не пользуюсь в Unity. И вот, такое количество запросов в update не будет генерировать кучу мусора?

Сакутин, давай несколько строчек кода с твоей игры, как это в процессе выглядит

Ну, не соглашусь. Если ты за это заплатил 500 рублей, значит человек справился с задачей за час. Более того, ты заказал код на фрилансе, что само собой намекает на то, что проект не долгоиграющий, его нужно сделать и забыть (например, для курсача в универе). Тогда какой смысл думать о рефакторинге и тд? В этом смысла нет. Чел реально годно справился с задачей по соотношению цена/качества. Он написал код, который кое-как, но работает, протестировал всё это дело. И справился с этим за час. У тебя пусть и с объяснениями ушло минут 40 только лишь на рефакторинг. А теперь посчитай, за сколько бы ты сделал задачу с нуля. Ушло бы часа 2-3. Теперь давай подумаем, стоит ли эта задача 3 часов? Если она прямолинейна, о масштабируемости говорить не приходиться? Нет, не стоит. Я конечно не знаю, может, что в задаче было написано, но вряд ли: "Нужно сделать задачу с расчётом на то, что это долгоиграющий проект, его будут дорабатывать другие люди, сделай качественно, можешь хоть несколько часов это делать, не торопись."

Не ну кнш Linq в Update это жестко.

как то начинал изучать c# забросил и через время пересел на Java и не могу найти канал похожий на твой ,очень все информативно , хорошо хоть языки похожие и можно все равно что то для себя подметить)))))

Кулдаун не имеет смысла устанавливать в редакторе, так как после первого же обнуления он приравнивается 1. Не надо путать сам счётчик с его верхней границей. Нужна отдельная переменная.

Можно ли в приложение добавить ролик с Ютюб или др.видеоресурса? Не прилетит после публикации своего софта в плеймаркет бан за автор.права роликов?

Менеджер кубов в апдейте не нуждается, тут надо сигналами работать. Если куб свободен, пусть посылает сигнал и вызывает поиск врага. Если куб сдох, пусть посылает сигнал, чтобы его могли удалить из списка. Нечего там апдейту делать.

Спойлер: чувак переделал говнокод в еще больший говнокод с претензией на правильность

Вот такого побольше, Ром) Это и полезно, и рожать ты можешь такого контента много, и сразу видно, к кому учиться надо идти)

На кого ваш курс рассчитан? На нулей, начинающих?

Не совсем понял для чего использовался IEnumerable в параметре метода FindClosest... Чтобы List с кубами протащить? А почему просто List в параметре не прописать? (вопрос от нуба, если что... камнями прошу не забрасывать).

Посмотрел 15 минут, ожидая увидеть какой-то реально криповый код уровня самых тайных и закрытых индийских школ кодинга. А увидел просто обычный код, который РАБОТАЕТ, и ЧСВ-шного школьника, который кукарекает. Было ли в ТЗ написано про стиль кода? Публичные поля ему не нравятся? Смешно видеть начинавшихся книжек про ООП и клин код и считающий, что это негласный стандарт языка. Хотя сам язык почему-то ничего не запрещает. Что уж говорить про то, что в шарп иногда залазят ребята из С, и пишут так, чтобы перформанс был, потому что это требуется. Самое глупое в видео - это попытка пересчитать заработок фрилансера за час кодинга на месячную зп. В офисе у тебя зп постоянно капает, работаешь ты или нет. А фрилансер большую часть времени заказы ищет.

Я думаю из-за того, что ты не захотел в этом видео разделять сущности, реализация CubeManager стала слишком мудрёной. Пока ты не начал добавлять методы расширения код и вправду стал очень понятным. Дальше мне кажется замудрил немного. Но Формат видео очень нравится. Жду выпуска про Бугаенко, очень интересно узнать твое мнение о нем.

я не фрилансер, работаю на себя, но даже я лучше бы написал за спасибо😐, может поэтому я ничего не зарабатываю

"У мастера хороший код - единственный стиль", - японский Сенсей Аригоша Харидударь

отличный формат

почему Вы не добавили в первом коде методы сеттеры начального состояния переменных?

А можно было просто вместо cooldownAttack сохранять время предыдущей атаки, а когда атакуем проверять, last - Time.deltaTime > 1, если это верно, то в last записать текущее deltaTime и атаковать?

В 360 код афигено видно

вообще что то жестко, вот у нас есть список Кубов а что будет если другие кубы убьют пару кубов из этого списка, пока мы убиваем куб? Там будут указатели на null, или как это будет работать, или мы получим доступ к неправильной области памяти? Метод дестрой при уничтожении куба он наверно переведет его в другое состояние, но поскольку есть ссылка на объект, он не будет удален сборщиком мусора?

Вы хотели мерседес по цене оки. Сколько платите то и получаете. 😁

Count() == 0, у тебя цикл по всей коллекции будет, лучше Any () == false

Похоже джун не реализовался и открыл свой канал на ютубе)) Ты ТЗ выдал тебе на писали под твой ТЗ код, что не так?

А почему Update private? Может не досмотрел где он используется в классе Cube?

Роман, вроде вы говорили что "мы никогда не используем статистические методы или классы" ? Или я что то не понимаю?

Ошибка копипасты в 29-30 строчке в методе поиска ближайшего. Можно было бы еще запоминать дальность от прошлого в переменную и с ней сравнивать. А еще квадрат магнитуды использовать вместо дистанции если хочется еще ускорить код ) Еще не уверен не будет ли просчитан куб у которого уже появился враг? мы выдаем текущему пустому кубу врага, и врагу говорим что мые го теперь враг. Но этот куб же дальше будет в листе свободных кубов в этом апдейте нет? Да и первый тоже останется. их по сути мне кажется удалять надо из листа заменив форич на фор. Еще ошибка в кубе была. Просто переменную недопереименов кое-где

Я, будучи питонистом, немного обескуражен таким количеством вложенных итераторов, изменением размерности списка внутри цикла по нему, а так же наличием такого большого кол-ва повторяющихся проверок. будь этот скрипт написан на питоне, и в игре было бы 5 FPS от силы. Ну всё же я понимаю, что NNS - это вообще отдельная тема для разговора.

Подскажите: как правильно получить все нужные объекты на сцене(как например в видео - получить список всех кубов)

Попахивает exception-ом при итерировании когда удалится элемент из _cubes. Подстройки перед вызовом функции под код внутри функции - печально. Постоянно дергать linq - печально.

KD-tree напрашивается для поиска ближайшего объекта

Роман, хватит делать такие переходы. Смотреть невозможно, делай прямые склейки, так намного лучше

ManagerManager - ведь должен же кто-то контролировать этих менеджеров.

13:47 я б еще else if сделал одной инструкцией, не выделяя блок else и if отдельно, раз у if нет парного else, но тут уже на вкус и цвет, может показаться, что в одну инструкцию будет чуть менее читабельно и понятно, хоть и чуть короче и красивее визуально

Код должен быстро и правильно работать. А с названиями переменных всем не угодишь)

Не лучше было бы в Cube поле с врагом заменить на рейкаст, а то выглядит очень странно?

По сравнению с предыдущими видосами этот на голову выше, т.к. видны рассуждения, которые приводят к хорошему коду

Роман, как вам идея просто показать скорость времени компиляции кода фрилансера и вашего? Заранее спасибо

А тебя не смущает в методе расширения GetComponent вызываемый каждый кадр? Он насколько я знаю работает не так уж и быстро

Closest всегда возвращает первый элемент списка, но это уже другая история KEKW

Почему не включишь режим вима? Очень много работаешь мышкой, это все можно ускорить

16:10 - почему паблик, почему не геттер и сеттер? => в Юнити не работает, это оператор Линк. Nearby - читается "ниа-бай", а не "нёби"

А может на 19:48 вместо Contains сделать !cube.HasEnemy? Ведь в another будут свободные кубы и если cube.HasEnemy будет false то это значит, что он находится в another. Не проще вообще цикл foreach изменить таким образом: ... if(!cube.HasEnemy) throw new InvalidOperationException(); var nearby = others.First(); float minDistance = Vector3.Distance(cube.transfrom.position, nearby.transform.position); foreach(var another in others) { float distance = Vector3.Distance(cube.transfrom.position, another.transform.position); if(distance < minDistance) { nearby = another; minDistance = distance; } } ... minDistance можно инициализировать максимальным значением float или бесконечностью (вроде в C# есть такое понятие). Вроде выглядит намного лучше, тем более если инициализировать minDistance по другому. Хотя мне не самому не нравится cube.transform.position и т.д. Так же метод findClosest является private и мы передаем в него IEnumerable без самого cube, но внутри проверяем чтобы его не было, для чего делается эта проверка? Мы же знаем, что передаем без cube. Это просто для себя? И я заметил, что часто используют Linq, и хотел спросить, а это не долго? Ну вот в коде используется Linq, чтобы убрать сам куб из списка свободных, не проще в foreach просто пропускать его? P.s. я C# учил несколько лет назад, и на нем уже давно ничего не писал, а с Unity ни разу не работал. И да писал в блокноте, скорей всего есть ошибки в коде, не стал вглядываться

Формат без говна намного приятнее, продолжай!

Всем привет! Надеюсь, что тут есть знатоки, кто сможет подсказать: Какой конкретно вид платформера следующие игры - castle crashers, tmnt rescue palooza и т.д. Вопрос конкретно в том, что это вроде как 2D платформер, но с возможностью вертикального перемещения или я совсем неправ? Хочу попробовать игру создать такого типа, буду очень благодарен.

зачем нижнее подчеркивания в названии переменных?

Мечтаю, чтобы Рома купил код у меня да ещё и ошибки разобрал

Почему ты "служебные" методы поместил вначало а не в конец класса? По значимости они нулевые, когда файл открываешь каждый раз мотать вниз?

Блин, это сильно повысит Аллоки, инициализации ексепшенов + линку которая тоже подбрасывает их.

Ничерта непонятно, но очень интересно. По-моему это уже код не на 500р в час, хотя не мне судить, я как раз пишу подобный говнокод как в начале...

Походу если после метода старт добавятся кубы на сцене, то старые кубы не будут их быть( А метод старт я так понял при инициализации объекта происходит

1. в проверке расстояния поломал логику 2. ближайший куб из списка - как-то нелогично, лучше ближайший из списка к кубу (cube.Closest(others)) 14:08 "мы к этому вернёмся после менеджера" - вернулся? я, честно, пока смотрел уже забыл

Как говорил кто-то умный "ты не работаешь плохо, потому что тебе мало платят, а тебе мало платят, потому что ты работаешь плохо"

Ну я понимаю, что сравнение нахождение дистанции между самим собой и проверка на то когда же она станет меньше самой себя это опечатка. Но блин. Называть метод Closest (где глагол?), запихивать в него геткомпонент и вызывать в апдейте это конечно мощно

Ты хотел типу var присвоить null? :)

предлагаю запретить if/for/while без фигурных скобок на законадательном уровне!

37:37 if(Vector3.Distance(another.position, another.position) < Vector3.Distance(another.position, another.position)) - лолчто?

У меня блин стилистика кода хорошая я знаю как делать не нужно, но я просто не могу программировать, я не знаю, что писать

12:58 почему _attackDistance и _speed объявлены как переменные, если они нигде не меняются? Их нужно было объявлять как константы.

Такой вопрос. Не то чтобы я гений c#, но меня немного смущает сначала отбор через select, а потом хождение по всем трансформам. Ну, конечно по сравнению с тем что было это вообще не проблема. Но все таки может кто-то обьяснить, так действительно правильно делать, или лучше проходить через 1 цикл используя сразу трансформ

а почему только 360р?

Лайв гораздо понятнее😁

Я вообще не врубаюсь зачем пытаться быстро печатать и ещё при этом делать ошибки. Можно же спокойно печатать код без спешки, когда ты собираешься что-то объяснять.

Юзайте райдер. Он автоматом почти все предложит перерефакторить, как Роман показал

За что заплатил, то получил! Почему-то никто это не учитывает. Программист оценивает свои знания и определяет лимиты при которых возьмётся за работу. Например если у меня нет опыта и я не знаю кучу тонкостей языка и механики работы "движка", то я могу предположить что могу сделать за 500 руб. Например если я имею большой опыт и знаю кучу тонкостей, то понимаю что здесь мне нужно проделать кучу работы и буду брать за эту же работу уже 10000 руб. Поэтому нечего удивляться, если эту стоимость предложил ты или программист, то надо предполагать что будет дерьмовый код.

Что такое фасадирование?)

Открою Visual Studio за 0.99$

7:04 Update писать двойной for это уже грех, FindTag :(( fps 0.1

Зачем я это смотрю, я же джавист...

поставь себе райдер

Заголовок выглядит как "Заказал УЖАСНЫЙ код у фрилансера за 500 рублей".... Ну раз заказал УЖАСНЫЙ код - как можно удивляться, что код ужасный? Что заказывал - то и получил? Зачем заказывал-то "ужасный" код?1 😁😁😆

Сначала норм, потом терпимо, затем сложно, и в итоге приздец что за магия. Надо поднимать свой скилл в C#, а не тоже буду гавнокод за 500р в час делать.

Скажите, почему public это плохо?

ничего не понятно, но что то понятно.