Code inspection: Member can be made protected (non-private accessibility)
Category: Common Practices and Code Improvements
ID: MemberCanBeProtected.Global
EditorConfig: resharper_member_can_be_protected_global_highlighting=[error|warning|suggestion|hint|none]
Language: C#, VB.NET
Requires SWA: Yes
tip
JetBrains Rider is smart enough to determine where a particular class member is actually being used. If it isn’t used outside the type or derived types, JetBrains Rider will offer to mark the member as private
. If the member is used in derived classes too, JetBrains Rider will offer to mark it as protected
.
Note that these recommendations need to be considered before being automatically applied. After all, you may be creating a public API, or you may be making types intended to be accessed in non-standard ways (for example, reflection). In this case, constraining visibility might not be a good idea. If necessary, you can always suppress the inspection.
For the solution-wide inspection to work, you need to enable at least one of the following:
Simplified global usage checking: select Show unused non-private type members when solution-wide analysis is off on the Editor | Inspection Settings page of JetBrains Rider settings CtrlAlt0S.
Solution-wide analysis: select Enable solution-wide analysis on the Editor | Inspection Settings page of JetBrains Rider settings CtrlAlt0S.
Note that even if the reported member has no direct usages in your solution, there could be cases where it is used indirectly — for example, via reflection — or it could just be designed as public API. In all those cases, you would want to suppress the usage-checking inspection for the member in one of the following ways:
The recommended way is to decorate the implicitly used member with a code annotation attribute. There are two attributes for this purpose: [UsedImplicitly] and [PublicAPI], which are functionally similar, but let you and your teammates understand how the member is actually used.
You can also suppress usage-checking inspections with any custom attribute. To do so, mark the definition of that attribute with the [MeansImplicitUse] attribute.
And finally, you can suppress a specific usage-checking inspection as any other code inspection with a suppression comment or a suppression attribute.