separate acronym count and check

Author: jjw24

Flow.Launcher.Infrastructure/StringMatcher.cs

@@ -72,7 +72,7 @@ public MatchResult FuzzyMatch(string query, string stringToCompare, MatchOption
             var fullStringToCompareWithoutCase = opt.IgnoreCase ? stringToCompare.ToLower() : stringToCompare;
             var queryWithoutCase = opt.IgnoreCase ? query.ToLower() : query;
 
-            var querySubstrings = queryWithoutCase.Split(new[] {' '}, StringSplitOptions.RemoveEmptyEntries);
+            var querySubstrings = queryWithoutCase.Split(new[] { ' ' }, StringSplitOptions.RemoveEmptyEntries);
             int currentQuerySubstringIndex = 0;
             var currentQuerySubstring = querySubstrings[currentQuerySubstringIndex];
             var currentQuerySubstringCharacterIndex = 0;
@@ -92,7 +92,7 @@ public MatchResult FuzzyMatch(string query, string stringToCompare, MatchOption
                 // If acronyms matching successfully finished, this gets the remaining not matched acronyms for score calculation
                 if (currentAcronymQueryIndex >= query.Length && acronymsMatched == query.Length)
                 {
-                    if (IsAcronym(stringToCompare, compareStringIndex))
+                    if (IsAcronymCount(stringToCompare, compareStringIndex))
                         acronymsTotalCount++;
                     continue;
                 }
@@ -117,9 +117,11 @@ public MatchResult FuzzyMatch(string query, string stringToCompare, MatchOption
 
                         currentAcronymQueryIndex++;
                     }
+                }
 
+                if (IsAcronymCount(stringToCompare, compareStringIndex))
                     acronymsTotalCount++;
-                }
+
                 // Acronym end
 
                 if (allQuerySubstringsMatched || fullStringToCompareWithoutCase[compareStringIndex] !=
@@ -214,14 +216,46 @@ public MatchResult FuzzyMatch(string query, string stringToCompare, MatchOption
         }
 
         private bool IsAcronym(string stringToCompare, int compareStringIndex)
+        {
+            if (IsAcronymChar(stringToCompare, compareStringIndex) || IsAcronymNumber(stringToCompare, compareStringIndex))
+                return true;
+
+            return false;
+        }
+
+        // When counting acronyms, treat a set of numbers as one acronym ie. Visual 2019 as 2 acronyms instead of 5
+        private bool IsAcronymCount(string stringToCompare, int compareStringIndex)
+        {
+            if (IsAcronymChar(stringToCompare, compareStringIndex))
+                return true;
+
+            if (char.IsNumber(stringToCompare[compareStringIndex]) || char.IsDigit(stringToCompare[compareStringIndex]))
+            {
+                return compareStringIndex switch
+                {
+                    int i when i == 0 => true,
+                    int i when char.IsWhiteSpace(stringToCompare[i - 1]) => true,
+                    _ => false,
+                };
+            }
+
+            return false;
+        }
+
+        private bool IsAcronymChar(string stringToCompare, int compareStringIndex)
         {
             if (char.IsUpper(stringToCompare[compareStringIndex]) ||
-                    char.IsNumber(stringToCompare[compareStringIndex]) ||
-                    char.IsDigit(stringToCompare[compareStringIndex]) ||
-                    compareStringIndex == 0) //0 index means char is the start of the compare string, which is an acronym
+                    compareStringIndex == 0 || //0 index means char is the start of the compare string, which is an acronym
+                    compareStringIndex != 0 && char.IsWhiteSpace(stringToCompare[compareStringIndex - 1]))
                 return true;
 
-            if (compareStringIndex != 0 && char.IsWhiteSpace(stringToCompare[compareStringIndex - 1]))
+            return false;
+        }
+
+        private bool IsAcronymNumber(string stringToCompare, int compareStringIndex)
+        {
+            if (char.IsNumber(stringToCompare[compareStringIndex]) ||
+                char.IsDigit(stringToCompare[compareStringIndex]))
                 return true;
 
             return false;

Flow.Launcher.Test/FuzzyMatcherTest.cs

@@ -163,7 +163,11 @@ public void WhenGivenQueryString_ThenShouldReturn_TheDesiredScoring(
         [TestCase("vs", VisualStudioCode, SearchPrecisionScore.Regular, true)]
         [TestCase("vc", VisualStudioCode, SearchPrecisionScore.Regular, true)]
         [TestCase("vts", VisualStudioCode, SearchPrecisionScore.Regular, false)]
+        [TestCase("vcs", VisualStudioCode, SearchPrecisionScore.Regular, false)]
         [TestCase("wt", "Windows Terminal From Microsoft Store", SearchPrecisionScore.Regular, false)]
+        [TestCase("vsp", "Visual Studio 2019 Preview", SearchPrecisionScore.Regular, true)]
+        [TestCase("vsp", "2019 Visual Studio Preview", SearchPrecisionScore.Regular, true)]
+        [TestCase("2019p", "Visual Studio 2019 Preview", SearchPrecisionScore.Regular, true)]
         public void WhenGivenDesiredPrecision_ThenShouldReturn_AllResultsGreaterOrEqual(
             string queryString,
             string compareString,