Changeset 4790 for branches/ParameterBinding/HeuristicLab.Core/3.3/ItemBinding.cs

Timestamp:

11/15/10 09:41:51 (14 years ago)

Author:

abeham

Message:

#1258

• Added detection if a certain link in the chain implements INotifyPropertyChanged (still missing -> fire only on a change to the "right" property)
• Added optional parameter LambdaExpression in the binding
• Changed cloning behavior of binding -> bindings have to be cloned only after the clone is fully constructed

File:

• branches/ParameterBinding/HeuristicLab.Core/3.3/ItemBinding.cs (modified) (12 diffs)

branches/ParameterBinding/HeuristicLab.Core/3.3/ItemBinding.cs

@@ -7 +7 @@
 using System.Reflection;
 using System.Reflection.Emit;
+using System.ComponentModel;
+using System.Linq.Expressions;
 
 namespace HeuristicLab.Core {
@@ -22 +24 @@
     [Storable]
     private bool bound;
-
+    [Storable]
+    private LambdaExpression bindingExpression;
+
+    private Delegate bindingFunc;
     private static object evhCacheLock = new object();
     private static Dictionary<Type, DynamicMethod> eventHandlerCache = new Dictionary<Type, DynamicMethod>();
@@ -33 +38 @@
       this.source = cloner.Clone(original.source);
       this.sourcePath = original.sourcePath;
+      this.bindingExpression = original.bindingExpression;
+      if (this.bindingExpression != null)
+        this.bindingFunc = this.bindingExpression.Compile();
       if (original.bound) Bind();
     }
@@ -42 +50 @@
       this.bound = false;
     }
+    public ItemBinding(IDeepCloneable target, string targetPath, IDeepCloneable source, string sourcePath, LambdaExpression func)
+      : this(target, targetPath, source, sourcePath) {
+      this.bindingExpression = func;
+      this.bindingFunc = this.bindingExpression.Compile();
+    }
 
     public IDeepCloneable Clone(Cloner cloner) {
@@ -54 +67 @@
     private void AfterDeserialization() {
       if (bound) RegisterEventHandlers();
+      if (bindingExpression != null)
+        bindingFunc = bindingExpression.Compile();
     }
 
@@ -71 +86 @@
       object current = source;
       foreach (string property in properties) {
-        TryHookToChangedEvent(current, property + "Changed");
+        TryHookToChangedEvent(current, property);
         MethodInfo mInfo = current.GetType().GetProperty(property, Flags).GetGetMethod(true);
         object next = mInfo.Invoke(current, null);
@@ -83 +98 @@
       object current = source;
       foreach (string property in properties) {
-        TryUnhookFromChangedEvent(current, property + "Changed");
+        TryUnhookFromChangedEvent(current, property);
         MethodInfo mInfo = current.GetType().GetProperty(property, Flags).GetGetMethod(true);
         object next = mInfo.Invoke(current, null);
@@ -95 +110 @@
       string[] sourceProperties = sourcePath.Split('.');
       object cSource = source;
-      #region Navigate to source property
+      #region Navigate to source property (readd bindings if necessary)
       bool reregister = false;
       for (int i = 0; i < sourceProperties.Length; i++) {
         if (cSource == null) return;
         string property = sourceProperties[i];
-        if (reregister) TryHookToChangedEvent(cSource, property + "Changed");
+        if (reregister) TryHookToChangedEvent(cSource, property);
         if (cSource == sender) reregister = true;
-        object next = cSource.GetType().GetProperty(property, Flags).GetGetMethod(true).Invoke(cSource, null);
+        PropertyInfo pInfo = cSource.GetType().GetProperty(property, Flags);
+        if (pInfo == null) return;
+        object next = pInfo.GetGetMethod(true).Invoke(cSource, null);
         cSource = next;
       }
@@ -112 +129 @@
       #region Navigate to target property
       for (int i = 0; i < targetProperties.Length - 1; i++) {
-        cTarget = cTarget.GetType().GetProperty(targetProperties[i], Flags).GetGetMethod(true).Invoke(cTarget, null);
+        PropertyInfo pInfo = cTarget.GetType().GetProperty(targetProperties[i], Flags);
+        if (pInfo == null) return;
+        cTarget = pInfo.GetGetMethod(true).Invoke(cTarget, null);
         if (cTarget == null) return;
       }
@@ -119 +138 @@
       #endregion
 
-      if (cSource == null && !targetSetter.GetParameters().First().ParameterType.IsValueType
+      if (bindingFunc != null) {
+        targetSetter.Invoke(cTarget, new object[] { bindingFunc.DynamicInvoke(new object[] { cSource }) });
+      } else if (cSource == null && !targetSetter.GetParameters().First().ParameterType.IsValueType
         || cSource != null && cSource.GetType().IsValueType)
         targetSetter.Invoke(cTarget, new object[] { cSource });
@@ -128 +149 @@
 
     private bool TryHookToChangedEvent(object obj, string property) {
-      EventInfo eInfo = obj.GetType().GetEvent(property, Flags);
-      if (eInfo != null) {
-        Type[] methodParams = eInfo.EventHandlerType.GetMethod("Invoke").GetParameters().Select(x => x.ParameterType).ToArray();
-        if (!eventHandlerCache.ContainsKey(methodParams[1]))
-          CreateAndAddDynamicEventHandler(eInfo, methodParams);
-        eInfo.AddEventHandler(obj, eventHandlerCache[methodParams[1]].CreateDelegate(eInfo.EventHandlerType, this));
-      }
-      return eInfo != null;
+      INotifyPropertyChanged npc = (obj as INotifyPropertyChanged);
+      if (npc != null) {
+        npc.PropertyChanged += new PropertyChangedEventHandler(source_PropertyChanged);
+        return true;
+      } else {
+        EventInfo eInfo = obj.GetType().GetEvent(property + "Changed", Flags);
+        if (eInfo != null) {
+          Type[] methodParams = eInfo.EventHandlerType.GetMethod("Invoke").GetParameters().Select(x => x.ParameterType).ToArray();
+          if (!eventHandlerCache.ContainsKey(methodParams[1]))
+            CreateAndAddDynamicEventHandler(eInfo, methodParams);
+          eInfo.AddEventHandler(obj, eventHandlerCache[methodParams[1]].CreateDelegate(eInfo.EventHandlerType, this));
+        }
+        return eInfo != null;
+      }
     }
 
     private bool TryUnhookFromChangedEvent(object obj, string property) {
-      EventInfo eInfo = obj.GetType().GetEvent(property, Flags);
-      if (eInfo != null) {
-        Type[] methodParams = eInfo.EventHandlerType.GetMethod("Invoke").GetParameters().Select(x => x.ParameterType).ToArray();
-        if (eventHandlerCache.ContainsKey(methodParams[1]))
-          eInfo.RemoveEventHandler(obj, eventHandlerCache[methodParams[1]].CreateDelegate(eInfo.EventHandlerType, this));
-      }
-      return eInfo != null;
+      INotifyPropertyChanged npc = (obj as INotifyPropertyChanged);
+      if (npc != null) {
+        npc.PropertyChanged -= new PropertyChangedEventHandler(source_PropertyChanged);
+        return true;
+      } else {
+        EventInfo eInfo = obj.GetType().GetEvent(property, Flags);
+        if (eInfo != null) {
+          Type[] methodParams = eInfo.EventHandlerType.GetMethod("Invoke").GetParameters().Select(x => x.ParameterType).ToArray();
+          if (eventHandlerCache.ContainsKey(methodParams[1]))
+            eInfo.RemoveEventHandler(obj, eventHandlerCache[methodParams[1]].CreateDelegate(eInfo.EventHandlerType, this));
+        }
+        return eInfo != null;
+      }
+    }
+
+    private void source_PropertyChanged(object sender, PropertyChangedEventArgs e) {
+      ExecuteBinding(sender);
     }
 
@@ -152 +189 @@
         if (!eventHandlerCache.ContainsKey(methodParams[1])) {
           Type[] instanceMethodParams = new Type[] { GetType() }.Concat(methodParams).ToArray();
-          DynamicMethod dynamicEventHandler = new DynamicMethod("sourceChanged", null, instanceMethodParams, GetType());
+          DynamicMethod dynamicEventHandler = new DynamicMethod("source_Changed", null, instanceMethodParams, GetType());
           MethodInfo executeBindingMethodInfo = GetType().GetMethod("ExecuteBinding", Flags);
           ILGenerator iLG = dynamicEventHandler.GetILGenerator();