Use preferred overloads of string.Split() (#23712)

* Use preferred overloads of string.Split

* Revert TrimEntries for netstandard2.0

Revert usage of StringSplit.TrimEntries for projects that target netstandard2.0.

Co-authored-by: Levi Broderick <[email protected]>

Author: martincostello

src/Hosting/Hosting/src/Internal/WebHostOptions.cs

@@ -76,21 +76,8 @@ public IEnumerable<string> GetFinalHostingStartupAssemblies()
 
         private IReadOnlyList<string> Split(string value)
         {
-            if (string.IsNullOrWhiteSpace(value))
-            {
-                return Array.Empty<string>();
-            }
-
-            var list = new List<string>();
-            foreach (var part in value.Split(';', StringSplitOptions.RemoveEmptyEntries))
-            {
-                var trimmedPart = part;
-                if (!string.IsNullOrEmpty(trimmedPart))
-                {
-                    list.Add(trimmedPart);
-                }
-            }
-            return list;
+            return value?.Split(';', StringSplitOptions.TrimEntries | StringSplitOptions.RemoveEmptyEntries)
+                ?? Array.Empty<string>();
         }
     }
 }

src/Http/Routing/src/ParameterPolicyActivator.cs

@@ -100,9 +100,7 @@ private static IParameterPolicy CreateParameterPolicy(IServiceProvider servicePr
             }
             else
             {
-                var arguments = !string.IsNullOrEmpty(argumentString)
-                    ? argumentString.Split(',').Select(argument => argument.Trim()).ToArray()
-                    : Array.Empty<string>();
+                var arguments = argumentString?.Split(',', StringSplitOptions.TrimEntries) ?? Array.Empty<string>();
 
                 // We want to find the constructors that match the number of passed in arguments
                 // We either want a single match, or a single best match. The best match is the one with the most

src/Mvc/Mvc.Core/src/BindAttribute.cs

@@ -71,15 +71,6 @@ public Func<ModelMetadata, bool> PropertyFilter
         }
 
         private static IEnumerable<string> SplitString(string original)
-        {
-            if (string.IsNullOrEmpty(original))
-            {
-                return Array.Empty<string>();
-            }
-
-            var split = original.Split(',').Select(piece => piece.Trim()).Where(piece => !string.IsNullOrEmpty(piece));
-
-            return split;
-        }
+            => original?.Split(',', StringSplitOptions.TrimEntries | StringSplitOptions.RemoveEmptyEntries) ?? Array.Empty<string>();
     }
 }

src/Mvc/Mvc.ViewFeatures/src/RemoteAttributeBase.cs

@@ -173,19 +173,7 @@ private static void MergeAttribute(IDictionary<string, string> attributes, strin
         }
 
         private static IEnumerable<string> SplitAndTrimPropertyNames(string original)
-        {
-            if (string.IsNullOrEmpty(original))
-            {
-                return Array.Empty<string>();
-            }
-
-            var split = original
-                .Split(',')
-                .Select(piece => piece.Trim())
-                .Where(trimmed => !string.IsNullOrEmpty(trimmed));
-
-            return split;
-        }
+            => original?.Split(',', StringSplitOptions.TrimEntries | StringSplitOptions.RemoveEmptyEntries) ?? Array.Empty<string>();
 
         private void CheckForLocalizer(ClientModelValidationContext context)
         {