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(-)
(limited to 'prolog/problems/lists')
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