CA2222: Do not decrease inherited member visibility
| Item | Value |
|---|---|
| RuleId | CA2222 |
| Category | Microsoft.Usage |
| Breaking change | Non-breaking |
Cause
A private method in an unsealed type has a signature that is identical to a public method declared in a base type. The private method is not final.
Note
This rule has been deprecated. For more information, see Deprecated rules.
Rule description
Don't change the access modifier for inherited members. Changing an inherited member to private does not prevent callers from accessing the base class implementation of the method. If the member is made private and the type is unsealed, inheriting types can call the last public implementation of the method in the inheritance hierarchy. If you must change the access modifier, either the method should be marked final or its type should be sealed to prevent the method from being overridden.
How to fix violations
To fix a violation of this rule, change the access to be non-private. Alternatively, if your programming language supports it, you can make the method final.
When to suppress warnings
Do not suppress a warning from this rule.
Example
The following example shows a type that violates this rule.
using System;
namespace UsageLibrary
{
public class ABaseType
{
public void BasePublicMethod(int argument1) {}
}
public class ADerivedType:ABaseType
{
// Violates rule: DoNotDecreaseInheritedMemberVisibility.
// The compiler returns an error if this is overridden instead of new.
private new void BasePublicMethod(int argument1){}
}
}