TGViewer
Moth Moth @namlessmoth · 44 subscribers
Post #319 111
Я исправил предыдущие ошибки и решил пойти немного дальше перед тем как получить ответ

Вот решение: https://github.com/NoNameDeleted/BotsCollectors/tree/01f339eb8496d894bebad85fb3af69745a07bdb1

И вот отзыв по нему:

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

Мои мысли: вобщем зря наверное я начал делать следующее задание (колонизация) которое является продолжением этого задания (боты сборщики) потому что я начал накидывать некоторые "намётки" и не доделал их, думая что проверяющий поймёт что они не являются частью задания и пропустит недоделанные части, но нет
Ну ничё, я уже почти доделал второе задание ещё не сдав первое
И кстати это самое последнее задание в блоке юнити и после него я могу с гордостью заявить что прошел юнити на 100%
  • 🔥 2
  • 👍 1
More from @namlessmoth
  1. Mar 7, 2026Как сломать босса одним прокачанным навыком Причём первую заратану я победил точно так же,…
  2. Mar 3, 2026Наиграл уже 150+ часов и даже не прошел половину игры Мне капец как не везёт почти в каждо…
  3. Feb 17, 2026Один месяц подписки на впн Вдруг кому надо https://t.me/gruVPNbot?start=g_UZhunlWNfFC_2127…
  4. Feb 16, 2026оставлю тут, чтобы не потерять вам смотреть нельзя, это только для меня
  5. Feb 13, 2026Вчера сделал всего три задания и все три оказались не правильными 1) Задание: Проведите ре…
  6. Feb 12, 2026Оказалось что всё-таки можно сдавать несколько заданий за один раз. Я вчера сдал 7 заданий…
Threads Profile ViewerView any public Threads profile without an account.Open ThreadLook →Writing with AI? Make it sound human.Metric37 rewrites AI drafts so they read naturally. Free AI detector, 1,500 words free.Try Metric37 →