Constante não campos não devem estar visíveis

Causa

Um campo estático público ou protegido não é constante nem ele é somente leitura.

Descrição da regra

Campos estático que não são constantes nem leitura - apenas não são thread-safe.Acesso a tal campo deve ser controlado com cuidado e requer técnicas de programação avançadas para a sincronização de acesso ao objeto de classe.Como esses são difíceis de habilidades para aprender e mestre e teste de tal objeto apresenta seus próprios desafios, os campos estático são melhor utilizados para armazenar dados que não é alterado.Esta regra se aplica a bibliotecas; aplicativos não devem expor todos os campos.

Como corrigir violações

Para corrigir uma violação dessa regra, verifique o campo estático constante ou somente leitura.Se não for possível, recrie o tipo a ser usado um mecanismo alternativo sistema autônomo uma propriedade do thread-safe que gerencia o acesso de thread-safe ao campo base.Perceba que problemas sistema autônomo contenção de bloquear e deadlocks podem afetar o desempenho e o comportamento da biblioteca.

Quando suprimir avisos

É seguro suprimir um aviso da regra, se você estiver desenvolvendo um aplicativo e, portanto, têm controle total sobre o acesso para o tipo de campo estático.Designers de biblioteca devem eliminar um aviso da regra; usando campos estático não constante pode fazer usando a biblioteca difícil para os desenvolvedores para usar corretamente.

Exemplo

O exemplo a seguir mostra um tipo que violam essa regra.

Imports System

Namespace UsageLibrary

Public Class SomeStaticFields
    ' Violates rule: AvoidNonConstantStatic;
    ' the field is public and not a literal.
    Public Shared publicField As DateTime = DateTime.Now

    ' Satisfies rule: AvoidNonConstantStatic.
    Public Shared ReadOnly literalField As DateTime = DateTime.Now

    ' Satisfies rule: NonConstantFieldsShouldNotBeVisible;
    ' the field is private.
    Private Shared privateField As DateTime = DateTime.Now
End Class 
End Namespace

using System;

namespace UsageLibrary
{
   public class SomeStaticFields
   {
      // Violates rule: AvoidNonConstantStatic;
      // the field is public and not a literal.
      static public DateTime publicField = DateTime.Now;

      // Satisfies rule: AvoidNonConstantStatic.
      public static readonly DateTime literalField = DateTime.Now;

      // Satisfies rule: NonConstantFieldsShouldNotBeVisible;
      // the field is private.
      static DateTime privateField = DateTime.Now;
   }
}