diff options
| author | crobibero <cody@robibe.ro> | 2020-12-14 07:53:56 -0700 |
|---|---|---|
| committer | crobibero <cody@robibe.ro> | 2020-12-14 07:53:56 -0700 |
| commit | a515ecbada489bb79e8bcce6fd1aabe6d81ce235 (patch) | |
| tree | ade1b1f5020a4f006e18547b5535253dd44c3c82 | |
| parent | ee23d06154133ffc506dda3a8ef0e09d20c03c6c (diff) | |
Use range operator to get subarray
| -rw-r--r-- | Jellyfin.Data/Entities/User.cs | 22 |
1 files changed, 7 insertions, 15 deletions
diff --git a/Jellyfin.Data/Entities/User.cs b/Jellyfin.Data/Entities/User.cs index cc85a0b85..725531a7d 100644 --- a/Jellyfin.Data/Entities/User.cs +++ b/Jellyfin.Data/Entities/User.cs @@ -430,14 +430,17 @@ namespace Jellyfin.Data.Entities // Convert array of {string} to array of {T} var converter = TypeDescriptor.GetConverter(typeof(T)); var stringValues = val.Split(Delimiter); - var parsedValues = new object[stringValues.Length]; var convertedCount = 0; + var parsedValues = new T[stringValues.Length]; for (var i = 0; i < stringValues.Length; i++) { try { - parsedValues[i] = converter.ConvertFromString(stringValues[i].Trim()); - convertedCount++; + var parsedValue = converter.ConvertFromString(stringValues[i].Trim()); + if (parsedValue != null) + { + parsedValues.SetValue(parsedValue, convertedCount++); + } } catch (FormatException) { @@ -445,18 +448,7 @@ namespace Jellyfin.Data.Entities } } - var typedValues = new T[convertedCount]; - var typedValueIndex = 0; - for (var i = 0; i < parsedValues.Length; i++) - { - if (parsedValues[i] != null) - { - typedValues.SetValue(parsedValues[i], typedValueIndex); - typedValueIndex++; - } - } - - return typedValues; + return parsedValues[..convertedCount]; } /// <summary> |
