Finished the first version of introduction section.

2 files changed, 65 insertions, 74 deletions

diff --git a/python/problems/introduction/average/common.py b/python/problems/introduction/average/common.py
index 0a2b689..c84ef72 100644
--- a/python/problems/introduction/average/common.py
+++ b/python/problems/introduction/average/common.py
@@ -1,28 +1,30 @@
 # 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 = 189
 group = 'introduction'
-number = 2
+number = 4
 visible = True
 
 solution = '''\
 a = float(input('Ocena [Ana]? '))
 b = float(input('Ocena [Benjamin]? '))
 c = float(input('Ocena [Cilka]? '))
-print('Povpre?je:', (a + b + c) / 3)
+print('Povprečje:', (a + b + c) / 3)
 print('Srednja vrednost:', a + b + c - min(a, b, c) - max( a, b, c))
 '''
 
 hint_type = {
     'plan': Hint('plan'),
     'eval_expression': Hint('eval_expression'),
-    'name_error': HintSequence('name_error', 4),
-    'unsupported_operand': HintSequence('unsupported_operand', 4),
-    'error': HintSequence('error', 2),
-    'radians': HintSequence('radians', 3),
+    'name_error': Hint('name_error'),
+    'unsupported_operand': Hint('unsupported_operand'),
+    'error': Hint('error'),
+    'radians': Hint('radians'),
     'printing': Hint('printing'),
 }
 
@@ -47,47 +49,53 @@ def test(python, code):
     ]
 
     # List of outputs: (expression result, stdout, stderr, exception).
+    print ("ena")
     answers = python(code=code, inputs=test_in, timeout=1.0)
     outputs = [ans[1] for ans in answers]
 
     n_correct = 0
-    for output, c in zip(outputs, test_out):
-        if string_almost_equal(output, c[0]) and \
-           string_almost_equal(output, c[1]):
+    tin = None
+    print (outputs)
+    print (test_out)
+    for i, (output, correct) in enumerate(zip(outputs, test_out)):
+        if string_almost_equal(output, correct[0]) and \
+           string_almost_equal(output, correct[1]):
             n_correct += 1
+        else:
+            tin = test_in[i][1].replace('\n', ' ')
+            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, '1\n1\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}}]
         elif 'TypeError' in exc:
             return [{'id':'type_error', 'args': {'message': exc}}]
         else:
-            return [{'id':'error', 'args': {'message': exc}}]
-
-    # show plan if student is lost
-    # a) empty progam
-    # b) there is not input (we can do it here, since we have no input hint)
-    if not code or (not has_token_sequence(code, ['input'])):
-        return [{'id': 'plan'}]
+            return exc_hint
 
     # Help to compute average (if /3 is not program)
-    if not has_token_sequence(code, ['/', '3']):
+    if not has_token_sequence(tokens, ['/', '3']):
         return [{'id': 'average'}]
 
     # If student is not using functions min or max, then he is
     # not computing median in the right way
-    if not has_token_sequence(code, ['min']) or \
-       not has_token_sequence(code, ['max']):
+    if not has_token_sequence(tokens, ['min']) or \
+       not has_token_sequence(tokens, ['max']):
         return [{'id': 'median'}]
 
     # program is working correctly for mean but not for median
@@ -95,7 +103,7 @@ def hint(python, code):
         return [{'id': 'median'}]
 
     # 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/average/sl.py b/python/problems/introduction/average/sl.py
index 2824320..d493aa2 100644
--- a/python/problems/introduction/average/sl.py
+++ b/python/problems/introduction/average/sl.py
@@ -1,90 +1,73 @@
 # coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
 
 id = 189
 name = 'Povprečna ocena'
 slug = 'Povprečna ocena'
 
 description = '''\
-<p>Napiši program, ki mu uporabnik vpiše oceno, ki so jo pri matematiki dobili Ana, Benjamin in Cilka.
-Program naj izračuna in izpiše povprečno oceno ter srednjo vrednost. Sprogramiraj slednjo brez
-uporabe pogojnih stavkov ali česa podobno "naprednega". Konkretno, uporabljaj le funkcije input,
-print, min in max. Namig: min in max lahko prejmeta poljubno število argumentov. Pomisli tudi na
-to, da imaš samo tri osebe; pri štirih ta trik ne bi vžgal. Primer izvajanja programa:</p>
+<p>Napiši program, ki mu uporabnik vpiše oceno, ki so jo pri matematiki dobili Ana, Benjamin in Cilka.</p>
+<p>Program naj izračuna in izpiše povprečno oceno ter srednjo vrednost. Sprogramiraj slednjo brez
+uporabe pogojnih stavkov ali česa podobno "naprednega". Konkretno, uporabljaj le funkcije <code>input</code>,
+<code>print</code>, <code>min</code> in <code>max</code>.</p>
+<p>Namig: <code>min</code> in <code>max</code> lahko prejmeta poljubno število argumentov. Pomisli tudi na
+to, da imaš samo tri osebe; pri štirih ta trik ne bi vžgal. </p>
+<p>Primer izvajanja programa:</p>
 <pre>
 Ocena [Ana]? 2
 Ocena [Benjamin]? 4
 Ocena [Cilka]? 5
-Povpreče: 3.6666666666666665
+Povprečje: 3.6666666666666665
 Srednja vrednost: 4.0
 </pre>
 '''
 
-general_exception = {
-    'error_head' : '''\
-<p>Napaka:</p>
+average =  ['''\
+<p>Povprečje izračunamo kot vsoto vseh elementov, ki jo delimo s številom elementov. </p>''',
+            '''\
+<p>Konkretno v
+našem primeru:</p>
 <pre>
-[%=message%]
+p = (a + b + c) / 3
 </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>
-    ''',
-}
+median = ['''\
+<p>Če imamo samo 3 vrednosti in odstranimo najmanjšo in največjo vrednost, nam ostane srednja
+vrednost.</p>''',
+          '''\
+<p>V programu to najlažje izvedemo tako, da vrednosti seštejemo in potem vsoti odštejemo
+najmanjšo in največjo vrednost.</p>
+''']
 
-hint = {
-    'plan': '''\
+plan = [ '''\
 <p>Ista strategija kot pri predhodnih nalogah: 1) preberi ocene Ane, Benjamina in Cilke, 2) izračunaj
 povprečno vrednost in srednjo vrednost ter 3) izpiši.</p>
 ''',
+         average,
+         median]
 
-    'average': '''
-<p>Povprečje izračunamo kot vsoto vseh elementov, ki jo delimo s številom elementov. Konkretno v
-našem primeru:</p>
-<pre>
-p = (a + b + c) / 3
-</pre>
-''',
+hint = {
+    'average': average,
 
-    'median': '''
-<p>Če imamo samo 3 vrednosti in odstranimo najmanjšo in največjo vrednost, nam ostane srednja
-vrednost. V programu to najlažje izvedemo tako, da vrednosti seštejemo in potem vsoti odštejemo
-najmanjšo in največjo vrednost.</p>
-''',
+    'median': median,
     
     '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:
-<pre>
-print("Ime mi je", ime, ".")
-</pre>''',
+<p> V Pythonu izpisujemo s funkcijo <code>print</code>.</p>''',
 
-    'name_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['name_error'], '''
+    '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>'''],
 
-    'type_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['type_error'], '''
+    '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>
 '''],
-
-    'error' : [general_exception['error_head'], general_exception['general'],]
 }