From 876801eeff57b10d87a25d2ffaab338dc915dbdd Mon Sep 17 00:00:00 2001 From: Aleksander Sadikov 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.

Pa še drug razlog za ta vrstni red je: tako bomo vsi imeli enako, standardno, rešitev in bomo conc/3 uporabljali na enak način. To je zelo pomembno za kasneje.

+''', + + 'insertion_into_second_list': ''' +

Je tvoj rekurzivni klic oblike conc(T, [H|L2], ...)? Ne vstavljaj glave prvega seznama v +drugi seznam v rekurzivnem klicu. Pusti rekurziji, da sama poskrbi za združevanje repa prvega seznama T +z drugim seznamom L2, potem pa (ob vračanju rezultata) dodaj glavo prvega seznama +H na pravo mesto.

+''', + + 'two_heads': '''\ +

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.

+''', + + 'two_heads_markup': '''\ +

Res potrebuješ dve glavi?

+''', + + 'forcing_result_onto_recursion': ''' +

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.

+

Je tvoj rekurzivni klic oblike conc(T, L2 [H|...])? S tem vsiljuješ rekurziji +da mora vrniti tudi glavo, ki si jo prej že dal začasno stran. Glavo moraš na koncu na primerno mesto +vstaviti ti. Skratka, glavo H dodaj izven rekurzivnega klica.

''', 'final_hint': '''\ -- cgit v1.2.1