Small changes too fahrnheit and pythagorean problems.

2 files changed, 78 insertions, 78 deletions

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>