Refactoring von Legacy-Code in JavaScript: Schritt-für-Schritt-Anleitung

Online Python-Trainer für Anfänger

Lernen Sie Python einfach ohne theoretische Überlastung. Lösen Sie praktische Aufgaben mit automatischer Überprüfung und schreiben Sie Code direkt im Browser.

Kurs starten

Refactoring von Legacy-Code in JavaScript: Wie man Chaos in Ordnung verwandelt

Die Arbeit mit Legacy-Code ist ein unvermeidlicher Teil des Entwicklerlebens. Früher oder später stößt jeder auf ein Projekt, das in veraltetem JavaScript geschrieben wurde, in dem Variablen a1, temp oder data heißen und die Logik unter Hunderten von Zeilen in einer einzigen Datei begraben liegt. Das Refactoring solchen Codes verbessert nicht nur die Lesbarkeit – es senkt die Wartungskosten, beschleunigt das Hinzufügen neuer Funktionen und reduziert die Anzahl der Fehler.

In diesem Artikel werden wir untersuchen, wie man an das Refactoring von Legacy-JavaScript-Code in einer Node.js-Umgebung herangeht, welche Tools man verwendet und wie man ein funktionierendes System nicht kaputt macht.

Warum Legacy-Code gefährlich ist und warum man ihn refactoren sollte

Legacy-Code (vom engl. legacy – Erbe) ist Code, der bereits in der Produktion läuft, aber unter Verletzung moderner Standards, ohne Tests und mit schlechter Architektur geschrieben wurde. Die Hauptprobleme solchen Codes sind:

Verständnisschwierigkeit. Fehlende Modularität, lange Funktionen, magische Zahlen.
Hohe Kopplung. Eine Änderung in einem Teil zerstört einen anderen, der auf den ersten Blick nicht damit zusammenhängt.
Fehlende Tests. Jede Änderung ist ein Risiko.
Veraltete Patterns. Verwendung von var anstelle von const/let, Callback-Hell, Fehlen von async/await.

Refactoring bedeutet nicht, alles von Grund auf neu zu schreiben. Es ist die schrittweise Verbesserung des Codes, ohne sein externes Verhalten zu ändern. Das Ziel ist es, den Code sauberer, sicherer und leichter wartbar zu machen.

Refactoring-Strategie: Wo anfangen

Bevor man den Code ändert, muss man einen Plan entwickeln. Chaotische Änderungen verschlimmern die Situation nur. Hier ist eine bewährte Strategie:

Dokumentieren Sie das aktuelle Verhalten. Notieren Sie, was jede Funktion tut, welche Eingabedaten sie erwartet und was sie zurückgibt.
Schreiben Sie Tests. Auch wenn es keine Unit-Tests gibt, schreiben Sie Integrationstests, die die wichtigsten Szenarien überprüfen. Verwenden Sie jest oder mocha.
Isolieren Sie Änderungen. Arbeiten Sie jeweils an einem Modul oder einer Funktion. Versuchen Sie nicht, alles auf einmal zu refactoren.
Wenden Sie kleine Schritte an. Eine Änderung – ein Commit. Wenn etwas kaputt geht, kann man leicht zurückrollen.

Beispiel: Sie haben eine Funktion, die Daten aus einer Datei verarbeitet und das Ergebnis speichert. Schreiben Sie zuerst einen Test, der überprüft, dass bei der Eingabedatei A die Ausgabedatei B entsteht. Dann refactoren Sie das Innere.

Typische Probleme von Legacy-JavaScript und ihre Lösungen

1. Callback-Hell → async/await

Eines der häufigsten Probleme in altem Node.js-Code sind verschachtelte Callbacks. Sie machen den Code unlesbar und schwer zu debuggen.

Schlechter Code:

function getUserData(userId, callback) {    db.query('SELECT * FROM users WHERE id = ?', [userId], function(err, user) {        if (err) return callback(err);        db.query('SELECT * FROM orders WHERE user_id = ?', [user.id], function(err, orders) {            if (err) return callback(err);            callback(null, { user, orders });        });    });}

Nach dem Refactoring:

async function getUserData(userId) {    const user = await db.query('SELECT * FROM users WHERE id = ?', [userId]);    const orders = await db.query('SELECT * FROM orders WHERE user_id = ?', [user.id]);    return { user, orders };}

Der Code ist kürzer, verständlicher und leichter zu debuggen geworden. Fehler werden über try/catch behandelt.

2. Globale Variablen und Mutation

In Legacy-Code trifft man oft auf globale Variablen, die in verschiedenen Teilen des Programms geändert werden. Dies führt zu unerwarteten Seiteneffekten.