Magic Numbers

Magic Numbers sind Zahlen, die dem Leser jedoch nichts sagen.

public void ApproveDocument(int status)
{
    if (status == 1) 
    {
        ..
    }
    else if (status == 2) 
    {
        ..
    }
}

Der Status 1 und 2 sagt und überhaupt nichts über den eigentlichen Status bzw. den Wert dahinter aus.

---

Wir können das beheben, in dem wir, wie in diesem Beispiel eine Enumeration nutzen, oder die Werte als Konstanten speichert.

public void ApproveDocument(int status)
{
    const int PENDING = 1;
    const int SIGNED = 2;

    if (status == PENDING) 
    {
        ..
    }
    else if (status == SIGNED) 
    {
        ..
    }
}