Жёсткое код-ревью 😱: джунесса кричала, а я учил Java

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

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

  • @alexandr6055
    @alexandr6055 21 день назад +1

    а мне нравится, подписался! Вывозит конечно харизма парня) Как вариант я бы предложил скидывать сюда гит, где была бы ветка до рефакторинга и вторая после. для наглядности)

    • @java_legko
      @java_legko  21 день назад

      от души! спасибо большое)

  • @Java_Beginner-Dev
    @Java_Beginner-Dev Месяц назад +5

    Посмотрел видео чтобы увидеть как "джунесса кричала, а автор учил Java". Не увидел ни того ни другого)))) Магия кликабельных заголовков в действии)

    • @java_legko
      @java_legko  Месяц назад

      это осталось за кадром 🤣

  • @exxes5187
    @exxes5187 23 дня назад +1

    а с какой ты страны, что у тебя ai работает в идее?

    • @java_legko
      @java_legko  23 дня назад +1

      сам из РБ, но живу в Польше

  • @vitaly8430
    @vitaly8430 Месяц назад +3

    А чего лучше использовать 23? Любая версия может существовать, может чел на проект собесился с Java 18

    • @java_legko
      @java_legko  Месяц назад

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

    • @evgenyshagov7329
      @evgenyshagov7329 Месяц назад +2

      А почему не LTS (17, 21)?

  • @MuradSultanbaev
    @MuradSultanbaev Месяц назад

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

    • @stumppstumpp7189
      @stumppstumpp7189 Месяц назад

      отказаться

    • @java_legko
      @java_legko  Месяц назад

      заходи в телегу t.me/javezka/156
      посмотрим твой пр

    • @stumppstumpp7189
      @stumppstumpp7189 Месяц назад

      @@java_legko не надо 😭😭😭😭😭🥺

  • @EgorFrade
    @EgorFrade Месяц назад

    1) 0:50 вот к java 18 доколупался, а то, что maven используется вместо gradle - это норм? двойные стандарты ?
    2) 3:00 - это уже оверинжиниринг пошёл. Вкупе с захардкоженной строкой и статическим методом мечтать о расширении всей системы - опять двойные стандарты
    3) 7:04 очень странные телодвижения с преобразованием char в string я совсем не понял
    4) 8:12 isWin: назвать тот метод, который по мимо всего прочего печатает отгаданное слово - так себе
    5) 10:34 "тернарный оператор ВСЕГДА упрощает чтение кода" - ну да, ну да
    6) 15:44 тут бы подошел какой-нибудь StringBuilder и не нужно городить тройные кавычки
    7) 22:40 ну да, куда же нам без стримов
    Ну и последнее: У меня есть стойкое подозрение, что на слове с удвоенной буквой (например : tree) будет не правильно отображаться отгаданное слово
    Код ревью - дело не благородное и очень субъективное.

    • @java_legko
      @java_legko  Месяц назад

      привет) спасибо за коммент)
      1) согласен, я бы со старта градл выбирал
      2) отпишу снизу
      3) решил вместо двух одинаковых вызовов сделать одну переменную - в принципе можно и без этого обойтись)
      4) тут действительно код плохо написан - две ответственности сразу
      5) согласен) не всегда
      6) тут не согласен) а для чего тогда эта фича с тремя кавычками тогда нужна еще?)
      7) по мне декларативный стиль стримов выигрывает в поддержке)
      согласен про ревью) поэтому стараюсь всегда попросить 2 разных разработчиков посмотреть мой код, каждый видит со своей перспективы и своего опыта
      в тг канале писал превью к этому видео: Сделал одним дублем, потому что устал от установок джунов, что все должно быть идеально с первого раза
      Часто курсы по проге наполнены или копированием уже готового кода, или же обрезаны/смонтированы так, что места для ошибок не остается. А это приводит к обманчивой мысли о том, что код можно написать идеально сразу. Хотя ошибки допускают все и на них мы учимся лучше, чем на чем-либо еще
      поэтому спасибо за ревью)

    • @torlant3742
      @torlant3742 Месяц назад +4

      Что не так с мавеном, кроме того, что без плагинов приходится быть xml программистом? В больших рабочих проектах разницы в скорости сборки проекта не наблюдаю.

    • @java_legko
      @java_legko  Месяц назад

      @@torlant3742 понимаю, что странно скидывать сравнение производительности градла и мавена с сайта градла :D но все же gradle.org/maven-and-gradle/

    • @EgorFrade
      @EgorFrade Месяц назад

      @@torlant3742 Всё хорошо с мавеном, просто это было сказано в контексте что java старая. Так-то да: в легаси мавен это норм, а вот в новых проектах стараются использовать грэдл, так как он более читабелен, да и синтаксис groovy/kotlin поприятнее чем xml, а вообще это всё на любителя

  • @sorggold1522
    @sorggold1522 Месяц назад +1

    класс, я подписался

  • @siarhei4045
    @siarhei4045 Месяц назад

    А давайте подождем когда код-ревью сделает AI ассистент и какие варианты изменения кода предлагает IDE, на этом слепим контент :D
    С каких пор обращение к функции с запоминанием ее результата и сравнение его в дальнейшем стало проще чем два раза вызывать функцию?

    • @java_legko
      @java_legko  Месяц назад

      хорошая идея для видео, спасибо)
      да, там увлекся 😅 всякие преждевременные оптимизации, jit и так далее не учел

  • @SverrirLavr
    @SverrirLavr Месяц назад +2

    начал бы с тестов
    диз