You have been using cut and an unsafe form of negation. Both have to be used with extreme care. An immediate fix would be to guard your program against uses it is not designed for:
unique(X, Xs) :-
when_si(ground(X+Xs), your_unique(X, Xs)).
This uses when_si/2
(formerly called iwhen/2
) which is similar to when/2
except that it does not delay:
:- meta_predicate(when_si(+, 0)).
when_si(Cond, G_0) :-
when(Cond, ( Called = true, G_0 ) ),
( var(Called) -> throw(error(instantiation_error,_)) ; true ).
Above works for systems providing when/2
. Below is for any ISO conforming system:
when_si(Cond, G_0) :-
( when_condition(Cond)
-> ( Cond -> G_0 ; throw(error(instantiation_error,_)) )
; throw(error(domain_error(when_condition, Cond),_))
).
when_condition(C) :-
var(C),
!,
throw(error(instantiation_error,_)).
when_condition(ground(_)).
when_condition(nonvar(_)).
when_condition(?=(_, _)).
when_condition(( A, B )) :-
when_condition(A),
when_condition(B).
when_condition(( A ; B )) :-
when_condition(A),
when_condition(B).
On the other hand, it gets quite frustrating receiving instantiation errors all the time instead of real answers. So, let's make your program really pure!
Your second rule
unique(X, [_|Es]) :-
unique(X, Es).
reads declaratively, right-to-left (that :-
is an ←
)
Provided X
is a unique element of the list Es
, then X
is a unique element of the list [_|Es]
.
in other words: Whenever I know that X
is unique in Es
, it will be also unique with any further element in Es
. That conclusion is not true, consider to extend the list by X
! You need some extra condition. Also, your first rule needs to be reformulated. This uses non_member/2
:
unique(X, [X|Es]) :-
non_member(X, Es).
unique(X, [E|Es]) :-
dif(X, E),
unique(X, Es).
And here is another way using tfilter/3
:
unique(X, Es) :-
tfilter(=(X), Es, [_]).
The most efficient is probably the following which uses if_/3
of library(reif)
:
unique(X, [E|Es]) :-
if_(X = E, non_member(E, Es), unique(X, Es) ).