Code Inspection: Field can be made readonly (non-private accessibility)
This is a solution-wide code inspection. It only works when the solution-wide analysis is enabled.
Say you have decided to make a
Person class, initialized only via the constructor, but with publicly available fields. You go ahead and implement the following:
If solution-wide inspection is enabled, ReSharper can detect that the fields are only being assigned in the constructor and offers to create an additional safeguard: by marking them
readonly, we get to ensure that neither this class nor its users will inadvertently assign new values to these fields.
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 page of ReSharper options (Alt+R, O).
Solution-wide analysis: select Enable solution-wide analysis on the page of ReSharper options (Alt+R, O).
Note that even if the reported field 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 field in one of the following ways:
The recommended way is to decorate the implicitly used field 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 field 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.