Как минимизировать риск поломки функционала при доработке legacy-системы?

Понимание проблемы

Работа с legacy-системами — одна из самых сложных задач в разработке. Код часто не имеет тестов, документация устарела, а бизнес-логика запутана. Любое изменение может привести к неожиданным поломкам. Основная цель — внести изменения с минимальным риском регрессии.

Основные стратегии

• Strangler Fig Pattern: постепенно заменяйте части старой системы новыми, пока старая не перестанет использоваться. Это позволяет откатывать изменения без простоя.
• Feature Toggles: внедряйте новый функционал под флагом, чтобы можно было быстро отключить его в случае проблем.
• Тестирование перед изменениями: сначала напишите тесты на существующее поведение (characterization tests), затем вносите правки. Это создаёт "сеть безопасности".
• Маленькие шаги: делайте коммиты как можно меньше, чтобы в случае ошибки легко откатиться.

Практический пример

Допустим, у вас есть функция расчёта скидки в legacy-коде:

function calculateDiscount(price, userType) {
  // legacy code without tests
  if (userType === 'vip') return price * 0.9;
  if (userType === 'regular') return price * 0.95;
  return price;
}

Перед рефакторингом напишите тест, фиксирующий текущее поведение:

test('calculateDiscount returns correct values', () => {
  expect(calculateDiscount(100, 'vip')).toBe(90);
  expect(calculateDiscount(100, 'regular')).toBe(95);
  expect(calculateDiscount(100, 'guest')).toBe(100);
});

Теперь можно безопасно рефакторить, например, вынести логику в конфиг или добавить новые типы пользователей.

Вывод

Применяйте комбинацию изолированных изменений, тестирования и постепенного рефакторинга. Это позволяет модернизировать legacy-систему с контролируемым риском и сохранить доверие бизнеса.