Finished the first version of introduction section.

2 files changed, 106 insertions, 78 deletions

diff --git a/python/problems/introduction/fast_fingers_2/common.py b/python/problems/introduction/fast_fingers_2/common.py
index d562f7b..958940d 100644
--- a/python/problems/introduction/fast_fingers_2/common.py
+++ b/python/problems/introduction/fast_fingers_2/common.py
@@ -1,11 +1,12 @@
 # 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 = 191
 group = 'introduction'
-number = 2
+number = 6
 visible = True
 
 solution = '''\
@@ -17,31 +18,78 @@ t = time.time()
 print('Koliko je ', x, ' krat ', y, '? ')
 z = float(input())
 if x * y == z:
-    print('Odgovor je pravilen.')
+    print(True)
 else:
-    print('Odgovor ni pravilen.')
+    print(False)
 print('Za razmišljanje ste porabili', time.time() ­ t, 's.')
 '''
 
+random_code = '''\
+import random
+random.randint = lambda x, y: {}
+'''
+
 hint_type = {
-    'plan': HintSequence('plan', 2),
     'random': Hint('random'),
-    'name_error': HintSequence('name_error', 4),
-    'type_error': HintSequence('type_error', 4),
-    'error': HintSequence('error', 2),
-    'if_clause': HintSequence('if_clause', 2),
+    'name_error': Hint('name_error'),
+    'type_error': Hint('type_error'),
+    'error': Hint('error'),
+    'if_clause': Hint('if_clause'),
+    'final_hint': Hint('final_hint')
 }
 
 def test(python, code):
-    passed = True
-    hints = [{'id': 'test_results', 'args': {'passed': 0, 'total': 0}}]
+    # List of inputs: (expression to eval, stdin).
+    test = [(5,'25\n'),
+            (6, '29\n'),
+            (1, '1\n'),
+            (7, '65\n'),
+            (9, '81\n')]
+
+    test_out = [
+        True,
+        False,
+        True,
+        False,
+        True
+    ]
+
+   # List of outputs: (expression result, stdout, stderr, exception).
+
+    n_correct = 0
+    tin = None
+    for ti, t in enumerate(test):
+        # change code, so that it replaces random values
+        # hook randint
+        tcode = random_code.format(t[0]) + code
+
+        answers = python(code=tcode, inputs=[(None, t[1])], timeout=1.0)
+        output = answers[0][1]
+        print ("out", output)
+        print (answers)
+
+        if str(test_out[ti]) in output and str(not test_out[ti]) not in output:
+            n_correct += 1
+        else:
+            tin = '{t[0]}*{t[0]}={t[1]}'.format(t=t)
+            tout = test_out[ti]
+
+    passed = n_correct == len(test)
+    hints = [{'id': 'test_results', 'args': {'passed': n_correct, 'total': len(test)}}]
+    if tin:
+        hints.append({'id': 'problematic_test_case', 'args': {'testin': str(tin), 'testout': str(tout)}})
+    else:
+        hints.append({'id': 'final_hint'})
     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, '5\n')]
+    test_in = [(None, '16\n')]
     answer = python(code=code, inputs=test_in, timeout=1.0)
     exc = answer[0][3]
+    exc_hint = get_exception_desc(answer[0][3])
     # if have an exception!
     if exc:
         if 'NameError' in exc:
@@ -49,21 +97,16 @@ def hint(python, code):
         elif 'TypeError' in exc:
             return [{'id':'type_error', 'args': {'message': exc}}]
         else:
-            return [{'id':'error', 'args': {'message': exc}}]
+            return exc_hint
 
     # First: if student does not import random, tell him about that module
-    if not has_token_sequence(code, ['random']) or \
-       not has_token_sequence(code, ['randint']):
+    if not has_token_sequence(tokens, ['random']) or \
+       not has_token_sequence(tokens, ['randint']):
         return [{'id': 'random'}]
 
-    # show plan if student is lost
-    # a) empty progam or
-    # b) there is not input (we can do it here, since we have no input hint)
-    if len(code.strip()) < 5 or (not has_token_sequence(code, ['input'])):
-        return [{'id': 'plan'}]
 
     # Student will have to test whether result is correct or not
-    if not has_token_sequence(code, ['if']):
+    if not has_token_sequence(tokens, ['if']):
         return [{'id' : 'if_clause'}]
 
     return None
diff --git a/python/problems/introduction/fast_fingers_2/sl.py b/python/problems/introduction/fast_fingers_2/sl.py
index da2751c..5b70044 100644
--- a/python/problems/introduction/fast_fingers_2/sl.py
+++ b/python/problems/introduction/fast_fingers_2/sl.py
@@ -1,4 +1,6 @@
 # coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
 
 id = 191
 name = 'Hitri prsti 2'
@@ -7,54 +9,42 @@ slug = 'Hitri prsti 2'
 description = '''\
 <p>Napiši program, podoben prejšnjemu, vendar naj ne vpraša vedno, koliko je 6 krat 7, temveč naj si
 izmišlja naključna vprašanja iz poštevanke. Program bo torej izžrebal dve števili med 1 in 10 in izpisal
-račun s tema dvema številoma namesto s 6 in 7. Tokrat naj program tudi preveri, ali je uporabnik
-pravilno izračunal produkt.</p>
+račun s tema dvema številoma namesto s 6 in 7. Tokrat naj program izpiše True, če je uporabnik
+pravilno izračunal produkt, drugače naj izpiše False. </p>
 <pre>
 Koliko je 6 krat 3? UPORABNIK VTIPKA 18
-Odgovor je pravilen.
+True
 Za razmišljanje ste porabili 2.1922357082366943 s.
 </pre>
-<p>Pri tej nalogi prav tako ni testnih primerov.</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 = {
-    'random': '''\
-<p>Če napišemo na začetek programa</p>
+random = ['''\
+<p>V modulu <code>random</code> imate funkcijo <code>randint</code>,
+ki vrača naključno celo število. </p>''',
+          '''\
+<p>Če napišemo na začetek programa:</p>
 <pre>
 from random import *
 </pre>
-<p>dobimo (med drugim) tudi funkcijo <code>randint(x,y)</code>, ki vrne naključno
+<p>dobimo (med drugim) dostop do funkcije <code>randint(x,y)</code>, ki vrne naključno
 celo število med <code>x</code> in <code>y</code>:</p>
 <pre>
 st = randint(1, 10)
-</pre>''',
+</pre>''']
 
-    'plan': ['''\
+if_clause = ['''
+<p>Program se mora obnašati ustrezno rezultatu. Če izračunamo pravilno, bo izpisal True, drugače
+False. To omogoča pogojni stavek. </p>''',
+'''<p> Primer pogojnega stavka <code>if</code>:
+<pre>
+if a == b: # dvopičje na koncu pogoja!
+    print(True)
+else:
+    print(False)
+</pre>'''
+    ],
+
+plan = ['''\
 <p>Razširimo plan iz prejšnje naloge:</p>
 <ol>
     <li>Izmisli si dve naključni števili </li>
@@ -66,39 +56,34 @@ st = randint(1, 10)
     <li>Izpiši </li>
 </ol>
 ''',
-'''<p>Pri vprašanju za rezultat produkta moramo navesti tudi vrednosti
-dveh spremenljivk. Najlažje bo, če uporabite dve vrstici:<p>
-<pre>
-print("Koliko je", x, " * ", y, "?")
-rezultat = float(input())
-</pre>'''],
+        random,
+        if_clause]
 
-    'if_clause': ['''
-<p>Program se mora obnašati ustrezno rezultatu. Če izračunamo pravilno, bo napisal "Odgovor je pravilen", drugače
-"Odgovor ni pravilen". To omogoča pogojni stavek. </p>''',
-'''<p> Primer pogojnega stavka <code>if</code>:
+hint = {
+    'random': random,
+
+    'if_clause': if_clause,
+
+    'final_hint': '''\
+<p><b>Odlično!</b> Za konec pa še zanimivost:
+Pri tej nalogi stavka <code>if</code> niti ne potrebujemo, saj bi lahko napisali le:</p>
 <pre>
-if a == b: # dvopičje na koncu pogoja!
-    print("a je enak b")
-else:
-    print("a ni enak b")
-</pre>'''
-    ],
+print(a == b)
+</pre>
+<p>kar bi izpisalo rezultat izraza a == b. Poskusi!</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'],]
 }