首页 > 其他 > 详细

Mutable fields should not be "public static"

时间:2020-09-30 16:45:30      阅读:589      评论:0      收藏:0      [点我收藏+]

https://rules.sonarsource.com/csharp/RSPEC-2386

https://github.com/SonarSource/sonar-dotnet/issues/3636

 public class CustomHelper
    {
        public static List<string> MandatoryFields = new List<string>
        {
            "First name",
            "Last name"
        };
    }

修改为,需要引用

https://docs.microsoft.com/en-us/dotnet/api/system.collections.immutable.immutablelist.create?view=netcore-3.1

  public class CustomHelper
    {
        public static IImmutableList<string> MandatoryFields = ImmutableList.Create(
            "First name",
            "Last name"
        );
    }

 

public static mutable fields of classes which are accessed directly should be protected to the degree possible. This can be done by reducing the accessibility of the field or by changing the return type to an immutable type.

This rule raises issues for public static fields with a type inheriting/implementing System.Array or System.Collections.Generic.ICollection<T>.

 

No issue is reported:

  • If the type of the field inherits/implements one (at least) of the following types:
    • System.Collections.ObjectModel.ReadOnlyCollection<T>
    • System.Collections.ObjectModel.ReadOnlyDictionary<TKey, TValue>
    • System.Collections.Immutable.IImmutableArray<T>
    • System.Collections.Immutable.IImmutableDictionary<TKey, TValue>
    • System.Collections.Immutable.IImmutableList<T>
    • System.Collections.Immutable.IImmutableSet<T>
    • System.Collections.Immutable.IImmutableStack<T>
    • System.Collections.Immutable.IImmutableQueue<T>
  • If the field is readonly and is initialized inline with an immutable type (i.e. inherits/implements one of the types in the previous list) or null. 

 

Mutable fields should not be "public static"

原文:https://www.cnblogs.com/chucklu/p/13754685.html

(0)
(0)
   
举报
评论 一句话评论(0
关于我们 - 联系我们 - 留言反馈 - 联系我们:wmxa8@hotmail.com
© 2014 bubuko.com 版权所有
打开技术之扣,分享程序人生!