Don't call INPC until after setting the property.

Author: davidbritch

UserInterface/ListView/SwitchEntryTwoBinding/twoWayBinding/twoWayBinding/Model/light.cs

@@ -1,22 +1,59 @@
-using System;
-using System.ComponentModel;
-using Xamarin.Forms;
+using System.ComponentModel;
 using System.Runtime.CompilerServices;
+using Xamarin.Forms;
 
 namespace twoWayBinding
 {
-	public class light : INotifyPropertyChanged
+    public class light : INotifyPropertyChanged
 	{
 		public event PropertyChangedEventHandler PropertyChanged;
 
 		private string _name;
 		private string _comment;
 		private Color _color;
 		private bool _isOn;
-		public string name { get{ return _name;} set{ OnPropertyChanged (); _name = value;} }
-		public string comment { get{return _comment;} set{OnPropertyChanged ();_comment = value;} }
-		public Color color { get{ return _color;} set{ OnPropertyChanged (); _color = value;} }
-		public bool isOn { get{ return _isOn;} set{OnPropertyChanged (); OnPropertyChanged ("isNotOn"); _isOn = value;} }
+
+		public string name
+		{
+			get	{ return _name;}
+			set
+			{
+				_name = value;
+				OnPropertyChanged();
+			}
+		}
+
+		public string comment
+		{
+			get{return _comment;}
+			set
+			{
+				_comment = value;
+				OnPropertyChanged();
+			}
+		}
+
+		public Color color
+		{
+			get{ return _color;}
+			set
+			{
+				_color = value;
+				OnPropertyChanged();
+			}
+		}
+
+		public bool isOn
+		{
+			get{ return _isOn;}
+			set
+			{
+				_isOn = value;
+				OnPropertyChanged ();
+				OnPropertyChanged ("isNotOn");
+			}
+		}
+
 		public bool isNotOn{ get { return !_isOn; } }
 
 		public light ()
@@ -26,13 +63,15 @@ public light ()
 			this.color = Color.Blue;
 			this.comment = "Bedroom";
 		}
+
 		public light(bool isOn, string name, Color color, string comment)
 		{
 			this.isOn = isOn;
 			this.name = name;
 			this.color = color;
 			this.comment = comment;
 		}
+
 		void OnPropertyChanged([CallerMemberName] string propertyName = null)
 		{
 			var handler = PropertyChanged;

UserInterface/ListView/SwitchEntryTwoBinding/twoWayBinding/twoWayBinding/View/EntryPage.xaml

@@ -1,12 +1,17 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<ContentPage xmlns="http://xamarin.com/schemas/2014/forms" xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml" x:Class="twoWayBinding.EntryPage" Title="Naming Panel" xmlns:local="clr-namespace:twoWayBinding;assembly=twoWayBinding">
-    <ContentPage.Content>
-        <ListView x:Name="listView" SeparatorVisibility="None" ItemsSource="{x:Static local:HomeViewModel.lights}">
-            <ListView.ItemTemplate>
-                <DataTemplate>
-                    <EntryCell Label="{Binding comment}" Text="{Binding name}" />
-                </DataTemplate>
-            </ListView.ItemTemplate>
-        </ListView>
-    </ContentPage.Content>
+<ContentPage xmlns="http://xamarin.com/schemas/2014/forms"
+             xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
+             xmlns:local="clr-namespace:twoWayBinding;assembly=twoWayBinding"
+             x:Class="twoWayBinding.EntryPage"
+             Title="Naming Panel">
+    <ListView x:Name="listView"
+              SeparatorVisibility="None"
+              ItemsSource="{x:Static local:HomeViewModel.lights}">
+        <ListView.ItemTemplate>
+            <DataTemplate>
+                <EntryCell Label="{Binding comment}"
+                           Text="{Binding name}" />
+            </DataTemplate>
+        </ListView.ItemTemplate>
+    </ListView>
 </ContentPage>