Winform中TextBox与ListBox绑定问题：INotifyPropertyChanged未生效

Let's break down what's going wrong here and fix it step by step. I've run into similar binding quirks in VB.NET with WinForms and ViewModels before, so this should be straightforward once we align all the pieces.

1. Fix the INotifyPropertyChanged Implementation in CarViewModel #

First, let's make sure your ViewModel's property change notifications are firing correctly. Your fix to target SelectedCarModelName instead of CarName was on the right track, but we need to ensure the property setter only triggers the event when the value actually changes.

Here's the corrected ViewModel code:

Public Class CarViewModel
    Implements INotifyPropertyChanged

    Private _selectedCar As Car
    Public Property SelectedCar As Car
        Get
            Return _selectedCar
        End Get
        Set(value As Car)
            ' Only trigger change if the selected car actually differs
            If Not Equals(_selectedCar, value) Then
                _selectedCar = value
                ' Notify that the SelectedCar property has changed
                OnPropertyChanged(NameOf(SelectedCar))
            End If
        End Set
    End Property

    ' Property to hold your list of cars from EF
    Private _carList As List(Of Car)
    Public Property CarList As List(Of Car)
        Get
            Return _carList
        End Get
        Set(value As List(Of Car))
            _carList = value
            OnPropertyChanged(NameOf(CarList))
        End Set
    End Property

    ' INotifyPropertyChanged boilerplate
    Public Event PropertyChanged As PropertyChangedEventHandler Implements INotifyPropertyChanged.PropertyChanged

    Protected Overridable Sub OnPropertyChanged(propertyName As String)
        RaiseEvent PropertyChanged(Me, New PropertyChangedEventArgs(propertyName))
    End Sub
End Class

Note: If you need changes to individual Car properties (like editing the model name in the TextBox) to sync back to the ViewModel, make sure your Car model class also implements INotifyPropertyChanged.

2. Correct BindingSource Setup & Control Bindings #

Your BindingSource approach is solid, but the binding path was incorrect—you need to target the nested properties of SelectedCar (like CarName, Brand) instead of the SelectedCar object itself. Also, we need to link the ListBox's selected item directly to the ViewModel to trigger updates.

Here's how to set this up in your Form:

Private _viewModel As CarViewModel
Private _bindingSource As BindingSource

Private Sub Form1_Load(sender As Object, e As EventArgs) Handles MyBase.Load
    ' Initialize ViewModel and load data via EF
    _viewModel = New CarViewModel()
    _viewModel.CarList = LoadCarsFromDatabase() ' Replace with your EF query

    ' Initialize BindingSource with the ViewModel as its data source
    _bindingSource = New BindingSource()
    _bindingSource.DataSource = _viewModel

    ' Bind ListBox to the CarList property
    ListBox1.DataSource = _bindingSource
    ListBox1.DisplayMember = "CarName" ' Choose the property to show in the ListBox
    ' Critical: Link ListBox's selected item to the ViewModel's SelectedCar
    ListBox1.DataBindings.Add("SelectedItem", _bindingSource, "SelectedCar", True, DataSourceUpdateMode.OnPropertyChanged)

    ' Bind TextBoxes to the nested properties of SelectedCar
    txtCarName.DataBindings.Add("Text", _bindingSource, "SelectedCar.CarName", True, DataSourceUpdateMode.OnPropertyChanged, String.Empty)
    txtBrand.DataBindings.Add("Text", _bindingSource, "SelectedCar.Brand", True, DataSourceUpdateMode.OnPropertyChanged, String.Empty)
    txtYear.DataBindings.Add("Text", _bindingSource, "SelectedCar.Year", True, DataSourceUpdateMode.OnPropertyChanged, String.Empty)
End Sub

' Example method to load cars from your SQL Server DB via EF
Private Function LoadCarsFromDatabase() As List(Of Car)
    Using context As New YourDbContext() ' Replace with your EF DbContext class
        Return context.Cars.ToList()
    End Using
End Function

3. Why Your Previous Approach Failed #

• Incorrect Binding Path: When you bound the TextBox to bs, "SelectedCarModelName", you were pointing to the entire Car object, not its CarName property. WinForms can't display an object as text by default (unless you override ToString() on Car, which isn't ideal for binding).
• Uninitialized ViewModel Risk: If _CarViewModel was Nothing when you created the BindingSource, the source would be empty—even if you set the ViewModel later, the binding wouldn't pick it up without reassigning the DataSource.
• Missing ListBox-ViewModel Link: Without binding the ListBox's SelectedItem to the ViewModel's SelectedCar, selecting an item wouldn't update the ViewModel at all. That's why the getter ran but no data flowed to the TextBox.

Final Quick Checks #

• Ensure your Car class has public, gettable/settable properties for CarName, Brand, and Year (EF generates these automatically if you used database-first or model-first).
• Test setting a default selected car in the ViewModel's constructor (e.g., SelectedCar = CarList.FirstOrDefault() if the list isn't empty) to confirm TextBoxes populate on load.

内容的提问来源于stack exchange，提问作者Robert DeSautel