Skip to main content

Property setters are just wrong!

In general I think the use of property setters on classes is a sign of bad class design and a lack of OO principles - in most cases when you modify the 'state' of a class some 'behaviour' is invoked whether it be implicit or explicit;

e.g. when I change the address on a user account I want to make sure the address object has been populated (at least).

So these days I've started to design classes that have public getters & private setters, and if you want to modify state you are required to call a method, e.g. 'ChangeAddress'. This is nothing new, there are plenty of examples out there.

So in the example below 'Token' is property that's defined when the object is constructed and 'Path' is a property which is modified by calling the method 'ChangePath'.
    public class File : ObservableEntity<int>
{
private string _path;
private Guid _token;

private readonly IExtractFileProperties _propertiesExtractor = new FilePropertiesExtractor();


public File()
{
_token = Guid.NewGuid();
}


public virtual string Path
{
get { return _path; }
private set { ChangePropertyAndNotify(ref _path, value, x => Path); }
}

public virtual Guid Token
{
get { return _token; }
private set { ChangePropertyAndNotify(ref _token, value, x => Token); }
}


public File ChangePath(string path)
{
Path = path;

Size = _propertiesExtractor.Size(Path);
Format = _propertiesExtractor.Format(Path);
FileCreatedOn = _propertiesExtractor.CreatedOn(Path);
FileLastModified = _propertiesExtractor.LastModified(Path);

return this;
}
}


Now this attitude has been going well as it's made my classes more behaviour rich with better encapsulation & abstraction.

The only problem I've come across is when I'm trying to re-hydrate objects from the database where the properties I'm attempt to set have some kind of special behaviour - properties representing timestamps & versions that are historically set in the database via some function.

So in the following example I've got 3 properties - 'CreatedOn' 'LastModified' & 'Version'. I don't want to expose a method to set this values - I want them to be immutable...

But this is going to be a problem with an ORM, how am I going to update these values when saving or updating - I want them appear to be auto-updating from a users perspective.
    public abstract class ObservableEntity<T> : IEntity<T>, INotifyPropertyChanged
{
public event PropertyChangedEventHandler PropertyChanged = delegate { };

private T _id;

private DateTime? _createdOn;
private DateTime? _lastModified;
private int? _version;

public virtual T Id
{
get { return _id; }
private set { ChangePropertyAndNotify(ref _id, value, x => Id); }
}

public virtual DateTime? CreatedOn
{
get { return _createdOn; }
private set { ChangePropertyAndNotify(ref _createdOn, value, x => CreatedOn); }
}

public virtual DateTime? LastModified
{
get { return _lastModified; }
private set { ChangePropertyAndNotify(ref _lastModified, value, x => LastModified); }
}

public virtual int? Version
{
get { return _version; }
private set { ChangePropertyAndNotify(ref _version, value, x => Version); }
}

protected void ChangePropertyAndNotify<T2>(ref T2 value, T2 newValue, Expression<Func<object, T2>> expression)
{
value = newValue;
Notify(expression);
}

protected void Notify<T2>(Expression<Func<object, T2>> expression)
{
var propertyName = ((MemberExpression)expression.Body).Member.Name;
PropertyChanged(this, new PropertyChangedEventArgs(propertyName));
}
}

Now when using nHibernate as your ORM this becomes very easy because you can attach Listeners and do custom actions.

So I've got a custom action when NH saves or updates an entity to set these properties with the correct values.
public class EntitySaveOrUpdateEventListener : IPreInsertEventListener, IPreUpdateEventListener
{
public bool OnPreInsert(PreInsertEvent @event)
{
var hasType = (@event.Entity.GetType().GetInterface(typeof(IVersionable).Name, true) != null);
if (!hasType)
return false;

var now = DateTime.Now;
var createdOnIndex = new List<string>(@event.Persister.PropertyNames).FindIndex(x => x == "CreatedOn");
@event.State[createdOnIndex] = now;
SetPrivateVariable("CreatedOn", now, @event.Entity);

var lastModifiedIndex = new List<string>(@event.Persister.PropertyNames).FindIndex(x => x == "LastModified");
@event.State[lastModifiedIndex] = now;
SetPrivateVariable("LastModified", now, @event.Entity);

var versionIndex = new List<string>(@event.Persister.PropertyNames).FindIndex(x => x == "Version");
var version = GetPrivateVariable<int?>("Version", @event.Entity);

version = version.GetValueOrDefault() + 1;
@event.State[versionIndex] = version;
SetPrivateVariable("Version", version, @event.Entity);

return false;
}

public bool OnPreUpdate(PreUpdateEvent @event)
{
var hasType = (@event.Entity.GetType().GetInterface(typeof(IVersionable).Name, true) != null);
if (!hasType)
return false;

var now = DateTime.Now;
var lastModifiedIndex = new List<string>(@event.Persister.PropertyNames).FindIndex(x => x == "LastModified");
@event.State[lastModifiedIndex] = now;
SetPrivateVariable("LastModified", now, @event.Entity);

var versionIndex = new List<string>(@event.Persister.PropertyNames).FindIndex(x => x == "Version");
var version = GetPrivateVariable<int?>("Version", @event.Entity);

version = version.GetValueOrDefault() + 1;
@event.State[versionIndex] = version;
SetPrivateVariable("Version", version, @event.Entity);
return false;
}

public void SetPrivateVariable(string name, object value, object entity)
{
var entityType = entity.GetType();

var pi = entityType.GetProperty(name, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance);
pi.ReflectedType.BaseType.InvokeMember(name,
BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.SetProperty | BindingFlags.Instance,
null,
entity,
new[] { value });
}

public T GetPrivateVariable<T>(string name, object entity)
{
var entityType = entity.GetType();

var pi = entityType.GetProperty(name, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance);

return (T)pi.GetValue(entity, null);
}
}

Now if you follow this code, when an entity is saved for the first time then the 'CreatedOn' & 'LastModified' property values are set to the current date & time and the 'Version' property is incremented (the initial value is 0) via the private backing properties. When the entity is updated only the 'LastModified' & 'Version' properties are set via the private backing properties.

This event listener is then instantiated via the fluent NH config.
private static ISessionFactory CreateSessionFactoryImpl()
{
return Fluently.Configure()
.Database(MsSqlConfiguration.MsSql2005.ConnectionString(c => c.FromConnectionStringWithKey("Pronunciations")).ShowSql())
.Mappings(m => m.FluentMappings.AddFromAssemblyOf<IMap>())
.ExposeConfiguration(c =>
{
c.EventListeners.PreInsertEventListeners = new IPreInsertEventListener[] { new EntitySaveOrUpdateEventListener() };
c.EventListeners.PreUpdateEventListeners = new IPreUpdateEventListener[] { new EntitySaveOrUpdateEventListener() };
})
.BuildSessionFactory();
}



Awkward Coder

Comments

Popular posts from this blog

Implementing a busy indicator using a visual overlay in MVVM

This is a technique we use at work to lock the UI whilst some long running process is happening - preventing the user clicking on stuff whilst it's retrieving or rendering data. Now we could have done this by launching a child dialog window but that feels rather out of date and clumsy, we wanted a more modern pattern similar to the way <div> overlays are done on the web. Imagine we have the following simple WPF app and when 'Click' is pressed a busy waiting overlay is shown for the duration entered into the text box. What I'm interested in here is not the actual UI element of the busy indicator but how I go about getting this to show & hide from when using MVVM. The actual UI elements are the standard Busy Indicator coming from the WPF Toolkit : The XAML behind this window is very simple, the important part is the ViewHost. As you can see the ViewHost uses a ContentPresenter element which is bound to the view model, IMainViewModel, it contains 3 child v...

Showing a message box from a ViewModel in MVVM

I was doing a code review with a client last week for a WPF app using MVVM and they asked ' How can I show a message from the ViewModel? '. What follows is how I would (and have) solved the problem in the past. When I hear the words ' show a message... ' I instantly think you mean show a transient modal message box that requires the user input before continuing ' with something else ' - once the user has interacted with the message box it will disappear. The following solution only applies to this scenario. The first solution is the easiest but is very wrong from a separation perspective. It violates the ideas behind the Model-View-Controller pattern because it places View concerns inside the ViewModel - the ViewModel now knows about the type of the View and specifically it knows how to show a message box window: The second approach addresses this concern by introducing the idea of messaging\events between the ViewModel and the View. In the example ...

WPF tips & tricks: Dispatcher thread performance

Not blogged for an age, and I received an email last week which provoked me back to life. It was a job spec for a WPF contract where they want help sorting out the performance of their app especially around grids and tabular data. I thought I'd shared some tips & tricks I've picked up along the way, these aren't probably going to solve any issues you might be having directly, but they might point you in the right direction when trying to find and resolve performance issues with a WPF app. First off, performance is something you shouldn't try and improve without evidence, and this means having evidence proving you've improved the performance - before & after metrics for example. Without this you're basically pissing into the wind, which can be fun from a developer point of view but bad for a project :) So, what do I mean by ' Dispatcher thread performance '? The 'dispatcher thread' or the 'UI thread' is probably the most ...