rspec/rules/S3260/csharp/rule.adoc

57 lines
876 B
Plaintext
Raw Normal View History

== Why is this an issue?
2021-09-21 09:00:14 +02:00
``++private++`` classes and records aren't visible outside of their assemblies anyway, so if they're not extended inside the assemblies, they should be made explicitly non-extensible with the addition of the ``++sealed++`` keyword.
2021-04-28 16:49:39 +02:00
=== Noncompliant code example
2021-04-28 16:49:39 +02:00
2022-02-04 17:28:24 +01:00
[source,csharp]
2021-04-28 16:49:39 +02:00
----
private class MyClass // Noncompliant
{
// ...
}
2021-09-21 09:00:14 +02:00
private record MyRecord // Noncompliant
{
// ...
}
2021-04-28 16:49:39 +02:00
----
=== Compliant solution
2021-04-28 16:49:39 +02:00
2022-02-04 17:28:24 +01:00
[source,csharp]
2021-04-28 16:49:39 +02:00
----
private sealed class MyClass
{
// ...
}
2021-09-21 09:00:14 +02:00
private sealed record MyRecord
{
// ...
}
2021-04-28 16:49:39 +02:00
----
ifdef::env-github,rspecator-view[]
'''
== Implementation Specification
(visible only on this page)
=== Message
Mark this class/record "sealed".
'''
== Comments And Links
(visible only on this page)
=== on 20 Jul 2015, 11:30:41 Tamas Vajk wrote:
\[~ann.campbell.2] LGTM
endif::env-github,rspecator-view[]