Что делать если плохой код в проекте?

Стратегия работы с legacy-кодом в проекте

Когда сталкиваешься с плохим кодом (legacy code), важно действовать системно, а не эмоционально. Вот пошаговая стратегия, которую я выработал за 10+ лет работы с разными codebases.

1. Анализ и оценка ситуации

Первым делом нужно понять масштаб проблемы:

// Пример проблемного кода, с которым можно столкнуться
function processData(data) {
    // 500 строк кода в одной функции
    // Глобальные переменные
    // Смешение логики и представления
    // Отсутствие обработки ошибок
    if (window.globalConfig) {
        for (var i = 0; i < data.length; i++) {
            // Магические числа
            if (data[i].status == 3) {
                // Жесткие зависимости
                $("#result").append("<div>" + data[i].name + "</div>");
            }
        }
    }
}

Ключевые шаги анализа:

• Составить инвентаризацию проблемных мест
• Оценить связанность модулей
• Определить критические бизнес–процессы
• Выявить наиболее частые баги в этой части кода

2. Приоритизация рефакторинга

Не пытайтесь переписать всё сразу. Используйте принцип бритвы Оккама — начинайте с минимальных изменений, дающих максимальный эффект.

Моя система приоритетов:

1. Критические проблемы — код, вызывающий падения приложения
2. Высокочастотные изменения — модули, которые чаще всего модифицируются
3. Бизнес–критические компоненты — влияют на доход или ключевые метрики
4. Новые фичи вокруг legacy кода — рефакторинг как часть разработки новой функциональности

3. Практические техники рефакторинга

Медленный, безопасный рефакторинг

// Постепенное улучшение с TypeScript
// Вместо:
function getUser(id) {
    return fetch('/api/user/' + id);
}

// Постепенно:
interface User {
    id: number;
    name: string;
}

async function getUserSafe(id: number): Promise<User> {
    try {
        const response = await fetch(`/api/user/${id}`);
        return await response.json() as User;
    } catch (error) {
        console.error('Failed to fetch user', error);
        throw error;
    }
}

Техника "Wrap and Replace"

• Создайте новую чистую реализацию
• Оберните старый код новой абстракцией
• Постепенно переносите вызовы на новый интерфейс
• Удаляйте старый код только после полного перехода

4. Внедрение гарантий качества

Без тестов рефакторинг опасен:

// Начинаем с простых тестов
describe('Legacy Module', () => {
    test('should maintain existing behavior', () => {
        // Сохраняем старое поведение
        const result = legacyFunction(input);
        expect(result).toEqual(expectedValue);
    });
});

Поэтапное внедрение:

1. Интеграционные тесты для критических сценариев
2. Модульные тесты для новых изменений
3. Статический анализ (ESLint, TypeScript)
4. CI/CD проверки перед мержем

5. Командные практики и коммуникация

Культурные аспекты часто важнее технических:

• Документируйте decisions — почему меняем код
• Проводите code reviews — даже для небольших улучшений
• Создайте "правила игры" — стандарты для нового кода
• Образовывайте команду — регулярные сессии по clean code

6. Работа с зависимостями

Для монолитных legacy–систем:

// Стратегия изоляции
// 1. Выделяем четкие границы
const legacyAdapter = {
    // Изолируем взаимодействие со старым кодом
    processData: (data) => {
        // Вызов старой логики
        return oldComplexFunction(data);
    }
};

// 2. Новый код использует только адаптер
const newFeature = {
    useLegacyData: (data) => {
        // Четкая граница
        return legacyAdapter.processData(cleanData);
    }
};

7. План отката и мониторинг

Всегда имейте escape plan:

• Фиксируйте поведение до изменений (скриншоты, логи)
• Внедряйте feature toggles для постепенного включения
• Настройте мониторинг ключевых метрик
• Подготовьте rollback процедуру

Золотые правила

1. Не исправляйте то, что не сломано — если код работает и не меняется, иногда лучше оставить как есть
2. Рефакторите по мере необходимости — при добавлении фич, исправлении багов
3. Измеряйте прогресс — отслеживайте метрики качества (complexity, test coverage)
4. Будьте терпеливы — улучшение legacy–кода это марафон, а не спринт

Ключевой insight: Часто проблема не в коде как таковом, а в отсутствии стратегии его эволюции. Лучший подход — постепенное, контролируемое улучшение с постоянной обратной связью от системы и бизнеса.