Search code examples
listrecursionprolog

Reaching end of list in prolog


I've been given the question:

Define a predicate ordered/1, which checks if a list of integers is correctly in ascending order. For example, the goal ordered([1,3,7,11]) should succeed, as should the goal ordered([1,3,3,7]), whereas the goal ordered([1,7,3,9]) should fail.

So far I have this:

ordered([]).    
ordered([N, M|Ns]):-
    append(M, Ns, Tail),
    ordered(Tail),
    N =< M.

But it fails on every list.

I have deduced that the reason it fails is because it reaches the end number in the list then tries to compare that number against an empty list. Obviously this fails because you can't compare an integer to an empty list. Even if you could and it, say, returned 0 for an empty list, it would still return false as the number would be greater than 0, not less than.

I can't find a solution... Any ideas? Thanks, Jon.


Edit

So, some slightly amended code:

ordered([]).
ordered([N]):-
    N >= 0.
ordered([N, M|Ns]):-
    append(M, Ns, Tail),
    ordered(Tail),
    N =< M.

This now works for ordered([1]), but bigger lists still don't run correctly.

Should I include something like ordered([N, M|Ns]) in the definition?


Solution

  • Well that, in the end, was rediculously easy to fix.

    Here is the correct code.

    ordered([]).
    
    ordered([N, M|Ns]):-
     append([M], Ns, Tail),
     ordered(Tail),
     N =< M.
    
    ordered([M]).
    

    ordered([M]). deals with the single-element list as described above.

    The real root of my problem was not including [] around the M in the append function.

    Whats the ettiquette regarding awarding the correct answer? You've both helped muchly.

    Jon