From 820e032e90b31507c4084148a099a0a159e01583 Mon Sep 17 00:00:00 2001 From: Aleksander Sadikov Date: Mon, 14 Mar 2016 16:28:57 +0100 Subject: Plans for divide/3 added. --- prolog/problems/lists/divide_3/common.py | 46 +++++++++++++++++++++---------- prolog/problems/lists/divide_3/sl.py | 45 ++++++++++++++---------------- prolog/problems/lists/permute_2/common.py | 2 +- 3 files changed, 53 insertions(+), 40 deletions(-) diff --git a/prolog/problems/lists/divide_3/common.py b/prolog/problems/lists/divide_3/common.py index 544dbc4..34b7fe9 100644 --- a/prolog/problems/lists/divide_3/common.py +++ b/prolog/problems/lists/divide_3/common.py @@ -8,7 +8,7 @@ from server.hints import Hint, HintPopup import server.problems id = 115 -number = 8 +number = 7 visible = True facts = None @@ -26,8 +26,10 @@ hint_type = { 'base_case': Hint('base_case'), 'recursive_case': Hint('recursive_case'), 'timeout': Hint('timeout'), - '[]_should_not_succeed': Hint('[]_should_not_succeed'), - 'list_returned': Hint('list_returned'), + 'base_case_arbitrary': Hint('base_case_arbitrary'), + 'second_base_case_missing': Hint('second_base_case_missing'), + 'unsuccessful_conc_use': Hint('unsuccessful_conc_use'), + 'forcing_result_onto_recursion': Hint('forcing_result_onto_recursion'), } test_cases = [ @@ -76,28 +78,44 @@ def hint(code, aux_code): return [{'id': 'eq_instead_of_equ_markup', 'start': m[0], 'end': m[1]} for m in marks] + \ [{'id': 'eq_instead_of_equ'}] + # general unsuccessful use of conc (not appropriate for this exercise) + if prolog.util.Token('NAME', 'conc') in tokens: + return [{'id': 'unsuccessful_conc_use'}] + # recursion is getting bigger and bigger - # succeeds when asked to return the last element of an empty list - if prolog.engine.ask_truthTO(engine_id, 'last_elem([], _)'): - return [{'id': '[]_should_not_succeed'}] - # returns a list, not an element as required - if prolog.engine.ask_one(engine_id, 'last_elem([qQ], [qQ])'): # 'last_elem([qQ], X), is_list(X)' with ask_one? - return [{'id': 'list_returned'}] + # forcing result onto recursion; case of divide(T, [H1|L1], [H2|L2]) + if not prolog.engine.ask_truthTO(engine_id, 'divide([a,b,c], _, _)') and \ + prolog.engine.ask_truthTO(engine_id, + 'asserta( divide([cob], [yowza,cob], [brix]) ), divide([yowza,brix,cob], [yowza,cob], [brix]), retract( divide([cob], [yowza,cob], [brix]) )'): + return [{'id': 'forcing_result_onto_recursion'}] + + # base case succeeds with arbitrary result + if prolog.engine.ask_truthTO(engine_id, 'divide([], L1, L2), (var(L1) ; var(L2))') or \ + prolog.engine.ask_truthTO(engine_id, 'divide([q], L1, L2), (var(L1) ; var(L2))') or \ + prolog.engine.ask_truthTO(engine_id, 'divide([q], [q|L1], L2), (var(L1) ; var(L2))'): + return [{'id': 'base_case_arbitrary'}] + + # one base case ok, the other one is missing (or bad) + # only trigger when recursion also works, because that makes more sense to the student + # than immediately telling him or her that there is another base case missing + if prolog.engine.ask_truthTO(engine_id, + 'divide([a,b], [a], [b]), \+ divide([a,b,c], [a,c], [b]) ; \+ divide([a,b], [a], [b]), divide([a,b,c], [a,c], [b])'): + return [{'id': 'second_base_case_missing'}] # missing/failed base case - if not prolog.engine.ask_truthTO(engine_id, 'last_elem([], [], [])'): + if not prolog.engine.ask_truthTO(engine_id, 'divide([], [], [])'): return [{'id': 'base_case'}] # target predicate seems to always be false - if not prolog.engine.ask_truthTO(engine_id, 'last_elem(_, _)') and \ - not prolog.engine.ask_truthTO(engine_id, 'last_elem([_,_,_,_,_,_], _)'): + if not prolog.engine.ask_truthTO(engine_id, 'divide([_,_,_,_,_], _, _)') and \ + not prolog.engine.ask_truthTO(engine_id, 'divide([_,_,_,_,_,_], _, _)'): return [{'id': 'predicate_always_false'}] - # base case works, the recursive doesn't (but it doesn't timeout) + # base cases work, 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, 'last_elem([qa,qb,qQ,qc], qc)'): + if not prolog.engine.ask_truth(engine_id, 'divide([qa,qb,qc,qd], [qa,qc], [qb,qd])'): return [{'id': 'recursive_case'}] except socket.timeout as ex: diff --git a/prolog/problems/lists/divide_3/sl.py b/prolog/problems/lists/divide_3/sl.py index 23340cc..93bc699 100644 --- a/prolog/problems/lists/divide_3/sl.py +++ b/prolog/problems/lists/divide_3/sl.py @@ -39,29 +39,37 @@ implicitno že kar v argumentih predikata (glavi stavka).

''', 'base_case': '''\ -

Si pomislil na robni pogoj? Kaj je najbolj enostaven primer? Kaj, če ima seznam samo en element?

+

Si pomislil na robni pogoj? Kaj je najbolj enostaven primer? Kaj, če je seznam prazen?

''', - '[]_should_not_succeed': '''\ -

Kako si lahko uspešno našel zadnji element v praznem seznamu? Verjetno potrebuješ drugačen robni pogoj.

+ 'base_case_arbitrary': '''\ +

Kako je lahko rezultat delitve seznama poljuben seznam oz. karkoli?

+

Če je tvoj robni pogoj v stilu divide([], _, _) ali divide([X], [X|_], ...), +ga še enkrat premisli: kaj je rezultat delitve, kaj vračaš? Robni pogoj je vedno dokončno specificirana +rešitev, tu načeloma ni neznank (_ ali neinicializiranih spremenljivk) v tem kar se vrača.

''', - 'list_returned': '''\ -

Vračaš seznam namesto elementa.

+ 'second_base_case_missing': '''\ +

Rekurzija se ne konča vedno uspešno. Sta morda dva različna primera kako se lahko izteče? Saj veš, +sodo in liho ;) Je morda potreben še kakšen robni pogoj?

''', - 'clumsy_conc_use': '''\ -

Uporabljaš conc/3? Zanimiva ideja. Ne pozabi, da mora drugi seznam, ki ga konkateniraš -biti dolžine ena, če hočeš doseči to kar želiš. Torej vzorec oblike [X], kajne?

+ 'unsuccessful_conc_use': '''\ +

Uporabljaš conc/3? Pri tej nalogi to ni najboljša ideja, ker conc/3 deli "v kosih", +težko boš prišel do posameznih elementov. Poskusi raje brez.

''', - 'unsuccessful_conc_use': '''\ -

Uporabljaš conc/3? Zanimiva ideja, da se rešiti tudi tako. Vendar boš moral še malo premisliti. -Ne pozabi, conc/3 ima tri argumente, vsi so seznami. Premisli, kakšen vzorec potrebuješ...

+ '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 divide(T, [H1|...], [H2|...])? S tem vsiljuješ rekurziji +da mora vrniti tudi obe glavi, ki jih sploh ne pozna, ker si jih ti ravnokar vzel stran! To moraš +narediti ti z (obdelanimi) rezultati, ki jih rezurzija vrne. Skratka, elementa H1 in H2 +dodaj izven rekurzivnega klica.

''', 'recursive_case': '''\ -

Robni primer deluje. Kaj pa rekurzivni, splošni, primer?

+

Robni primeri delujejo. Kaj pa rekurzivni, splošni, primer?

''', 'predicate_always_false': '''\ @@ -75,18 +83,5 @@ da je X hkrati starš in sestra od Y ali kaj podobno z 'timeout': '''\

Je morda na delu potencialno neskončna rekurzija? Kako se bo ustavila?

Morda pa je kriv tudi manjkajoč, neustrezen ali preprosto nekompatibilen (s splošnim primerom) robni pogoj?

-''', - - 'final_hint': '''\ -

Zanimivost: se spomniš kako smo rekli, da lahko predikat conc/3 uporabljamo za iskanje vzorcev? -Tudi zadnji element seznama je na nek način vzorec. Kaj se zgodi, če konkateniram poljuben seznam _ -in seznam dolžine ena (v tem vrstnem redu)? Seznam dolžine ena seveda zapišemo kot [Element].

-

Poskusi prolog vprašati tole:

-

?- conc(_, [Element], [a,b,c,d,e,f,q]).

-

Znaš sedaj dobiti zadnji element s pomočjo conc/3? To bo že še prišlo prav. Seveda pa je dostop -do zadnjega elementa še vedno potraten, O(n). Zato, če ni važno od kje ali kam dostopati, se splača vedno delati -s prvim elementom.

-

Kaj pa naredi tole? ;)

-

?- conc([a,b,c], [q], L).

''', } diff --git a/prolog/problems/lists/permute_2/common.py b/prolog/problems/lists/permute_2/common.py index 3ebbbd6..e39300b 100644 --- a/prolog/problems/lists/permute_2/common.py +++ b/prolog/problems/lists/permute_2/common.py @@ -5,7 +5,7 @@ import prolog.engine import server.problems id = 107 -number = 9 +number = 8 visible = True facts = None -- cgit v1.2.1