Files
codeql/csharp/ql/test/query-tests/Bad Practices/Control-Flow/ConstantCondition/ConstantCondition.cs
2025-03-11 14:35:04 +01:00

170 lines
3.2 KiB
C#

using System;
using System.Collections;
using System.Diagnostics;
class ConstantCondition
{
const bool Field = false;
void M1(int x)
{
if (Field) // GOOD: Allow conditional execution based on constant field
;
const bool local = false;
if (local) // GOOD: Allow conditional execution based on local constant
;
try
{
throw new ArgumentNullException("x");
}
finally
{
if (x > 1) // No 'false' successor (instead a 'throw[ArgumentNullException]' successor)
throw new Exception();
}
}
int M2(bool? b) => (b ?? false) ? 0 : 1; // GOOD
bool M3(double d) => d == d; // BAD: but flagged by cs/constant-comparison
}
class ConstantNullness
{
void M1(int i)
{
var j = ((string)null)?.Length; // $ Alert
var s = ((int?)i)?.ToString(); // $ Alert
var k = s?.Length; // GOOD
k = s?.ToLower()?.Length; // GOOD
}
void M2(int i)
{
var j = (int?)null ?? 0; // $ Alert
var s = "" ?? "a"; // $ Alert
j = (int?)i ?? 1; // $ Alert
s = ""?.CommaJoinWith(s); // $ Alert
s = s ?? ""; // GOOD
s = (i == 0 ? s : null) ?? s; // GOOD
var k = (i == 0 ? s : null)?.Length; // GOOD
}
}
class ConstantMatching
{
void M1()
{
switch (1 + 2)
{
case 2: // $ Alert
break;
case 3: // $ Alert
break;
case int _: // GOOD
break;
}
}
void M2(string s)
{
switch ((object)s)
{
case int _: // $ Alert
break;
case "": // GOOD
break;
}
}
void M3(object o)
{
switch (o)
{
case IList _: // GOOD
break;
}
}
string M4(object o)
{
return o switch
{
_ => o.ToString() // $ Alert
};
}
string M5(object o)
{
return o switch
{
"" => " ",
_ => o.ToString() // GOOD
};
}
void M6(bool b1, bool b2)
{
if (!b1)
return;
if (!b2)
return;
if (b1 && b2) // $ Alert
return;
}
string M7(object o)
{
return o switch
{
(string s, _) => s, // GOOD
(_, string s) => s, // GOOD
_ => "" // GOOD
};
}
string M8(int i)
{
return i switch
{
_ when i % 2 == 0 => "even", // GOOD
_ => "odd" // GOOD
};
}
string M9(int i)
{
switch (i)
{
case var _: // $ Alert
return "even";
}
}
string M10(int i)
{
switch (i)
{
case var _ when i % 2 == 0: // GOOD
return "even";
case var _: // GOOD
return "odd";
}
}
}
class Assertions
{
void F()
{
Debug.Assert(false ? false : true); // GOOD
}
}
static class Ext
{
public static string CommaJoinWith(this string s1, string s2) => s1 + ", " + s2;
}