Исправляем очень плохой код | Clean Code
Переглядів 15,111
День томуЗдравствуйте, мои дорогие, это вторая часть видео про клин код, практическая. Наглядно разбираем ошибки и изучаем на практике, что же такое чистый код и принципы его написания.
🔥 Новый поток авторского тренинга Сергея Немчинского GRASP and GoF Design patterns стартует уже 01.02.2024
Регистрация 👉 go.foxminded.ua/4b4M3JI
🧠 Регистрируйся прямо сейчас на менторинг
⬇️ Учись у профи ⬇️
🎓 Менторинг:
JAVA - go.foxminded.ua/3u5V4BF
PYTHON - go.foxminded.ua/48F0dj8
C#/.NET - go.foxminded.ua/3O987tc
ANDROID - go.foxminded.ua/3HoXZbL
FRONT-END (ANGULAR, REACT) - go.foxminded.ua/3O8V62E
SALESFORCE Developer - go.foxminded.ua/47IxGIq
UI/UX дизайн - go.foxminded.ua/3S6s58L
Unreal Engine - go.foxminded.ua/422Ry7S
QA Automation - go.foxminded.ua/3u128Qa
IOS разработка - go.foxminded.ua/3tQ8aTL
PHP - go.foxminded.ua/4be51ho
Unity - go.foxminded.ua/4aV0fFc
NODE.JS - go.foxminded.ua/3S7wc4w
GOLANG - go.foxminded.ua/48XgqQt
Обучение на проекте - go.foxminded.ua/427KoPu
Курсы для новичков - go.foxminded.ua/3O601kQ
🎓Продвинутые курсы для состоявшихся девелоперов:
Enterprise patterns - go.foxminded.ua/3S1g6tn -
Алгоритмы и структуры данных - go.foxminded.ua/424yIwZ
C# NEXT - go.foxminded.ua/4aZcb8V
🔧 Пробное техническое собеседование со специалистом уровня Senior Developer/ Team Leader - go.foxminded.ua/426HL0H
Тайминг:
00:00 - Вступление
00:37 - Названия
06:29 - GRASP and GoF Design patterns
07:03 - Методы
09:42 - Комментарии
13:54 - Плохие комментарии
15:41 - Форматирование
16:23 - Объекты и структуры
18:14 - Классы
19:59 - Правила простого дизайна
👔 Карьерная консультация с Сергеем Немчинским - go.foxminded.ua/48ZGP05
Есть вопросы по обучению в FoxmindEd? Пишите нам в телеграм - t.me/foxminded
Вы можете стать спонсором канала и получать плюшки - / @sergeynemchinskiy
❤ FoxmindEd в Instagram: / foxminded.ua
Сайт FoxmindEd для новичков: go.foxminded.ua/3O9iLQv
Сайт для разработчиков уровня мидл+: go.foxminded.ua/47GqufK
FoxmindEd в ФБ: / foxmindedco
FoxmindEd в Instagram: / foxminded.ua
Мой Telegram: t.me/nemchinskiyOnBusiness
Для деловых запросов: youtube@foxminded.ua
@SergeyNemchinskiy 3 місяці тому
🔥 Новый поток авторского тренинга Сергея Немчинского GRASP and GoF Design patterns стартует уже 01.02.2024 Регистрация 👉 go.foxminded.ua/4b4M3JI
@nikolayrykov 3 місяці тому
Сергей, запишите, пожалуйста, видео на тему 'Декомпозиция предметной области'. Существующее видео плохого качества, да и мало кто его найдёт на канале. А тема очень нужная!
@user-ym7zd2ni5o 3 місяці тому
Поддерживаю, будет очень интересно, с учетом нового опыта и актуальных примеров
@user-ut2lu1kz1v 2 місяці тому
Да, отличная идея
@ME-fv1ug 3 місяці тому
Было бы интересно узнать от Сергея Немчинского, подробный способ рефакторинга, от Мартина Фаулера, превращение метода в класс.
@shoorick77 3 місяці тому
01:15 Помимо переменной цикла (которая не только 𝑖, но и 𝑗, и даже 𝑘, когда циклы вложенные) есть ещё один случай - когда это координаты, тогда 𝑥, 𝑦 и 𝑧 - вполне нормальные имена. Есть ещё один случай - в языке Perl в функциях сравнения используют однобуквенные переменные 𝑎 и 𝑏, но это уже чисто перловая особенность, которая актуальна только внутри таких функций - в иных местах perlstyle и Perl Best Practices советуют не делать так.
@jewgenijmoldawski3306 2 місяці тому
Вот еще: c-counter, s-sum, l-length, n-name, e-exception
@shoorick77 2 місяці тому
@@jewgenijmoldawski3306 мне из всего этого набора часто попадалось только e.
@borisisavnin9983 3 місяці тому
Очень наглядно, лаконично и ёмко! Спасибо!
@alexandrapersukova 3 місяці тому
Спасибо вам!
@__ali__97 3 місяці тому
Сергей вы с этим циклом роликов про клин код прям в самое время! я только начал искать инфу и обучаться клин кодингу 👍
@alexandrapersukova 3 місяці тому
Делитесь потом успехами.
@evgeny910 3 місяці тому
Спасибо большое, интересно было бы так же посмотреть про рефакторинг
@alexandrapersukova 3 місяці тому
Отлично, запишем
@denisskyter4526 3 місяці тому
Все круто , Сергей , если можно , то давайте больше роликов с показание кода и вашем комментированием
@ozimandias1858 Місяць тому
Очень понравилось!
@user-ng6nx1uo5u 3 місяці тому
Спасибо за видео. Такой вопрос: как вы относитесь к такой штуке в VS Code как Codeium, я тут попробовал, мне понравилось, что он за тебя пишет ту часть, которую тебе лень писать, тебе остается только править под свои требования. Какие подводные камни, кроме потакания своей лени?
@valpalchenko3982 3 місяці тому
Интересно, расскажите про рефакторинг
@Sernik_z_rodzynkamii 3 місяці тому
Я джаваскриптизер, понравился анализ. Спасибо!
@hammerok3932 3 місяці тому
Больше примеров богу примеров! Спасибо.
@alexandrapersukova 3 місяці тому
😊
@DIMONIKOS 3 місяці тому
Продолжайте!
@alexandrapersukova 3 місяці тому
Будем)
@_GyG_ 3 місяці тому
Перемнные из одного символа, такие как X и Y, в контексте позиционирования элемента тоже понятны
@user-ym7zd2ni5o 3 місяці тому
9:58 Настал тот час, когда в разговоре Сергея появились украинизмы))
@alexandrapersukova 3 місяці тому
😅
@VanTwo 3 місяці тому
5:50, я бы сказал AddAndPrint это уже реализационное углубление в названии метода, чем нарушение srp. Что тоже является ошибкой 9:30, стоило бы так-же добавить что рекомендованная длина метода - до 20 линий
@vitaliy0192 3 місяці тому
Сергей, возможно вы сможете рассказать в своем видео. В интернете или у ИИ не смог найти ответ на вопрос. Вот чего я пока что вообще не понимаю: В вакансиях на java developer в требованиях часто указана kafka. Если я java developer, то кроме получения или отправки сообщений в топик kafka, что ещё мне надо знать или уметь?
@EngineeringLabChannel 3 місяці тому
Не впевнений в останньому прикладі, що створення "під капотом" об"єкта stream, ітерація по ньому обрахування суми (в деякому сенсі reduce) + додаткова робота для GC буде працювати швидше, як стверджує автор, за класичний for-loop. Щодо інших переваг зміненого коду - погоджуюсь.
@nabludatel4230 3 місяці тому
Сергей, запишите пожалуйста видео как строить код, как разбивать на функции , как выделять классы, конфигурации,
@kobalt-tv-777 3 місяці тому
Интересно было
@alexandrapersukova 3 місяці тому
Спасибо
@vahagn-gishyan 3 місяці тому
Считается ли нарушением принципа Single Responsibility, если метод одновременно выполняет определенные действия и осуществляет логирование процесса?
@user-ym7zd2ni5o 3 місяці тому
Мабуть, це найкраще корисне відео на каналі у цьому році. Грунтовне пояснення, доступні приклади. Треба збарегти його собі у закладки, і періодично переглядати.
@alexandrapersukova 3 місяці тому
ми раді, що вам це відео було корисним
@n0ikolay 3 місяці тому
Сергей, это лайк, это подписка, и даже мать его КОЛОКОЛЬЧИК на канал!
@alexandrapersukova 3 місяці тому
отлично! :)
@PredatorAlpha2 3 місяці тому
Как по мне, короткое название метода process() вполне уместно, особенно если класс наследуется от абстрактного AbstractProcessor, где этот метод абстрактный.. Но тогда нужно максимально понятное название подклассов. Тогда такие классы-процессоры можно вызывать, допустим, по расписанию, для выполнения некоторый действий.
@user-fb6of7uy4w 3 місяці тому
ине всё интересно)☺️
@alexandrapersukova 3 місяці тому
отлично, включите колокольчик, чтобы не пропускать новые видео и эфиры)
@Mr43046721 3 місяці тому
Интересно узнать про пример рефакторинга от М. Фаулера, про то, как красиво сделать метод классом
@alexandrapersukova 3 місяці тому
Ваш ответ записан. Сделаем! :)
@georgepro8481 3 місяці тому
Было бы хорошо поиметь исходный код в гите и его рефакторинг в бранче. Чтобы попытаться самостоятельно повторить весь путь.
@nikolaybaranov2213 3 місяці тому
Сергей! Спасибо за прекрасный очередной «урок» в доступном изложении. Clean Code - это то, что очень важно для нас, human’ов. Но что Вы скажете про тренд на то, что AI заменит кодеров и всё то, что для нас важно, станет вообще не важным? Вот первый попавшийся ролик-дискуссия на эту тему ukposts.info/have/v-deo/gaNoZJpqsKWC158.html Интересно Ваше мнение на тему актуальности олдскульных подходов. Думаю, оно достойно отдельного разговора.
@user-ho7rl2me3r 3 місяці тому
использование счетчика і в цикле - единственньій случай, когда значение переменной из одной букві всем понятно.. (1:30) class Point {double x, y, z;}; class Vector {double x, y, z;}; ну да, ну да...
@postoronny 3 місяці тому
В интерпретируемых языках короткие имена быстрее обрабатываются ;)
@user-qi3pr7df3x 3 місяці тому
Блин то класс это только "Моделька и поведение" все остальное от лукавого, то "Используйте simple domain model"
@spiritfrombook 3 місяці тому
Хотим видео про рефакторинг, превращение метода в класс
@user-un4on4oc3l 3 місяці тому
Будет понятно, переменная "х" и "y"))
@vermilinguas 3 місяці тому
Скажіть будь-ласка, а оплата частинами можлива? 😁
@alexandrapersukova 3 місяці тому
Вітаю! Вас цікавить тренінг GRASP and GoF Design patterns? На нього немає оплати частинами як і на інші тренінги. А якщо ви про менторинг, то оплата відбувається помісячно.
@vermilinguas 3 місяці тому
@@alexandrapersukova дякую за відповідь
@alexandrapersukova 3 місяці тому
@@vermilinguas І вам! Якщо ще будуть питання, то я буду рада відповісти)
@YacobMonar 3 місяці тому
По поводу первого примера("не сингл респонсибилити") метод возвращает результат. Если просто написать что-то типа System.out.println(d + e); это ж по сути выполнение одной задачи... Почему это не сингл матьевореспонсибилити!? Может в классе больше нечего будет принтить и потому нету смысла создавать отдельный метод...
@dim3787 3 місяці тому
Хлопцы, 15:04 - бегом за знаниями - дякуйте
@vasyltokarskyi5743 3 місяці тому
А де можна записатись на кар‘єрну консультацію?
@alexandrapersukova 3 місяці тому
Перше посилання після таймінгу в описі під цим відео
@romankuznetsov4601 3 місяці тому
книга "Чистый код" спорная во многом
@SemenNaduiev 3 місяці тому
Про тудушки не погоджусь, іноді виникає пробелма, яку потрібно вирішити в іншому таску, щоб скоуп твого таску не розростався, ти ствролєш ТУДУ, але З ПОСИЛАННЯМ на створену таску/багу, яка буде в скоупі якогось релізу або беклозі і буде пофікшена
@sergeypekar1058 3 місяці тому
Первый пример как будто обфускатором прошелся
@user-nu2jz1sb4s 3 місяці тому
Вот никогда, кстати, не понимал, зачем джависты в DTO пишут геттеры и сеттеры? Чего там инкапсулировать? Если поменяется формат входящих данных (или исходящих), по любому этот DTO и всё, что с ним связанно, будет переписано. Типа засовывать в DTO логику, чтоб старые геттеры работали с новыми данными? Но он тогда перестаёт быть DTO, разве нет?
@TheLoveKusano 3 місяці тому
инкапсуляция - одна из основ ооп. и да, через сеттеры можно реализовать валидацию, а через геттеры, например, иммутабельность
@user-nu2jz1sb4s 3 місяці тому
@@TheLoveKusanoНу если берём чистый паттер DTO, то он ничего валидировать не должен, он должен чисто содержать данные. "Data Transfer Object, в отличие от business object или data access object не должен содержать какого-либо поведения." - из вики.
@jewgenijmoldawski3306 2 місяці тому
В настоящее время, согласен, это пережиток прошлого т. к. то для чего были getter и setter сейчас решается по-другому.
@user-vu6hn4ul2i 3 місяці тому
12:50 к черту подробности и комментарии, откуда в этом методе взялась data? В параметрах я её не вижу, а если это член класса, то где this? 🤔
@user-if3cj1uf3d 3 місяці тому
это поле класса. В джаве для обращения к полю класса this необязательно. this используется только в случае, если есть несколько полей / параметров с таким именем, и нужно явно указать что нужно именно поле класса
@user-vu6hn4ul2i 3 місяці тому
@@user-if3cj1uf3d прикольно. Выглядит противоречащим философии Java, но, похоже, это так.
@user-if3cj1uf3d 3 місяці тому
@@user-vu6hn4ul2i не совсем понимаю что ты имеешь ввиду под философией джава. В ней сплошь и рядом встречаются места где что то можно не писать, если это не является необходимым
@postoronny 3 місяці тому
А я плохим кодом считаю сишные фигурные скобки. Не сами скобки, разумеется, а их расположение: когда открывающая расположена в конце строки, попробуй догадаться, какая из закрывающих в этой (а, может быть, и не в этой вовсе) лестнице к ней относится... Я всегда ставлю открывающую точно над закрывающей. Да, это требует лишней строки, зато код сразу становится читабельным.
@PredatorAlpha2 3 місяці тому
Вопрос привычки. Я раньше больше любил как Вы, сейчас для себя держусь правила - для коротких блоков до пяти строк без вложенных пишу по сишному, для более длинных и с вложенными - фигурные скобки одна под другой. Исключение - конструкции циклов и начало класса и методов - там по сишному всегда
@user-zi8zw3yf2t 3 місяці тому
1:30 j, k
@redneck_prm5429 3 місяці тому
Интересно, лет через десять до жаберов дойдет, что лепить везде геттеры и сеттеры лишено смысла, или так и продолжат цепляться за "а у нас так принято"?
@PredatorAlpha2 3 місяці тому
У нас, джавистов, часто используется доступ к полям объекта со стороны сторонних библиотек, которые ничего про объект не знают, кроме имени полей - например при сериализации или обработчики вьюх для MVC. И им гораздо быстрее работать через геттеры сеттеры, чем через голую рефлексию. Да и хотел бы я на тебя посмотреть, если тебе понадобиться подменить объект его проксей, например для ленивой загрузки по сети, если на нем не будет геттеров и сеттеров.
@redneck_prm5429 3 місяці тому
@@PredatorAlpha2 Если библиотека знает имена полей, то зачем там рефлексия? Да и лепить их где попало только на случай "а вдруг нам понадобиться" - как раз приводит к тому, что в итоге лепят уже просто потому, что "так принято", без подключения мозга.
@jewgenijmoldawski3306 2 місяці тому
Сейчас да, простые геттеры/сеттеры не нужны, т.к. доступ к полям в основном осуществляется через фреймворки. Раньше имело смысл, но эти времена прошли.
@sergeiseivach 3 місяці тому
Ну последний пример, такое себе... Был нормальный оверлоадинг, простой и понятный любому джуну, купившему вчера книжку по программированию. А такой синтаксический сахар с многоточиями и стримом из библиотеки, нужно пять раз перечитать прежде чем понять и удивиться, нафига так намудрили. И уж точно, в отличие от первого, не понятный программисту, пришедшему из другого языка.
@user-iv5co4lu9g 3 місяці тому
Джиннннееееееееееерикиииии эээхххххе хе хээээээээээ
@DimaVort 3 місяці тому
Я би клас із першого прикладу назвав counter а не calculator. Це краще відображає його суть.
@viacheslavkarpenko2162 3 місяці тому
Вітаю! ukposts.info/have/v-deo/a4GKarCZhZ-Jumg.html Поясніть будь ласка чому клас User не може мати методів save print а клас UserService має методи saveUser та printUser? На перший погляд можна реалізувати все в класі User. І наче логічніше винести зберігання, видалення, оновлення, редагування юзера в окремі сервіси наприклад SaveUserService і там зробити метод save. Таким чином клас матиме тільки одну причину зміни наприклад оновлення логіки збереження юзера. Дякую)
@YT_User376 3 місяці тому
У меня есть вопрос. Можно учить языки программирования в 14 лет?
@alexandrapersukova 3 місяці тому
Можно, а почему нет)
@jewgenijmoldawski3306 2 місяці тому
Я учил детей программированию. Из опыта: в 14 можно, но большинству приходится по ходу дела очень много объяснять вокруг, в зависимости от приложения. Например системы координат, если это игры.
@user-zy5nn1om9e 3 місяці тому
Сергію, ви ж хоч кажіть, що у вас є україномовний канал, ато я вже половину відео з нього подивився (через рекомендації), і весь час думав, що це відео з цього, основного каналу😅
@alexandrapersukova 3 місяці тому
Охх. знайшли, підписалися?)
@alexandr1408 3 місяці тому
200 строк кода?.. пффф, слабо как-то. На нашем проекте (достался по наследству) методы по 1000-2000 строк, притом это методы в самом контроллере. Прям портянка со всей непростой логикой. Думаю, всем и так понятно, что там со слоями, зависимостями, ответственностями и пр. никому ненужными придумками ))
@georgepro8481 3 місяці тому
" ... примеры кода будут на Джаве, ну а какой ещё язык я должен был выбрать ...". К сожалению Мартин Фаулер в переиздании 2019 года выбрал Javasript.. Предатель!
@dmitrysavkin5981 3 місяці тому
Про комментарии. Немцыв постоянно на немецком пишут, пишут названия методов на немецком. И ничего. Придет индус и будет матерится? А после индуса мы)?
@MERKYRIY-fn5rw 3 місяці тому
Ну дуже "початковий" рівень коду.
@alexandrapersukova 3 місяці тому
так, відео для початківців
@user-zi8zw3yf2t 3 місяці тому
Слишком простые и очевидные примеры
@arttex7yuart763 3 місяці тому
а можна таке ж відео тільки Українською? Бо нічого не зрозуміло.
@alexandrapersukova 3 місяці тому
якщо є запит, то спробуємо зняти. дякуємо за ідеї!
@arttex7yuart763 3 місяці тому
@@alexandrapersukova ви вважає це ідеєю? Українська мова - це вже ідея? Тоді питання, ви Українська компанія чи ні? Якщо ні, то питань немає, лише одне, для чого прапор у куті використовувати? Якщо ж Українська компанія, то чому за майже 2 роки повномасштабної війни, ви так і не перейшли на державну мову?
@alexandrapersukova 3 місяці тому
@@arttex7yuart763 У нас є окремий україномовний канал - Сергій Немчинський: Кодерська вітальня. А цей канал для всіх наших глядачів, які не знають української, а на рф, рб ми як і не працювали так і не працюємо.
@user-zi8zw3yf2t 3 місяці тому
И как же ты так быстро забыл русский язык?
@alexandrapersukova 3 місяці тому
@@user-zi8zw3yf2t это к чему вопрос?
@paul-emilelecoqdeboisbaudr555 3 місяці тому
Немчинский - самый опасный подвид инфоцыгана: если у Мишустина на лице клеймо инфоцыгана, то этот товарищ на вид - вполне адекватен
@user-qh5hz3jo4g 3 місяці тому
Аргументы?
@user-rb4ow1gz2v 3 місяці тому
Сергію, ви ж наче перейшли на українську мову.... Війна ще не закінчилася, ворог той самий, давайте продовжувати робити українське іт україномовним 😢
@smaxim02 3 місяці тому
Сергій зараз веде два канали. Цей лишився російськомовним, для інших країн, або людей які все ще розмовляють російською. Інший канал ведеться суто українською, частина контенту також перейшла туди. Питання в тому що(я так розумію) канали та ютуб забезпечують більшу частину від притоку клієнтів, а на україномовному зараз лише 10к підписників. Плюс це відразу мінус частина аудиторії з тої самої Прибалтики або Казахстану
@kirillperov3843 3 місяці тому
Так є окремій українськомовний канал
@alexandrapersukova 3 місяці тому
@@smaxim02 Все правильно, ми як не працювали з рф і рб так і не будемо, тому цей контент для людей з інших країн, які не володіють українською.
Sergey Nemchinskiy
Переглядів 19 тис.
Onliner
Переглядів 915 тис.
Sergey Nemchinskiy
Переглядів 32 тис.