UnityMirror/xNode
1
0
Fork 0
mirror of https://github.com/Siccity/xNode.git synced 2025-12-20 09:16:01 +08:00
Code

Minor OnValidate optimization

Browse Source 
Code now only runs on selected node
This commit is contained in:
Unknown 2017-09-23 02:43:29 +02:00
parent c66ac47ea9
commit 97bbc236be
1 changed files with 15 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
Scripts/Editor/NodeEditorGUI.cs
View File

@ -138,7 +138,14 @@ public partial class NodeEditorWindow {
Event e = Event.current;
if (e.type == EventType.Repaint) portConnectionPoints.Clear();
BeginWindows();
//Selected node is hashed before and after node GUI to detect changes
int nodeHash = 0;
System.Reflection.MethodInfo onValidate = null;
if (selectedNode != null) {
onValidate = selectedNode.GetType().GetMethod("OnValidate");
if (onValidate != null) nodeHash = selectedNode.GetHashCode();
}
BeginZoomed(position, zoom);
//if (e.type == EventType.Repaint) portRects.Clear();
foreach (Node node in graph.nodes) {
@ -157,11 +164,6 @@ public partial class NodeEditorWindow {
nodeEditor.target = node;
//Node is hashed before and after node GUI to detect changes
int nodeHash = 0;
var onValidate = node.GetType().GetMethod("OnValidate");
if (onValidate != null) nodeHash = node.GetHashCode();
nodeEditor.OnNodeGUI();
if (e.type == EventType.Repaint) {
foreach (var kvp in nodeEditor.portRects) {
@ -169,9 +171,6 @@ public partial class NodeEditorWindow {
}
}
//If a change in hash is detected, call OnValidate method. This is done through reflection because OnValidate is only relevant in editor, and thus, the code should not be included in build.
if (onValidate != null && nodeHash != node.GetHashCode()) onValidate.Invoke(node, null);
GUILayout.EndVertical();
GUILayout.FlexibleSpace();
@ -181,6 +180,12 @@ public partial class NodeEditorWindow {
GUILayout.EndArea();
}
EndZoomed(position, zoom);
EndWindows();
//If a change in hash is detected in the selected node, call OnValidate method.
//This is done through reflection because OnValidate is only relevant in editor,
//and thus, the code should not be included in build.
if (selectedNode != null) {
if (onValidate != null && nodeHash != selectedNode.GetHashCode()) onValidate.Invoke(selectedNode, null);
}
}
}