I’ve got a wizard which needs to validate if a user is logged in and then at the end validate that all the details they have entered are correct.
The issue is I’m not sure where to put the validation logic. At the moment I have a BuyMembership class which has validation on the Buy() method. However this won’t be called at the beginning of the wizard where I need to validate whether the user is unique and is eligible for buying a membership.
For this validation I’ve created a BuyMembershipValidation class which validates if the user is eligible.
The issue now is that I have to pass a different parameter object to the BuyMembershipValidation and the BuyMembership classes. This means the data is split.
Is there a better way of doing it. Should I only load part of the information into the BuyMembership class for the initial validation and then load the rest after?
Update:
I need to validate when they enter the wizard (check if they are logged in already) and if they aren’t then they will register as a new user otherwise I have to check if they have the right settings to buy membership as all users can’t buy a membership. That’s why I need two sets of validation. One for whether they’re eligible and another for the actual data they enter to register. However I would like to recheck that they are eligible when they make the final transaction just in case they somehow made it past the first wizard step or if the web service (where I’m doing the logic) is called from somewhere else at a later point.
Another Update:
I’ve added an answer with my decision.
TIA, Jonathan.
I’ve decided to go with just the one class. The reason is that there is common logic / behaviour in the IsEligible and Buy methods. One determines if the inputs meet the eligibility requirements and the other validates that all the details needed to buy the membership are valid. I don’t really see this as authentication related. The authentication is done in a separate web control and I just check the Identity.IsAuthenticated and Username to see if they are logged in. I then check their username and if they are eligible to buy then I let them continue, otherwise show an error message. In terms of input validation I already have validation controls on my webpage but the validation I’m concerned with here is server side business logic validation. For that reason I think it should be in the business logic class. Not some validation class separate from the logic that performs the payment. This may just be my way of looking at it but I think this fits well with the BDD ideas of keeping validation inside the entity that holds the data.