Small changes too fahrnheit and pythagorean problems.

4 files changed, 168 insertions, 163 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>
diff --git a/python/problems/introduction/pythagorean_theorem/common.py b/python/problems/introduction/pythagorean_theorem/common.py
index 405a39c..193527e 100644
--- a/python/problems/introduction/pythagorean_theorem/common.py
+++ b/python/problems/introduction/pythagorean_theorem/common.py
@@ -1,6 +1,7 @@
 # coding=utf-8
 
-from python.util import has_token_sequence, string_almost_equal
+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 = 188
@@ -52,47 +53,50 @@ 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 string_almost_equal(output, float(correct)):
             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, '3\n4\n')]
     answer = python(code=code, inputs=test_in, timeout=1.0)
     exc = answer[0][3]
-    # if have an exception!
+    exc_hint = get_exception_desc(answer[0][3])
     if exc:
         if 'NameError' in exc:
             return [{'id':'name_error', 'args': {'message': exc}}]
-        if 'unsupported operand' in exc or 'TypeError' in exc:
+        elif 'unsupported operand' in exc or 'TypeError' in exc:
             return [{'id':'unsupported_operand', 'args': {'message': exc}}]
-
-    # show plan if student is lost
-    if len(code.strip()) < 5 or (not has_token_sequence(code, ['input']) and
-                    (has_token_sequence(code, ['pi']) or 
-                     has_token_sequence(code, ['sin']) 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']) or \
-       not has_token_sequence(code, ['float']):
+    if not has_token_sequence(tokens, ['input']) or \
+       not has_token_sequence(tokens, ['float']) or \
+       not has_token_sequence(tokens, ['=']):
         return [{'id': 'no_input_call'}]
 
     # if tokens sqrt or ** are not in code, we have to teach them how to 
     # use math functions.
-    if  (not has_token_sequence(code, ['sqrt']) or
-        not has_token_sequence(code, ['**'])):
+    if  (not has_token_sequence(tokens, ['sqrt']) or
+        not has_token_sequence(tokens, ['**'])):
         return [{'id' : 'math_functions'}]
 
     # 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/pythagorean_theorem/sl.py b/python/problems/introduction/pythagorean_theorem/sl.py
index 15c498a..7f26e00 100644
--- a/python/problems/introduction/pythagorean_theorem/sl.py
+++ b/python/problems/introduction/pythagorean_theorem/sl.py
@@ -1,94 +1,90 @@
 # coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
 
 id = 188
 name = 'Pitagorov izrek'
 slug = 'Pitagorov izrek'
 
 description = '''\
-<p>Npiši program, ki uporabnika vpraša po dolžinah katet pravokotnega trikotnika in
+<p>Napiši program, ki uporabnika vpraša po dolžinah katet pravokotnega trikotnika in
 izpiše dolžino hipotenuze. '''
 
-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 razdelimo na tri dele kot pri Fahrenheitih:</p>
-<ol>
-    <li>Preberi vrednosti katet (a,b = ?)</li>
-    <li>Izračunaj dolžino hipotenuze c (c = …)</li>
-    <li>Izpis dolžine hipotenuze (print … )</li>
-</ol>
-''',
-
-    'no_input_call': '''\
-<p>Tako kot pri prejšnji nalogi uporabimo funkcijo input, le da tokrat preberemo 
-dve vrednosti in jih shranimo v dve spremenljivki:</p>
+no_input_call = ['''\
+<p>Tako kot pri prejšnji nalogi za branje uporabimo funkcijo <code>input</code></p>''',
+                 '''\
+<p>Preberemo dve vrednosti in jih shranimo v dve spremenljivki:</p>
 <pre>
  a = float(input("Prva kateta: "))
  b = float(input("Druga kateta: "))
-</pre>
-<p> Imeni <code>a</code> in <code>b</code> sta spremenljivki (angl. variable). 
-Spremenljivke uporabljamo, kadar želimo kakšno vrednost shraniti, ki jo bomo 
+</pre>''',
+                '''\
+<p> Imeni <code>a</code> in <code>b</code> sta spremenljivki (angl. variable).
+Spremenljivke uporabljamo, kadar želimo kakšno vrednost shraniti, ki jo bomo
 potrebovali kasneje v programu. Imena spremenljivk so lahko poljubno
  dolga, v našem primeru bi jim lahko rekli tudi <code>kateta_a</code> in <code>kateta_b</code>.
- Pri programiranju velja, da izbiramo taka imena spremenljivk, 
+ Pri programiranju velja, da izbiramo taka imena spremenljivk,
  ki bodo naredila program berljiv. </p>
- ''',
+ '''
+]
 
-    
-    'math_functions': ['''
-<p>Dolžina hipotenuze je kvadratni koren vsote kvadratov katet oz.</p>
-<p>c = sqrt(a**2 + b**2)</p>
-<p>Dvojni znak za množenje ** je potenciranje. Funkcija <code>sqrt</code> 
-izračuna kvadratni koren. Vendar pa funkcija <code>sqrt</code> ni vgrajena v Python, 
-temveč se nahaja v ločenem matematičnem modulu oz. zbirki  matematičnih funkcij. 
+math_functions =  ['''\
+<p>Dolžina hipotenuze je kvadratni koren vsote kvadratov katet.</p>''',
+                   '''\
+<pre>
+c = sqrt(a**2 + b**2)
+</pre>
+<p>Dvojni znak za množenje ** je potenciranje. Funkcija <code>sqrt</code>
+izračuna kvadratni koren. ''',
+                   '''\
+Funkcija <code>sqrt</code> ni vgrajena v Python,
+temveč se nahaja v ločenem matematičnem modulu oz. zbirki  matematičnih funkcij.
 Da bi lahko dostopali do teh funkcij, moramo ta modul najprej uvoziti:</p>
 <pre>
 from math import *
 </pre>
-<p>Poleg funkcije <code>sqrt</code> matematični modul vsebuje še vrsto 
-uporabnih matematičnih funkcij, kot so:   log, exp, trigonometrične funkcije, itd. 
+<p>Poleg funkcije <code>sqrt</code> matematični modul vsebuje še vrsto
+uporabnih matematičnih funkcij, kot so:   log, exp, trigonometrične funkcije, itd.
 Opis modula najdete v Pythonovi dokumentaciji.</p>
-'''],
+''']
 
-    'printing': ['''
-<p> V Pythonu izpisujemo s funkcijo <code>print. Č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.  Recimo, da imamo spremenljivko <code>ime</code>,
+ki vsebuje naše ime, potem lahko napišemo:</p>
 <pre>
 print("Ime mi je", ime, ".")
 </pre>'''],
 
-    'name_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['name_error'], '''
+plan = ['''\
+<p>Program razdelimo na tri dele kot pri Fahrenheitih:</p>
+<ol>
+    <li>Preberi vrednosti katet (a,b = ?)</li>
+    <li>Izračunaj dolžino hipotenuze c (c = …)</li>
+    <li>Izpis dolžine hipotenuze (print … )</li>
+</ol>
+''',
+        no_input_call,
+        math_functions,
+        printing]
+
+hint = {
+    'no_input_call': no_input_call,
+
+    'math_functions': math_functions,
+
+    'printing': printing,
+
+    '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
      izpisati napačno spremenljivko?</p>'''],
 
-    'unsupported_operand' : [general_exception['error_head'], general_exception['general'],
-    general_exception['type_error'], '''
+    'unsupported_operand' : [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>