跟踪时可以快速看到问题:
?- trace, pairwise_min( [(25,25),(24,24),(23,23),(22,22),(21,21),(20,20),(19,19),(18,18),(17,17),(16,16),(15,15),(14,14),(13,13),(12,12),(11,11),(10,10),(9,9),(8,8),(7,7),(6,6),(5,5),(4,4),(3,3),(2,2),(1,1)], X ).
Call: (7) pairwise_min([ (25, 25), (24, 24), (23, 23), (22, 22), (21, 21), (20, 20), (19, 19), (..., ...)|...], _G737) ?
... snip ...
Call: (30) pairwise_min([ (2, 2), (1, 1)], (_G1065, _G1066)) ?
Call: (31) pairwise_min([ (1, 1)], (_G1068, _G1069)) ?
Exit: (31) pairwise_min([ (1, 1)], (1, 1)) ?
Call: (31) 1>2 ?
Fail: (31) 1>2 ?
Redo: (30) pairwise_min([ (2, 2), (1, 1)], (_G1065, _G1066)) ?
Call: (31) pairwise_min([ (1, 1)], (_G1065, _G1066)) ?
Exit: (31) pairwise_min([ (1, 1)], (1, 1)) ?
Call: (31) 1=<2 ?
Exit: (31) 1=<2 ?
Exit: (30) pairwise_min([ (2, 2), (1, 1)], (1, 1)) ?
Call: (30) 1>3 ?
Fail: (30) 1>3 ?
Redo: (29) pairwise_min([ (3, 3), (2, 2), (1, 1)], (_G1062, _G1063)) ?
Call: (30) pairwise_min([ (2, 2), (1, 1)], (_G1062, _G1063)) ?
Call: (31) pairwise_min([ (1, 1)], (_G1068, _G1069)) ?
Exit: (31) pairwise_min([ (1, 1)], (1, 1)) ?
Call: (31) 1>2 ?
Fail: (31) 1>2 ?
Redo: (30) pairwise_min([ (2, 2), (1, 1)], (_G1062, _G1063)) ?
Call: (31) pairwise_min([ (1, 1)], (_G1062, _G1063)) ?
Exit: (31) pairwise_min([ (1, 1)], (1, 1)) ?
Call: (31) 1=<2 ?
Exit: (31) 1=<2 ?
pairwise_min(T, ...)
基本上,问题是你在这两种情况下都会打电话,即使在第二种情况下它不会有所不同。尾部的成对最小值是尾部的成对最小值,无论当前元素如何检查它。
一个好的解决方案是通过使用显式条件来消除第二条规则,如下所示:
pairwise_min( [X], X ) :- !.
pairwise_min( [(A,B)|T], (RA,RB) ) :-
pairwise_min(T, (A1,B1)), !,
(B1 > B -> (RA = A, RB = B)
; (RA = A1, RB = B1)).
对我来说,可读性的一个重大障碍是,我并没有真正理解你试图用你的成对最小值实现什么样的排序。但无论如何,将来将其提取到自己的谓词中将是一个好主意。我对你的最低要求没有信心;不是真的@</2
会做你想做的事吗?如果是这种情况,您可以使用非常紧凑的尾递归折叠来做到这一点:
minimum([X|Xs], Min) :- minimum(Xs, X, Min).
minimum([], Min, Min).
minimum([X|Xs], MinSoFar, Min) :-
(X @< MinSoFar -> minimum(Xs, X, Min)
; minimum(Xs, MinSoFar, Min)).
如果不是这种情况,您可以编写自己的谓词min_pair/2
来比较两对,并使用它来代替@</2
并获得好处。或者从pairwise_min/2
上面的修改中调用它,看看尾调用优化的好处。
我认为您在改进此版本的性能方面会遇到一些麻烦。