Reply To: Feature Request: Variable popups in scripts

May 30, 2017 at 11:47 pm #12202

ReimG

Participant

Yes exactly that’s what I meant.

I needed something simple and fast for my game so I quickly created this simple classes:

[Serializable]

public class NCVariable

{

public string Name;

public string FieldName;

public int IntValue;

public float FloatValue;

public bool BoolValue;

public string StringValue;

public Vector3 VectorValue;

}

[Serializable]

public class NCVariableArray

{

public Blackboard Board;

public List<NCVariable> Variables = new List<NCVariable>();

}

and then added editor for it:

[CustomPropertyDrawer(typeof(NCVariableArray))]

public class NCVariableArrayEditor : PropertyDrawer

{

private const string BLACKBOARD = "Board", VARIABLES = "Variables";

private string[] _variableNames;

private Dictionary<NCVariable, int> _selected = new Dictionary<NCVariable, int>();

private NCVariableArray _target;

private ReorderableList _list;

private ReorderableList GetList(SerializedProperty property)

{

if (_target == null || _target.Board == null)

return null;

_variableNames = _target.Board.GetVariableNames();

return _list ?? (_list = new ReorderableList(property.serializedObject, property, true, true, true, true)

{

drawElementCallback = (rect, index, active, focused) =>

{

var element = _list.serializedProperty.GetArrayElementAtIndex(index);

rect.y += 2;

rect.height -= 4;

var variable = _target.Variables[index];

var position = new Rect(rect.x, rect.y, rect.width / 3 - 5, rect.height);

if (!_selected.ContainsKey(variable))

{

_selected.Add(variable, 0);

var selected = _variableNames.Select((v, i) => new {Var = v, Index = i})

.FirstOrDefault(v => v.Var == variable.Name);

if (selected != null)

_selected[variable] = selected.Index;

}

_selected[variable] = EditorGUI.Popup(position, _selected[variable], _variableNames);

variable.Name = _variableNames[_selected[variable]];

position = new Rect(rect.x + rect.width / 3, rect.y, 2 * rect.width / 3, rect.height);

var targetVar = _target.Board.GetVariable(_target.Variables[index].Name);

var value = variable.GetType().GetFields().LastOrDefault(f => f.FieldType == targetVar.varType);

if (value == null)

{

EditorGUI.HelpBox(position, "Variable type not supported", MessageType.Warning);

return;

}

variable.FieldName = value.Name;

var valueProperty = element.FindPropertyRelative(value.Name);

EditorGUI.PropertyField(position, valueProperty, GUIContent.none);

drawHeaderCallback = rect => { EditorGUI.LabelField(rect, "Variables"); },

});

}

public override float GetPropertyHeight(SerializedProperty property, GUIContent label)

{

var list = GetList(property.FindPropertyRelative(VARIABLES));

var height = list == null ? .0f : list.GetHeight();

return height + EditorGUIUtility.singleLineHeight;

}

public override void OnGUI(Rect position, SerializedProperty property, GUIContent label)

{

if (_target == null)

{

_target = fieldInfo.GetValue(property.serializedObject.targetObject) as NCVariableArray;

if (_target == null) return;

}

EditorGUI.PropertyField(new Rect(position.x, position.y, position.width, EditorGUIUtility.singleLineHeight), property.FindPropertyRelative(BLACKBOARD));

if (_target.Board == null) return;

position.y += EditorGUIUtility.singleLineHeight;

var list = GetList(property.FindPropertyRelative(VARIABLES));

if (list == null) return;

_list.DoList(position);

}

Maybe it’s not practical (I’m storing additional variables that aren’t needed and only take memory, and I only support only few types), but for my purposes it suffice. Maybe if I have time to implement something more sophisticated I’ll try to cram all fields into one ‘object’ field and serialize it.

Also I love that you have ‘varType’ field. So cool 😀

Thanks, and cheers!