From 876801eeff57b10d87a25d2ffaab338dc915dbdd Mon Sep 17 00:00:00 2001
From: Aleksander Sadikov <aleksander.sadikov@fri.uni-lj.si>
Date: Sat, 12 Mar 2016 00:49:44 +0100
Subject: Hints for conc/3 added.

---
 prolog/problems/lists/conc_3/common.py | 29 +++++++++++++++++++++++++----
 prolog/problems/lists/conc_3/sl.py     | 24 ++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 4 deletions(-)

diff --git a/prolog/problems/lists/conc_3/common.py b/prolog/problems/lists/conc_3/common.py
index 3ed35bc..fd97865 100644
--- a/prolog/problems/lists/conc_3/common.py
+++ b/prolog/problems/lists/conc_3/common.py
@@ -26,6 +26,10 @@ hint_type = {
     'timeout': Hint('timeout'),
     'final_hint': Hint('final_hint'),
     'second_list_iteration': Hint('second_list_iteration'),
+    'forcing_result_onto_recursion': Hint('forcing_result_onto_recursion'),
+    'two_heads_markup': HintPopup('two_heads_markup'),
+    'two_heads': Hint('two_heads'),
+    'insertion_into_second_list': Hint('insertion_into_second_list'),
 }
 
 test_cases = [
@@ -77,15 +81,32 @@ def hint(code, aux_code):
         # recursion is getting bigger and bigger
 
 
-#        # missing/failed base case
-#        if not prolog.engine.ask_truthTO(engine_id, 'conc([], [qa,qb,qc], [qa,qb,qc])'):
-#            return [{'id': 'base_case'}]
-
         # shortening the second list instead of first list
         if prolog.engine.ask_truthTO(engine_id,
            'conc([a,b,c], [1,2,3,4], L), (L = [1,2,3,4,a,b,c] ; L = [4,3,2,1,a,b,c])'):
             return [{'id': 'second_list_iteration'}]
 
+        # inserting first list's head into the second list in recursive call: conc(T, [H\L2], L)
+        if prolog.engine.ask_truthTO(engine_id,
+           'conc([a,b,c,d,e], [1,2,3,4], L = [e,d,c,b,a,1,2,3,4])'):
+            return [{'id': 'insertion_into_second_list'}]
+
+        # forcing result onto recursion; case of conc(T, L2, [H|L])
+        if not prolog.engine.ask_truthTO(engine_id, 'conc([a,b,c,d,e], _, _)') and \
+           prolog.engine.ask_truthTO(engine_id,
+           'asserta( conc([b,c],[1,2],[yowza,b,c,1,2]) ), conc([yowza,b,c],[1,2],L), retract( conc([b,c],[1,2],[yowza,b,c,1,2]) )'):
+            return [{'id': 'forcing_result_onto_recursion'}]
+
+        # using two heads indicating that the user wants heads from both lists
+        # TODO: Tim, it would be nice/safer to catch this just for a single clause, not whole code.
+        if prolog.util.Token('NAME', 'H1') in tokens and \
+           (prolog.util.Token('NAME', 'H') in tokens or prolog.util.Token('NAME', 'H2') in tokens):
+            targets = [prolog.util.Token('NAME', 'H1'), prolog.util.Token('NAME', 'H2')]
+            marks = [(t.pos, t.pos + len(t.val)) for t in tokens if t in targets]
+            if marks:
+                return [{'id': 'two_heads_markup', 'start': m[0], 'end': m[1]} for m in marks] + \
+                       [{'id': 'two_heads'}]
+
         # missing/failed base case
         if not prolog.engine.ask_one(engine_id, 'conc([], [qa,qb,qc], [qa,qb,qc])'):
             return [{'id': 'base_case'}]
diff --git a/prolog/problems/lists/conc_3/sl.py b/prolog/problems/lists/conc_3/sl.py
index 1fd0500..8519870 100644
--- a/prolog/problems/lists/conc_3/sl.py
+++ b/prolog/problems/lists/conc_3/sl.py
@@ -64,6 +64,30 @@ Morda npr. v rekurziji zmanjšuješ prvi seznam, ustaviš pa se pri praznem drug
 Raje zmanjšuj prvi seznam in pusti drugega kot je.</p>
 <p>Pa še drug razlog za ta vrstni red je: tako bomo vsi imeli enako, standardno, rešitev in bomo
 <code>conc/3</code> uporabljali na enak način. To je zelo pomembno za kasneje.</p>
+''',
+
+    'insertion_into_second_list': '''
+<p>Je tvoj rekurzivni klic oblike <code>conc(T, [H|L2], ...)</code>? Ne vstavljaj glave prvega seznama v
+drugi seznam v rekurzivnem klicu. Pusti rekurziji, da sama poskrbi za združevanje repa prvega seznama <code>T</code>
+z drugim seznamom <code>L2</code>, potem pa (ob vračanju rezultata) dodaj glavo prvega seznama
+<code>H</code> na pravo mesto.</p>
+''',
+
+    'two_heads': '''\
+<p>Res potrebuješ dve glavi? Poskušaš morda zmanjševati hkrati oba seznama? To ni dobra ideja, tako boš
+res težko prišel do pravilne rešitve. Raje zmanjšuj samo prvi seznam in pusti drugega kot je.</p>
+''',
+
+    'two_heads_markup': '''\
+<p>Res potrebuješ dve glavi?</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>conc(T, L2 [H|...])</code>? S tem vsiljuješ rekurziji
+da mora <emph>vrniti</emph> tudi glavo, ki si jo prej že dal začasno stran. Glavo moraš na koncu na primerno mesto
+vstaviti ti. Skratka, glavo <code>H</code> dodaj izven rekurzivnega klica.</p>
 ''',
 
     'final_hint': '''\
-- 
cgit v1.2.1