Разбираю и объясняю пример хорошего и плохого кода

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

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

  • @fellainthewagon7166
    @fellainthewagon7166 3 года назад +45

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

  • @fess932
    @fess932 3 года назад +41

    вывод - языки без строгой типизации позволяют ломать себе ногу передавая обьект туда куда надо передавать uint

    • @PostoiParovoz
      @PostoiParovoz 3 года назад

      В таких языках надо делать проверку и типизацию (валидацию) внутри функции.

    • @anotherone3641
      @anotherone3641 3 года назад

      а что скажешь насчет ассемблера?

    • @fess932
      @fess932 3 года назад +1

      @@anotherone3641 ассемблер это лучший способ сломать себе все что возможно))

  • @VladykaVladykov
    @VladykaVladykov 3 года назад +100

    Интересная стратегия: мусор на входе, мусор на выходе. Хмм, надо подумать о внедрении

    • @mamkinproger
      @mamkinproger 3 года назад +12

      Я уже пакеты с мусором поставил как делает автор) А если серьезно - очень полезное видео) Заставляет задуматься и понять, что мой код - го*но...

    • @КириллЧе-я5ы
      @КириллЧе-я5ы 3 года назад +1

      Паттерн стратегия - поменяйте алгоритм в процессе работы на мусорный...

    • @arkadysm
      @arkadysm 3 года назад

      Вообще-то мусор на входе, ошибка на выходе, хотя я тебя понял)

    • @Alexander-lp2qy
      @Alexander-lp2qy 3 года назад

      Мусор ломится в квартиру

  • @ВикторСмольяков-щ4в
    @ВикторСмольяков-щ4в 3 года назад +57

    Никогда не рассматривал такие примеры с факториалом с такой стороны... Вроде пример банальный, а тонкостей оказывается очень много. Спасибо!

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

      Хвостовые рекурсии наши друзья
      А исключения переполнения стека - признаки плохого программиста

  • @norzqq
    @norzqq 3 года назад +147

    Из этого ролика я узнал в каком стиле я проектирую и пишу все свои функции - мусор на входе, мусор на выходе :peepo_cry:

  • @placid3495
    @placid3495 3 года назад +14

    Спасибо за ролик! Подобной информации надо больше, таких разборов не хватает.

  • @blacktrub
    @blacktrub 3 года назад +26

    если достаточно долго писать на javascript то вообще перестаёшь доверять людям и начинаешь обрабатывать всё подряд, будто каждый хочет тебе подсунуть undefined

  • @drapasYT
    @drapasYT 3 года назад +8

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

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

      Никто не пишет рекурсии вызывающие переполнение стека. Это ужасный код.
      Юзаем хвостовые рекурсии вместо этого

  • @mikhailkh8560
    @mikhailkh8560 3 года назад +5

    Обожаю такие выпуски! Всегда есть о чем погуглить и чему поучится после!

  • @arhitutorials
    @arhitutorials 3 года назад +23

    Название видео не соответствует содержанию. Надо было назвать "Страдания JS разработчиков")

  • @АзимАзимов-ч1й
    @АзимАзимов-ч1й 3 года назад +1

    Было очень полезно. Больше таких обучающих роликов. Спасибо🙃

  • @peregrint
    @peregrint 3 года назад +6

    Отличный ролик! Я ещё больше полюбил TypeScript)

  • @aronin4392
    @aronin4392 3 года назад +24

    Блин, я не программист, но автор канала так четко и понятно рассказывает, что даже мне абсолютно все понятно. Респект!

    • @kalobyte
      @kalobyte 3 года назад

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

  • @СтаніславСлободян-ч6у

    Чтобы "докрутить" "мусор на входе - ничего на выходе", осталось добавить:
    if (n > 170) return Infinity;

  • @olezhonnv3215
    @olezhonnv3215 3 года назад +3

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

  • @dasauser
    @dasauser 3 года назад +16

    Жаль, что в жс не придумали тайп хинтинг.
    Хорошо, что для жс придумали тайпскрипт.

  • @lexxkrt
    @lexxkrt 3 года назад +14

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

    • @yaroslav8609
      @yaroslav8609 3 года назад

      TS в помощь

    • @qwertymangames1800
      @qwertymangames1800 4 месяца назад +1

      @@yaroslav8609 ts всё равно транслирует код в js и выполняться код будет в js.
      Максимум чем ts может помочь - это дать подсказки до запуска кода.

  • @xdef42
    @xdef42 3 года назад +29

    Предлагаю сделать видео, где Соер просто 15 минут будет быстро говорить умные слова, за Макконнела, белые/черные списки и стратегии "мусор на входе - мусор на выходе" и т.д. больше спасибо

  • @enkryp
    @enkryp 3 года назад +4

    Отличный пример того, что !(n > 0) и (n

  • @drVatman
    @drVatman 3 года назад +9

    Мне кажется, тут уже недостаток джаваскрипта. Слева более понятная с точки зрения алгоритма функция (без специфики языка), но её поведение получается более сложным из-за отсутствия строгих типов. Наверное, хорошим тут был бы третий вариант, с проверкой типа, если мы берем общий случай, а не конкретно джаваскрипт.

  • @aMR5AvDeG
    @aMR5AvDeG 3 года назад +6

    "мусор на входе - мусор на выходе"
    мое кредо

  • @richarddaniel7088
    @richarddaniel7088 3 года назад +2

    Сначала был скептически натроен, но сейчас ставлю лайкос и подписка. Не знаю на сколько ты профи по факту, но темы очень правильные.

  • @kai3341
    @kai3341 3 года назад

    Касательно ключевой идеи "не усложнять код снаружи" -- жЫрный лайк! Спасибо!
    Касательно проверки -- упущен тот момент, что в коде без проверки условия о том, что с отрицательными числами, оказывается, не работаем, будет произведено несколько тысяч рекурсивных вызовов данной функции. Что вовсе не небольшой оверхед
    Дальше оффтоп и лирика. Один из вариантов решения -- разбить функцию на 2. Первая -- фронт-энд -- принимает данные и выполняет проверки, в случае успеха возвращает результат рекурсивной функции, а сама рекурсивная функция максимально простая. Таким образом мы получим максимальную производительность при минимальном оверхеде
    Не была рассмотрена хвостовая рекурсия. Ввиду отсутствия в JS оптимизации хвостовой рекурсии, использование хвостовой рекурсии в JS не делает код хорошим или плохим -- оно погоды не делает. Однако алгоритм может быть реализован на языке, поддерживающим оптимизацию хвостовой рекурсии -- и там этот момент будет очень принципиальным
    Читаю комментарии: строгая типизация решает одну проблему, порождая другую. Оказывается, в JS помимо Number есть ещё и BigInt -- в случае строгой типизации ваш код просто перестанет работать с этим типом. В других языках числовых типов бывает ещё больше. На примере python это Decimal, numpy.int* -- и это вершина айсберга. Если заниматься мacтypбaциeй с типами -- куча клиентского кода перестанет работать без объективных на то причин. Д -- достижение.

  • @dmitrygurinovich58
    @dmitrygurinovich58 3 года назад +2

    Насколько я понял основная идея этого видео в том, что поведение функции должно быть детерминированным. И я с этим полностью согласен.
    Но примеры на JS лично для меня только добавляют путаницы.
    Я, признаюсь, плохо знаю JS. Видимо поэтому у меня осталось только больше вопросов, чем ясности:
    - я правильно понимаю, что в правой функции выбрасываются одинаковые ексепшены для разных инпутов?
    - Если ексепшены разные - то это мало отличается от левой функции, так как надо их все обработать.
    - Если они все одинаковые и сообщают о переполнении буфера - то, вроде бы лучше, но не совсем понятно, как это поможет вызывающему коду понять, что он передал неправильный аргумент?
    - И вообще, если, я увижу, что при передаче неверного аргумента в функцию она выбросила ексепшен, который говорит о том, что она пыталась выполнить свою логику с этим заведомо неверным типом аргумента, мне будет не по себе. Просто представьте, что функция не считает факториал, а обновляет состояние в базе данных, например.

  • @grafgeleon2420
    @grafgeleon2420 2 года назад

    Как только услышал пример с рекурсией, рука сама потянулась выключать видосик. Но я решил всё таки посмотреть, включил заново и я не пожалел. Пример реально хороший, объяснения хорошие, того говна что я ожидал услышать НЕ_БЫЛО. Я доволен.

  • @PostoiParovoz
    @PostoiParovoz 3 года назад +2

    Оба примера - плохой код.
    Должна быть валидация входящих данный и должно выбрасывать ошибку, либо в крайнем случае 1 (в зависимости от задачи).
    Также должно быть преобразование типов - это же JS *!!!
    Левый код более правильный. Валидация слабая, но она есть! Надо просто сделать это лучше.
    Потому что потом , когда во время разработки будет пойман результат Exception, хотя по факту будет ожидаться выполнение, а не переполнение, потому что якобы должно поступать небольшое число на вход - вот тогда разработчик "хорошего кода" будет рвать последние волосы на мягкой точке, ища что же не так.
    Так для примера на 8:00 идёт рассуждение "либо результат, либо Exception". А на каком результате он будет наступать? На 20000, на 10486? А если комп и движок JS из будущего и потянет и даже 100 000 на расчёт?
    * данная функция из-за особенностей JS автоматически будет приводить всё в числа. Но "0" ноль, она не переварит! Что выдаст factorial("0")?
    А если математическая функция использует сложение?

    • @S0ERDEVS
      @S0ERDEVS  3 года назад

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

    • @PostoiParovoz
      @PostoiParovoz 3 года назад +1

      @@S0ERDEVS ага, а потом разгребать багрепорты от пользователей. Вот так джуны почитают вас, ещё и формы проверять не будут, потому что наивный подход на практике.
      У вас вероятно свой опыт умения проектирования, а на практике - чем больше вариантов учтёшь, чем больше проверок и тестов предусмотришь, тем проще будет поддерживать. Не придётся искать ошибки. И хорошо, если эта функция не жизненноважные. А то может со счёта уйти пару миллиончиков долларов и будешь всю жизнь отрабатывать.

    • @S0ERDEVS
      @S0ERDEVS  3 года назад

      @PostoiParovoz причем тут мой опыт? Открой любую публичную популярную библиотеку и посмотри как они написаны. Было бы желание, а информации на счет проектирования более чем достаточно. Если в коде куча проверок и явных приведений - это первый признак, что никто его не проектировал, а код написан по интуиции.

  • @Игорь-ч6ф3и
    @Игорь-ч6ф3и 3 года назад +2

    Если идти по пути проверок, я б эту функцию завернул в ещё одну функцию, где бы и разместил проверки. Чтоб ещё и рекурсивно их не повторять.

  • @ИльяЛюбашов
    @ИльяЛюбашов 3 года назад +1

    Интересное видео! Спасибо!
    А как насчет того, чтобы правую часть в try..catch засунуть, и в случае исключения возвращать NaN

  • @wylysypydystyshky
    @wylysypydystyshky 3 года назад

    Всё понятно, спасибо. ECMAScript - сплошная головная боль. Когда его начал изучать и практиковаться, тогда полдня потратил, чтобы найти ошибку при неявном преобразовании типов - проверял переменную (взятую с тега input ) на равенство нулю, а в самой переменной то-ли пустая строка строка была, то ли "0"...
    С тех неявное преобразование не люблю. Ученики с трудом въезжают в эту тему. Наоборот, пусть лучше целое 2 и вещественное 2.0 не будут равняться друг другу. Пусть лучше 2 и 2.0 вообще не складываются.
    Беру на вооружение термины "Белый список" и "Чёрный список" в операторе ветвления. Буду ученикам также объяснять составные логические выражения.

  • @MrAnetto
    @MrAnetto 2 года назад +1

    Классическое GIGO - garbage in, garbage out. Наглядно

  • @SimpleException
    @SimpleException 3 года назад +3

    Автор, спасибо. Действительно очень интересный пример.
    Но после просмотра меня не покидает чувство, что оба примера плохие. Просто один (правый) получилось вписать в одну из стратегий программирования, а второй пример - нет.
    Ни один из примеров не выполняет свою функцию с точки зрения математики. Факториа́л - функция, определённая на множестве неотрицательных целых чисел. Ключевой, на мой взгляд, момент - "целые неотрицательные числа". Т.е. пример слева "пытался" выполнить эти условия, но выполнил только половину, а пример справа даже не пытался. Возможно, что это и хорошо, по крайней мере именно эту мысль, как я понял, Вы и пытались донести, но тут скорее применима Теорема Эскобара.

  • @Algok17
    @Algok17 3 года назад +8

    А что на счёт языков со статической типизацией?

    • @dasauser
      @dasauser 3 года назад +5

      А что насчёт них?
      Они молодцы.
      Они делают то, что соер предлагает прописывать руками, а именно ограничивать функцию от данных, с которыми она не работает, добавляя полотна проверок.
      Так что с ними все ок.
      Т.е. если параметр функции был условно integer n, то при мусоре на входе было бы исключение.
      И все норм. 3 состояния результата работы функции: число, бесконечность (это вроде как тоже число в жсе, я хз), и экзепшен.

    • @ForwardMyMonkey
      @ForwardMyMonkey 3 года назад

      @@dasauser uint

    • @Algok17
      @Algok17 3 года назад

      @@dasauser а что на счёт проверки x

    • @0imax
      @0imax 3 года назад +2

      @@Algok17 unsigned int - и число всегда будет неотрицательным.

    • @Algok17
      @Algok17 3 года назад

      @@0imax ну это для данного примера подходит, а что если в агрументы передаётся класс, части которого могут быть некорректными и из-за этого не отработает дальнейшая логика?

  • @ArtyomShakurov
    @ArtyomShakurov 3 года назад

    Не успел поставить лайк после завершения, но вернулся и поставил! Спасибо за разбор!

  • @НиколайПрибитков
    @НиколайПрибитков 3 года назад

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

  • @АндрійБілецький-о2ж

    Отличное видео, побольше бы таких.

  • @BigCrus
    @BigCrus 3 года назад

    а есть книга, в которой поподробней про все эти принципы (типа мусор на входе - мусор на выходе) можно почитать? это у Макконела написано?

    • @S0ERDEVS
      @S0ERDEVS  3 года назад

      Да, про мусор и не только писал Макконнел, у него же про защитное программирование, про контракты лучше всего у Мейера написано, так же можно кое что у Лармана прочитать по безопасному программированию.

    • @BigCrus
      @BigCrus 3 года назад

      @@S0ERDEVS понял-принял, спасибо. был бы очень признателен если подскажешь название книг

  • @ЗахарИгнатьев-е3э
    @ЗахарИгнатьев-е3э 3 года назад

    Затея интересная, яп не известный.
    Это же все самое простое и примитивное, что дается на курсе по алгоритмам.
    И из квадратичного делается линейный алгоритм исполнения.
    Но это же правда просто, неделя - две над одним псевдокодом и решено...
    Запомнил...
    А вот решение задачи с 4 вложенными циклами и создание линейного алгоритма, это на Яндекс стажировке выдали, второй из 6 задачей, вот тут только мастер поможет)
    Спасибо.

  • @mormorrowind403
    @mormorrowind403 3 года назад +1

    Спасибо, за разбор. Очень познавательно

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

    Здравствуйте! Меня волнует один вопрос: вы говорите, что, обработав одну ошибку в правой функции(пример с факториалом), мы обработаем одновременно и остальные. А как же правило, что ошибки мы обрабатываем каждую отдельно? Try{} catch ( SomeError1 e){} catch (SomeError2 e) ….

  • @antonpankrat4594
    @antonpankrat4594 3 года назад +6

    может так еще лучше
    function factorial(n)
    {
    return fact(n);
    }
    function fact(n)
    {
    if (n === 0) return 1;
    return n * fact(n-1));
    }
    таким образом мы уберем из рекурсии проверку (многократную) хоть белого хоть черного списка

    • @enkryp
      @enkryp 3 года назад

      Такой подход требует, чтобы во время рекурсивных вызовов входные данные оставались корректными, гарантии чего наш код не даёт. Например, при проверке "входное значение должно быть числом большим или равным единице" указанный код крашнется при n=1.2

    • @eugeneponomarov7429
      @eugeneponomarov7429 3 года назад

      Да, но у нас вместо одной функции теперь две, что не очень красиво

    • @antonpankrat4594
      @antonpankrat4594 3 года назад

      @@enkryp

    • @fess724
      @fess724 3 года назад

      Чтобы не создавать 2 глобальные функции лучше поступить так:
      function factorial(n) {
      if (!Number.isInteger(n)) {
      throw new TypeError('Variable n must be and integer type.');
      }
      if (n < 0) {
      throw new RangeError('Variable n must be positive integer.');
      }
      return (subf = (i) => {
      if (i === 0) return 1;
      return i * subf(i - 1);
      })(n);
      }

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

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

  • @КириллЧе-я5ы
    @КириллЧе-я5ы 3 года назад +1

    А если не использовать рекурсию в данном примере, а применить цикл, как быстро произойдёт переполнение?..

  • @keksinjo
    @keksinjo 3 года назад

    чтобы избежать переполнения стека:
    function sum(n, s = 0) {
    return (n) ? () => sum(n - 1, s + n) : s
    }
    function trampoline(fn) {
    return function (...args) {
    let res = fn.apply(null, args)
    while (typeof res === 'function') {
    res = res()
    }
    return res
    }
    }
    const num = 1000000
    const trampSum = trampoline(sum)
    const res = trampSum(num)
    console.log(res)

  • @AlexM-gn7bp
    @AlexM-gn7bp 3 года назад

    Отличное видео. Побольше бы именно такого контента

  • @niceandquickly
    @niceandquickly 3 года назад

    Почему не рассмотрено исправление левого кода с учетом белого списка и выброса исключения? Или это тоже плохой вариант, почему?

  • @ФагимСадыков-ъ6с
    @ФагимСадыков-ъ6с 2 года назад

    На любом вменяемом языке тип int убрал бы возможность передать передать 1.2, {}, [] остался бы только вариант < 0, uint снимает и её. Поэтому если нужен JS хотя бы используйте TS

    • @ФагимСадыков-ъ6с
      @ФагимСадыков-ъ6с 2 года назад

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

  • @nurk9928
    @nurk9928 3 года назад

    Тут ещё нужно смотреть на ЯП.
    В golang, на котором в основном я пишу код, левый вариант лучше.
    При < 0 возвращается ошибка, которую необходимо обработать. При невалидных входных данных (< 0) функция уйдёт в бесконечную рекурсию.

    • @nurk9928
      @nurk9928 3 года назад +1

      Я снова не досмотрел видео и написал комментарий.
      Прошу прощения.

  • @savvakirichenko8419
    @savvakirichenko8419 3 года назад +3

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

    • @ANTGPRO
      @ANTGPRO 3 года назад +1

      для начала добавить проверку типа аргумента

  • @stanislavsh6582
    @stanislavsh6582 3 года назад +1

    На самом деле все довольно сложно.
    Если более комплексный кейс рассмотреть, допустим с парсингом Http - все будет не так уж и очевидно. А люди из команды еще тебе и скажут, что им лень обрабатывать исключения и потому твой код не должен их бросать. Ну и в итоге - все придет к тому, что у тебя все публичные функции будут выглядеть как-то так: IResult, а в контексте веба - всегда 200 и пусть тот кто получает результат - смотрит, все ли нормально.

    • @liravesnovaya242
      @liravesnovaya242 3 года назад

      По поводу лени обработки: пускай тогда просто делают пустой catch раз уж так лень. По мне, подход: "появилась проблема, выдай исключение и умри" в разы лучше мусора на выходе, потому что мусор может потом уйти очень далеко от источника, и источник уже можно и не найти.

    • @stanislavsh6582
      @stanislavsh6582 3 года назад +1

      ​@@liravesnovaya242 Да я сам так же считаю(относительно принципа fail-fast), но что поделать, я же не один в команде. Потому и сказал что в конечном итоге все сведется к IResult. Как по мне - тупое решение, потому что не дает до конца понять - что не так, но что поделать.

  • @mykhailostepanishchev6472
    @mykhailostepanishchev6472 3 года назад

    очень интересно,спасибо !

  • @spappinventor2850
    @spappinventor2850 3 года назад

    А почему в правой функции не перехватить Exception и вернуть Nan ? И вообще если надо вернуть хорошо или плохо, то надо функции немногие варианты плохо и превратить в одно возвращаемое плохо!

  • @almasabdrazak5089
    @almasabdrazak5089 3 года назад

    чем мне джава нравится
    int factorial(int n) throws NegativeValueException,IntegerOverFlowException
    не валидные типы отсечет компилятор , также функция показывает какую ошибку можно ожидать
    в golang принято возвращать значение и error
    func factorin(number int) (int,e)
    в случае ошибки e будет != null , но тут неявно в чем причина ошибки и мне как пользователю метода не понятно что делать с цепочкой вызова

  • @azamatk4302
    @azamatk4302 3 года назад +10

    TypeScript поможет избежать мусора на входе. Типизация + аннотация параметра функции.

    • @robertdowneyjr8494
      @robertdowneyjr8494 3 года назад +5

      Не поможет, если сам разработчик дундук и поставил условный any как тип.
      Тут надо понимать что и зачем, язык лишь даёт (или не даёт) инструменты для этого.

    • @azamatk4302
      @azamatk4302 3 года назад +1

      @@robertdowneyjr8494 В таком случае поможет проверка в аннотации. Чтоб обойти, дундуку придется лезь в аннотацию и там ломать проверки.

    • @robertdowneyjr8494
      @robertdowneyjr8494 3 года назад +2

      @@azamatk4302 Вы не верите в могущество идиотизма? :)

    • @vetalmontolio8075
      @vetalmontolio8075 3 года назад +1

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

    • @KopoLPedov
      @KopoLPedov 3 года назад +2

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

  • @СергейНовожилов-я6я

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

    • @enkryp
      @enkryp 3 года назад +1

      Посмотрите в сторону архитектуры Flux и хранилищ типа NgXs/Redux

    • @redeyes256
      @redeyes256 3 года назад

      Если некоторые данные используются разными компонентыми, то их можно хранить в виде стейта в общем компоненте прородителе, это если без всяких state management. По хорошему конечно же завести глобальное хранилище состояния приложения, например вышеупомянутый редакс, и из него получать конкретные курски стейта в разных компонентах.
      "например чекбокс на одной странице должен менять состояние элементов на другой". Это штуку тоже можно редаксом решить, просто изменения состояния чекбокса должно запускать определенный actionCreator, который будет менять глобальное состояние, от которого и будет зависеть тот самый компонент, который вы хотите менять чек боксом, и не важно в каком месте, потому что как только тот кусок состояния в редаксе, от которого он зависит, изменится, он автоматически перерисуется. Так же черед редаксовские actionCreator's можно отправлять ассинхронные запросы на сервер, если подключить redux-thunk, что тоже добавляет удобства. Чтобы понять что к чему в редаксе потребуется определенное время и пытливость ума, но если есть время и желание, то это решение должно вам помочь.

  • @protiv_bio
    @protiv_bio 3 года назад

    Хм, здесь что-то очень специфичное относительно языка, что требует объяснения. Я не понял, как проверка на отрицательное число в итоге приводит к разным результатам в еще двух случаях?

  • @VorobyevAlexander
    @VorobyevAlexander 2 года назад

    Мне кажется, если уж дорабатывать левый код, то правильнее было бы бросить исключение в случае не валидных входных данных. это так же позволит разделять исключения "общие" и относящиеся к бизнес логике задачи.

  • @kalobyte
    @kalobyte 3 года назад +4

    еще раз убедился, что яваскрипт гавно
    как можно было додуматься до нан в качестве результата некорректной операции?
    а если функция и не должна выдавать числа, а строки там или массив или даже объект?

  • @Shtirlic_Isaev
    @Shtirlic_Isaev 3 года назад

    Исчерпывающе! Но мне вариант с белым списком кажется предпочтительнее,так как операция факториал определена на неотрецательных целых числах.

  • @bunnyboss75
    @bunnyboss75 3 года назад +6

    Что это за 25 кадр? Я уже чувствую желания писать код разными кисточками на графическом планшете)

  • @takiekakmi7532
    @takiekakmi7532 3 года назад +2

    Хм, хоть я и питонист, тем не менее - интересная концепция, особенно при переполнении стека (хотя это будет сделать проблематично - скорее комп положишь...) Ну и отсюда понимание, что питонячий обработчик исключений - просто топчик

    • @viktor_borodin
      @viktor_borodin 3 года назад +1

      Не знаю, на си и c++ - изи, причём если не сталкивался с таким, то можно по первому разу сильно удивиться, когда делаешь какой-то итеративный расчёт.

  • @Адриан-у4т
    @Адриан-у4т 3 года назад +3

    Лайк за Макконнелла!

  • @solitarysubscriber6575
    @solitarysubscriber6575 3 года назад +2

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

    • @S0ERDEVS
      @S0ERDEVS  3 года назад

      На чем основывается утверждение? Переполнение стека в JS в данном случае произойдет для тебя так же моментально как и выход. Страх ради страха?

    • @solitarysubscriber6575
      @solitarysubscriber6575 3 года назад +3

      @@S0ERDEVS А на медленном компьютере? А если это происходит тысячу раз подряд? Тоже будет "так же моментально"?
      Что это вообще за больная логика: "компьютер быстрый, давайте делать лишние (!) действия, не писать же нам более аккуратный код в конце концов"?
      Поверить не могу, что приходится это объяснять серьёзному специалисту. Но, как я уже сказал, это объясняет, почему веб-страницы долго грузятся и медленно работают, а браузеру потреблять 2гб ОЗУ для отображения картинок и текста (!!) считается нормой. Печально, очень печально.

    • @S0ERDEVS
      @S0ERDEVS  3 года назад +3

      @Solitary Subscriber
      Поверить не могу, что для тебя нормальная ситуация - тысяча исключений подряд, в моей практике вероятность исключительной ситуации в 0.01% - это уже очень много, но даже в таком случае, если посчитать вероятность того что такие события произойдут 1000 раз подряд, то окажется, что это практически нереально (очень уж маленькая вероятность такой ситуации).
      Т.е. у тебя получается, что ради того чтобы сэкономить процессорное время 1 раз на миллиард вызовов (на самом деле даже больше, но считать точно не хочу), ты в каждом из этих вызовов готов делать доп проверку (которая не бесплатно выполняется).
      Вот поэтому и получаются медленные приложения, что пихаете в каждую функцию проверки, которых можно избежать, если перестать руководствоваться интуицией, а подучить тервер и посчитать реально насколько вероятно наступление события, ради которого вы вносите неопределенность и сложность в программу, убивая производительность в нормальном потоке выполнения.
      Еще раз подумай - речь идет не о нормальном потоке исполнения, а о исключительной ситуации, т.е. в нормальном потоке вообще не будут использоваться доп. ресурсы. В твоем же случае ты хочешь экономить ради редкого исключения, повышая нагрузку на нормальный поток.

    • @niklkelbon3662
      @niklkelbon3662 3 года назад

      @@S0ERDEVS просто язык гавно вот и всё

  • @Uni-Coder
    @Uni-Coder 3 года назад

    Услышав про мусор, ожидал услышать что-нибудь про контракты.
    Хотя не знаю, есть ли какое-то подобие контрактов в JS.

  • @vetalmontolio8075
    @vetalmontolio8075 3 года назад +1

    Шикарный разбор!

  • @alexarakcheyev9189
    @alexarakcheyev9189 3 года назад

    Интересно конечно, но по-моему этот конкретный пример чито js заморочка. Например, я начинал с Паскаля, как учебного языка и потом сразу пошли C и С++. Там такое даже принципиально не возможно. Если тип возврата специфицирован, то и будешь сидеть и думать, что же вернуть при ошибке или вообще результат возварщать через указатель или ссылку в аргументах функции. А в return возвращать только код результата или ошибки.

  • @Maiq-The_Liar
    @Maiq-The_Liar 3 года назад

    14:09 а в случае с NaN нам не придется добавлять аналогичные проверки на isNaN(i)?

    • @S0ERDEVS
      @S0ERDEVS  3 года назад

      Здесь имелся в виду популярный среди JS-ов подход, когда выход из функции делается через пустой return и по факту эта функция возвращает не осмысленное значение, а undefined. Т.е. не конкретно выбор между NaN и undefined, а в любом непонятном случае. В результате в программе постоянно возникают ситуации, когда неопределенные переменные просачиваются в функции и появляется привычка проверять на undefined на всякий случай.

    • @Maiq-The_Liar
      @Maiq-The_Liar 3 года назад

      @@S0ERDEVS да, я это понял. Просто пытаюсь уловить момент, чем проверка на undefined хуже проверки на любое другое возвращаемое "ничего" (NaN, null)?

    • @S0ERDEVS
      @S0ERDEVS  3 года назад +3

      @@Maiq-The_Liar Тем что если ты ожидаешь число, то NaN - это число (по типу), и тебе не надо делать проверки на NaN чтобы оперировать с ним как с числом, т.е. например, (undefied).toFixed() - выдаст ошибку, NaN.toFixed() - нет.
      Про null, там я сказал null-object, т.е. объект-пустышка.

    • @Maiq-The_Liar
      @Maiq-The_Liar 3 года назад +1

      @@S0ERDEVS уловил, благодарю.

    • @TwilightSun32
      @TwilightSun32 3 года назад

      @@S0ERDEVS по факту получается NaN это null-object для числа. Вроде логично, но как-то раньше не приходило в голову.

  • @ДмитрийРоманенков
    @ДмитрийРоманенков 3 года назад

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

    • @TwilightSun32
      @TwilightSun32 3 года назад +1

      он очень быстро кончается в жабаскрипте. можно в браузере проверить, падает почти мгновенно. там просто ограничение на глубину вызова стоит, оно не ждёт фактического окончания памяти

    • @enkryp
      @enkryp 3 года назад +1

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

    • @ДмитрийРоманенков
      @ДмитрийРоманенков 3 года назад +1

      @@enkryp Ну да, в этом смысле понятно, что правый вариант лучше. Просто они оба плохие)

  • @ХаимМохави
    @ХаимМохави 3 года назад +1

    А ещё будет?

  • @TheKwadriga
    @TheKwadriga 3 года назад +1

    Мне кажется, что вторая функция тоже плохая. Да, работать она будет предсказуемо, но каждый раз, когда на вход попадает что-то кроме целочисленного значения >= 0 ждать, когда переполнится стек - это так себе идея. С точки зрения производительности это очень плохо. И я не уверен, что она будет работать одинаково на разных ОС и разных архитектурах железа (я, как программист, вообще не должен об этом думать - лучше просто написать функцию так, чтобы: 1) никакого переполнения стека не происходило вообще никогда, 2) было не важно, на какой платформе это будет запускаться. Чтобы этого добиться, пред условием выхода из рекурсии (n === 0), нужно добавить проверку корректности входящих значений. Не уверен, что это корректный код на js, но будем считать, что это некий псевдокод:
    if (typeof n !== integer || n < 0) {
    throw new Exception('Invalid parameter');
    }
    if (n

    • @S0ERDEVS
      @S0ERDEVS  3 года назад +2

      Стек на js это условности. Не надо путать со стеком на си. Переполнение стека в js безопасно в силу гарантий самого языка.
      Что касается производительности то преждевременная оптимизация - зло.
      В данном случае переполнение стека очень недорогое поведение.
      И не надо забывать что это исключительная ситуация, если в программе высокие шансы получить вместо данных мусор, то надо разбираться с дизайном системы в целом.

    • @TheKwadriga
      @TheKwadriga 3 года назад +1

      @@S0ERDEVS Ага, т. е., в данном случае можно считать, что сам язык делает эту проверку входящих значений. Ну да, тогда никакого смысла нет проверять самостоятельно и вторая функция вполне удовлетворительна.

    • @ANTGPRO
      @ANTGPRO 3 года назад +1

      @@S0ERDEVS Причем здесь JS к проверке стэка вызовов? не позорься))

  • @avecoder
    @avecoder 3 года назад +2

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

  • @EdwVee
    @EdwVee 3 года назад

    На первой минуте какой-то чистой джаваскриптовый пример. Если б были типы, то n выбрать как uint и всегда все будет выполняться идеально без ошибок.

  • @МаксимХвостов-м1й
    @МаксимХвостов-м1й 3 года назад

    Еще один интересный метод реализации - через построение бесконечного списка - обращение к нему производится только по индексу поэтому получается "мусор на входе - исключение на выходе". Пример на Haskell:
    factorial = scanl (*) 1 [1..]
    :t factorial -- Тип списка: factorial :: [Integer]
    factorial !! (-1) -- Обращение к списку: *** Exception: Prelude.!!: negative index
    factorial !! 3 -- 6
    factorial !! 300 -- Большое число не буду сюда копировать.
    factorial !! 30000 -- Очень большое число.

  • @111KriegeR111
    @111KriegeR111 3 года назад +4

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

  • @klev1983
    @klev1983 3 года назад +1

    В плане кода да. В плане юзабилити мусор на входе на выходе купи пожалуйста эти наушники. Найдется все, даже то, что ты не искал.

  • @surenkirakosyan5244
    @surenkirakosyan5244 3 года назад

    А почему просто не выбрасить исключения вместо оператора return? Очевидно же, что программист хотел избежать ненужных операций, которые приводили бы к ошибке "Maximum call stack size exceeded", но выбрал неудобный путь.

  • @trell_7842
    @trell_7842 3 года назад +5

    я 1 кто пытался поймать кадр на 07:12 и очень сильно разочаровался ???((((

  • @ДобрыйЧеловек-я6о
    @ДобрыйЧеловек-я6о 3 года назад +9

    Код хороший. Код плохой. Код злой.

  • @АлександрКостюченко-у4х

    спасибо за контент

  • @ac130kz
    @ac130kz 3 года назад

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

  • @bohdanyatsyshyn1662
    @bohdanyatsyshyn1662 3 года назад +4

    С видео становится понятно что не стоит писать на js :/

  • @nekosora6036
    @nekosora6036 3 года назад +1

    Есть хороший подход, см. статью на Хабре "Парсите, а не валидируйте". Не знаю, насколько это применимо к JS, но сам по себе подход хороший

    • @ANTGPRO
      @ANTGPRO 3 года назад +2

      в JS придется и парсить и валидировать, автор видео совсем забыл базовые вещи для школьников

    • @nekosora6036
      @nekosora6036 3 года назад

      @@ANTGPRO какой автор?

    • @woodstickbong
      @woodstickbong 3 года назад +1

      @@nekosora6036 автор видео же

  • @КириллЧе-я5ы
    @КириллЧе-я5ы 3 года назад +1

    Этакий контракт, за мусор на входе сервер не отвечает, отвечает клиент...

  • @kirillriman3611
    @kirillriman3611 3 года назад

    Мудро и познавательно

  • @sovrinfo
    @sovrinfo 3 года назад

    Спасибо за ролик!

  • @агатакристи-г3ы
    @агатакристи-г3ы 3 года назад +3

    Жень, ударение такое "Сти́вен Макко́ннелл "

  • @andreyvolkov3117
    @andreyvolkov3117 3 года назад

    Мне кажется или картинка стала лучше?

  • @vlatterran
    @vlatterran 3 года назад +16

    Из этого всего мы делаем вывод - js плох, ибо у него есть ничего, и другое ничего

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

      NaN есть во многих языках. Это результат вычисления к примеру 0/0
      Это число, но не число при этом.
      Inf/-Inf тоже в любом языке есть, это число в типах с плавающей точкой.
      undefined фишка js. По сути аналог null, но не null так как у null есть своё отдельное значение в js.
      А исключение Exeption тоже есть во многих языках, к примеру Java, C#
      Откуда взялся NaN? Сначала функция вернула просто return; -> undefined. Потом undefined попытались математически умножить с числом и получили число "нечисло", то есть NaN.

  • @user-dz3bi7qt6i
    @user-dz3bi7qt6i 3 года назад

    Отличное видео!

  • @ianpaddington3187
    @ianpaddington3187 3 года назад

    а че бы не проверить тип параметра? явное лучше, чем неявное. в правой функции отсутствие побочных эффектов достигается за счёт того, что нет условия ‘n < 1’. всем программистам в команде нужно знать и держать в голове, что доп проверка приводит к сайд-эффектам.

  • @AlexRuban...
    @AlexRuban... 3 года назад

    в java действительно нужно === поставить?)))

  • @ВіталійКостак-ю9в
    @ВіталійКостак-ю9в 3 года назад

    Круто... Подписался

  • @romanradchenko3569
    @romanradchenko3569 3 года назад

    Стратегия 2 интересна, но также не до конца понятно какое должно быть поведение в месте где вызывается функция, тогда в таком месте должен быть try/catch.

    • @ANTGPRO
      @ANTGPRO 3 года назад +1

      Конечно. Обе функции бездарные, с точки зрения JS.
      Но этот приколист еще убрал проверку на отрицательный аргумент в надежде на exception.
      Трэш.

  • @0imax
    @0imax 3 года назад

    В любой непонятной ситуации кидайте exception :)

  • @ovanse
    @ovanse 3 года назад +4

    Джун: ничерта не понятно, но очень интересно 😂

    • @covid-2284
      @covid-2284 3 года назад +4

      Не думаю что я - джун. Я никто вообще... Но что, зачем, как и почему - мне понятно.
      Думаю тут в принципе ничего такого нет. Ну разве что гуманитарий запутается...

    • @computercomputer3293
      @computercomputer3293 3 года назад

      @@covid-2284 гуманитарий. Не запутался :)

  • @NarkotikAsd
    @NarkotikAsd 3 года назад

    Спасибо!"!!!!!!!!!!!!!!

  • @anotherone3641
    @anotherone3641 3 года назад

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

  • @GenaTolstij
    @GenaTolstij 3 года назад

    О. Хорошо однако, элементарнейшие вещи но хорошо

  • @bpospanov
    @bpospanov 3 года назад

    отличное видео

  • @vladimirv.443
    @vladimirv.443 3 года назад

    Очень полезно, но очень растянуто, суть ясна уже в первые минуты

  • @johnstrayk5208
    @johnstrayk5208 3 года назад +1

    Спасибо, расширили сознание.

  • @woodstickbong
    @woodstickbong 3 года назад

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