Рефакторинг сеттеров и интерфейсов

Открываем новую рубрику практического рефакторинга. Сегодня у нас разбор кода компонента OAuth 2.0 Server. Сделаем форк репозитория и настроим локальный запуск тестов перед отправкой Pull Request.

Займёмся упрощением кода сервера. Найдём и устраним избыточность интерфейсов. Удалим неиспользуемый код. Увидим неудобство наличия сеттеров в сервисе и рассмотрим способы избавления от них. Избавимся от зависимости от конкретного класса путём внедрения интерфейса. И сделаем интерфейсы компактными.

  • 00:00:35 - Примут или нет Pull Request
  • 00:02:32 - Делаем Fork и клонируем
  • 00:04:25 - Подключение upstream
  • 00:07:12 - Локальный запуск
  • 00:10:12 - Рабочая версия PHP
  • 00:11:52 - Обзор компонента
  • 00:16:33 - Формирование ответов
  • 00:18:51 - Неудобство кода с сеттерами
  • 00:20:38 - Формирование редиректов
  • 00:22:16 - Избыточность интерфейса
  • 00:23:27 - Разделение интерфейсов
  • 00:25:39 - Устройство сервера ресурсов
  • 00:29:51 - Неиспользуемые параметры
  • 00:31:36 - Упрощение создания валидатора
  • 00:34:39 - Скрытая неожиданность
  • 00:36:45 - Альтернативы промежуточных сеттеров
  • 00:39:36 - Интерфейс для сеттера
  • 00:44:45 - Верить ли процентам тестового покрытия
  • 00:49:00 - Обзор результата
Скрытый контент
Комментарии (3)
Руслан

Спасибо!

Ответить
Владимир

Очень полезно, спасибо

Ответить
d.arnautov

Спасибо, полезный материал

Ответить
Зарегистрируйтесь или войдите чтобы оставить комментарий

Или войти через:

Google
GitHub
Yandex
MailRu
  • 1Рефакторинг сеттеров и интерфейсов