مشاركة عبر


ca1044: يجب أن لا تكون خصائص الكتابة فقط

TypeName

PropertiesShouldNotBeWriteOnly

CheckId

ca1044

Category

Microsoft.تصميم

تعطيل تغيير

فصل

السبب

عامة أو محمية خاصية له ملحق التعيين ولكن لم يكن ملحق get.

وصف القاعدة

يحصل accessors توفر حق الوصول للقراءة إلى خاصية و التعيين accessors توفر حق الوصول للكتابة. أثناء ذلك هو مقبولة والضروري غالباً ما تحتوي على خاصية للقراءة فقط ومنع إرشادات التصميم باستخدام خصائص كتابة فقط لأنه يسمح لمستخدم بتعيين قيمة وثم منع مستخدم من عرض القيمة لا يوفر أي الأمان. أيضا، بدون حق الوصول للقراءة، الولاية الكائنات المشتركة ولا يمكن عرضه، التي تحد من فائدة الخاصة بهم.

كيف إلى الإصلاح انتهاكات

إلى إصلاح انتهاكا لهذه قاعدة، إضافة ملحق يحصل إلى الخاصية. بدلاً من ذلك، إذا كان سلوك خاصية كتابة فقط هو ضروريا، خذ بعين الاعتبار تحويل th هو الخاصية لأسلوب.

عند إلى منع التحذيرات

هو المستحسن أن لك القيام بمنع تحذير من القيم بالموضع لا هو قاعدة.

مثال

في المثال التالي، BadClassWithWriteOnlyPropertyهو نوع بالكتابة فقط خاصية. GoodClassWithReadWritePropertyيحتوي على تعليمات برمجية الذي تم تصحيحه.

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; 
         } 
      }
   }
}