Принцип хорошего кода KISS ("keep it simple, stupid", "keep it short and simple")

  Рет қаралды 83,385

Sergey Nemchinskiy

Sergey Nemchinskiy

Күн бұрын

О чем говорит принцип кода KISS ("keep it simple, stupid", "keep it short and simple"), что делать не стоит, а что делать стоит, чтобы код был хорошим.
Курс, о котором идет речь в видео: PYTHON - bit.ly/2HaK8tU
Другие курсы для новичков:
JAVA - bit.ly/3koGeMj
JAVA Start - bit.ly/32GgZ28
Инструментарий JAVA - bit.ly/3mws6Cr
Automation QA (Java) - bit.ly/3myHDlp
ANDROID - bit.ly/2FMc86s
C#/.NET - bit.ly/32GM19U
C# START - bit.ly/2H84pQF
FRONT-END - bit.ly/3kq1vVR
WORDPRESS Developer - bit.ly/2RyADqp
SALESFORCE Developer - bit.ly/3koGLOj
UI/UX дизайн - bit.ly/2RFsK28
Project management - bit.ly/32GnAt7
Обучение на проекте - bit.ly/3mvBwhB
Продвинутые курсы для состоявшихся девелоперов:
GRASP and GoF Design patterns - bit.ly/3mvKifo
Enterprise patterns - bit.ly/2FLZOD4
Сайт Foxminded: bit.ly/2RIchdz
Foxminded в ФБ: / foxmindedco
FoxmindEd в Instagram: / foxminded.ua
Foxminded в VK: foxminded
Мой Telegram: t.me/nemchinskiyOnBusiness
Мой блог: www.nemchinsky.me
0:00 - вступление
0:25 - в чем заключается принцип KISS
0:53 - не имеет смысла реализовывать дополнительные функции
1:17 - не стоит перегружать интерфейс теми опциями, которые не будут нужны большинству пользователей
4:05 - бессмысленно делать реализацию сложной бизнес-логики, которая учитывает абсолютно все возможные варианты
5:12 - не имеет смысла беспредельно увеличивать уровень абстракции
6:02 - бессмысленно закладывать в проект избыточные функции «про запас»
8:34 - рекламная пауза
9:37 - не стоит подключать огромную библиотеку, если вам от неё нужна лишь пара функций
11:12 - не стоит подключать огромную библиотеку, если вам от неё нужна лишь пара функций
11:50 - абсолютная математическая точность или предельная детализация нужны не всегда
12:38 - precision design is fragile
16:35 - Принципы простого дизайна от Кента Бека

Пікірлер: 252
@user-dl3xy5lf7u
@user-dl3xy5lf7u 3 жыл бұрын
Расскажите, пожалуйста, когда наследование а когда делегирование:)
@aleksanderm1947
@aleksanderm1947 3 жыл бұрын
И что это вообще такое
@user-be2cy2fb9u
@user-be2cy2fb9u 2 жыл бұрын
@@aleksanderm1947 азазаза, тестировщик ...)
@ultraduck100
@ultraduck100 Жыл бұрын
@@user-be2cy2fb9u html coder
@KonstantinSafutin
@KonstantinSafutin 3 жыл бұрын
Спасибо за ваш труд! Расскажите потом пожалуйста про делегирование и наследование и что где следует применять)
@user-cx1ve6df5q
@user-cx1ve6df5q 3 жыл бұрын
Работодателя: - Как вы относитесь к Kiss? Я: *поцелуй* Работодатель: - Эээ вы не так меня поняли, но вы приняты...
@MrPesiik
@MrPesiik 3 жыл бұрын
Работодателя: - Как вы относитесь к Kiss? Я: ай воз мейд фо ловинг ю бэйби! Работодатель: - Эээ вы не так меня поняли, но вы приняты...
@olehdiatlenko4463
@olehdiatlenko4463 3 жыл бұрын
я-то думаю почему ни разу не слышал на собеседовании об этом принципе
@hands_in_code
@hands_in_code 3 жыл бұрын
как же шикарно наткнуться на этот канал! Спасибо!
@pavel_savelov
@pavel_savelov 3 жыл бұрын
Отличный преподаватель! Вчера даже сел после видео задачи решать. Спасибо за интересный контент.
@sultanbaimuratov4748
@sultanbaimuratov4748 3 жыл бұрын
Сергей, вы молодец, спасибо вам за контент
@akkh6971
@akkh6971 3 жыл бұрын
Спасибо вам! Я только учусь, но вы так интересно рассказываете, что даже новичку интересно!
@vladsmith4981
@vladsmith4981 Жыл бұрын
Серёг, отличная подача, приятно слушать, спасибо за твой контент А теперь я понадеюсь что у тебя великолепное чувство юмора: Я половину видео думал что ты "подглядываешь в монитор" который "висит слева сверху" :) Потом конечно всё понял
@PythonDevelopment
@PythonDevelopment 3 жыл бұрын
Спасибо, Сергей. Один из лучших каналов, хоть я и питонист))
@Dmitrii-Zhinzhilov
@Dmitrii-Zhinzhilov Жыл бұрын
Сергей, благодарю!
@user-rg6gd9xh7u
@user-rg6gd9xh7u 3 жыл бұрын
Очень полезное видео, спасибо !
@lyloo6577
@lyloo6577 3 жыл бұрын
Еще хотелось бы добавить поговорку "Усложнять просто, упрощать сложно", многие программисты не следуют принципу KISS потому что им лень думать, как сделать проще и понятнее. Пишут код в стиле "что вижу то пою"
@Mr43046721
@Mr43046721 3 жыл бұрын
Под чаёк в обед самое то. Ждем байки про то, когда лучше юзать наследование, а когда делегирование!
@Brick87Game87
@Brick87Game87 3 жыл бұрын
Спасибо за видео! Про наследование было бы прямо очень интересно =)
@olegssh6452
@olegssh6452 3 жыл бұрын
Это то что мне нужно было посмотреть очень давно, столько времени ушло в чёрную дыру🥲
@Nik-jj6bz
@Nik-jj6bz 3 жыл бұрын
слушаю с интересом. расскажите про делегирование и наследование, будет полезно. спасибо.
@apristen
@apristen 2 жыл бұрын
да всё просто: делегируешь код джуну, он его наследует... :-D)))
@Nik-jj6bz
@Nik-jj6bz 2 жыл бұрын
@@apristen очень спасибо. 👍
@adeusexmachina
@adeusexmachina 3 жыл бұрын
Кстати пример с полями для ИНН и скайпа - это как раз на самом деле яркий пример КИСС. Вот было поле для ИНН и программист не делал его валидацию или проверки, потому что ему это не задавали. И вот конечный пользователь уже нашёл этому полю применение не по назначению. Как только бизнес-требования расширились (или опомнились что не делали валидацию и нанесли в систему мусора) - добавили проверку поля. И это благо, что все эти поля с ценными уникальными данными не стёрлись системой, которая сочла их ошибкой. А если бы программист проявил инициативу заранее сам и предусмотрел применение поля не по назначению - он бы отошел от КИСС, но при этом не позволил бы применять поле не по назначению. Часто продакт-менеджер не обладает всей широтой представления о поведении пользователей. И валидация полей - это одно из немногих, чего часто нет в бизнес требованиях, но должно быть с точки зрения опыта программиста. Как минимум валидация показывает самому оператору, что он неввёл одну цифру или опечатался и продублировал одну цифру. Поэтому принципу КИСС нужно следовать умно и валидацию делать не через задницу, а так чтобы пользователь мог ввести любой текст, а на выходе получить уведомление об ошибке если она есть, форматируя как нужно внутри системы и не озадачивая этим пользователя. Главное нужное количество цифр. А если в бизнес требованиях вообще нет проверок и функциональностей, которые с точки зрения программиста должны быть как базис диалога пользователя и системы - он должен уточнить это отдельно и зафиксировать: "да, тут валидацию не делаем, а вот тут делаем". И чем больше программист сам обладает опытом нахождения подобных уязвимостей или слабостей интерфейсов, тем больше он добавит проверок, которые не заложил менеджер. Опять же сейчас всё обсуждается с продакт менеджером и оунером, если делается по аджайлу.
@aleksprimetv
@aleksprimetv 3 жыл бұрын
Самое интересное запросить у программиста поле для ИНН, а вводить туда скайп и потом еще возмущаться...
@adeusexmachina
@adeusexmachina 3 жыл бұрын
@@yagami23100 если бы его организация потеряла все данные из валидированных полей - это было бы своеобразным злом, и именно в таком контексте должен был быть озвучен тот случай. Типа вот программист делал по КИСС и не предусмотрел заранее, а пользователи потом потеряли данные. Но автор сделал другой вывод: что программист и дальше не должен был валидировать поле ради удобства тех, кто применил его не по назначению. Это уже двойной стандарт и непоследовательность. Концепция Кисс не должна быть выше обновляющихся бизнес-требований для конечного исполнителя, как предлагает автор, типа ну раз сделали изначально без валидации (согласно Кисс) то и пусть навеки-вечные так и остаётся для сохранения лояльности клиентов. Или что, программист должен был пойти наперекор новым бизнес требованиям? Спорить с решениями оунера и продакта? Встать на сторону клиентов, которые там хранят свои скайпы, телефоны, счета банков, адреса и прочую муть, засоряющую базу данных?))
@adeusexmachina
@adeusexmachina Жыл бұрын
​@@chikenmacnugget В одном из последних видео про уход из программирования он рассказал, что давно перешел на руководящую должность в своем бизнесе про обучение и не пишет код. Это значит уже начал утрачивать некоторые важные нюансы мышления программиста уровня мидла-сеньора, которым особенно в стартапах как раз и платят выше, чтобы они закрывали своим опытом дыры в ТЗ, а не следовали КИСС, мол, дак не просили же, вот я и не сделал. Осознанное применение КИСС не делая валидаций если их нет в ТЗ - это признак хитрожопости - ведь потом за разгребвание мусора из системы тоже придется платить деньги. Но это простительно стажеру джуниору, который только обретает полноценное мышление и опыт и ему дополнительные задачи брать в тягость. А по хорошему при наборе в команду мидлов и сеньоров нужно делать тест на мышление. И между хитрожопым КИСС-истом и опытным Абстракционистом выбирать второго, главное все его инициативы заверять перед реализацией, добавляя в ТЗ и смету задним числом. Это реально ценные кадры.
@savannah633
@savannah633 3 жыл бұрын
Приятный по софт общению Сергей Немчинский !) А главное по объяснению - ничего не потеряно.
@user-gq7gl3dv2r
@user-gq7gl3dv2r 3 жыл бұрын
Сергей, спасибо за ваши видео, очень интересно и полезно. Выпустите видео про наследование и делегирование.
@drandru83
@drandru83 3 жыл бұрын
Нашел пару фраз что бы поспорить, но опыт говорит что это для спора в курилке, а не на всеобщем обозрении ))) Категорически плюсую видео!
@exploidrice787
@exploidrice787 3 жыл бұрын
Дядя Сережа, перед тем как рассказать про "разницу использования наследования и делегирования", расскажите про делегирование, что, как, почему. Коротко и понятно, как вы умеете - спасибо)))
@romantsyupryk3009
@romantsyupryk3009 3 жыл бұрын
Спасибо вам большое за ето видео.
@Smartorcute
@Smartorcute 2 жыл бұрын
Спасибо большое!
@ruslansivalnev209
@ruslansivalnev209 3 жыл бұрын
а давай про наследование и делегирование! - звучит прям как то чего не хватало)
@vsleontyev
@vsleontyev 3 жыл бұрын
Круто, что я живу по этому принципу
@PodzolovDm
@PodzolovDm 3 жыл бұрын
Keep it simple, suka
@ikorjefocur
@ikorjefocur 3 жыл бұрын
Данный принцип в первую очередь больше всего пересекается с моим любимым принципом "х.х. и в продакшн".
@user-tf5bs6de6z
@user-tf5bs6de6z 3 жыл бұрын
Всё POOP (practical object-oriented programming) такое.
@olegkot3362
@olegkot3362 3 жыл бұрын
Сергей, хотелось бы как можно больше примеров из Вашего опыта, как и в этом видео
@SergeyNemchinskiy
@SergeyNemchinskiy 3 жыл бұрын
стараюсь :)
@YujinK11
@YujinK11 3 жыл бұрын
Видосик про наследование +++
@georgekravtsov6198
@georgekravtsov6198 3 жыл бұрын
Запишите, пожалуйста, про монолит!
@vladgonchar
@vladgonchar 3 жыл бұрын
Не всегда согласен с теоретическим мнением и выводами ведущего. Но что несомненно ценно - его богатый практический опыт! Слушаю его "байки про былое" и нахожу их очень полезными, особенно для молодых программистов. Например, да - пользователи могут использовать программу таким "странным" образом, какой и не приходил в голову разработчику программы.
@OlavLadnav
@OlavLadnav 3 жыл бұрын
В разработке проектов многое зависит от заказчика, его требований и области применения продукта. Например, если работаешь с постоянным заказчиком длительное время и знаешь его "особенности". Например, когда в изначальном ТЗ от заказчика не присутствуют определенные фичи, а 2 дня после релиза заказчик просит - "Ой, а нам тут нужно ещё это, это и это", ты добавляешь, делаешь релиз, через 2 дня просьба повторяется. Со временем, уже зная специфику своих клиентов мы начали добавлять в проекты тонну фич, которые могут понадобиться заказчику, в результате он доволен как слон. Но это единичный случай, повторюсь, всё зависит от менталитета заказчика.
@vladzaiko5012
@vladzaiko5012 Жыл бұрын
@Sergey Nemchinskiy Так что по итогу с видео про делегирование ?
@user-jj5rt7dd9e
@user-jj5rt7dd9e Жыл бұрын
Много параметров в конфигурация нужна что бы потом адаптировать под нагрузку и скорость. Размер порции выполненой за раз, скорость обновления отрисовки. Параметры для системных админов. И иногда приходится для них написать интерфейс чтобы туда не вносили не допустимые значения(устранять конфликтующие значения).
@niolonq
@niolonq 3 жыл бұрын
16:17 Сделать второй телефон дополнительный. А так, заполнение данных, мне кажется, должно иметь минимально необходимые данные и кучу всяких приписок и дополнений на усмотрение пользователя
@7453060
@7453060 3 жыл бұрын
В системах написанных студентами или динозаврами я встречал перегруженность конструкциями, которые были не задействованы или не доделаны вовсе. Причиной тому было что бросались на задачу как быки на ширму, сам был таким. Далее встречал только контрактный принцип. Даже когда с foris billing делали "расчёты побратски" все ненужное скрыли за фасадом.
@daniil4299
@daniil4299 3 жыл бұрын
Принцип KISS очень субъективен, для одного кодера одно просто и очевидно, для другого другое. Нет общих стандартов простоты )))
@Freqat
@Freqat 3 жыл бұрын
Сергей, у меня вопрос не по теме. Что вы можете сказать про RxJava и про технологии ректиного программирования?
@Elenasn
@Elenasn 3 жыл бұрын
какая стильная футболочка у этого розового мишки, сидящего возле чашки :)
@f1am3d
@f1am3d 10 ай бұрын
Принципы KISS и YAGNI в отношении задач работают только тогда, когда задачи написаны корректно и исчерпывающе. А не так, когда тебе говорят - "ну мы тут описали все основное, а ты дальше сам должен понимать что требуется клиенту". Потомо делаешь все по инструкции без отступлений, и получаешь такой диалог: - "А почему вот этого нет?" - "Потому что этого нет в требованиях." - "Так это же и так понятно!"
@user-tr8xe1oo5n
@user-tr8xe1oo5n 3 жыл бұрын
Сейчас как раз джанго начал изучать самостоятельно. Скажите пожалуйста а сколько ж курс стоит? А то я по ссылке не нашел цену
@olego5633
@olego5633 2 жыл бұрын
Хм, насчёт валидация. Мы писали биллинг для телефонной станции. Однажды выяснилось что несколько абонентов при поиске по фамилии не попадают в результат. Как оказалось, вместо букв Ч были цифры 4, вместо О цифра 0. Мы офигели, Сразу исправили, накинули валидатор. Оказалось что операторы абонентского отдела были старые телеграфистки, а в телеграфе есть замена букв цифрами для экономии символов.
@at0m1x191919
@at0m1x191919 3 жыл бұрын
Класс.
@user-mr1ii2wn2w
@user-mr1ii2wn2w 3 жыл бұрын
Просим про делегирование и наследование
@demt_kern
@demt_kern 3 жыл бұрын
Расскажите, куда делась вязаная лисичка?)
@semremal
@semremal 3 жыл бұрын
Lombok - это не только автогенерация. Это и @SneakThrows, и исключение сгенереных методов из репорта code coverage. Кроме того, сгенеренный руками код (пусть даже и в IDE) надо поддерживать. К примеру, не забывать добавлять новые поля в toString(). Lombok позволяет этого избегать.
@andrewzaitsev6668
@andrewzaitsev6668 3 жыл бұрын
Видео Must have для большинства java-программистов кодящих на spring и spring boot.
@user-ey5xw2nx9s
@user-ey5xw2nx9s 2 жыл бұрын
Для всех программистов :)
@als-creator
@als-creator 2 жыл бұрын
По имеющимся сообщениям, акроним был придуман Кларенсом Джонсоном, ведущим инженером Lockheed Skunk Works (создатели Lockheed U-2, SR-71 Blackbird и многих других). В то время как уже несколько десятилетий популярно использование расшифровки «Keep it simple, stupid», Джонсон расшифровал KISS как «Keep it simple stupid» (без запятой) и эта трактовка до сих пор используется многими авторами (в английском языке, в отличие от русского, запятая используется для обособления (выделения) обращения достаточно редко). В ней не было никакого скрытого смысла, что инженер был глуп; как раз наоборот. Этот принцип лучше всего иллюстрируется историей, когда Джонсон вручил команде инженеров-авиаконструкторов набор инструментов, поставив им условие: механик среднего уровня должен суметь отремонтировать реактивный самолёт, который они проектировали, в полевых условиях только с этими инструментами. Таким образом, «stupid» относится к отношению между тем, что всё ломается, и сложностью необходимого для этого ремонта.
@lego12239nn
@lego12239nn 3 жыл бұрын
Насчёт проверки формата тел.номера. Забьём на неё и огребём в будущем проблемы, когда захотим что-то в автоматическом режиме с ним делать. Например, искать дубли, завести тел.книжку, звонки через встроенный sip, сбор статистики и т.п. Поэтому нифига подобного - входные данные, которые должны иметь какой-либо формат(типа номер телефона или email), должны и проверятся относительно его и приводится к нему(убираем тире или наоборот расставляем однообразно в тел.номере и т.п.). Если надо иметь несколько номеров, то пожалуйста - разрешаем писать несколько через запятую. Но формат надо проверять. Опыт автоматизации подобных вещей показывает, что в будущем с подобными полями(которые не проверялись сразу) могут быть проблемы.
@GenaTolstij
@GenaTolstij 3 жыл бұрын
Про валидацию полей очень несогласен. Юзер не должен вносить в поля то, что не предусмотрено. Но он должен это не делать с первого дня жизни системы, конечно если он 5 лет писал Скайп в ИНН, а тут резко ему зарезали опцию, то это нельзя, но если б тебе изначально говорилось "иди в жопу, пиши цифры", то хрен бы ты туда скайпов напихал, а скорее написал бы разрабу "Бэн, ай нид хелп, добавь поле ИНН мне нада" и всё. Плохо не то, что он валидацию добавил, а то, что не сделал это изначально.
@vladgonchar
@vladgonchar 3 жыл бұрын
Можно перекрикивать как метод доказательства своей правоты. Но опыт подсказывает, что в большинстве случаев валидировать надо только те поля, значения которых впоследствии будут использоваться/читаться другими программами автоматически и на этих значениях будет строиться другая бизнес логика.
@GenaTolstij
@GenaTolstij 3 жыл бұрын
@@vladgonchar , привет xss дыркам
@SoccerSorcerer
@SoccerSorcerer 2 жыл бұрын
Совершенно согласен. Ну и ещё одна проблема - незнание авторами упомянутого софта потребностей своих клиентов, и, как следствие, вывод нерелевантной информации без возможности кастомизации
@agentk2023
@agentk2023 3 жыл бұрын
Подробнее плиз о делегировании
@kehula3404
@kehula3404 3 жыл бұрын
зачастую, один и тот же софт продается разным заказчикам, и потом для заказчиков делают доработки - вот и появляется куча различных настроек, которые нужны только кому -то одному. Монолит, лучшая архитектура в мире, гениальные аналитики-архитекторы - наше всё
@agidel2012
@agidel2012 3 жыл бұрын
про делегирование и наследование было бы здорово
@alex_chugaev
@alex_chugaev 3 жыл бұрын
Гугл в помощь
@skynowa2626
@skynowa2626 3 жыл бұрын
где купить хороший микрофон?
@Epenckorn
@Epenckorn Жыл бұрын
блин, во смотрю сейчас свой курсач 2009 года, консольные крестики нолики на С++ на 2500 строк и свою личную CMS на 800 строк на PHP 2019 года. Смеяться и плакать хочется.
@nikolayvinogradov3566
@nikolayvinogradov3566 3 жыл бұрын
Здравствуйте, Сергей. Говорят, что хороший стиль написания кода - это писать его от интерфейсов. И такой вопрос в каких случаях лучше применять интерфейсы к классам? Не нужно же писать интерфейсы и тыкать их в программе для всех классов.
@SergeyNemchinskiy
@SergeyNemchinskiy 3 жыл бұрын
вообще-то нудно. но писать именно от интерфейсов очень сложно
@7453060
@7453060 3 жыл бұрын
Стоп! Если строим систему с нуля мы же первым делом пишем интерфейсы, потом тесты, тестируем парадигму на устойчивость и только после этого приступаем к конкретной реализации. Как в данном контексте можно даже лишний параметр получить?
@turchik5763
@turchik5763 3 жыл бұрын
Сергей, а не будет ли видео про Xamarin?
@AlexS-gn9tq
@AlexS-gn9tq 3 жыл бұрын
он не специалист по .NET, а про мобиьную разработку и тем более Xamarin вообще не в курсе. Сергей Java dev из мира кровавого энтерпрайза.
@VladiqLot
@VladiqLot 2 жыл бұрын
Нужно понимать где заканчивается KISS, и начинается лень с пофигизмом😀
@alexxx4434
@alexxx4434 3 жыл бұрын
Я нахожу это ироничным, что видео про KISS само не следует принципу KISS. )
@FerelUltra
@FerelUltra 3 жыл бұрын
Не следует, потому что разжевывает?
@maxlich9139
@maxlich9139 3 жыл бұрын
фолыч первый и второй - топ!) (я про автарку)
@venom5583
@venom5583 3 жыл бұрын
Так уж устроен мозг, что нейронные связи формируются лучше через сотню примеров и уточнений, чем одно утверждение. Иначе все бы учились по справочнику.
@alexxx4434
@alexxx4434 3 жыл бұрын
@@venom5583 Через сотню примеров и уточнений уже забываешь/замылевается первоначальный принцип, погребенный под кучей всей этой второстепенной информации. "Знание некоторых принципов легко возмещает незнание некоторых фактов" - К. Гельвеций Кстати, это изречение я опять же от Лиса услышал )
@venom5583
@venom5583 3 жыл бұрын
@@alexxx4434 Но как ты получишь знание без примеров и уточнений на что эти принципы распространяются, а на что нет, чтобы принцип не превращался в религию? Вот к примеру сказали тебе KISS YAGNI и ты будучи студентом понимаешь это как не писать геттеры, сеттеры, интерфейсы, хардкодить константы магическими числами для экономии строк) Так что знание принципов не возмещает незнания фактов если эти принципы противоречат друг другу и у тебя недостаточно опыта чтобы определить оптимальный компромисс между ними.
@Andrey_Knoroz
@Andrey_Knoroz 3 жыл бұрын
Нужны ли знания дискриминантной математики в изучении языка Java ?
@vadympylypchenko5801
@vadympylypchenko5801 3 жыл бұрын
Ещё один голос за inheritance vs composition
@user-zl5fj4vl1t
@user-zl5fj4vl1t 2 жыл бұрын
что не так с хибернейтом для валидации бинов? У хибера же есть проект Hibernate Validator, который специально для этого сделан? Геттеры и Сеттеры это мусор в коде, лучше заменить аннотацией. Да и ломбок работает только во время компиляции - его зависимостей не будет после её окончания.
@user-qv4hn6qq4n
@user-qv4hn6qq4n 3 жыл бұрын
В KISS тоже надо знать меру и писать аккуратно и дальновидно. Вот пример с портом из видео, да можно захардкодить порт и это будет работать, а можно сделать свойство (гетер, сетер) "PORT". которое будет так же возвращать всего лишь захардкоженный порт, дополнительных трудозатрат нет, но при этом, потом, если тебе надо будет получать этот сраный порт каким-то способом с подвыподвертом, тебе не придется искать по всему приложению где же ты использовал этот порт.
@SergeyNemchinskiy
@SergeyNemchinskiy 3 жыл бұрын
согласен. Я под "захордкодить" имел в виду, конечно, вынести это значение в константу и ее везде использовать
@user-qv4hn6qq4n
@user-qv4hn6qq4n 3 жыл бұрын
@@SergeyNemchinskiy стоит уточнить, а то зрители будут потом рассказывать "Немчинский разрешил" :)
@charinav
@charinav 3 жыл бұрын
Наследование? Делегирование?
@realex2007
@realex2007 8 ай бұрын
Даже с позиции теории надёжности, чем система проще тем она более надёжная и безотказная.
@skarmendor4827
@skarmendor4827 3 жыл бұрын
С одной стороны безумно плюсовал, но потом подумал... Работаю рук.группы эксплуатации/техпода. И в работе сталкивался с таким числом жути... Но если к ней родимой приложить эти принципы, то с небольшой натяжкой - подходят) Но эта "натяжка" как раз и создаёт максимум гемора. Часто разрабы нахардкодят так, что потом добавить небольшие элементы удобства обслуживания этого добра - это будет аж изменение архитектуры. И потом каждую команду разрабов и продуктологов приходится убеждать вложиться и переделать - чем раньше, тем лучше... Особенно при интеграционных взаимодействиях, чтобы можно было прикрутить и мониторинг, и логи нормальные и элементы контроля админки... А все потому что задачи ставили от бизнес потребностей с принципом кисс Упуская поддержку в принципе ( Да, или косяк архитекторов, или бизнес уговаривает удешевить... Ведь это дорого. Хотя потом оказывается ещё дороже, когда приходится или переделывать, или нанимать и обучать в огромном количестве сходящих с ума операторов. В общем и целом - принцип при наличии мозга реально работает. Но, имхо, для многих смотрящих и некоторых говнокодеров целевой мыслью может показаться, что проще - это главное. По крайней мере, такой червячок остался после двух просмотров. Был бы рад вашим комментариям или вообще дико плюсовал отдельному видео по учёту нюансов, важных для дальнейшей поддержки продукта, если он не мелкий или не планируется таким в конечной точке. А вообще - спасибо за все, что вы делаете!
@adudyak
@adudyak 2 жыл бұрын
ржу с голоса, который задействован для "а давайте" программистов
@maxlich9139
@maxlich9139 3 жыл бұрын
Еще порой люди любят усложнят логику методов, классов и их взаимодействия
@user-se8gj4yu2b
@user-se8gj4yu2b 3 жыл бұрын
Неудачный пример с полем для идентификатора ИНН. На самом деле они были правы, что типизировали его. Вот представьте, что кто-то вписал в поле ИНН со скайпа ник, который может быть понятный ему одному, и кто-то отправил таки в налоговую отчет с ИНН в виде имени скайпа или же какой-то другой программист решил облегчить работу бухгалтера и написал классную программу, упрощающую отправку отчетов в налоговую. Как Вы думаете какая будет реакция у налоговой? А какая будет реакция у бухгалтера, когда вернулась ошибка из налоговой на следующий день, а время сдачи отчета было вчера? Главная их ошибка была, это то что нужно было изначально типизировать это поле. Ваша же ошибка была в том, что если в этом поле использовался ИНН, то нужно было использовать его по назначению, а не говорить, что ах какие не хорошие разработчики, решили провалидировать это поле.
@MrVagrod
@MrVagrod 3 жыл бұрын
- Как решить проблему с избыточным количеством уровней абстракции? - Добавить новый уровень абстракции
@mityaneversky4247
@mityaneversky4247 3 ай бұрын
Возможно они стали использовать поле ИНН для общения с другими сервисами принимающие строгий формат.
@IvanIvanov-wn7ep
@IvanIvanov-wn7ep 2 жыл бұрын
KISS конфликтует с возможным требованием расширяемости и переиспользования в будущем. Обычная диалектика. И между этими сторонами нужно соблюдать баланс. Если конечно продукт не в стиле "сделал и долой с плеч"
@user-ef4yx3mu2q
@user-ef4yx3mu2q 3 жыл бұрын
А не получается ли, что KISS противоречит SOLID? в частности в вопросах наследования и использования интерфесов, которые заранее предпологают их использования для созданий новых классов в будущем?
@0imax
@0imax 3 жыл бұрын
При правильном подходе к разработке интерфейс является принадлежностью того, КТО вызывает, а не того, КОГО вызывают. В таком случае, если реализация интерфейса сильно не отклоняется от требований вызывающей стороны, то всё нормально. Опять же, всё зависит от конкретной ситуации. В одной использование целого фреймворка ради одной маленькой функции будет неприемлемо, в другой это может быть нормальным решением.
@nocomments9061
@nocomments9061 Жыл бұрын
@@0imax "того, КТО вызывает" -- вы имеете ввиду интерфейс зависит от прав пользователя в программе? Тогда на каждое право свой interface и они дальше комбинруются, и динамически добавляются пользователю при логине?
@0imax
@0imax Жыл бұрын
@@nocomments9061 Речь про программный интерфейс, не графический.
@nocomments9061
@nocomments9061 Жыл бұрын
@@0imax я про программный тоже, понял вас так.
@0imax
@0imax Жыл бұрын
@@nocomments9061 речь не о правах доступа. Вот Вы пишите программу, вызывая у объектов какие-то методы. Другой программист пишет сами эти классы. Набор методов (т.е. интерфейс) этих классов - это Ваша принадлежность, Вам должно быть удобно их вызывать. Таким образом можно легко заменить одни классы на другие, просто сделав ещё одну реализацию того же интерфейса. Если же интерфейс, т.е. набор методов, диктует разработчик вызываемых классов, то для замены одного класса на другой придётся весь вызывающий их код переписать под интерфейс нового класса.
@user-fh8yv8bq1j
@user-fh8yv8bq1j 3 жыл бұрын
То что ты думаешь , не то что вводит заказчик. . Чем меньше строк кода тем меньше багов.! круто. но ! чем больше красивых строк Python тем больше в x^степине больше багов?
@OlegKiyashko
@OlegKiyashko 3 жыл бұрын
код удаленный из проекта но оставшийся в гите, там и останется навечно. при необходимости восстановления функционала будет написан заново если конечно не найдется ктото, кто помнит, что такое есть...
@user-xx1dm8qb5r
@user-xx1dm8qb5r 3 жыл бұрын
не в противоречие принципу KISS, но не могу не заметить: что касается полей валидации - их таки нужно валидировать очень аккуратно и точно (и вобще все, что приходит от пользователя), потому что в один прекрасный день в поле вы получите что-то типа "do_bad_shit(); :)
@user-dj8fs3kz2j
@user-dj8fs3kz2j 3 жыл бұрын
Плиз 'когда наследование а когда делегирование'
@VasiliyZavoyko
@VasiliyZavoyko 3 жыл бұрын
Вообще данный принцип, в указанной редакции, прерогатива архитектора и аналитиков. В коде обычно говорят о ненужной сложности функций.
@DmitriySergeev
@DmitriySergeev 3 жыл бұрын
Сергей, очень интересные видео делаете! Большое вам спасибо! Позвольте дать совет, подумайте не стоит ли исключить английские слова в разговорной речи, типа "really"..
@vladislavlen9307
@vladislavlen9307 2 жыл бұрын
Аналогия с рабочими - весёлая. Это же ровно как и в работе программиста бывает.
@DVBLEX
@DVBLEX 3 жыл бұрын
go и що видео про YAGNI
@qr46654
@qr46654 3 жыл бұрын
Вот так, раньше, в прошлом веке, например, винда умещалась на нескольких дискетах, эксель 4.0 - на одной и мог запускаться прямо с неё!, а сейчас..., но функционал изменился не настолько радикально
@0imax
@0imax 3 жыл бұрын
Это наглядная демонстрация нарушения принципа KISS, т.к. тот же эксель большинство пользователей не используют и на 10% всех его возможностей))
@qr46654
@qr46654 3 жыл бұрын
@@0imax Эксель - это массовый продукт "для всех', а у каждого потребителя - свои потребности, вот каждый и использует необходимые ему возможности. Размеры растут за счёт импорта ненужных библиотек и всего того, что было сказано в этом ролике. Функционал, с того времени, - почти не изменился.
@0imax
@0imax 3 жыл бұрын
@@qr46654 Если что - это была шутка :)
@qr46654
@qr46654 3 жыл бұрын
@@0imax шутки шутками, но это не прикольно, когда нужно тратить деньги, получая, в основном, "улучшения" в виде визуальных эффектов
@0imax
@0imax 3 жыл бұрын
@@qr46654 Это да. Скайп до попадания в руки m$ был вполне приличным...
@user-kw5sz3mg1s
@user-kw5sz3mg1s 3 жыл бұрын
Наверно в силу того что я не программист не совсем согласен с выпуском. Может большая часть принципом предназначена для оутсорс разработки. Для продуктовой компании, которая собирается развиваться десятки лет надо заложить все что можно, если на это требуется разумное количество ресурсов
@TatsuLeo
@TatsuLeo 2 жыл бұрын
Валидация в поле ИНН обязательна.
@digitalsocium9638
@digitalsocium9638 3 жыл бұрын
Расскажите пожалуйста подробнее
@SergeyNemchinskiy
@SergeyNemchinskiy 3 жыл бұрын
о чем именно?
@digitalsocium9638
@digitalsocium9638 3 жыл бұрын
@@SergeyNemchinskiy ну дык о чем в ролике говорилось, сказали просить - я прошу, соблюдаю социальный контракт по повышению метрик под роликами в обмен на контент =)
@FirstNameLastName-fz8pg
@FirstNameLastName-fz8pg 3 жыл бұрын
Здравствуйте, мои дорогие, меня зовут Сергей Сириусли... )))
@ni55an
@ni55an 3 жыл бұрын
2:10 "Сирили"?
@viperviper7775
@viperviper7775 2 жыл бұрын
физические константы хардкодить? и жить без sv_gravity?
@xxxbunx
@xxxbunx 2 жыл бұрын
Максимально просто, как в Linux/Unix.
@13Balck
@13Balck 2 жыл бұрын
Не сделал валидацию на телефон, а через год прилетела таска сделать автоматическую звонилку, у нас же база номеров есть...
@user-yu3qo5um5k
@user-yu3qo5um5k 3 жыл бұрын
маечка типа i am stupid ибо мелкий шрифт плохо виден
@DimaVort
@DimaVort 3 жыл бұрын
Очень сомнительно, что разработчики по своей инициативе добавили ту валидацию ИНН. Скорее всего это было по заявкам пользоватетей. Есть такой отчет 1ДФ. Кому и на сколько ты продал или оказал услуг. Вот если там в поле ИНН ошибешься, то в лучшем случае лишний раз побегаешь по кабинетам налоговой, а в худшем попадешь на штраф. Раз программа была давно, то никаких Ме ДОК-ов тогда не было и отчеты надо было на дискетках таскать и в распечатке.
@alexeffect1765
@alexeffect1765 3 жыл бұрын
16:17 ахахахах
@durbich
@durbich 3 жыл бұрын
Про ненужную валидацию: У меня попытка заказа пиццы иногда превращается в квест, т.к. сайты требует ввести название улицы в адресе, а в моём мухосранске улиц нет. И всё. И меня сайт посылает. Ладно, ввожу "улицы нет". Сайт проверяет по гугл карте и выдаёт, что в моём мухосранске нет улицы с названием "нет улицы" (ДА ПОТОМУ ЧТО В МОЁМ МУХОСРАНСКЕ НЕТ УЛИЦ. Просто город FOO, дом BAR)
@venom5583
@venom5583 3 жыл бұрын
В мухосранске без улиц есть доставка пиццы с сайтом?
@devcrghome7061
@devcrghome7061 3 жыл бұрын
понапридумали же... Считаю доп функционал нужен, это продукт. А вот при доработке - только ТЗ, ни шагу в сторону.
@jgkdmdevienjjgg8866
@jgkdmdevienjjgg8866 3 жыл бұрын
это называется оверенжениринг. если делать максимально просто и тупо - получится полная херня неподдерживаемая. тут скорее дело такое что если ты хочешь запихнуть какую нибудь дополнительную функциональность и она прямо сейчас не нужна - нужно четко понимать понадобится она в ближайшем будущем или нет. если да, то лучше ее сделать. но это надо четко осознавать. очередной принцип который если бездумно применять везде - получится хрень. нужно везде балансировать)
@vladislavkotsuba7202
@vladislavkotsuba7202 3 жыл бұрын
Согласен с Вашим высказыванием, но я бы добавил. Если вещь понадобится в ближайшем будущем, код был должен быть организован таким образом, чтобы добавление этой "будущей" функциональности требовало минимальных усилий. Т.е не нужно сразу реализовывать все что может понадобиться. Код должен быть гибок к новым изменениям.
@alekseyshibayev5243
@alekseyshibayev5243 3 жыл бұрын
Почему всего 10 процентов лайков? Вы что не уважаете Сергея?
@vitalijuskolinko9011
@vitalijuskolinko9011 3 жыл бұрын
Где грань между всем понятным и сложным кодом? Мне, например, stream или Atomic'и легко, а другому - космос.
@WalkHB2
@WalkHB2 3 жыл бұрын
2:30 Только хотел сказать, что, по факту, любой современный фреймворк (из мира PHP точно) нарушает принцип KISS, потому что имеет в себе сильно больше, чем нужно для 90% задач, где этот фреймворк используется. С другой стороны, в велосипедах, даже простых, тоже ничего хорошего нет. В общем, нет идеальных решений, во всем есть недостатки.
@my-little-shorts
@my-little-shorts 3 жыл бұрын
Но 10% всё ещё где то используется, а значит этот функционал нужен. Автор видео говорил о том, что не нужно делать то, что не просили или то, что никто не будет использовать в ближайшем будущем. Но если хотя бы 1% пользователей например использует какой то функционал который большинство не использует - это уже много.
@ScreedRed
@ScreedRed 3 жыл бұрын
Так а зачем использовать фреймворк, от которого требуется только 10% функционала?) Проще уж подключить нужные либы и использовать их. P.S. Это ваш субъективный взгляд. Очень много работаю с Laravel, за всё время пригодился, пожалуй, весь его функционал, кроме Mix. На большей части проектов так же используется сильно больше чем 10%.
Правильные методы по Clean Code
28:29
Sergey Nemchinskiy
Рет қаралды 77 М.
Navigating Healthcare Data
43:07
MongoDB
Рет қаралды 9
where is the ball to play this?😳⚽
00:13
LOL
Рет қаралды 8 МЛН
Uma Ki Super Power To Dekho 😂
00:15
Uma Bai
Рет қаралды 44 МЛН
I MADE A CARDBOARD SWING!#asmr
00:40
HAYATAKU はやたく
Рет қаралды 30 МЛН
Из PHP в Go или как уйти втуда и невернуться
23:36
Дамп чердачины
Рет қаралды 2,6 М.
Принцип хорошего кода YAGNI ("You aren't gonna need it")
15:08
Принцип хорошего кода DRY (dont repeat yourself)
16:20
Sergey Nemchinskiy
Рет қаралды 70 М.
where is the ball to play this?😳⚽
00:13
LOL
Рет қаралды 8 МЛН