Add Join<T>(char, XxxCollection) overloads

Author: xin9le

src/ZString.Unity/Assets/Scripts/ZString/ZString.cs

@@ -55,6 +55,29 @@ public static string Join<T>(char separator, params T[] values)
             }
         }
 
+        /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
+        public static string Join<T>(char separator, List<T> values)
+        {
+            var sb = new Utf16ValueStringBuilder(true);
+            try
+            {
+                var count = values.Count;
+                for (int i = 0; i < count; i++)
+                {
+                    if (i != 0)
+                    {
+                        sb.Append(separator);
+                    }
+                    sb.Append(values[i]);
+                }
+                return sb.ToString();
+            }
+            finally
+            {
+                sb.Dispose();
+            }
+        }
+
         /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
         public static string Join<T>(char separator, ReadOnlySpan<T> values)
         {
@@ -105,6 +128,26 @@ public static string Join<T>(char separator, IEnumerable<T> values)
             }
         }
 
+        public static string Join<T>(char separator, ICollection<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IList<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IReadOnlyList<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IReadOnlyCollection<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
         /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
         public static string Join<T>(string separator, params T[] values)
         {

src/ZString/ZString.cs

@@ -55,6 +55,29 @@ public static string Join<T>(char separator, params T[] values)
             }
         }
 
+        /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
+        public static string Join<T>(char separator, List<T> values)
+        {
+            var sb = new Utf16ValueStringBuilder(true);
+            try
+            {
+                var count = values.Count;
+                for (int i = 0; i < count; i++)
+                {
+                    if (i != 0)
+                    {
+                        sb.Append(separator);
+                    }
+                    sb.Append(values[i]);
+                }
+                return sb.ToString();
+            }
+            finally
+            {
+                sb.Dispose();
+            }
+        }
+
         /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
         public static string Join<T>(char separator, ReadOnlySpan<T> values)
         {
@@ -105,6 +128,26 @@ public static string Join<T>(char separator, IEnumerable<T> values)
             }
         }
 
+        public static string Join<T>(char separator, ICollection<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IList<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IReadOnlyList<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
+        public static string Join<T>(char separator, IReadOnlyCollection<T> values)
+        {
+            return Join(separator, values.AsEnumerable());
+        }
+
         /// <summary>Concatenates the elements of an array, using the specified seperator between each element.</summary>
         public static string Join<T>(string separator, params T[] values)
         {

tests/ZString.Tests/JoinTest.cs

@@ -63,6 +63,35 @@ public void JoinOverloads2()
             ZString.Join("_,_", (IReadOnlyCollection<int>)new[] { 1, 2, 3 }).Should().Be(string.Join("_,_", new[] { 1, 2, 3 }));
         }
 
+        [Fact]
+        public void JoinOverloads3()
+        {
+            ZString.Join(',', new string[] { }.ToList()).Should().Be(string.Join(',', new string[0]));
+            ZString.Join(',', new[] { 1 }.ToList()).Should().Be(string.Join(',', new[] { 1 }));
+            ZString.Join(',', new[] { 1, 2 }.ToList()).Should().Be(string.Join(',', new[] { 1, 2 }));
+            ZString.Join(',', new[] { 1, 2, 3 }.ToList()).Should().Be(string.Join(',', new[] { 1, 2, 3 }));
+
+            ZString.Join(',', (IList<int>)new int[] { }).Should().Be(string.Join(',', new string[0]));
+            ZString.Join(',', (IList<int>)new[] { 1 }).Should().Be(string.Join(',', new[] { 1 }));
+            ZString.Join(',', (IList<int>)new[] { 1, 2 }).Should().Be(string.Join(',', new[] { 1, 2 }));
+            ZString.Join(',', (IList<int>)new[] { 1, 2, 3 }).Should().Be(string.Join(',', new[] { 1, 2, 3 }));
+
+            ZString.Join(',', (IReadOnlyList<int>)new int[] { }).Should().Be(string.Join(',', new string[0]));
+            ZString.Join(',', (IReadOnlyList<int>)new[] { 1 }).Should().Be(string.Join(',', new[] { 1 }));
+            ZString.Join(',', (IReadOnlyList<int>)new[] { 1, 2 }).Should().Be(string.Join(',', new[] { 1, 2 }));
+            ZString.Join(',', (IReadOnlyList<int>)new[] { 1, 2, 3 }).Should().Be(string.Join(',', new[] { 1, 2, 3 }));
+
+            ZString.Join(',', (ICollection<int>)new int[] { }).Should().Be(string.Join(',', new string[0]));
+            ZString.Join(',', (ICollection<int>)new[] { 1 }).Should().Be(string.Join(',', new[] { 1 }));
+            ZString.Join(',', (ICollection<int>)new[] { 1, 2 }).Should().Be(string.Join(',', new[] { 1, 2 }));
+            ZString.Join(',', (ICollection<int>)new[] { 1, 2, 3 }).Should().Be(string.Join(',', new[] { 1, 2, 3 }));
+
+            ZString.Join(',', (IReadOnlyCollection<int>)new int[] { }).Should().Be(string.Join(',', new string[0]));
+            ZString.Join(',', (IReadOnlyCollection<int>)new[] { 1 }).Should().Be(string.Join(',', new[] { 1 }));
+            ZString.Join(',', (IReadOnlyCollection<int>)new[] { 1, 2 }).Should().Be(string.Join(',', new[] { 1, 2 }));
+            ZString.Join(',', (IReadOnlyCollection<int>)new[] { 1, 2, 3 }).Should().Be(string.Join(',', new[] { 1, 2, 3 }));
+        }
+
         [Fact]
         public void CooncatNullTest()
         {