Piotr Łętowski

changes.txt

@@ -0,0 +1,12 @@
+1. zmiana nazw iteratorów
+2. usunięcie nadmiarowej funkcji
+3. usunięcie nadmiarowych printów, oraz nadmiarowych słów z printów(np something went horribly wrong)
+4. przeniesienie import na górę kodu i zmiana z importowania wszyskiego za pomocą "*" na import konktetnych funkcji
+5. poprawa ogólnej estetyki kodu (wcięcia w odpowiednich miejscach, odpowiednie ilości odstępu między blokami kodu)
+6. zmiana nazwy funkcji na bardziej zrozumiałą
+7. wstawienie pętli for zamiast kilku takich samych operacji pod rząd
+8. ususnięcie zmiennych globalnych
+9. ujednolicenie nazw zmiennych
+10. poprawienie komentarza, aby nie był jedną długą linijką, oraz skrócenie go by był bardziej zrozumiały
+11. poszerzenie try-except
+12. zmiana -1 na ":" przy zapisie pliku

file157.py

@@ -1,30 +1,37 @@
+from file87 import piContainer, calculate
+
+
 def enumerate(pi):
-    for hello in pi.a:
-        print(f'Next element: {hello}')
-from file87 import *
+    for i in pi.a:
+        print(f'Next element: {i}')
+
+
 try:
-    my_pi = pi_container()
-    pi_gen = foo(5)
-    my_pi.mth(pi_gen.__next__())
-    my_pi.mth(pi_gen.__next__())
-    my_pi.mth(pi_gen.__next__())
-    my_pi.mth(pi_gen.__next__())
-    my_pi_2 = pi_container()
-    my_pi.mth(pi_gen.__next__())
-    my_pi.mth(pi_gen.__next__())
-    my_pi.mth(pi_gen.__next__())
+    my_pi = piContainer()
+    pi_gen_1 = calculate(5)
+
+    for i in range(6):
+        my_pi.mth(pi_gen_1.__next__())
+
+    my_pi_2 = piContainer()
+
+    pi_gen_2 = calculate(194)
+    for i in range(23):
+        my_pi_2.mth(next(pi_gen_2))
+
+    my_pi_3 = piContainer()
+    pi_gen_3 = calculate(6)
+    my_pi_3.mth([i for i in list(pi_gen_3)])
+
+    print('my first pi')
+    enumerate(my_pi)
+
+    print('my second pi')
+    enumerate(my_pi_2)
+
+    new_file = open('pi_approximation.txt', 'w')
+    new_file.write(f'my best pi: {my_pi_3.a[:]}')
+    new_file.close()
+
 except:
-    print('something went horribly wrong :(')
-pIgEn3 = foo(194)
-for the_variable_that_contains_next_approximations_of_pi_from_generator in range(23):
-    my_pi_2.mth(next(pIgEn3))
-my_pi_3 = pi_container()
-pi_gen = foo(6)
-my_pi_3.mth([i for i in list(pi_gen)])
-print('my first pi')
-enumerate(my_pi)
-print('my second pi')
-enumerate(my_pi_2)
-new_file = open('some-file.txt', 'w')
-new_file.write(f'my best pi: {my_pi_3.a[-1]}')
-new_file.close()
+    print('something went wrong')

file87.py

@@ -1,8 +1,5 @@
-b = 0
-c = 0
 
-
-class pi_container:
+class piContainer:
     def __init__(self, a=list()):
         self.a = a
 
@@ -11,25 +8,17 @@ def mth(self, x):
             self.a += x
         else:
             self.a.append(x)
-
-
-def foo(x):
-    global b
-    global c
+# this is a important operation in calculating pi that is
+# provided in a separate file in this repository,
+# please analyse this file before using!
+def calculate(x):
     b = 0
-    c=1
-    for hello in range(x):
-        if hello % 2 == 0:
-            b += 4 / c#this is a very important operation in calculateing pi according to documentation that is provided in a seperate file in this repository, please analyse this file before using!
+    c = 1
+    for i in range(x):
+        if i % 2 == 0:
+            b += 4 / c
         else:
             b -= 4 / c
         c += 2
         yield b
     yield 'finished'
-
-def enumerate(pi: pi_container):
-    for hello in pi.a:
-        print(hello)
-
-
-print('All functions are defined')