Александр Бармин - SOLIDный чистый код на простых примерах

แชร์
ฝัง
  • เผยแพร่เมื่อ 22 ต.ค. 2024

ความคิดเห็น • 91

  • @Isildan88
    @Isildan88 2 ปีที่แล้ว +4

    Боже, насколько же это офигенный материал! 🤩 Автор, спасибо тебе от души!

  • @katorabian
    @katorabian 4 ปีที่แล้ว +11

    Чудесное выступление, отличная подача. В силу опыта, я не смог всё переварить, но Вы, Александр, меня очень впечатлили) никогда еще не видел настолько хорошо подготовленных докладов)

  • @bakhbk
    @bakhbk 4 ปีที่แล้ว +3

    Отличный содержательный доклад, без воды!

  • @theAntoshka
    @theAntoshka 4 ปีที่แล้ว +7

    Очень интересный, доступный и подробный доклад, спасибо!

  • @sysadm85
    @sysadm85 4 ปีที่แล้ว +12

    Хороший доклад, видно что рассказчик неплохо разбирается в теме!

  • @МихаилБолотов-н5о
    @МихаилБолотов-н5о 4 ปีที่แล้ว +8

    Самый недооцененный доклад о принципах SOLID!

  • @lexbekker
    @lexbekker ปีที่แล้ว +7

    На 24:25 мы захардкодили в класс через new наши фильтры, и теперь класс:
    1) Отвечает за создание списка фильтров и самих фильтров, которые мы будем применять к посту (Мы буквально явно создаём объекты внутри метода)
    2) За применение созданного списка фильтров к посту
    3) За сохранение нашего поста.
    Не много ответственности для одного класса?

  • @ЮраЧорнота
    @ЮраЧорнота 3 ปีที่แล้ว +1

    Очень крутой доклад, без воды и понятно

  • @poorshakespeare7099
    @poorshakespeare7099 3 ปีที่แล้ว

    В первые три минуты уже услышал дельные вещи. Лайк сразу.

  • @BC-uj4nu
    @BC-uj4nu 2 ปีที่แล้ว

    Потрясающий доклад. Очень структурированно, понятно и наглядно. Особенно с последней буквой (D).

  • @Облакоизроз
    @Облакоизроз 2 ปีที่แล้ว +1

    Талантливо!!! Жаль, что нет собственного канала на TH-cam 🙁

  • @СтаниславСидоров-ы6м
    @СтаниславСидоров-ы6м วันที่ผ่านมา

    Чистый код очень субьективная вещь, в вопросах речь зашла про золотую середину, проблема в том что ни одной достоверной матемвтически доказанной метрики/теоремы для этого нет, с солидом схожая история вопросы из зала в часности это подсветили, так же они не совсем универсальны и было бы лучше если бы они были переведены на прикладной уровень конкретного языка, например на RoR на котором я сейчас пишу код нет чистокровных интерфейсов да есть депенденси иньекшен как способ заигрывания с интерфкйсами или абстракциями но это не тоже самое, самое важный патерн это публичная оферта в рамках продукта - брейнсошиал

  • @natanagar
    @natanagar 4 ปีที่แล้ว +11

    Принцип Liskov Substitutional вообще об иерархии и о предсказуемости поведения в иерархии

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      Можете расшифровать ? Принцип до боли простой конкретный, вы написали нечто очень абстрактное

    • @sergeydostovalov6180
      @sergeydostovalov6180 3 ปีที่แล้ว +1

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

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      @@sergeydostovalov6180 вообще мимо Liskov Substitution

    • @sergeydostovalov6180
      @sergeydostovalov6180 3 ปีที่แล้ว

      @@isey2851 а тут что тогда написано? ru.wikipedia.org/wiki/%D0%9F%D1%80%D0%B8%D0%BD%D1%86%D0%B8%D0%BF_%D0%BF%D0%BE%D0%B4%D1%81%D1%82%D0%B0%D0%BD%D0%BE%D0%B2%D0%BA%D0%B8_%D0%91%D0%B0%D1%80%D0%B1%D0%B0%D1%80%D1%8B_%D0%9B%D0%B8%D1%81%D0%BA%D0%BE%D0%B2

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      @@sergeydostovalov6180 чел, явно не то, что ты написал

  • @vitaliiplagov
    @vitaliiplagov 3 ปีที่แล้ว

    Хороший доклад. Спасибо!

  • @valeriyemelyanov9090
    @valeriyemelyanov9090 4 ปีที่แล้ว +1

    Интересно. Спасибо!

  • @НиколайСтецюк-в8з
    @НиколайСтецюк-в8з 3 ปีที่แล้ว

    ОООчень полезное видео для меня оказалось, так как изучаю java )))

  • @qdreadknight
    @qdreadknight 6 หลายเดือนก่อน

    Отличный доклад. Как написать код, в котором куча классов, между которыми сложно понять, что вообще происходит

  • @alexmur07
    @alexmur07 4 ปีที่แล้ว +2

    24:52. можно еще сделать чтоб интерфейс фильтра имел дефолтную реальзацию принимал объект preparationservice и вызывал у него метод инит в который передавал бы себя а там происходило бы добавление в лист объекта фильтра

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      Вы сейчас описали Registry Pattern.

    • @bormanbor8740
      @bormanbor8740 ปีที่แล้ว

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

    • @lexbekker
      @lexbekker ปีที่แล้ว +1

      @@bormanbor8740 По идее, нужно создавать новый класс, в котором создаёшь новый список фильтров и применяешь их к изображению, но да, слишком много ответственностей на один класс

  • @ViseDen
    @ViseDen 2 ปีที่แล้ว

    35:47 похоже ошибка, в "Реализации сервисного слоя" имплементация сервиса зависит от интерфейса DAO, а наоборот

  • @НикитаЗамалютдинов
    @НикитаЗамалютдинов 2 ปีที่แล้ว

    В моменте с комментарием не проще было через event (Observable) запилить ? Сервис нотификаций подписан на эвент комментария - гораздо логичнее чем городить какие то левые классы. Об этом кстати был вопрос и ответ. Делегат круче работает в данном конкретном примере.

  • @dmitryf6069
    @dmitryf6069 2 ปีที่แล้ว +5

    Принцип разделения интерфейсов объяснен правильно, но пример показан совершенно обратный сказанному. Интерфейс оставили один, только добавили к нему дополнительный метод. Теперь все реализующие классы должны будут тоже изменить свои реализации.
    Принцип инверсии зависимостей вообще никак не связан ни с интерфейсами, ни со Спрингом, например. Вопрос в том, получает ли класс свои зависимости сам, или ему их предоствляют. Таким образом DI можно осуществить и полностью на конкретных классах, без абстрактных классов или интерфейсов.
    По остальным принципам тоже есть вопросы...

    • @Ajanni55
      @Ajanni55 2 ปีที่แล้ว +2

      ты путаешь dependecy inversion c ioc/dependency injection

  • @pashnyovv
    @pashnyovv ปีที่แล้ว +2

    неймінг з Impl в кінці те що треба 😂

  • @marinajuzova1356
    @marinajuzova1356 3 ปีที่แล้ว

    i, j, k -это говорили что имена переменных целые , если начинались с этих букв в FORTRAN

  • @timarejepkuliev7532
    @timarejepkuliev7532 2 ปีที่แล้ว

    Классный ментор

  • @eaglesm1993
    @eaglesm1993 4 ปีที่แล้ว +1

    на скорости 1.5 вообще пушка

  • @alexdayneko5084
    @alexdayneko5084 3 ปีที่แล้ว

    Почему пост и комменты содержат ссылку на юзера, а не наоборот? Это результат какого-то долгого анализа или есть какое-то правило?

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      у поста или коммента один овнер, а у юзера их может быть много. ManyToOne/OneToMany

    • @alexdayneko5084
      @alexdayneko5084 3 ปีที่แล้ว

      @@isey2851 но ведь с точки зрения реального мира, у юзера есть посты и комменты, тогда именно юзер должен содержать список постов или комментов. Мы же сейчас не про БД, а про модель в бизнес-логике.

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      @@alexdayneko5084 не. Если говорить про объектную модель, то нет. Комменты у поста это не свойство поста, как и посты у юзера. А вот у коммента свойства аля text, header, publicationDate, publisher и тд

    • @alexdayneko5084
      @alexdayneko5084 3 ปีที่แล้ว

      @@isey2851 все равно не понимаю. Может быть есть какие-нибудь книги или статьи с объяснением? Например, почему не может быть список постов свойством юзера? Это же отношение композиции, ведь пост юзера не может существовать без юзера

    • @isey2851
      @isey2851 3 ปีที่แล้ว

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

  • @garikhakobyan3013
    @garikhakobyan3013 ปีที่แล้ว

    где доступны коды?

  • @ChoVasche
    @ChoVasche 3 ปีที่แล้ว +1

    Поднимите руку, кто считает, что лучше утром, а не вечером.

  • @MrGribovod
    @MrGribovod 6 หลายเดือนก่อน

    9:22 Train-o-Gram

  • @FF-ne2qz
    @FF-ne2qz 2 ปีที่แล้ว +6

    Ребзя, вы все такие рассказываете о солидах и так далее. Но куда не прийди на проект - кругом пизда и адища. Еще и применять эти принципы на практике не пробовали?

    • @cheevauva
      @cheevauva 2 ปีที่แล้ว +2

      Ну так очевидно же, SOLID это про чистый ООП, а то что у автора в презентации это процедурка завернутая в классы(сервисы).

  • @NickRuster
    @NickRuster 2 ปีที่แล้ว

    смотря данное видео я чувствую что я умнею XD прилив сил

  • @mishgunplay6783
    @mishgunplay6783 ปีที่แล้ว +1

    9:47 - отношения без направлений на диаграмме классов - жирный дизлайк.

  • @ostanislav
    @ostanislav 3 ปีที่แล้ว +3

    Неверно истолкованы принципы solid, а именно l и i, коллеги, проверять такие вещи надо перед публикацией...

    • @ghefest3817
      @ghefest3817 3 ปีที่แล้ว +6

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

  • @thatlinuxguy9902
    @thatlinuxguy9902 3 ปีที่แล้ว +4

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

    • @dmitryf6069
      @dmitryf6069 3 ปีที่แล้ว +2

      Особенно на галере. Там друг другу мердж-реквесты не отклоняют.

    • @lexxx1994
      @lexxx1994 3 ปีที่แล้ว

      только потом настанет момент, когда нельзя уже реализовать быстро, а то и вовсе нельзя реализовать без перепиливания. Говнокод - это техдолг

  • @LordZiegfrid
    @LordZiegfrid 4 ปีที่แล้ว +3

    Слишком быстро говорит. Без пауз с перемотками невозможно понять.

    • @protiv_bio
      @protiv_bio 4 ปีที่แล้ว

      На 1.5 самое то. Слайды на паузу, ну круто что есть пауза

  • @millfreedom
    @millfreedom 4 ปีที่แล้ว +20

    Отвратительный доклад. Монотонная речь, не раскрыты принципы SOLID (L, I - вообще не по теме, докладчик и сам, очевидно, не понимает этого принципа), примеры - не выразительные...

    • @GamDevRus
      @GamDevRus 2 ปีที่แล้ว +1

      и не говори, он же поверил дяде бобу, который неверно сформулировал свои принципы SOLID, на самом деле дядя боб имел ввиду совсем не то ..

  • @arthurmorgan8328
    @arthurmorgan8328 5 ปีที่แล้ว +4

    Вступление жесть

  • @danilx6239
    @danilx6239 ปีที่แล้ว

    Кажется этот доклад для тех, кто уже разбирается в принципах SOLID...
    Почему слайд с описанием всех 5ти принципов так быстро проматывается?
    Кажется автор делал доклад на 1 час 20 минут, а ему сказали ужать на 45 минут. Дизлайк.

    • @pavelb2451
      @pavelb2451 3 หลายเดือนก่อน

      Тоже заметил такое. Смотрел на скорости воспроизведения 1.5.
      Может из-за этого?

  • @vladimirlazarev2267
    @vladimirlazarev2267 3 ปีที่แล้ว +2

    3:40
    Это всё рассказанное - не стандарты, а практики, техники, паттерны...
    А стандарт - тоже есть. Уже лет 40 наверное.
    ЕСПД называется :)
    14:43
    Серьёзно - стало лучше?
    Было, пусть и в циклах, но - всё по делу и любой джун поймёт что написано.
    А предлагается добавить в код какие-то сущности, не относящиеся к задаче: stream. flatMap, map, collect...
    И всё для чего? Чтобы код стал понятен только тем, кто знает, что это за сущности?

    • @lexxx1994
      @lexxx1994 3 ปีที่แล้ว +6

      по поводу второго - так это же стримы из Java 8, непонятны только для тех, кто с ними не работает.

    • @vladimirlazarev2267
      @vladimirlazarev2267 3 ปีที่แล้ว +2

      @@lexxx1994 Это понятно, но всему своё место.
      Усложнять код только "потому что могу" - так себе идея.

    • @lexxx1994
      @lexxx1994 3 ปีที่แล้ว

      @@vladimirlazarev2267 согласен, "keep it simple, stupid" никто не отменял)

    • @isey2851
      @isey2851 3 ปีที่แล้ว

      @@vladimirlazarev2267 читаемость кода увеличилась в разы. Kiss никуда не делся, а писать код с кучей вложенных циклов потому что нет других идей никак не тянет на ход конём

    • @lexbekker
      @lexbekker ปีที่แล้ว

      Создаётся только одна сущность - коллекция, а всё, что вы описали - это методы Stream API, функциональное программирование, когда ты вместо того, чтобы писать КАК РЕАЛИЗОВАТЬ, пишешь буквально ЧТО РЕАЛИЗОВАТЬ, а добрые дяденьки уже реализовали простые вещи за тебя, такие как фильтрация, преобразование одного объекта в другой и т.д.

  • @dark_evil77
    @dark_evil77 4 ปีที่แล้ว +1

    мечтаю попасть в команду где нет говнокодеров!

    • @GamDevRus
      @GamDevRus 2 ปีที่แล้ว

      нет таких :))

    • @pavelb2451
      @pavelb2451 3 หลายเดือนก่อน

      Спустя 3 года.
      Мечтаю попасть в команду. Можно с говнокодерами.

  • @ВиталийГордиенко-ч2к
    @ВиталийГордиенко-ч2к 4 ปีที่แล้ว +5

    максимально тяжело слушать несвязанную речь.

  • @ForestBeast1
    @ForestBeast1 ปีที่แล้ว +1

    Отличная реклама epam (нет)
    Из полезного - только рекомендация книги

  • @ДарьяЗакамская-б4ъ
    @ДарьяЗакамская-б4ъ 2 ปีที่แล้ว

    Метод getLikeNotificationsRecipients(), добавленный для того, чтоб избежать нарушения принципа L, сам нарушает принцип L ...

    • @Mikst74
      @Mikst74 15 วันที่ผ่านมา

      это точно, этот метод прям кровь из глаз вызвал. Пост ничего не знает про какие то там уведомления, не то, что кому из посылать, даже о их существовании не догадывается. 🤦‍♂️

  • @ОлександрПаскар-ж6х
    @ОлександрПаскар-ж6х 2 ปีที่แล้ว

    Очень крутой доклад, без воды и понятно!!!