Kiedy powinieneś refaktoryzować, zastępować lub wycofywać system legacy?

Question

Accepted Answer

Systemy legacy są niewygodne, ale często wartościowe – prowadzą biznes. Decyzja o **refaktoryzacji, zastąpieniu lub wycofaniu** musi być napędzana kosztem i wartością, a nie niechęcią do starego kodu. "To stare i brzydkie" nie jest powodem; "to nas kosztuje" to jest.

## Dlaczego to ważne

```text
REFACTOR (improve in place) when:
  - Core logic is sound; it's holding business value
  - Problems are localized; you can improve incrementally

REPLACE (rewrite/migrate) when:
  - It blocks important work and can't be incrementally fixed
  - Maintenance cost > rebuild cost over a sensible horizon
  - It's a growing security or reliability liability

RETIRE (remove entirely) when:
  - Its value no longer justifies any maintenance cost
  - Usage has dwindled; a simpler path exists
```

## Uważaj na pułapkę pełnego przepisania

Pełne przepisania są kusujące i często katastrofalne – zajmują więcej czasu niż oczekiwane, ponownie wprowadzają stare błędy, a system, który działał, tymczasem się psuje. Preferuj inkrementalną refaktoryzację lub stopniowe zastępowanie (strangler-fig). Sięgaj do pełnego przepisania tylko wtedy, gdy ścieżki inkrementalne są naprawdę niemożliwe.

## Konkretny przykład

Monolit jest bolesny, ale działa dobrze i nie jest twoim wąskim gardłem – refaktoryzuj najgorsze moduły, nie przepisuj. Ale system na wersji bazy danych, która utraci wsparcie bezpieczeństwa za rok, bez ścieżki aktualizacji – to wymuszone zastępienie, i planujesz je celowo.

## Pułapka

Decyzje emocjonalne: zastępowanie systemów wystarczająco dobrych, bo są niemodne, lub przyleganie do umierających z powodu poniesionych kosztów. Decyduj na podstawie kosztu i wartości.

## Dlaczego to ważne

Niepotrzebne przepisania marnują lata i ponownie wprowadzają rozwiązane problemy; zaniedbane legacy staje się dryfującym zagrożeniem.

Prawidłowe podjęcie tej decyzji, na podstawie dowodów, a nie estetyki, to jeden z najwyższego ryzyka sądów, które podejmuje senior TL.