Improve method matching logic in InlineQueryHandler.cs

Introduce a list to store potential matching methods and correct the condition to skip non-matching methods. Remove previous parameter conversion logic and add all matching methods to the list. After the loop, handle cases where exactly one or multiple methods match by converting parameters or throwing an AmbiguousMatchException, respectively. This ensures precise method selection or appropriate error handling.

Author: bvdcode

Sources/TelegramBot/Handlers/InlineQueryHandler.cs

@@ -26,6 +26,7 @@ public InlineQueryHandler(IReadOnlyCollection<MethodInfo> controllerMethods, Upd
             }
             var inlineQuery = update.CallbackQuery;
             string command = inlineQuery.Data!;
+            List<MethodInfo> methods = new List<MethodInfo>();
             foreach (var method in controllerMethods)
             {
                 var attributes = method.GetCustomAttributes(typeof(InlineCommandAttribute), false);
@@ -53,20 +54,26 @@ public InlineQueryHandler(IReadOnlyCollection<MethodInfo> controllerMethods, Upd
                                 _args.Add(incomingCommandParts[i]);
                             }
                         }
-                        if (match)
+                        if (!match)
                         {
-                            var args = _args.ToArray();
-                            bool converted = ObjectHelpers.TryConvertParameters(method, args);
-                            if (converted)
-                            {
-                                _args.Clear();
-                                _args.AddRange(args);
-                                return method;
-                            }
+                            continue;
                         }
+                        methods.Add(method);
                     }
                 }
             }
+            if (methods.Count == 1)
+            {
+                var args = _args.ToArray();
+                ObjectHelpers.TryConvertParameters(methods[0], args);
+                _args.Clear();
+                _args.AddRange(args);
+                return methods[0];
+            }
+            else if (methods.Count > 1)
+            {
+                throw new AmbiguousMatchException("Multiple methods found with the same command and arguments: " + command);
+            }
             return null;
         }