CA1044: Właściwości nie powinny być tylko do zapisu

TypeName	PropertiesShouldNotBeWriteOnly
CheckId	CA1044
Kategoria	Microsoft.Design
Złamanie zmiany	Złamanie

Przyczyna

Właściwości publiczne lub chronione ma akcesor operacji set, ale nie ma akcesora operacji get.

Opis reguły

Get akcesorów zapewniają dostęp do właściwości odczytu oraz akcesor zestawu zapewniają dostęp do zapisu.Chociaż jest to dopuszczalne i często konieczne mają właściwość tylko do odczytu, wytyczne projektowania Zabroń używania właściwości tylko do zapisu.To dlatego, pozwalając użytkownikowi ustawić wartość, a następnie uniemożliwia użytkownikowi wyświetlanie wartości nie zapewnia żadnych zabezpieczeń.Także bez dostępu do odczytu, stan obiektów udostępnionych nie można przeglądać, które ograniczają ich użyteczność.

Jak naprawić naruszenia

Aby rozwiązać problem naruszenie tej zasady, należy dodać akcesora operacji get do właściwości.Alternatywnie Jeśli konieczne jest zachowanie właściwości tylko do zapisu, należy rozważyć konwersję tej właściwości do metody.

Kiedy do pomijania ostrzeżenia

Stanowczo zaleca się, aby nie Pomijaj ostrzeżenie od tej reguły.

Przykład

W poniższym przykładzie BadClassWithWriteOnlyProperty jest typ właściwości tylko do zapisu.GoodClassWithReadWritePropertyzawiera kod poprawiony.

Imports System

Namespace DesignLibrary

   Public Class BadClassWithWriteOnlyProperty

      Dim someName As String

      ' Violates rule PropertiesShouldNotBeWriteOnly.
      WriteOnly Property Name As String
         Set 
            someName = Value
         End Set 
      End Property

   End Class

   Public Class GoodClassWithReadWriteProperty

      Dim someName As String

      Property Name As String
         Get 
            Return someName
         End Get 

         Set 
            someName = Value
         End Set 
      End Property

   End Class

End Namespace

using System;

namespace DesignLibrary
{
   public class BadClassWithWriteOnlyProperty
   {
      string someName;

      // Violates rule PropertiesShouldNotBeWriteOnly.
      public string Name 
      { 
         set 
         { 
            someName = value; 
         } 
      }
   }

   public class GoodClassWithReadWriteProperty
   {
      string someName;

      public string Name 
      { 
         get 
         { 
            return someName; 
         } 
         set 
         { 
            someName = value; 
         } 
      }
   }
}