Худшая реализация оружия на C# в Unity? Космический шутер \ Code Review

Поділитися
Вставка
  • Опубліковано 15 лис 2024

КОМЕНТАРІ • 50

  • @mesa.
    @mesa. 4 роки тому +21

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

    • @Michael_Sh
      @Michael_Sh 4 роки тому +2

      Потому что половина анализа кода это просто вкусовщина. Хотя есть интересные вещи.

    • @dictatorship7413
      @dictatorship7413 3 роки тому +3

      @@Michael_Sh, ну, не то чтобы вкусовщина. Кодстайл - достаточно важно. Но, обычно, кодстайл зависит от компании/организации. А если её нет, то можно воспользоваться рекомендациями Майкрософт.

  • @DeadRabbitCanDance
    @DeadRabbitCanDance 4 роки тому +21

    РОМАН, Сделай курс по тому как правильно. С паттернами, со всей фигней, но не забудь про то, что не всё делается моментально. Последовательный перенос из функциональных парадигм в .... Или потом быстро всё перенести в тот же ECS. Быстро! Быстро и понятно. Или вынести Физику отдельно от Графики, это ж постоянная потребность - ведь на сервере не нужна графика. И создать систему адекватного обмена состояниями... чтобы в одном и том же классе была кучка параметров, которые нужно синхронизировать и которые не нужно синхронизировать. Как РЕФАКТОРИТЬ без анусболи? И не говори только "Начните изучать всё с 0".

  • @victorpavlovitch
    @victorpavlovitch 4 роки тому +29

    Активируй винду

  • @kolesnikovproduction4857
    @kolesnikovproduction4857 4 роки тому +5

    Заметил, что Ваш ученик всегда старался использовать Update. Но можно было обойтись и FixedUpdate в геймплейных элементах. А в Update чекать только устройства ввода.
    По красоте кода ученик молодец.
    Можно было упомянуть, что атрибуты можно прописывать через запятые, например, если используется Header и SerializeField только для одного поля. Также можно удалить неиспользуемые библиотеки, такие как System.Collection.
    Увидел, что вообще не используется тип var. Мог бы писать строчки по короче.

    • @nooftube2541
      @nooftube2541 4 роки тому +1

      Аттрибуты через запятую это один из самых вредных советов :)
      Читать сложнее становится. Но иногда (sic!) бывает удобно, но точно не в этом случае.

    • @kolesnikovproduction4857
      @kolesnikovproduction4857 4 роки тому

      @@nooftube2541 Скорей всего нужно было уточнить, что это относится к code style команды разработки. "Читается легче", это в принципе понятие индивидуальное.

  • @svoboda6930
    @svoboda6930 4 роки тому +38

    weapon - випон. понимаю.

    • @maximilianoeaston6283
      @maximilianoeaston6283 3 роки тому

      i know it is kind of randomly asking but do anyone know of a good site to watch new tv shows online ?

    • @ariesgregory1072
      @ariesgregory1072 3 роки тому

      @Westley Leonardo Yup, been using flixzone for months myself =)

    • @maximilianoeaston6283
      @maximilianoeaston6283 3 роки тому

      @Westley Leonardo Thanks, I signed up and it seems like a nice service :) I really appreciate it!!

    • @westleyleonardo1473
      @westleyleonardo1473 3 роки тому

      @Maximiliano Easton You are welcome xD

  • @alexanderalexander1637
    @alexanderalexander1637 4 роки тому +2

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

  • @darksega1
    @darksega1 4 роки тому +1

    Все правильно Роман. В книжках тоже азы на 500 листов растягивают, а денег дофига просят.

  • @vklymenko
    @vklymenko 4 роки тому +1

    Всё видео не смотрел, но для "денежных" операций вместо float лучше использовать что-то другое, например decimal, потому что float очень "примерно" показывает значение, подходит только для position/rotation, к пример 0.999997 вместо 1

    • @nooftube2541
      @nooftube2541 4 роки тому

      Ну это все таки игра а не банк :)

    • @vklymenko
      @vklymenko 4 роки тому +1

      @@nooftube2541 если в игре что-то покупается за реальные деньги (in-app purchases) то это уже не игра, а банк = )

    • @nooftube2541
      @nooftube2541 4 роки тому

      @@vklymenko там вроде не реальные деньги были.
      Если реальные, то да - стоит использовать децимал.

  • @Tera-h7e
    @Tera-h7e 4 місяці тому

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

  • @ellisisland9845
    @ellisisland9845 4 роки тому +4

    Ваш сайт постоянно валится, нет возможности заходить постоянно. Выводится ошибка "Service Unavailable
    HTTP Error 503. The service is unavailable."

  • @olehfilipchuk8257
    @olehfilipchuk8257 4 роки тому +4

    ага - и рефренсы прикольные "C:/Users/Rushana/Desktop/C# Light/Lesson8/Space Battle/Library/ScriptAssemblies/..."

    • @ЕвгенийСкворцов-т2ш
      @ЕвгенийСкворцов-т2ш 4 роки тому

      Что в них прикольного?)

    • @GOLEM-lg1yn
      @GOLEM-lg1yn 4 роки тому

      @@ЕвгенийСкворцов-т2ш видимо он тупо создал этот код в процессе просмотра видео.

  • @Beep22
    @Beep22 Рік тому

    Я серанул, подумал что у меня ключ винды слетел)))

  • @КушанСамарасингхе
    @КушанСамарасингхе 4 роки тому +4

    Где ссылка про делегаы и собития?

  • @ЫыКек
    @ЫыКек 4 роки тому +7

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

    • @victorpavlovitch
      @victorpavlovitch 4 роки тому +3

      Согласен, тоже не увидел проблемы использования сеттеров по прямому назначению.

    • @nightyonetwothree
      @nightyonetwothree 4 роки тому +4

      в том что можно было бы сделать это компактно, особенно с учётом на рост колличества этих самых свойств

    • @nooftube2541
      @nooftube2541 4 роки тому +2

      @@nightyonetwothree рефекторинг ради рефакторинга и код ради кода - это крайне плохой подход.
      А если в классе будет много методов, будете тоже количество методов уменьшать из за того что "много методов"? (Я не говорю о нарушении SRP)
      Проперти это такие же методы, и уменьшать их количество только потому что "они слишком много места занимают", это что то какой то крайне дурацкий подход.
      В крайнем случае есть ctrl shift m

    • @АлександрБычко-п9ъ
      @АлександрБычко-п9ъ 2 роки тому

      @@nooftube2541 в этом случае можно использовать тип данных не поддерживающий отрицательные значения и не нужно писать кучу проверок.

    • @nooftube2541
      @nooftube2541 2 роки тому

      @@АлександрБычко-п9ъ это точно ответ мне?

  • @Den-zanoza
    @Den-zanoza 3 роки тому +1

    Увеличь разрешение! Глаза выжег на телефоне смотреть...

  • @АлександрБычко-п9ъ

    Да уж без событий хорошо расширяемый код не напишешь, а вы этому не учите. Это можно сказать основа ООП.

  • @nooftube2541
    @nooftube2541 4 роки тому

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

  • @АртёмС-п1в
    @АртёмС-п1в 4 роки тому +9

    как всегда нехрена не понятно

    • @ПростоПроги
      @ПростоПроги 4 роки тому +1

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

  • @tim4ig
    @tim4ig 3 роки тому

    Ещë не смотрел, но уверяю, мой код хуже, он даже не заработал)))

  • @LuCkyRead
    @LuCkyRead 4 роки тому

    Я хочу научится программировать и создавать игры подскажите где это можно сделать?

    • @valerysemenkin6024
      @valerysemenkin6024 4 роки тому +1

      Читай книги по программированию,их очень очень много.Выбери язык,к примеру Python,загугли какую лучше книгу читать,точнее по который лучше учить.ИЗИ
      Хочешь делать игры,ты для начала должен уметь программировать,разумеется. Выбрав движок гуглишь туториалы по разработке,какие либо уроки.Черпаешь информацию из интернета, книг,да откуда угодно.Тебя не кто и нигде научит ,учись сам.Удачи!

    • @NicoFye
      @NicoFye 4 роки тому +5

      В создании игр самое главное графика. Программирование не такое сложное и юзер его не видит, поэтому можно просто писать говнокод и игра будет получаться успешной (для начала сойдет). А вот если в команде нет художника/3D моделлера/аниматора, то ты со своими идеальными навыками программирования можешь разве что текстовую адвенчуру запилить и сидеть курить бамбук.

    • @valerysemenkin6024
      @valerysemenkin6024 4 роки тому +4

      @@NicoFye в играх главное механика,или же идея.Плохой код значит кривая игра,не оптимизированная,она будет вызвать отторжение у игрока,он просто бросит игру.3D моделингу,анимации и изобразительному искусству не так уж и сложно научится.Это обычные навыки,их развить можно за короткий срок,а моделинг так вообще за месяц упорной работы даст большие плоды. Я со своими идеальными идеальными навыками программирования буду востребован в геймдеве,и буду способен делать хорошие и рабочие игры,которые работают стабильно.Самое сложно это КОД!Чтоб писать его хорошо нужно около года учить язык,параллейно практикуя,всё остальное не так сложно как код!

    • @skarabeydm
      @skarabeydm 4 роки тому +1

      @@NicoFye нууу графику можно и накопать в интернете, а вот с кодом по сложнее

    • @ilyas3507
      @ilyas3507 4 роки тому +1

      @@NicoFye Нет,это чертовски важно. Графика-это хорошо,но без кода,или его нормальной реализации - очень туго. Графику можно научиться рисовать,или же иметь художника.

  • @ОлегМашков-я8х
    @ОлегМашков-я8х 2 роки тому +2

    Туповатое видео... для сравнения показал бы как надо, например на UML схеме что не так.👎

  • @ravenempty3562
    @ravenempty3562 4 роки тому +7

    Дизлайк за кликбейт ^-^

    • @kamrielus
      @kamrielus 4 роки тому

      Ну уж точно не лучшая)

  • @smilerok
    @smilerok 4 роки тому +5

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