Hints for dup/2 added.

2 files changed, 41 insertions, 50 deletions

diff --git a/prolog/problems/lists/dup_2/common.py b/prolog/problems/lists/dup_2/common.py
index a27eb4b..0ed3914 100644
--- a/prolog/problems/lists/dup_2/common.py
+++ b/prolog/problems/lists/dup_2/common.py
@@ -24,11 +24,10 @@ hint_type = {
     'predicate_always_false': Hint('predicate_always_false'),
     'base_case': Hint('base_case'),
     'timeout': Hint('timeout'),
-    'lost_heads': Hint('lost_heads'),
     'recursive_case': Hint('recursive_case'),
-    'leading_heads_all_x': Hint('leading_heads_all_x'),
-    'ins_results_in_empty_list': Hint('ins_results_in_empty_list'),
-    'ins_results_in_arbitrary_result': Hint('ins_results_in_arbitrary_result'),
+    'forcing_result_onto_recursion': Hint('forcing_result_onto_recursion'),
+    'base_case_arbitrary': Hint('base_case_arbitrary'),
+    'base_case_missing_[]': Hint('base_case_missing_[]'),
 }
 
 test_cases = [
@@ -78,35 +77,34 @@ def hint(code, aux_code):
         # recursion is getting bigger and bigger
 
 
-        # inserting an element succeeds with an emtpy list result
-        if prolog.engine.ask_truthTO(engine_id, 'insert(_, _, [])'):
-            return [{'id': 'ins_results_in_empty_list'}]
+        # forcing result onto recursion; case of dup(T, [H,H|L]) or dup(T, [H,H|T])
+        if not prolog.engine.ask_truthTO(engine_id, 'dup([a,b,c,d,e], _)') and \
+           (prolog.engine.ask_truth(engine_id,
+           'asserta( dup([b], [a,a,yowza]) ), dup([a,b], [yowza]), retract( dup([b], [a,a,yowza]) )') or \
+           prolog.engine.ask_truth(engine_id,
+           'asserta( dup([b], [a,a,b]) ), dup([a,b], [b]), retract( dup([b], [a,a,b]) )')):
+            return [{'id': 'forcing_result_onto_recursion'}]
 
-        # inserting an element succeeds with an arbitrary result
-        if prolog.engine.ask_truth(engine_id, 'insert(_, _, L), var(L)'):
-            return [{'id': 'ins_results_in_arbitrary_result'}]
+
+        # base case succeeds with arbitrary result
+        if prolog.engine.ask_truthTO(engine_id, 'dup([], L), var(L)'):
+            return [{'id': 'base_case_arbitrary'}]
+
+        # base case does not work for empty list
+        if prolog.engine.ask_truthTO(engine_id, 'dup([q], [q,q]), \+ dup([], [])'):
+            return [{'id': 'base_case_missing_[]'}]
 
         # missing/failed base case
-        if not prolog.engine.ask_one(engine_id, 'insert(qQ, [qa,qb,qc], [qQ,qa,qb,qc])'):
+        if not prolog.engine.ask_one(engine_id, 'dup([], [])'):
             return [{'id': 'base_case'}]
 
         # target predicate seems to always be false
-        if not prolog.engine.ask_truth(engine_id, 'insert(_, [_,_,_,_,_,_], _)'):
+        if not prolog.engine.ask_truth(engine_id, 'dup([_,_,_], _)'):
             return [{'id': 'predicate_always_false'}]
 
-        # forgot to put list head(s) back in the list when deleting an element
-        if prolog.engine.ask_truth(engine_id, 'insert(q, [a,b,c,d,x,y,z], [q,x,y,z])'):
-            return [{'id': 'lost_heads'}]
-
-        # used [X|T] instead of more general [H|T] in recursive case
-        if prolog.engine.ask_truth(engine_id, \
-                'findall(NL, insert(q, [a,b,c], NL), [[q,a,b,c], [q,q,b,c], [q,q,q,c], [q,q,q,q]]).') and \
-           not prolog.engine.ask_truth(engine_id, 'insert(q, [a,b,c,x,y,z], [a,b,c,q,x,y,z])'):
-            return [{'id': 'leading_heads_all_x'}]
-
         # base case works, the recursive doesn't (but it doesn't timeout)
         # this may be left as the last, most generic hint
-        if not prolog.engine.ask_truth(engine_id, 'insert(qQ, [qa,qb,qc], [qa,qb,qQ,qc])'):
+        if not prolog.engine.ask_truth(engine_id, 'dup([qa,qb,qc], [qa,qa,qb,qb,qc,qc])'):
             return [{'id': 'recursive_case'}]
 
 
diff --git a/prolog/problems/lists/dup_2/sl.py b/prolog/problems/lists/dup_2/sl.py
index eeefbd0..1513dda 100644
--- a/prolog/problems/lists/dup_2/sl.py
+++ b/prolog/problems/lists/dup_2/sl.py
@@ -27,18 +27,28 @@ potem je to skupaj ravno podvojen seznam.</p>
 hint = {
     'eq_instead_of_equ': '''\
 <p>Operator <code>==</code> je strožji od operatorja <code>=</code> v smislu, da je za slednjega dovolj,
-da elementa lahko naredi enaka (unifikacija). Morda z uporabo <code>=</code> narediš predikat
-<code>insert/3</code> delujoč tudi v kakšni drugi smeri. To bo kasneje znalo priti prav!</p>
+da elementa lahko prilagodi (unifikacija).</p>
 <p>Seveda pa lahko nalogo rešiš brez obeh omenjenih operatorjev, spomni se, da lahko unifikacijo narediš
 implicitno že kar v argumentih predikata (glavi stavka).</p>
 ''',
 
     'eq_instead_of_equ_markup': '''\
-<p>Morda bi bil bolj primeren operator za unifikacijo (=)?</p>
+<p>Morda bi šlo brez tega?</p>
+''',
+
+    'base_case_missing_[]': '''\
+<p>Tvoj robni pogoj je sicer smiselen, a ne deluje za poseben primer: prazen seznam. Malce ga predelaj.
+Ne naredi pa dveh robnih pogojev, ker bo to prineslo nov problem: pravilne rešitve se bodo podvajale.</p>
+''',
+
+    'base_case_arbitrary': '''\
+<p>Kako je lahko rezultat podvajanja praznega seznama poljuben seznam oz. karkoli?
+Matematiki bi rekli: "Dvakrat nič je nič in ne karkoli."</p>
+<p>Če je tvoj robni pogoj <code>dup([], _).</code>, ga še enkrat premisli: kaj je rezultat podvajanja?</p>
 ''',
 
     'base_case': '''\
-<p>Si pomislil na robni pogoj? Na katero mesto v seznamu najlažje vstaviš nek element?</p>
+<p>Si pomislil na robni pogoj? Kakšen seznam najlažje "podvojiš"?</p>
 ''',
 
     'recursive_case': '''\
@@ -58,28 +68,11 @@ da je <code>X</code> hkrati starš in sestra od <code>Y</code> ali kaj podobno z
 <p>Morda pa je kriv tudi manjkajoč, neustrezen ali preprosto nekompatibilen (s splošnim primerom) robni pogoj?</p>
 ''',
 
-    'ins_results_in_empty_list': '''\
-<p>Kako je lahko rezultat vstavljanja v seznam prazen seznam?</p>
-<p>Če je to tvoj robni pogoj, ga še enkrat premisli: kaj je rezultat vstavljanja?</p>
-''',
-
-    'ins_results_in_arbitrary_result': '''\
-<p>Kako je lahko rezultat vstavljanja v seznam poljuben seznam oz. karkoli?</p>
-<p>Če je to tvoj robni pogoj, ga še enkrat premisli: kaj je rezultat vstavljanja?</p>
-''',
-
-    'lost_heads': '''\
-<p>Element je vstavljen, ampak vsi pred njim so se pa izgubili, kajne?
-Si pozabil dati glavo nazaj na začetek seznama, ko se vračaš iz rekurzije?</p>
-<p>Poskusi postaviti naslednje vprašanje prologu in preglej <emph>vse</emph> rešitve:</p>
-<p><code>?- insert(q, [a,b,c,d], L).</code></p>
-''',
-
-    'leading_heads_all_x': '''\
-<p>Si morda pozabil (copy/paste?) in uporabil <code>[X|T]</code> namesto bolj splošnega
-<code>[H|T]</code> v rekurzivnem primeru?</p>
-<p>Od spodnjih dveh vprašanj prologu prvo deluje, drugo pa ne.</p>
-<p><code>?- insert(d, [d,d,d,d,e,f,g], L).</code></p>
-<p><code>?- insert(d, [a,b,c,d,e,f,g], L).</code></p>
+    'forcing_result_onto_recursion': '''
+<p>Ne vsiljuj rekurziji kaj naj vrne, prepusti se ji. To je tisti del, ko narediš predpostavko,
+če je ta izpolnjena, potem bo tvoje pravilo delovalo za večji primer.</p>
+<p>Je tvoj rekurzivni klic oblike <code>dup(T, [H,H|...])</code>? S tem vsiljuješ rekurziji
+da mora <emph>vrniti</emph> tudi podvojeno glavo. To moraš narediti ti z (obdelanim) rezultatom, ki ga rezurzija vrne.
+Skratka, [H,H] dodaj izven rekurzivnega klica.</p>
 ''',
 }