📢 ССЫЛКА НА TELEGRAM канал: t.me/lightcode_group ⚡ ПОДДЕРЖКА канала на boosty: boosty.to/lightcode

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

Тебе нужно делать цикл видео программирование на мемах !!!)!)!))!))!)🤧

Категорически крутой контент! Все понятно и доступно 🔥

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

Спасибо

Даже я все поняла 😁👍🏻

Крутой продакшн. Респект!

В последнем примере, где вы стали заранее готовить результаты всех сравнений, ваша оптимизация просто опасна. А вдруг, например, свойство user.IsLocked вычисляется относительно долго? До оптимизации оно вычислялось последним, после кучи других проверок, и не каждый раз. Вы "отрефакторили" код так, что теперь все значения для условия вычисляются обязательно и всегда, в том числе и наше потенциально тяжёлое свойство.

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

насколько на технике эпл удобно кодить на Си шарпе?

Последний пример я бы написал немного по другому. Каждый if вынес бы отдельно с прерывание дальнейшего выполнения. Зачем нам дальше выполнять код, если нет оредера или юзера? if (order == null || user == null) return false; И так далее, код будет так же читаем и без вложеностей.

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

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

Спасибо, хоть и являюсь новичком, но понял все, о чем вы говорили в данном видеоролике. Но почему не использовать partial в 3 рефакторинге, можно же создать 2 одинаковых метода с разными аргументами, раз у нас номер карты и email, то можно uint cardNumber сделать и аналогично с e-mail, но уже в string? Я немного слабоват еще в таком

какой симпатяжка🥹🥹🥹🥹😘🥹🥹

Жизненно 1:15

Если интересно, можно почитать книгу, Чистый код Роберта Мартина

Я бы по другому сделал мне не нравится код рефакторинга в начале мы 3 раза извлекаем продукт хотя можно это сделать 1 раз и передавать уже продукт аргументом в методы а не ID продукта. С интерфейсами вообще все просто можно сказать что интерфейсы обеспечивают стандартизацию, позволяя классам имплементировать общие методы и свойства. Это средство унификации, обеспечивая согласованный способ взаимодействия различных классов. Из этого следует вопрос а зачем нам все это надо на что можно просто ответить что это нужно для упрощения добавления новых функциональностей, обеспечивая единообразный способ взаимодействия между классами и улучшая читаемость и понимание кода. То определение что интерфейс служит контрактом бла бла бла мне в корне не нравится…

4й - не надо так плз, имхо более правильный рефакторинг в данном случае это инвертирование условия: т.е. вместо if (a) { if (b) { if (c) ...логика... }}} else { return false; } лучше if (!a) { return false; } if (!b) { return false; } if (!c) { return false; } ...логика... это легче читать чем вложенные в тернарники тернарники и полотно && || условий

автору самому бЫ подучиться перед тем, как учить других. В четвертом примере название метода CanProcessOrder неудачно, поскольку описЫвает невнятное состояние вместо действия. Лучше бЫло бЫ назвать CheckOrderProcessAbility или CheckOrderProcess. Так же в коде есть антипаттерн magic numbers - єто 0 и 10. Их нужно вЫнести в константЫ.

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

Чат подсказал тоже неплохое сокращение кода, хотя хз, норм это или нет ) public class OrderProcessor { public bool CanProcessOrder(Order order, int stockCount, User user, bool isExpressShipping) { return order != null && user != null & order.Status == OrderStatus.Pending && stockCount > 0 && user.IsActive && !user.IsLocked && (!isExpressShipping || stockCount >= 10); } }