Small changes too fahrnheit and pythagorean problems.

2 files changed, 90 insertions, 85 deletions

diff --git a/python/problems/introduction/fahrenheit_to_celsius/common.py b/python/problems/introduction/fahrenheit_to_celsius/common.py
index 2e6c9b7..2962c55 100644
--- a/python/problems/introduction/fahrenheit_to_celsius/common.py
+++ b/python/problems/introduction/fahrenheit_to_celsius/common.py
@@ -1,6 +1,7 @@
 # coding=utf-8
 
-from python.util import has_token_sequence
+from python.util import has_token_sequence, string_almost_equal, \
+    string_contains_number, get_tokens, get_numbers, get_exception_desc
 from server.hints import Hint, HintSequence
 
 id = 180
@@ -46,19 +47,28 @@ def test(python, code):
     outputs = [ans[1] for ans in answers]
 
     n_correct = 0
-    for output, correct in zip(outputs, test_out):
+    tin = None
+    for i, (output, correct) in enumerate(zip(outputs, test_out)):
         if correct in output:
             n_correct += 1
+        else:
+            tin = test_in[i][1]
+            tout = correct
 
     passed = n_correct == len(test_in)
     hints = [{'id': 'test_results', 'args': {'passed': n_correct, 'total': len(test_in)}}]
+    if tin:
+        hints.append({'id': 'problematic_test_case', 'args': {'testin': str(tin), 'testout': str(tout)}})
     return passed, hints
 
 def hint(python, code):
+    tokens = get_tokens(code)
+
     # run one test first to see if there are any exceptions
     test_in = [(None, '212\n')]
-    exc = python(code=code, inputs=test_in, timeout=1.0)[0][3]
-    # have an exception!
+    answer = python(code=code, inputs=test_in, timeout=1.0)
+    exc = answer[0][3]
+    exc_hint = get_exception_desc(answer[0][3])
     if exc:
         if 'NameError' in exc:
             return [{'id':'name_error', 'args': {'message': exc}}]
@@ -68,31 +78,22 @@ def hint(python, code):
             return [{'id':'unsupported_operand', 'args': {'message': exc}}]
         elif 'TypeError' in exc:
             return [{'id':'unsupported_operand', 'args': {'message': exc}}]
-
-
-    # the trick is to decide when to show the plan and when the first hint. 
-    # I implemented a simple idea: show plan, when code contains something from the 
-    # solution, but not input. 
-    if len(code.strip()) < 5 or (not has_token_sequence(code, ['input']) and
-                    (has_token_sequence(code, ['32']) or 
-                     has_token_sequence(code, ['=', '5']) or 
-                     has_token_sequence(code, ['print']))):
-        return [{'id': 'plan'}]
-
+        else:
+            return exc_hint
 
     # if input is not present in code, student needs to learn about input
-    if not has_token_sequence(code, ['input']):
+    if not has_token_sequence(tokens, ['input']):
         return [{'id': 'no_input_call'}]
 
     # if tokens * or / or = are not in code, we have to teach them how to 
     # evaluate expressions.
-    if (not has_token_sequence(code, ['/']) or
-        not has_token_sequence(code, ['*']) or
-        not has_token_sequence(code, ['='])):
+    if (not has_token_sequence(tokens, ['/']) or
+        not has_token_sequence(tokens, ['*']) or
+        not has_token_sequence(tokens, ['='])):
         return [{'id' : 'expressions_python'}]
 
     # student is not using print function
-    if not has_token_sequence(code, ['print']):
+    if not has_token_sequence(tokens, ['print']):
         return [{'id' : 'printing'}]
 
     return None
diff --git a/python/problems/introduction/fahrenheit_to_celsius/sl.py b/python/problems/introduction/fahrenheit_to_celsius/sl.py
index d040adb..5b12d26 100644
--- a/python/problems/introduction/fahrenheit_to_celsius/sl.py
+++ b/python/problems/introduction/fahrenheit_to_celsius/sl.py
@@ -1,4 +1,6 @@
 # coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
 
 id = 180
 name = 'Pretvarjanje iz Fahrenheitov v Celzije'
@@ -9,40 +11,10 @@ description = '''\
 stopinjah, program pa jo izpiše v Celzijevih. Med temperaturama pretvarjamo po
 formuli C = 5/9 (F – 32).</p>'''
 
-general_exception = {
-    'error_head' : '''\
-<p>Napaka:</p>
-<pre>
-[%=message%]
-</pre>''',
-
-    'general': '''\
-<p>Pri razumevanju napake sta pomembni dve vrstici. V predzadnji vrstici je
-napisana lokacija napake (line ...), v zadnji vrstici pa izvemo za kakšno
-napako gre.</p>''',
-
-    'name_error' : '''\
-<p>Napaka <code>NameError</code> pomeni, da uporabljate nedefinirano vrednost:
-ali vrednost spremenljivke ni določena ali uporabljate funkcijo, ki ni
-uvožena.</p>''',
-
-    'type_error': '''\
-<p>TypeError napaka pomeni, da želite izvesti operacijo na nedovoljenih tipih.
-Npr., če želite sešteti niz in število ali klicati funkcijo, čeprav tisto ni
-funkcija, itd.</p>''',
-}
-
-hint = {
-    'plan': '''\
-<p>Program izvedemo v treh korakih:</p>
-<ol>
-  <li>Vprašanje za temperaturo v Fahrenheitih (F = ?).</li>
-  <li>Izračun temperature v Celzijih: C = 5/9 (F – 32)</li>
-  <li>Izpis temperature v Celzijih (izpiši C).</li>
-</ol>''',
-
-    'no_input_call': '''\
-<p>Uporabnika lahko nekaj vprašamo s funkcijo <code>input</code>. Funkcija
+no_input_call = ['''\
+<p>Uporabnika lahko nekaj vprašamo s funkcijo <code>input</code>.</p>''',
+                 '''\
+<p>Funkcija
 <code>input</code> sprejme kot argument niz (<em>angl.</em> string), ki se
 prikaže uporabniku kot vprašanje in vrne niz, ki ga je uporabnik napisal. Nize
 zapisujemo v narekovaje (lahko so enojni ali dvojni). Npr., naslednja
@@ -51,58 +23,89 @@ vrstica:</p>
 ime = input("Kako ti je ime?")
 </pre>
 <p>pokliče funkcijo <code>input</code>, ki povpraša uporabnika po imenu in si
-shrani uporabnikov odgovor v spremenljivko <code>ime</code>.</p>''',
+shrani uporabnikov odgovor v spremenljivko <code>ime</code>.</p>''']
 
-    'expressions_python': '''\
-<p>Računanje: če v Pythonu napišemo izraz, se bo ta izračunal. Npr., če
-napišemo</p>
+expressions_python = ['''\
+<p>Če v Pythonu napišemo izraz, se bo ta izračunal.</p>''',
+                      '''\
+<p>Če napišemo</p>
 <pre>
-3 + 6
+3 + 6 * 5
 </pre>
-<p>bo Python seštel vrednosti 3 in 6 in ... rezultat pozabil. Kadar pa želimo
-rezultat shraniti, za to uporabimo <em>prireditveni stavek</em>, kjer na levo
+<p>bo Python izračunal <code>3 + 6 * 5</code>.</p>''',
+                      '''\
+<p>Kadar želimo rezultat shraniti, za to uporabimo <em>prireditveni stavek</em>, kjer na levo
 napišemo ime spremenljivke, na desno pa izraz:</p>
 <pre>
 c = 2 * a * (3 + b)
 </pre>
-<p>Kot vidite, izraz lahko uporablja tudi spremenljivke.</p>''',
+''']
 
-    'printing': '''\
-<p>V Pythonu izpisujemo s funkcijo <code>print</code>. Če želimo izpisati več
-elementov, jih ločimo z vejico.  Recimo, da imamo spremenljivko
-<code>ime</code>, ki vsebuje naše ime, potem lahko napišemo:
+printing = ['''\
+<p>V Pythonu izpisujemo s funkcijo <code>print</code>.</p>''',
+            '''\
+</p>Če želimo izpisati več elementov, jih ločimo z vejico.
+Imejmo spremenljivko <code>ime</code>, ki vsebuje naše ime, potem:
 <pre>
 print("Ime mi je", ime, ".")
-</pre>''',
+</pre>''']
+
+plan = ['''\
+<p>Program izvedemo v treh korakih:</p>
+<ol>
+  <li>Vprašanje za temperaturo v Fahrenheitih (F = ?).</li>
+  <li>Izračun temperature v Celzijih: C = 5/9 (F – 32)</li>
+  <li>Izpis temperature v Celzijih (izpiši C).</li>
+</ol>''',
+        no_input_call,
+
+
+        ]
+
+hint = {
+    'plan': '''\
+<p>Program izvedemo v treh korakih:</p>
+<ol>
+  <li>Vprašanje za temperaturo v Fahrenheitih (F = ?).</li>
+  <li>Izračun temperature v Celzijih: C = 5/9 (F – 32)</li>
+  <li>Izpis temperature v Celzijih (izpiši C).</li>
+</ol>''',
+
+    'no_input_call': no_input_call,
+
+    'expressions_python': expressions_python,
+
+    'printing': printing,
 
     'name_error' : [
-        general_exception['error_head'],
-        general_exception['general'],
-        general_exception['name_error'],
+        mod.general_msg['error_head'],
+        mod.general_msg['general_exception'],
+        mod.general_msg['name_error'],
         '''\
-<p>Verjetno uporabljate spremenljivko, ki nima vrednosti. Ali v izrazu za
-izračun uporabljate napačno spremenljivko? Ali pri izpisu morda poskušate
+<p>Verjetno uporabljaš spremenljivko, ki nima vrednosti. Ali v izrazu za
+izračun uporabljaš napačno spremenljivko? Ali pri izpisu morda poskušaš
 izpisati napačno spremenljivko?</p>'''
     ],
 
     'unsupported_operand' : [
-        general_exception['error_head'],
-        general_exception['general'],
-        general_exception['type_error'],
+        mod.general_msg['error_head'],
+        mod.general_msg['general_exception'],
+        mod.general_msg['type_error'],
         '''\
 <p>Verjetni razlog: funkcija <code>input</code> vrača vrednost tipa niz, ki jo
 moramo najprej pretvoriti v tip <code>float</code>, če želimo z njo
 računati:</p>
 <pre>
 v = float(input(" ...
-</pre>
-<p>Zakaj je tako, boste razumeli kasneje, zaenkrat je dovolj, da to poznate.</p>
+</pre>''',
+        '''\
 <p>Na primeru pretvarjanja temperatur:</p>
 <pre>
 fniz = input("Temperatura [F]: ")
 f = float(fniz)
-</pre>
-<p>ali krajše, združeno v eno vrstico:</p>
+</pre>''',
+        '''\
+<p>Krajše, združeno v eno vrstico:</p>
 <pre>
 f = float(input("Temperatura [F]: "))
 </pre>
@@ -111,16 +114,17 @@ npr. "10". Če imamo v nizu tudi kakšno črko, bo Python javil napako.</p>'''
     ],
 
     'not_callable' : [
-        general_exception['error_head'],
-        general_exception['general'],
-        general_exception['type_error'],
+        mod.general_msg['error_head'],
+        mod.general_msg['general_exception'],
+        mod.general_msg['type_error'],
+        '''\
+<p>V programu poskušaš uporabiti število kot funkcijo, a to ne gre.</p>''',
         '''\
-<p>V programu poskušate uporabiti število kot funkcijo, a to ne gre.
-Verjetni razlog: Ali ste v izrazu pozabili na znak *. Pri pretvorbi temperatur bi namesto:</p>
+<p>Verjetni razlog: Ali si v izrazu pozabil na znak *? Pri pretvorbi temperatur bi namesto:</p>
 <pre>
 C = 5/9(F – 32)
 </pre>
-<p>morali napisati: </p>
+<p>moral napisati: </p>
 <pre>
 C = 5/9 * (F – 32).
 </pre>