Merge pull request Flow-Launcher#3859 from dcog989/Calculator-accepts-flexible-separator

Smart thousands and decimals for Calculator

Author: VictoriousRaptor

Plugins/Flow.Launcher.Plugin.Calculator/Languages/en.xaml

@@ -4,7 +4,9 @@
     xmlns:system="clr-namespace:System;assembly=mscorlib">
 
     <system:String x:Key="flowlauncher_plugin_caculator_plugin_name">Calculator</system:String>
-    <system:String x:Key="flowlauncher_plugin_caculator_plugin_description">Allows to do mathematical calculations.(Try 5*3-2 in Flow Launcher)</system:String>
+    <system:String x:Key="flowlauncher_plugin_caculator_plugin_description">Perform mathematical calculations (including hexadecimal values). Use ',' or '.' as thousand separator or decimal place.</system:String>
+    <system:String x:Key="flowlauncher_plugin_calculator_not_a_number">Not a number (NaN)</system:String>
+    <system:String x:Key="flowlauncher_plugin_calculator_expression_not_complete">Expression wrong or incomplete (Did you forget some parentheses?)</system:String>
     <system:String x:Key="flowlauncher_plugin_calculator_copy_number_to_clipboard">Copy this number to the clipboard</system:String>
     <system:String x:Key="flowlauncher_plugin_calculator_output_decimal_separator">Decimal separator</system:String>
     <system:String x:Key="flowlauncher_plugin_calculator_output_decimal_separator_help">The decimal separator to be used in the output.</system:String>

Plugins/Flow.Launcher.Plugin.Calculator/Main.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Collections.Generic;
 using System.Globalization;
+using System.Linq;
 using System.Runtime.InteropServices;
 using System.Text.RegularExpressions;
 using System.Windows.Controls;
@@ -14,6 +15,9 @@ public class Main : IPlugin, IPluginI18n, ISettingProvider
     {
         private static readonly Regex RegValidExpressChar = MainRegexHelper.GetRegValidExpressChar();
         private static readonly Regex RegBrackets = MainRegexHelper.GetRegBrackets();
+        private static readonly Regex ThousandGroupRegex = MainRegexHelper.GetThousandGroupRegex();
+        private static readonly Regex NumberRegex = MainRegexHelper.GetNumberRegex();
+
         private static Engine MagesEngine;
         private const string Comma = ",";
         private const string Dot = ".";
@@ -23,6 +27,16 @@ public class Main : IPlugin, IPluginI18n, ISettingProvider
         private Settings _settings;
         private SettingsViewModel _viewModel;
 
+        /// <summary>
+        /// Holds the formatting information for a single query.
+        /// This is used to ensure thread safety by keeping query state local.
+        /// </summary>
+        private class ParsingContext
+        {
+            public string InputDecimalSeparator { get; set; }
+            public bool InputUsesGroupSeparators { get; set; }
+        }
+
         public void Init(PluginInitContext context)
         {
             Context = context;
@@ -45,26 +59,17 @@ public List<Result> Query(Query query)
                 return new List<Result>();
             }
 
+            var context = new ParsingContext();
+
             try
             {
-                string expression;
-
-                switch (_settings.DecimalSeparator)
-                {
-                    case DecimalSeparator.Comma:
-                    case DecimalSeparator.UseSystemLocale when CultureInfo.CurrentCulture.NumberFormat.NumberDecimalSeparator == ",":
-                        expression = query.Search.Replace(",", ".");
-                        break;
-                    default:
-                        expression = query.Search;
-                        break;
-                }
+                var expression = NumberRegex.Replace(query.Search, m => NormalizeNumber(m.Value, context));
 
                 var result = MagesEngine.Interpret(expression);
                 if (result?.ToString() != "NaN" && result is not Function && !string.IsNullOrEmpty(result?.ToString()))
                 {
                     decimal roundedResult = Math.Round(Convert.ToDecimal(result), _settings.MaxDecimalPlaces, MidpointRounding.AwayFromZero);
-                    string newResult = ChangeDecimalSeparator(roundedResult, GetDecimalSeparator());
+                    string newResult = FormatResult(roundedResult, context);
 
                     return new List<Result>
                     {
@@ -100,43 +105,153 @@ public List<Result> Query(Query query)
             return new List<Result>();
         }
 
-        private bool CanCalculate(Query query)
+        /// <summary>
+        /// Parses a string representation of a number, detecting its format. It uses structural analysis
+        /// and falls back to system culture for truly ambiguous cases (e.g., "1,234").
+        /// It populates the provided ParsingContext with the detected format for later use.
+        /// </summary>
+        /// <returns>A normalized number string with '.' as the decimal separator for the Mages engine.</returns>
+        private string NormalizeNumber(string numberStr, ParsingContext context)
         {
-            // Don't execute when user only input "e" or "i" keyword
-            if (query.Search.Length < 2)
+            var systemGroupSep = CultureInfo.CurrentCulture.NumberFormat.NumberGroupSeparator;
+            int dotCount = numberStr.Count(f => f == '.');
+            int commaCount = numberStr.Count(f => f == ',');
+
+            // Case 1: Unambiguous mixed separators (e.g., "1.234,56")
+            if (dotCount > 0 && commaCount > 0)
             {
-                return false;
+                context.InputUsesGroupSeparators = true;
+                if (numberStr.LastIndexOf('.') > numberStr.LastIndexOf(','))
+                {
+                    context.InputDecimalSeparator = Dot;
+                    return numberStr.Replace(Comma, string.Empty);
+                }
+                else
+                {
+                    context.InputDecimalSeparator = Comma;
+                    return numberStr.Replace(Dot, string.Empty).Replace(Comma, Dot);
+                }
             }
 
-            if (!RegValidExpressChar.IsMatch(query.Search))
+            // Case 2: Only dots
+            if (dotCount > 0)
             {
-                return false;
+                if (dotCount > 1)
+                {
+                    context.InputUsesGroupSeparators = true;
+                    return numberStr.Replace(Dot, string.Empty);
+                }
+                // A number is ambiguous if it has a single Dot in the thousands position,
+                // and does not start with a "0." or "."
+                bool isAmbiguous = numberStr.Length - numberStr.LastIndexOf('.') == 4
+                                   && !numberStr.StartsWith("0.")
+                                   && !numberStr.StartsWith(".");
+                if (isAmbiguous)
+                {
+                    if (systemGroupSep == Dot)
+                    {
+                        context.InputUsesGroupSeparators = true;
+                        return numberStr.Replace(Dot, string.Empty);
+                    }
+                    else
+                    {
+                        context.InputDecimalSeparator = Dot;
+                        return numberStr;
+                    }
+                }
+                else // Unambiguous decimal (e.g., "12.34" or "0.123" or ".123")
+                {
+                    context.InputDecimalSeparator = Dot;
+                    return numberStr;
+                }
             }
 
-            if (!IsBracketComplete(query.Search))
+            // Case 3: Only commas
+            if (commaCount > 0)
             {
-                return false;
+                if (commaCount > 1)
+                {
+                    context.InputUsesGroupSeparators = true;
+                    return numberStr.Replace(Comma, string.Empty);
+                }
+                // A number is ambiguous if it has a single Comma in the thousands position,
+                // and does not start with a "0," or ","
+                bool isAmbiguous = numberStr.Length - numberStr.LastIndexOf(',') == 4
+                                   && !numberStr.StartsWith("0,")
+                                   && !numberStr.StartsWith(",");
+                if (isAmbiguous)
+                {
+                    if (systemGroupSep == Comma)
+                    {
+                        context.InputUsesGroupSeparators = true;
+                        return numberStr.Replace(Comma, string.Empty);
+                    }
+                    else
+                    {
+                        context.InputDecimalSeparator = Comma;
+                        return numberStr.Replace(Comma, Dot);
+                    }
+                }
+                else // Unambiguous decimal (e.g., "12,34" or "0,123" or ",123")
+                {
+                    context.InputDecimalSeparator = Comma;
+                    return numberStr.Replace(Comma, Dot);
+                }
             }
 
-            if ((query.Search.Contains(Dot) && GetDecimalSeparator() != Dot) ||
-                (query.Search.Contains(Comma) && GetDecimalSeparator() != Comma))
-                return false;
+            // Case 4: No separators
+            return numberStr;
+        }
 
-            return true;
+        private string FormatResult(decimal roundedResult, ParsingContext context)
+        {
+            string decimalSeparator = context.InputDecimalSeparator ?? GetDecimalSeparator();
+            string groupSeparator = GetGroupSeparator(decimalSeparator);
+
+            string resultStr = roundedResult.ToString(CultureInfo.InvariantCulture);
+
+            string[] parts = resultStr.Split('.');
+            string integerPart = parts[0];
+            string fractionalPart = parts.Length > 1 ? parts[1] : string.Empty;
+
+            if (context.InputUsesGroupSeparators && integerPart.Length > 3)
+            {
+                integerPart = ThousandGroupRegex.Replace(integerPart, groupSeparator);
+            }
+
+            if (!string.IsNullOrEmpty(fractionalPart))
+            {
+                return integerPart + decimalSeparator + fractionalPart;
+            }
+
+            return integerPart;
         }
 
-        private static string ChangeDecimalSeparator(decimal value, string newDecimalSeparator)
+        private string GetGroupSeparator(string decimalSeparator)
         {
-            if (string.IsNullOrEmpty(newDecimalSeparator))
+            // This logic is now independent of the system's group separator
+            // to ensure consistent output for unit testing.
+            return decimalSeparator == Dot ? Comma : Dot;
+        }
+
+        private bool CanCalculate(Query query)
+        {
+            if (query.Search.Length < 2)
             {
-                return value.ToString();
+                return false;
             }
 
-            var numberFormatInfo = new NumberFormatInfo
+            if (!RegValidExpressChar.IsMatch(query.Search))
             {
-                NumberDecimalSeparator = newDecimalSeparator
-            };
-            return value.ToString(numberFormatInfo);
+                return false;
+            }
+
+            if (!IsBracketComplete(query.Search))
+            {
+                return false;
+            }
+
+            return true;
         }
 
         private string GetDecimalSeparator()

Plugins/Flow.Launcher.Plugin.Calculator/MainRegexHelper.cs

@@ -19,4 +19,10 @@ internal static partial class MainRegexHelper
                     @"[\+\%\-\*\/\^\., ""]|[\(\)\|\!\[\]]" +
                     @")+$", RegexOptions.Compiled)]
     public static partial Regex GetRegValidExpressChar();
+
+    [GeneratedRegex(@"[\d\.,]+", RegexOptions.Compiled)]
+    public static partial Regex GetNumberRegex();
+
+    [GeneratedRegex(@"\B(?=(\d{3})+(?!\d))", RegexOptions.Compiled)]
+    public static partial Regex GetThousandGroupRegex();
 }

Plugins/Flow.Launcher.Plugin.Calculator/NumberTranslator.cs

@@ -2,8 +2,8 @@
   "ID": "CEA0FDFC6D3B4085823D60DC76F28855",
   "ActionKeyword": "*",
   "Name": "Calculator",
-  "Description": "Perform mathematical calculations (including hexadecimal values)",
-  "Author": "cxfksword",
+  "Description": "Perform mathematical calculations (including hexadecimal values). Use ',' or '.' as thousand separator or decimal place.",
+  "Author": "cxfksword, dcog989",
   "Version": "1.0.0",
   "Language": "csharp",
   "Website": "https://github.com/Flow-Launcher/Flow.Launcher",