Improve error handling in FindRoot, FindMaximum, FindMinimum

Author: axkr

symja_android_library/matheclipse-core/src/main/java/org/matheclipse/core/reflection/system/FindMaximum.java

@@ -124,9 +124,11 @@ public IExpr evaluate(IAST ast, int argSize, IExpr[] options, EvalEngine engine,
     } catch (MathIllegalStateException mise) {
       if (mise.getSpecifier().equals(LocalizedCoreFormats.MAX_COUNT_EXCEEDED)) {
         Object[] parts = mise.getParts();
-        // Failed to converge to the requested accuracy or precision within `1` iterations.
-        return Errors.printMessage(ast.topHead(), "cvmit", F.list(F.$str(parts[0].toString())),
-            engine);
+        if (parts != null && parts.length >= 1) {
+          // Failed to converge to the requested accuracy or precision within `1` iterations.
+          return Errors.printMessage(ast.topHead(), "cvmit", F.list(F.$str(parts[0].toString())),
+              engine);
+        }
       }
       // `1`.
       return Errors.printMessage(ast.topHead(), "error", F.list(F.$str(mise.getMessage())), engine);

symja_android_library/matheclipse-core/src/main/java/org/matheclipse/core/reflection/system/FindMinimum.java

@@ -174,9 +174,11 @@ public IExpr evaluate(IAST ast, int argSize, IExpr[] options, EvalEngine engine,
     } catch (MathIllegalStateException mise) {
       if (mise.getSpecifier().equals(LocalizedCoreFormats.MAX_COUNT_EXCEEDED)) {
         Object[] parts = mise.getParts();
-        // Failed to converge to the requested accuracy or precision within `1` iterations.
-        return Errors.printMessage(ast.topHead(), "cvmit", F.list(F.$str(parts[0].toString())),
-            engine);
+        if (parts != null && parts.length >= 1) {
+          // Failed to converge to the requested accuracy or precision within `1` iterations.
+          return Errors.printMessage(ast.topHead(), "cvmit", F.list(F.$str(parts[0].toString())),
+              engine);
+        }
       }
       // `1`.
       return Errors.printMessage(ast.topHead(), "error", F.list(F.$str(mise.getMessage())), engine);

symja_android_library/matheclipse-core/src/main/java/org/matheclipse/core/reflection/system/FindRoot.java

@@ -441,13 +441,18 @@ private static IExpr univariateFindRoot(IExpr listOfEquations, IAST varValuePair
           IExpr result = optimizeSupplier.get();
           // engine.evalBlock(optimizeSupplier, list);
           return F.list(F.Rule(list.arg1(), result));
-        } catch (MathIllegalStateException miae) {
-          if (miae.getSpecifier() == LocalizedCoreFormats.CONVERGENCE_FAILED) {
-            // Failed to converge to the requested accuracy or precision within `1` iterations.
-            return Errors.printMessage(S.FindRoot, "cvmit", F.list(F.ZZ(maxIterations)), engine);
+        } catch (MathIllegalStateException mise) {
+          if (mise.getSpecifier() == LocalizedCoreFormats.CONVERGENCE_FAILED) {
+            Object[] parts = mise.getParts();
+            if (parts != null && parts.length >= 1) {
+              // Failed to converge to the requested accuracy or precision within `1` iterations.
+              return Errors.printMessage(S.FindRoot, "cvmit", F.list(F.$str(parts[0].toString())),
+                  engine);
+            }
+
           }
           // `1`.
-          return Errors.printMessage(S.FindRoot, "error", F.list(F.$str(miae.getMessage())),
+          return Errors.printMessage(S.FindRoot, "error", F.list(F.$str(mise.getMessage())),
               engine);
         } catch (MathRuntimeException mre) {
           if (mre.getSpecifier() == LocalizedCoreFormats.NOT_BRACKETING_INTERVAL