CODEWITHSUNDEEP
c#wpfmvvminotifypropertychangedfield-accessors
Trevor Hickey
Trevor Hickey

Reputation: 37836

What is the ideal way to define property changes from a Model to a View in MVVM?

I'm attempting to refactor some code in a MVVM architecture.
The Model has public values that are changed directly.

The UI listens for changes in these values.

Below is the event signaling code: 

public string LoadFilename   { get { return _loadFilename;   } set { _loadFilename   = value; OnPropertyChanged(); } }
public string SaveFilename   { get { return _saveFilename;   } set { _saveFilename   = value; OnPropertyChanged(); } }
public string OneSafFilename { get { return _oneSafFilename; } set { _oneSafFilename = value; OnPropertyChanged(); } }

public bool IsSaveEnabled           { get { return _isSaveEnabled;           } set { _isSaveEnabled           = value; OnPropertyChanged(); } }
public bool IsLoadEnabled           { get { return _isLoadEnabled;           } set { _isLoadEnabled           = value; OnPropertyChanged(); } }
public bool IsLoadCheckpointEnabled { get { return _isLoadCheckpointEnabled; } set { _isLoadCheckpointEnabled = value; OnPropertyChanged(); } }
public bool IsCheckpointEnabled     { get { return _isCheckpointEnabled;     } set { _isCheckpointEnabled     = value; OnPropertyChanged(); } }
public bool IsScenariosEnabled      { get { return _isScenariosEnabled;      } set { _isScenariosEnabled      = value; OnPropertyChanged(); } }

Here is the OnPropertyChanged function: 

    public event PropertyChangedEventHandler PropertyChanged;

    protected virtual void OnPropertyChanged([CallerMemberName] string propertyName = null)
    {
        var handler = PropertyChanged;
        if (handler != null) handler(this, new PropertyChangedEventArgs(propertyName));
    }

This seems like a lot of boilerplate for something that should be natural in MVVM.
I'd like to make it more concise, but I'm not sure where to start.

With views listening to the above properties, what should the getters and setters look like?

Upvotes: 0

Views: 99

Answers (3)

Henk Holterman
Henk Holterman

Reputation: 273219

Implementing the INPC always remains the ugly part of WPF/XAML. With a good base class it could reduce to { get { return _loadFilename; } set { Set(ref _loadFilename, value); } } but that's about as compact as it will get. Resharper has support for (refactoring to) this.

Btw, your code is also missing the optimization guard if(value != _loadFilename) . So a BindableBase base class is definitely a good idea.

Upvotes: 3

Ag71191
Ag71191

Reputation: 83

private type m_name;
    public type name
    {
        get { return m_name; }
        set
        {
            m_name = value;
            if (PropertyChanged != null)
            { PropertyChanged(this, new PropertyChangedEventArgs("name")); }
        }
    }

I use the above code for property changes that require a listener. type is the type of the property and name is the name of the property (change 'type' and 'name' anywhere seen in the code)

For a great tutorial on MVVM that explains more about property changes and code snippets, check out Jerry Nixon on youtube: https://www.youtube.com/watch?v=cB7KdYPQw1k It's a little outdated but still very helpful!

Upvotes: -1

Richard Szalay
Richard Szalay

Reputation: 84734

You may want to look into something like Fody's PropertyChanged, which rewrites the IL at build time to inject the change notification code.

Upvotes: 2

Related Questions