3

タイトルが示すように、検証クラスがモデルのすべてのプロパティにアクセスできるようにするのは良い考えかどうか疑問に思っています。理想的には、一部のフィールドでは、有効かどうかを確認するために 10 以上のフィールドが必要になるため、これを行いたいと考えています。10 個以上のパラメーターを持つ関数を使用することはできますが、使用したくはありません。それとも、モデルとバリデーターが互いに結びつきすぎてしまうのでしょうか? ここに私が言いたいことの小さな例があります。ただし、このコードは無限ループになるため機能しません。

Class User
    Private m_UserID
    Private m_Validator

    Public Sub Class_Initialize()
    End Sub

    Public Property Let Validator(value)
        Set m_Validator = value

        m_Validator.Initialize(Me)
    End Property

    Public Property Get Validator()
        Validator = m_Validator
    End Property

    Public Property Let UserID(value)
        m_UserID = value
    End property

    Public Property Get UserID()
        UserID = m_Validator.IsUserIDValid()
    End property End Class

Class Validator
    Private m_User

    Public Sub Class_Initialize()
    End Sub

    Public Sub Initialize(value)
        Set m_User = value
    End Sub

    Public Function IsUserIDValid()
        IsUserIDValid = m_User.UserID > 13
    End Function End Class

Dim mike : Set mike = New User 

mike.UserID = 123456  mike.Validator = New Validator

Response.Write mike.UserID

私が正しく、それが良い考えである場合、get プロパティ UserID を使用して無限ループを修正するにはどうすればよいですか?

ありがとうございました。

解決

<!-- #include file = "../lib/Collection.asp" -->

<style type="text/css">

td { padding: 4px; }

td.error 
{
    background: #F00F00;
}

td.warning 
{
    background: #FC0;
}

</style>

<%

Class UserModel
    Private m_Name
    Private m_Age
    Private m_Height

    Public Property Let Name(value)
        m_Name = value
    End Property

    Public Property Get Name()
        Name = m_Name
    End Property

    Public Property Let Age(value)
        m_Age = value
    End Property

    Public Property Get Age()
        Age = m_Age
    End Property

    Public Property Let Height(value)
        m_Height = value
    End Property

    Public Property Get Height()
        Height = m_Height
    End Property
End Class

Class NameValidation
    Private m_Name

    Public Function Init(name)
        m_Name = name
    End Function

    Public Function Validate()
        Dim validationObject

        If Len(m_Name) < 5 Then
            Set validationObject = New ValidationError
        Else
            Set validationObject = New ValidationSuccess
        End If

        validationObject.CellValue = m_Name

        Set Validate = validationObject
    End Function
End Class

Class AgeValidation
    Private m_Age

    Public Function Init(age)
        m_Age = age
    End Function

    Public Function Validate()
        Dim validationObject

        If m_Age < 18 Then
            Set validationObject = New ValidationError
        ElseIf m_Age = 18 Then
            Set validationObject = New ValidationWarning
        Else
            Set validationObject = New ValidationSuccess
        End If

        validationObject.CellValue = m_Age

        Set Validate = validationObject
    End Function
End Class

Class HeightValidation
    Private m_Height

    Public Function Init(height)
        m_Height = height
    End Function

    Public Function Validate()
        Dim validationObject

        If m_Height > 400 Then
            Set validationObject = New ValidationError
        ElseIf m_Height = 324 Then
            Set validationObject = New ValidationWarning
        Else
            Set validationObject = New ValidationSuccess
        End If

        validationObject.CellValue = m_Height

        Set Validate = validationObject
    End Function
End Class

Class ValidationError
    Private m_CSSClass
    Private m_CellValue

    Public Property Get CSSClass()
        CSSClass = "error"
    End Property

    Public Property Let CellValue(value)
        m_CellValue = value
    End Property

    Public Property Get CellValue()
        CellValue = m_CellValue
    End Property
End Class

Class ValidationWarning
    Private m_CSSClass
    Private m_CellValue

    Public Property Get CSSClass()
        CSSClass = "warning"
    End Property

    Public Property Let CellValue(value)
        m_CellValue = value
    End Property

    Public Property Get CellValue()
        CellValue = m_CellValue
    End Property
End Class

Class ValidationSuccess
    Private m_CSSClass
    Private m_CellValue

    Public Property Get CSSClass()
        CSSClass = ""
    End Property

    Public Property Let CellValue(value)
        m_CellValue = value
    End Property

    Public Property Get CellValue()
        CellValue = m_CellValue
    End Property
End Class

Class ModelValidator
    Public Function ValidateModel(model)
        Dim modelValidation : Set modelValidation = New CollectionClass

        ' Validate name
        Dim name : Set name = New NameValidation
        name.Init model.Name
        modelValidation.Add name

        ' Validate age
        Dim age : Set age = New AgeValidation
        age.Init model.Age
        modelValidation.Add age

        ' Validate height
        Dim height : Set height = New HeightValidation
        height.Init model.Height
        modelValidation.Add height

        Dim validatedProperties : Set validatedProperties = New CollectionClass
        Dim modelVal
        For Each modelVal In modelValidation.Items()
            validatedProperties.Add modelVal.Validate()
        Next

        Set ValidateModel = validatedProperties
    End Function
End Class

Dim modelCollection : Set modelCollection = New CollectionClass

Dim user1 : Set user1 = New UserModel
user1.Name = "Mike"
user1.Age = 12
user1.Height = 32
modelCollection.Add user1

Dim user2 : Set user2 = New UserModel
user2.Name = "Phil"
user2.Age = 18
user2.Height = 432
modelCollection.Add user2

Dim user3 : Set user3 = New UserModel
user3.Name = "Michele"
user3.Age = 32
user3.Height = 324
modelCollection.Add user3

' Validate all models in the collection
Dim modelValue
Dim validatedModels : Set validatedModels = New CollectionClass
For Each modelValue In modelCollection.Items()
    Dim objModelValidator : Set objModelValidator = New ModelValidator
    validatedModels.Add objModelValidator.ValidateModel(modelValue)
Next

%>

<table>
    <tr>
        <td>Name</td>
        <td>Age</td>
        <td>Height</td>
    </tr>
    <%

    Dim r, c
    For Each r In validatedModels.Items()
        %><tr><%
        For Each c In r.Items()
            %><td class="<%= c.CSSClass %>"><%= c.CellValue %></td><%        
        Next
        %></tr><%
    Next

    %>
</table>

生産するソリューションイメージ

完璧ではありませんが、私が始めたものよりもはるかに優れています. 基本的に、decorator パターンを使用することにしました。私の次のステップは、ほとんどの場合、各検証から Init() 関数を削除し、それを SetModel() 関数などに置き換えることです。そうすれば、各検証でモデル内のすべてのプロパティにアクセスできます。

皆さんありがとう。

4

2 に答える 2

2

私は通常、モデル全体を検証するバリデーターを定義します。この場合、User を受け取り、検証エラーのリストを含む ValidationResult を返すメソッドを持つ UserValidator クラスを用意します。

これにより、検証に影響を与えずに User クラスの実装を変更できます (たとえば、新しいプロパティを追加するたびに新しいメソッドを Validator クラスに追加したり、メソッドの署名方法を変更したい場合にメソッド シグネチャを変更したりする必要はありません)。 UserID が検証されるなど）。

于 2010-05-28T17:57:26.623 に答える
1

バリデーターにモデル全体を検証させることは正しいと思います。無限ループを破るために、バリデーターに値を渡すことができます

Public Property Get UserID()
     UserID = m_Validator.IsUserIDValid(m_userID)
End property 

// in Validator
Public Function IsUserIDValid(userID)
    IsUserIDValid = userID > 13
End Function

または、カプセル化を希望する場合は、検証なしでプロパティにアクセスするための Friend 関数を追加できます。

Public Property Get UserID()
     UserID = m_Validator.IsUserIDValid()
End property 

Friend Function GetUserID()
   GetUserID = m_userID
End Function

// in Validator
Public Function IsUserIDValid()
    // "private" access - to get the unvalidated property
    IsUserIDValid = m_user.GetUserID > 13
End Function

これを行う 3 つ目の方法は、オブジェクトを検証から分離することです。基本クラスは、検証なしですべてのプロパティを定義します。次に、検証を追加する子クラスを定義します。

class User
    Private m_userID
    Public Property Get UserID()
         UserID = m_userID
    End property 
End Class

class ValidatedUser inherits User
   Public Overrides Property Get UserID()
       if (m_userID<15)
           // handle invalid case, e.g. throw exception with property that is invalid
       UserID = m_userID
   End Property

   Public Function Validate()
    ' class-level validation
   End Function
End Class

最後のバリエーションでは、委任を使用して、基本的なユーザー プロパティを検証済みのものから分離します。実装する必要があるため、User を抽象クラスにします。1 つは検証あり、もう 1 つは検証なしです。

Class MustInherit User
   Public MustInherit Property Get UserID()
End Class

' A simple implementation of User that provides the properties
Class DefaultUser Inherits User
   Private m_UserID
   Public Overrides Property Get UserID()
      UserID = m_UserID
   End Property   
End Class

Class ValidatedUser Inherits User
   private Validator m_validator
   private User m_User

   Public Property Let Validator(value)
        Set m_Validator = value
        m_Validator.Initialize(m_User)
        ' note that validator uses m_User - this breaks the infinite recursion
    End Property

   Public Overrides Property Let UserID(value)
      m_User.UserID = value;
   End Property

   Public Overrides Property Get UserID()
      UserID = m_validator.IsUserValid();
   End Property
End Class

最後の例では、ValidatedUser は元のコードと似ていますが、主な違いは ValidatedUser 自体にプロパティ値がないことです。すべてのプロパティ アクセサーを m_User オブジェクトに委譲します。Validator は、検証なしで単純なプロパティを提供する m_user オブジェクトを使用するため、無限再帰はなくなります。

現在、プロパティの取得時に検証が行われます。これは、データを使用する前に検証し、プロパティが割り当てられるときに一時的な検証エラーを回避するために行われると思います。プロパティ レベルの検証に加えて、オブジェクトのすべてのプロパティ (特に複数プロパティの制約に関係するもの) をチェックする "オブジェクト全体" の検証メソッドを定義することもできます。たとえば、A+B+C < 50 という制約がある場合、AB と C を個別のプロパティとしてチェックすると、その条件 (A+B+C<50) が 3 回評価されることになります。これは不要であり、混乱を招きます。実際には 3 つのプロパティすべてに問題がある場合、エラーは 1 つの特定のプロパティに表示されるためです。

上記のすべてが Validation を User クラスにバインドするため、クライアントは検証を気にせずに User を使用できます。このアプローチには利点と欠点があります。利点は透過性です。クライアントは User オブジェクトを使用して、明示的に要求しなくてもバックグラウンドで検証を取得できます。欠点は、検証がモデルと非常に密接に結びついていることです。別の方法は、検証を User オブジェクトから完全に分離することです。これは検証を分離するだけでなく、「オブジェクト全体」の検証も提供します。例えば

' User is now a simple class (like DefaultUser above '
' with just properties, no validation '
Class UserValidator

   Public Function Validate(user)
     ' validate the given user object, return a list of
     ' validation errors, each validation error object
     ' that describes the property or properties
     ' that caused the validation error and why it's an error     
     ' E.g. '
     Dim ve As ValidationError 
     ve = new ValidationError
     ve.obj = user;   ' the object that failed validation
     ve.property = "userID"
     ve.msg = "userId must be < 15"
     ' potentially put several of these in a list and return to caller     
   End
End Class

User を操作するコードは、変更を行った後に Validate を明示的に呼び出す必要がありますが、これは通常問題にはならず、制御のレベルは自動的に行われるよりもはるかに優れています。（私の経験では、「自動」アクションは邪魔になるため、ほとんどの場合、ある時点で元に戻す必要があります。）

思った以上に書きました。これがお役に立てば幸いです。

PS: 私はあまり VB をやらないので、ときどき発生する構文エラーについてはご容赦ください。私は OO プログラマーなので、原則が正しいことは知っています。そして、「asp-classic」タグに気付きました-一部の例では、クラシックaspでは使用できない機能を使用していますが、別のバリデーターコード-最後の例はクラシックaspで問題ないはずです。

于 2010-06-01T21:30:39.777 に答える