[NotEmptyCollection] to validate empty list/array

[laurentopia]

probably by design but would be nice ™

[AliAlbarrak]

I suggest using a property like [ArraySize] but with no editor behavior. That will be useful in more scenarios

It should be defined like

    [AttributeUsage(AttributeTargets.Field)]
    public class CollectionSize : Attribute
    {
        public int Min = 0;
        public int Max = int.MaxValue;
    }

And used like

    [CollectionSize(Min = 1)] // Equivalent to [NotEmptyCollection]
    public List<Transform> endOfBarrels;
    
    [CollectionSize(Min = 1, Max = 10)] // Size must be between 1 and 10 inclusive
    public List<Transform> endOfBarrels;
    
    [CollectionSize(Max = 10)] // Can be empty but size has to be less than or equal 10
    public List<Transform> endOfBarrels;

[TylerTemp]

@AliAlbarrak This is a much better idea.

I think I'll put it into existing API, either ListDrawerSettings or ArraySize

[laurentopia]

ArraySize already exists, it hard sets the array size

[TylerTemp]

ArraySize already exists, it hard sets the array size

It can be like this:

• ArraySize(10) is the same as the old one, set the fixed size to 10
• ArraySize(5, 10) means min=5, and max=10

Then, ArraySize(min: 1) will do the trick: at least 1 element is expected.

This at least can keep the compatibility with the old API

[TylerTemp]

Hi,

this feature has been added to 3.3.8

1. ArraySize allows to set range
2. If you have ListDrawerSettings enabled, the Add and Remove buttons will be disabled/enabled accordingly if you also have ArraySize

For example:

• [ArraySize(3)] will make the array size fixed to 3
• [ArraySize(1, 5)] will make the array size range to 1-5 (both included)
• [ArraySize(min: 1)] will make the array size at least 1 (no max value specific). Useful to require a non-empty array.

[ArraySize(1, 3), ListDrawerSettings] public int[] oneToThreeWithSettings;

[laurentopia]

That's so nice! Thanks!