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

Einführung

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

Warum das wichtig ist

Warum das wichtig ist

Einführung von Fehlern: Ein Tippfehler ändert === zu = führt nicht zu einem Syntaxfehler, sondern ändert nur das Verhalten stillschweigend. Die Bedingung wird auf den zugewiesenen Wert (wahr/falsch) ausgewertet, nicht auf das Vergleichsergebnis.

Lesbarkeit des Codes: Die Leser erwarten, dass Conditionals Werte testen und nicht verändern. Wenn beides gleichzeitig geschieht, müssen die Betreuer nachvollziehen, welche Variablen wann geändert 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 Konditionalen machen es unklar, ob dies beabsichtigt oder ein Tippfehler ist. Das erste Beispiel könnte ein Fehler sein, bei dem === beabsichtigt war, und das zweite vermischt Regex-Matching mit Zuweisungen, 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 das wichtig ist: Durch die Trennung von Zuweisung und Bedingung wird die Absicht kristallklar. Die Leser sehen sofort, dass Benutzer wird zuerst extrahiert und dann getestet. Das Ergebnis der Regex-Übereinstimmung wird erfasst und dann ausgewertet. Keine Mehrdeutigkeit, kein kognitiver Mehraufwand, und Tippfehler wie = gegen === deutlich werden.

Schlussfolgerung

Die Trennung von Zuweisungen und Konditionalen ist eine einfache Regel, die eine ganze Klasse von Fehlern verhindert. Der kognitive Mehraufwand beim Parsen kombinierter Operationen überwiegt jeden vermeintlichen Vorteil der Kürze. Klarer, expliziter Code, in dem Zuweisung und Auswertung getrennte Operationen sind, verbessert die Lesbarkeit, reduziert Fehler und macht die Codeüberprüfung effektiver.