[NodeCanvas] Support for DIctionary

This topic has 0 replies, 2 voices, and was last updated 2 years, 6 months ago by Gavalakis.

Viewing 2 posts - 1 through 2 (of 2 total)

Author

Posts

October 2, 2023 at 4:56 pm #13014

leanon00

Participant

Hello again! Made dictionaries with SO as key viable (if they were not..)

Why? Because scriptable object as keys for dictionaries are better that strings (more flexible, less typos errors etc)

EditorUtils.GUIEditors.cs (lines: 223 – 253)

if ( GUILayout.Button("Add Element") ) {

if (!typeof(UnityObject).IsAssignableFrom(keyType))

{

object newKey = null;

if (keyType == typeof(string))

{

newKey = string.Empty;

}

else

{

newKey = Activator.CreateInstance(keyType);

}

if (dict.Contains(newKey))

{

Logger.LogWarning(string.Format("Key '{0}' already exists in Dictionary", newKey.ToString()), "Editor");

return dict;

}

keys.Add(newKey);

}

//CustomPatch: Added support for dictionary SO keys.

else if (typeof(ScriptableObject).IsAssignableFrom(keyType))

{

keys.Add(ScriptableObject.CreateInstance(keyType));

}

else

{

Logger.LogWarning("Can't add a 'null' Dictionary Key", "Editor");

return dict;

}

values.Add(valueType.IsValueType ? Activator.CreateInstance(valueType) : null);

}

October 9, 2023 at 12:17 pm #13015

Gavalakis

Keymaster

Hello again!

I will have to double check if everything works correctly before adding this because by convention dictionaries are only supported with string for KEY in various places in code (both serialization and other reflection utilities wise).

Thanks! 🙂

Author

Posts

Viewing 2 posts - 1 through 2 (of 2 total)

You must be logged in to reply to this topic.