1

I am having a binding issue I wasn't able to figure out for the past two days. I have thoroughly went through most of the relevant threads on SO, and I still wasn't able to pinpoint where my error lies.

The issue I'm having is with one of the textboxes in my program. The purpose of it is to show the file the user has selected from the file browser. I have bound the text property of it to a string called parameterFileSelected but the textbox never updates even though debugging seems to be showing that the iNotifyPropertyChanged is called and executed properly.

Please help me take a look at my code below if there are any mistakes in my code.

The textbox is part of an xaml called GenerateReports and this view is tied to the GenerateReportsViewModel as follows:

Code for setting datacontext to GenerateReportsViewModel

<Grid > <Grid.DataContext> <vm:GenerateReportsViewModel/> </Grid.DataContext> <Grid.ColumnDefinitions> ....

Code for TextBox. I have tried removing the Twoway mode, changing it to Oneway and removing the mode but there is no difference. 

<TextBox Grid.Column="2" Grid.Row="1" Margin="5" Text="{Binding parameterFileSelected, Mode=Twoway, UpdateSourceTrigger=PropertyChanged}" ></TextBox>

To get the file browser and then to pass the selected file result to the GenerateReportsViewModel, this is the function in the codebehind file. The genviewmodel is initialized in the beginning of the codebehind file as GenerateReportsViewModel genViewModel = new GenerateReportsViewModel();

private void ParaFileButtonClick(object sender, RoutedEventArgs e) { OpenFileDialog openFileDialog = new OpenFileDialog(); if (openFileDialog.ShowDialog() == true) { DataContext = genViewModel; genViewModel.updateParameterFileSelected(openFileDialog.FileName.ToString()); } }

This is the code that's called in GenerateReportsViewModel to update the parameterFileSelected string the textbox is bound to. 

class GenerateReportsViewModel : ViewModelBase { private string _parameterFileSelected; public string parameterFileSelected { get { return _parameterFileSelected; } set { SetValue(ref _parameterFileSelected, value); } } public void updateParameterFileSelected(string parameterFile) { parameterFileSelected = parameterFile; } }

Here is the ViewModelBase the viewmodel is attached to. 

public class ViewModelBase : INotifyPropertyChanged { public event PropertyChangedEventHandler PropertyChanged; public void SetValue<T>(ref T property, T value, [CallerMemberName] string propertyName = null) { if (property != null) { if (property.Equals(value)) return; } OnPropertyChanged(propertyName); property = value; } protected virtual void OnPropertyChanged([CallerMemberName] string propertyName = null) { var handler = PropertyChanged; if (handler != null) handler(this, new PropertyChangedEventArgs(propertyName)); } } }

EDIT Working Solution after Applying Kevin's Suggestions

For simplicity sake, the Datacontext was set in the XAML.

<Grid> <Grid.DataContext> <vm:GenerateReportsViewModel x:Name="generateReportsViewModel"/> </Grid.DataContext>

Then, I call the string the textbox was bound to, in the viewmodel directly from code behind.

 private void ParaFileButtonClick(object sender, RoutedEventArgs e) { OpenFileDialog openFileDialog = new OpenFileDialog(); if (openFileDialog.ShowDialog() == true) { generateReportsViewModel.parameterFileSelected = openFileDialog.FileName.ToString(); } }

The ViewModel now uses Kevin's ViewModelBase:

public class GenerateReportsViewModel : ViewModelBase { public string parameterFileSelected { get { return this.GetValue<string>(); } set { this.SetValue(value); } } }

Thank you Kevin for your solution. Now my 2-day-long problem is solved.

I found out that my previous ViewModelBase was calling iNotifyPropertyChanged but somehow when the View was updated, the value was null instead.

Improve this question

1 Answer 1

Reset to default
1

I'm trying to understand why using the ref keyword in your viewModel. I learned a nice way to create the BaseViewModel from the Classon and Baxter book which you can find below. The view-model implements the INotifyPropertyChanged like you did. What you did with [CallerMemberName] is great, it's really magical the way we can reference to our properties thanks to it.

The view model uses a the dictionary to store its properties. It uses a pretty neat trick of looking through the dictionnary keys to see if we contain the string name of the property.Otherwise, we will return a default T value.

 public class CommonBaseViewModel: INotifyPropertyChanged { private Dictionary<string, object> Values { get; set; } protected CommonBaseViewModel() { this.Values = new Dictionary<string, object>(); } public event PropertyChangedEventHandler PropertyChanged; protected T GetValue<T>([CallerMemberName] string name=null) { if (this.Values.ContainsKey(name)) { return (T)this.Values[name]; } else { return default(T); } } protected void SetValue(object value, [CallerMemberName] string name = null) { this.Values[name] = value; //notify my property this.OnPropertyChanged(new PropertyChangedEventArgs(name)); } protected void OnPropertyChanged([CallerMemberName] string name=null) { this.OnPropertyChanged(new PropertyChangedEventArgs(name)); } protected virtual void OnPropertyChanged(PropertyChangedEventArgs e) { if(this.PropertyChanged != null) { this.PropertyChanged(this, e); } } }

As for your GenerateReportViewModel, with the common view model that I provided you, your class then becomes : 

 public class GenerateReportsViewModel : CommonViewModelBase { private string _parameterFileSelected; public string parameterFileSelected { get { return _parameterFileSelected; } set { SetValue(ref _parameterFileSelected, value); } } get { return this.GetValue<string>(); } set { this.SetValue(value); } public void updateParameterFileSelected(string parameterFile) { parameterFileSelected = parameterFile; } }

Oh before I forgot, I don't know if it was your intention, but your GenerateReportViewModel is private. This has some impact on your code. Don't forget that by defaut, classes are private!

As for your code behind, even though it could be consider bad practice, I recommend that you have a private field (OpenFileDialog _openFileDialog)that you construct while initializing your page. Because doing it each time your clicking your button is going to consume more data that you need your application to.

//EDIT I have review my code,and it seemed that the property was not programmed correctly. public class GenerateReportsViewModel : CommonViewModelBase { 

 private string _parameterFileSelected; public string parameterFileSelected { get { return this.GetValue<string>(); } set { this.SetValue(value); } public void updateParameterFileSelected(string parameterFile) { parameterFileSelected = parameterFile; } }

More about my comment about constructing the page and binding the view model. While creating your page, you have to create the view-model for that page and then bind it to the data context. I don't know what you do in your code, but I could provide with this sample such as 

public GenerateReportView() { InitializeComponent(); //Some operations var generateReportViewModel = new GenerateReportViewModel(); this.DataContext = generateReportViewModel; }
Improve this answer
Sign up to request clarification or add additional context in comments.

6 Comments

Hi, thank you for responding. I have made my viewmodel to be a public class now, and have updated my viewmodelbase with the code you have provided. The textbox binding still does not work currently. I have another observable collection bound to a listview in my viewmodel and that is working as intended with your viewmodelbase as well as my previous viewmodelbase. Any idea what I could do further to try to fix the problem? Thanks again :)
When you're constructing your page, I see that you're not actually binding your viewModel as soon as your page is created ! What you're trying to do, is to bind the viewModel in your button event
I'm not entirely sure if the datacontext has been set for the viewModel properly but in the xaml of the view, I have bound to the viewModel in the grid <Grid > <Grid.DataContext> <vm:GenerateReportsViewModel/> </Grid.DataContext>
My get and set for the parameterFileSelected is already like the one you have provided in your Edits. I have added in a line to set the DataContext to generateReportViewModel after the InitializeComponent(); line. The textbox binding is still not working for me. There were no data binding errors in the output window as well. Really at a loss :(
Oh and for the creation of the viewmodel, I have done it like this: public partial class GenerateReports : UserControl { GenerateReportsViewModel genViewModel = new GenerateReportsViewModel();
|

Start asking to get answers

Find the answer to your question by asking.

Ask question

Explore related questions

See similar questions with these tags.