Business Validation Logic Code Smell

Question

Consider the following code:

partial class OurBusinessObject {
    partial void OnOurPropertyChanged() {
        if(ValidateOurProperty(this.OurProperty) ==          


        

Answer 1

You are validating a change after it has been done? Validation should be done before the busyness property is altered.

Answering your questing: the solution presented in that code snippet can generate big issues in production, you don't know whether the default value appeared there due to invalid input or just because something else set the value to the default