Skip to content

Commit 06ee71d

Browse files
committed
Tidy up HasMandatoryInAllParameterAttributes
1 parent 35d42c1 commit 06ee71d

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

Rules/AvoidDefaultValueForMandatoryParameter.cs

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,16 +83,15 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
8383
private static bool HasMandatoryInAllParameterAttributes(ParameterAst paramAst)
8484
{
8585
var parameterAttributes = paramAst.Attributes.OfType<AttributeAst>()
86-
.Where(attr => string.Equals(attr.TypeName?.Name, "parameter", StringComparison.OrdinalIgnoreCase))
87-
.ToList();
86+
.Where(attr => string.Equals(attr.TypeName?.Name, "parameter", StringComparison.OrdinalIgnoreCase));
8887

89-
return parameterAttributes.Count > 0 &&
88+
return parameterAttributes.Any() &&
9089
parameterAttributes.All(attr =>
91-
attr.NamedArguments?.OfType<NamedAttributeArgumentAst>()
90+
attr.NamedArguments.OfType<NamedAttributeArgumentAst>()
9291
.Any(namedArg =>
93-
string.Equals(namedArg?.ArgumentName, "mandatory", StringComparison.OrdinalIgnoreCase) &&
92+
string.Equals(namedArg.ArgumentName, "mandatory", StringComparison.OrdinalIgnoreCase) &&
9493
Helper.Instance.GetNamedArgumentAttributeValue(namedArg)
95-
) == true
94+
)
9695
);
9796
}
9897

0 commit comments

Comments
 (0)