fix(string): OnlyNumbers bug

NetDevPack · Aug 13, 2021 · e52746b · e52746b
1 parent 5e4873b
commit e52746b
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/NetDevPack/Utilities/StringUtils.cs b/src/NetDevPack/Utilities/StringUtils.cs
@@ -290,7 +290,7 @@ public static string OnlyNumbers(this string str)
 
                 onlyNumbers[lastIndex++] = c;
             }
-            Array.Resize(ref onlyNumbers, onlyNumbers.Length);
+            Array.Resize(ref onlyNumbers, lastIndex);
             return new string(onlyNumbers);
         }
     }

diff --git a/tests/NetDevPack.Tests/Utilities/StringTest.cs b/tests/NetDevPack.Tests/Utilities/StringTest.cs
@@ -27,5 +27,14 @@ public void Should_Urlize(string content, string expected)
         {
             content.Urlize().Should().Be(expected);
         }
+
+        [Theory]
+        [InlineData("86456-221", "86456221")]
+        [InlineData("864aoeuaoeu56-221", "86456221")]
+        [InlineData("#@eee221oeu#@ aoeu a,34", "22134")]
+        public void Should_Only_Get_Numbers(string content, string expected)
+        {
+            content.OnlyNumbers().Should().Be(expected);
+        }
     }
 }