Как правильно ограничить выбор enum значений в инспекторе для разных типов предметов?

Question

[MinZeroMax]

Столкнулся с проблемой в системе предметов.

Было:
Базовый ItemConfig с общим enum ItemType (оружие, броня и т.д.)
Наследники WeaponConfig и ClothesConfig

Проблема:
В инспекторе можно было назначить оружию тип брони или наоборот — это ломало логику.

Сейчас сделал так:
Разделил enum'ы: WeaponType (меч, лук) и ArmorType (шлем, доспех)
В WeaponConfig оставил только WeaponType, в ClothesConfig — только ArmorType

Вопрос:
Это нормальный подход, или есть более правильные способы? Хочется:
Чтоб в инспекторе нельзя было выбрать не те типы
Сохранить понятную структуру кода

Если знаете варианты лучше — подскажите. Спасибо!

namespace MainProject.Items
{
    public enum ItemType
    {
        None,
        //clothes
        Armor,
        Boots,
        Helmet,
        //weapons
        WeaponOneHanded,  
        WeaponTwoHanded,  
        OffHand
    }
    public enum WeaponType
    {
        None,
        WeaponOneHanded,
        WeaponTwoHanded,
        OffHand
    }

    public enum ArmorType
    {
        None,
        Helmet,
        Armor,
        Boots
    }
}

using UnityEngine;

namespace MainProject.Items.Config
{
    public abstract class ItemConfig : ScriptableObject
    {
        [field: SerializeField] public Sprite Icon { get; private set; }
        [field: SerializeField] public string Name { get; private set; }
        [field: SerializeField] public int MaxStack { get; private set; }
        public abstract ItemType ItemType { get; }

    }
}

using UnityEngine;

namespace MainProject.Items.Config
{
    [CreateAssetMenu(fileName = "ClothesConfig", menuName = "Configs/Items/Clothes")]
    public class ClothesConfig : ItemConfig
    {
        [field: SerializeField] public int PhysProtection { get; private set; }
        [field: SerializeField] public int MagProtection { get; private set; }
        [field: SerializeField] public ArmorType ArmorType { get; private set; }
        public override ItemType ItemType
        {
            get
            {   
                return ArmorType switch
                {
                    ArmorType.Helmet => ItemType.Helmet,
                    ArmorType.Armor => ItemType.Armor,
                    ArmorType.Boots => ItemType.Boots,
                    _ => ItemType.None
                };
            }
        }
    }
}

using UnityEngine;

namespace MainProject.Items.Config
{
    [CreateAssetMenu(fileName = "WeaponConfig", menuName = "Configs/Items/Weapon")]
    public class WeaponConfig : ItemConfig
    {
        [field: SerializeField] public int PhysDamage { get; private set; }
        [field: SerializeField] public int MagDamage { get; private set; }
        [field: SerializeField] public WeaponType WeaponType { get; private set; }

        public override ItemType ItemType
        {
            get
            {
                return WeaponType switch
                {
                    WeaponType.WeaponOneHanded => ItemType.WeaponOneHanded,
                    WeaponType.WeaponTwoHanded => ItemType.WeaponTwoHanded,
                    WeaponType.OffHand => ItemType.OffHand,
                    _ => ItemType.None
                };
            }
        }
    }
}

Comments

[K0TlK]

У тебя в комментах к ItemType написано то, что должно быть в ItemType.

ItemType {
    Clothes,
    Weapon
}

Можешь избавиться от наследников ItemConfig, все их поля перекинуть в айтем конфиг, а для айтем конфиг написать кастомный инспектор, который будет отображать нужные поля, в зависимости от типа предмета.

[freeExec]

Самый простой вариант, написать валидатор за 3 секунды, который укажет, где поле заполнено не верно.
В юнити даже подспорье есть - встроенный метод OnValidate
Конечно кастомный инспектор молодёжно и красиво, но уж больно хлопотно.

[K0TlK]

freeExec,

public enum ItemType : byte {
    Clothes,
    Weapon
}

[CreateAssetMenu(fileName = "Item Config", menuName = "Configs/Item")]
public class ItemConfig : ScriptableObject {
    public ItemType Type;
    public float    Damage;
    public float    SwingSpeed;
    public int      Armor;
    public float    Weight;
}

[CustomEditor(typeof(ItemConfig))]
public class ItemConfigEditor : Editor {

    public override void OnInspectorGUI() {
        var typeProperty = serializedObject.FindProperty("Type");

        EditorGUILayout.PropertyField(typeProperty);

        var type = (ItemType)typeProperty.enumValueIndex;

        switch(type) {
            case ItemType.Clothes : {
                DrawProperty("Armor");
                DrawProperty("Weight");
            } break;

            case ItemType.Weapon : {
                DrawProperty("Damage");
                DrawProperty("SwingSpeed");
            } break;
        }

        serializedObject.ApplyModifiedProperties();
    }

    private void DrawProperty(string name) {
        EditorGUILayout.PropertyField(serializedObject.FindProperty(name));
    }
}

5 минут. Крайне хлопотно.