Говнокод из популярной игры, частая ошибка и криптостойкие алгоритмы в C# и Unity | КодПатруль #2

Говнокода много в нашем мире, и хорошо не только его видеть но и уметь грамотно переписать. Снова показал вам техники рефакторинга на языке программирования C# которые также вам будут полезны в Unity, ASPNet и других прикладных технологиях.
СПОНСОР:
Admix - admixplay.com/
Discord - discordapp.com/invite/GjJsPhY
До 31 Августа можете выиграть $25,000 - admixplay.com/boost/
БОРИС (Бесплатное Обучение Разработке Игр) - ijunior.ru/boris/promo01?...
Обучение с нуля с гарантией трудоустройства - ijunior.ru/unity-start?...
ШКОЛА ПРОГРАММИРОВАНИЯ ЯЮНИОР - holymonkey_sandbox
INSTA - / ijunior_school
Наш новый канал с уроками - / @user-wq2dk1kn2v
МОЯ КНИГА - csharpbook.sakutin.ru
ГРУППА КАНАЛА - holymonkey
Я В VK - rsakutin
ЧАТ В ТЕЛЕГЕ - t.me/csharp_faggots_fan_club

[@rsakutin]

Сори за косячки, видос записываю болея бронхитом с температурой в 39 :( Также важная деталь: курс с нуля до результата с 3600 в месяц доступен только до конца августа, потом цена вырастет на 30 процентов. Если записаться сейчас то скидочная цена заморозится на весь период рассрочки - ijunior.ru/unity-start?

[@victorbrin5085]

☝️Это ваш курс? Иди реклама?

[@rsakutin]

@@victorbrin5085 Наш)

[@victorbrin5085]

@@rsakutin вписался

[@g3k0s20]

Блин, ну что за детский сад? )) Ты позиционируешь себя как бывалый кодер. При этом, ты просто адово рофлишь практически с любого куска кода. И допускаешь типичные ошибки... Какого черта записывать этот хлам, если тебе хреново?? Как показывает практика, пока мы делаем обзор кода, мы ниибаца спецы. А как садимся кодить, так на деле - рядовой мидл в лучшем случае... И после того как тебя разнесли в коментах, ты просто пишешь, "сори за косячки" ))) Садись - Двойка!

[@rsakutin]

@@g3k0s20 садись на пенис!

[@AqilonStudio]

“YandereDev: A lesson in coding” - видос на ютубе с кусками кода из популярной Инди игры, кладезь, хватит надолго

[@tupolegolassomethingelse455]

О боже мой...

[@Mikasey]

О даа, яндере дев это величайший говнокодер сия мира

[@user-fx5pz7st7c]

Я ржал от начала и до конца как вне себя, мне нужно больше этого дерьма.

[@janedoe6182]

В первом написанном тобой примере генерации пароля длинна итоговой строки будет превышать требуемую. Во втором примере, вместо получения длины массива через стандартное свойство массива, ты зачем-то делаешь это через универсальный метод Linq, который делает лишние проверки и приводит аргумент перечислимого типа к ICollection знающему свою длинну. Random проинициализировать случайным числом можно так: var random = new Random(Guid.NewGuid().GetHashCode()); Массив допустимых символов можно проинициализировать так: var characters = Enumerable.Range('0', 10).Concat(Enumerable.Range('A', 26)) .Concat(Enumerable.Range('a', 26)).Select(Convert.ToChar).ToArray(); Получить результат одной строкой при помощи Linq можно так: string result = String.Concat(Enumerable.Range(0, length).Select(n => characters[random.Next(0, characters.Length)]));

[@user-nu2wg3rx6n]

Курс - Красивый говнокод за 24 часа для чайников ;-))) Сначала находим себе сложности, а потом героически их преодолеваем

[@podtangusom4002]

Знаете,ошибки делают все,а ошибки в свою очередь делают нас лучше.И худшее ,что может сделать человек-высмеивать косяки других людей,за счет этого самоутверждаясь.

[@gaitavr1992]

Первый пример вообще от реальности оторванный, этот конфиг будет еще той огромной колбасой, плюс не упомянут метод Equals с перегрузкой и без. Кстати, а где третий пример говнокода? Я увидел 2 говнокода и три исправленных варианта

[@Scrayerful]

ужасный подход: взять код, который предназначается джунам и для простоты понимания состоит из понятных инструкций и городить из этого работу с исключениями в элементарной операции (подробнее ниже) Да, обработка исключений это хорошо, но сможет ли новичок правильно обработать исключение ? А проконтроллировать после этого сборщик ? Добавление дополнительной конструкции Config.PositiveStateKeyword Почему в итоге не return value == "on" ? То, что ты описал хорошо, когда повторяемость кода большая, но если подобные функции необходимы часто, то тут проблема скорее в архитектуре *предполагаю, что данный код взят из какого-либо видео, но ссылки на него не оставлено **думаю, что преобразование string -> bool не уместно вообще, поэтому функция на момент создания обречена быть неудачной

[@Sagrell94]

Ты не прав в каждом своем высказывании) 1. По подходу я написал коммент. 2. Про сборщик мусора ты решил сделать вид, что ты шаришь, но это самое бессмысленное, что я слышал) 3. Config.PositiveStateKeyword это "константа". Ты наверное считаешь периметр круга в коде как 2*3.14f*R 4. Преобразование string в bool не уместно?)) А как по твоему работает парсинг СТРОКИ в ОБЪЕКТЫ?)

[@user-qu3xn6uc9o]

я который еще не джун такие конструкции давно не использую и как уже говорил автор во многих видео : говнокод это не проще,это хуже

[@alyxmp4749]

Каждый код - говнокод (С) Роман

[@Sagrell94]

Почти в каждом комменте вижу, как люди несут такую чушь. Многие пишут, что его подход - ужасен. Подход не ужасен, а бессмысленнен, новички все равно не смотрят такие видео, поэтому эти видео скорее делаются для себя. Но он говорит абсолютно логичные и правильные вещи. Нельзя при показе кода новичкам жертвовать качеством этого кода. Тебе же учитель математики не говорит: "сейчас мы изучаем сложение, поэтому не обращайте внимания, что у меня кучу ошибок в вычитании". Человек думает, что так нужно писать и так пишет дальше. Приведу пример: я хотел разобраться в шейдерах, но не сильно запаривать и не читать огромные книги. В итоге искал ролики на KZbin и мне попадались сплошные эти "горе Гоши Дудари", и сколько я не смотрел таких роликов, я не мог сесть и написать любой шейдер нормально, потому что они не раскрывали суть и не обучали тебя, они показывали как они говнокодом решили какую-то конкретную задачу. Но потом я нашел Catlike Coding, где чел разбирает шейдеры на уровне их компиляции и объясняет вообще все. И за четверть времени которое потратил на этих идиотов, научился писать довольно сложные шейдеры и дальше уже сам разбираться во всем. Если ты делаешь контент для новичков, то делай уроки вида "Как работает цикл for" и постепенно вводи в материал, а не "Как нихуя не зная, написать AI за 30 мин". Это информационный мусор, который только вредит. Хотя скорее всего это пишут зрители Гоши Дударя, которые думают, что они разработчики игр, не понимая, что они по уровню развития сейчас на школьной программе 9 класса.

[@Hitori_FtoL]

Роман, оставь, плиз, ссылку на твой гитхаб) Просто хочется увидеть хороший код, для того, чтобы подчеркнуть некоторые фишки

[@user-nf6uw9cv4q]

А ответа нет? У гуру нет гитхаба странно?

[@dadoo6912]

@@user-nf6uw9cv4q если ты увидишь сеньора с богатым, постоянно обновляющимся гитхабом, единственное, что о нем можно сказать, так то, что он нихуя не работает

[@xezdx]

Насчет экспшн в конфиге это больше зависит от соглашений, принятых для программы. Если принято ошибки "замалчивать", а не бесить юзера всплывающими окошками, то достаточно {return key=="on"}, если нет, то проверку стоит делать отдельно типа if(!isKeyword(key)){ ...error... } else { ok=isEnabled(key); } Проверять значения из конфига выбросом исключения не стоит. Исключения не должны быть частью программы и "нормальным" результатом выполнения функции.

[@Slko]

whiteList[b % whiteList.Length] Если делать криптографически стойкую генерацию паролей, то так *НЕЛЬЗЯ* делать, если количество вариантов значений b (в данном случае 256) не делится нацело на whiteList.Length (в данном случае как раз не делится, т.к. 26+26+10=62). Каждое из значений 0-7 встречаются 5 раз (255 % 62 == 7, 255/62=4.xyz), а значения 8+ встречаются 4 раза, то есть есть bias к маленьким числовым значениям.

[@rsakutin]

Крутяк, спасибо!

[@Slko]

@@rsakutin Если интересно, как правильно сделать, то самый простой вариант такой: Нужно проверять, входит ли случайное число в наш диапазон (0-61), и если не входит, то пропускать его и брать следующее случайное значение, и так пока не наберём нужное количество символов.

[@xBezzx]

Slko я нихуя не понял но очень интересно , не останавливайся пиши

[@artur8904]

@@xBezzx маленьких цифр в паролях будет больше

[@Lucio11a]

Admix, это, конечно, круто... но реклама сбербанка в средневековой таверне, эт было бы слишком сильно :D

[@Priboy313]

Я первый пример вижу так: был рычаг из простой конструкции и я более чем уверен, что где-то в коде он вызывался ИСКЛЮЧИТЕЛЬНО с он/офф. А стал переусложнённый космический аппарат с навесом из конфига и прочей лабуды. Всего один вопрос - зачем?

[@Alex-net-brn]

Чувак, ты в первом примере, где проверка value == "on", ВСЁ СЛОМАЛ!!! ))) Функция возвращала true, ТОЛЬКО когда value существует и равно "on". Если в конфиге нет этой строки, или она закоменчена, то твой супер код начнёт сыпать эксепшены )) Пример такой строки: god_mode = "on" смекаешь умник? :))

[@LordZiegfrid]

Вот тебе ревью по первому примеру: 1. Сравнение через "==" для строк зависит от региональных настроек. 2. "== false". 3. Название переменных и методов не отражает суть.

[@riskiofi7571]

не отражает, но это важно. чтобы не запутаться в своём же скрипте, ну например сел за проект который делал 2 года назад. и ты ничего не поймёшь так как воиды будут называться"aboba A" и с переменными также

[@user-gk4sn1th8c]

Есть множество методов решения проблемы. Так почему все, кроме одного единственного, сразу говно? Во мне пробуждается сомневающийся во всём перфекционист. Как убедиться в правильности решения проблемы? Может потому, что мы человеки еще не стали как "1"? Не "перепишем говно код", а "Люди! Я могу предложить более совершенный код!" :-).

[@mikhailshpirko1877]

4:42 по идее, если в метод передать "On" или "ON" то будет либо эксепшн (в зависимости от того как имплементирован метод IsBinaryStateKeyword) либо метод вернет false. Как вы считаете, правильно ли в таком случае возвращать true и осуществить сравнение без учета регистра?

[@rsakutin]

Я бы делал регистронезависимым решение

[@maxm1079]

@@rsakutin а я бы не делал, иначе те кто настраивает софт вам хуй выкрутят, из за того что у вас on != ON, on != true (и тд.). так что и вы неплохо своего говна добавили. первый пример вообще чисто рефакторинг кода сделали, изначальный код не является уязвимым

[@f14shm4n]

@@maxm1079 изначальный код болеет тем же самым, при "ON" выдаст "false", но в принципе согласен с вами, в оригинальном коде нужно просто добавить IgnoreCase

[@stanislaw8179]

Привет, у меня есть книга "ЯП С# 7 и платформы .NET и .NET CORE" от Эндрю Троелсен и Филипп Джепикс Какой уровень можно получить, прочитав эту книгу, если я с полного нуля, без книг и тп, смотря видео на ютубе понял ООП, создал 2д на юньке по типу дудла и несколько приложений ВинФормс с разным наклоном, но при этом, я прохожу тесты на junior на 40%-60%?

[@badguy6915]

сакутин:выы гавнооо!!! мы:мыыы гавнооо!!! простые смертные(непрограммисты): ...

[@SergeyGlukhachev]

Встречал как-то такое: public enum Sample { Value0, Value1, Value2, ... } public Sample SomMethod(string inputString) { switch (inputString) { case "Value0": return Sample.Value0; case "Value1": return Sample.Value1; case "Value2": return Sample.Value2; ... } }

[@FloatingGames]

А что делать если я не понимаю?

[@FloatingGames]

Хотя я понял, это жесть, а можешь написать как надо или что подрузумевалось?

[@SergeyGlukhachev]

@@FloatingGames docs.microsoft.com/ru-ru/dotnet/api/system.enum.parse

[@serghein845]

@@FloatingGames когда не понимаешь, надо лезть в инет и разбираться! так обычно все делают )

[@serghein845]

Мужик, ты красава! самый правильный ответ ) тольк в нём нет состояние "default", которое и будет исключением.

[@g_r_u_n_d_y]

Зачем кастить char в int и обратно, если можно одним char обойтись? `for (char c = 'a'; c

[@FloatingGames]

А что значит yield return, ну просто никак не могу понять

[@g_r_u_n_d_y]

@@FloatingGames Справка msdn: docs.microsoft.com/en-us/dotnet/csharp/language-reference/keywords/yield При использовании этого ключевого слова внутри функции, компилятор преобразует ее так, чтобы она возвращала итератор. Это позволяет просто создавать итерируемые объекты, по сути односвязные списки, по которым можно пройтись например, с использованием `foreach`, при этом каждая итерация отделяется с помощью yield

[@FloatingGames]

@@g_r_u_n_d_y а куда буква запишется тогда? Я так понял этот метод yield используется при переопределении метода getIEnumerator, да?

[@g_r_u_n_d_y]

@@FloatingGames Я так понял этот метод yield используется при переопределении метода getIEnumerator, да? Не совсем. Сам класс c методом GetEnumerator генерируется компилятором. В полученной реализации IEnumerator при вызове `MoveNext()` происходит выполнение метода до первого yield, значением свойства Current устанавливается значение после `yield return` в данном случае текущий символ. При следующем вызове - выполнение до следующего yield и так далее.

[@bratttn]

а зачем так сложно с генерацией рандом строки из словаря? есть же в си шарпе Fare (xeger), загоняешь в него регулярку с фиксированной длиной строки и бум! одна строчка кода.

[@7453060]

Роман, почему генерация строки не метод расширения? Очевидно, что если мне необходимо особое поведение StringBuilder, я создаю метод расширения и в момент использования оборачиваю в using. Далее мне не нужны сами символы на этапе генерации, мне нужны только их коды в таблице символов, значит мне нужно смещать iterator по матрице, способ смещения определяет стойкость алгоритма.

[@_kk_]

Потому что это нарушит принцип единственной ответственности. Разве нет?

[@sashayakimov6]

Здравствуйте, сколько стоит сейчас курс?

[@user-ni5nt2bs8j]

Недавно в проекте ЕПАМ был найден такой код. Это самое бессмысленное и беспощадное, что я видел. private static int stringSize(String s) { int size = 0; for(int i = 0; i < s.length(); i++) { size++; } return size; }

[@Dadadadam999]

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

[@AlexStraga]

Балин, мы ток с другом начали учить делфи и то превращали числа в чары, а не создавали списки чаров в ручную)

[@morozokgaming5921]

а для чего вам делфи, если не секрет? 🤔

[@AlexStraga]

@@morozokgaming5921 потому что была книга по делфи для чайников) Это было более 17 лет назад.)

[@morozokgaming5921]

@@AlexStraga я думал вы его в 2020 начали учить))

[@morozokgaming5921]

@@AlexStraga и чего добился за 17 лет? если не секрет)

[@AlexStraga]

@@morozokgaming5921 Честно говоря не много ) Интернета тогда у меня не было (да и нормальный не еле дышаший адсл ток 2 года назад появился), учиться было сложно, потом устроился в айти сфере в отрасль не касаясь прогерства. Кодинг был хобби. И вот только полтора года назад вернулся в кодинг, прошел курсец и думаю что теперь дело пойдет хоть как то. Друг же ушел в веб разработку

[@vitaly6838]

Я когда только начинал даже так не писал

[@olegzorin3911]

Пример с генерацией пароля немного надуман. Да белый список - это правильно, но в конечном итоге вы же храните у себя не пароль, а его хеш. И как бы вы круто не генерировали пароль, но если потом храните, упаси Бог, 8-битный хеш - то уже будет не принципиально как сгенерирован пароль. Да даже 256 битные пароли брутфорсятся за вменяемое время на гпушках. Привет от радужных таблиц. Так что если думать о безопасности, то начинать надо с хеш-функции (повышать время на взлом), хранения этих хешей (не позволять спереть бд, чтобы брутфорсить на гпу) и защищать от перебора саму форму (long request - увеличиваем время взлома, fail2ban - усложняем жизнь на возню с прокси и увеличиваем время). А вообще сделать непробиваемую защиту не возможно - можно лишь сделать экономически не целесообразным взлом. Если вы храните только фантики для пользователей со странными никами, то так-то можно пароль и в открытую хранить ))

[@ARTUT]

0:00 Camtasia 9?

[@user-jy1rb5xn7s]

Как ты относишься к антипатерну Softcode ?

[@pirnogion]

Я называю это жидкий код, ибо протекает во все щели.

[@ZEkA10000]

Ты обозреваешь любой код или целенаправленно выбираешь код написанный на яп С#?

[@intivi5277]

Он проталкивает свои говнокурсы поэтому щедро набрасывает на вентилятор.

[@sudormrfall]

Не могу понять от видео к видео, я джун или мастер...

[@mrkvchch]

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

[@rsakutin]

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

[@sergeykarpov4683]

Интересно услышать мнение о курсе ITVDN "ASP .Net Core Developer", стоит ли использовать данный ресурс для изучения функционала C# языка? P.s. Отлично отфильтрована информация, только не переходите на "умирающий бас", неприятно разбирать слова.

[@rsakutin]

Уже есть на канале

[@Roman_SBB]

Что-то БОРИС ссылку на почту не присылает

[@user-sp8eb8wf2q]

Гавнокод (помню из старых твоих обзорах на unity3d курсы): Нашел kzbin.info/www/bejne/fJS6eXSsrtSpiNU Что-то такое вроде было if (points < 100) { star1.Enabled = false; star2.Enabled = false; star1.Enabled = false; }else if (points > 300) { star1.Enabled = true; star2.Enabled = false; star1.Enabled = false; }else if (points > 500) { star1.Enabled = true; star2.Enabled = true; star1.Enabled = false; }else if (points > 1000) { star1.Enabled = true; star2.Enabled = true; star1.Enabled = true; }

[@DoGGy1110]

Про 2ой случай. Это погрешности декомпиляции. При декомпеляции любая конкатинация строк делается через массив.

[@50z10p47]

Поясните нуфагу. При конкатенации строк/массивов мы создаём новый объект. При использовании stringbuilder мы имеем все тот же объект. Если не лезет - capacity увеличивается в 2 раза до тех пор пока не влезет или не превысит MaxCapacity. То есть даже после декомпиляции это не должно выглядеть конкатенацией строк так как принцип работы немножечко совсем другой, разве нет?

[@user-nf6uw9cv4q]

Сакутин где увидеть твой гитхаб?

[@user-ix8bh3ox9f]

Такие примеры вселяют мне уверенность что я уже всё таки не джун)

[@qwertyqwertyi620]

Жду код ревью YandereDev'a...

[@melonplayground5975]

в первом примере как я понял value должно иметь только on или off, так почему бы string не заменить на enum ?

[@rsakutin]

Строка приходить из конфигурации строковой

[@serghein845]

можно узнать почему такие великие учителя, которые преподают курсы, и делают из людей гуру программирования, не делают свои успешные проекты? или не работают над большими крутыми проектами? Мне просто интересно! p.s. Не знаю ни одного крутого прогера, который думал бы учить людей программированию, потому, что у него нет времени и он зазнавшийся дятел (ну обычно так).

[@rsakutin]

Не знаю ни одного крутого программиста который не вёл бы блог, не писал бы книги или не снимал на KZbin

[@serghein845]

​@@rsakutin если так судить, то хороших прогеров очень мало, так как я не видел много документаций или уроков доступных о паттернах программирования, отличае с# от f#! или же в крутых компаниях, которые на удивление выпускают крутые игры или софт, все пишут статьи ) Я не говорю о гугл разработчиках или фейсбук, которые сидят в долинке, и чтобы продвинуться по карьерной лестнице пишут диссертации, лишь для того, чтобы их заметили в той гуще крутых прогеров, которая их окружает. p.s. Но вы ведь не статейки пишете, а преподаёте, что забирает уверен кучу времени, чтобы из обычного пользователя компа сделать их гуру программирования. Этио же не за вечер и не за неделю сел и написал статейку )

[@rsakutin]

@@serghein845 Вы не смогли найти кучу материала по паттернам и отличию C# и F#? У вас инвалидность?

[@rsakutin]

@@serghein845 3600 - это цена курса в рублях. Вы говорить любите а думать и читать нет. :(

[@serghein845]

@@rsakutin ну тогда пишите нормально, чтобы и читалось доступно и понятно! Потому, как "курс с нуля до результата с 3600 в месяц" - это не одно и тоже, как например "курс за 3600 в месяц - с нуля до результата " или хотя бы "курс с нуля до результата за 3600 в месяц". Поэтому я и задал основной вопрос, так как понял совсем другое!

[@user-pp9mz1lh9b]

Уникальнейший полезный контент! Прямо таки уникальный в своей нише. Продолжай творить!

[@mrmatrix228]

4:37 сравниваем булевую переменную. ОК

[@lancwork]

сравнение с false более читаемое чем отрицание

[@user-kp8db4nz6n]

Не знаю как у вас, но у меня триггер на «!». Сразу норм читается, особенно если булевые переменные сразу называть трушными названиями или с приставкой “Is”

[@BashkaMen]

Валидация в конфиге, ммммм

[@dmitriiyokohamaman8523]

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

[@rsakutin]

Какой опыт программирования у вас? Какие комании? Какие проекты?

[@dmitriiyokohamaman8523]

Я нигде не работал по профессии программист.

[@rsakutin]

@@dmitriiyokohamaman8523 А что подпездываете тогда во втором видео уже?

[@dmitriiyokohamaman8523]

в@@rsakutin Не "подп-здываю", а "По Царски" комментирую. Что бы у Вас больше просмотров было.

[@rsakutin]

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

[@montykay6003]

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

[@lolmemavich135]

а еще за "Привет, мои маленькие любители говнокода" - отсылка к Тесаку, тоже респект)

[@isildurgir]

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

[@dizzyark9221]

Роман что то у вас с сайтом. Пытался записаться но... Видимо что то случилось, да. Регистрация не работает!

[@araslanrus]

Видимо говнокод на сайте не позволяет

[@user-yt9un2uo6r]

Можешь обозреть код yanderedeva?)))

[@7453060]

Почему "on" "off" не enum?

[@LordZiegfrid]

Потому что это строки.

[@Sagrell94]

@@LordZiegfrid +, господи челы настолько Васьки) Это парсинг строки, ало. Какой enum? Или ты предлагаешь спарсить "on" "off" в enum, чтоб потом перевести в bool?

[@Dadadadam999]

Тут скорее вопрос, кто вообще хранит двухзначное значение в виде строки. =Д Как бы не проще ли в конфиге хранить сразу же, bool переменную, чем потом писать такую вот дичь.)

[@Bond007in]

Если "из популярной игры" может не "обосрались".

[@selim5772]

А что за игра?

[@darkhorse2219]

Можешь конструктивно раскритиковать мой не дописанный говнокод на данном этапе ? using System.Collections; using System.Collections.Generic; using UnityEngine; using UnityEngine.UI; public class carController : MonoBehaviour { [Header("Keyboard")] [Space(10)] public KeyCode Forward = KeyCode.W; public KeyCode backTo = KeyCode.S; public KeyCode right = KeyCode.D; public KeyCode left = KeyCode.A; public KeyCode Acceleration = KeyCode.LeftShift; public KeyCode brake = KeyCode.Space; [Header("Speed")] [Space(10)] public float speed; public float minSpeed =50f; public float accelerationSpeed = 100f; public float reversSpeed = -30f; public float smoothSpeed = 0.1f; [Header("Fuel")] [Space(10)] public float fuelConsumption = 100f; public Scrollbar fuelS; [Header("Others")] [Space(10)] public float rotateUgol = 20f; Rigidbody rig; Transform trans; public float xRotate; public float rotatePodom = 2; public float yRotate; public Transform[] wheels; public TriggerCollider[] peredZad; // Use this for initialization void Start () { rig = GetComponent (); trans = GetComponent (); Cursor.visible = false; Cursor.lockState = CursorLockMode.Locked; } // Update is called once per frame void FixedUpdate () { if (fuelS.size == 0) { rig.useGravity = true; rig.freezeRotation = false; } else { if (speed < 0.1 && speed > -0.1) speed = 0f; if (Input.GetKey (brake)) { if (speed > 0) speed -= smoothSpeed * 10; if (speed < 0) speed += smoothSpeed * 10; rig.velocity = transform.forward * speed; } if (fuelS != null) fuelS.size -= Time.deltaTime / fuelConsumption; //trans.Rotate (Input.GetAxis ("Mouse X") * rotateUgol, 0f, 0f); if (!Input.GetKey (backTo) && !Input.GetKey (Forward)) { if (speed > 0) speed -= smoothSpeed * 5; if (speed < 0) speed += smoothSpeed; rig.velocity = transform.forward * speed; } if (Input.GetKey (Forward) && !peredZad[0].isTrigger) { if (Input.GetKey (left)) { yRotate -= Mathf.Lerp (speed / rotateUgol, speed / rotateUgol * 0.8f, speed / rotateUgol * 0.8f); } if (Input.GetKey (right)) { yRotate += Mathf.Lerp (speed / rotateUgol, speed / rotateUgol * 0.8f, speed / rotateUgol * 0.8f); } xRotate += Input.GetAxis ("Mouse Y") * rotatePodom; xRotate = Mathf.Clamp (xRotate, -50, 50); trans.localEulerAngles = new Vector3 (xRotate, yRotate, trans.localEulerAngles.z); rig.velocity = transform.forward * speed; if ((speed < minSpeed && Input.GetKey (Acceleration) && speed != minSpeed) || speed < 0) speed += smoothSpeed * 2; else if (speed < minSpeed && speed != minSpeed) speed += smoothSpeed; else { speed = Mathf.Clamp (speed, minSpeed, accelerationSpeed); if (Input.GetKey (Acceleration) && speed < accelerationSpeed) { speed += smoothSpeed; } else if (speed > minSpeed + 1) { speed -= smoothSpeed; } } } if(!Input.GetKey(left) && !Input.GetKey(right)) yRotate = trans.localEulerAngles.y; if (Input.GetKey (backTo) && !peredZad[1].isTrigger) { xRotate += Input.GetAxis ("Mouse Y") * rotatePodom; xRotate = Mathf.Clamp (xRotate, -50, 50); trans.localEulerAngles = new Vector3 (xRotate, yRotate, trans.localEulerAngles.z); rig.velocity = transform.forward * speed; if (speed != -reversSpeed) speed -= smoothSpeed * 10; speed = Mathf.Clamp (speed, reversSpeed, accelerationSpeed); if (Input.GetKey (left)) { yRotate -= Mathf.Lerp (speed / rotateUgol, speed / rotateUgol * 0.8f, speed / rotateUgol * 0.8f); } if (Input.GetKey (right)) { yRotate += Mathf.Lerp (speed / rotateUgol, speed / rotateUgol * 0.8f, speed / rotateUgol * 0.8f); } } } } }

[@GameDevTutorials2023]

Я бы глянул обзор кода канала Sharp Accent

[@sh_nero]

В С# такой стиль писать "condition == false" вместо "!condition"?

[@your-salmon]

Сакутин говорил, что восклицательный знак легко не заметить.

[@jz4887]

@@your-salmon ну, его, наверное, не заметит только в шары долбящийся.

[@jz4887]

Нет, это не стиль такой. Вот сам не понимаю зачем этот позор с "== false" писать.

[@lunarlife7850]

@@jz4887 Для нормальной читабельности

[@jz4887]

@@lunarlife7850 код отлично и без этого читается

[@hewston2733]

Вообще это из С++, но вдруг кому-то захочется в этом поковыряться, писал от скуки на паре: int fun(int *arr, int sz, int elem = 0){ return elem==0?(arr[elem]+fun(arr, sz, elem+1))/ sz: elem !=sz-1?arr[elem]+fun(arr,sz,elem+1):arr[elem]; }

[@hewston2733]

Выводит среднее арифметическое числового массива (наверное)

[@nikolainikk3916]

3:07 по-моему, это тоже говнокод. Поправьте, если не прав. Ты эксепшн кидаешь и юзаешь зачем-то new? Зачем тебе тут понадобилось вообще динамическое выделение памяти?

[@TedFanat]

Ору) Это С#, a не С++))

[@nikolainikk3916]

@@TedFanat всё, извиняюсь) спасибо что поправили)

[@evilraccoon6538]

Есть два разных подхода. 1. Критиковать чужое, старательно выбирая слабые примеры. 2. Создавать свое, превосходящее чужое, ожидая что это оценят. Так вот, ребятки, программисты это люди умственного труда. Оставлю-ка я вам тут задачку: кто из этих двух примеров будет лУчшим специалистом, а кто пустышкой? И оставлю тут готовое решение: идите отсюда, вас тут ничему толковому не научат. Он же токсичен. Нельзя инфицироваться таким образом мышления.

[@sergeyfirsov9100]

Твой то код где ?

[@kirillbukato4164]

Прошу прощения, а как этот ваш c# в плане секса, А? М?

[@nikolay-5074]

Отличный пример, но легче весь код просто удалить так как он не преследует никакой цели и не нужен.

[@russianempiremapper408]

что за игра

[@nikolay-5074]

Кликер Clickbayter.

[@--.--]

4:53

[@user-bb5xw8bd8w]

Очередное не знание матчасти: if (value == "on") return true else bla-bla-bla - натуральный говнокод вместо одной строчки

[@avocadocoder185]

Гоблин в мире C#

[@debez3716]

ахахах

[@purplep3466]

блин, Роме нужно похожий голос еще сделать

[@Diyozen]

Я Вас категорически приветствую. Этот код - тупое говно тупого говна. Говнополя и говнометоды с говноаргументами для отсталых дебилов. Дети, не пишите говно тупого говна. А на сегодня всё, до новых встреч.

[@adiks09]

Те кто писал про yandere, спасибо, я ослеп

[@darkstormheute]

А нельзя было написать: if (value == "on") return true else if (value == "off") return false else throw new ArgumentException("...") ?

[@hitaciry258]

а еще можно без else

[@darkstormheute]

@@hitaciry258 нет, была-бы ошибка

[@kirillsviderski4739]

@@darkstormheute else реально не нужен тут

[@darkstormheute]

@@kirillsviderski4739 а да, затупил

[@intivi5277]

Можно и нужно так как вы предложили. Только без else. Не берите в голову примеры быдлооптимизации из видео.

[@biohazardmars]

Зделай обзор на ютуб программиста Simple Code у него полезные.

[@-boiadeiro-]

а у вас разве нет другой либы для "настоящего" рандома? *мимо-не-сишарпер*

[@xezdx]

Нафиг такие сложности с генерацией? Псевдокод: pwd=md5(random()).substring(...) - сколько нужно вплоть до 32 символов. И вообще такие пароли фиг запомнишь даже 6 символов. Лучше делать пароли из пары обычных слов типа "каша-малаша" - подбирать рандомом дольше, а запомнить легче.

[@giawsuy4781]

У меня есть еще один вариант получения алфавита: List Alphabet = new List(Enumerable.Range('A', 'Z' - 'A' + 1).Select(c => (char)c).ToList());

[@MrJloa]

Плять, 2021 год, пароли из ascii символов... Для адептов кодировок отличных от utf8 в аду отдельный котёл с бесами. Вообще за такое административку давать надо.

[@That-Is-All-Hat]

Только у Ромы Сакутина качественный код, а у всех остальных - говно. Покупайте курсы Ромы Сакутина.

[@khudoyorkhonmirjalolov3006]

готов в разносу своего кода public class EnemySpawner : MonoBehaviour { public ObjectPooler _objectPooler; public Transform[] SpawnPoint; public float NextDeamonTimer = 2f, NextSkeletonTimer = 1f, DeamonStartSpawnTime = 5f; private float _deamonTimer = 0f, _skeletTimer = 0f, _deamonStartSpawnTime = 0f; private void Start() { _objectPooler.SpawnFromPool("Deamon", SpawnPoint[Random.Range(0, SpawnPoint.Length)].position, Quaternion.identity); _objectPooler.SpawnFromPool("Skeleton", SpawnPoint[Random.Range(0, SpawnPoint.Length)].position, Quaternion.identity); } private void FixedUpdate() { SkeletonSpawn(); DeamonSpawn(); } private void SkeletonSpawn() { _skeletTimer += Time.deltaTime; if(_skeletTimer >= NextSkeletonTimer) { _objectPooler.SpawnFromPool("Skeleton", SpawnPoint[Random.Range(0, SpawnPoint.Length)].position, Quaternion.identity); _skeletTimer = 0; } } private void DeamonSpawn() { if (_deamonStartSpawnTime = NextDeamonTimer) { _objectPooler.SpawnFromPool("Deamon", SpawnPoint[Random.Range(0, SpawnPoint.Length)].position, Quaternion.identity); _deamonTimer = 0f; } } } }

[@levrisfirst3291]

Самое смешное, что раз так автор разбирается в говне, то скорее всего сам такое же говно и пишет :)

[@50z10p47]

Опыт - сын ошибок трудных

[@znsoft]

Не скорее всего, а точно пишет. Он даже в этом ролике показал свой говнокод. Его испражвления не лучше

[@isildurgir]

В первом примере интересная Ваша логика. В коде четко же написано, что если не "on" только тогда false. То есть в принципе пофиг что там кто то вводит что угодно даже yes, всегда будет false. Да конечно там нет эксепшин, но разве это ответственность этого метода, пусть тот кто передает в метод значение и занимается его обработкой. Это же синглреспонсибилити в прямом действии. Как такого можно не знать. Согласен если б это действительно была программа для какой то банковской системы безопасности, там где эксепшин на эксепшине сидит и нужно все обрабатывать, но для игр это перебор и полный бред.

[@user-by1ct1rc7p]

using System.Collections; using System.Collections.Generic; using UnityEngine; public class SWIPE_MOVE : MonoBehaviour { public bool tap, swipeLeft, swipeRight; private bool isDraging = false; private Vector2 startTouch, swipeDelta; public GameObject player; private void Update() { tap = swipeLeft = swipeRight = false; #region Standalone Input if (Input.GetMouseButtonDown(0)) { tap = true; isDraging = true; startTouch = Input.mousePosition; } else if (Input.GetMouseButtonDown(0)) { Reset(); } #endregion #region Mobile Input if (Input.touches.Length > 0) { if (Input.touches[0].phase == TouchPhase.Began) { tap = true; isDraging = true; startTouch = Input.touches[0].position; } else if (Input.touches[0].phase == TouchPhase.Ended || Input.touches[0].phase == TouchPhase.Canceled) { isDraging = false; Reset(); } } #endregion swipeDelta = player.transform.position; if (isDraging) { if (Input.touches.Length > 0) swipeDelta = Input.touches[0].position - startTouch; else if (Input.GetMouseButton(0)) swipeDelta = (Vector2)Input.mousePosition - startTouch; } //Dead zone if (swipeDelta.magnitude > 125) { float x = swipeDelta.x; float y = swipeDelta.y; if (Mathf.Abs(x) > Mathf.Abs(y)) { //Right or left if (x < 0) swipeLeft = true; else swipeRight = true; } Reset(); } } private void Reset() { startTouch = swipeDelta = Input.mousePosition; } public Vector2 SwipeDelta { get { return swipeDelta; } } public bool SwipeLeft { get { return swipeLeft; } } public bool SwipeRight { get { return swipeRight; } } } Thats my shit!

[@user-by1ct1rc7p]

using System.Collections; using System.Collections.Generic; using UnityEngine; public class Swipe_test : MonoBehaviour { public SWIPE_MOVE swipeControls; public Transform player; //public Vector3 desiredPosition; private void Update() { if (swipeControls.SwipeLeft) player.transform.position -= Vector3.right; if (swipeControls.SwipeRight) player.transform.position += Vector3.right; player.transform.position = new Vector3(player.transform.position.x, player.transform.position.y, player.transform.position.z); } }

[@kyv1r]

Все классно, но можешь говорить чуть громче пожалуйста?

[@user-uv3gw2zk2d]

Мда, почему это говно вылетает по рекомендациям....

[@user-wb3fc9zn7j]

Посмотри Yandere Dev Code. Там контента на долго хватит) 4069 строк кода............

[@FloatingGames]

А в чем прикол? Я просто не в теме)) расскажи

[@user-wb3fc9zn7j]

@@FloatingGames у него там весь код состоит из if/else.. И это 4069 строк занимает в скрипте. Видео на ютубе посмотри, угарнешь.

[@Yangoira]

Нихуя не понятно, но очень интеречно

[@CidKagenouS]

Оооода

[@nikemoscow8881]

Если не on то false. Ну и в чем баг? Некорректно? - да, но не будет бага.

[@Dadadadam999]

Вполне может быть. Скорее зависит от того, какая реализация на основе этого флага будет создана. Если от значения off будет зависеть, выполнение каких, либо условий, то могут полезть баги, если например конфиг криво спарсится. Однако это не критично и проблема быстро находится через дебагер. Тут в целом проблема, что очевидно булева переменная в конфиге описана при помощи стрингов.

[@oleksandrlysiuk3971]

Блин , я фронт разраб, и смотрю это, зачем?))

[@cheburek5116]

А я вообще не кодер, но смотрю

[@Wansery]

3:05 это тоже говнокод какой-то. Зачем нам вообще нужно парсить строку в бул, если мы точно знаем, что on это true, а off это false? Почему сразу не передавать бул? Если уж на то пошло, то можно просто сделать switch case (хз какой аналог на шарпах) нет кучи дополнительных условий и защита от ошибок. А так, ты просто обернул говнокод в шапку из говна, ещё не забыв свои говнокурсы прорекламировать

[@despasus]

Очень прикольный контент, жаль только, что он ограничивается сишарпом и гейдевом

[@gachigazm6169]

Зачем так голос искажаешь? Типу фишка такая? Бывает не оч приятный такой асмр

[@rsakutin]

У меня бронхит :(

[@nurlanakylbekov7266]

#1

[@NagisaHa]

Да ты за.. надоел обсирать Гошу в первую очередь.

[@oleksandrlysiuk3971]

Nagisa так Гоша реально говнокодит)

[@iamdan8203]

Кто бы ни читал этот комментарий, знай, что *ГОСПОДЬ ИИСУС ХРИСТОС ЛЮБИТ ТЕБЯ!* Whoever reads this comment, know that *LORD JESUS CHRIST LOVES YOU!*

[@user-wl5hd3yf9w]

Еба весь ролик реклама своих курсов

[@calchome2768]

Да нахрен надо так писать... Через 3 мес забудете про сложности, которыми напичкали код... И поддерживать его станет гораздо сложнее... В общем Гоша дударь лучший! Лучше говнистей, но понятнее!

[@LordHellas]

Как же тяжело вас слушать... Представляю такую подачу материала на ваших курсах.

[@user-sp8eb8wf2q]

он не преподает, там другой препод

[@rsakutin]

@@user-sp8eb8wf2q Я преподаю, и на мои лекции отзывы что сложно слушать

[@50z10p47]

@@rsakutin no pain - no gain хули

[@asddsaasddsaasddsaasddsa]

Я, наверно, открою Америку, но на юнити игры делают не программисты и им до качества кода совершенно нет дела.

[@FloatingGames]

Почему ты так думаешь?