Andrey C# Expert, по итогу на ~1400$ тестовое выполнено, только чуть защитить его на собеседовании? В принципе тестовое тянет на такую цену? За код спасибо. 5:20 согласен, не выглядит полезным. не нужно мокать все сервисы но в комментах уже обсудили 6:50 asset Provider. Лучше сразу исп generic, тип без cast 8:00 минусы сервис локатора. 8:10 антипаттерн 8:40 без PlayerPrefs = binaryFormatter 9:10 в этом месте binary formatter не нужен 9:30 лучше с async "using" declaration -> try-finaly -> dispose -> fileStream сам закроется IL 11:20 на асинхронный сложно исправлять. до самого ядра дописывать 15:30 вместо такого хардкода лучше SO 18:30 лишние слова. лучше вообще без exception 20:50 нужно разделение на save date и модель данных json, protobuf 24:20 временно увеличил severity подсказок, чтобы понять сколько всего в проекте небрежностей 25:20 предпочитает в средних и мелк проектах throw Exception вместо Debug.Log лучше в первые месяцы помучиться, но потом ядро будет стабильнее 25:40 "data?.Some()" = скрытие бага 26:20 Factory не нужно для каждого объекта, особенно если он создаётся 1 раз 28:00 реальный проект должен оплачиваться 28:20 отказали из-за незнания библиотеки. Согласен, жидкий ответ.

Спасибо за видео! Как по мне - это очень удачный формат видео. Я имею в виду код ревью + рефакторинг. Я считаю что у Вас отлично получается указывать на ошибки, комментировать и сразу же в коде рефакторить их. С нетерпением жду новых видосов и всего самого наилучшего! :)

Я ниче не хочу сказать, но этот проект должен стоить больше 1400$ :)

Посмотрел на одном дыхании. Хотелось бы модет где-то схемы архитектуры нарисованные, но если имеет смысл. А так очень понравилось, что и детали и глобально интересные вещи подмечаешь

Пили еще код ревью, пжалста !!! Топ контент.

Большое спасибо за разбор! Приятно видеть фидбэк более опытных программистов. Кстати после собеседования написали, что как у них откроется вакансия на джуна, то напишут мне, но сообщений пока нет :)

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

О, я кажется натыкался на это задание. Не стал делать, тк реально выглядит как реализация фич

Делал подобное тестовое задание на джуна, тз очень схоже, как минимум начало и конец текста точь-в-точь одинаковы. То ли я неопытен и тз нормально составлять именно так, то ли это просто мошеннический шаблон)) Меня кстати не взяли, проигнорировали. А потом чат с работодателем и вовсе пропал...

8:35 дак это и есть Чисты DI с внедрением через конструктор. Сам сервис локатор немного для другого в этой архитектуре должен быть использован. А DI-Контейнеры на то и нужны чтобы зависимости не прокидывать в глубь вручную. Тут сервис локатор заменяет контейнер. И есть мнение от авторов этой архитектуры что контейнер передавать в фабрику это позволительно, т.к. там идёт сборка объекта и это инфраструктурная часть кода (это уже мое мнение).

Кстати спасибо за совет в конце видео, что не стоит делать такие тестовые задания бесплатно Самому лично приходилось делать такие тестовые задания, в итоге меня не взяли т.к. я не знал способов внедрения монетизации в игру, не знал Firebase, не знал как делать деплой через webgl на сайты yandex Я действительно их не знал, и подумал что их надо долго изучать, и в принципе не приводил агрументы что потрачу время на их изучение

Люди, помогите, пожалуйста как научится понимать как строить архитектуру приложения? Все говорят про k-syndicate, что это архитектура под копирку, но что здесь плохого, если эта архитектура удачная. У этих людей за плечами много опыта, уж явно они больше знают джунов. Читал книгу Роберта Найтстрома про геймдев паттерны, ведь по сути архитектура строится на паттернах, ну и конечно же прошёл курс от k-syndicate по архитектуре. Когда прохожу тестовые, то говорят проект не гибкий и не способен расширяться и не понимаю каким образом ещё более можно расширять проект. Посоветуйте книжки по архитектуру приложений, желательно на c# или другой похожий язык как c++ например. Если у вас есть другое мнение и архитектура игры и приложения это разные совершенно вещи, то скажите тогда как в таком случае действовать

Полезный видос как для новичка (который даже не собирается писать на юнити). Лучше учиться на чужих и реальных ошибках, понимаешь как лучше писать код, спасибо)

Класс! Спасибо за видео. Иногда интересно посмотреть, что и как люди пишут. Но вот есть несколько нюансов, которые бросились в глаза. Хотел бы их поднять на обсуждение или просто обратить внимание. 0. Действительно, архитектура местами прям копирует K-Syndicate. Но вот там, где нужно писать самому - начинает просаживаться. Некоторые места, наоборот, опущены. В общем, интересный симбиоз. 1. "DI не используется" (c). На самом деле, используется. Здесь не используются DI-контейнеры (как, например, Zenject). Прокидывание зависимостей ручками - это т.н. "pure DI" (если вам нужны термины). Спор о том, что лучше "pure DI" или DI-контейнеры - вечен и идёт ещё от отцов-основателей техник DI в C# (например, того же Марка Симана, который пришёл к pure DI). Однако, однозначного мнения здесь нет. Главное понимать, зачем и почему ты используешь тот или иной подход. 2. "IRandomService - бесполезный" (с). Не совсем. Благодаря выносу абстракции с рандомом, ты получаешь дополнительный контроль над процессом и логикой рандомизации. Например, здесь, ты можешь подсунуть другую реализацию, которая всегда будет возвращаться одно и то же число. Это удобно для тестов (даже без учёта юнит-тестов и интеграционных). Кроме того, ты защищаешь свой рандом от сайд эффектов. UnityEngine.Random небезопасен из-за того, что сид может меняться в любой точке проекта из-за его статики. 3. По поводу повторяющийхся строк в описании исключений (таймкод 17:37). Есть ещё вариант просто-напросто сделать кастомное исключение и выбрасывать его, унаследовав от нужного. Тогда не было бы необходимости повторять строки.

Нормально выполненное задание для ТЗ. Не факт что автор сделает лучше. Открой любой пронект и можно сделать подобный ролик

Интересные видео,, с Вами можно как то связаться? Интересует клиент сервер на c# unity и в целом возможность переноса игры на движёк юнити или воссоздание и ли оно того?

Интересно, а можно ли Вам проект на обзор скинуть? Джуны, но вроде пишем понятно

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

А не слишком ли перегруженная архитектура, стейтмашина, кастомный проброс зависимостей вместо использования готового DI?

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

Добрый день. Не нашёл ваших контактов. Есть ли у вас платная услуга код ревью?

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

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

Привет, работу не ищешь случайно?) У нас создается новая команда под юнити и это первая юнити команда в компании(другие игры были под веб на typescript), пока еще вроде никого не нашли, удаленка, компания из США, но все в основном из стран снг, не гк, проекты которые мы делали довольно таки серьезные, все игры онлайн, ситибилдеры и шутер от первого лица, нфти, крипта и подобное

Привет, а куда можно в личку написать?

9:40 Ну насколько я понимаю он используется для сохранения данных в бинарном формате чтобы пользователь не мог отредактировать Это как бы прогресс игрока и так же защита данных от редактирования со стороны пользователя платформы, так что вполне оправдано его использование

Получается не разбор тестового, а скорее разбор архитектуры от K-Syndicate :) Ничего нового вообще не было добавлено. Я конечно сам ее юзаю в некотором варианте, но если бы делал тестовое, что наверное бы делал что-то свое. Ведь каждый джун уже пишет так, как конкурировать если все одинаково?)

у меня есть джуниоровское тестовое задание 3х летней давности когда я был в программировании месяца 1.5 пойдет на обзор? :D

ЗП очень маленькое для Мидла((( Так везде в гемдеве?

1400 долларов за миддла ну да это жестко)

Я не могу понять что вы все прицепились к тестовому заданию? Оно супер элементарное учитывая что чел идёт на мидла. Само задание умещается на одной странице документа, где большую часть занимают расписанеые статы. А чел взял и буквально НАСРАЛ в код. Тебя просили срать? Такой сотрудник никому нахрен не всрался, по факту такие люди будут только мешать/спорить/тратить время впустую. Есть задача, она четко прописана - тебе надо её сделать. ВСЁ! Не сидеть придумывать кучу сервисов и всякой херни которые никогда не будут использованы и на которые ты потратишь в 200 раз больше времени чем должен был, и недайбох кому то другому кроме тебя придется это читать.

Такое впечетление, что после курсов синдиката код написан)

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

Что-то есть в папке Resources? Fail ! Из всего делаем гребанные адрэсибэлс и ничего не засовываем в ресорсэс. Нах** столько классов и интерфейсов - элемент в grid layout - гребанный префаб с одним скриптом на нем и один скрипт с интерфейсом на все окно, еще файл другой на настройки и модели данных и не е**м никому мозги.

очень круто, но очень тихо(

что это вообще такое? Ниче не понимаю... Где такому уровню учат?

Соглашаться на тестовые задания - это ошибка

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

Код под копирку с курса @KSyndicate по архитектуре :D

Привет!!! Как с тобой можно связаться? Хотел бы пообщаться