Skip to content

Conversation

@srutzky
Copy link

@srutzky srutzky commented Apr 24, 2020

  1. Added ExtractNumbersRegEx_Replace() method since Regex.Replace() does not require a collection or a loop, so this might be more efficient than using Regex.Split().

  2. Added test wrapper method to reference new ExtractNumbersRegEx_Replace() test method.

  3. Added test cases (both 40 and 4000 character versions), derived from the following T-SQL:

    DECLARE @TestD NVARCHAR(4000) = REPLICATE(N'a', 400) + REPLICATE(N'1', 400) +
        REPLICATE(N'b', 400) + REPLICATE(N'2', 300) + REPLICATE(N'c', 400) +
        REPLICATE(N'3', 300) + REPLICATE(N'd', 400) + REPLICATE(N'4', 300) +
        REPLICATE(N'e', 400) + REPLICATE(N'5', 300) + REPLICATE(N'f', 400);
    SELECT @TestD;

    and:

    DECLARE @TestE NVARCHAR(40) = REPLICATE(N'a', 4) + REPLICATE(N'1', 4) +
        REPLICATE(N'b', 4) + REPLICATE(N'2', 3) + REPLICATE(N'c', 4) +
        REPLICATE(N'3', 3) + REPLICATE(N'd', 4) + REPLICATE(N'4', 3) +
        REPLICATE(N'e', 4) + REPLICATE(N'5', 3) + REPLICATE(N'f', 4);
    SELECT @TestE;

NOTE: Unfortunately, I have not had, nor will have, time to test these changes. But everything looks correct 😸 .

Take care,
Solomon...
https://SqlQuantumLift.com/
https://SqlQuantumLeap.com/
https://SQLsharp.com/

srutzky added 2 commits April 23, 2020 23:57
Added ExtractNumbersRegEx_Replace() method since Regex.Replace() does not require a collection or a loop, so this _might_ be more efficient than using Regex.Split().
1) Added test wrapper method to reference new Regex.Replace() test method.

2) Added test cases (both 40 and 4000 character versions), derived from the following T-SQL:
    ```sql
    DECLARE @testD NVARCHAR(4000) = REPLICATE(N'a', 400) + REPLICATE(N'1', 400) + REPLICATE(N'b', 400) + REPLICATE(N'2', 300) + REPLICATE(N'c', 400) + REPLICATE(N'3', 300) + REPLICATE(N'd', 400) + REPLICATE(N'4', 300) + REPLICATE(N'e', 400) + REPLICATE(N'5', 300) + REPLICATE(N'f', 400);
    SELECT @testD;
    ```
    and:
    ```sql
    DECLARE @teste NVARCHAR(40) = REPLICATE(N'a', 4) + REPLICATE(N'1', 4) + REPLICATE(N'b', 4) + REPLICATE(N'2', 3) + REPLICATE(N'c', 4) + REPLICATE(N'3', 3) + REPLICATE(N'd', 4) + REPLICATE(N'4', 3) + REPLICATE(N'e', 4) + REPLICATE(N'5', 3) + REPLICATE(N'f', 4);
    SELECT @teste;
    ```
@srutzky srutzky changed the title srutzky-RegExReplace_and_NewTestCase Add RegEx.Replace test and new test case with more variation Apr 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant