Vier oder mehr verkettete `if`/`else if`-Zweige, die denselben Wert per `==` mit einer Konstante vergleichen. Ein ausgeschriebenes `switch`.
Jedes Beispiel zeigt genau die Codeform, nach der die Regel sucht, und dann die kleinste Version, die die Absicht bewahrt, ohne Risiken zu verstecken oder Rauschen hinzuzufügen.
if (status == "pending")
return Color.Gray;
else if (status == "active")
return Color.Green;
else if (status == "suspended")
return Color.Orange;
else if (status == "closed")
return Color.Red;
return Color.Black;Deterministischer Zeilenabgleich: verfolgt ein `if (x == ...)` mit nachfolgenden `else if (x == ...)`-Zweigen auf derselben Variable und beanstandet die Kette ab vier Zweigen.
return status switch
{
"pending" => Color.Gray,
"active" => Color.Green,
"suspended" => Color.Orange,
"closed" => Color.Red,
_ => Color.Black,
};Nutze ein `switch`-Statement oder -Ausdruck, oder ein Dictionary von Handlern, wenn jeder Zweig einen Wert auf eine Aktion abbildet.
Jeder neue Fall wird zu einem weiteren `else if`. Agenten verlängern die Kette, statt sie umzubauen, also wächst sie mit jedem Feature.
ai-slop/csharp-if-ladder
ai-slop
Deterministischer Zeilenabgleich: verfolgt ein `if (x == ...)` mit nachfolgenden `else if (x == ...)`-Zweigen auf derselben Variable und beanstandet die Kette ab vier Zweigen.
Ketten mit drei oder weniger Zweigen, Ketten, die die Variable wechseln oder andere Operatoren nutzen, sowie bestehende `switch`-Statements werden nicht beanstandet.
Teil des C#-Regelsatzes von aislop. Vollwertige C#-Unterstützung, einschließlich C#-spezifischer AI-Slop-Regeln, kam mit aislop v0.14.1.