不允许在 Guard 中使用函数。需要替代实施的建议

Question

我正在尝试使用fermats method 创建一个素数分解器。

此行产生错误

find_factors(A, B, FactorThis) when is_a_square(B) == true ->

对本地/导入函数 is_a_square/1 的调用在警卫中是非法的

我看到的唯一可能的替代方案是在函数中使用某种 case 语句。我正在避免这种情况，因为它可能会搞砸尾递归。我是一个 Erlang 菜鸟。 还有哪些其他方法可以实现此功能？

get_int_part_of_sqrt(N) ->
    trunc(math:sqrt(N)).

is_a_square(N) ->
    get_int_part_of_sqrt(N) * get_int_part_of_sqrt(N) == N.

calculate_new_b(A, FactorThis) ->
    NewB = trunc(abs((A * A) - FactorThis)),
    io:format("Calculate_new_b A^2 ~w- FT ~w= NB ~w ~n",[A*A,FactorThis,NewB]),

find_factors(A, B, FactorThis) when is_a_square(B) == true ->
    io:format("find_factors true ~w ~w~n", [A, B]),
    {ok, A + get_int_part_of_sqrt(B), A - get_int_part_of_sqrt(B)};

find_factors(A, B, FactorThis) ->
    io:format("find_factors false ~w ~w~n", [A, B]),
    NewA = A + 1,
    NewB = calculate_new_b(NewA, FactorThis),
    find_factors(NewA, NewB, FactorThis).

Research1

Research2

已编辑。 调用calculate_new_b时的固定参数

添加了缺少的 get_int_part_of_sqrts。

Answer 1

Erlang 刻意限制了哪些函数可以调用警卫。 Here's 近期讨论了这样做的理由、优点和缺点。

解决它的唯一方法是使用case。你可以很容易地重写这段代码来使用case：

find_factors(A, B, FactorThis) ->
    case is_a_square(B) of
        true -> io:format("      find_factors true ~w ~w~n", [A, B]),
                {ok, A + B, A - B};

        false-> io:format("      find_factors false ~w ~w~n", [A, B]),
                NewA = A + 1,
                NewB = calculate_new_b(NewA, FactorThis),
                find_factors(NewA, NewB, FactorThis).

请注意，上面的代码仍然是正确的尾递归。

（我稍微修改了你的代码，去掉了我猜你不应该有的部分）

Answer 2

这是围绕该问题进行重构的另一种方法。

在调用者处添加所需的保护函数作为参数。这将它从一个可能有副作用的函数变成了没有副作用的真或假。然后直接模式匹配就可以了。

main() ->
    List2 = find_factors_2 (10, 5, 105, is_a_square(5)),
    io:format("method 2 ~w~n", [List2]).

find_factors_2(A, B, _FactorThis, true) ->
    Offset = get_int_part_of_sqrt(B),
    {A + Offset, A - Offset};

find_factors_2(A, _B, FactorThis, false) ->
    NewA = A + 1,
    NewB = calculate_new_b(NewA, FactorThis),
    find_factors_2(NewA, NewB, FactorThis, is_a_square(NewB)).