1) Удаляйте неиспользуемые using

2) UnitsSpawner - переменная Unit unit в функции SpawnUnit не нужна

3) HasFreeUnits - можно сократить public bool HasFreeUnits => _freeUnits.Count > 0; и кстати это свойство не используется в коде

4) ResourceStorage - лучше пишите методы событий Unity в порядке их вызова (Awake выше OnEnable, Start ниже OnEnable)

5) ResourceStorage - если класс называется хранилищем ресурсов, то он наверное должен выступать только складом, а не базой данных для бронирования ресурсов, для этого нужен отдельный класс

6) можно не делать явное сравнение с правдой, например (isAlive == true) или isAlive != false). Результатом сравнения будет всё тоже значение, поэтому эта операция не нужна

7) Unit стр 41 - вместо постоянного ручного перемещения объекта можно просто прекрепить его как дочерний объект и открепить когда довезём

8) Base

1. после { и перед } пустые строки не нужны

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

9) Builder - метод BuildBase ничего не делает, буквально, в нём кода нет

10) Player

1. поле _isBuildMode не используется

2. у всех членов класса должны быть явно указаны модификаторы доступа

3. переменные должны именоваться в стиле camelCase

1) if (Vector3.Distance(positionNearby, transform.position) < _sphere.radius) - "Vector3.Distance - Это очень затратная операция из-за вычисления корня в конце. У процессора с этим туго. Везде где возможно лучше вместо дистанции использовать sqrMagnitude, это даже в документации написано (просто сравниваешь не с самой дистанцией, а с ее квадратом). https://news.1rj.ru/str/KaDR_gamedev/83
2) public event Action<Resource> UnitUnloadResource;
3) public event Action ReadyForNewTask;
4) public event Action<Resource> UnitAimedAtResource; - события именуются в прошедшем времени с окончанием -ed или -ing
5) StartCoroutine(nameof(MoveToResourse), resource);
6) StartCoroutine(nameof(MoveToBase), resource); - немного странное решение. Почему сразу не написать StartCoroutine(MoveToBase (resource));
7) if (_resourceStorage.HasFreeStorageSpace == true) - не делайте явную проверку с true. Ведь результатом сравнения будет всё тоже значение
_base.GetComponent<ResourceDistributor>().Initialize();, а если на базе нет компонента ResourceDistributor?
8) if (_generator == null)
_generator = FindObjectOfType<ResourceGenerator>(); - я бы предпочел в OnValidate выводить ошибку что поле не заполнено

1) Удаляйте неиспользуемые using

2) Base

1. FindObjectOfType - не используйте методы Find для получения ссылок на компоненты объектов со сцены, это неявное обращения к объектам и трудозатратное по производительности, также и с ResourceStorage

2. в OnDisable происходят подписки вместо отписок

3) можно не делать явное сравнение с правдой, например (isAlive == true) или isAlive != false). Результатом сравнения будет всё тоже значение, поэтому эта операция не нужна

4) ResourceDistributor - более 1 пустой строки подряд быть не должно

5) UnitsDepot - SendFreeUnitToGathering - условные операторы отделяются от остального кода пустой строкой с двух сторон

6) Builder

1. переменные должны именоваться в стиле camelCase

2. Input.GetMouseButtonDown(0) - магия

Продолжаем работать в проекте задачи Боты сборщики, добавим новый функционал:

1. Когда на базу принесли три единицы ресурсов, она тратит их на создание нового юнита, который ведет себя также, как и старые юниты.

2. У каждой базы должна быть своя коллекция ресурсов.

3. Игрок может установить флаг базы на уровне, кликнув на базу, а потом на любую точку плоскости. Флаг невозможно установить за пределами карты, а также у одной базы не может быть больше одного флага. Если в момент клика флаг базы уже находится на уровне, то он должен быть переставлен в новое место.

4. Когда флаг базы поставлен, ее поведение меняется, устанавливается новый приоритет. По этому приоритету база ожидает сбора 5 единиц ресурсов, а потом тратит их на отправку свободного юнита к флагу, чтобы тот построил там новую базу. С этого момента отправленный юнит принадлежит новой базе.

5. При постройке новой базы флаг исчезает и поведение старой базы возвращается к постройке новых юнитов.

6. Нельзя построить новую базу, если у вас остался один юнит.

Привет!
Сейчас у тебя ресурсы попадают в свободные напрямую из спавнера. Всем классам должно быть все равно на то, откуда ресурсы появляются на карте. Важен сам факт наличия ресурса. Сделай поиск ресурса полностью через сканнер.

Так же релизить ресурс напрямую в спавнер не нужно. У тебя появляется лишняя зависимость на спавнер. Заставь спавнер подписываться на событие объекта, который он заспавнил. Тогда, чтобы зарелизить ресурс, базе достаточно вызвать resource.Release.

Использовать FindObjectOfType не нужно. Сделай у базы метод инициализации. Туда и передавай все необходимое.

У тебя сейчас у бота несколько корутин. Если хочешь, можешь переделать на что то подобное:
public IEnumerator Collect(Resource resource)

{

yield return _botRotation.SmoothLookAt(resource.transform); // обычный метод IEnumerable

yield return _botMovement.MoveTo(resource.transform);

_botAnimator.PlayLift();

yield return new WaitUntil(() => _botAnimator.IsLifting);

yield return _botRotation.SmoothLookAt(_homeBase.transform);

yield return _botMovement.MoveTo(_homeBase.transform);

MissionCompleted?.Invoke(resource);

} - все методы будут выполняться последовательно.

|         Название         |             Размер              |     комментарий      |
| ------------------------ | ------------------------------- | -------------------- |
|          godot           |             ~ 40 мб             |    Без изменений     |
|          godot           |             ~ 10 мб             | Перекомпилированный  |
|          unity           |            ~ 10.7 мб            |          3д          |
|          unity           |            ~ 7.7 мб             |          2д          |
|          p5.js           |             4.5 мб              |          3д          |
|          defold          |             3.5 мб              |          3д          |
|         Heaps io         |             2.5 мб              |          3д          |
|         Picotron         |             1.9 мб              |          2д          |
|         Scratch          |             1.8 мб (Turbowarp)  |          2д          |
|         Gdevelop 3д      |             1.6 мб              |          3д          |
|        Game maker        |             1.6 мб              |          2д          |
|          Phaser          |             1.5 мб              |          2д          |
|        Construct         |             1.5 мб              |          2д          |
|          Ct.js           | 1.1 мб + 0.8 мб иконки = 1.9 мб |          2д          |
|         Gdevelop 2д      |             930 Кб              |          2д          |
|         three.js         |             700 мб              |          3д          |
|         Pixi.js          |             460 Кб              |          2д          |
|          Kaplay          |             160 Кб              |          2д          |
|         Point.js         |              96 кб              |          2д          |
|        Little JS         |             7-8 Кб              | Версия для 13k джема |
| CanvasRenderingContext2D |               0 б               |          2д          |