C# и Unity так ещё не унижали! CodeReview подписчиков

Sdílet
Vložit
  • čas přidán 3. 11. 2021
  • Улучшу ваш код и подниму вашу зарплату в два раза - ijunior.ru/napilnik?...
    Обучение разработке игр с нуля с гарантией трудоустройства - ijunior.ru/unity-start?...
    Одежда - ijunior.ru/wear?...
    МОЯ КНИГА - csharpbook.sakutin.ru
    Я В VK - rsakutin
    INSTA - / sakutinhuytin
    ЯЮниор:
    Группа - holymonkey_sandbox
    ЧАТ В ТЕЛЕГЕ - t.me/csharp_faggots_fan_club
    Discord - / discord

Komentáře • 119

  • @rsakutin
    @rsakutin  Před 2 lety +9

    На курсе напильник я лично провожу ревью вашего кода - ijunior.ru/napilnik?

    • @MrBastards
      @MrBastards Před 2 lety +1

      Рома напомни, какие книги читать по языку C#?

    • @yoou6726
      @yoou6726 Před 2 lety

      @@MrBastards а ты точно поймёшь их, просто читая?

    • @MrBastards
      @MrBastards Před 2 lety

      @@yoou6726 их я точно пойму, если они достойные

    • @user-co8ek3bj5n
      @user-co8ek3bj5n Před 2 lety

      тест

    • @user-bg9gl9fl9y
      @user-bg9gl9fl9y Před 2 lety

      А если упороться и IsMoving спарсить в int и в рассчете скорости бега или высоты прыжка всегда умножать на Int(IsMoving)?
      Типо если мы двигаться не можем, то мы умножим скорость на ноль => стоим на месте, а если может ( =1 ) , то умножаем на 1 и получается что получается.
      Или это больше похоже больше на костыли, чем фичу?

  • @pom01ka
    @pom01ka Před 2 lety +70

    Начал смотреть этот канал неделю назад,понял ,что я ничтожество ,спасибо

    • @Makhran
      @Makhran Před rokem +6

      Не расстраивайся, главное ты понял кто ты) 👍
      P.s. А сейчас кто?

    • @pom01ka
      @pom01ka Před rokem +2

      @@Makhran Сейчас я бог программирования.А если серьезно ,то за эти 11 месяцев я сильно вырос в коддинге)

    • @_resfar_26
      @_resfar_26 Před rokem

      @@pom01ka блин это очень круто

    • @GLOK7.62
      @GLOK7.62 Před 9 měsíci

      А сейчас как дела? 😅

    • @user-qf9so9hk2v
      @user-qf9so9hk2v Před 5 měsíci

      ему некогда отвечать он взламывает пентагон@@GLOK7.62

  • @shaikhel
    @shaikhel Před 2 lety +25

    Нужен ли английский айтишнику?
    -Чарактер

  • @user-hu2vm8vx2n
    @user-hu2vm8vx2n Před 2 lety +97

    После этого видео хрупкая дружба с Хауди опять рассыпется 😭

    • @Garneg
      @Garneg Před 2 lety +6

      Ничего личного, просто плохой код

  • @-it394
    @-it394 Před 2 lety +52

    Спасибо за разбор! Все по делу:) Если будет ещё такая возможность, то постараюсь уделить больше времени коду и учесть косяки, в этот раз поздно узнал про данную затею с ревью, и пару вещей действительно не успел нормально почистить(свитч например ахпхаха). А пока буду учиться дальше!)

    • @vikkia4011
      @vikkia4011 Před 2 lety +4

      Молодец.

    • @fgggg1463
      @fgggg1463 Před 2 lety +4

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

    • @-it394
      @-it394 Před 2 lety +1

      @@fgggg1463 Спасибо большое!)

    • @-it394
      @-it394 Před 2 lety

      @@vikkia4011 спасибо!)

    • @-it394
      @-it394 Před 2 lety

      @@Lipatoff1992 Согласен, тут у меня отголоски физики) Хочется разграничить понятие модуля вектора скорости и самого вектора скорости, но писать вектор вектор скорости тупо, и вот вырождается что-то подобное) На самом деле над названиями и пониманием того как кратко и исчерпывающе называть методы и поля мне надо поработать. А по поводу умножения вектора на число я знаю, просто с моей реализацией было неудобно умножать весь вектор сразу и пришлось вот так изгаляться, согласен, что не очень красиво) Но буду стараться улучшать код, спасибо за замечания и советы!

  • @bomb5994
    @bomb5994 Před 2 lety +5

    "Когда это handler, оно что-то handle, обрабатывает, а когда ты что-то делаешь, ты что-то делаешь" 😎

  • @trell_7842
    @trell_7842 Před 2 lety +7

    11:15 в новых версиях юнити она кеширует все объекты с тегом MainCamera и будет брать первое валидное значение из кеша. Правда, каждый раз будет вызываться геткомпонент.

  • @naxa1ka
    @naxa1ka Před 2 lety +10

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

  • @AltarastLD
    @AltarastLD Před 2 lety +1

    Подскажите пожалуйста, как может влиять на оптимизацию psx shader для unity (Или PS1 стилистика в целом) ?

  • @krushiler398
    @krushiler398 Před 2 lety +3

    Про isStoped или stoped. Это зависит от тех, кто писал код. Например, в гугловских библиотеках для java/kotlin исполбзуется is. В тех же вьюшках есть isEnabled, isFocusable и т.д. Ну и впринципе на андроид это как-то принято писать

  • @v_karna
    @v_karna Před 2 lety +2

    Я вижу, Роман тоже понял, что приставка "Game Dev" прибавляет популярности:) (без шуток)

  • @Cyneprepou
    @Cyneprepou Před rokem

    Спасибо за видео. Подскажите про курс "напильник": сколько он повремени, а то на сайте непонятно написано.
    Спасибо.

  • @abyssfrk6388
    @abyssfrk6388 Před 2 lety +3

    Роман, нужно больше дружбы с хоуди!

  • @ewerestr
    @ewerestr Před 2 lety

    Привет. Ты обозреваешь только Unity проекты или прочие C# проекты тоже?

  • @user-it9vj7vw2z
    @user-it9vj7vw2z Před 2 lety +2

    Ты говорил, что скинешь ссылки на их каналы. Где?

  • @nice2meetu92
    @nice2meetu92 Před 2 lety

    Йо, умею неплохо моделить, можешь дать совет как попасть в индустрию, где искать команду и тд.тп

  • @user-lm8ed9js4e
    @user-lm8ed9js4e Před 5 měsíci

    @rsakutin как к питону относишься?

  • @user-im8gq7om5h
    @user-im8gq7om5h Před 2 lety +1

    аналогия с кружкой до слёз! 🤣

  • @AibekDandaev
    @AibekDandaev Před rokem

    захзааз
    вы лучший Роман!)

  • @steel_beard
    @steel_beard Před 2 lety

    Если ваш код полон, то вы программист, иначе вы подписчик😂. Шутка про стаканы просто зашла...

  • @bug7894
    @bug7894 Před 2 lety +1

    чел который написал код на другом языке и связал по FFI:
    "Never gonna give you up, never gonna let you down"

  • @kawaikami
    @kawaikami Před 2 lety

    [было весело. лукусик]

  • @andreiivanov6921
    @andreiivanov6921 Před 2 lety +2

    Ну ссылка на курс есть так что все в порядке)))

  • @victorpolyakov3031
    @victorpolyakov3031 Před 2 lety

    Охох. Штош - у меня много работы по рефакторингу xDD

  • @user-pg1ed3jw1p
    @user-pg1ed3jw1p Před 2 lety

    А ты берешь ревью игры сделанной на WPF ?)

  • @MrDarts-ff6jq
    @MrDarts-ff6jq Před 2 lety +1

    Никого не смутило, что хауди полез править код плагина?

  • @ZEkA10000
    @ZEkA10000 Před 2 lety +4

    7:30
    Я вот тоже подумал над изменением кода в MoveCharacter.
    Это хорошая практика которую я видел конечно в основном на Ruby... да не суть важно.
    И я согласен что лучше сделать так:
    public void MoveCharacter(Vector3 vector3) {
    if (IsStopped) return;
    ...
    }
    Но я также не понял смысла делать IsStopped как метод если он работает как обычная булева переменная.

    • @evgenijtsvetkov7008
      @evgenijtsvetkov7008 Před 2 lety +2

      Суть в том что все проверки можно завернуть в методы, для удобного чтения.
      Например, как вот тут :
      if(Vector3.Angel(...) > 0 && !IsStoped)
      { //другой код }
      завернуть в
      if(CanNotRotate())
      return;
      private bool CanNotRotate() =>
      Vector3.Angel(...) > 0 && !IsStoped;
      читаешь такой метод, если не может вращаться то выходим из метода, а если интересует проверка как почему, заходишь в метод и смотришь какие там проверки и условия
      Или как в видео было сказано, сначала проверяешь
      if(CanRotate())
      Rotate();

  • @WoozyRacer
    @WoozyRacer Před 2 lety

    Роман, а что у тебя за татуха на шее?

  • @slava6105
    @slava6105 Před 2 lety

    6:00 главное, что на предлог обратил внимание, а на правописание забил: stopped с двумя p

  • @ricardomilos857
    @ricardomilos857 Před 2 lety +12

    Я считаю что флаг нужно называть с is, а событие без is
    Считаю что так более читаемо. Хотя у вас в конвенции .NET методы пишутся с большой буквы и при чтении мы знаем что является методом, а что переменной. Но не в случае свойств. Исходя из этого, лучше добавлять Is в начале в булевого свойства

  • @Priboy313
    @Priboy313 Před 2 lety +4

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

    • @extrageniuz
      @extrageniuz Před 2 lety

      хахахахах

    • @rsakutin
      @rsakutin  Před 2 lety

      Звучит правдоподобно но я бы не акцентировал на этом внимание если он не делал так регулярно

  • @zatr1k600
    @zatr1k600 Před 2 lety +2

    Здраствуйте, Роман. Вчера на своём канале я выложил ролик с обзором кода моего проекта. Буду рад, если вы раскритикуете его в следующем видео по этой рубрике. Думаю, что ваша критика поможет мне понять свои не очевидные ошибки, которые я допустил и сам не заметил. Скорее всего, из-за моего малого опыта в разработке на юности таких ошибок найдется немало. Я к критике всегда относился положительно и считаю, что она является двигателем прогресса! Особенно она даст большой толчок, ведь я еще нахожусь на ранних этапах изучения юнити!

  • @Shirosugia
    @Shirosugia Před 2 lety

    Реклама напильника вышла на новый уровень стёба.

  • @FERZiNANDoS
    @FERZiNANDoS Před 2 lety

    Оу! Да он говорит как настоящий миссионер!

  • @elezthemdevs
    @elezthemdevs Před 8 měsíci

    Ну у новичков вот так, у Романа тоже уверен вот так делали обзор кода когда он только начинал)

  • @FZ530
    @FZ530 Před rokem

    9:54 я думал это у меня плеер слетел до качества 144 px, я аж дернулся пока не понял что это так задумано, подловил, красава

  • @DarkW1zard
    @DarkW1zard Před 2 lety +2

    Мы с хауди помирились и началось 14:55 :)

  • @user-ct3pn9ib6f
    @user-ct3pn9ib6f Před 2 lety

    так у хауди хо класс из ассета, хотя может конкретно этот кусочек он для видео написал сам.

  • @mrakov
    @mrakov Před 2 lety

    Ещё пару видео с рекламой напильника, и я запишусь

  • @T1m0the0S
    @T1m0the0S Před 2 lety

    Должно быть три кружки. Третья во всех остальных случаях

  • @Golanhast
    @Golanhast Před 2 lety +3

    4 минуты назад, обожаю...

  • @tashchan
    @tashchan Před rokem

    1:21
    С третьим фанатом угар))

  • @dimon4071
    @dimon4071 Před 2 lety

    Чел а где твои большие проекты ?

  • @EdikTVSet
    @EdikTVSet Před 2 lety +6

    А шото как бы какое-то видео про разрушение объектов будет? Ты же обещал. Мы договаривались(

    • @rsakutin
      @rsakutin  Před 2 lety +10

      Я обязательно им займусь, думаю к концу месяца выйдет

  • @sergei-fil
    @sergei-fil Před 2 lety +1

    Что второго участника слабо разобрал, а человек старался, записывал.

  • @user-wo9we6ue9f
    @user-wo9we6ue9f Před 2 lety

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

    • @UltraChort
      @UltraChort Před 2 lety +4

      23 часа в сутки

    • @Osarikas
      @Osarikas Před 2 lety

      @@UltraChort чет мало

  • @themihanoid5020
    @themihanoid5020 Před 8 měsíci

    Зацепился взгляд на нескольких моментах из видео:
    1. В начале видео автор произносит C Sharp, затем называет переменную isMovement и [ЧА]рактер. Пропагандируешь англоязычное произношение - иди до конца (по мере необходимости качай английский)
    2. Про класс GravityHandler ничего толком не сказал кроме "ну такоое"
    Там ведь куда больше замечаний (по делу) можно было добавить:
    Что такое вообще вектор VelocityDirection?
    В одном случае он константный 0.5 (и не зависит от deltaTime), в другом - к нему прибавляется "сила" умноженная на время (хотя под силой на самом деле тут имеется в виду ускорение)
    И почему он назван Direction. Cлово direction можно (и нужно) опустить в данном контексте, потому что под "направлением" обычно имеется в виду нормализованный вектор, тут это явно "speed" c направлением, то есть просто "velocity"
    3. Про дубляж кода в свиче - не могу согласиться. Не так просто там написать недублирующийся код. Вернее как, его можно сделать более читаемым - да, но если вчитаться, там из дублирующегося - сами методы (не параметры методов) Translate и Distance. Проверки разные, знаки переменной разные, в одну функцию сложно будет вынести, а в две - так ли нужно? (Возможно, если дать им говорящие названия типа ApplyGravityWhileGrounded, ApplyGravityWhileFlying, но так ли много в этом смысла)

  • @morfeus23
    @morfeus23 Před rokem

    Ничего не понятно, но очень интересно

  • @CoolinarGameDeveloper
    @CoolinarGameDeveloper Před 2 lety

    Ром сделай разьеб "eccentric games" ребята продают говнокурсы по 50к, грустно(

  • @suslikest3708
    @suslikest3708 Před 2 lety

    Ты чем то по деятельности лев против напоминаешь я думаю если бы го...о кодеры собирались кодить в пятницу вечером в парке то ты бы рейдил их отбирая ноуты и выкидывая их в мусорку😅😂🤣

  • @reosfire
    @reosfire Před 2 lety +4

    есть два стула SitOnFirst(false);

    • @phat80
      @phat80 Před 2 lety +3

      string result = SitOn() ? "piki" : "hui";
      Public static bool SitOn() {
      ….
      }
      Остается реализовать метод SitOn и игра готова!

  • @user-ux6kd2qc5y
    @user-ux6kd2qc5y Před 2 lety

    Внизу экрана это вмятина??? )))

  • @MrArtrynk
    @MrArtrynk Před 2 lety

    почему так написали... да потому что можем) странно, но за _variable меня часто сношали сеньеры , им эта _ не нравилась (я техарт) , до сих пор дергается глаз.

  • @DarkW1zard
    @DarkW1zard Před 2 lety +1

    Рома хватит фигней страдать устройся уже на работу, в кефир например :))

  • @pickramag
    @pickramag Před 2 lety

    Сделай разбор кода Dani

  • @skander8447
    @skander8447 Před rokem

    а теперь прикиньте, смотреть это видео, с 2 по матике, 3 по инглиш, геометрия-эт еще чё такое? и еще под водочку, ни.я не понял но очень интересно

  • @user-sb4kj4pt4o
    @user-sb4kj4pt4o Před 2 lety

    Неуж то звук подтянули!

  • @forfedix1019
    @forfedix1019 Před 2 lety

    Где видосы?)

  • @smoke_sss_3811
    @smoke_sss_3811 Před rokem

    Вася мне друг, но истина дороже!

  • @theairblow
    @theairblow Před 2 lety

    Помню ещё в 10 писал прям такую жирную игру на юнити, в итоге все было засрано говнокодом и я только исправил сохранения (Был BinaryConverter, стал JSON), а часть функций вообще не работала.
    Жалко что сурсы просрал. Называется, забыл на кнопочку Publish нажать.

  • @solidesuu
    @solidesuu Před 2 lety

    Чет ты более-менее проревьюил только первого чела, а нахалку нахально скипнул и на хауди чуть-чуть погнал

  • @vatyunga
    @vatyunga Před 2 lety

    Кто такой черектер? Знаю только керектер.

  • @eugenesolyanov3515
    @eugenesolyanov3515 Před 2 lety

    очень много придирок, мол я вот люблю так, а оно не так, буууу

    • @user-gi4pl5wh8e
      @user-gi4pl5wh8e Před rokem

      по факту весь его ревью был только +- в проверке стайлгайда.
      слабенько

  • @SergeySvotin
    @SergeySvotin Před 9 měsíci

    2:10 я 4 раза послушал, но так и не понял, видимо, проблемы с английским - это крест вашей компании)

  • @user-yy8yv6sj4y
    @user-yy8yv6sj4y Před 11 měsíci

    еще как унижали и еще сильнее, например я унижал пока пытался разобраться с unity

  • @user-kn7yv8lu1b
    @user-kn7yv8lu1b Před 2 lety

    За Хабуди Хабудая посмеялся

  • @puripuripupuripu2987
    @puripuripupuripu2987 Před 2 lety

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

  • @World_Dota_Champions
    @World_Dota_Champions Před 2 lety

    Посмотрел, подумал.... UE4 не так уж и плох.

  • @MooNDesigns
    @MooNDesigns Před 2 lety +1

    12:30 забавно что rider не ругается на public поля с _

    • @bloodrik
      @bloodrik Před 2 lety

      ругаться должен решарпер, он скорее всего выключен или специфически настроен.

  • @flashbackmovie8792
    @flashbackmovie8792 Před 2 lety

    обзери код канала "Лавка разработчика". Крутой чел, пишет лучше тебя

  • @fuad2069
    @fuad2069 Před 2 lety

    Не прилагательное, а verb to be

  • @qdnr
    @qdnr Před 2 lety

    А че нельзя дизать

  • @user-zy2km1xe9j
    @user-zy2km1xe9j Před 2 lety

    Набор слов

  • @Lockhurt1322
    @Lockhurt1322 Před 2 lety

    ООО , крутяк тя разблокировали

  • @re1el697
    @re1el697 Před 2 lety +1

    Я python программист и могу не понять некоторые детали в C# коде, но как я понял код прикольный, но он не логичный можно было бы упростить его если посидеть над задачей ну минут 20.

    • @user-td3ev4pk3p
      @user-td3ev4pk3p Před 2 lety

      Что на змее программируешь?

    • @re1el697
      @re1el697 Před 2 lety

      @@user-td3ev4pk3p в основном телеграм, ВК, Дискорд ботов

    • @MercifulMarkiz
      @MercifulMarkiz Před rokem

      Питон сильно отличается от шарпа

  • @user-zh2ug9ij8p
    @user-zh2ug9ij8p Před 2 lety +1

    Роман, Здравствуй. Ты хейтишь Хауди Хо за плохой код. Я нашёл для тебя новый вызов. YANDERE DEV. Программист. Создаёт игру в одиночку. Работает над ней уже 7 лет. Очень популярен на западе. Хотелось бы посмотреть на то, как у тебя с него бомбанёт😂

    • @rsakutin
      @rsakutin  Před 2 lety +1

      Уже есть видео)

  • @user-km5vf3go8g
    @user-km5vf3go8g Před rokem

    На каком языке он вообще говорит, шляпа не понятная или понятно только автору ))))))))

  • @user-fz6mn4re4e
    @user-fz6mn4re4e Před 2 lety +2

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

    • @redeboyjavacglsl724
      @redeboyjavacglsl724 Před 2 lety

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

    • @user-qb8lx2uq5j
      @user-qb8lx2uq5j Před 2 lety

      @@redeboyjavacglsl724 Ты смеешься? Коммерчески успешные приложения начинаются с миллионов скачек. Курсы да, наверняка прибыльны. Но коммерчески успешных приложений у Романа точно нет, по крайней мере в том публичном аккаунте, про который я знаю.

    • @rsakutin
      @rsakutin  Před 2 lety +1

      Потому что игры уходят на аккаунт издателя когда у них начинаются более-менее метрики

  • @projectcool945
    @projectcool945 Před rokem

    Очередной Масон

  • @snegopadich1471
    @snegopadich1471 Před 2 lety

    Что за наборы символов, нихера не понял, удали канал. Куча бреда и непонятных слов, слишком сложно(

  • @artempc3204
    @artempc3204 Před 2 lety +1

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

  • @nikolassmakovsky4032
    @nikolassmakovsky4032 Před 2 lety

    if (мне понравилось видео == true)
    {
    Console.WriteLine("Лайк")
    }
    else
    {
    Console.WriteLine("Говно")
    }
    Console.WriteLine("Лайк")
    P.S. Код говно