Программисты-самоучки. Зачем нужен код-ревью 🔍

แชร์
ฝัง
  • เผยแพร่เมื่อ 27 ม.ค. 2025

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

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

    И правда, самый настоящий урок !) Было интересно и познавательно, спасибо за видео

  • @user-vito-kuk
    @user-vito-kuk 4 ปีที่แล้ว +10

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

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

      👍

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

      а что Вы писали для себя тяп-ляп, а что писали не тяп-ляп?) примеры покажете?=)

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

      @@sergekochubey9305 Я бы сказал так. Когда я писал скрипт чтобы получить текстовую версию доклада с youtube, мне было все равно как по итогу будет это все выглядеть. Главное - чтобы работало. В итоге получил python + bash. Просто 2 файлика с полотном кода без классов чисто на функциях и переменных (классы из библиотек были). Но так как это разовое решение сиюминутной хотелки - абсолютно все равно. Выполнил и забыл. Удалил через месяц.

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

    блин, учу руби уже 3 месяца, про многое из видео не знал, спасибо!

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

      Некоторые годами учат и не знают

    • @брониславботексный
      @брониславботексный 4 ปีที่แล้ว

      есть уроки Роман Пушкин бесплатно выложены

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

      @@брониславботексный автор кода на видео смотрел эти уроки, но это всё не важно

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

    Спасибо. Познавательно.

  • @ДимаКозуб-ю3г
    @ДимаКозуб-ю3г 4 ปีที่แล้ว +13

    Я один воспринял это, как полноценный урок по повышению читаемости и рациональности кода?

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

      Надеюсь, что нет :)

  • @брониславботексный
    @брониславботексный 4 ปีที่แล้ว +3

    Круто он все разложил мне зашло

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

    Насчёт пробелов - это косяк саблайма - в нём по умолчанию стоят табы, нужно заменить на пробелы.
    А насчёт скобки ставить обязательно - это кто так сказал и почему? Я вот не ставлю обычно )

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

      скобки улучшают читаемость.

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

      @@crissquest3258 то есть в delete улучшают читаемость, а в puts нет? Просто я не совсем понял разницу. Скобки везде улучшают читаемость, кроме puts? Или в каких ещё функциях улучшают? Какой критерий?

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

      Про настройку sublime: ruby.qkspace.com/ruby-nastroyka-sublime

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

      @@dervish80 Это принятый в сообществе стиль кодирования: github.com/rubocop-hq/ruby-style-guide#method-invocation-parens

  • @soulcode9941
    @soulcode9941 4 ปีที่แล้ว +8

    Че там говорить половину он бы мог поправить просто поставив Рубокопа

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

    9:29 на php тоже обычно так не пишут, скорее всего было бы while(!empty($products)) { ... }

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

    Ну не у всех есть деньги на дорогостоящие курсы.
    Половину можно рубокопом исправить. Если автору его проще так читать, пусть пишет так. Постепенно поймёт что его можно сократить)
    Все равно спасибо за ревью.

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

      Всё так!

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

    хороший урок!

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

    Класс!!! Спасибо большое за разъяснения. То есть, если пройти ваш курс - то статус самоучки снимается?

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

      Serge Kochubey и если даже не наш пройти - тоже. По определению :)

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

      если пройти их курс , станешь беднее на на 80 штук (+- прыгает).

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

      @@valentinavalentina9449 а потом богаче на 50-80 с каждым месяцем

  • @Nikita-tp1up
    @Nikita-tp1up 4 ปีที่แล้ว +2

    Подскажите, как поставить последнюю версию ruby on rails для винды.
    В railsinstaller идут старые сборки.

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

      Лучше в винде ruby on rails не устанавливать, но на свой страх и риск можно поставить rubyinstaller.ru и потом просто поставить gem install rails

    • @Nikita-tp1up
      @Nikita-tp1up 4 ปีที่แล้ว

      @@installero подскажите, я вот решил начать изучение языка.
      Выбирал между java, python, ruby.
      Мне посоветовали ruby. Почитав, понял что без рельс руби учить нет смысла, он не востребован.
      Потратив день, пришел к выводу, что на винде заниматься программированием на ruby on rails нет смысла, из-за несовместимости.
      Вопрос, насколько к правильному выводу я пришёл? Может быть всё таки для обучения с нуля можно учиться программировать в руби на винде?
      Не хочется зря тратить время и пойти по неправильному пути.

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

      @@Nikita-tp1up выбирай java .

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

      @@Nikita-tp1up > Почитав, понял что без рельс руби учить нет смысла, он не востребован
      Ну на рельсах, например, написан GitHub. На руби пишут даже в Сбербанке, к примеру. А так вакансий меньше, чем на других языках, угу. Но когда будете учиться, вам количество вакансий не сильно поможет. Вы java просто можете не доучить до уровня достаточного для выхода на работу.
      Вот видос про это: th-cam.com/video/PcVcZEKfwWE/w-d-xo.html

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

      @@installero, а в чем проблема установить rails на винду? Почему на "свой страх и риск"?

  • @брониславботексный
    @брониславботексный 4 ปีที่แล้ว +3

    Руби это сила

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

    👍

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

    Преподавать вы явно умеете !👏

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

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

    • @ВладимирЧерепович-н9ц
      @ВладимирЧерепович-н9ц 4 ปีที่แล้ว

      На самом деле я просто скопировал из условий задачи, этот массив к себе в программу:-) Как там были кавычки, так и оставил.

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

      есть такая штука - экранирование: зубная\ паста. Все по делу сказано в видео...кароч

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

      %w[зубная\ паста хлеб молоко капуста] - ответ на ваш вопрос, держу в курсе. Вместо того, чтобы самостоятельно нагуглить эту информацию, начинаете топить другого чела не по делу.

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

      Речь была о читаемости кода. Если вы всегда пишете код и не думаете о том, что его будет кто-то читать, то рано или поздно вас вычислят по коммиту)

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

      Это не придирка, я ж говорю, можно оставить и так (9:20)
      Вариант 1: заэкранировать, как тут ребята подсказывают
      Вариант 2: да, переписать, не вижу большой в том беды. Руби -- это не про то, что написал один раз и на века. Руби любят как раз за гибкость, код легко понять и легко поменять. Ещё именно поэтому в руби так любят тесты.

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

    size != 0 || any? Удобней ли ?

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

      А вы на руби пишете?

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

      @@goodprogrammer нет. но с концепцией языка знаком. И если мне не изменяет память , автор языка Мацумото , возносит в самый верх - лёгкость в понимании (чтение) кода, даже если это уменьшает производительность. То есть пусть будет 10 строк кода в замен 5 , если это упростит понимание .
      Человек не знакомый с синтаксическим сахаром руби легко поймёт условие arr.size != 0 и Бог его знает что там под капотом метода "any".
      Ну опять же, мой дилетантский взгляд на язык. Но если уж его создатель сказал - что мой язык для того - чтобы всем было его легко читать и понимать ,и вы выбрали этот язык - то надо придерживаться идей создателя . Не в коем случае не преследую цели навязать или "научить" , но если посмотреть на том же codeware в best practice раздела языка Ruby , то там ВСЕГДА максимально короткий код и почти всегда надо сильно вникать , чтобы понять результат его работы . Но ведь язык то не про это :)

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

      @@crissquest3258 .size != 0 -- понятно любому программисту (который знает перевод слова size), .any? -- понятно любому, кто знает английский язык

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

      @@goodprogrammer хах...ну да, я же именно об этом написал выше О_о
      пс: на курсах вы также аргументируете ?:)

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

      @@crissquest3258 > ну да, я же именно об этом написал выше
      В параллельной ветке я уже высказался про ваши навыки формулирования мыслей письменно (спойлер: не очень).
      На курсах мы "читаем мысли" студентов, чтобы понять, что они хотели сказать/спросить. В комментариях на ютубе мы подобным не занимаемся. Идите на codewars, выберите руби и попробуйте сдать там решение с .size != 0. Посмотрите, насколько ваши представления соответствуют реальности.

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

    А нахер он нужен этот руби?

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

      что бы заработать денег ))))

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

      @@ixtirochiavlodlar Тогда уж лучше JAVA или C sharp

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

      @@TheKirsna нет, не лучше :)

  • @FilmicFil
    @FilmicFil 4 ปีที่แล้ว +6

    Самый бесполезный it канал

    • @user-vito-kuk
      @user-vito-kuk 4 ปีที่แล้ว +11

      Самый бесполезный отзыв на этом канале ...

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

      А какой самый полезный?

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

      таки самый бесприбыльный комментарий. Это не it канал

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

      Ну ну ... не перегибайте :) Ребята используют канал в первую очередь как витрину для своих курсов и соответственно представленный материал направлен на популяризацию оных. Какой то особой цели в развитии самого канала и обучающего контента на канале они явно не преследуют.
      Ну и позволю себе ответить на вопрос : "А какой самый полезный?" . Вы знаете , я был невероятно удивлён обнаружив канал (Тимофей Хирьянов) , для русскоязычного ютуба такой контент - редкость. И если уж человек решил заняться освоением программирования - то ему стоит начать с этих лекций.

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

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