КОД КАК У СЕНЬОРА. РЕФАКТОРИНГ

Всем привет! С вами первая серия проекта «Код Раковского». Где я, Александр Раковский, Senior Java разработчик компании ITentika, расскажу о том, что считаю важным и интересным в сфере программирования.

Каких-то жестких правил тут не будет, главное, запомните:
Здесь не любят костыли и велосипеды.
Здесь не терпят код без тестов.
Здесь чтут отцов аджайла.
Здесь суровое экстремальное программирование.

[CoRecYT]

Какой сочный видос. Один из самых годных по рефакторингу, что я видел. Однозначно лайк.

[ulcuber]

Разбивать на методы нужно по уровням абстракции. Я насмотрелся на кучу безсвязных методов. Для рефакторинга бывает нужно свалить всё в один метод, чтобы поделить всё заново. Самое частое для веба в проектах любого размера -- запросы к базе в неожиданных местах. В основном нужно отделять получение данных от рассчётов. Часто логику просто двигают из одного класса в другой, чтобы удовлетворить какое-нибудь требование, вроде "контроллер должен быть коротким", но смысла в этом нет, если у тебя получается Noname класс, делающий то же самое. Если нельзя назвать назначение класса, то скорее всего он был создан не для решения задачи, а для переноса логики, чтобы спрятать.

[ok-tzvw]

Видос - супер! Кратко, понятно и доступно! Можно показать как разрабам, так и менеджменту.

[beispiel]

Очень приятная манера повествования и дикция, уши в восторге

[byjinmo]

Топ, однозначно. И техническая сторона, и философская, и взаимоотношения.

[Nisitka]

Спасибо за видео! Все понятно и доступно

[sliceem]

Блин категорически крутой контент! Как раз в процессе.

[rryrmlp]

Спасибо, теперь появилось хоть и смутное, но понимание того, что же такое рефакторинг

[urbanfreestylerone]

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

[foooxich]

Прекрасное видео, спасибо большое автору

[maximlyakhov]

видео топ, но семантика имени класса ChildrensPrice заставляет задуматься о контексте))

[antonchekulaev]

Братан, хорош, давай, давай, вперёд. Контент в кайф, можно еще? Вообще красавчик! Можно вот этого почаще?

[nomugop]

Сильно брат! Надеюсь однажды достигну твоего уровня код рефакторинга ;), а лучше сразу буду писать как надо.

[bog_d]

Знаю родителей автора видео, классные мужики!

[youtubeyh]

Только за озвучку уже 2 лайка заслуживает!

[TurboKotk]

слова настоящего синьора, мое почтение! единственное есть пару спорных моментов, многие это подметили, а в остальном балдеж)

[swhcpns]

катастрофически одобряю
контент полезный

[spamcolector]

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

Одна проблема если ты пытаешься разобраться что автор пишет ты понимаешь что ускоренное Перемотка здесь вообще ни к месту. Приходится постоянно останавливать видео и разбираться хотя и понимаю зачем было это сделано

[___avka___]

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

[cymzomw]

Не ну это за гранью конечно.
Методы по 3-4 строки - еще ладно, на любителя, может кому-то и правда удобнее иметь 1000 методов вместо 20. Лично меня учили, что метод должен быть глубоким, то есть делать какое-то значимое и понятное изменение, решать какую-то структурную часть задачи. А не просто быть потому что нам код слишком длинным показался.
Но вызов одного и того же метода несколько раз, просто чтобы не создавать переменную - это край уже. А если метод изменится и станет жрать больше ресурсов, тогда что? Будете все его вызовы искать? А когда у вас такие методы вообще везде и имя им легион - так что конкретно одного тормозящего нет, а все коматозят приложение примерно поровну - тогда что? Опять полный рефакторинг?
Нельзя так откровенно забивать на оптимизацию. Преувеличивать её роль конечно тоже не стоит, но тут прямо полное пренебрежение же.
Что вообще значит "только когда она (оптимизация) требуется"? Кому требуется? Типа пока заказчик претензии не выставит, то и оптимизировать не будем что ли?