Regel
Nicht platzieren Zuweisungen innerhalb von Konditionalen. 
Mischen von Zuweisung und Bedingung Logik macht Code fehleranfällig
und schwieriger zu verstehen. Trennen Sie Zuweisungen von logischen Prüfungen. 

Unterstützte Sprachen:** JavaScript, TypeScript, Python, PHP

Einleitung

Zuweisungsoperatoren innerhalb von Bedingungsanweisungen sind eine häufige Fehlerquelle, die Compiler und Linter oft übersehen. Der klassische Fehler ist die Verwendung von = (Zuweisung) anstelle von == oder === (Vergleich) in einer if-Anweisung, aber das Problem geht tiefer. Selbst beabsichtigte Zuweisungen in Bedingungsanweisungen erzeugen Code, der schwer zu lesen, zu überprüfen und zu debuggen ist. Wenn Zuweisung und Auswertung in derselben Zeile stattfinden, müssen Lesende mental analysieren, welche Operation Vorrang hat und welcher Wert tatsächlich getestet wird.

Warum es wichtig ist

Warum es wichtig ist

Fehlereinführung: Ein Tippfehler ändert === zu = verursacht keinen Syntaxfehler, sondern ändert das Verhalten stillschweigend. Die Bedingung wird zum zugewiesenen Wert (truthy/falsy) ausgewertet, nicht zum Vergleichsergebnis.

Code-Lesbarkeit: Lesende erwarten, dass Konditionale Werte testen, nicht modifizieren. Wenn beides gleichzeitig geschieht, müssen Wartende nachvollziehen, welche Variablen wann modifiziert werden.

Code-Beispiele

❌ Nicht konform:

function processUser(userData) {
    if (user = userData.user) {
        console.log(`Processing user: ${user.name}`);
        return user.id;
    }
    return null;
}

function validateInput(value) {
    if (result = value.match(/^\d{3}-\d{2}-\d{4}$/)) {
        return result[0];
    }
    return false;
}

Warum es falsch ist: Die Zuweisungen innerhalb von Bedingungsanweisungen machen unklar, ob dies beabsichtigt oder ein Tippfehler ist. Das erste Beispiel könnte ein Fehler sein, bei dem === beabsichtigt war, und das zweite mischt Regex-Matching mit Zuweisung, was den Codefluss schwer nachvollziehbar macht.

✅ Konform:

function processUser(userData) {
    const user = userData.user;
    if (user) {
        console.log(`Processing user: ${user.name}`);
        return user.id;
    }
    return null;
}

function validateInput(value) {
    const result = value.match(/^\d{3}-\d{2}-\d{4}$/);
    if (result) {
        return result[0];
    }
    return false;
}

Warum dies wichtig ist: Die Trennung der Zuweisung von der Bedingung macht die Absicht glasklar. Leser erkennen sofort, dass Benutzer wird zuerst extrahiert, dann getestet. Das Regex-Match-Ergebnis wird erfasst, dann ausgewertet. Keine Mehrdeutigkeit, kein kognitiver Overhead und Tippfehler wie = vs === offensichtlich werden.

Fazit

Zuweisungen von Bedingungen zu trennen, ist eine einfache Regel, die eine ganze Klasse von Fehlern verhindert. Der kognitive Aufwand beim Parsen kombinierter Operationen überwiegt jeden vermeintlichen Vorteil der Kürze. Klarer, expliziter Code, bei dem Zuweisung und Auswertung getrennte Operationen sind, verbessert die Lesbarkeit, reduziert Fehler und macht Code-Reviews effektiver.