Added problem "counting".

Some other small text corrections.

5 files changed, 233 insertions, 9 deletions

diff --git a/python/problems/lists_and_for/contains_42/sl.py b/python/problems/lists_and_for/contains_42/sl.py
index 9477e29..fe6b57e 100644
--- a/python/problems/lists_and_for/contains_42/sl.py
+++ b/python/problems/lists_and_for/contains_42/sl.py
@@ -19,10 +19,12 @@ Seveda mora program delati za poljubne sezname in ne samo za seznam iz primera.<
 '''
 
 for_loop = ['''\
-<p>Čez elemente v seznamu se najlažje sprehodimo s <b>for</b> zanko.
+<p>Pregledati bo treba vse elemente v seznamu <code>xs</code>''',
+            '''\
+<p>Najlažje bo s <b>for</b> zanko.
 ''',
              '''\
-<p>Poskusi, kaj naredita naslednji dve vrstici:</p>
+<p>Poskusii naslednji dve vrstici:</p>
 <pre>
 for x in xs:
     print (x)
@@ -30,21 +32,25 @@ for x in xs:
              '''\
 <p>V zgornjem primeru z zanko <code>for</code> Pythonu naročimo naj se sprehodi čez seznam <code>xs</code>
 in na vsakem koraku trenutni element seznama shrani v spremenljivko <code>x</code>.
-Kaj naj Python naredi s to spremenljivko, mu naročimo v zamaknjenih vrsticah.
+Kaj naj Python naredi s to spremenljivko, je zapisano v zamaknjenih vrsticah.
 Tokrat vrednost le izpišemo.</p>''']
 
 if_clause = ['''\
-<p><code>Poglej, ali je število 42?</code> Uporabite pogojni stavek <b>if</b>!</p>''',
+<p><code>Preveri, ali imamo število 42?</code></p>''',
+             '''\
+<p>Uporabi pogojni stavek <b>if</b>!</p>''',
              '''\
 <pre>
 if x == 42:
 </pre>''']
 
 seen_42 = ['''\
-<p>Kako si lahko zapomnimo, da smo <b>videli 42</b>? Uporabi novo spremenljivko!</p>
+<p>Zapomni si, da si našel 42!</p>''',
+           '''\
+<p>Uporabi novo spremenljivko!</p>
 ''',
               '''\
-<p>Spremenljivko na začetku nastavimo na False, npr.:</p>
+<p>Spremenljivko na začetku nastavimo na False:</p>
 <pre>
 videl42 = False
 </pre>
diff --git a/python/problems/lists_and_for/contains_string/sl.py b/python/problems/lists_and_for/contains_string/sl.py
index 2f24458..d177463 100644
--- a/python/problems/lists_and_for/contains_string/sl.py
+++ b/python/problems/lists_and_for/contains_string/sl.py
@@ -18,10 +18,12 @@ xs = ['foo', 'bar', 'baz', 'Waldo', 'foobar']
 '''
 
 for_loop = ['''\
-<p>Čez elemente v seznamu se najlažje sprehodimo s <b>for</b> zanko.
+<p>Pregledati bo treba vse elemente v seznamu <code>xs</code>''',
+            '''\
+<p>Najlažje bo s <b>for</b> zanko.
 ''',
              '''\
-<p>Poskusi, kaj naredita naslednji dve vrstici:</p>
+<p>Poskusii naslednji dve vrstici:</p>
 <pre>
 for x in xs:
     print (x)
@@ -29,7 +31,7 @@ for x in xs:
              '''\
 <p>V zgornjem primeru z zanko <code>for</code> Pythonu naročimo naj se sprehodi čez seznam <code>xs</code>
 in na vsakem koraku trenutni element seznama shrani v spremenljivko <code>x</code>.
-Kaj naj Python naredi s to spremenljivko, mu naročimo v zamaknjenih vrsticah.
+Kaj naj Python naredi s to spremenljivko, je zapisano v zamaknjenih vrsticah.
 Tokrat vrednost le izpišemo.</p>''']
 
 if_clause = ['''\
@@ -39,6 +41,7 @@ if_clause = ['''\
 if x == 'Waldo':
 </pre>''']
 
+
 plan = ['''\
 <p><b>Plan</b> je enak kot pri prvi nalogi:
 <pre>
diff --git a/python/problems/lists_and_for/counting/common.py b/python/problems/lists_and_for/counting/common.py
new file mode 100644
index 0000000..2d59182
--- /dev/null
+++ b/python/problems/lists_and_for/counting/common.py
@@ -0,0 +1,109 @@
+# coding=utf-8
+
+import re
+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
+
+id = 203
+group = 'lists_and_for'
+number = 3
+visible = True
+
+solution = '''\
+xs = [42, 5, 4, -7, 2, 12, -3, -4, 11, 42, 2]
+
+stevec = 0
+for x in xs:
+    if x == 42:
+        stevec += 1
+print("Število 42 se v seznamu pojavi", stevec, "krat.")
+'''
+
+hint_type = {
+    'no_xs': Hint('no_xs'),
+    'for_loop': Hint('for_loop'),
+    'if_clause': Hint('if_clause'),
+    'printing': Hint('printing'),
+    'print_out_for': Hint('print_out_for'),
+    'final_hint': Hint('final_hint')
+}
+
+def test(python, code):
+    test_xs = [[42, 5, 4, -7, 2, 42, -3, -4, 11, 42, 2],
+               [42, 5, 4, -7, 2, 12, 42, -4, 11, 2],
+               [5, 4, -7, 2, 12, -3, -4, 11, 2],
+               [],
+               [42],
+               [42, 42],
+               [1, 2, 3, -42],
+               [1, 2, 3, 42, -42]]
+    test_out = [
+        3,
+        2,
+        0,
+        0,
+        1,
+        2,
+        0,
+        1
+    ]
+
+    n_correct = 0
+    tin = None
+    for xs_i, xs in enumerate(test_xs):
+        # change code to contain new xs instead of the one
+        # given by user
+        tcode = re.sub(r'^xs\s*=\s*\[.*?\]',
+                       'xs = ' + str(xs),
+                       code,
+                       flags = re.DOTALL | re.MULTILINE)
+
+        # use python session to call tcode
+        answers = python(code=tcode, inputs=[(None, None)], timeout=1.0)
+        output = answers[0][1]
+
+        if str(test_out[xs_i]) in output:
+            n_correct += 1
+        else:
+            tin = test_xs[xs_i]
+            tout = test_out[xs_i]
+
+    passed = n_correct == len(test_xs)
+    hints = [{'id': 'test_results', 'args': {'passed': n_correct, 'total': len(test_xs)}}]
+    if tin:
+        hints.append({'id': 'problematic_test_case', 'args': {'testin': str(tin), 'testout': str(tout)}})
+    if passed:
+        hints.append({'id': 'final_hint'})
+    return passed, hints
+
+def hint(python, code):
+    # run one test first to see if there are any exceptions
+    answer = python(code=code, inputs=[(None, None)], timeout=1.0)
+    exc = get_exception_desc(answer[0][3])
+    if exc: return exc
+
+    tokens = get_tokens(code)
+
+    # if has no xs, tell him to ask for values
+    if not has_token_sequence(tokens, ['xs', '=', '[']):
+        return [{'id' : 'no_xs'}]
+
+    # student does not have while or for: instruct him on loops
+    if not has_token_sequence(tokens, ['while']) and \
+       not has_token_sequence(tokens, ['for']):
+        return [{'id' : 'for_loop'}]
+
+    if not has_token_sequence(tokens, ['if']):
+        return [{'id' : 'if_clause'}]
+
+    # student is not using print function
+    if not has_token_sequence(tokens, ['print']):
+        return [{'id' : 'printing'}]
+
+    # student is not using print at the beginning of line
+    if not has_token_sequence(tokens, ['\n', 'print']):
+        return [{'id' : 'print_out_for'}]
+
+
+    return None
diff --git a/python/problems/lists_and_for/counting/en.py b/python/problems/lists_and_for/counting/en.py
new file mode 100644
index 0000000..805b37c
--- /dev/null
+++ b/python/problems/lists_and_for/counting/en.py
@@ -0,0 +1,16 @@
+# coding=utf-8
+
+id = 193
+name = 'Contains 42'
+slug = 'Contains 42'
+
+description = '''\
+<p>(translation missing)</p>'''
+
+hint = {
+    'plan': '''\
+<p>(translation missing)</p>''',
+
+    'no_input_call': '''\
+<p>(translation missing)</p>''',
+}
diff --git a/python/problems/lists_and_for/counting/sl.py b/python/problems/lists_and_for/counting/sl.py
new file mode 100644
index 0000000..67dc323
--- /dev/null
+++ b/python/problems/lists_and_for/counting/sl.py
@@ -0,0 +1,90 @@
+# coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
+
+
+id = 203
+name = 'Štej'
+slug = 'Štej'
+
+
+description = '''\
+<p>Napiši program, ki v podanem seznamu prešteje število ponovitev števila 42. 
+Seznam <code>xs</code> definiraj na vrhu programa, kot v prejšnjih nalogah,
+izpis pa naj bo takšen: "Število 42 se v seznamu pojavi 2 krat."'''
+
+for_loop = ['''\
+<p>Pregledati bo treba vse elemente v seznamu <code>xs</code>''',
+            '''\
+<p>Najlažje s <code>for</code> zanko.
+''',
+             '''\
+<p>Poskusi naslednji dve vrstici:</p>
+<pre>
+for x in xs:
+    print (x)
+</pre>''',
+             '''\
+<p>V zgornjem primeru z zanko <code>for</code> Pythonu naročimo naj se sprehodi čez seznam <code>xs</code>
+in na vsakem koraku trenutni element seznama shrani v spremenljivko <code>x</code>.
+Kaj naj Python naredi s to spremenljivko, je zapisano v zamaknjenih vrsticah.
+V danem primeru vrednost le izpišemo.</p>''']
+
+if_clause = ['''\
+<p>Preveri, ali imamo število 42?</p>''',
+         '''\
+<p>Uporabi pogojni stavek <code>if</code>!</p>''',
+         '''\
+<pre>
+if x == 42:
+</pre>''']
+
+count_42 = ['''\
+<p>Zapomni si, da si našel 42! Povečaj števec!</p>''',
+            '''\
+<p>Števec na začetku nastavimo na 0</p>
+<pre>
+stevec = 0
+</pre>
+<p>in ga tekom zanke ustrezno spreminjamo.''']
+
+plan = ['''\
+<p><b>Plan</b>: Preglej vse elemente v seznamu in vsakič, ko srečaš 42, povečaj 
+števec za 1.</p>
+''',
+        '''\
+<p>Bolj podroben plan: </p>
+<pre>
+Za vsak element v seznamu
+    Poglej, ali je 42?
+        Če je, povečaj števec.
+Izpiši števec.
+</pre>
+<p>Zdaj pa je potrebno le še slovenščino prevesti v Python.</p>
+''',
+        for_loop,
+        if_clause,
+        count_42]
+
+hint = {
+    'no_xs': ['''\
+<p>Program mora imeti na začetku definiran seznam <code>xs</code>.</p>'''],
+
+    'for_loop': for_loop,
+
+    'if_clause': if_clause,
+
+    'printing': ['''\
+<p>Izpiši rezultat.</p>'''],
+
+    'print_out_for': ['''\
+<p>Pazi, da izpišeš rezultat izven zanke!</p>'''],
+
+    'final_hint:': ['''\
+ <p>Nalogo lahko rešiš bistveno hitreje, če poznaš metodo <code>count</code></p>
+ <pre>
+ xs = [42, 5, 4, -7, 2, 12, -3, -4, 11, 42, 2]
+ print('Število 42 se v seznamu pojavi', xs.count(42), 'krat.')
+</pre>''']
+
+}