Делаем тестовое задание, которое дали на собеседовании

Поделиться
HTML-код
  • Опубликовано: 1 ноя 2024

Комментарии • 78

  • @Voshod___
    @Voshod___ 4 месяца назад

    Было очень познавательно и интересно, спасибо! =)

  • @SeliverstovMusic
    @SeliverstovMusic 10 месяцев назад +2

    проверки isFirst/isSecond лучше сделать через наследование от Parent
    class Parent:
    def isFirst(self):
    return isinstance(self, First)
    @property
    def isSecond(self):
    return isinstance(self, Second)
    @isSecond.setter
    def isSecond(self, value):
    raise AttributeError

  • @Geskawary234
    @Geskawary234 11 месяцев назад +1

    Огромное спасибо, мне как раз классы надо повторить было. Тут как раз вся база!

  • @dmitryshkinin8030
    @dmitryshkinin8030 Год назад +2

    Идея для контента очень интересная. С удовольствием подобное бы смотрел.
    Поставил на паузу, решил примерно за 45 минут. Выложил на гитхаб (но коммент с ссылкой был удален). Сейчас смотрю как задачу решил автор :)

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад

      Да, ютюб удаляет комментарии где ссылки .(

    • @julesbois2122
      @julesbois2122 7 месяцев назад

      Тоже поставил на паузу.. Решал несколько часов) За два подхода, ночами. Решил, теперь боюсь смотреть и на свой код, и продолжение видео.
      upd:
      гитхаб.ком /trinitatis/test_case_by_Pomidor

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  7 месяцев назад

      Хах. Нормально! Главное что получилось! Я думаю что это задача не на красоту кода, и не на то чтобы разгадать какую-то тайную схему абстракций.
      Просто такая , какая есть)

    • @julesbois2122
      @julesbois2122 6 месяцев назад

      @@SeniorPomidorDeveloper Я, кажется, тоже потерял свой комментарий, на котрый Вы уже ответили: сначала он был, а позже я отредактировал его, хотел вставить ссылку на гитхаб, но не напрямую, а на русском, через пробелы, чтобы не выглядело как ссылка. Но, ИсКин Ютуба раскусил меня - удалили весь коммент целиком))
      Ну да ладно.

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  6 месяцев назад

      @julesbois2122 хах. Бывает )

  • @vladimirsmolkotin2619
    @vladimirsmolkotin2619 11 месяцев назад

    Интересное задание. Мне бы точно пришлось потратить на это несколько часов. А про property вряд ли бы вспомнил. Спасибо за контент

  • @ChristineJohnston-ud9ej
    @ChristineJohnston-ud9ej Год назад

    important tnks!

  • @nikolaymatveychuk6145
    @nikolaymatveychuk6145 5 месяцев назад

    Вот потому и не позвонили, видимо :) Кроме того, что код должен работать, он ещё и должен быть адекватно структурирован. Ну правда, у Вас isFirst определяется в дереве наследования не на том же уровне, на котором возникает сам класс First :) То же самое и с Second. Но методы isFirst и isSecond нельзя реализовывать в самих классах First и Second, так как это породит сильное зацепление двух классов, которые знать друг о друге ничего не должны. Однако принципы SOLID запрещают определить поведение в Parent, а потом First и Second сделать его потомками и там изменить поведение. Из этого делаем вывод, что Parent, First и Second не связаны родством, а являются классами одного уровня абстракции. При этом First и Second - это классы-состояния, а Parent поставляет интерфейс определения этого состояния. Потому решено: A и B должны иметь множественное наследование Parent+First и Parent+Second, при этом Parent должен поставлять методы isFirst и isSecond, которые проверяют self на isinstance соответствующего класса. Возможно авторы теста предполагали что-то другое, но я только такую структуру вижу адекватной, без лишних связей и с возможностью расширения.

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  5 месяцев назад

      Да наверное я просто не понял это задание. Надеюсь что они сами его поняли. Надо уже их пригласить в студию, пусть расскажут наконец)

  • @TieoZ
    @TieoZ Год назад +2

    Я бы ещё предложил методы из фёрст и из секонд перенести в классы Фёрст и Секонд (по названиям немного напрашивается), и я бы возможно ещё засунул в парент бульку из фёрст, и в конструкторах фёрста и секонда устанавливал её либо в тру либо в фалс, и в методах проверки превращал бы это в инт.
    Это уберёт магические числа из методов, и должно быть прозрачно для понимания потому что код находится в классах фёрст и секонд.
    И да, на падение тестов это не повлияет, но мы же понимаем что если тесты не пройдут код даже читать не будут, а вот если пройдут, то далее будут смотреть уже качество кода и навёрнутую архитектуру, а на сколько кандидат разгадал заложенные в тесты идеи.
    И в завершение вопрос, а щас подобная практика в найме встреречается, и если да то есть примеры актуальных заданий?

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад +2

      Тут уже несколько человек написали и каждый в этом задании увидел какую-то свою идею) это как в песнях Егора Летова ) А мне вот не кажется что тут была какая-то заложенная идея, которую кандидат должен разгадать. По моему , тут все очень просто.
      По поводу магических чисел и переменных в одну букву, они не плохи сами по себе. Плохо когда за ними скрываются какие-то осмысленные значения, смысл которых мы таким образом теряем. А к примеру, в математических функциях и задачках только магические числа и отдельные буквы и используют . По тому что изначально они не несут какого -то предметного смысла. И в этой задаче тоже очевидно что его нет.
      По поводу того, какие сейчас дают задания на собеседованиях, не могу сказать. Я их уже много лет не проходил. Я их сейчас провожу и даю там задание на разработку кое -какого мини-приложения.

    • @TieoZ
      @TieoZ Год назад +1

      @@SeniorPomidorDeveloper Ну так в этом же и смысл, докопаться до истины обсудить детали и разобрать всё по косточкам.
      А идеи имхо должны быть заложены, иначе зачем задание, просто проверить что человек понимает в наследование, знает что такое эксепшен и что существует декоратор проперти? Гораздо интересней посравнивать как люди интепретируют представленный пример TDD, какими архитектурными приёмами владеют (и именно владеют, а не просто знают, потому что тут это надо увидеть и применить, а не описать), имхо это самая ценная часть задание, и кстати если в конторе хорошо построена TDD, то задача может быть весьма близка в боевой работе, и она становится максимально релевантным заданием.
      Кстати понравилась идея с фабриками, и я бы даже предположил что А и Б это не классы, а просто функции возвращающие объекты Фёрст и Секонд, но их нейминг намекает на обратное, по этому наверно всё же нет.

    • @mikeofs1304
      @mikeofs1304 Год назад

      @@TieoZ А если добавятся еще и Third b тд и вов сех них городить is{number} методы, учитывая что по заданию полагается что любой наследник паррента их реализует? Можно конечно это в паренте прописать, но это то же плохой дизайн поскольку из идеологии ООП полагается что родитель ничего не должен знать о наследниках. Тут миксин как высказался человек уже выше подходит лучше всего

    • @TieoZ
      @TieoZ Год назад

      @@mikeofs1304 Я исхожу из логики что инфу о том что в методах из Фёрст и из Секонд стоит опираться на некоторую общую бульку которая устанавливается для классов Фёрст и Секонд (и по этому само поле как общие данные мне хочется поднять в Парента и устанавливать при вызове супер конструктора). Кстати идея с третим класом натолкнула на мысль что это должна быть не булька, а инт.
      На счёт миксинов, не сразу понял как их сюда приложить, но пока писал это полотно, и обдумывал варианты, наконец понял что имелось в виду (да, с начала не понял, но мне можно, я не программист :Р) И вообще да, это выглядит логичнее, за одним нюансом, разве не является в таком случае проблемой то, что для проверки на инстанс этот миксин должен знать о всех наследниках на которые он должен проверять инстанс в методах?

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад

      Вы столько написали интересных идей, что теперь просто обязаны предоставить свои варианты 😀

  • @andreyo.945
    @andreyo.945 Год назад

    Интересно

  • @Фома-ж7я
    @Фома-ж7я 10 месяцев назад

    Вы говорили, что есть у вас на Udemy курсы, думаю стоит ссылку на них оставить где-то в своем ютуб (в описании к видео или на странице сообщество). А то вдруг там есть что-то интересное, ведь ваши курсы на ютубе - огонь!

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  10 месяцев назад

      Да пока так ничего и не получилось снять для Udemy ( делал один курс, но не доделал

  • @ВячаФродо
    @ВячаФродо Год назад +1

    Случайно зашел на ваш канал. Очень понравилась идея решения тестовых заданий в видеороликах. Можете утешить мой интересен, ответив на вопрос: "Сколько у Вас лет практики в коммерческих проектах и какой стек технологий?"

  • @ivanlino3747
    @ivanlino3747 9 месяцев назад

    Привет, расскажи пожалуйста про свой последний опыт поиска работы (за границей), что спрашивают?) Искал ли когда либо работу на американскую компанию (через посредника или самостоятельно)? Спасибо.

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  9 месяцев назад +1

      Да, работал и в Американских и в других заграничных компаниях, но очень давно работу не искал. Раньше просто на Хабр карьера и прочих Российских ресурсах находил такие вакансии. Сейчас надо как-то по-другому искать. Говорят что LinkedIn неплохо работает в этом смысле

  • @marcelosacred
    @marcelosacred 10 месяцев назад

    Выложите курс и на степик пожалуйста. Люди из РФ не смогут на юдеми их купить

  • @ibrahimoglu
    @ibrahimoglu Год назад

    👍

  • @albukerke3322
    @albukerke3322 Год назад +1

    Эх какие простые задания были…
    Сейчас просят сайт создать на джанго, с бд и логикой, и прочим.. короче задание на недельку) и это на интерна..

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад

      😄

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад

      Для сайта на Джанго есть курс )

    • @Gribori
      @Gribori Год назад +1

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

    • @igorratnik2357
      @igorratnik2357 Год назад

      ​@@Griboriи таких контор пруд пруди.

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад +2

      Вообще конечно офигели. У меня как-то давно была начальница , которая попросила сделать тестовое дизайнера , мобильное приложение дизайн, потом не взяла ее на работу, а результат отдала iOS программисту , он немножко охренел от такого и уволился )

  • @KiberHawk
    @KiberHawk Год назад +3

    У меня для вас плохие новости :) За 10 лет вы так и не решили задачу :)
    1. Очевидно что Parent - корень иерархии классов.
    2. Если присмотреться к формулам, то очевидно, что обе формулы используют некое свойство (третий множитель), а значит это свойство общее для А и В, а значит это свойство (i) должно находится в Parent
    3. Тогда у Parent должен быть конструктор с аргументом, который через наследование Second долетает до B.
    4. Тогда конструктор А (без аргументов) - это перегруженый конструктор Parent с default значением.
    5. Очевидно что А.func - это перегруженый метод B.func. А значит B.func должен находится в Parent, a A.func может перегружаться либо в А, либо в First
    6. Очевидно что isFirst и isSecond - относятся к классам First и Second

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад +3

      😭

    • @KiberHawk
      @KiberHawk Год назад +3

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

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад +2

      🤣

    • @nikolaymatveychuk6145
      @nikolaymatveychuk6145 5 месяцев назад

      Вы не сможете сделать Parent основой иерархии классов не нарушив принципы SOLID и не породив ненужного зацепления между First и Second. По крайней мере я не смог. И не смотря на то, что я у автора видео вижу тоже ну прям очевидные ошибки (как Вы и указали, например, что у него isFirst и isSecond это методы, которые явно появляются слишком поздно, когда First и Second классы давно существуют). Но всё же у Вас тоже ошибка в том, что Вы решили от Parent всё унаследовать, ну или я не вижу чего-то, что видите Вы, и потому у меня Parent в основании всё ломает, а Вы смогли как-то обойти эту проблему. Если да, поделитесь со мной, пожалуйста, как Вы это решили.

    • @KiberHawk
      @KiberHawk 5 месяцев назад

      @@nikolaymatveychuk6145 Ну давайте разбираться.
      Из справки по isinstance:
      The isinstance() function checks if the object (first argument) is an instance or subclass of classinfo class (second argument).
      Ассерты в строчках 20 и 22, говорят нам что класс А является подклассом First и одновременно подклассом Parent. Множественного наследования нам не завезли, поэтому возможны два варианта. A extends First и First extends Parent, или A extends Parent и Parent extends First.
      Запомним эти варианты, посмотрим на такие же ассерты для B: В extends Second и Second extends Parent, или B extends Parent и Parent extends Second.
      Явно видно что вариант A extends Parent и Parent extends First и B extends Parent и Parent extends Second. Невозможен по причине отсутствия множественного наследования.
      У нас остаются три варианта:
      1. A extends First и First extends Parent и B extends Parent и Parent extends Second
      2. A extends Parent и Parent extends First и В extends Second и Second extends Parent
      3. A extends First и First extends Parent и В extends Second и Second extends Parent
      Первый и второй варианты выглядят очень запутанными иерархиями, к тому же имя класса Parent, явно на что-то намекает :D

  • @artemunix5223
    @artemunix5223 10 месяцев назад

    микросервисные монолиты скоро покажешь как делать?)

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  10 месяцев назад

      Я только монолитные микросервисы умею )

    • @artemunix5223
      @artemunix5223 10 месяцев назад

      @@SeniorPomidorDeveloper покажешь как умеешь мы посомтрим :D

  • @clauseclause6640
    @clauseclause6640 Год назад +7

    Ответ не принял бы, потому что у классов есть общее поведение, но оно не вынесено в родительские классы, а реализация не консистентная (в одном случае num, в другом - с, в одном случае через self обращаешься, в другом через имя класса. isFirst и isSecond очевидно должны быть вынесены в миксин и возвращать не 0 или 1 а True или False (должна быть проверка на isinstance в методе)

    • @clauseclause6640
      @clauseclause6640 Год назад +2

      assert(0) я бы тоже поправил на assert 0

    • @mikeofs1304
      @mikeofs1304 Год назад +1

      да , задание намекает на то, что испытуемый должен прежде всег овостановить ирархию классов и логику ее построения, в идеале видимо понять что эт овообще такое, а не просто зачекаться по ассертам. Например вероятнее всего i это какая то базовая характеристика сущности, и 3 ее значение поумолчанию, а иначе она инится явно в классе - том же B.Ну и как вы замечаете это должно быт ьв паренте

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад +1

      @mikeofs1304 присылайте варианты. Интересно, какая это могла бы быть иерархия классов.

    • @mikeofs1304
      @mikeofs1304 Год назад +1

      @@SeniorPomidorDeveloper хорошо завтранакидаю, а то в РФ ночь уже)) Сейчас кстати дошло что A и B это вообще не классы скорее всего, а фабрики. Потому что рефлексией проверяются не они

    • @mikeofs1304
      @mikeofs1304 Год назад +1

      @@SeniorPomidorDeveloper По поводу сигнатуры абстракт метода можно и лучше сделать, она слишком широкая, но лень уже на него время тратить
      from abc import ABC, abstractmethod
      class MyError(Exception):
      """custom error"""
      class Parent(ABC):
      def __init__(self, i=3):
      self.i = i
      @abstractmethod
      def fnc (self, *args) -> int:
      """function for extending"""
      class First(Parent):
      def fnc(self, one: int) ->int:
      if one == 7:
      raise MyError("Error text")
      return one**2*self.i
      class Second(Parent):
      def fnc(self, one: int, two: int) ->int:
      return one*two*self.i
      class ClassChecker():
      def isFirst(self) -> bool:
      return isinstance(self, First)
      @property
      def isSecond(self) -> bool:
      return isinstance(self, Second)
      class A(ClassChecker, First):
      """class A"""
      class B(ClassChecker, Second):
      """class B"""

  • @mikeofs1304
    @mikeofs1304 Год назад +5

    Кстати вы переменную i в классе A как переменную класса объявили, что в общем то считается плохой практикой если вы не пометили ее как прайват, и в целом предполагаете использовать не для внутреней логики в инстансах, а и во внешних каких то операциях. По сути через A.i получается доступ ко всем i экземпляров этого класса , что может повлечь в случае ошибок применения(и использования вашего кода как библиотечного другими разарабами) очень слабопредсказуемое поведение. Тут как раз проперти подходит гораздо лучше, поскольку позволяет контролировать доступ

    • @clauseclause6640
      @clauseclause6640 Год назад +1

      Это не плохая практика, это дефолтное значение, плохая практика это менять A.i

    • @mikeofs1304
      @mikeofs1304 Год назад

      @@clauseclause6640 плохая практика A._i менять

  • @Developer_python_
    @Developer_python_ Год назад +3

    Вітаю, добра тобі! Як ти дивишся на те щоби зняти пару роликів про сесії в Джанго? Типу що з ними робити( корзина, аутентифікація і тд) Дякую

  • @ИгорьВасилевич-э4р
    @ИгорьВасилевич-э4р 11 месяцев назад +1

    Бредозадание , джун какойто придумал

  • @siriusdiamond
    @siriusdiamond Год назад

    +

  • @IliaDementev
    @IliaDementev Год назад

    Гарри Дюбуа это вы?

  • @justman073
    @justman073 Год назад +2

    А я в моменте с AttributeError решил изменить логику переприсваивания атрибута, про @property не знал
    def __setattr__(self, name, value):
    if name == "isSecond" and value == 2:
    raise AttributeError
    super.__setattr__(self, name, value)

    • @SeniorPomidorDeveloper
      @SeniorPomidorDeveloper  Год назад

      Наверное хардкодить имя метода в setattr это не очень гибкое решение. А вцелом да, вполне подходит

    • @igorratnik2357
      @igorratnik2357 Год назад

      Вы после super скобочки забыли))

    • @justman073
      @justman073 Год назад

      @@igorratnik2357 Действительно, как-то прошляпил